Convert various VarCache prefs to static prefs
Categories
(Firefox :: General, task, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: n.nethercote, Assigned: n.nethercote)
References
Details
Attachments
(23 files)
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review |
Specifically, ones where a VarCache is created when no pref value exists. These are blocking me from completing bug 1569526.
![]() |
Assignee | |
Comment 1•6 years ago
|
||
![]() |
Assignee | |
Comment 2•6 years ago
|
||
It's unused.
Depends on D40001
![]() |
Assignee | |
Comment 3•6 years ago
|
||
Depends on D40002
![]() |
Assignee | |
Comment 4•6 years ago
|
||
Depends on D40003
![]() |
Assignee | |
Comment 5•6 years ago
|
||
Depends on D40004
![]() |
Assignee | |
Comment 6•6 years ago
|
||
Depends on D40005
Updated•6 years ago
|
Updated•6 years ago
|
![]() |
Assignee | |
Comment 7•6 years ago
|
||
Depends on D40006
![]() |
Assignee | |
Comment 8•6 years ago
|
||
Depends on D40153
![]() |
Assignee | |
Comment 9•6 years ago
|
||
Depends on D40154
![]() |
Assignee | |
Comment 10•6 years ago
|
||
Depends on D40155
![]() |
Assignee | |
Comment 11•6 years ago
|
||
Depends on D40156
![]() |
Assignee | |
Comment 12•6 years ago
|
||
Depends on D40157
![]() |
Assignee | |
Comment 13•6 years ago
|
||
Depends on D40158
![]() |
Assignee | |
Comment 14•6 years ago
|
||
Depends on D40159
![]() |
Assignee | |
Comment 15•6 years ago
|
||
Depends on D40160
![]() |
Assignee | |
Updated•6 years ago
|
Comment 16•6 years ago
|
||
![]() |
Assignee | |
Comment 17•6 years ago
|
||
Depends on D40162
![]() |
Assignee | |
Comment 18•6 years ago
|
||
Depends on D40338
![]() |
Assignee | |
Comment 19•6 years ago
|
||
Also remove HTMLVideoElement::InitStatics(), which is no longer needed.
Depends on D40339
![]() |
Assignee | |
Comment 20•6 years ago
|
||
Depends on D40340
![]() |
Assignee | |
Comment 21•6 years ago
|
||
It's never modified anywhere, and so there's no real point in having it.
The patch also removes the now-empty nsCORSListenerProxy::Startup().
Depends on D40341
![]() |
Assignee | |
Comment 22•6 years ago
|
||
As a result, it would become visible in about:config. So the patch also
modifies about:config so that doesn't happen.
Depends on D40342
Comment 23•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/62db71f2544d
https://hg.mozilla.org/mozilla-central/rev/1ad56915268a
https://hg.mozilla.org/mozilla-central/rev/94aaa71664cb
https://hg.mozilla.org/mozilla-central/rev/6774e173a05e
https://hg.mozilla.org/mozilla-central/rev/32d5b0eb343a
https://hg.mozilla.org/mozilla-central/rev/3c246eb3943a
Updated•6 years ago
|
Comment 24•6 years ago
|
||
Comment 25•6 years ago
|
||
Comment 26•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ff00fe45c3d2
https://hg.mozilla.org/mozilla-central/rev/a035512b4f44
https://hg.mozilla.org/mozilla-central/rev/0502c02439eb
https://hg.mozilla.org/mozilla-central/rev/09628a9faf59
https://hg.mozilla.org/mozilla-central/rev/ce74181517ed
https://hg.mozilla.org/mozilla-central/rev/0e363460f740
https://hg.mozilla.org/mozilla-central/rev/0ecc35590409
https://hg.mozilla.org/mozilla-central/rev/57c05745399b
https://hg.mozilla.org/mozilla-central/rev/533cf0ae7f5e
https://hg.mozilla.org/mozilla-central/rev/ae1e48d740a7
https://hg.mozilla.org/mozilla-central/rev/8e72c66c3076
https://hg.mozilla.org/mozilla-central/rev/bb1723162580
Updated•6 years ago
|
![]() |
Assignee | |
Comment 27•6 years ago
|
||
Fix overly-long lines, add '.' to the end of sentences, put things in a more
sensible order, etc. (No functional changes.)
Depends on D40343
![]() |
Assignee | |
Comment 28•6 years ago
|
||
The patch also removes RFP_TIMER_VALUE_DEFAULT
and RFP_JITTER_VALUE_DEFAULT
because they are unused.
Depends on D40903
Updated•6 years ago
|
Updated•6 years ago
|
Updated•6 years ago
|
![]() |
Assignee | |
Updated•6 years ago
|
Comment 29•6 years ago
|
||
Comment 30•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c76bbd0f9a43
https://hg.mozilla.org/mozilla-central/rev/df1b7d72c13e
https://hg.mozilla.org/mozilla-central/rev/3fcd5e2f0344
https://hg.mozilla.org/mozilla-central/rev/bc6225570e8b
https://hg.mozilla.org/mozilla-central/rev/6f0426e233d6
Description
•