Closed
Bug 1250603
Opened 7 years ago
Closed 7 years ago
update "WebappRT" comments or remove app restrictions on components
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox48 fixed)
RESOLVED
FIXED
mozilla48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: myk, Assigned: myk)
References
Details
Attachments
(1 file)
6.41 KB,
patch
|
gps
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•7 years ago
|
||
(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.
Assignee | ||
Comment 2•7 years ago
|
||
(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.
Updated•7 years ago
|
Attachment #8741955 -
Flags: review?(gps) → review+
Comment 4•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/02101aad5897
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Updated•5 years ago
|
Component: Build Config → General
Product: Firefox → Firefox Build System
Updated•5 years ago
|
Target Milestone: Firefox 48 → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•