Android: Improve Composite-with-no-GPU-channel handling

Android: Improve Composite-with-no-GPU-channel handling

The current code simply bails out of Composite() when the channel
is not established yet (or lost).
This seems to work in practice, but following the code it looks like
we would
- generally still think that the Composite task is pending
- in OnVSync think that we missed the window and post again
- further attempts to schedule, including when the channel eventually
  gets established, will think it's already pending and going to happen
  without the need to post a task
- but it will really only happen if OnVSync gets called again
  (somebody happened to request vsync) and we think we missed the frame
  and schedule again.

Improve this by
- tracking when we deferred Composite because we are waiting
  for the channel
- bail out of PostComposite if we are in this condition
- when established unset and post Composite

Committed: https://crrev.com/a0e92afeb69eb7cca3fddeb86aa1238d51d672b3
Cr-Commit-Position: refs/heads/master@{#305298}

[no sievers, 2014-11-21 20:59:59]

sievers@chromium.org changed reviewers:
+ jdduke@chromium.org

[no sievers, 2014-11-21 21:00:17]

Jared, can you take a look?

[jdduke (slow), 2014-11-21 21:16:35]

lgtm

https://codereview.chromium.org/750643003/diff/1/content/browser/renderer_hos...
File content/browser/renderer_host/compositor_impl_android.cc (right):

https://codereview.chromium.org/750643003/diff/1/content/browser/renderer_hos...
content/browser/renderer_host/compositor_impl_android.cc:281:
weak_factory_.GetWeakPtr()));
Maybe |DCHECK(!defer_composite_for_gpu_channel_);| here?

[no sievers, 2014-11-21 21:19:19]

https://codereview.chromium.org/750643003/diff/1/content/browser/renderer_hos...
File content/browser/renderer_host/compositor_impl_android.cc (right):

https://codereview.chromium.org/750643003/diff/1/content/browser/renderer_hos...
content/browser/renderer_host/compositor_impl_android.cc:281:
weak_factory_.GetWeakPtr()));
On 2014/11/21 21:16:34, jdduke wrote:
> Maybe |DCHECK(!defer_composite_for_gpu_channel_);| here?

Done.

[no sievers, 2014-11-21 21:19:23]

The CQ bit was checked by sievers@chromium.org

[commit-bot: I haz the power, 2014-11-21 21:20:15]

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

[commit-bot: I haz the power, 2014-11-21 22:13:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-11-21 22:13:38]

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