Handle Esc key press event in Local NTP.

chrome/browser/ui/search/instant_extended_interactive_uitest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <sstream>
 
 #include "base/command_line.h"
 #include "base/prefs/pref_service.h"
 #include "base/string_util.h"
 #include "base/stringprintf.h"
@@ skipping 1912 matching lines @@
   SetOmniboxText("");
 
   EXPECT_TRUE(GetIntFromJS(
       overlay,
       "chrome.embeddedSearch.searchBox.nativeSuggestions.length",
       &num_autocomplete_results));
   EXPECT_EQ(0, num_autocomplete_results);
 }
 
 // Test that hitting Esc to clear the omnibox works. http://crbug.com/231744.
-// TODO(sreeram): reenable once ESC bug is actually fixed.
-IN_PROC_BROWSER_TEST_F(InstantExtendedTest, DISABLED_EscapeClearsOmnibox) {
+IN_PROC_BROWSER_TEST_F(InstantExtendedTest, EscapeClearsOmnibox) {
+  // Use only the local overlay.

[samarth, 2013/05/05 23:49:30]

This shouldn't be necessary given the call to SetInstantEnabled below.  Also,
why force the local only page at all?  We can't really test (or want to test)
its behavior in this test.

[kmadhusu, 2013/05/06 00:11:16]

hmm. No specific reason to force this test for local only page. I will
investigate more and update this test.

+  CommandLine::ForCurrentProcess()->AppendSwitch(
+      switches::kEnableLocalOnlyInstantExtendedAPI);
+  ASSERT_TRUE(chrome::IsLocalOnlyInstantExtendedAPIEnabled());
   ASSERT_NO_FATAL_FAILURE(SetupInstant(browser()));
-  FocusOmniboxAndWaitForInstantOverlayAndNTPSupport();
+
+  // The second argument indicates to use only the local overlay and NTP.
+  instant()->SetInstantEnabled(true, true);
+
+  // Focus omnibox and confirm overlay isn't shown.
+  FocusOmniboxAndWaitForInstantOverlaySupport();
 
   // Navigate to the Instant NTP, and wait for it to be recognized.
   content::WindowedNotificationObserver instant_tab_observer(
       chrome::NOTIFICATION_INSTANT_TAB_SUPPORT_DETERMINED,
       content::NotificationService::AllSources());
   ui_test_utils::NavigateToURLWithDisposition(browser(),
                                               GURL(chrome::kChromeUINewTabURL),
                                               CURRENT_TAB,
                                               ui_test_utils::BROWSER_TEST_NONE);
   instant_tab_observer.Wait();
 
-  content::WebContents* contents =
-      browser()->tab_strip_model()->GetActiveWebContents();
+  // Type a query. Verify that the query is shown in the omnibox.
+  SetOmniboxText("mojo");
+  EXPECT_EQ("mojo", GetOmniboxText());
 
-  // Type a query. Verify that the query is seen by the page.
-  SetOmniboxText("mojo");
-  std::string query;
-  EXPECT_TRUE(GetStringFromJS(contents, "chrome.embeddedSearch.searchBox.value",
-                              &query));
-  EXPECT_EQ("mojo", query);
-
-  EXPECT_TRUE(content::ExecuteScript(contents,
-                                     "onChangeCalls = submitCount = 0;"));
-
-  // Hit Escape, and verify that the page sees that the query is cleared.
+  // Hit Escape, and verify that the omnibox text is cleared.
   SendEscape();
-  EXPECT_TRUE(GetStringFromJS(contents, "chrome.embeddedSearch.searchBox.value",
-                              &query));
-  EXPECT_EQ("", query);
   EXPECT_EQ("", GetOmniboxText());

[samarth, 2013/05/05 23:49:30]

This should be true, but isn't really testing everything.  We should test that
we sent a keypress event.  Generally we do this in tests by counting the IPCs in
the test js file.  See UpdateSearchState() and the test instant file
(https://code.google.com/p/chromium/codesearch#chromium/src/chrome/test/data/i...).

[kmadhusu, 2013/05/06 00:11:16]

I completely agree. This test can be improved. I will refer to
UpdateSearchState() and will update the test.

-
-  EXPECT_TRUE(UpdateSearchState(contents));
-  EXPECT_LT(0, on_change_calls_);
-  EXPECT_EQ(0, submit_count_);
 }
 
 IN_PROC_BROWSER_TEST_F(InstantExtendedTest, OnDefaultSearchProviderChanged) {
   InstantService* instant_service =
       InstantServiceFactory::GetForProfile(browser()->profile());
   ASSERT_NE(static_cast<InstantService*>(NULL), instant_service);
 
   // Setup Instant.
   ASSERT_NO_FATAL_FAILURE(SetupInstant(browser()));
   FocusOmniboxAndWaitForInstantOverlayAndNTPSupport();
@@ skipping 297 matching lines @@
   ASSERT_TRUE(AddSearchToHistory(ASCIIToUTF16("query"), 10000));
   BlockUntilHistoryProcessesPendingRequests();
 
   SetOmniboxText("quer");
 
   // Should get 2 suggestions from SearchProvider:
   //   - SWYT for "quer"
   //   - Search history suggestion for "query"
   EXPECT_EQ(2, CountSearchProviderSuggestions());
 }