cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead, including inside LayerTreeHost
and the Scheduler.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck
ScheduledActionDrawAndSwap -> ScheduledActionSubmitCompositorFrame
SwapThrottled -> SubmitThrottled

R=enne@chromium.org, khushalsagar@chromium.org, sunnyps@chromium.org, sky@chromium.org, boliu@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/9d124421c3113ae9e87d92570a0d187a72d4154a
Cr-Commit-Position: refs/heads/master@{#425245}

[danakj, 2016-10-11 00:28:53]

Description was changed from

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck

R=enne@chromium.org, khushalsagar@chromium.org, sky@chromium.org
BUG=606056
==========

to

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck

R=enne@chromium.org, khushalsagar@chromium.org, sky@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[danakj, 2016-10-11 00:52:33]

Description was changed from

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck

R=enne@chromium.org, khushalsagar@chromium.org, sky@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead, including inside LayerTreeHost
and the Scheduler.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck
ScheduledActionDrawAndSwap -> ScheduledActionSubmitCompositorFrame

R=enne@chromium.org, khushalsagar@chromium.org, sky@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[danakj, 2016-10-11 00:52:51]

Description was changed from

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead, including inside LayerTreeHost
and the Scheduler.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck
ScheduledActionDrawAndSwap -> ScheduledActionSubmitCompositorFrame

R=enne@chromium.org, khushalsagar@chromium.org, sky@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead, including inside LayerTreeHost
and the Scheduler.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck
ScheduledActionDrawAndSwap -> ScheduledActionSubmitCompositorFrame
SwapThrottled -> SubmitThrottled

R=enne@chromium.org, khushalsagar@chromium.org, sky@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[danakj, 2016-10-11 00:57:38]

Description was changed from

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead, including inside LayerTreeHost
and the Scheduler.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck
ScheduledActionDrawAndSwap -> ScheduledActionSubmitCompositorFrame
SwapThrottled -> SubmitThrottled

R=enne@chromium.org, khushalsagar@chromium.org, sky@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead, including inside LayerTreeHost
and the Scheduler.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck
ScheduledActionDrawAndSwap -> ScheduledActionSubmitCompositorFrame
SwapThrottled -> SubmitThrottled

R=enne@chromium.org, khushalsagar@chromium.org, sunnyps@chromium.org,
sky@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[danakj, 2016-10-11 00:57:39]

danakj@chromium.org changed reviewers:
+ sunnyps@chromium.org

[danakj, 2016-10-11 00:59:38]

Description was changed from

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead, including inside LayerTreeHost
and the Scheduler.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck
ScheduledActionDrawAndSwap -> ScheduledActionSubmitCompositorFrame
SwapThrottled -> SubmitThrottled

R=enne@chromium.org, khushalsagar@chromium.org, sunnyps@chromium.org,
sky@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead, including inside LayerTreeHost
and the Scheduler.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck
ScheduledActionDrawAndSwap -> ScheduledActionSubmitCompositorFrame
SwapThrottled -> SubmitThrottled

R=enne@chromium.org, khushalsagar@chromium.org, sunnyps@chromium.org,
sky@chromium.org, boliu@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[danakj, 2016-10-11 00:59:38]

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

[danakj, 2016-10-11 01:00:26]

enne: overall
sunnyps: cc/scheduler/
boliu: c/b/ and c/r/android/
khushal: blimp/
sky: services/

[danakj, 2016-10-11 01:00:42]

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

[danakj, 2016-10-11 01:00:51]

Patchset #2 (id:20001) has been deleted

[danakj, 2016-10-11 01:00:55]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2016-10-11 01:01:38]

Dry run: 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-10-11 01:05:25]

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

[commit-bot: I haz the power, 2016-10-11 01:05:26]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[danakj, 2016-10-11 01:27:08]

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

[commit-bot: I haz the power, 2016-10-11 01:27:29]

Dry run: 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-10-11 01:46:30]

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

[commit-bot: I haz the power, 2016-10-11 01:46:31]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[sky, 2016-10-11 02:43:25]

LGTM - question I raised below can be resolved separately.

https://codereview.chromium.org/2409923002/diff/60001/services/ui/public/cpp/...
File services/ui/public/cpp/compositor_frame_sink.cc (right):

