Log on app install start and finish

RESOLVED FIXED in Firefox 27

Status

defect
P2
normal
RESOLVED FIXED
6 years ago
Last year

People

(Reporter: jryans, Assigned: jryans)

Tracking

Trunk
Firefox 27

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Currently there's no indication that an app install is going on, and the process can take a while, leaving it unclear what is happening.
Attachment #810800 - Flags: review?(poirot.alex)
Comment on attachment 810800 [details] [diff] [review]
Log during app install in App Manager

Review of attachment 810800 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/devtools/app-manager/content/projects.js
@@ +231,5 @@
>      }
> +    return installPromise.then(() => {
> +      this.connection.log("Install completed.");
> +    }, () => {
> +      this.connection.log("Install failed.");

By doing that, you are eating the error, so that we no longer get the alert on error.
Attachment #810800 - Flags: review?(poirot.alex)
(In reply to Alexandre Poirot (:ochameau) from comment #3)
> By doing that, you are eating the error, so that we no longer get the alert
> on error.

Ah, good catch! :)
Attachment #810800 - Attachment is obsolete: true
Attachment #813318 - Flags: review?(poirot.alex)
Comment on attachment 813318 [details] [diff] [review]
Log during app install in App Manager v2

Review of attachment 813318 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good.
Attachment #813318 - Flags: review?(poirot.alex) → review+
Someone broke BC tests, trying again: https://tbpl.mozilla.org/?tree=Try&rev=4892daf969c1
Keywords: checkin-needed
Whiteboard: [land-in-fx-team]
https://hg.mozilla.org/integration/fx-team/rev/64479cdfb3a6
Keywords: checkin-needed
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/64479cdfb3a6
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 27
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.