Add extra_data_field_ in gpu::SyncToken converter and remove two array converters

Add extra_data_field_ in gpu::SyncToken converter and remove two array converters

BUG=565553
TBR=ben@chromium.org

Committed: https://crrev.com/090488ff3ff55782b02a1b2965bdbe2acd0b8f53
Cr-Commit-Position: refs/heads/master@{#366758}

[Peng, 2015-12-23 17:07:42]

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

[Peng, 2015-12-23 17:07:42]

Hi Fady, PTAL. Thanks.

[Fady Samuel, 2015-12-23 17:13:02]

Could you please give some more background? What's this extra field for?

[Peng, 2015-12-23 17:13:03]

Description was changed from

==========
Add extra_data_field_ in gpu::SyncToken converter and remove two array
converters

BUG=None
==========

to

==========
Add extra_data_field_ in gpu::SyncToken converter and remove two array
converters

BUG=565553
==========

[Peng, 2015-12-23 17:16:04]

On 2015/12/23 17:13:02, Fady Samuel wrote:
> Could you please give some more background? What's this extra field for?

See the comment inline [1].

[1]
https://code.google.com/p/chromium/codesearch#chromium/src/gpu/command_buffer...

[Peng, 2015-12-23 17:16:06]

On 2015/12/23 17:13:02, Fady Samuel wrote:
> Could you please give some more background? What's this extra field for?

See the comment inline [1].

[1]
https://code.google.com/p/chromium/codesearch#chromium/src/gpu/command_buffer...

[Fady Samuel, 2015-12-23 17:33:41]

lgtm

[Peng, 2015-12-23 17:34:09]

The CQ bit was checked by penghuang@chromium.org

[commit-bot: I haz the power, 2015-12-23 17:34:18]

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

[commit-bot: I haz the power, 2015-12-23 17:41:10]

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

[commit-bot: I haz the power, 2015-12-23 17:41:11]

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

[Peng, 2015-12-23 18:35:19]

Description was changed from

==========
Add extra_data_field_ in gpu::SyncToken converter and remove two array
converters

BUG=565553
==========

to

==========
Add extra_data_field_ in gpu::SyncToken converter and remove two array
converters

BUG=565553
TBR=ben@chromium.org
==========

[Peng, 2015-12-23 18:35:24]

The CQ bit was checked by penghuang@chromium.org

[commit-bot: I haz the power, 2015-12-23 18:35:43]

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

[commit-bot: I haz the power, 2015-12-23 18:39:12]

Description was changed from

==========
Add extra_data_field_ in gpu::SyncToken converter and remove two array
converters

BUG=565553
TBR=ben@chromium.org
==========

to

==========
Add extra_data_field_ in gpu::SyncToken converter and remove two array
converters

BUG=565553
TBR=ben@chromium.org
==========

[commit-bot: I haz the power, 2015-12-23 18:39:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-23 18:40:06]

Description was changed from

==========
Add extra_data_field_ in gpu::SyncToken converter and remove two array
converters

BUG=565553
TBR=ben@chromium.org
==========

to

==========
Add extra_data_field_ in gpu::SyncToken converter and remove two array
converters

BUG=565553
TBR=ben@chromium.org

Committed: https://crrev.com/090488ff3ff55782b02a1b2965bdbe2acd0b8f53
Cr-Commit-Position: refs/heads/master@{#366758}
==========

[commit-bot: I haz the power, 2015-12-23 18:40:07]

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