1273017 - Move CompositorThreadHolder into its own file

Status: RESOLVED FIXED

(Core :: Graphics: Layers, defect)

Description

[David Anderson [:dvander] - inactive, e-mail if emergency]

A bunch of files include CompositorBridgeParent just to access the compositor message loop. To make it less tempting to use this class for random stuff, I'd like to split CompositorThreadHolder into its own file and move the message loop access there.

Comment 1

[David Anderson [:dvander] - inactive, e-mail if emergency]

Attachment: part 1, separate file

Move CompositorThreadHolder into a separate file, and fix up callers to GetCompositorThreadHolder.

This (and the following patches) should all preserve existing behavior.

Comment 2

[David Anderson [:dvander] - inactive, e-mail if emergency]

Attachment: part 2, move CompositorLoop()

Comment 3

[David Anderson [:dvander] - inactive, e-mail if emergency]

Attachment: part 3, move IsInCompositorThread()

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f18799f8f8c6
https://hg.mozilla.org/integration/mozilla-inbound/rev/79da877c9d70
https://hg.mozilla.org/integration/mozilla-inbound/rev/d869cb9a9e09

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/f18799f8f8c6
https://hg.mozilla.org/mozilla-central/rev/79da877c9d70
https://hg.mozilla.org/mozilla-central/rev/d869cb9a9e09