Add asserts for shared mutex.

Add asserts for shared mutex.

BUG=skia:

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

[herb_g, 2015-08-19 18:35:29]

herb@google.com changed reviewers:
+ mtklein@google.com

[herb_g, 2015-08-19 18:37:31]

The CQ bit was checked by herb@google.com

[commit-bot: I haz the power, 2015-08-19 18:37:47]

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

[commit-bot: I haz the power, 2015-08-19 18:37:48]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2015-08-20 00:37 UTC

[commit-bot: I haz the power, 2015-08-19 18:40:08]

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

[commit-bot: I haz the power, 2015-08-19 18:40:09]

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-...)

[mtklein, 2015-08-19 19:00:52]

https://codereview.chromium.org/1285973003/diff/20001/src/core/SkSharedMutex.h
File src/core/SkSharedMutex.h (right):

https://codereview.chromium.org/1285973003/diff/20001/src/core/SkSharedMutex....
src/core/SkSharedMutex.h:32: void assertHeld() const;
Let's do something like this to make sure these are totally no-ops in release.

#ifdef SK_DEBUG
  void assertHeld() const;
  void assertHeldShared() const;
#else
  void assertHeld() const {}
  void assertHeldShared() const {}
#endif

[herb_g, 2015-08-19 20:30:50]

https://codereview.chromium.org/1285973003/diff/20001/src/core/SkSharedMutex.h
File src/core/SkSharedMutex.h (right):

https://codereview.chromium.org/1285973003/diff/20001/src/core/SkSharedMutex....
src/core/SkSharedMutex.h:32: void assertHeld() const;
On 2015/08/19 19:00:51, mtklein wrote:
> Let's do something like this to make sure these are totally no-ops in release.
> 
> #ifdef SK_DEBUG
>   void assertHeld() const;
>   void assertHeldShared() const;
> #else
>   void assertHeld() const {}
>   void assertHeldShared() const {}
> #endif

Done.

[herb_g, 2015-08-19 20:30:55]

The CQ bit was checked by herb@google.com

[commit-bot: I haz the power, 2015-08-19 20:31:12]

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

[mtklein_C, 2015-08-19 20:39:35]

mtklein@chromium.org changed reviewers:
+ mtklein@chromium.org

[mtklein_C, 2015-08-19 20:39:35]

lgtm

[commit-bot: I haz the power, 2015-08-19 20:40:16]

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