cc: Remove right quad for fullscreen overlays.

cc: Remove right quad for fullscreen overlays.

When a fullscreen quad is promoted to overlay, it should be removed
from quad_list to be GL composited.

This CL fixes a bug where OverlayStrategyFullscreen assumed that the
first quad would always be the one to remove.

BUG=b/29430506
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/7c8c70e76262bd9a2cfc45cd85a10cdeb69a25ea
Cr-Commit-Position: refs/heads/master@{#414759}

[Daniele Castagna, 2016-08-26 01:21:58]

Description was changed from

==========
Remove right quad for fullscreen overlays.

When a fullscreen quad is promoted to overlay, it should be removed
from quad_list to be GL composited.

This CL fixes a bug where OverlayStrategyFullscreen assumed that the
first quad would always be the one to remove.


BUG=b/29430506
==========

to

==========
Remove right quad for fullscreen overlays.

When a fullscreen quad is promoted to overlay, it should be removed
from quad_list to be GL composited.

This CL fixes a bug where OverlayStrategyFullscreen assumed that the
first quad would always be the one to remove.


BUG=b/29430506
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[Daniele Castagna, 2016-08-26 01:22:01]

The CQ bit was checked by dcastagna@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-26 01:22:26]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Daniele Castagna, 2016-08-26 01:22:40]

Description was changed from

==========
Remove right quad for fullscreen overlays.

When a fullscreen quad is promoted to overlay, it should be removed
from quad_list to be GL composited.

This CL fixes a bug where OverlayStrategyFullscreen assumed that the
first quad would always be the one to remove.


BUG=b/29430506
==========

to

==========
cc: Remove right quad for fullscreen overlays.

When a fullscreen quad is promoted to overlay, it should be removed
from quad_list to be GL composited.

This CL fixes a bug where OverlayStrategyFullscreen assumed that the
first quad would always be the one to remove.


BUG=b/29430506
==========

[Daniele Castagna, 2016-08-26 01:22:49]

dcastagna@chromium.org changed reviewers:
+ reveman@chromium.org

[Daniele Castagna, 2016-08-26 01:22:49]

We were still GL compositing even when we were replacing the main plane.
This patch should fix it.

[reveman, 2016-08-26 01:37:37]

lgtm

[reveman, 2016-08-26 03:05:02]

On 2016/08/26 at 01:37:37, reveman wrote:
> lgtm

would be awesome if you could also add a unit test for this

[Daniele Castagna, 2016-08-26 03:16:46]

Description was changed from

==========
cc: Remove right quad for fullscreen overlays.

When a fullscreen quad is promoted to overlay, it should be removed
from quad_list to be GL composited.

This CL fixes a bug where OverlayStrategyFullscreen assumed that the
first quad would always be the one to remove.


BUG=b/29430506
==========

to

==========
cc: Remove right quad for fullscreen overlays.

When a fullscreen quad is promoted to overlay, it should be removed
from quad_list to be GL composited.

This CL fixes a bug where OverlayStrategyFullscreen assumed that the
first quad would always be the one to remove.


BUG=b/29430506
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[Daniele Castagna, 2016-08-26 03:17:04]

On 2016/08/26 at 03:05:02, reveman wrote:
> On 2016/08/26 at 01:37:37, reveman wrote:
> > lgtm
> 
> would be awesome if you could also add a unit test for this

Done.

[reveman, 2016-08-26 16:34:02]

The CQ bit was checked by reveman@chromium.org

[reveman, 2016-08-26 16:34:03]

lgtm

[commit-bot: I haz the power, 2016-08-26 16:34:55]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-26 18:44:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-26 19:16:15]

Description was changed from

==========
cc: Remove right quad for fullscreen overlays.

When a fullscreen quad is promoted to overlay, it should be removed
from quad_list to be GL composited.

This CL fixes a bug where OverlayStrategyFullscreen assumed that the
first quad would always be the one to remove.


BUG=b/29430506
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Remove right quad for fullscreen overlays.

When a fullscreen quad is promoted to overlay, it should be removed
from quad_list to be GL composited.

This CL fixes a bug where OverlayStrategyFullscreen assumed that the
first quad would always be the one to remove.

BUG=b/29430506
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/7c8c70e76262bd9a2cfc45cd85a10cdeb69a25ea
Cr-Commit-Position: refs/heads/master@{#414759}
==========

[commit-bot: I haz the power, 2016-08-26 19:16:16]

Patchset 2 (id:??) landed as
https://crrev.com/7c8c70e76262bd9a2cfc45cd85a10cdeb69a25ea
Cr-Commit-Position: refs/heads/master@{#414759}