Comment #3 on issue 8659 by [email protected]: Accelerator handling Linux problems. http://code.google.com/p/chromium/issues/detail?id=8659
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=11759
------------------------------------------------------------------------
r11759 | [email protected] | 2009-03-16 12:51:56 -0700 (Mon, 16 Mar 2009)
| 12 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser.cc?r1=11759&r2=11758
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser.h?r1=11759&r2=11758
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser_window.h?r1=11759&r2=11758
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/browser_window_cocoa.h?r1=11759&r2=11758
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/browser_window_cocoa.mm?r1=11759&r2=11758
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/browser_toolbar_gtk.cc?r1=11759&r2=11758
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/browser_toolbar_gtk.h?r1=11759&r2=11758
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/browser_window_gtk.cc?r1=11759&r2=11758
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/browser_window_gtk.h?r1=11759&r2=11758
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_widget_host_view_gtk.cc?r1=11759&r2=11758
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/web_contents_view_gtk.cc?r1=11759&r2=11758
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/frame/browser_view.cc?r1=11759&r2=11758
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/frame/browser_view.h?r1=11759&r2=11758
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/test_browser_window.h?r1=11759&r2=11758
Linux accelerators cleanup:
- Give renderer a chance to handle accelerators before browser does.
- Handle browser accelerators that aren't attached to any particular UI
element in BrowserWindowGtk rather than in BrowserToolbarGtk
- Use Browser::ExecuteCommand() to handle accelerator activation
- Switch a random void* to gfx::NativeWindow
- Enable three browser commands on linux : Focus Location, Focus Search,
Open file
This fully enables ctrl-l, ctrl-k, and ctrl-o. This fixes copy-pasta in the
omnibox. This fixes the problem Dean described with
<http://www.quirksmode.org/js/keys.html>.
bug=8659
Review URL: http://codereview.chromium.org/42190
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
