Closed
Bug 1691935
Opened 4 years ago
Closed 4 years ago
Context menu Save Image As is broken again after tb-fission
Categories
(Thunderbird :: General, defect)
Thunderbird
General
Tracking
(thunderbird_esr78 unaffected, thunderbird86 affected)
RESOLVED
FIXED
87 Branch
Tracking | Status | |
---|---|---|
thunderbird_esr78 | --- | unaffected |
thunderbird86 | --- | affected |
People
(Reporter: mkmelin, Assigned: darktrojan)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
in nsContextMenu.js we used to call this.setTarget(node) which set up imageURL.
We don't do that now (setTarget seems unused), so in saveImage() this.imageURL is undefined and it's all downhill from there.
Reporter | ||
Updated•4 years ago
|
Assignee: nobody → geoff
status-thunderbird86:
--- → affected
Assignee | ||
Comment 1•4 years ago
|
||
Replaces two fields that were only set by setTarget and removes setTarget, which is now unused.
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/653d18dfbf42
Fix and test context menu Save Image As. r=mkmelin
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•4 years ago
|
Target Milestone: --- → 87 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•