Comment #19 on issue 30707 by [email protected]: Keyboard shortcuts failed to work on Bookmark sync Sign in Authentication dialog
http://code.google.com/p/chromium/issues/detail?id=30707

The reason for this problem described in comment 14 is right. However, I don't think it's necessary to split the code into a separated class. The keyboard handling code of HtmlDialog is much simpler than the browser window, because it's not necessary to handle any browser specific shortcut keys in a html dialog. We just need to implement the {Pre,}HandleKeyboardEvent() methods of each HtmlDialogTabContentsDelegate implementation classes to pass the keyboard events to system default handler. For windows, it's DefWindowProc. For Mac, it's [window redispatchEvent:event]. But nothing
to do for Linux.

--
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

Reply via email to