Open Bug 1815916 Opened 2 years ago Updated 2 years ago

drag drop double upload

Categories

(Core :: Widget: Gtk, defect, P5)

Firefox 111
defect

Tracking

()

UNCONFIRMED

People

(Reporter: zlice555, Unassigned)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression)

Attachments

(3 files)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/111.0

Steps to reproduce:

  • Linux
  • Fluxbox
  • ROX Filer (gtk2)

Open a private browser.
Drag and drop an image for upload (imgur and other sites)

Actual results:

Get 2 images uploaded

Expected results:

only 1 image

 4:57.52 INFO: Using local file: /tmp/tmpnr2_zg41/fda8e2847c51-shippable--autoland--target.tar.bz2 (downloaded in background)
 4:57.52 INFO: Running autoland build built on 2022-12-16 17:44:07.134000, revision fda8e284
 5:04.00 INFO: Launching /tmp/tmpo2l9tgg4/firefox/firefox
 5:04.00 INFO: Application command: /tmp/tmpo2l9tgg4/firefox/firefox imgur.com -profile /tmp/tmp7u68iua7.mozrunner
 5:04.00 INFO: application_buildid: 20221216171818
 5:04.00 INFO: application_changeset: fda8e2847c515505b7f4fd2b906e768433331127
 5:04.00 INFO: application_name: Firefox
 5:04.00 INFO: application_repository: https://hg.mozilla.org/integration/autoland
 5:04.00 INFO: application_version: 110.0a1
Was this integration build good, bad, or broken? (type 'good', 'bad', 'skip', 'retry', 'back' or 'exit' and press Enter): good
 5:14.69 INFO: Narrowed integration regression window from [35fd7fb9, b9c2ea53] (3 builds) to [fda8e284, b9c2ea53] (2 builds) (~1 steps left)
 5:14.69 INFO: No more integration revisions, bisection finished.
 5:14.69 INFO: Last good revision: fda8e2847c515505b7f4fd2b906e768433331127
 5:14.69 INFO: First bad revision: b9c2ea5329e3e99c805f6fdd9e9e8bd6f41d1f0d
 5:14.69 INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=fda8e2847c515505b7f4fd2b906e768433331127&tochange=b9c2ea5329e3e99c805f6fdd9e9e8bd6f41d1f0d

The Bugbug bot thinks this bug should belong to the 'Core::Widget: Gtk' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: Untriaged → Widget: Gtk
Product: Firefox → Core

Firefox is not mentioned at all in the steps to reproduce. Please provide clear steps to reproduce which leave no room for interpretation.

Flags: needinfo?(zlice555)

Open a private browser - in firefox*

?

Drag and drop an image for upload (imgur and other sites)

Drag and drop an image from where?

Sorry, I didn't think it was that unclear.

Here is a video. Let me know if you have any other questions.

Flags: needinfo?(zlice555)

For clarification - the left is "ROX filer" file-manager, the right is Firefox, the Window Manager is Fluxbox.

Blocks: linuxdad
Priority: -- → P3

:evilpie, since you are the author of the regressor, bug 1732528, could you take a look?

For more information, please visit auto_nag documentation.

Flags: needinfo?(evilpies)
Keywords: regression

Unlike most other file manager that I am aware of, "ROX filer" seems to provide both the file URI via "text/uri-list" and the actual image data via "image/png". Other file managers just provide the file: URI of the dragged file.

I would not necessarily consider this a Firefox bug.

Flags: needinfo?(evilpies)
Priority: P3 → P5
Attached patch dnd_noimg.patchSplinter Review

Thanks, was able to patch ROX so it doesn't dupe. However ROX has been dead for some time now (sourceforge, 2005) so there will not be a patch on that end.

(In reply to Tom S [:evilpie] from comment #9)

Unlike most other file manager that I am aware of, "ROX filer" seems to provide both the file URI via "text/uri-list" and the actual image data via "image/png". Other file managers just provide the file: URI of the dragged file.

I would not necessarily consider this a Firefox bug.

I wonder why do we do the drag twice then? Is that a a bug on the webpage which fetches both attachments?

It happens on more than imgur. And not in chromium, and obviously not before that commit.

Attached patch dnd_noimg2.patchSplinter Review

Update to that ROX patch. Other one broke the ROX pinboard when dragging items which I don't do very often. Minor testing seems to work :)

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: