Web Inspector: Remove unused code from contextmenu of element panel
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Mar 2013 04:47:29 +0000 (04:47 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Mar 2013 04:47:29 +0000 (04:47 +0000)
https://bugs.webkit.org/show_bug.cgi?id=111658

Patch by Seokju Kwon <seokju.kwon@gmail.com> 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):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@145032 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/inspector/front-end/ElementsPanel.js [changed mode: 0644->0755]
Source/WebCore/inspector/front-end/ElementsTreeOutline.js [changed mode: 0644->0755]

index 4e61c53..78ff018 100644 (file)
@@ -1,3 +1,17 @@
+2013-03-06  Seokju Kwon  <seokju.kwon@gmail.com>
+
+        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  <jchaffraix@webkit.org>
 
         [CSS Grid Layout] Extend our grammar to support 2 positions for grid-{row|column}
old mode 100644 (file)
new mode 100755 (executable)
index 578a00f..7f0fc15
@@ -338,7 +338,7 @@ WebInspector.ElementsPanel.prototype = {
         }
 
         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();
old mode 100644 (file)
new mode 100755 (executable)
index b62bef5..8ce7f0e
@@ -513,7 +513,7 @@ WebInspector.ElementsTreeOutline.prototype = {
     {
         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");