Title: [112656] trunk/Source/WebCore
Revision
112656
Author
vse...@chromium.org
Date
2012-03-30 06:10:07 -0700 (Fri, 30 Mar 2012)

Log Message

Web Inspector: ScriptsNavigator should open scripts with single click (not double click).
https://bugs.webkit.org/show_bug.cgi?id=82723

Reviewed by Pavel Feldman.

This patch makes ScriptsNavigator open scripts with single click.

* inspector/front-end/ScriptsNavigator.js:
(WebInspector.NavigatorScriptTreeElement.prototype.onattach):
(WebInspector.NavigatorScriptTreeElement.prototype._onclick):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (112655 => 112656)


--- trunk/Source/WebCore/ChangeLog	2012-03-30 12:50:31 UTC (rev 112655)
+++ trunk/Source/WebCore/ChangeLog	2012-03-30 13:10:07 UTC (rev 112656)
@@ -1,3 +1,16 @@
+2012-03-30  Vsevolod Vlasov  <vse...@chromium.org>
+
+        Web Inspector: ScriptsNavigator should open scripts with single click (not double click).
+        https://bugs.webkit.org/show_bug.cgi?id=82723
+
+        Reviewed by Pavel Feldman.
+
+        This patch makes ScriptsNavigator open scripts with single click.
+
+        * inspector/front-end/ScriptsNavigator.js:
+        (WebInspector.NavigatorScriptTreeElement.prototype.onattach):
+        (WebInspector.NavigatorScriptTreeElement.prototype._onclick):
+
 2012-03-30  Vineet Chaudhary  <rgf...@motorola.com>
 
         IDLParser.pm should be able to parse sequence<T> as method argument.

Modified: trunk/Source/WebCore/inspector/front-end/ScriptsNavigator.js (112655 => 112656)


--- trunk/Source/WebCore/inspector/front-end/ScriptsNavigator.js	2012-03-30 12:50:31 UTC (rev 112655)
+++ trunk/Source/WebCore/inspector/front-end/ScriptsNavigator.js	2012-03-30 13:10:07 UTC (rev 112656)
@@ -716,6 +716,7 @@
     onattach: function()
     {
         WebInspector.BaseNavigatorTreeElement.prototype.onattach.call(this);
+        this.listItemElement.addEventListener("click", this._onclick.bind(this), false);
         if (this._uiSourceCode.isSnippet)
             this.listItemElement.addEventListener("contextmenu", this._handleContextMenuEvent.bind(this), false);
     },
@@ -723,7 +724,7 @@
     /**
      * @param {Event} event
      */
-    ondblclick: function(event)
+    _onclick: function(event)
     {
         this._navigator.scriptSelected(this.uiSourceCode);
     },
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to