Omnibox: increase omnibox minimum width

Omnibox width was 10 characters (plus some extra for page actions and location bar bubbles), and is now 20 characters.

Bug: 849784
Change-Id: Iaacab5244b5f1c21156151f49e0405e844f09451
Reviewed-on: https://chromium-review.googlesource.com/1135147Reviewed-by: Jonathon Kereliuk <kereliuk@chromium.org>
Reviewed-by: Andrey Kosyakov <caseq@chromium.org>
Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Reviewed-by: Tommy Li <tommycli@chromium.org>
Reviewed-by: Justin Donnelly <jdonnelly@chromium.org>
Commit-Queue: manuk hovanesian <manukh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578623}

chrome/browser/devtools/devtools_sanity_interactive_browsertest.cc

@@ -122,10 +122,10 @@ class DevToolsManagerDelegateTest : public InProcessBrowserTest {
 };
 
 IN_PROC_BROWSER_TEST_F(DevToolsManagerDelegateTest, NormalWindowChangeBounds) {
-  browser()->window()->SetBounds(gfx::Rect(100, 100, 500, 600));
-  CheckWindowBounds(gfx::Rect(100, 100, 500, 600));
+  browser()->window()->SetBounds(gfx::Rect(100, 100, 600, 600));
+  CheckWindowBounds(gfx::Rect(100, 100, 600, 600));
   UpdateBounds();
-  CheckWindowBounds(gfx::Rect(200, 100, 500, 400));
+  CheckWindowBounds(gfx::Rect(200, 100, 600, 400));
 }
 
 #if defined(OS_MACOSX)

chrome/browser/prefs/pref_service_browsertest.cc

@@ -28,7 +28,7 @@
 typedef InProcessBrowserTest PreservedWindowPlacement;
 
 IN_PROC_BROWSER_TEST_F(PreservedWindowPlacement, PRE_Test) {
-  browser()->window()->SetBounds(gfx::Rect(20, 30, 500, 600));
+  browser()->window()->SetBounds(gfx::Rect(20, 30, 600, 600));
 }
 
 // Fails on Chrome OS as the browser thinks it is restarting after a crash, see
@@ -40,7 +40,7 @@ IN_PROC_BROWSER_TEST_F(PreservedWindowPlacement, PRE_Test) {
 #endif
 IN_PROC_BROWSER_TEST_F(PreservedWindowPlacement, MAYBE_Test) {
   gfx::Rect bounds = browser()->window()->GetBounds();
-  gfx::Rect expected_bounds(gfx::Rect(20, 30, 500, 600));
+  gfx::Rect expected_bounds(gfx::Rect(20, 30, 600, 600));
   ASSERT_EQ(expected_bounds.ToString(), bounds.ToString());
 }
 

chrome/browser/ui/views/omnibox/omnibox_view_views.cc

@@ -330,7 +330,7 @@ bool OmniboxViewViews::IsImeComposing() const {
 }
 
 gfx::Size OmniboxViewViews::GetMinimumSize() const {
-  const int kMinCharacters = 10;
+  const int kMinCharacters = 20;
   return gfx::Size(
       GetFontList().GetExpectedTextWidth(kMinCharacters) + GetInsets().width(),
       GetPreferredSize().height());

chrome/test/chromedriver/test/run_py_tests.py

@@ -974,8 +974,8 @@ class ChromeDriverTest(ChromeDriverBaseTestWithWebServer):
     self._driver.SetWindowSize(size[0], size[1])
     self.assertEquals(size, self._driver.GetWindowSize())
 
-    self._driver.SetWindowSize(600, 400)
-    self.assertEquals([600, 400], self._driver.GetWindowSize())
+    self._driver.SetWindowSize(640, 400)
+    self.assertEquals([640, 400], self._driver.GetWindowSize())
 
   def testWindowMaximize(self):
     self._driver.SetWindowPosition(100, 200)
@@ -986,19 +986,19 @@ class ChromeDriverTest(ChromeDriverBaseTestWithWebServer):
     self.assertNotEqual([500, 300], self._driver.GetWindowSize())
     # Set size first so that the window isn't moved offscreen.
     # See https://bugs.chromium.org/p/chromedriver/issues/detail?id=297.
-    self._driver.SetWindowSize(600, 400)
+    self._driver.SetWindowSize(640, 400)
     self._driver.SetWindowPosition(100, 200)
     self.assertEquals([100, 200], self._driver.GetWindowPosition())
-    self.assertEquals([600, 400], self._driver.GetWindowSize())
+    self.assertEquals([640, 400], self._driver.GetWindowSize())
 
   def testWindowMinimize(self):
     handle_prefix = "CDwindow-"
     handle = self._driver.GetCurrentWindowHandle()
     target = handle[len(handle_prefix):]
     self._driver.SetWindowPosition(100, 200)
-    self._driver.SetWindowSize(600, 400)
+    self._driver.SetWindowSize(640, 400)
     rect = self._driver.MinimizeWindow()
-    expected_rect = {u'y': 200, u'width': 600, u'height': 400, u'x': 100}
+    expected_rect = {u'y': 200, u'width': 640, u'height': 400, u'x': 100}
 
     #check it returned the correct rect
     for key in expected_rect.keys():

chrome/test/data/extensions/api_test/window_update/sizing/test.js

@@ -6,7 +6,7 @@ var pass = chrome.test.callbackPass;
 
 var finalTop = 400;
 var finalLeft = 10;
-var finalWidth = 580;
+var finalWidth = 640;
 var finalHeight = 401;
 
 var chromeWindow = null;