Reduce viewport arguments parsing noise in the DevTools console

Source/web/tests/ViewportTest.cpp

 /*
  * Copyright (C) 2013 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 2964 matching lines @@
 
     registerMockedHttpURLLoad("viewport/viewport-warnings-2.html");
 
     FrameTestHelpers::WebViewHelper webViewHelper;
     webViewHelper.initializeAndLoad(m_baseURL + "viewport/viewport-warnings-2.html", true, 0, &webViewClient, setViewportSettings);
 
     Page* page = webViewHelper.webViewImpl()->page();
     PageScaleConstraints constraints = runViewportTest(page, 320, 352);
 
     EXPECT_EQ(1U, webViewClient.messages.size());
-    EXPECT_EQ(WebConsoleMessage::LevelError, webViewClient.messages[0].level);
+    EXPECT_EQ(WebConsoleMessage::LevelWarning, webViewClient.messages[0].level);
     EXPECT_STREQ("The key \"wwidth\" is not recognized and ignored.", webViewClient.messages[0].text.utf8().c_str());
 
     EXPECT_EQ(980, constraints.layoutSize.width());
     EXPECT_EQ(1078, constraints.layoutSize.height());
     EXPECT_NEAR(0.33f, constraints.initialScale, 0.01f);
     EXPECT_NEAR(0.33f, constraints.minimumScale, 0.01f);
     EXPECT_NEAR(5.0f, constraints.maximumScale, 0.01f);
     EXPECT_TRUE(page->viewportDescription().userZoom);
 }
 
 TEST_F(ViewportTest, viewportWarnings3)
 {
     ConsoleMessageWebViewClient webViewClient;
 
     registerMockedHttpURLLoad("viewport/viewport-warnings-3.html");
 
     FrameTestHelpers::WebViewHelper webViewHelper;
     webViewHelper.initializeAndLoad(m_baseURL + "viewport/viewport-warnings-3.html", true, 0, &webViewClient, setViewportSettings);
 
     Page* page = webViewHelper.webViewImpl()->page();
     PageScaleConstraints constraints = runViewportTest(page, 320, 352);
 
     EXPECT_EQ(1U, webViewClient.messages.size());
-    EXPECT_EQ(WebConsoleMessage::LevelError, webViewClient.messages[0].level);
+    EXPECT_EQ(WebConsoleMessage::LevelWarning, webViewClient.messages[0].level);
     EXPECT_STREQ("The value \"unrecognized-width\" for key \"width\" is invalid, and has been ignored.",
         webViewClient.messages[0].text.utf8().c_str());
 
     EXPECT_NEAR(64.0f, constraints.layoutSize.width(), 0.01);
     EXPECT_NEAR(70.4f, constraints.layoutSize.height(), 0.01);
     EXPECT_NEAR(5.0f, constraints.initialScale, 0.01f);
     EXPECT_NEAR(5.0f, constraints.minimumScale, 0.01f);
     EXPECT_NEAR(5.0f, constraints.maximumScale, 0.01f);
     EXPECT_TRUE(page->viewportDescription().userZoom);
 }
@@ skipping 30 matching lines @@
     registerMockedHttpURLLoad("viewport/viewport-warnings-5.html");
 
     FrameTestHelpers::WebViewHelper webViewHelper;
     webViewHelper.initializeAndLoad(m_baseURL + "viewport/viewport-warnings-5.html", true, 0, &webViewClient, setViewportSettings);
 
     Page* page = webViewHelper.webViewImpl()->page();
     PageScaleConstraints constraints = runViewportTest(page, 320, 352);
 
     EXPECT_EQ(5U, webViewClient.messages.size());
 
-    EXPECT_EQ(WebConsoleMessage::LevelError, webViewClient.messages[0].level);
+    EXPECT_EQ(WebConsoleMessage::LevelWarning, webViewClient.messages[0].level);
     EXPECT_STREQ("The value \"device-width;\" for key \"width\" is invalid, and has been ignored.",
         webViewClient.messages[0].text.utf8().c_str());
 
     EXPECT_EQ(WebConsoleMessage::LevelWarning, webViewClient.messages[1].level);
     EXPECT_STREQ("The value \"1.0;\" for key \"initial-scale\" was truncated to its numeric prefix.",
         webViewClient.messages[1].text.utf8().c_str());
 
     EXPECT_EQ(WebConsoleMessage::LevelWarning, webViewClient.messages[2].level);
     EXPECT_STREQ("The value \"1.0;\" for key \"maximum-scale\" was truncated to its numeric prefix.",
         webViewClient.messages[2].text.utf8().c_str());
@@ skipping 20 matching lines @@
 
     registerMockedHttpURLLoad("viewport/viewport-warnings-6.html");
 
     FrameTestHelpers::WebViewHelper webViewHelper;
     webViewHelper.initializeAndLoad(m_baseURL + "viewport/viewport-warnings-6.html", true, 0, &webViewClient, setViewportSettings);
 
     Page* page = webViewHelper.webViewImpl()->page();
     PageScaleConstraints constraints = runViewportTest(page, 320, 352);
 
     EXPECT_EQ(1U, webViewClient.messages.size());
-    EXPECT_EQ(WebConsoleMessage::LevelError, webViewClient.messages[0].level);
+    EXPECT_EQ(WebConsoleMessage::LevelWarning, webViewClient.messages[0].level);
     EXPECT_STREQ("The value \"\" for key \"width\" is invalid, and has been ignored.",
         webViewClient.messages[0].text.utf8().c_str());
 
     EXPECT_NEAR(64.0f, constraints.layoutSize.width(), 0.01);
     EXPECT_NEAR(70.4f, constraints.layoutSize.height(), 0.01);
     EXPECT_NEAR(5.0f, constraints.initialScale, 0.01f);
     EXPECT_NEAR(5.0f, constraints.minimumScale, 0.01f);
     EXPECT_NEAR(5.0f, constraints.maximumScale, 0.01f);
     EXPECT_TRUE(page->viewportDescription().userZoom);
 }
 
+TEST_F(ViewportTest, viewportWarnings7)
+{
+    ConsoleMessageWebViewClient webViewClient;
+
+    registerMockedHttpURLLoad("viewport/viewport-warnings-7.html");
+
+    FrameTestHelpers::WebViewHelper webViewHelper;
+    webViewHelper.initializeAndLoad(m_baseURL + "viewport/viewport-warnings-7.html", true, 0, &webViewClient, setViewportSettings);
+
+    Page* page = webViewHelper.webViewImpl()->page();
+    runViewportTest(page, 320, 352);
+
+    EXPECT_EQ(0U, webViewClient.messages.size());
+}
+
 } // namespace