Change ozone async swap to fix overlay scheduling

Change ozone async swap to fix overlay scheduling

We keep a list of all pending frames insize the GLSurface. Each frame contains
a list of overlays. In the future it can contain overlay sync point info too.
When ready, we set the ready state and submit all completed frames in order

BUG=470185
TEST=link and rockchip

Committed: https://crrev.com/77f08ab335da814a065dce6859a674b29e9c1294
Cr-Commit-Position: refs/heads/master@{#324053}

[achaulk, 2015-04-02 17:40:34]

achaulk@chromium.org changed reviewers:
+ alexst@chromium.org

[alexst (slow to review), 2015-04-02 19:48:53]

https://codereview.chromium.org/1054073002/diff/1/ui/gl/gl_surface_ozone.cc
File ui/gl/gl_surface_ozone.cc (right):

https://codereview.chromium.org/1054073002/diff/1/ui/gl/gl_surface_ozone.cc#n...
ui/gl/gl_surface_ozone.cc:212: }
I think you are missing:

unsubmitted_frames_.back()->ScheduleOverlayPlanes(widget_);
unsubmitted_frames_.back()->overlays.clear();

On intel there is no sync ext, but we need to schedule overlays anyway.

[achaulk, 2015-04-02 19:53:16]

Ah yes, missed that path

[alexst (slow to review), 2015-04-02 20:19:29]

lgtm

TEST=link and rockchip

[achaulk, 2015-04-02 20:20:05]

The CQ bit was checked by achaulk@chromium.org

[commit-bot: I haz the power, 2015-04-02 20:22:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1054073002/20001

[commit-bot: I haz the power, 2015-04-02 21:16:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-02 21:16:58]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[dbehr, 2015-04-02 21:37:33]

dbehr@chromium.org changed reviewers:
+ dbehr@chromium.org

[dbehr, 2015-04-02 21:37:33]

Wouldn't it be better to add overlays.clear() to
PendingFrame::ScheduleOverlayPlanes() instead of calling it afterwards every
time?

[achaulk, 2015-04-07 15:53:14]

The CQ bit was checked by achaulk@chromium.org

[achaulk, 2015-04-07 15:53:14]

The patchset sent to the CQ was uploaded after l-g-t-m from alexst@chromium.org
Link to the patchset: https://codereview.chromium.org/1054073002/#ps40001
(title: " ")

[commit-bot: I haz the power, 2015-04-07 15:54:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1054073002/40001

[commit-bot: I haz the power, 2015-04-07 16:22:44]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-04-07 16:23:29]

Patchset 3 (id:??) landed as
https://crrev.com/77f08ab335da814a065dce6859a674b29e9c1294
Cr-Commit-Position: refs/heads/master@{#324053}