1358755 - Cannot save image by drag and drop to other than Finder/Desktop

Status: RESOLVED FIXED

(Core :: DOM: Copy & Paste and Drag & Drop, defect)

Description

[Des]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:53.0) Gecko/20100101 Firefox/53.0
Build ID: 20170413192749

Steps to reproduce:

1. Open Nightly
2. Open https://addons.cdn.mozilla.net/static/img/icons/firefox.png in current tab
3. Drag and drop the image to a third part file management tool (eg Forklift) rather than Desktop


Actual results:

The image can be dragged, but when dropped, nothing happens.


Expected results:

The image can be dragged, and when drop it, it's saved to desktop.

This is largely the issue in Bug 1325770 (thus I have largely copied the report), which appears to have been resolved only insofar as relates to interoperability with Finder either as the Desktop or the File Management tool.  However, the issue remains with any other file management tool (eg. Forklift, Double Commander, Commander One)

Comment 1

[Emil Pasca [:emilpasca]]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:55.0) Gecko/20100101 Firefox/55.0

I have tested this issue on Mac OS 10.12 x64 with the latest Firefox release (53.0) and the latest Nightly (55.0a1-20170425030221) and managed to reproduce it following the steps described in the description.
The image can be dragged and when dropped to a third part file management tool (eg Forklift), nothing happens.

Comment 2

[Neil Deakin]

Works fine in 52 however, I would guess that this is almost certainly a regression from 1235162.

Comment 3

[Stephen A Pohl [:spohl]]

Attachment: Patch

Comment 4

[Stephen A Pohl [:spohl]]

[Tracking Requested - why for this release]:
Regression from bug 1235162.

Comment 5

[Julien Cristau [:jcristau]]

Regression in 53, the fix should probably be uplifted to beta54; tracking for 54/55.

Comment 6

[Liz Henry (:lizzard) (relman/hg->git project)]

Duplicate of bug 1330470?

Comment 7

[Liz Henry (:lizzard) (relman/hg->git project)]

If it isn't then the same idea applies, let's verify the fix on m-c and then try to uplift to 54.

Comment 8

[Stephen A Pohl [:spohl]]

(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #6)
> Duplicate of bug 1330470?

No, these are two separate problems with two separate patches.

Comment 9

[Stephen A Pohl [:spohl]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/0c91cb327e453b6dfe8f4064ed1f71d594bdab5b
Bug 1358755: Fix dragging of files from Firefox to third-party file management applications on macOS. r=mstange

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/0c91cb327e45

Comment 11

[Nathan Froyd [:froydnj]]

Please request uplift on this when you get a chance.  Thanks!

Comment 12

[Stephen A Pohl [:spohl]]

Comment on attachment 8867830 [details] [diff] [review]
Patch

Approval Request Comment
[Feature/Bug causing the regression]: bug 1235162
[User impact if declined]: When files (such as images) are being dragged from Firefox to third-party file management applications on OSX/macOS, they cannot be dropped like they used to and the drag will fail.
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: yes
[Needs manual test from QE? If yes, steps to reproduce]: Yes, preferably. Steps in comment 0.
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: minimally
[Why is the change risky/not risky?]: This is a very small (2 loc) and well-understood change.
[String changes made/needed]: none

Comment 13

[Julien Cristau [:jcristau]]

Comment on attachment 8867830 [details] [diff] [review]
Patch

moving uplift request from aurora to beta, since aurora is going away.

Comment 14

[Gerry Chang [:gchang]]

Hi Emil,
Can you help check if this issue is fixed in the latest nightly?

Comment 15

[Gerry Chang [:gchang]]

Comment on attachment 8867830 [details] [diff] [review]
Patch

Fix a regression about drag & drop. Beta54+. Should be in 54 beta 11.

Comment 16

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/6441d136fc1c

Comment 17

[Bogdan Maris, Desktop Test Engineering]

Reproduced the initial issue using affected build 54 beta 8 only on Windows and macOS and verified that it's fixed on Firefox 54 beta 11. 
On Ubuntu 16.04 we could not reproduce the initial issue on affected build using Konqueror, Nautilus and Dolphin but we could verify it works using latest Firefox 54 beta 11.

Comment 18

[Stephen A Pohl [:spohl]]

Thanks for verifying. This was an issue on OSX only. Adding platform info to make this clear.