Another attempt at fixing cmd-left/right.

webkit/api/src/EditorClientImpl.cpp

 /*
  * Copyright (C) 2006, 2007 Apple, Inc.  All rights reserved.
  * Copyright (C) 2009 Google, Inc.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 342 matching lines @@
 //
 
 static const unsigned CtrlKey = 1 << 0;
 static const unsigned AltKey = 1 << 1;
 static const unsigned ShiftKey = 1 << 2;
 static const unsigned MetaKey = 1 << 3;
 #if PLATFORM(DARWIN)
 // Aliases for the generic key defintions to make kbd shortcuts definitions more
 // readable on OS X.
 static const unsigned OptionKey  = AltKey;
+
+// Do not use this constant for anything but cursor movement commands. Keys
+// with cmd set have their |isSystemKey| bit set, so chances are the shortcut
+// will not be executed. Another, less important, reason is that shortcuts
+// defined in the renderer do not blink the menu item that they triggered.  See
+// http://crbug.com/25856 and the bugs linked from there for details.
 static const unsigned CommandKey = MetaKey;
 #endif
 
 // Keys with special meaning. These will be delegated to the editor using
 // the execCommand() method
 struct KeyDownEntry {
     unsigned virtualKey;
     unsigned modifiers;
     const char* name;
 };
@@ skipping 533 matching lines @@
     notImplemented();
 }
 
 void EditorClientImpl::setInputMethodState(bool enabled)
 {
     if (m_webView->client())
         m_webView->client()->setInputMethodEnabled(enabled);
 }
 
 } // namesace WebKit