Revert of SkOnce: 2 bytes -> 1 byte

Revert of SkOnce: 2 bytes -> 1 byte (patchset #4 id:60001 of https://codereview.chromium.org/1904483003/ )

Reason for revert:
bust the roll

Original issue's description:
> SkOnce: 2 bytes -> 1 byte
> 
> This uses the same logic we worked out for SkOncePtr to reduce
> the memory footprint of SkOnce from a done byte and lock byte
> to a single 3-state byte:
> 
>   - NotStarted: no thread has tried to run fn() yet
>   - Active:     a thread is running fn()
>   - Done:       fn() is complete
> 
> Threads which see Done return immediately.
> Threads which see NotStarted try to move to Active, run fn(), then move to Done.
> Threads which see Active spin until the active thread moves to Done.
> 
> This additionally fixes a too-weak memory order bug in SkOncePtr,
> and adds a big note to explain.
> 
> BUG=skia:
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1904483003
> 
> Committed: https://skia.googlesource.com/skia/+/df02d338be8e3c1c50b48a3a9faa582703a39c07

TBR=herb@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Committed: https://skia.googlesource.com/skia/+/9134686fd9ce384e9163b44942538d033b5bff11

[mtklein_C, 2016-04-20 20:02:01]

Created Revert of SkOnce: 2 bytes -> 1 byte

[commit-bot: I haz the power, 2016-04-20 20:02:04]

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

[commit-bot: I haz the power, 2016-04-20 20:02:11]

Description was changed from

==========
Revert of SkOnce: 2 bytes -> 1 byte (patchset #4 id:60001 of
https://codereview.chromium.org/1904483003/ )

Reason for revert:
bust the roll

Original issue's description:
> SkOnce: 2 bytes -> 1 byte
> 
> This uses the same logic we worked out for SkOncePtr to reduce
> the memory footprint of SkOnce from a done byte and lock byte
> to a single 3-state byte:
> 
>   - NotStarted: no thread has tried to run fn() yet
>   - Active:     a thread is running fn()
>   - Done:       fn() is complete
> 
> Threads which see Done return immediately.
> Threads which see NotStarted try to move to Active, run fn(), then move to
Done.
> Threads which see Active spin until the active thread moves to Done.
> 
> This additionally fixes a too-weak memory order bug in SkOncePtr,
> and adds a big note to explain.
> 
> BUG=skia:
> GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
> 
> Committed:
https://skia.googlesource.com/skia/+/df02d338be8e3c1c50b48a3a9faa582703a39c07

TBR=herb@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
==========

to

==========
Revert of SkOnce: 2 bytes -> 1 byte (patchset #4 id:60001 of
https://codereview.chromium.org/1904483003/ )

Reason for revert:
bust the roll

Original issue's description:
> SkOnce: 2 bytes -> 1 byte
> 
> This uses the same logic we worked out for SkOncePtr to reduce
> the memory footprint of SkOnce from a done byte and lock byte
> to a single 3-state byte:
> 
>   - NotStarted: no thread has tried to run fn() yet
>   - Active:     a thread is running fn()
>   - Done:       fn() is complete
> 
> Threads which see Done return immediately.
> Threads which see NotStarted try to move to Active, run fn(), then move to
Done.
> Threads which see Active spin until the active thread moves to Done.
> 
> This additionally fixes a too-weak memory order bug in SkOncePtr,
> and adds a big note to explain.
> 
> BUG=skia:
> GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
> 
> Committed:
https://skia.googlesource.com/skia/+/df02d338be8e3c1c50b48a3a9faa582703a39c07

TBR=herb@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Committed:
https://skia.googlesource.com/skia/+/9134686fd9ce384e9163b44942538d033b5bff11
==========

[commit-bot: I haz the power, 2016-04-20 20:02:12]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/9134686fd9ce384e9163b44942538d033b5bff11

[herb_g, 2016-04-20 20:36:16]

lgtm