Move AcceleratedSurfaceBuffersSwappedParams to content/common

Move AcceleratedSurfaceBuffersSwappedParams to content/common

This struct is used for browser<=>gpu IPC. As with EstablishChannelParams,
this CL moves this struct to content/common and removes its usage
from within content/common/gpu

BUG=596290
Committed: https://crrev.com/521c72b8e01db6716892e82b8fb5d23864766ae7
Cr-Commit-Position: refs/heads/master@{#383117}

[Fady Samuel, 2016-03-24 03:48:03]

fsamuel@chromium.org changed reviewers:
+ dcheng@chromium.org, piman@chromium.org

[Fady Samuel, 2016-03-24 03:48:04]

+piman@: overall review.
+dcheng@: IPC

PTAL Thanks!

[Fady Samuel, 2016-03-24 03:49:28]

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

[commit-bot: I haz the power, 2016-03-24 03:49:35]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1830513003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1830513003/20001

[dcheng, 2016-03-24 03:50:11]

rs lgtm for ipcchanges.

[commit-bot: I haz the power, 2016-03-24 04:05:24]

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

[commit-bot: I haz the power, 2016-03-24 04:05:25]

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

[Fady Samuel, 2016-03-24 04:14:36]

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

[commit-bot: I haz the power, 2016-03-24 04:14:43]

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

[commit-bot: I haz the power, 2016-03-24 05:25:28]

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

[commit-bot: I haz the power, 2016-03-24 05:25:28]

Dry run: This issue passed the CQ dry run.

[piman, 2016-03-24 18:02:21]

lgtm

https://codereview.chromium.org/1830513003/diff/40001/content/gpu/gpu_child_t...
File content/gpu/gpu_child_thread.cc (right):

https://codereview.chromium.org/1830513003/diff/40001/content/gpu/gpu_child_t...
content/gpu/gpu_child_thread.cc:337: const std::vector<ui::LatencyInfo>&
latency_info) {
nit: make this "std::vector<ui::LatencyInfo> latency_info", and use std::move
both on the caller side and here below, to save a copy.

[Fady Samuel, 2016-03-24 18:17:53]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-03-24 18:17:54]

The patchset sent to the CQ was uploaded after l-g-t-m from dcheng@chromium.org,
piman@chromium.org
Link to the patchset: https://codereview.chromium.org/1830513003/#ps60001
(title: "Addressed piman's comment")

[Fady Samuel, 2016-03-24 18:18:03]

CQ'ed. Thanks!

https://codereview.chromium.org/1830513003/diff/40001/content/gpu/gpu_child_t...
File content/gpu/gpu_child_thread.cc (right):

https://codereview.chromium.org/1830513003/diff/40001/content/gpu/gpu_child_t...
content/gpu/gpu_child_thread.cc:337: const std::vector<ui::LatencyInfo>&
latency_info) {
On 2016/03/24 18:02:21, piman wrote:
> nit: make this "std::vector<ui::LatencyInfo> latency_info", and use std::move
> both on the caller side and here below, to save a copy.

Done.

[commit-bot: I haz the power, 2016-03-24 18:18:20]

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

[commit-bot: I haz the power, 2016-03-24 19:19:42]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-03-24 19:21:25]

Description was changed from

==========
Move AcceleratedSurfaceBuffersSwappedParams to content/common

This struct is used for browser<=>gpu IPC. As with EstablishChannelParams,
this CL moves this struct to content/common and removes its usage
from within content/common/gpu

BUG=596290
==========

to

==========
Move AcceleratedSurfaceBuffersSwappedParams to content/common

This struct is used for browser<=>gpu IPC. As with EstablishChannelParams,
this CL moves this struct to content/common and removes its usage
from within content/common/gpu

BUG=596290

Committed: https://crrev.com/521c72b8e01db6716892e82b8fb5d23864766ae7
Cr-Commit-Position: refs/heads/master@{#383117}
==========

[commit-bot: I haz the power, 2016-03-24 19:21:26]

Patchset 4 (id:??) landed as
https://crrev.com/521c72b8e01db6716892e82b8fb5d23864766ae7
Cr-Commit-Position: refs/heads/master@{#383117}