Address post-commit comments from codereview.chromium.org/371059 and codereview.chromium.org/293019

chrome/browser/renderer_host/render_widget_host_view_mac.mm

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/renderer_host/render_widget_host_view_mac.h"
 
 #include "base/histogram.h"
 #import "base/scoped_nsobject.h"
 #include "base/string_util.h"
 #include "base/sys_string_conversions.h"
@@ skipping 647 matching lines @@
   //
   // If this object's retainCount is 1, the only reference is the one held by
   // keepSelfAlive.  All other references may have been destroyed in the
   // RenderWidgetHost::ForwardKeyboardEvent call above if it resulted in tab
   // closure.  Were it not for that single reference, this object would
   // already be deallocated.  In that case, there's no point in calling
   // -interpretKeyEvents:.
   if ([self retainCount] > 1 && [theEvent type] == NSKeyDown) {
     [self interpretKeyEvents:[NSArray arrayWithObject:theEvent]];
 
-    // We don't get insertText: calls if ctrl is down, so synthesize a keypress
-    // event for that case. Note that this makes our behavior deviate from the
-    // windows and linux versions of chrome (however, see http://crbug.com/13891
-    // ), but it makes us similar to how Safari behaves.
+    // We don't get insertText: calls if ctrl is down and not even a keyDown:
+    // call if cmd is down, so synthesize a keypress event for these cases.
+    // Note that this makes our behavior deviate from the windows and linux
+    // versions of chrome (however, see http://crbug.com/13891 ), but it makes
+    // us behave similar to how Safari behaves.
     if ([theEvent modifierFlags] & (NSControlKeyMask | NSCommandKeyMask) &&
         lastKeyPressedEvent_.get() != theEvent &&
         [[theEvent characters] length] > 0 &&
         renderWidgetHostView_->render_widget_host_) {
       NativeWebKeyboardEvent event([[theEvent characters] characterAtIndex:0],
                                    ToWebKitModifiers([theEvent modifierFlags]),
                                    base::Time::Now().ToDoubleT());
       // We fire menu items on keydown, we don't want to activate menu items
       // twice.
       event.skip_in_browser = true;
@@ skipping 657 matching lines @@
                                  base::Time::Now().ToDoubleT());
     renderWidgetHostView_->render_widget_host_->ForwardKeyboardEvent(event);
   } else {
     renderWidgetHostView_->render_widget_host_->ImeConfirmComposition(
         UTF8ToUTF16([im_text UTF8String]));
   }
   renderWidgetHostView_->im_composing_ = false;
 }
 
 @end