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)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla17

People

(Reporter: mounir, Assigned: mounir)

References

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
Fabrice, feel free to take that review from Justin ;)
Attachment #654712 - Flags: review?(justin.lebar+bug)
Component: Webapp Runtime → DOM: Apps
Product: Firefox → Core
QA Contact: jsmith
Attachment #654712 - Flags: review?(fabrice)
Comment on attachment 654712 [details] [diff] [review]
Patch

Starting this review now.
Attachment #654712 - Flags: review?(fabrice)
Summary: Stop uninstall all installed apps in WebApps tests → Stop uninstalling all installed apps in WebApps tests
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+
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
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: