Revert of Replaced cc::SurfaceId::nonce_ with base::UnguessableToken

Revert of Replaced cc::SurfaceId::nonce_ with base::UnguessableToken (patchset #25 id:480001 of https://codereview.chromium.org/2379653006/ )

Reason for revert:
Looks like this added a static initializer to surface_manager.cc via the change to unguessable_token

https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/28400/steps/sizes/logs/stdio

Original issue's description:
> Replaced cc::SurfaceId::nonce_ with base::UnguessableToken
>
> base::SurfaceId::nonce_ is an uint64 generated at random to make the entire
> surface id unguessable.
> base::UnguessableToken is a wrapper for 2 uint64 (high_ and low_).
> high_ and low_ are generated at random using base::UnguessableToken::Create().
> base::UnguessableToken achieves the same goal as the original
> cc::SurfaceId::nonce_ with better security and uses more common code.
>
> BUG=649800
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
>
> Committed: https://crrev.com/6b01e58ff708c17f74b560fe1b57f7733b78da53
> Cr-Commit-Position: refs/heads/master@{#431294}

TBR=danakj@chromium.org,tsepez@chromium.org,fsamuel@chromium.org,jbroman@chromium.org,haraken@chromium.org,vmpstr@chromium.org,sky@chromium.org,dcheng@chromium.org,staraz@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=649800
Committed: https://crrev.com/d75809ef13714bcc4e960779d2178d9f99783147
Cr-Commit-Position: refs/heads/master@{#431330}

[Dirk Pranke, 2016-11-10 20:03:02]

The CQ bit was checked by dpranke@chromium.org

[Dirk Pranke, 2016-11-10 20:03:03]

Created Revert of Replaced cc::SurfaceId::nonce_ with base::UnguessableToken

[commit-bot: I haz the power, 2016-11-10 20:03:41]

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-11-10 20:05:11]

Description was changed from

==========
Revert of Replaced cc::SurfaceId::nonce_ with base::UnguessableToken (patchset
#25 id:480001 of https://codereview.chromium.org/2379653006/ )

Reason for revert:
Looks like this added a static initializer to surface_manager.cc via the change
to unguessable_token

https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/28400/steps...

Original issue's description:
> Replaced cc::SurfaceId::nonce_ with base::UnguessableToken
> 
> base::SurfaceId::nonce_ is an uint64 generated at random to make the entire
> surface id unguessable.
> base::UnguessableToken is a wrapper for 2 uint64 (high_ and low_).
> high_ and low_ are generated at random using base::UnguessableToken::Create().
> base::UnguessableToken achieves the same goal as the original
> cc::SurfaceId::nonce_ with better security and uses more common code.
> 
> BUG=649800
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
> 
> Committed: https://crrev.com/6b01e58ff708c17f74b560fe1b57f7733b78da53
> Cr-Commit-Position: refs/heads/master@{#431294}

TBR=danakj@chromium.org,tsepez@chromium.org,fsamuel@chromium.org,jbroman@chro...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=649800
==========

to

==========
Revert of Replaced cc::SurfaceId::nonce_ with base::UnguessableToken (patchset
#25 id:480001 of https://codereview.chromium.org/2379653006/ )

Reason for revert:
Looks like this added a static initializer to surface_manager.cc via the change
to unguessable_token

https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/28400/steps...

Original issue's description:
> Replaced cc::SurfaceId::nonce_ with base::UnguessableToken
> 
> base::SurfaceId::nonce_ is an uint64 generated at random to make the entire
> surface id unguessable.
> base::UnguessableToken is a wrapper for 2 uint64 (high_ and low_).
> high_ and low_ are generated at random using base::UnguessableToken::Create().
> base::UnguessableToken achieves the same goal as the original
> cc::SurfaceId::nonce_ with better security and uses more common code.
> 
> BUG=649800
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
> 
> Committed: https://crrev.com/6b01e58ff708c17f74b560fe1b57f7733b78da53
> Cr-Commit-Position: refs/heads/master@{#431294}

TBR=danakj@chromium.org,tsepez@chromium.org,fsamuel@chromium.org,jbroman@chro...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=649800
==========

[commit-bot: I haz the power, 2016-11-10 20:05:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-10 20:14:22]

Description was changed from

==========
Revert of Replaced cc::SurfaceId::nonce_ with base::UnguessableToken (patchset
#25 id:480001 of https://codereview.chromium.org/2379653006/ )

Reason for revert:
Looks like this added a static initializer to surface_manager.cc via the change
to unguessable_token

https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/28400/steps...

Original issue's description:
> Replaced cc::SurfaceId::nonce_ with base::UnguessableToken
> 
> base::SurfaceId::nonce_ is an uint64 generated at random to make the entire
> surface id unguessable.
> base::UnguessableToken is a wrapper for 2 uint64 (high_ and low_).
> high_ and low_ are generated at random using base::UnguessableToken::Create().
> base::UnguessableToken achieves the same goal as the original
> cc::SurfaceId::nonce_ with better security and uses more common code.
> 
> BUG=649800
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
> 
> Committed: https://crrev.com/6b01e58ff708c17f74b560fe1b57f7733b78da53
> Cr-Commit-Position: refs/heads/master@{#431294}

TBR=danakj@chromium.org,tsepez@chromium.org,fsamuel@chromium.org,jbroman@chro...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=649800
==========

to

==========
Revert of Replaced cc::SurfaceId::nonce_ with base::UnguessableToken (patchset
#25 id:480001 of https://codereview.chromium.org/2379653006/ )

Reason for revert:
Looks like this added a static initializer to surface_manager.cc via the change
to unguessable_token

https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/28400/steps...

Original issue's description:
> Replaced cc::SurfaceId::nonce_ with base::UnguessableToken
>
> base::SurfaceId::nonce_ is an uint64 generated at random to make the entire
> surface id unguessable.
> base::UnguessableToken is a wrapper for 2 uint64 (high_ and low_).
> high_ and low_ are generated at random using base::UnguessableToken::Create().
> base::UnguessableToken achieves the same goal as the original
> cc::SurfaceId::nonce_ with better security and uses more common code.
>
> BUG=649800
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
>
> Committed: https://crrev.com/6b01e58ff708c17f74b560fe1b57f7733b78da53
> Cr-Commit-Position: refs/heads/master@{#431294}

TBR=danakj@chromium.org,tsepez@chromium.org,fsamuel@chromium.org,jbroman@chro...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=649800

Committed: https://crrev.com/d75809ef13714bcc4e960779d2178d9f99783147
Cr-Commit-Position: refs/heads/master@{#431330}
==========

[commit-bot: I haz the power, 2016-11-10 20:14:23]

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