Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(561)

Unified Diff: chrome/browser/views/tab_contents_container_view.cc

Issue 21298: Make accelerators work in interstitial pages (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/tab_contents/interstitial_page.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/views/tab_contents_container_view.cc
===================================================================
--- chrome/browser/views/tab_contents_container_view.cc (revision 9548)
+++ chrome/browser/views/tab_contents_container_view.cc (working copy)
@@ -178,6 +178,10 @@
bool TabContentsContainerView::ShouldLookupAccelerators(
const views::KeyEvent& e) {
+ // Don't look-up accelerators if we are showing a non-crashed WebContents.
+ // We'll first give the page a chance to process the key events. If it does
+ // not process them, they'll be returned to us and we'll treat them as
+ // accelerators then.
if (tab_contents_ && !tab_contents_->is_crashed() &&
tab_contents_->AsWebContents())
return false;
« no previous file with comments | « chrome/browser/tab_contents/interstitial_page.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698