Closed Bug 1638618 Opened 4 years ago Closed 4 years ago

Theme changes (such as LookAndFeelCache) aren't mirrored to preallocated processes

Categories

(Core :: Widget, defect)

defect

Tracking

()

RESOLVED DUPLICATE of bug 1156934

People

(Reporter: jesup, Unassigned)

References

Details

(Whiteboard: [fission])

Per bug 1156934, Theme changes (including LookAndFeelCache, such as the prefers-reduced-motion values) are mirrored to all active PresContexts. However, a preallocated process doesn't have a PresContext, so it doesn't get notified of the change. If we start the process, it gets the current state of the LookAndFeelCache. If the context changes while it's still in the preallocation state, it doesn't get updated. When it is finally moved to be an active process, it will have the incorrect state for the LookAndFeel cache. This affects e10s and fission preallocated processes (with bug 1602757 to enable them in fission), though it's more likely to be noticed in fission.

This would also affect e10s processes which are cached for reuse via Provide() in ContentParent.cpp, and any future short-term caching of Fission webIsolated processes.

Solving bug 1156934 would solve this bug.

Fission Milestone: --- → ?

Tracking for Fission M7 Beta

Jim will ask gcp.

Fission Milestone: ? → M7

The severity field is not set for this bug.
:jimm, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(jmathies)

This was fixed by bug 1156934.

Status: NEW → RESOLVED
Closed: 4 years ago
Flags: needinfo?(jmathies)
Resolution: --- → DUPLICATE

Clearing Fission Milestone for bugs resolved as duplicates. We don't need to track duplicates.

Fission Milestone: M7 → ---
You need to log in before you can comment on or make changes to this bug.