cros: Add app mode restrictions.

chrome/browser/ui/views/frame/browser_view.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/views/frame/browser_view.h"
 
 #include <algorithm>
 
 #include "base/auto_reset.h"
 #include "base/command_line.h"
@@ skipping 2443 matching lines @@
   free(accelerators);
 #else
   views::FocusManager* focus_manager = GetFocusManager();
   DCHECK(focus_manager);
 
   // Let's fill our own accelerator table.
   const std::vector<chrome::AcceleratorMapping> accelerator_list(
       chrome::GetAcceleratorList());
   for (std::vector<chrome::AcceleratorMapping>::const_iterator it =
            accelerator_list.begin(); it != accelerator_list.end(); ++it) {
+#if defined(OS_CHROMEOS)

[sky, 2013/03/04 22:25:14]

Don't we want this when in ash mode too?

[xiyuan, 2013/03/04 22:38:32]

Good point. Removed.

+    // Only allow accelerators of white listed commands to pass through.
+    static const bool is_app_mode = chrome::IsRunningInForcedAppMode();

[sky, 2013/03/04 22:25:14]

Is there a compelling reason to make this static?

[xiyuan, 2013/03/04 22:38:32]

Moved it outside the loop and removed "static".

+    if (is_app_mode && !chrome::IsCommandAllowedInAppMode(it->command_id))
+      continue;
+#endif
+
     ui::Accelerator accelerator(it->keycode, it->modifiers);
     accelerator_table_[accelerator] = it->command_id;
 
     // Also register with the focus manager.
     focus_manager->RegisterAccelerator(
         accelerator, ui::AcceleratorManager::kNormalPriority, this);
   }
 #endif
 }
 
@@ skipping 249 matching lines @@
 
   Browser* modal_browser =
       chrome::FindBrowserWithWebContents(active_dialog->web_contents());
   if (modal_browser && (browser_ != modal_browser)) {
     modal_browser->window()->FlashFrame(true);
     modal_browser->window()->Activate();
   }
 
   AppModalDialogQueue::GetInstance()->ActivateModalDialog();
 }