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

Unified Diff: chrome/renderer/render_view.cc

Issue 244044: Address review comments. (Closed)
Patch Set: To boldy use the same |switch| style as the rest of our code Created 11 years, 3 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 | « no previous file | chrome/renderer/render_view_unittest_mac.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/render_view.cc
diff --git a/chrome/renderer/render_view.cc b/chrome/renderer/render_view.cc
index ac31fdedf0aa618d5c1119d4746813f04c922309..8bcc309fbbad1245565265f658d93a0236d7960d 100644
--- a/chrome/renderer/render_view.cc
+++ b/chrome/renderer/render_view.cc
@@ -1520,9 +1520,9 @@ bool RenderView::handleCurrentKeyboardEvent() {
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.
+ // In gtk and cocoa, 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.
if (!frame->executeCommand(WebString::fromUTF8(it->name),
WebString::fromUTF8(it->value)))
break;
« no previous file with comments | « no previous file | chrome/renderer/render_view_unittest_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698