1314189 - Only create a RemoteVideoDecoder if the current LayerManager is using the GPU process

Status: RESOLVED FIXED

(Core :: Graphics: Layers, defect)

Description

[Matt Woodrow (:mattwoodrow)]

Attachment: right-knows-compositor-type

Comment 1

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

Comment on attachment 8806190 [details] [diff] [review]
right-knows-compositor-type

Review of attachment 8806190 [details] [diff] [review]:
-----------------------------------------------------------------

::: gfx/ipc/GfxMessageUtils.h
@@ +900,5 @@
>  
>    static bool Read(const Message* aMsg, PickleIterator* aIter, paramType* aResult)
>    {
>      bool result = ReadParam(aMsg, aIter, &aResult->mParentBackend) &&
> +                  ReadParam(aMsg, aIter, &aResult->mParentProcessId) &&

Pre-existing, but this really should have been called "mParentProcessType".

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/070a68ae4962
Only create a RemoteVideoDecoder if the current LayerManager is using the GPU process. r=dvander

Comment 3

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/070a68ae4962