Make a single GrSingleOwner in GrContext

Make a single GrSingleOwner in GrContext

TBR=bsalomon@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1563703004

Committed: https://skia.googlesource.com/skia/+/f9bc796e0dbd31674c22b34761913ee6e8fdd66a

[joshua.litt, 2016-01-06 19:06:24]

Description was changed from

==========
Make a single GrSingleOwner in GrContext

BUG=skia:
==========

to

==========
Make a single GrSingleOwner in GrContext

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[joshualitt, 2016-01-06 19:06:54]

joshualitt@google.com changed reviewers:
+ joshualitt@google.com, robertphillips@google.com

[joshualitt, 2016-01-06 19:06:54]

I'm open to alternative ideas, this one seems not so bad.

[joshualitt, 2016-01-07 16:04:13]

On 2016/01/06 19:06:54, joshualitt wrote:
> I'm open to alternative ideas, this one seems not so bad.

ping

[robertphillips, 2016-01-07 16:44:11]

lgtm + 2 documentation suggestions

https://codereview.chromium.org/1563703004/diff/1/include/gpu/GrContext.h
File include/gpu/GrContext.h (right):

https://codereview.chromium.org/1563703004/diff/1/include/gpu/GrContext.h#new...
include/gpu/GrContext.h:394: // In debug builds we guard against improper thread
handling
// This guard is passed to the GrDrawingManager and, from there, to all the
GrDrawContexts. 

?

https://codereview.chromium.org/1563703004/diff/1/src/gpu/GrDrawingManager.h
File src/gpu/GrDrawingManager.h (right):

https://codereview.chromium.org/1563703004/diff/1/src/gpu/GrDrawingManager.h#...
src/gpu/GrDrawingManager.h:82: GrDrawTarget::Options      
fOptionsForDrawTargets;
// In debug builds we guard against improper thread handling

[joshualitt, 2016-01-07 18:59:58]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2016-01-07 18:59:59]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/1563703004/#ps20001
(title: "tweaks")

[commit-bot: I haz the power, 2016-01-07 19:00:01]

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

[commit-bot: I haz the power, 2016-01-07 19:01:03]

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

[commit-bot: I haz the power, 2016-01-07 19:01:04]

Try jobs failed on following builders:
  skia_presubmit-Trybot on client.skia.fyi (JOB_FAILED,
http://build.chromium.org/p/client.skia.fyi/builders/skia_presubmit-Trybot/bu...)

[joshualitt, 2016-01-07 19:15:10]

Description was changed from

==========
Make a single GrSingleOwner in GrContext

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Make a single GrSingleOwner in GrContext

TBR=bsalomon@google.com
BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[joshualitt, 2016-01-07 19:15:19]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2016-01-07 19:15:19]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/1563703004/#ps40001
(title: "nit")

[commit-bot: I haz the power, 2016-01-07 19:15:28]

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

[commit-bot: I haz the power, 2016-01-07 19:32:41]

Description was changed from

==========
Make a single GrSingleOwner in GrContext

TBR=bsalomon@google.com
BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Make a single GrSingleOwner in GrContext

TBR=bsalomon@google.com
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/+/f9bc796e0dbd31674c22b34761913ee6e8fdd66a
==========

[commit-bot: I haz the power, 2016-01-07 19:32:42]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/f9bc796e0dbd31674c22b34761913ee6e8fdd66a

[joshualitt, 2016-01-07 21:17:56]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/1565303003/ by joshualitt@google.com.

The reason for reverting is: breaking asan .