Bug 1578084 - Add onCompletedImage to other context conditions. r=frg a=frg
authorIan Neal <iann_cvs@blueyonder.co.uk>
Tue, 01 Oct 2019 16:59:17 +0200
changeset 32317 fe41cc38a07256e7992bbf89d5e84b49589b0a84
parent 32316 b61f35e0c01dada21bc6bc910de8f157b2024fb3
child 32318 1562bf096151922a86b717c6883c3ab7fa2efee6
push id220
push userfrgrahl@gmx.net
push dateTue, 01 Oct 2019 15:01:16 +0000
treeherdercomm-esr60@e71c6693877f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersfrg, frg
bugs1578084
Bug 1578084 - Add onCompletedImage to other context conditions. r=frg a=frg
suite/base/content/nsContextMenu.js
--- a/suite/base/content/nsContextMenu.js
+++ b/suite/base/content/nsContextMenu.js
@@ -191,17 +191,18 @@ nsContextMenu.prototype = {
     // Save/send link depends on whether we're in a link.
     if (this.onSaveableLink)
       goSetMenuValue("context-savelink",
                      this.autoDownload ? "valueSave" : "valueSaveAs");
     this.showItem("context-savelink", this.onSaveableLink);
     this.showItem("context-sendlink", this.onSaveableLink);
 
     // Save image depends on having loaded its content, video and audio don't.
-    showSave = this.onLoadedImage || this.onStandaloneImage || this.onCanvas;
+    showSave = (this.onLoadedImage && this.onCompletedImage) ||
+               this.onStandaloneImage || this.onCanvas;
     if (showSave)
       goSetMenuValue("context-saveimage",
                      this.autoDownload ? "valueSave" : "valueSaveAs");
     this.showItem("context-saveimage", showSave);
     this.showItem("context-savevideo", this.onVideo);
     this.showItem("context-saveaudio", this.onAudio);
     this.showItem("context-video-saveimage", this.onVideo);
     if (this.onVideo)