Closed Bug 1155245 Opened 7 years ago Closed 7 years ago

The app status is not set correctly for hosted certified apps


(Core Graveyard :: DOM: Apps, defect)

Windows 8.1
Not set


(firefox40 fixed)

Tracking Status
firefox40 --- fixed


(Reporter: amac, Assigned: amac)




(1 file, 2 obsolete files)

1. Enable developer mode (set 
pref("dom.apps.developer_mode", true);)
2. Boot and install any hosted app that is certified
3. Check the app status at /data/local/webapps/webapps.json


The appStatus should be 3 (Certified)


The appStatus is 1. And if the app uses web components on any other thing that explicitly check for the status, it doesn't work.
Assignee: nobody → amac.bug
Attachment #8593435 - Flags: review?(fabrice)
Comment on attachment 8593435 [details] [diff] [review]
V1. Set the correct app status for hosted apps on developer mode

Review of attachment 8593435 [details] [diff] [review]:

::: dom/apps/Webapps.jsm
@@ +3107,5 @@
>      // don't update the permissions yet.
>      if (!aData.isPackage) {
>        if (supportUseCurrentProfile()) {
> +        // Until now we were not setting correctly the appStatus for hosted apps, and that didn't matter because
> +        // hosted apps couldn't be other than installed anyway. That's not true anymore...

Remove this comment. Historians will look at the blame & bugzilla.
Attachment #8593435 - Flags: review?(fabrice) → review+
Attachment #8593435 - Attachment is obsolete: true
Test run looks (mostly) green, requesting checkin.
Keywords: checkin-needed
Your B2G/Mulet test failures look legit...
213 INFO TEST-UNEXPECTED-FAIL | dom/apps/tests/test_app_enabled.html | Test timed out. - expected PASS
Keywords: checkin-needed
Actually this kinda looks a lot like Bug 1115788. In fact, all of those tests seem to have a lot of oranges judging by the suggestions... The code added here only makes a single change if the developer_mode preference is true. And the only test where that's true is on test_install_dev_mode.html. So for all the rest, the code that's being executed is the same with and without this patch.
Keywords: checkin-needed
The failures aren't happening in production, especially at the high frequency of your test run. Please post a green Try run to verify that your patch isn't at fault rather than asking me to accept it on faith and risk a tree closure if wrong.
Keywords: checkin-needed
And it's green... rerequesting checkin
Keywords: checkin-needed
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.