Changeset 116850 in webkit

Timestamp:

May 12, 2012 5:02:08 AM (12 years ago)

Author:

loislo@chromium.org

Message:

Web Inspector: Load context menu item has to be shown for all Profiles panel.
​https://bugs.webkit.org/show_bug.cgi?id=86290

Reviewed by Pavel Feldman.

• inspector/front-end/ProfilesPanel.js:

(WebInspector.ProfilesPanel.prototype._handleContextMenuEvent):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/ProfilesPanel.js (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-05-12  Ilya Tikhonovsky  <loislo@chromium.org>
+
+        Web Inspector: Load context menu item has to be shown for all Profiles panel.
+        https://bugs.webkit.org/show_bug.cgi?id=86290
+
+        Reviewed by Pavel Feldman.
+
+        * inspector/front-end/ProfilesPanel.js:
+        (WebInspector.ProfilesPanel.prototype._handleContextMenuEvent):
+
 2012-05-12  Pavel Feldman  <pfeldman@chromium.org>
 

trunk/Source/WebCore/inspector/front-end/ProfilesPanel.js

@@ -224 +224 @@
 
     this._createFileSelectorElement();
+    this.element.addEventListener("contextmenu", this._handleContextMenuEvent.bind(this), true);
+
 }
 
@@ -391 +393 @@
     {
         var element = event.srcElement;
-        while (element && !element.treeElement)
+        while (element && !element.treeElement && element !== this.element)
             element = element.parentElement;
         if (!element)
             return;
-        if (element.treeElement.handleContextMenuEvent)
+        if (element.treeElement && element.treeElement.handleContextMenuEvent) {
             element.treeElement.handleContextMenuEvent(event);
+            return;
+        }
+        var contextMenu = new WebInspector.ContextMenu();
+        contextMenu.appendItem(WebInspector.UIString("Load profile\u2026"), this._fileSelectorElement.click.bind(this._fileSelectorElement));
+        contextMenu.show(event);
     },