Closed Bug 920478 Opened 8 years ago Closed 8 years ago

[app manager] if the launch path of an app is missing or invalid, we don't show any warning


(DevTools Graveyard :: WebIDE, defect, P1)



(Not tracked)

Firefox 27


(Reporter: paul, Assigned: ochameau)




(1 file, 5 obsolete files)

No description provided.
Assignee: nobody → poirot.alex
This adds a bunch of tests against the launch path in the validator
and comes with unit tests.
Comment on attachment 816491 [details] [diff] [review]
Print errors when something is wrong with app launch path r=paul

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

::: browser/devtools/app-manager/app-validator.js
@@ +54,5 @@
>      this.error(strings.formatStringFromName("validator.invalidManifestURL", [manifestURL], 1));
>      deferred.resolve(null);
>      return deferred.promise;
>    }
> + |= Ci.nsIRequest.LOAD_BYPASS_CACHE | Ci.nsIRequest.INHIBIT_CACHING;

This additional flag was required for tests to behave sanely. So it may fix bug 925219.
Attachment #816491 - Flags: review?(paul)
Comment on attachment 816491 [details] [diff] [review]
Print errors when something is wrong with app launch path r=paul

> + |= Ci.nsIRequest.LOAD_BYPASS_CACHE | Ci.nsIRequest.INHIBIT_CACHING;

This is important for bug 925219 and we need it in aurora.
But we can't land the whole patch in Aurora (strings).

Can you fix that in bug 925219, or have 2 different patches in this bug?
Attachment #816491 - Flags: review?(paul) → review+
Attached patch Factored out the cache fix (obsolete) — Splinter Review
Attachment #816491 - Attachment is obsolete: true
I had to modify bug 926332 per review comments.
The getTestFile method is now getTestFilePath and returns a file path instead of nsIFile.
I also fixed a test failure on windows.

I'm carrying over the review as it is really simple updates, but feel free to take another look at it.
Attachment #816502 - Attachment is obsolete: true
Attachment #817128 - Flags: review+
Attachment #817869 - Flags: review+
Test are green and dependency (bug 926332) is ready to land.
Keywords: checkin-needed
Flags: in-testsuite+
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Sorry, I had to back out this change because it depends on bug 926332 which was backed out:
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 27
Product: Firefox → DevTools
Product: DevTools → DevTools Graveyard
You need to log in before you can comment on or make changes to this bug.