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

Unified Diff: third_party/WebKit/LayoutTests/svg/as-image/data-font-in-css-invalid-data-url-crash.html

Issue 1699323002: [Closed][SVG 4/4] Finish font subresource loading of SVG synchronously in SVGImage::dataChanged() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: auto-Rebase Created 4 years, 9 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/svg/as-image/data-font-in-css-invalid-data-url-crash.html
diff --git a/third_party/WebKit/LayoutTests/svg/as-image/data-font-in-css-invalid-data-url-crash.html b/third_party/WebKit/LayoutTests/svg/as-image/data-font-in-css-invalid-data-url-crash.html
new file mode 100644
index 0000000000000000000000000000000000000000..a704daed512d5756af09425b02ad3db76487953f
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/svg/as-image/data-font-in-css-invalid-data-url-crash.html
@@ -0,0 +1,26 @@
+<html>
+<head>
+<script src="../../resources/js-test.js"></script>
+<script>
+description("Test that loading of SVG that contains a font data URL completes before <img> and <body> onload handler is called, and thus doesn't cause assertion failure.");
+jsTestIsAsync = true;
+
+var count = 0;
+function doTest() {
+ var svg = document.getElementById('img');
+ var canvas = document.createElement("canvas");
+ var ctx = canvas.getContext("2d");
+ ctx.drawImage(svg, 0, 0);
+
+ // Test finishes when both <body> and <img>'s onload() are called.
+ ++count;
+ if (count == 2) {
+ finishJSTest();
+ }
+}
+</script>
+</head>
+<body onload="debug('onload of body'); doTest()">
+<img src="resources/data-font-in-css-invalid-data-url.svg" id="img" onload="debug('onload of img'); doTest()">
+</body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698