Move SurfaceSequenceGenerator => cc/surfaces

Move SurfaceSequenceGenerator => cc/surfaces

SurfaceSequenceGenerator is useful in mus in the short term and so
this CL moves it somewhere where we can use it.

BUG=647852
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/3439656732dc29f07bcc7800ff0226179ecde149
Cr-Commit-Position: refs/heads/master@{#423532}

[Fady Samuel, 2016-10-06 02:40:59]

Description was changed from

==========
Move SurfaceSequenceGenerator => cc/surfaces

SurfaceSequenceGenerator is useful in mus in the short term and so
this CL moves it somewhere where we can use it.

BUG=647852
==========

to

==========
Move SurfaceSequenceGenerator => cc/surfaces

SurfaceSequenceGenerator is useful in mus in the short term and so
this CL moves it somewhere where we can use it.

BUG=647852
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[Fady Samuel, 2016-10-06 02:41:15]

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

[Fady Samuel, 2016-10-06 02:41:22]

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

[commit-bot: I haz the power, 2016-10-06 02:41:47]

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-06 03:52:56]

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

[commit-bot: I haz the power, 2016-10-06 03:52:59]

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

[ajuma, 2016-10-06 13:43:20]

lgtm

https://codereview.chromium.org/2397103002/diff/1/cc/surfaces/surface_sequenc...
File cc/surfaces/surface_sequence_generator.h (right):

https://codereview.chromium.org/2397103002/diff/1/cc/surfaces/surface_sequenc...
cc/surfaces/surface_sequence_generator.h:21: class CC_EXPORT
SurfaceSequenceGenerator {
I think you want CC_SURFACES_EXPORT here

[Fady Samuel, 2016-10-06 13:48:38]

Thanks Ali! CQ'ing.

https://codereview.chromium.org/2397103002/diff/1/cc/surfaces/surface_sequenc...
File cc/surfaces/surface_sequence_generator.h (right):

https://codereview.chromium.org/2397103002/diff/1/cc/surfaces/surface_sequenc...
cc/surfaces/surface_sequence_generator.h:21: class CC_EXPORT
SurfaceSequenceGenerator {
On 2016/10/06 13:43:20, ajuma wrote:
> I think you want CC_SURFACES_EXPORT here

Done.

[Fady Samuel, 2016-10-06 13:48:44]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-10-06 13:48:44]

The patchset sent to the CQ was uploaded after l-g-t-m from ajuma@chromium.org
Link to the patchset: https://codereview.chromium.org/2397103002/#ps20001
(title: "Addressed comment")

[commit-bot: I haz the power, 2016-10-06 13:49:01]

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-06 13:52:25]

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

[commit-bot: I haz the power, 2016-10-06 13:52:26]

Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Fady Samuel, 2016-10-06 13:54:30]

On 2016/10/06 13:52:26, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
>
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
>   android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
>
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)
>   linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
>
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

I'd like to keep surface_sequence_generator with surface_id source_set, which
means I don't need an export at all, since it's not in a different component.

[Fady Samuel, 2016-10-06 13:56:47]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-10-06 13:56:47]

The patchset sent to the CQ was uploaded after l-g-t-m from ajuma@chromium.org
Link to the patchset: https://codereview.chromium.org/2397103002/#ps40001
(title: "Remove export")

[commit-bot: I haz the power, 2016-10-06 13:57:09]

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-06 15:01:46]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-06 15:04:20]

Description was changed from

==========
Move SurfaceSequenceGenerator => cc/surfaces

SurfaceSequenceGenerator is useful in mus in the short term and so
this CL moves it somewhere where we can use it.

BUG=647852
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
Move SurfaceSequenceGenerator => cc/surfaces

SurfaceSequenceGenerator is useful in mus in the short term and so
this CL moves it somewhere where we can use it.

BUG=647852
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/3439656732dc29f07bcc7800ff0226179ecde149
Cr-Commit-Position: refs/heads/master@{#423532}
==========

[commit-bot: I haz the power, 2016-10-06 15:04:21]

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