Changeset 57590 in webkit

Timestamp:

Apr 14, 2010 11:18:34 AM (14 years ago)

Author:

eric@webkit.org

Message:

2010-04-14 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: Ctrl-L (Clear History) does nothing on Windows
​https://bugs.webkit.org/show_bug.cgi?id=37579

• inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/ConsoleView.js (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-04-14  Alexander Pavlov  <apavlov@chromium.org>
+
+        Reviewed by Joseph Pecoraro.
+
+        Web Inspector: Ctrl-L (Clear History) does nothing on Windows
+        https://bugs.webkit.org/show_bug.cgi?id=37579
+
+        * inspector/front-end/ConsoleView.js:
+        (WebInspector.ConsoleView):
+
 2010-04-14  Anton Muhin  <antonm@chromium.org>
 

trunk/WebCore/inspector/front-end/ConsoleView.js

@@ -94 +94 @@
 
     var shortcut;
+
+    shortcut = WebInspector.KeyboardShortcut.makeKey("k", WebInspector.KeyboardShortcut.Modifiers.Meta);
+    // This case requires a separate bound function as its isMacOnly property should not be shared among different shortcut handlers.
+    this._shortcuts[shortcut] = this.requestClearMessages.bind(this);
+    this._shortcuts[shortcut].isMacOnly = true;
+
     var clearConsoleHandler = this.requestClearMessages.bind(this);
-
-    shortcut = WebInspector.KeyboardShortcut.makeKey("k", WebInspector.KeyboardShortcut.Modifiers.Meta);
-    this._shortcuts[shortcut] = clearConsoleHandler;
-    this._shortcuts[shortcut].isMacOnly = true;
     shortcut = WebInspector.KeyboardShortcut.makeKey("l", WebInspector.KeyboardShortcut.Modifiers.Ctrl);
     this._shortcuts[shortcut] = clearConsoleHandler;