Add guards to GrTextureProvider

Add guards to GrTextureProvider

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

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

[joshua.litt, 2016-01-07 19:00:49]

Description was changed from

==========
Add guards to GrTextureProvider

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

to

==========
Add guards to GrTextureProvider

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

[joshualitt, 2016-01-07 19:02:32]

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

[joshualitt, 2016-01-07 19:02:32]

There are a couple of inlined functions in the header, I can add guards there if
you prefer.

[robertphillips, 2016-01-07 19:07:02]

https://codereview.chromium.org/1567983002/diff/1/include/gpu/GrTextureProvid...
File include/gpu/GrTextureProvider.h (right):

https://codereview.chromium.org/1567983002/diff/1/include/gpu/GrTextureProvid...
include/gpu/GrTextureProvider.h:193: GrGpu* fGpu;
// Guard for single-threaded access. Owned by GrContext.

?

[joshualitt, 2016-01-07 19:18:09]

On 2016/01/07 19:07:02, robertphillips wrote:
>
https://codereview.chromium.org/1567983002/diff/1/include/gpu/GrTextureProvid...
> File include/gpu/GrTextureProvider.h (right):
> 
>
https://codereview.chromium.org/1567983002/diff/1/include/gpu/GrTextureProvid...
> include/gpu/GrTextureProvider.h:193: GrGpu* fGpu;
> // Guard for single-threaded access. Owned by GrContext.
> 
> ?

feedback incorporated

[robertphillips, 2016-01-07 19:19:40]

lgtm

[joshualitt, 2016-01-07 19:20:39]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2016-01-07 19:20:42]

The CQ bit was unchecked by joshualitt@google.com

[joshualitt, 2016-01-07 19:37:51]

The CQ bit was checked by joshualitt@google.com

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

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

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

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

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

Try jobs failed on following builders:
  Build-Mac10.8-Clang-x86_64-Release-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Mac10.8-Clang-...)
  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:56:53]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2016-01-07 19:56:53]

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/1567983002/#ps40001
(title: "mutable so mac doesn't complain")

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

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

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

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

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

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 20:04:36]

Description was changed from

==========
Add guards to GrTextureProvider

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

to

==========
Add guards to GrTextureProvider

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 20:34:50]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2016-01-07 20:34:50]

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/1567983002/#ps80001
(title: "got it")

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

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

[commit-bot: I haz the power, 2016-01-07 20:52:07]

Description was changed from

==========
Add guards to GrTextureProvider

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

to

==========
Add guards to GrTextureProvider

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/+/b30dd1db1d914b85a691b4724713ba1b0f16cd6c
==========

[commit-bot: I haz the power, 2016-01-07 20:52:08]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/b30dd1db1d914b85a691b4724713ba1b0f16cd6c

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

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

The reason for reverting is: breaking asan bot.