1444275 - Remove the *InAnyProcess() functions

Status: RESOLVED FIXED

(Core :: Preferences: Backend, enhancement)

Description

[Nicholas Nethercote [inactive]]

Prefs are only modified in the parent process... except for a few exceptions that I want to get rid of.

Comment 1

[Nicholas Nethercote [inactive]]

Attachment: Bug 1444275 - Don't set the telemetry pref in content processes.

It'll be set via the normal parent-to-child pref setting process.

Review commit: https://reviewboard.mozilla.org/r/227276/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/227276/

Comment 2

[Nicholas Nethercote [inactive]]

Attachment: Bug 1444275 - Remove the *InAnyProcess() functions.

All pref-modifying operations now only occur in the parent process. Hooray!

Review commit: https://reviewboard.mozilla.org/r/227278/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/227278/

Comment 3

[Mike Hommey [:glandium]]

Comment on attachment 8958347 [details]
Bug 1444275 - Don't set the telemetry pref in content processes.

https://reviewboard.mozilla.org/r/227276/#review234944

Comment 4

[Mike Hommey [:glandium]]

Comment on attachment 8958348 [details]
Bug 1444275 - Remove the *InAnyProcess() functions.

https://reviewboard.mozilla.org/r/227278/#review234948

Comment 5

[Nicholas Nethercote [inactive]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c35555a46b2cbb2a6bf4eaff05af71707024fa4c
Bug 1444275 - Don't set the telemetry pref in content processes. r=glandium

https://hg.mozilla.org/integration/mozilla-inbound/rev/93d47457969380040f9617fc9f067171d1acd563
Bug 1444275 - Remove the *InAnyProcess() functions. r=glandium

Comment 6

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/c35555a46b2c
https://hg.mozilla.org/mozilla-central/rev/93d474579693

Comment 7

[Nicholas Nethercote [inactive]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b6e90fe569620f82e5b519c144162ec977734d10
Bug 1444275 - Reinstate an assertion. r=me

Comment 8

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/b6e90fe56962