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

Unified Diff: chrome/browser/tab_contents/web_contents_view_gtk.cc

Issue 42190: Linux accelerators cleanup (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 9 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
Index: chrome/browser/tab_contents/web_contents_view_gtk.cc
===================================================================
--- chrome/browser/tab_contents/web_contents_view_gtk.cc (revision 11738)
+++ chrome/browser/tab_contents/web_contents_view_gtk.cc (working copy)
@@ -162,7 +162,6 @@
web_contents_->delegate()->SetFocusToLocationBar();
}
-
void WebContentsViewGtk::HandleKeyboardEvent(
const NativeWebKeyboardEvent& event) {
// The renderer returned a keyboard event it did not process. This may be a
@@ -180,8 +179,8 @@
break;
default:
// This may be an accelerator. Pass it on to GTK.
- gtk_accel_groups_activate(G_OBJECT(vbox_->window),
- event.os_event->keyval,
+ GtkWindow* window = GetTopLevelNativeView();
Dean McNamee 2009/03/16 19:19:02 GetTopLevelNativeView returning a Window still see
Evan Stade 2009/03/16 19:25:49 will change the name of that function in a follow
+ gtk_accel_groups_activate(G_OBJECT(window), event.os_event->keyval,
GdkModifierType(event.os_event->state));
}
}

Powered by Google App Engine
This is Rietveld 408576698