Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(68)

Unified Diff: third_party/WebKit/LayoutTests/fast/js/resources/image-preload-helper.js

Issue 1363233003: Make sure <url>s are being serialized according to spec (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix interpolation tests Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/fast/js/resources/image-preload-helper.js
diff --git a/third_party/WebKit/LayoutTests/fast/js/resources/image-preload-helper.js b/third_party/WebKit/LayoutTests/fast/js/resources/image-preload-helper.js
index 27ea17d143abaad3f1cd1f0b9144dee27b9b2804..b6fafdcc7bccce77684cb79a1204d9221e85bddd 100644
--- a/third_party/WebKit/LayoutTests/fast/js/resources/image-preload-helper.js
+++ b/third_party/WebKit/LayoutTests/fast/js/resources/image-preload-helper.js
@@ -13,7 +13,7 @@ function preloadImagesFromStyle(rootNode, imageCount, onComplete, failPattern) {
var imagesToLoad = [];
var seenUrls = {};
for (var i = 0; i < nodes.length; i++) {
- var urls = nodes[i].style.cssText.split(/url\w*\(([^)]*)\)/);
+ var urls = nodes[i].style.cssText.split(/url\w*\("([^)]*)"\)/);
for (var j = 1; j < urls.length; j += 2) {
// Attempt to convert URL to a relative path in order to have deterministic error messages.
var url = urls[j];

Powered by Google App Engine
This is Rietveld 408576698