fix shortcuts unexpectedly working in modal dialog

chrome/browser/chromeos/options/wifi_config_view.cc

Index: chrome/browser/chromeos/options/wifi_config_view.cc
diff --git a/chrome/browser/chromeos/options/wifi_config_view.cc b/chrome/browser/chromeos/options/wifi_config_view.cc
index a697b7a2fe09c18f71683e9a838e9c4b8b48ce61..a57b337094509b90fdc5efe3a5e2c6d939e9b6ac 100644
--- a/chrome/browser/chromeos/options/wifi_config_view.cc
+++ b/chrome/browser/chromeos/options/wifi_config_view.cc
@@ -25,6 +25,7 @@
 #include "ui/views/controls/combobox/combobox.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/controls/textfield/textfield.h"
+#include "ui/views/focus/focus_manager.h"
 #include "ui/views/layout/grid_layout.h"
 #include "ui/views/layout/layout_constants.h"
 #include "ui/views/widget/widget.h"
@@ -377,6 +378,7 @@ WifiConfigView::WifiConfigView(NetworkConfigView* parent, bool show_8021x)
 }
 
 WifiConfigView::~WifiConfigView() {
+  views::FocusManager::set_shortcut_handling_suspended(false);

[Daniel Erat, 2012/09/27 22:24:30]

the fact that FocusManager doesn't keep a count of how many times it's been
asked to suspend/unsuspend shortcut handling makes me a bit nervous.  is it
guaranteed that no one else can make additional calls to suspend/unsuspend while
this view is shown?

   if (cert_library_)
     cert_library_->RemoveObserver(this);
 }
@@ -880,6 +882,7 @@ void WifiConfigView::Init(WifiNetwork* wifi, bool show_8021x) {
 
   views::GridLayout* layout = views::GridLayout::CreatePanel(this);
   SetLayoutManager(layout);
+  views::FocusManager::set_shortcut_handling_suspended(true);
 
   const int column_view_set_id = 0;
   views::ColumnSet* column_set = layout->AddColumnSet(column_view_set_id);