Title: [145032] trunk/Source/WebCore
Revision
145032
Author
[email protected]
Date
2013-03-06 20:47:29 -0800 (Wed, 06 Mar 2013)

Log Message

Web Inspector: Remove unused code from contextmenu of element panel
https://bugs.webkit.org/show_bug.cgi?id=111658

Patch by Seokju Kwon <[email protected]> on 2013-03-06
Reviewed by Pavel Feldman.

No new tests (no functional changes).

* inspector/front-end/ElementsPanel.js:
(WebInspector.ElementsPanel.prototype._contextMenuEventFired.set get var):
* inspector/front-end/ElementsTreeOutline.js:
(WebInspector.ElementsTreeOutline.prototype.populateContextMenu):

Modified Paths

Property Changed

Diff

Modified: trunk/Source/WebCore/ChangeLog (145031 => 145032)


--- trunk/Source/WebCore/ChangeLog	2013-03-07 04:46:16 UTC (rev 145031)
+++ trunk/Source/WebCore/ChangeLog	2013-03-07 04:47:29 UTC (rev 145032)
@@ -1,3 +1,17 @@
+2013-03-06  Seokju Kwon  <[email protected]>
+
+        Web Inspector: Remove unused code from contextmenu of element panel
+        https://bugs.webkit.org/show_bug.cgi?id=111658
+
+        Reviewed by Pavel Feldman.
+
+        No new tests (no functional changes).
+
+        * inspector/front-end/ElementsPanel.js:
+        (WebInspector.ElementsPanel.prototype._contextMenuEventFired.set get var):
+        * inspector/front-end/ElementsTreeOutline.js:
+        (WebInspector.ElementsTreeOutline.prototype.populateContextMenu):
+
 2013-03-06  Julien Chaffraix  <[email protected]>
 
         [CSS Grid Layout] Extend our grammar to support 2 positions for grid-{row|column}

Modified: trunk/Source/WebCore/inspector/front-end/ElementsPanel.js (145031 => 145032)


--- trunk/Source/WebCore/inspector/front-end/ElementsPanel.js	2013-03-07 04:46:16 UTC (rev 145031)
+++ trunk/Source/WebCore/inspector/front-end/ElementsPanel.js	2013-03-07 04:47:29 UTC (rev 145032)
@@ -338,7 +338,7 @@
         }
 
         var contextMenu = new WebInspector.ContextMenu(event);
-        var populated = this.treeOutline.populateContextMenu(contextMenu, event);
+        this.treeOutline.populateContextMenu(contextMenu, event);
 
         if (WebInspector.experimentsSettings.cssRegions.isEnabled()) {
             contextMenu.appendSeparator();
Property changes on: trunk/Source/WebCore/inspector/front-end/ElementsPanel.js
___________________________________________________________________

Added: svn:executable

Modified: trunk/Source/WebCore/inspector/front-end/ElementsTreeOutline.js (145031 => 145032)


--- trunk/Source/WebCore/inspector/front-end/ElementsTreeOutline.js	2013-03-07 04:46:16 UTC (rev 145031)
+++ trunk/Source/WebCore/inspector/front-end/ElementsTreeOutline.js	2013-03-07 04:47:29 UTC (rev 145032)
@@ -513,7 +513,7 @@
     {
         var treeElement = this._treeElementFromEvent(event);
         if (!treeElement)
-            return false;
+            return;
 
         var isTag = treeElement.representedObject.nodeType() === Node.ELEMENT_NODE;
         var textNode = event.target.enclosingNodeOrSelfWithClass("webkit-html-text-node");
Property changes on: trunk/Source/WebCore/inspector/front-end/ElementsTreeOutline.js
___________________________________________________________________

Added: svn:executable

_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to