The default bug view has changed. See this FAQ.

Stop uninstalling all installed apps in WebApps tests

RESOLVED FIXED in mozilla17

Status

()

Core
DOM: Apps
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: mounir, Assigned: mounir)

Tracking

Trunk
mozilla17
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

14.43 KB, patch
Justin Lebar (not reading bugmail)
: review+
Details | Diff | Splinter Review
(Assignee)

Description

5 years ago
Created attachment 654712 [details] [diff] [review]
Patch

Fabrice, feel free to take that review from Justin ;)
Attachment #654712 - Flags: review?(justin.lebar+bug)

Updated

5 years ago
Component: Webapp Runtime → DOM: Apps
Product: Firefox → Core
QA Contact: jsmith
(Assignee)

Updated

5 years ago
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+
(Assignee)

Comment 3

5 years ago
https://hg.mozilla.org/mozilla-central/rev/ebe6e3330f63
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
Version: unspecified → Trunk
You need to log in before you can comment on or make changes to this bug.