Avoid scheduling unnecessary ProcessGL between GL draws.

Avoid scheduling unnecessary ProcessGL between GL draws.

SurfaceFactory::SubmitCompositorFrame might call glFlush. So calling it
during "kModeSync" stage (which does not allow GL) might result in extra
kModeProcess. Instead, submit the frame in "kModeDraw" stage to avoid
unnecessary kModeProcess.

BUG=527113
Committed: https://crrev.com/214d8f9ab144172967bf29e5b12ac5355304c749
Cr-Commit-Position: refs/heads/master@{#348808}

[hush (inactive), 2015-09-14 23:50:28]

hush@chromium.org changed reviewers:
+ boliu@chromium.org

[hush (inactive), 2015-09-14 23:50:28]

PTAL

[boliu, 2015-09-15 00:02:40]

https://codereview.chromium.org/1341533005/diff/20001/android_webview/browser...
File android_webview/browser/hardware_renderer.cc (right):

https://codereview.chromium.org/1341533005/diff/20001/android_webview/browser...
android_webview/browser/hardware_renderer.cc:100: if (child_frame_.get() &&
child_frame_->frame.get()) {
Ehh, pass the whole child_frame_ below, then you don't need the second half of
this conditional

Also keep the DCHECKs in CommitFrame. Crashing earlier is better

[hush (inactive), 2015-09-15 00:50:30]

https://codereview.chromium.org/1341533005/diff/20001/android_webview/browser...
File android_webview/browser/hardware_renderer.cc (right):

https://codereview.chromium.org/1341533005/diff/20001/android_webview/browser...
android_webview/browser/hardware_renderer.cc:100: if (child_frame_.get() &&
child_frame_->frame.get()) {
On 2015/09/15 00:02:40, boliu wrote:
> Ehh, pass the whole child_frame_ below, then you don't need the second half of
> this conditional
> 
> Also keep the DCHECKs in CommitFrame. Crashing earlier is better

Done.

[boliu, 2015-09-15 00:57:36]

lgtm

[hush (inactive), 2015-09-15 00:58:24]

The CQ bit was checked by hush@chromium.org

[commit-bot: I haz the power, 2015-09-15 00:58:43]

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

[commit-bot: I haz the power, 2015-09-15 01:29:49]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-09-15 01:30:24]

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

[commit-bot: I haz the power, 2015-09-23 12:41:03]

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