Closed Bug 1250603 Opened 4 years ago Closed 4 years ago

update "WebappRT" comments or remove app restrictions on components

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(firefox48 fixed)

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: myk, Assigned: myk)

References

Details

Attachments

(1 file)

After I remove the web runtime in bug 1238079, we won't need to restrict various components to a specific list of apps because of "WebappRT" (i.e. the desktop web runtime):

https://dxr.mozilla.org/mozilla-central/search?q=WebappRT+doesn%27t+need+these+instructions&redirect=true&case=false

However, in my testing, I've found that we still need to do so in at least some cases for other reasons, as a variety of tests fail otherwise with messages like:

> Assertion count 1 is greater than expected range 0-0 assertions.

In logs for debug builds, I also see many instances of warnings like:

> 03:17:17     INFO -  [Child 708] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file c:/builds/moz2_slave/try-w64-d-00000000000000000000/build/src/toolkit/xre/nsXREDirProvider.cpp, line 1408
> 03:17:17     INFO -  [Child 708] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file c:/builds/moz2_slave/try-w64-d-00000000000000000000/build/src/xpcom/base/nsSystemInfo.cpp, line 112
> 03:17:17     INFO -  [Child 708] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80040111: file c:/builds/moz2_slave/try-w64-d-00000000000000000000/build/src/netwerk/protocol/res/SubstitutingProtocolHandler.cpp, line 244
> 03:17:17     INFO -  [Child 708] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80040111: file c:/builds/moz2_slave/try-w64-d-00000000000000000000/build/src/netwerk/protocol/res/SubstitutingProtocolHandler.cpp, line 244
> 03:17:17     INFO -  [Child 708] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80040111: file c:/builds/moz2_slave/try-w64-d-00000000000000000000/build/src/netwerk/base/nsIOService.cpp, line 779
> 03:17:17     INFO -  [Child 708] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80040111: file c:\builds\moz2_slave\try-w64-d-00000000000000000000\build\src\netwerk\base\nsNetUtil.inl, line 180
> 03:17:17     INFO -  [Child 708] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80040111: file c:/builds/moz2_slave/try-w64-d-00000000000000000000/build/src/js/xpconnect/loader/mozJSComponentLoader.cpp, line 251
> 03:17:17     INFO -  [Child 708] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80040111: file c:/builds/moz2_slave/try-w64-d-00000000000000000000/build/src/js/xpconnect/loader/mozJSComponentLoader.cpp, line 1135

Followed by errors like:

> 03:17:17     INFO -  JavaScript error: jar:file:///C:/slave/test/build/application/firefox/omni.ja!/components/Weave.js, line 13: NS_ERROR_NOT_AVAILABLE: Component returned failure code: 0x80040111 (NS_ERROR_NOT_AVAILABLE) [nsIXPCComponents_Utils.import]
> 03:17:17     INFO -  [Child 708] WARNING: Cannot create startup observer : service,@mozilla.org/weave/service;1: file c:/builds/moz2_slave/try-w64-d-00000000000000000000/build/src/embedding/components/appstartup/nsAppStartupNotifier.cpp, line 81

And:

> 13:19:08     INFO -  JavaScript error: jar:file:///builds/slave/test/build/application/firefox/browser/omni.ja!/components/nsBrowserGlue.js, line 3222: NS_ERROR_XPC_GS_RETURNED_FAILURE: Component returned failure code: 0x80570016 (NS_ERROR_XPC_GS_RETURNED_FAILURE) [nsIJSCID.getService]
> 13:19:08     INFO -  [Child 2477] WARNING: Cannot create startup observer : service,@mozilla.org/browser/browserglue;1: file /builds/slave/try-l64-d-00000000000000000000/build/src/embedding/components/appstartup/nsAppStartupNotifier.cpp, line 81

So there's clearly another reason to restrict one or more of these components to a specified list of apps, and we should continue to do so.  But we should update the comment to reflect the current reason (or at least stop crediting WebappRT).

We might also be able to remove the restrictions for some of these components.  In my tests so far, removing them from browser/components/BrowserComponents.manifest and browser/components/feeds/BrowserFeeds.manifest seems to cause test failures, while removing them from browser/components/sessionstore/nsSessionStore.manifest doesn't seem to cause problems.  Unsure about services/sync/SyncComponents.manifest.
(In reply to Myk Melez [:myk] [@mykmelez] from comment #0)
> Unsure about services/sync/SyncComponents.manifest.

Update: removing these restrictions doesn't appear to affect services/sync/SyncComponents.manifest, so I'm removing them in bug 1250352.
(In reply to Myk Melez [:myk] [@mykmelez] from comment #1)
> (In reply to Myk Melez [:myk] [@mykmelez] from comment #0)
> > Unsure about services/sync/SyncComponents.manifest.
> 
> Update: removing these restrictions doesn't appear to affect
> services/sync/SyncComponents.manifest, so I'm removing them in bug 1250352.

Famous last words.  I'm reverting the removal over in bug 1264931, where I also identify xpcshell as a culprit.  So here's a patch that updates the rest of the comments that reference WebappRT so they pin the blame on the appropriate donkey.
Assignee: nobody → myk
Status: NEW → ASSIGNED
Attachment #8741955 - Flags: review?(gps)
Attachment #8741955 - Flags: review?(gps) → review+
https://hg.mozilla.org/mozilla-central/rev/02101aad5897
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Component: Build Config → General
Product: Firefox → Firefox Build System
Target Milestone: Firefox 48 → mozilla48
You need to log in before you can comment on or make changes to this bug.