Chromium Code Reviews| Index: chrome/browser/ui/search/instant_controller.cc |
| diff --git a/chrome/browser/ui/search/instant_controller.cc b/chrome/browser/ui/search/instant_controller.cc |
| index 734223f576163de72022b539f28ce94622af602d..d491709007c337e4078718d4db12dad8e01d138a 100644 |
| --- a/chrome/browser/ui/search/instant_controller.cc |
| +++ b/chrome/browser/ui/search/instant_controller.cc |
| @@ -358,14 +358,8 @@ bool InstantController::Update(const AutocompleteMatch& match, |
| // 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()) |
|
samarth
2013/05/04 16:37:35
Add a TODO here for sending an escape key press in
kmadhusu
2013/05/05 22:44:54
Done.
|
| - instant_tab_->Update(string16(), 0, 0, true); |
| + instant_tab_->EscKeyPressed(); |
| else |
| instant_tab_->Submit(full_text); |
| } |