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)
14.43 KB,
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
|
||
https://hg.mozilla.org/mozilla-central/rev/ebe6e3330f63
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
•