Changeset 55658

Timestamp:

04/19/2023 01:24:40 AM (3 years ago)

Author:

westonruter

Message:

Script Loader: Optimize performance of _wp_normalize_relative_css_links() by more than 2x.

• Replace preg_match_all() and its secondary str_replace() call with preg_replace_callback().
• Fix case where paths beginning with http and https (but not http: and https:) were erroneously not counted as relative.
• Improve code style and readability by consolidating conditions and returning once.
• Use str_starts_with() consistently instead of strpos().

Follow-up to [52036], [52695], and [52754].

Fixes #58069.
See #54243.

Location:

trunk

Files:

• src/wp-includes/script-loader.php (modified) (1 diff)
• tests/phpunit/tests/dependencies/styles.php (modified) (1 diff)

trunk/src/wp-includes/script-loader.php

@@ -2940 +2940 @@
  */
 function _wp_normalize_relative_css_links( $css, $stylesheet_url ) {
-        $has_src_results = preg_match_all( '#url\s*\(\s*[\'"]?\s*([^\'"\)]+)#', $css, $src_results );
-        if ( $has_src_results ) {
-                // Loop through the URLs to find relative ones.
-                foreach ( $src_results[1] as $src_index => $src_result ) {
-                        // Skip if this is an absolute URL.
-                        if ( 0 === strpos( $src_result, 'http' ) || 0 === strpos( $src_result, '//' ) ) {
-                                continue;
+        return preg_replace_callback(
+                '#(url\s*\(\s*[\'"]?\s*)([^\'"\)]+)#',
+                static function ( $matches ) use ( $stylesheet_url ) {
+                        list( , $prefix, $url ) = $matches;
+
+                        if ( ! (
+                                str_starts_with( $url, 'http:' )
+                                ||
+                                str_starts_with( $url, 'https:' )
+                                ||
+                                str_starts_with( $url, '//' )
+                                ||
+                                str_starts_with( $url, '#' )
+                                ||
+                                str_starts_with( $url, 'data:' )
+                        ) ) {
+                                // Build the absolute URL.
+                                $absolute_url = dirname( $stylesheet_url ) . '/' . $url;
+                                $absolute_url = str_replace( '/./', '/', $absolute_url );
+
+                                // Convert to URL related to the site root.
+                                $url = wp_make_link_relative( $absolute_url );
                         }
 
-                        // Skip if the URL is an HTML ID.
-                        if ( str_starts_with( $src_result, '#' ) ) {
-                                continue;
-                        }
-
-                        // Skip if the URL is a data URI.
-                        if ( str_starts_with( $src_result, 'data:' ) ) {
-                                continue;
-                        }
-
-                        // Build the absolute URL.
-                        $absolute_url = dirname( $stylesheet_url ) . '/' . $src_result;
-                        $absolute_url = str_replace( '/./', '/', $absolute_url );
-                        // Convert to URL related to the site root.
-                        $relative_url = wp_make_link_relative( $absolute_url );
-
-                        // Replace the URL in the CSS.
-                        $css = str_replace(
-                                $src_results[0][ $src_index ],
-                                str_replace( $src_result, $relative_url, $src_results[0][ $src_index ] ),
-                                $css
-                        );
-                }
-        }
-
-        return $css;
+                        return $prefix . $url;
+                },
+                $css
+        );
 }
 

trunk/tests/phpunit/tests/dependencies/styles.php

@@ -241 +241 @@
                                 'expected' => 'img {mask-image: url(\'data:image/svg+xml;utf8,<svg></svg>\');}',
                         ),
+                        'URLs with path beginning with http'           => array(
+                                'css'      => 'p {background:url( "http-is-awesome.png" );}',
+                                'expected' => 'p {background:url( "/wp-content/themes/test/http-is-awesome.png" );}',
+                        ),
+                        'URLs with path beginning with https'          => array(
+                                'css'      => 'p {background:url( "https-is-more-awesome.png" );}',
+                                'expected' => 'p {background:url( "/wp-content/themes/test/https-is-more-awesome.png" );}',
+                        ),
                 );
         }