Closed
Bug 600052
Opened 14 years ago
Closed 14 years ago
Add function that waits for a download to complete in the new Add-ons API
Categories
(Mozilla QA Graveyard :: Mozmill Tests, defect)
Mozilla QA Graveyard
Mozmill Tests
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: aaronmt, Assigned: whimboo)
References
Details
(Whiteboard: [lib])
Attachments
(2 files, 1 obsolete file)
|
(deleted),
patch
|
u279076
:
review+
|
Details | Diff | Splinter Review |
|
(deleted),
patch
|
u279076
:
review+
|
Details | Diff | Splinter Review |
Adding a function that waits for a download to complete in the new Add-ons API would make things much easier than having to use a timeout timer in tests. This bug might depend on bug 600051
|
Assignee | |
Updated•14 years ago
|
|
|
Assignee | |
Comment 2•14 years ago
|
||
Lets wait until bug 569813 has been checked in. Otherwise it's too much hassle to review.
Please add the tests to the blocking list which make use of this progress meter.
|
|
Assignee | |
Updated•14 years ago
|
Summary: [shared module] Add function that waits for a download to complete in the new Add-ons API → Add function that waits for a download to complete in the new Add-ons API
|
|
Assignee | |
Comment 3•14 years ago
|
||
So I had some trouble to get the install button and the install status elements. As given by Dave, those are located below another binding. That's why we have to search twice. It needed a little refactoring of the collector class too, which I will have to backport to older branches. Otherwise you will now have an installAddon, undo, and waitForDownloaded helper function.
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Attachment #480130 -
Flags: review?(anthony.s.hughes)
|
|
Assignee | |
Comment 4•14 years ago
|
||
Attachment #480130 -
Attachment is obsolete: true
Attachment #480131 -
Flags: review?(anthony.s.hughes)
Attachment #480130 -
Flags: review?(anthony.s.hughes)
Attachment #480131 -
Flags: review?(anthony.s.hughes) → review+
|
|
Assignee | |
Comment 5•14 years ago
|
||
Landed on default as:
http://hg.mozilla.org/qa/mozmill-tests/rev/62551d70805a
I will leave it open until we have landed the backport patch.
|
|
Assignee | |
Comment 6•14 years ago
|
||
This patch simply backports the changes for the DOMUtilsAPI to the older branches.
Attachment #480267 -
Flags: review?(anthony.s.hughes)
|
|
Assignee | |
Updated•14 years ago
|
Attachment #480131 -
Attachment description: Patch v1.1 → Patch v1.1 [checked-in]
Attachment #480267 -
Flags: review?(anthony.s.hughes) → review+
|
|
Assignee | |
Comment 7•14 years ago
|
||
Backports landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/7498938cb9cd (1.9.2)
http://hg.mozilla.org/qa/mozmill-tests/rev/7dee27d91bbc (1.9.1)
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 8•14 years ago
|
||
Move of Mozmill Test related project bugs to newly created components. You can
filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.
Product: Testing → Mozilla QA
Version: Trunk → unspecified
|
|
Assignee | |
Updated•13 years ago
|
Component: Mozmill Tests → Mozmill Shared Modules
|
|
Assignee | |
Updated•13 years ago
|
Component: Mozmill Shared Modules → Mozmill Tests
|
|
Assignee | |
Updated•13 years ago
|
Whiteboard: [shared module] → [lib]
|
||
Updated•5 years ago
|
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•