Revert of Added an extra sync token field for extra command buffer identification.

Revert of Added an extra sync token field for extra command buffer identification. (patchset #8 id:140001 of https://codereview.chromium.org/1489573003/ )

Reason for revert:
This is causing a large number of layout tests in virtual/threaded/animations to crash.

They're hitting the following assertion:
FATAL:scheduler.cc(203)] Check failed: state_machine_.pending_swaps() > 0 (0 vs. 0){"compositor_timing_history":

e.g. see https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.7%20%28dbg%29/builds/25835

Confirmed with a local bisect.

Original issue's description:
> Added an extra sync token field for extra command buffer identification.
>
> This CL adds an extra field within sync tokens without changing the size
> of the sync token. Currently this extra field is only used for the
> GPU channel implementation, storing the stream of the identified command
> buffer.
>
> BUG=514815
>
> Committed: https://crrev.com/f21e7e6e2b260d56fa54373dd662c81c9d955331
> Cr-Commit-Position: refs/heads/master@{#363231}

TBR=dcheng@chromium.org,dalecurtis@chromium.org,piman@chromium.org,sky@chromium.org,dyen@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=514815
Committed: https://crrev.com/5097a2103f86741a7d0e6567e57759be939182b0
Cr-Commit-Position: refs/heads/master@{#363290}

[ajuma, 2015-12-04 20:50:44]

Created Revert of Added an extra sync token field for extra command buffer
identification.

[commit-bot: I haz the power, 2015-12-04 20:52:27]

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

[commit-bot: I haz the power, 2015-12-04 20:56:51]

Description was changed from

==========
Revert of Added an extra sync token field for extra command buffer
identification. (patchset #8 id:140001 of
https://codereview.chromium.org/1489573003/ )

Reason for revert:
This is causing a large number of layout tests in virtual/threaded/animations to
crash.

They're hitting the following assertion:
FATAL:scheduler.cc(203)] Check failed: state_machine_.pending_swaps() > 0 (0 vs.
0){"compositor_timing_history":

e.g. see
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.7%20%28d...

Confirmed with a local bisect.

Original issue's description:
> Added an extra sync token field for extra command buffer identification.
> 
> This CL adds an extra field within sync tokens without changing the size
> of the sync token. Currently this extra field is only used for the
> GPU channel implementation, storing the stream of the identified command
> buffer.
> 
> BUG=514815
> 
> Committed: https://crrev.com/f21e7e6e2b260d56fa54373dd662c81c9d955331
> Cr-Commit-Position: refs/heads/master@{#363231}

TBR=dcheng@chromium.org,dalecurtis@chromium.org,piman@chromium.org,sky@chromi...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=514815
==========

to

==========
Revert of Added an extra sync token field for extra command buffer
identification. (patchset #8 id:140001 of
https://codereview.chromium.org/1489573003/ )

Reason for revert:
This is causing a large number of layout tests in virtual/threaded/animations to
crash.

They're hitting the following assertion:
FATAL:scheduler.cc(203)] Check failed: state_machine_.pending_swaps() > 0 (0 vs.
0){"compositor_timing_history":

e.g. see
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.7%20%28d...

Confirmed with a local bisect.

Original issue's description:
> Added an extra sync token field for extra command buffer identification.
> 
> This CL adds an extra field within sync tokens without changing the size
> of the sync token. Currently this extra field is only used for the
> GPU channel implementation, storing the stream of the identified command
> buffer.
> 
> BUG=514815
> 
> Committed: https://crrev.com/f21e7e6e2b260d56fa54373dd662c81c9d955331
> Cr-Commit-Position: refs/heads/master@{#363231}

TBR=dcheng@chromium.org,dalecurtis@chromium.org,piman@chromium.org,sky@chromi...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=514815
==========

[commit-bot: I haz the power, 2015-12-04 20:56:53]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-04 20:58:11]

Description was changed from

==========
Revert of Added an extra sync token field for extra command buffer
identification. (patchset #8 id:140001 of
https://codereview.chromium.org/1489573003/ )

Reason for revert:
This is causing a large number of layout tests in virtual/threaded/animations to
crash.

They're hitting the following assertion:
FATAL:scheduler.cc(203)] Check failed: state_machine_.pending_swaps() > 0 (0 vs.
0){"compositor_timing_history":

e.g. see
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.7%20%28d...

Confirmed with a local bisect.

Original issue's description:
> Added an extra sync token field for extra command buffer identification.
> 
> This CL adds an extra field within sync tokens without changing the size
> of the sync token. Currently this extra field is only used for the
> GPU channel implementation, storing the stream of the identified command
> buffer.
> 
> BUG=514815
> 
> Committed: https://crrev.com/f21e7e6e2b260d56fa54373dd662c81c9d955331
> Cr-Commit-Position: refs/heads/master@{#363231}

TBR=dcheng@chromium.org,dalecurtis@chromium.org,piman@chromium.org,sky@chromi...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=514815
==========

to

==========
Revert of Added an extra sync token field for extra command buffer
identification. (patchset #8 id:140001 of
https://codereview.chromium.org/1489573003/ )

Reason for revert:
This is causing a large number of layout tests in virtual/threaded/animations to
crash.

They're hitting the following assertion:
FATAL:scheduler.cc(203)] Check failed: state_machine_.pending_swaps() > 0 (0 vs.
0){"compositor_timing_history":

e.g. see
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.7%20%28d...

Confirmed with a local bisect.

Original issue's description:
> Added an extra sync token field for extra command buffer identification.
>
> This CL adds an extra field within sync tokens without changing the size
> of the sync token. Currently this extra field is only used for the
> GPU channel implementation, storing the stream of the identified command
> buffer.
>
> BUG=514815
>
> Committed: https://crrev.com/f21e7e6e2b260d56fa54373dd662c81c9d955331
> Cr-Commit-Position: refs/heads/master@{#363231}

TBR=dcheng@chromium.org,dalecurtis@chromium.org,piman@chromium.org,sky@chromi...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=514815

Committed: https://crrev.com/5097a2103f86741a7d0e6567e57759be939182b0
Cr-Commit-Position: refs/heads/master@{#363290}
==========

[commit-bot: I haz the power, 2015-12-04 20:58:12]

Patchset 1 (id:??) landed as
https://crrev.com/5097a2103f86741a7d0e6567e57759be939182b0
Cr-Commit-Position: refs/heads/master@{#363290}