Changeset 88942 in webkit

Timestamp:

Jun 15, 2011 9:00:34 AM (13 years ago)

Author:

pfeldman@chromium.org

Message:

2011-06-15 Pavel Feldman <​pfeldman@google.com>

Reviewed by Yury Semikhatsky.

Web Inspector: introduce context menu on ImageView with Copy and Open in new tab actions.
​https://bugs.webkit.org/show_bug.cgi?id=62727

• English.lproj/localizedStrings.js:
• inspector/front-end/ImageView.js: (WebInspector.ImageView.prototype._base64ToSize): (WebInspector.ImageView.prototype._contextMenu): (WebInspector.ImageView.prototype._copyImageURL): (WebInspector.ImageView.prototype._openInNewTab):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• English.lproj/localizedStrings.js (modified) (previous)
• inspector/front-end/ImageView.js (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-06-15  Pavel Feldman  <pfeldman@google.com>
+
+        Reviewed by Yury Semikhatsky.
+
+        Web Inspector: introduce context menu on ImageView with Copy and Open in new tab actions.
+        https://bugs.webkit.org/show_bug.cgi?id=62727
+
+        * English.lproj/localizedStrings.js:
+        * inspector/front-end/ImageView.js:
+        (WebInspector.ImageView.prototype._base64ToSize):
+        (WebInspector.ImageView.prototype._contextMenu):
+        (WebInspector.ImageView.prototype._copyImageURL):
+        (WebInspector.ImageView.prototype._openInNewTab):
+
 2011-06-15  Andrey Kosyakov  <caseq@chromium.org>
 

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

@@ -58 +58 @@
         imagePreviewElement.addStyleClass("resource-image-view");
         imageContainer.appendChild(imagePreviewElement);
+        imagePreviewElement.addEventListener("contextmenu", this._contextMenu.bind(this), true);
 
         this._container = document.createElement("div");
@@ -119 +120 @@
             size--;
         return size;
+    },
+
+    _contextMenu: function(event)
+    {
+        var contextMenu = new WebInspector.ContextMenu();
+        contextMenu.appendItem(WebInspector.UIString(WebInspector.useLowerCaseMenuTitles() ? "Copy image URL" : "Copy Image URL"), this._copyImageURL.bind(this));
+        contextMenu.appendItem(WebInspector.UIString(WebInspector.useLowerCaseMenuTitles() ? "Open image in new tab" : "Open Image in New Tab"), this._openInNewTab.bind(this));
+        contextMenu.show(event);
+    },
+
+    _copyImageURL: function(event)
+    {
+        InspectorFrontendHost.copyText(this.resource.url);
+    },
+
+    _openInNewTab: function(event)
+    {
+        WebInspector.openResource(this.resource.url, false);
     }
 }