Chromium Code Reviews| Index: chrome/renderer/render_view.cc |
| diff --git a/chrome/renderer/render_view.cc b/chrome/renderer/render_view.cc |
| index e86c27db86fc7c33685fcffceb6a224cf43f6bcd..542c0b8fef49af17dbbade6170416a1f2304ca54 100644 |
| --- a/chrome/renderer/render_view.cc |
| +++ b/chrome/renderer/render_view.cc |
| @@ -1512,13 +1512,18 @@ bool RenderView::handleCurrentKeyboardEvent() { |
| EditCommands::iterator it = edit_commands_.begin(); |
| EditCommands::iterator end = edit_commands_.end(); |
| + bool did_execute_command = false; |
| for (; it != end; ++it) { |
| + // In gtk, it's possible to bind multiple edit commands to one key (but it's |
| + // the exception). Once one edit command is not executed, it seems safest to |
| + // not execute the rest. |
|
jeremy
2009/09/29 18:12:56
Are we sure this isn't possible on OSX as well?
Ac
Nico
2009/09/30 03:51:35
Oh, this wasn't supposed to mean "only in gtk" but
|
| if (!frame->executeCommand(WebString::fromUTF8(it->name), |
| WebString::fromUTF8(it->value))) |
| break; |
| + did_execute_command = true; |
| } |
| - return true; |
| + return did_execute_command; |
| } |
| void RenderView::spellCheck( |