Closed Bug 1296021 Opened 3 years ago Closed 3 years ago

Fix static CompositorBridgeParent methods in GPUProcessManager

Categories

(Core :: Graphics, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: rhunt, Assigned: rhunt)

Details

Attachments

(1 file)

Right now there are a few calls to static methods on CompositorBridgeParent in GPUProcessManager. These won't work with a GPU process, and will need to be fixed somehow.

http://searchfox.org/mozilla-central/source/gfx/ipc/GPUProcessManager.cpp#505
Comment on attachment 8782035 [details]
Bug 1296021 - Forward DeallocateLayerTreeId to GPU process if there is one.

https://reviewboard.mozilla.org/r/72318/#review69934

SwapLayerTreeObservers is removed in bug 1279086. Unless you need that part for testing right now, I'd just skip it and wait for bug 1279086 to land.
Attachment #8782035 - Flags: review?(dvander) → review+
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/535bd44df96a
Forward DeallocateLayerTreeId to GPU process if there is one. r=dvander
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/535bd44df96a
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.