Closed
Bug 785161
Opened 12 years ago
Closed 12 years ago
Stop uninstalling all installed apps in WebApps tests
Categories
(Core Graveyard :: DOM: Apps, defect)
Core Graveyard
DOM: Apps
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla17
People
(Reporter: mounir, Assigned: mounir)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
justin.lebar+bug
:
review+
|
Details | Diff | Splinter Review |
Fabrice, feel free to take that review from Justin ;)
Attachment #654712 -
Flags: review?(justin.lebar+bug)
|
||
Updated•12 years ago
|
Component: Webapp Runtime → DOM: Apps
Product: Firefox → Core
QA Contact: jsmith
|
Assignee | |
Updated•12 years ago
|
Attachment #654712 -
Flags: review?(fabrice)
|
||
Comment 1•12 years ago
|
||
Comment on attachment 654712 [details] [diff] [review]
Patch
Starting this review now.
Attachment #654712 -
Flags: review?(fabrice)
|
|
||
Updated•12 years ago
|
Summary: Stop uninstall all installed apps in WebApps tests → Stop uninstalling all installed apps in WebApps tests
|
|
||
Comment 2•12 years ago
|
||
Comment on attachment 654712 [details] [diff] [review]
Patch
>+ navigator.mozApps.getInstalled().onsuccess = function() {
>+ is(this.result.length, 0, "should get no installed app");
>+
>+ for (var i=0; i != this.result.length ; ++i) {
>+ var app = this.result[i];
>+ dump("\n\n" + app.manifestURL + "\n\n");
>+ }
This loop is the debugging code you said you removed, right?
Attachment #654712 -
Flags: review?(justin.lebar+bug) → review+
|
|
Assignee | |
Comment 3•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
Version: unspecified → Trunk
|
||
Updated•7 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•