Have srcset respond to viewport changes

Source/core/css/parser/SizesAttributeParserTest.cpp

Index: Source/core/css/parser/SizesAttributeParserTest.cpp
diff --git a/Source/core/css/parser/SizesAttributeParserTest.cpp b/Source/core/css/parser/SizesAttributeParserTest.cpp
index 09352e62a3bd391ec87abed56d5575c819f7a05b..92a57d8b64271c201b47ae222ab377c3b916bb3a 100644
--- a/Source/core/css/parser/SizesAttributeParserTest.cpp
+++ b/Source/core/css/parser/SizesAttributeParserTest.cpp
@@ -14,49 +14,56 @@ namespace blink {
 
 typedef struct {
     const char* input;
-    const int output;
+    const unsigned effectiveSize;
+    const bool viewportDependant;
 } TestCase;
 
 TEST(SizesAttributeParserTest, Basic)
 {
     TestCase testCases[] = {
-        {"screen", 500},
-        {"(min-width:500px)", 500},
-        {"(min-width:500px) 200px", 200},
-        {"(min-width:500px) 50vw", 250},
-        {"(min-width:500px) 200px, 400px", 200},
-        {"400px, (min-width:500px) 200px", 400},
-        {"(min-width:5000px) 200px, 400px", 400},
-        {"(blalbadfsdf) 200px, 400px", 400},
-        {"0", 0},
-        {"-0", 0},
-        {"1", 500},
-        {"300px, 400px", 300},
-        {"(min-width:5000px) 200px, (min-width:500px) 400px", 400},
-        {"", 500},
-        {"  ", 500},
-        {" /**/ ", 500},
-        {" /**/ 300px", 300},
-        {"300px /**/ ", 300},
-        {" /**/ (min-width:500px) /**/ 300px", 300},
-        {"-100px, 200px", 200},
-        {"-50vw, 20vw", 100},
-        {"50asdf, 200px", 200},
-        {"asdf, 200px", 200},
-        {"(max-width: 3000px) 200w, 400w", 500},
-        {",, , /**/ ,200px", 200},
-        {"50vw", 250},
-        {"5em", 80},
-        {"5rem", 80},
-        {"calc(40vw*2)", 400},
-        {"(min-width:5000px) calc(5000px/10), (min-width:500px) calc(1200px/3)", 400},
-        {"(min-width:500px) calc(1200/3)", 500},
-        {"(min-width:500px) calc(1200px/(0px*14))", 500},
-        {"(max-width: 3000px) 200px, 400px", 200},
-        {"(max-width: 3000px) 20em, 40em", 320},
-        {"(max-width: 3000px) 0, 40em", 0},
-        {"(max-width: 3000px) 50vw, 40em", 250},
-        {0, 0} // Do not remove the terminator line.
+        {"screen", 500, true},
+        {"(min-width:500px)", 500, true},
+        {"(min-width:500px) 200px", 200, false},
+        {"(min-width:500px) 50vw", 250, true},
+        {"(min-width:500px) 200px, 400px", 200, false},
+        {"400px, (min-width:500px) 200px", 400, false},
+        {"40vw, (min-width:500px) 201px", 200, true},
+        {"(min-width:500px) 201px, 40vw", 201, false},
+        {"(min-width:5000px) 40vw, 201px", 201, false},
+        {"(min-width:500px) calc(201px), calc(40vw)", 201, false},
+        {"(min-width:5000px) calc(40vw), calc(201px)", 201, false},
+        {"(min-width:5000px) 200px, 400px", 400, false},
+        {"(blalbadfsdf) 200px, 400px", 400, false},
+        {"0", 0, false},
+        {"-0", 0, false},
+        {"1", 500, true},
+        {"300px, 400px", 300, false},
+        {"(min-width:5000px) 200px, (min-width:500px) 400px", 400, false},
+        {"", 500, true},
+        {"  ", 500, true},
+        {" /**/ ", 500, true},
+        {" /**/ 300px", 300, false},
+        {"300px /**/ ", 300, false},
+        {" /**/ (min-width:500px) /**/ 300px", 300, false},
+        {"-100px, 200px", 200, false},
+        {"-50vw, 20vw", 100, true},
+        {"50asdf, 200px", 200, false},
+        {"asdf, 200px", 200, false},
+        {"(max-width: 3000px) 200w, 400w", 500, true},
+        {",, , /**/ ,200px", 200, false},
+        {"50vw", 250, true},
+        {"5em", 80, false},
+        {"5rem", 80, false},
+        {"calc(40vw*2)", 400, true},
+        {"(min-width:5000px) calc(5000px/10), (min-width:500px) calc(1200px/3)", 400, false},
+        {"(min-width:500px) calc(1200/3)", 500, true},
+        {"(min-width:500px) calc(1200px/(0px*14))", 500, true},
+        {"(max-width: 3000px) 200px, 400px", 200, false},
+        {"(max-width: 3000px) 20em, 40em", 320, false},
+        {"(max-width: 3000px) 0, 40em", 0, false},
+        {"(max-width: 3000px) 50vw, 40em", 250, true},
+        {"(max-width: 3000px) 50px, 40vw", 50, false},
+        {0, 0, false} // Do not remove the terminator line.
     };
 
     MediaValuesCached::MediaValuesCachedData data;
@@ -75,8 +82,9 @@ TEST(SizesAttributeParserTest, Basic)
     RefPtr<MediaValues> mediaValues = MediaValuesCached::create(data);
 
     for (unsigned i = 0; testCases[i].input; ++i) {
-        int effectiveSize = SizesAttributeParser::findEffectiveSize(testCases[i].input, mediaValues);
-        ASSERT_EQ(testCases[i].output, effectiveSize);
+        SizesAttributeParser parser(mediaValues, testCases[i].input);
+        ASSERT_EQ(testCases[i].effectiveSize, parser.length());
+        ASSERT_EQ(testCases[i].viewportDependant, parser.viewportDependant());
     }
 }