LayoutTests/fast/hidpi/image-srcset-invalid-descriptor.html - Issue 649183007: Add console errors and usecounter when srcset candidates are dropped

Unified Diff: LayoutTests/fast/hidpi/image-srcset-invalid-descriptor.html

Issue 649183007: Add console errors and usecounter when srcset candidates are dropped (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: improved log messages Created 6 years, 2 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

« no previous file with comments | « LayoutTests/fast/hidpi/image-srcset-ignore-double-descriptor-expected.txt ('k') | LayoutTests/fast/hidpi/image-srcset-invalid-descriptor-expected.txt » ('j') | Source/core/html/parser/HTMLSrcsetParser.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: LayoutTests/fast/hidpi/image-srcset-invalid-descriptor.html

diff --git a/LayoutTests/fast/hidpi/image-srcset-invalid-descriptor.html b/LayoutTests/fast/hidpi/image-srcset-invalid-descriptor.html

index 514f9535272cadf4343226bc67f02f6df32f69b1..960c171a0e6d7197faa51f20efdcb84720b5d735 100644

--- a/LayoutTests/fast/hidpi/image-srcset-invalid-descriptor.html

+++ b/LayoutTests/fast/hidpi/image-srcset-invalid-descriptor.html

@@ -9,6 +9,14 @@

addEventListener("load", function() {

shouldBeTrue('document.getElementById("foo").clientWidth==200');

shouldBeTrue('document.getElementById("foo2").clientWidth==200');

+ shouldBeTrue('document.getElementById("foo3").clientWidth==200');

+ shouldBeTrue('document.getElementById("foo4").clientWidth==200');

+ shouldBeTrue('document.getElementById("foo5").clientWidth==200');

+ shouldBeTrue('document.getElementById("foo6").clientWidth==200');

+ shouldBeTrue('document.getElementById("foo7").clientWidth==200');

+ shouldBeTrue('document.getElementById("foo8").clientWidth==200');

+ shouldBeTrue('document.getElementById("foo9").clientWidth==200');

+ shouldBeTrue('document.getElementById("foo10").clientWidth==200');

}, false);

</script>

</head>

@@ -19,5 +27,13 @@

<div>The 2x resource has an intrinsic width of 200px, since it's a 400px wide image at 2x DPR.</div>

+ <img id="foo3" src="" srcset="resources/blue-100-px-square.png 100w 100w, resources/green-400-px-square.png 2x">

+ <img id="foo4" src="" srcset="resources/blue-100-px-square.png 100h 100h, resources/green-400-px-square.png 2x">

+ <img id="foo5" src="" srcset="resources/blue-100-px-square.png 100x 100h, resources/green-400-px-square.png 2x">

+ <img id="foo6" src="" srcset="resources/blue-100-px-square.png 100x 100w, resources/green-400-px-square.png 2x">

+ <img id="foo7" src="" srcset="resources/blue-100-px-square.png -100w, resources/green-400-px-square.png 2x">

+ <img id="foo8" src="" srcset="resources/blue-100-px-square.png -100h, resources/green-400-px-square.png 2x">

+ <img id="foo9" src="" srcset="resources/blue-100-px-square.png -1x, resources/green-400-px-square.png 2x">

+ <img id="foo10" src="" srcset="resources/blue-100-px-square.png 100h, resources/green-400-px-square.png 2x">

</body>

</html>