Fix the issue that omnibox is editable when menu is opened in aura.

ui/views/controls/menu/menu_controller.cc

 // Copyright (c) 2011 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 "ui/views/controls/menu/menu_controller.h"
 
 #include "base/i18n/case_conversion.h"
 #include "base/i18n/rtl.h"
 #include "base/time.h"
 #include "base/utf_string_conversions.h"
@@ skipping 871 matching lines @@
 
 #elif defined(USE_AURA)
 base::MessagePumpDispatcher::DispatchStatus
     MenuController::Dispatch(XEvent* xev) {
   if (exit_type_ == EXIT_ALL || exit_type_ == EXIT_DESTROYED) {
     aura::RootWindow::GetInstance()->GetDispatcher()->Dispatch(xev);
     return base::MessagePumpDispatcher::EVENT_QUIT;
   }
   switch (ui::EventTypeFromNative(xev)) {
     case ui::ET_KEY_PRESSED:
-      OnKeyDown(ui::KeyboardCodeFromNative(xev));
-      // OnKeyDown may have set exit_type_.
-      if (exit_type_ != EXIT_NONE)
+      if (!OnKeyDown(ui::KeyboardCodeFromNative(xev)))
         return base::MessagePumpDispatcher::EVENT_QUIT;
 
       // TODO(oshima): support SelectChar
-      break;
+      // See http://crbug.com/107869

[oshima, 2011/12/16 18:21:33]

nit: period at the end of both lines.

[jennyz, 2011/12/16 18:26:39]

Done.

+
+      return base::MessagePumpDispatcher::EVENT_PROCESSED;
     case ui::ET_KEY_RELEASED:
       return base::MessagePumpDispatcher::EVENT_PROCESSED;
     default:
       break;
   }
 
   // TODO(oshima): Update Windows' Dispatcher to return DispatchStatus
   // instead of bool.
   if (aura::RootWindow::GetInstance()->GetDispatcher()->Dispatch(xev) ==
       base::MessagePumpDispatcher::EVENT_IGNORED)
@@ skipping 1037 matching lines @@
   // is necessary to exit from nested loop (See Dispatch methods).
   // Send non-op event so that Dispatch method will always be called.
   // crbug.com/104684.
   if (exit_type_ == EXIT_ALL || exit_type_ == EXIT_DESTROYED)
     aura::RootWindow::GetInstance()->PostNativeEvent(ui::CreateNoopEvent());
 #endif
 }
 
 
 }  // namespace views