Allow SiteInstance for about:blank in new tab to be reused.

chrome/browser/ui/omnibox/omnibox_view_browsertest.cc

Index: chrome/browser/ui/omnibox/omnibox_view_browsertest.cc
diff --git a/chrome/browser/ui/omnibox/omnibox_view_browsertest.cc b/chrome/browser/ui/omnibox/omnibox_view_browsertest.cc
index 17826259c81f170f9c3645b0c796be56ddd20f28..687f9b5a14c38996b43fc9146d08094c9d10b7b7 100644
--- a/chrome/browser/ui/omnibox/omnibox_view_browsertest.cc
+++ b/chrome/browser/ui/omnibox/omnibox_view_browsertest.cc
@@ -599,7 +599,14 @@ class OmniboxViewTest : public InProcessBrowserTest,
               popup_model->result().default_match()->type);
 
     // Open the default match.
-    ASSERT_NO_FATAL_FAILURE(SendKey(ui::VKEY_RETURN, 0));
+    {
+      ui_test_utils::WindowedNotificationObserver observer(

[Peter Kasting, 2012/02/06 18:35:53]

Nit: Do we do this anywhere else in this file or other files, that would justify
making it into some sort of helper function?

[Charlie Reis, 2012/02/06 19:23:00]

Good call.  Turns out there's already a SendKeyAndWait helper, and I found one
other place that checks a URL and should be using it.  Look good?

+        content::NOTIFICATION_NAV_ENTRY_COMMITTED,
+        content::Source<content::NavigationController>(
+            &browser()->GetSelectedWebContents()->GetController()));
+      ASSERT_NO_FATAL_FAILURE(SendKey(ui::VKEY_RETURN, 0));
+      observer.Wait();
+    }
     GURL url = browser()->GetSelectedWebContents()->GetURL();
     EXPECT_STREQ(kSearchTextURL, url.spec().c_str());
 
@@ -616,7 +623,14 @@ class OmniboxViewTest : public InProcessBrowserTest,
               popup_model->result().default_match()->type);
 
     // Open the default match.
-    ASSERT_NO_FATAL_FAILURE(SendKey(ui::VKEY_RETURN, 0));
+    {
+      ui_test_utils::WindowedNotificationObserver observer(
+        content::NOTIFICATION_NAV_ENTRY_COMMITTED,
+        content::Source<content::NavigationController>(
+            &browser()->GetSelectedWebContents()->GetController()));
+      ASSERT_NO_FATAL_FAILURE(SendKey(ui::VKEY_RETURN, 0));
+      observer.Wait();
+    }
     url = browser()->GetSelectedWebContents()->GetURL();
     EXPECT_STREQ(kSearchSingleCharURL, url.spec().c_str());
   }
@@ -1157,12 +1171,7 @@ IN_PROC_BROWSER_TEST_F(OmniboxViewTest, AltEnter) {
   AltEnterTest();
 }
 
-// DISABLED http://crbug.com/80118
-#if defined(OS_LINUX)
-IN_PROC_BROWSER_TEST_F(OmniboxViewTest, DISABLED_EnterToSearch) {
-#else
 IN_PROC_BROWSER_TEST_F(OmniboxViewTest, EnterToSearch) {
-#endif  // OS_LINUX
   EnterToSearchTest();
 }