Support dropping xpi/jar files on the add-ons manager

VERIFIED FIXED in mozilla2.0b1

Status

()

Toolkit
Add-ons Manager
P1
normal
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: mossop, Assigned: mossop)

Tracking

Trunk
mozilla2.0b1
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +
in-litmus +

Firefox Tracking Flags

(blocking2.0 final+)

Details

(Whiteboard: [in-litmus-bug-week][rewrite])

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
In Firefox this just magically works because we are displaying in a content browser, but for apps that want to display the add-ons manager in its own window it won't. The solution is to add drop support for files to the add-ons manager UI.
Most of themes at AMO are packaged as .jar files, so the add-ons manager should also support dropping .jar files.
Summary: Support dropping xpi files on the add-ons manager → Support dropping xpi/jar files on the add-ons manager
(Assignee)

Updated

7 years ago
blocking2.0: --- → beta1+
(Assignee)

Updated

7 years ago
Blocks: 567640

Updated

7 years ago
Duplicate of this bug: 569151
(Assignee)

Updated

7 years ago
blocking2.0: beta1+ → final+
(Assignee)

Comment 3

7 years ago
Patch in progress.
Assignee: nobody → dtownsend
(Assignee)

Comment 4

7 years ago
I'm going to be able to automate some of the testing here, but we can't properly simulate a drag of files from outside the app in the test suites so we should make sure there is litmus coverage here.
Flags: in-testsuite?
Flags: in-litmus?
(Assignee)

Comment 5

7 years ago
Created attachment 451365 [details] [diff] [review]
patch rev 1

This adds a drop handler to the add-ons manager window and verifies that dropping files and urls on it will start installs.
Attachment #451365 - Flags: review?(bmcbride)
(Assignee)

Updated

7 years ago
Status: NEW → ASSIGNED
Comment on attachment 451365 [details] [diff] [review]
patch rev 1

I'm unfamiliar with the text/uri-list mimetype - it *sounds* like each item of this mimetype would be a list of URIs, but the code assumes its one URI per item. I'm guessing its converted to multiple items before we get to it. But if not, you'll need to handle that.

Wish we could avoid the install confirmation dialog, since its an explicit action. But I guess that needs to stay :\
Attachment #451365 - Flags: review?(bmcbride) → review+
(Assignee)

Comment 7

7 years ago
(In reply to comment #6)
> (From update of attachment 451365 [details] [diff] [review])
> I'm unfamiliar with the text/uri-list mimetype - it *sounds* like each item of
> this mimetype would be a list of URIs, but the code assumes its one URI per
> item. I'm guessing its converted to multiple items before we get to it. But if
> not, you'll need to handle that.

Should have mentioned, most of the handling of the different types is just copied from the old code and we never heard any complaints about it so I'm guessing it is correct enough.

> Wish we could avoid the install confirmation dialog, since its an explicit
> action. But I guess that needs to stay :\

I think we still need the conformation, just so users know what they're getting from urls etc., however we have a bug somewhere on ditching the countdown timer in that case which is probably a good idea.
(Assignee)

Comment 8

7 years ago
Landed: http://hg.mozilla.org/mozilla-central/rev/eb16fbc5530a.

As mentioned this is testing only as far as injecting pretend drag events to the UI. We should have some real manual testing of this too.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Flags: in-testsuite? → in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a6
Verified fixed on trunk with builds on all platforms like Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.3a6pre) Gecko/20100624 Minefield/3.7a6pre
Status: RESOLVED → VERIFIED
Test has been updated on Litmus:
https://litmus.mozilla.org/show_test.cgi?id=10306

Dave, do we have plans to allow D&D of .jar files into the browser? Right now that only works inside the Add-ons Manager.
Flags: in-litmus? → in-litmus+
Whiteboard: [rewrite] → [in-litmus-bug-week][rewrite]
(Assignee)

Comment 11

7 years ago
(In reply to comment #10)
> Test has been updated on Litmus:
> https://litmus.mozilla.org/show_test.cgi?id=10306
> 
> Dave, do we have plans to allow D&D of .jar files into the browser? Right now
> that only works inside the Add-ons Manager.

We never have before because jar files can be all sorts of things.
You need to log in before you can comment on or make changes to this bug.