https://codereview.chromium.org/2409923002/diff/60001/services/ui/public/cpp/...
services/ui/public/cpp/compositor_frame_sink.cc:57:
base::Bind(&CompositorFrameSink::SubmitCompositorFrameAck,
Any idea why this doesn't supply client_ directly?

[boliu, 2016-10-11 15:16:33]

lgtm

[Khushal, 2016-10-11 17:18:45]

lgtm

[enne (OOO), 2016-10-11 17:26:00]

lgtm

https://codereview.chromium.org/2409923002/diff/60001/cc/output/compositor_fr...
File cc/output/compositor_frame_sink_client.h (right):

https://codereview.chromium.org/2409923002/diff/60001/cc/output/compositor_fr...
cc/output/compositor_frame_sink_client.h:48: // The CompositorFrameSink is lost
when the ContextProviders held by it
Thanks for these comments in this file.  :)

https://codereview.chromium.org/2409923002/diff/60001/cc/scheduler/scheduler_...
File cc/scheduler/scheduler_state_machine.h (right):

https://codereview.chromium.org/2409923002/diff/60001/cc/scheduler/scheduler_...
cc/scheduler/scheduler_state_machine.h:171: bool SubmitThrottled() const;
bikeshedding: IsSubmitThrottled?

[sunnyps, 2016-10-11 18:32:49]

https://codereview.chromium.org/2409923002/diff/60001/cc/output/compositor_fr...
File cc/output/compositor_frame_sink.h (right):

https://codereview.chromium.org/2409923002/diff/60001/cc/output/compositor_fr...
cc/output/compositor_frame_sink.h:108: // DidReceiveCompositorFrameAck()
asynchronously when the frame has been
bikeshed: DidAckCompositorFrame? or OnCompositorFrameAck? or
DidReturnCompositorFrame?

https://codereview.chromium.org/2409923002/diff/60001/cc/output/compositor_fr...
File cc/output/compositor_frame_sink_client.h (right):

https://codereview.chromium.org/2409923002/diff/60001/cc/output/compositor_fr...
cc/output/compositor_frame_sink_client.h:39: // unregisted by giving a null
base::Closure.
nit: unregistered

https://codereview.chromium.org/2409923002/diff/60001/cc/scheduler/scheduler.h
File cc/scheduler/scheduler.h (right):

https://codereview.chromium.org/2409923002/diff/60001/cc/scheduler/scheduler....
cc/scheduler/scheduler.h:42: virtual DrawResult
ScheduledActionSubmitCompositorFrameForced() = 0;
I'd prefer these to be called ScheduledActionDraw because the return value is
related about drawing (why we checkerboarded).

https://codereview.chromium.org/2409923002/diff/60001/cc/scheduler/scheduler_...
File cc/scheduler/scheduler_state_machine.h (left):

https://codereview.chromium.org/2409923002/diff/60001/cc/scheduler/scheduler_...
cc/scheduler/scheduler_state_machine.h:198: void SetSwapUsedIncompleteTile(bool
used_incomplete_tile);
Can you clarify why this method was removed?

https://codereview.chromium.org/2409923002/diff/60001/cc/trees/layer_tree_hos...
File cc/trees/layer_tree_host_impl.h (right):

https://codereview.chromium.org/2409923002/diff/60001/cc/trees/layer_tree_hos...
cc/trees/layer_tree_host_impl.h:305: // TODO(danakj): This isn't "drawing" it's
"generating a frame".
aside: I think it's OK for this to be called drawing. All "drawing" that happens
in Chrome is just about converting a set of high level primitives into a set of
low level primitives (e.g. layers to render passes, render passes to GL, etc.) 
so I'm not sure distinguishing between them is necessary. OTOH we use the word
"draw" to describe this everywhere throughout cc e.g. in the scheduler, in
CFS::OnDraw interface between webview and cc, etc. I'd rather not change all
these to SubmitCompositorFrame (or GenerateFrame or CompositeFrame).

https://codereview.chromium.org/2409923002/diff/60001/cc/trees/single_thread_...
File cc/trees/single_thread_proxy.cc (left):

https://codereview.chromium.org/2409923002/diff/60001/cc/trees/single_thread_...
cc/trees/single_thread_proxy.cc:554: client_->DidPostSwapBuffers();
Should DidPostSwapBuffers also be renamed?

[danakj, 2016-10-13 22:02:48]

https://codereview.chromium.org/2409923002/diff/60001/services/ui/public/cpp/...
File services/ui/public/cpp/compositor_frame_sink.cc (right):

