Implemented StructTraits for cc::CopyOutputRequest

Implemented StructTraits for cc::CopyOutputRequest

MojoCompositorFrameSink needs to support copy requests, which means we need
mojo definition and struct traits for cc::CopyOutputRequests in order to send it
over mojo.

BUG=672071
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2655963006
Cr-Commit-Position: refs/heads/master@{#447277}
Committed: https://chromium.googlesource.com/chromium/src/+/345632b5cbddaa65124eb98bce6fa933fa211c35

[Saman Sami, 2017-01-26 16:05:08]

Description was changed from

==========
Implemented StructTraits for cc::CopyOutputRequest

BUG=672071
==========

to

==========
Implemented StructTraits for cc::CopyOutputRequest

BUG=672071
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Saman Sami, 2017-01-26 16:05:49]

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

[commit-bot: I haz the power, 2017-01-26 16:06:09]

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, 2017-01-26 16:13:48]

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

[commit-bot: I haz the power, 2017-01-26 16:13:49]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Saman Sami, 2017-01-26 22:16:10]

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

[commit-bot: I haz the power, 2017-01-26 22:16:48]

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, 2017-01-26 22:26:47]

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

[commit-bot: I haz the power, 2017-01-26 22:26:48]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Saman Sami, 2017-01-30 21:49:15]

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

[commit-bot: I haz the power, 2017-01-30 21:49:50]

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

[Saman Sami, 2017-01-30 21:53:03]

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

[commit-bot: I haz the power, 2017-01-30 21:53:58]

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

[Saman Sami, 2017-01-30 22:31:08]

Description was changed from

==========
Implemented StructTraits for cc::CopyOutputRequest

BUG=672071
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Implemented StructTraits for cc::CopyOutputRequest

BUG=672071
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[Saman Sami, 2017-01-30 22:31:08]

samans@chromium.org changed reviewers:
+ fsamuel@chromium.org

[Saman Sami, 2017-01-30 22:33:06]

PTAL

[commit-bot: I haz the power, 2017-01-30 23:26:26]

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

[commit-bot: I haz the power, 2017-01-30 23:26:27]

Dry run: This issue passed the CQ dry run.

[Fady Samuel, 2017-01-30 23:56:58]

lgtm

[Saman Sami, 2017-01-31 00:51:58]

samans@chromium.org changed reviewers:
+ enne@chromium.org, tsepez@chromium.org

[Saman Sami, 2017-01-31 00:52:51]

enne@: Please review cc/
tsepez@: Please review mojo

[Saman Sami, 2017-01-31 15:25:34]

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

[commit-bot: I haz the power, 2017-01-31 15:25:45]

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

[Saman Sami, 2017-01-31 15:30:05]

samans@chromium.org changed reviewers:
+ sky@chromium.org

[Saman Sami, 2017-01-31 15:31:04]

sky: Please review cc/output/DEPS

[Saman Sami, 2017-01-31 15:31:15]

PTAL

[Saman Sami, 2017-01-31 15:36:07]

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

[commit-bot: I haz the power, 2017-01-31 15:36:30]

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

[Tom Sepez, 2017-01-31 16:46:34]

lgtm

[commit-bot: I haz the power, 2017-01-31 16:47:43]

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

[commit-bot: I haz the power, 2017-01-31 16:47:44]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Saman Sami, 2017-01-31 16:52:12]

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

[commit-bot: I haz the power, 2017-01-31 16:52:34]

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

[Fady Samuel, 2017-01-31 17:02:53]

Please add a short description WHY you're adding StructTraits for our future
reference. Thanks!

[Saman Sami, 2017-01-31 17:15:12]

Description was changed from

==========
Implemented StructTraits for cc::CopyOutputRequest

BUG=672071
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Implemented StructTraits for cc::CopyOutputRequest

MojoCompositorFrameSink needs to support copy requests, which means we need
mojo definition and struct traits for cc::CopyOutputRequests in order to send it
over mojo.

BUG=672071
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[commit-bot: I haz the power, 2017-01-31 17:53:15]

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

[commit-bot: I haz the power, 2017-01-31 17:53:16]

Dry run: This issue passed the CQ dry run.

[sky, 2017-01-31 18:23:31]

LGTM

[enne (OOO), 2017-01-31 18:24:46]

enne@chromium.org changed reviewers:
- sky@chromium.org

[enne (OOO), 2017-01-31 18:24:47]

lgtm

https://codereview.chromium.org/2655963006/diff/60001/cc/ipc/copy_output_requ...
File cc/ipc/copy_output_request.mojom (right):

https://codereview.chromium.org/2655963006/diff/60001/cc/ipc/copy_output_requ...
cc/ipc/copy_output_request.mojom:10: // TODO(samans): Add source.
In general, I was expecting some sort of id here so that the recipient of the
response could know which request it went with, but I that's something that can
happen later.

[Saman Sami, 2017-01-31 18:26:45]

The CQ bit was checked by samans@chromium.org

[Saman Sami, 2017-01-31 18:26:46]

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

[commit-bot: I haz the power, 2017-01-31 18:27:32]

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, 2017-01-31 18:34:12]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1485887205572090,
"parent_rev": "3ea8c007ce51f08fcc6429be7957e3d75627d8e9", "commit_rev":
"345632b5cbddaa65124eb98bce6fa933fa211c35"}

[commit-bot: I haz the power, 2017-01-31 18:34:43]

Description was changed from

==========
Implemented StructTraits for cc::CopyOutputRequest

MojoCompositorFrameSink needs to support copy requests, which means we need
mojo definition and struct traits for cc::CopyOutputRequests in order to send it
over mojo.

BUG=672071
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Implemented StructTraits for cc::CopyOutputRequest

MojoCompositorFrameSink needs to support copy requests, which means we need
mojo definition and struct traits for cc::CopyOutputRequests in order to send it
over mojo.

BUG=672071
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2655963006
Cr-Commit-Position: refs/heads/master@{#447277}
Committed:
https://chromium.googlesource.com/chromium/src/+/345632b5cbddaa65124eb98bce6f...
==========

[commit-bot: I haz the power, 2017-01-31 18:34:45]

Committed patchset #7 (id:120001) as
https://chromium.googlesource.com/chromium/src/+/345632b5cbddaa65124eb98bce6f...