Collect @viewport before constructing RuleSets.

third_party/WebKit/Source/web/tests/WebFrameTest.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 2678 matching lines @@
   webViewHelper.webView()->setDefaultPageScaleLimits(0.25f, 5);
   webViewHelper.webView()->settings()->setWideViewportQuirkEnabled(true);
   webViewHelper.webView()->settings()->setUseWideViewport(false);
   webViewHelper.resize(WebSize(viewportWidth, viewportHeight));
 
   EXPECT_NEAR(1.0f, webViewHelper.webView()->pageScaleFactor(), 0.01f);
   EXPECT_NEAR(1.0f, webViewHelper.webView()->minimumPageScaleFactor(), 0.01f);
   EXPECT_NEAR(5.0f, webViewHelper.webView()->maximumPageScaleFactor(), 0.01f);
 }
 
-// TODO(rune@opera.com): Does not pass until we collect author @viewport rules
-// before constructing the RuleSets. https://crbug.com/332763
-TEST_P(ParameterizedWebFrameTest,
-       DISABLED_AtViewportInsideAtMediaInitialViewport) {
+TEST_P(ParameterizedWebFrameTest, AtViewportInsideAtMediaInitialViewport) {
   registerMockedHttpURLLoad("viewport-inside-media.html");
 
   FixedLayoutTestWebViewClient client;
   FrameTestHelpers::WebViewHelper webViewHelper;
   webViewHelper.initializeAndLoad(m_baseURL + "viewport-inside-media.html",
                                   true, nullptr, &client, nullptr,
                                   enableViewportSettings);
   webViewHelper.resize(WebSize(640, 480));
 
   EXPECT_EQ(2000, webViewHelper.webView()
                       ->mainFrameImpl()
                       ->frameView()
                       ->layoutSize()
                       .width());
 }
 
+TEST_P(ParameterizedWebFrameTest, AtViewportAffectingAtMediaRecalcCount) {
+  registerMockedHttpURLLoad("viewport-and-media.html");
+
+  FixedLayoutTestWebViewClient client;
+  FrameTestHelpers::WebViewHelper webViewHelper;
+  webViewHelper.initialize(true, nullptr, &client, nullptr,
+                           enableViewportSettings);
+  webViewHelper.resize(WebSize(640, 480));
+  FrameTestHelpers::loadFrame(webViewHelper.webView()->mainFrame(),
+                              m_baseURL + "viewport-and-media.html");
+
+  Document* document =
+      webViewHelper.webView()->mainFrameImpl()->frame()->document();
+  EXPECT_EQ(2000, webViewHelper.webView()
+                      ->mainFrameImpl()
+                      ->frameView()
+                      ->layoutSize()
+                      .width());
+
+  // The styleForElementCount() should match the number of elements for a single
+  // pass of computed styles construction for the document.
+  EXPECT_EQ(10u, document->styleEngine().styleForElementCount());
+  EXPECT_EQ(Color(0, 128, 0),
+            document->body()->computedStyle()->visitedDependentColor(
+                CSSPropertyColor));
+}
+
+TEST_P(ParameterizedWebFrameTest, AtViewportWithViewportLengths) {
+  registerMockedHttpURLLoad("viewport-lengths.html");
+
+  FixedLayoutTestWebViewClient client;
+  FrameTestHelpers::WebViewHelper webViewHelper;
+  webViewHelper.initialize(true, nullptr, &client, nullptr,
+                           enableViewportSettings);
+  webViewHelper.resize(WebSize(800, 600));
+  FrameTestHelpers::loadFrame(webViewHelper.webView()->mainFrame(),
+                              m_baseURL + "viewport-lengths.html");
+
+  FrameView* view = webViewHelper.webView()->mainFrameImpl()->frameView();
+  EXPECT_EQ(400, view->layoutSize().width());
+  EXPECT_EQ(300, view->layoutSize().height());
+
+  webViewHelper.resize(WebSize(1000, 400));
+
+  EXPECT_EQ(500, view->layoutSize().width());
+  EXPECT_EQ(200, view->layoutSize().height());
+}
+
 class WebFrameResizeTest : public ParameterizedWebFrameTest {
  protected:
   static FloatSize computeRelativeOffset(const IntPoint& absoluteOffset,
                                          const LayoutRect& rect) {
     FloatSize relativeOffset =
         FloatPoint(absoluteOffset) - FloatPoint(rect.location());
     relativeOffset.scale(1.f / rect.width(), 1.f / rect.height());
     return relativeOffset;
   }
 
@@ skipping 7478 matching lines @@
   webViewHelper.webView()->handleInputEvent(endEvent);
   webViewHelper.webView()->handleInputEvent(updateEvent);
 
   // Try a full Begin/Update/End cycle.
   webViewHelper.webView()->handleInputEvent(beginEvent);
   webViewHelper.webView()->handleInputEvent(updateEvent);
   webViewHelper.webView()->handleInputEvent(endEvent);
 }
 
 }  // namespace blink