https://codereview.chromium.org/2409923002/diff/60001/services/ui/public/cpp/...
services/ui/public/cpp/compositor_frame_sink.cc:57:
base::Bind(&CompositorFrameSink::SubmitCompositorFrameAck,
On 2016/10/11 02:43:25, sky wrote:
> Any idea why this doesn't supply client_ directly?

In similar cases its to use a weakptr but no reason to here.

[danakj, 2016-10-13 23:50:03]

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

[danakj, 2016-10-13 23:50:23]

PTAL sunny

https://codereview.chromium.org/2409923002/diff/60001/cc/output/compositor_fr...
File cc/output/compositor_frame_sink_client.h (right):

https://codereview.chromium.org/2409923002/diff/60001/cc/output/compositor_fr...
cc/output/compositor_frame_sink_client.h:39: // unregisted by giving a null
base::Closure.
On 2016/10/11 18:32:48, sunnyps wrote:
> nit: unregistered

Done.

https://codereview.chromium.org/2409923002/diff/60001/cc/scheduler/scheduler.h
File cc/scheduler/scheduler.h (right):

https://codereview.chromium.org/2409923002/diff/60001/cc/scheduler/scheduler....
cc/scheduler/scheduler.h:42: virtual DrawResult
ScheduledActionSubmitCompositorFrameForced() = 0;
On 2016/10/11 18:32:48, sunnyps wrote:
> I'd prefer these to be called ScheduledActionDraw because the return value is
> related about drawing (why we checkerboarded).

Done.

https://codereview.chromium.org/2409923002/diff/60001/cc/scheduler/scheduler_...
File cc/scheduler/scheduler_state_machine.h (left):

https://codereview.chromium.org/2409923002/diff/60001/cc/scheduler/scheduler_...
cc/scheduler/scheduler_state_machine.h:198: void SetSwapUsedIncompleteTile(bool
used_incomplete_tile);
On 2016/10/11 18:32:48, sunnyps wrote:
> Can you clarify why this method was removed?

The method doesn't exist only the definition does.

https://codereview.chromium.org/2409923002/diff/60001/cc/scheduler/scheduler_...
File cc/scheduler/scheduler_state_machine.h (right):

https://codereview.chromium.org/2409923002/diff/60001/cc/scheduler/scheduler_...
cc/scheduler/scheduler_state_machine.h:171: bool SubmitThrottled() const;
On 2016/10/11 17:26:00, enne wrote:
> bikeshedding: IsSubmitThrottled?

Went to IsDrawThrottled.

https://codereview.chromium.org/2409923002/diff/60001/cc/trees/layer_tree_hos...
File cc/trees/layer_tree_host_impl.h (right):

https://codereview.chromium.org/2409923002/diff/60001/cc/trees/layer_tree_hos...
cc/trees/layer_tree_host_impl.h:305: // TODO(danakj): This isn't "drawing" it's
"generating a frame".
On 2016/10/11 18:32:48, sunnyps wrote:
> aside: I think it's OK for this to be called drawing. All "drawing" that
happens
> in Chrome is just about converting a set of high level primitives into a set
of
> low level primitives (e.g. layers to render passes, render passes to GL, etc.)

> so I'm not sure distinguishing between them is necessary. OTOH we use the word
> "draw" to describe this everywhere throughout cc e.g. in the scheduler, in
> CFS::OnDraw interface between webview and cc, etc. I'd rather not change all
> these to SubmitCompositorFrame (or GenerateFrame or CompositeFrame).

Done.

https://codereview.chromium.org/2409923002/diff/60001/cc/trees/single_thread_...
File cc/trees/single_thread_proxy.cc (left):

https://codereview.chromium.org/2409923002/diff/60001/cc/trees/single_thread_...
cc/trees/single_thread_proxy.cc:554: client_->DidPostSwapBuffers();
On 2016/10/11 18:32:49, sunnyps wrote:
> Should DidPostSwapBuffers also be renamed?

Yes, ok I kept going and renamed this and that hit a bunch of more places. But I
feel like this patch is exploding in size so I'm going to undo that for now.

[commit-bot: I haz the power, 2016-10-13 23:50:24]

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

[sunnyps, 2016-10-14 01:08:47]

lgtm % nits

https://codereview.chromium.org/2409923002/diff/100001/cc/scheduler/scheduler...
File cc/scheduler/scheduler_state_machine.cc (right):

https://codereview.chromium.org/2409923002/diff/100001/cc/scheduler/scheduler...
cc/scheduler/scheduler_state_machine.cc:357: // Do not queue too many swaps.
nit: swaps -> draws

https://codereview.chromium.org/2409923002/diff/100001/cc/scheduler/scheduler...
cc/scheduler/scheduler_state_machine.cc:443: // We are waiting for previous
frame to be drawn, swapped and acked.
nit: swapped -> submitted

https://codereview.chromium.org/2409923002/diff/100001/cc/scheduler/scheduler...
File cc/scheduler/scheduler_unittest.cc (right):

https://codereview.chromium.org/2409923002/diff/100001/cc/scheduler/scheduler...
cc/scheduler/scheduler_unittest.cc:1499: void
SchedulerTest::ImplFrameSkippedAfterLateAck(
nit: Ack -> SubmitCompositorFrameAck

[danakj, 2016-10-14 01:19:31]

Done but

https://codereview.chromium.org/2409923002/diff/100001/cc/scheduler/scheduler...
File cc/scheduler/scheduler_unittest.cc (right):

https://codereview.chromium.org/2409923002/diff/100001/cc/scheduler/scheduler...
cc/scheduler/scheduler_unittest.cc:1499: void
SchedulerTest::ImplFrameSkippedAfterLateAck(
On 2016/10/14 01:08:47, sunnyps wrote:
> nit: Ack -> SubmitCompositorFrameAck

This makes test names go over 80 columns so I'll leave it for now?

[danakj, 2016-10-14 01:20:49]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-10-14 01:20:50]

The patchset sent to the CQ was uploaded after l-g-t-m from enne@chromium.org,
boliu@chromium.org, sky@chromium.org, khushalsagar@chromium.org,
sunnyps@chromium.org
Link to the patchset: https://codereview.chromium.org/2409923002/#ps120001
(title: "swap-to-submit: nits")

[commit-bot: I haz the power, 2016-10-14 01:21:00]

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-10-14 01:24:36]

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

[commit-bot: I haz the power, 2016-10-14 01:24:38]

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

[danakj, 2016-10-14 01:55:37]

The CQ bit was checked by danakj@chromium.org

[danakj, 2016-10-14 01:55:38]

The patchset sent to the CQ was uploaded after l-g-t-m from
sunnyps@chromium.org, enne@chromium.org, boliu@chromium.org, sky@chromium.org,
khushalsagar@chromium.org
Link to the patchset: https://codereview.chromium.org/2409923002/#ps140001
(title: "swap-to-submit: rebase")

[commit-bot: I haz the power, 2016-10-14 01:55:57]

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-10-14 03:16:05]

Description was changed from

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead, including inside LayerTreeHost
and the Scheduler.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck
ScheduledActionDrawAndSwap -> ScheduledActionSubmitCompositorFrame
SwapThrottled -> SubmitThrottled

R=enne@chromium.org, khushalsagar@chromium.org, sunnyps@chromium.org,
sky@chromium.org, boliu@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead, including inside LayerTreeHost
and the Scheduler.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck
ScheduledActionDrawAndSwap -> ScheduledActionSubmitCompositorFrame
SwapThrottled -> SubmitThrottled

R=enne@chromium.org, khushalsagar@chromium.org, sunnyps@chromium.org,
sky@chromium.org, boliu@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[commit-bot: I haz the power, 2016-10-14 03:16:08]

Committed patchset #6 (id:140001)

[commit-bot: I haz the power, 2016-10-14 03:18:05]

Description was changed from

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead, including inside LayerTreeHost
and the Scheduler.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck
ScheduledActionDrawAndSwap -> ScheduledActionSubmitCompositorFrame
SwapThrottled -> SubmitThrottled

R=enne@chromium.org, khushalsagar@chromium.org, sunnyps@chromium.org,
sky@chromium.org, boliu@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

This renames everything to do with "Swap" in the layer/delegating
compositors to be "Submit" instead, including inside LayerTreeHost
and the Scheduler.

CompositorFrameSink::SwapBuffers -> SubmitCompositorFrame
CompositorFrameSinkClient::DidSwapBuffersComplete ->
    DidReceiveCompositorFrameAck
LayerTreeHost::DidCompleteSwapBuffers ->
    DidReceiveCompositorFrameAck
ScheduledActionDrawAndSwap -> ScheduledActionSubmitCompositorFrame
SwapThrottled -> SubmitThrottled

R=enne@chromium.org, khushalsagar@chromium.org, sunnyps@chromium.org,
sky@chromium.org, boliu@chromium.org
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/9d124421c3113ae9e87d92570a0d187a72d4154a
Cr-Commit-Position: refs/heads/master@{#425245}
==========

[commit-bot: I haz the power, 2016-10-14 03:18:08]

Patchset 6 (id:??) landed as
https://crrev.com/9d124421c3113ae9e87d92570a0d187a72d4154a
Cr-Commit-Position: refs/heads/master@{#425245}