Title: [88920] trunk
- Revision
- 88920
- Author
- [email protected]
- Date
- 2011-06-15 05:09:18 -0700 (Wed, 15 Jun 2011)
Log Message
2011-06-14 Alexander Pavlov <[email protected]>
Reviewed by Yury Semikhatsky.
Web Inspector: "data" URLs not detected in CSS url(...) values properly
https://bugs.webkit.org/show_bug.cgi?id=62643
WebCore:
* inspector/front-end/inspector.js:
(WebInspector.completeURL):
LayoutTests:
* inspector/styles/styles-url-linkify-expected.txt:
* inspector/styles/styles-url-linkify.html:
Modified Paths
Diff
Modified: trunk/LayoutTests/ChangeLog (88919 => 88920)
--- trunk/LayoutTests/ChangeLog 2011-06-15 11:46:24 UTC (rev 88919)
+++ trunk/LayoutTests/ChangeLog 2011-06-15 12:09:18 UTC (rev 88920)
@@ -1,3 +1,13 @@
+2011-06-14 Alexander Pavlov <[email protected]>
+
+ Reviewed by Yury Semikhatsky.
+
+ Web Inspector: "data" URLs not detected in CSS url(...) values properly
+ https://bugs.webkit.org/show_bug.cgi?id=62643
+
+ * inspector/styles/styles-url-linkify-expected.txt:
+ * inspector/styles/styles-url-linkify.html:
+
2011-06-15 Mario Sanchez Prada <[email protected]>
Unreviewed, skipped accessibility test failing in GTK.
Modified: trunk/LayoutTests/inspector/styles/styles-url-linkify-expected.txt (88919 => 88920)
--- trunk/LayoutTests/inspector/styles/styles-url-linkify-expected.txt 2011-06-15 11:46:24 UTC (rev 88919)
+++ trunk/LayoutTests/inspector/styles/styles-url-linkify-expected.txt 2011-06-15 12:09:18 UTC (rev 88920)
@@ -1,7 +1,7 @@
-Tests that URLs are linked to and completed correctly. Bugs 51663, 53171
+Tests that URLs are linked to and completed correctly. Bugs 51663, 53171, 62643
-Partial URLs completed:
+URLs completed:
http://example.com/
http://example.com/moo
https://secure.com/moo
@@ -12,6 +12,7 @@
http://example.com/moo
http://example.com/foo?a=b
http://example.com/foo?a=b
+data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAEIAAABCAgMAAACeOuh7AAAABGdBTUEAAK/INwWK6QAAAAlQTFRF////AAAA////fu+PTwAAAAF0Uk5TAEDm2GYAAACHSURBVDjLxdLbDYAgDAVQGELn0R3oEHYf2KGdUqtE46OFRCP3oyTng1xCnWsaD5JRRtCkQ2YmkBkHRXqWJBn0j0TICbrsWVoWhRShCdcGyZCtHxMaUnVPRZ9KSbmBJdsX2vJVnwqRD0Rb4rpzgIbE/AI5NTnWAMvy5l0dXrfuLh5OCe5BmmYGXhTUxlQ5xJ8AAAAASUVORK5CYII=
Link for a URI from CSS document:
inspector/styles/resources/fromcss.png
Link for a URI from iframe inline stylesheet:
Modified: trunk/LayoutTests/inspector/styles/styles-url-linkify.html (88919 => 88920)
--- trunk/LayoutTests/inspector/styles/styles-url-linkify.html 2011-06-15 11:46:24 UTC (rev 88919)
+++ trunk/LayoutTests/inspector/styles/styles-url-linkify.html 2011-06-15 12:09:18 UTC (rev 88920)
@@ -13,7 +13,7 @@
InspectorTest.addResult(WebInspector.completeURL(baseURL, href));
}
- InspectorTest.addResult("Partial URLs completed:");
+ InspectorTest.addResult("URLs completed:");
completeURL("http://example.com", "/");
completeURL("http://example.com", "moo");
completeURL("http://example.com/", "https://secure.com/moo");
@@ -25,6 +25,9 @@
completeURL("http://example.com/foo", "?a=b");
completeURL("http://example.com/foo?c=d", "?a=b");
+ const dataURL = "data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAEIAAABCAgMAAACeOuh7AAAABGdBTUEAAK/INwWK6QAAAAlQTFRF////AAAA////fu+PTwAAAAF0Uk5TAEDm2GYAAACHSURBVDjLxdLbDYAgDAVQGELn0R3oEHYf2KGdUqtE46OFRCP3oyTng1xCnWsaD5JRRtCkQ2YmkBkHRXqWJBn0j0TICbrsWVoWhRShCdcGyZCtHxMaUnVPRZ9KSbmBJdsX2vJVnwqRD0Rb4rpzgIbE/AI5NTnWAMvy5l0dXrfuLh5OCe5BmmYGXhTUxlQ5xJ8AAAAASUVORK5CYII=";
+ completeURL("https://example.com/foo", dataURL);
+
function dumpHref()
{
var href;
@@ -70,7 +73,7 @@
</head>
<body _onload_="runAfterIframeIsLoaded()">
<p>
-Tests that URLs are linked to and completed correctly. Bugs <a href="" <a href=""
+Tests that URLs are linked to and completed correctly. Bugs <a href="" <a href="" <a href=""
</p>
<div id="local"></div>
<iframe src=""
Modified: trunk/Source/WebCore/ChangeLog (88919 => 88920)
--- trunk/Source/WebCore/ChangeLog 2011-06-15 11:46:24 UTC (rev 88919)
+++ trunk/Source/WebCore/ChangeLog 2011-06-15 12:09:18 UTC (rev 88920)
@@ -1,3 +1,13 @@
+2011-06-14 Alexander Pavlov <[email protected]>
+
+ Reviewed by Yury Semikhatsky.
+
+ Web Inspector: "data" URLs not detected in CSS url(...) values properly
+ https://bugs.webkit.org/show_bug.cgi?id=62643
+
+ * inspector/front-end/inspector.js:
+ (WebInspector.completeURL):
+
2011-06-15 Pavel Feldman <[email protected]>
Reviewed by Yury Semikhatsky.
Modified: trunk/Source/WebCore/inspector/front-end/inspector.js (88919 => 88920)
--- trunk/Source/WebCore/inspector/front-end/inspector.js 2011-06-15 11:46:24 UTC (rev 88919)
+++ trunk/Source/WebCore/inspector/front-end/inspector.js 2011-06-15 12:09:18 UTC (rev 88920)
@@ -1318,7 +1318,7 @@
if (href) {
// Return absolute URLs as-is.
var parsedHref = href.asParsedURL();
- if (parsedHref && parsedHref.scheme)
+ if ((parsedHref && parsedHref.scheme) || href.indexOf("data:") === 0)
return href;
}
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes