Move most things out of BrowserUtils.jsm
Categories
(Toolkit :: General, task, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox87 | --- | fixed |
People
(Reporter: kmag, Assigned: kmag)
References
(Blocks 1 open bug)
Details
Attachments
(12 files)
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review |
BrowserUtils.jsm is loaded in all processes, but most of it is only needed in the parent. It should be stripped down as much as possible.
Assignee | ||
Comment 1•4 years ago
|
||
Note that PromptUtils.objectToPropBag and the new propBagToObject function in
prompt_common.js do not do the recursive conversion that the BrowserUtils
versions attempted, but that code was completely broken, and would have failed
if it encountered a value which triggered recursion.
Assignee | ||
Comment 2•4 years ago
|
||
Assignee | ||
Comment 3•4 years ago
|
||
Assignee | ||
Comment 4•4 years ago
|
||
Assignee | ||
Comment 5•4 years ago
|
||
Assignee | ||
Comment 6•4 years ago
|
||
Assignee | ||
Comment 7•4 years ago
|
||
Assignee | ||
Comment 8•4 years ago
|
||
It's only used by the finder code, and can be much simpler.
Assignee | ||
Comment 9•4 years ago
|
||
It won't ever be used in most content processes.
Assignee | ||
Comment 10•4 years ago
|
||
They won't be used in most processes.
Assignee | ||
Comment 11•4 years ago
|
||
It will mostly not be used. This patch just duplicates it in the two modules
that use it since it's not long for this world, anyway.
Assignee | ||
Comment 12•4 years ago
|
||
This moves the exception prettifying to the script security manager for all JS
callers, where it is much cheaper and more consistently applied.
Updated•4 years ago
|
Updated•4 years ago
|
Comment 13•4 years ago
|
||
Comment 14•4 years ago
|
||
Comment 15•4 years ago
|
||
Backed out for causing failures on browser_fission_maxOrigins.js.
Backout link: https://hg.mozilla.org/integration/autoland/rev/c9058c70634bb051c37a87eaa03c91f2b38aee97
Failure log: https://treeherder.mozilla.org/logviewer?job_id=328048144&repo=autoland&lineNumber=5295
Comment 16•4 years ago
|
||
Assignee | ||
Updated•4 years ago
|
Comment 17•4 years ago
|
||
Comment 18•4 years ago
|
||
Backed out 12 changesets (Bug 1685801) for causing bc failures in browser_ctrlTab.js
Backout link: https://hg.mozilla.org/integration/autoland/rev/60f948492e6c
Failure log: https://treeherder.mozilla.org/logviewer?job_id=328116517&repo=autoland&lineNumber=14475
Assignee | ||
Updated•4 years ago
|
Comment 19•4 years ago
|
||
Comment 20•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/48e72e2fe184
https://hg.mozilla.org/mozilla-central/rev/85147dd0a1cc
https://hg.mozilla.org/mozilla-central/rev/d1e122651111
https://hg.mozilla.org/mozilla-central/rev/2e0fe366d6c0
https://hg.mozilla.org/mozilla-central/rev/e75da77e19ac
https://hg.mozilla.org/mozilla-central/rev/11c9171434b7
https://hg.mozilla.org/mozilla-central/rev/7eca8293846d
https://hg.mozilla.org/mozilla-central/rev/3f912d15c129
https://hg.mozilla.org/mozilla-central/rev/a0572c936024
https://hg.mozilla.org/mozilla-central/rev/7c1f3e4fbffd
https://hg.mozilla.org/mozilla-central/rev/4ba93ae387f2
https://hg.mozilla.org/mozilla-central/rev/f5d08efaa765
Description
•