Handle Esc key press event in Local NTP.

chrome/browser/ui/search/instant_controller.cc

 // Copyright 2012 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 "chrome/browser/ui/search/instant_controller.h"
 
 #include "base/metrics/histogram.h"
 #include "base/string_util.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
@@ skipping 346 matching lines @@
           // Ensure we don't send the omnibox text to a random webpage (the new
           // tab), by comparing the old and new WebContents.
           if (escape_pressed &&
               instant_tab_->contents() == browser_->GetActiveWebContents()) {
             // If the omnibox is blank, send an onchange("") instead of an
             // onsubmit(""). This is to avoid confusion with the onsubmit("")
             // that we send when the user hits Enter to navigate to a URL.
             // onchange("") is used for a similar situation with the overlay
             // (when the overlay is dismissed because the user hit Escape); it
             // does the right thing for committed tabs as well.
-            if (full_text.empty())
-              instant_tab_->Update(string16(), 0, 0, true);
-            else
+            if (full_text.empty()) {
+              if (UsingLocalPage())

[samarth, 2013/05/04 01:04:17]

Let's send the right event for both local and remote.  We can fix the
server-side code too.

[kmadhusu, 2013/05/04 03:34:38]

Done.

+                OnEscKeyPressed();

[samarth, 2013/05/04 01:04:17]

Just call instant_tab_->OnEscKeyPressed() here. No need for the other function
below.

[kmadhusu, 2013/05/04 03:34:38]

Done.

+              else
+                instant_tab_->Update(string16(), 0, 0, true);
+             } else {
               instant_tab_->Submit(full_text);
+            }
           }
         } else if (!full_text.empty()) {
           // If |full_text| is empty, the user is on the NTP. The overlay may
           // be showing custom NTP content; hide only if that's not the case.
           HideOverlay();
         }
       } else if (full_text.empty()) {
         // The user is typing, and backspaced away all omnibox text. Clear
         // |last_omnibox_text_| so that we don't attempt to set suggestions.
         last_omnibox_text_.clear();
@@ skipping 1299 matching lines @@
     }
   }
 
   return false;
 }
 
 bool InstantController::UsingLocalPage() const {
   return (instant_tab_ && instant_tab_->IsLocal()) ||
       (!instant_tab_ && overlay_ && overlay_->IsLocal());
 }
+
+void InstantController::OnEscKeyPressed() const {
+  DCHECK(extended_enabled_);
+  if (!instant_tab_ && !overlay_)
+    return;
+
+  if (instant_tab_)
+    instant_tab_->EscKeyPressed();
+  else
+    overlay_->EscKeyPressed();
+}