Add guards around access to font cache global fields.

Add guards around access to font cache global fields.

BUG=589439
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2082423002

Committed: https://skia.googlesource.com/skia/+/632d004677de66a985d05c25a25e25272c740ede

[herb_g, 2016-06-22 15:40:01]

Description was changed from

==========
Add guards around access to font cache global fields.

BUG=589439
==========

to

==========
Add guards around access to font cache global fields.

BUG=589439
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2082423002
==========

[herb_g, 2016-06-22 15:41:11]

Description was changed from

==========
Add guards around access to font cache global fields.

BUG=589439
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2082423002
==========

to

==========
Add guards around access to font cache global fields.

BUG=589439
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2082423002
==========

[herb_g, 2016-06-22 15:41:11]

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

[herb_g, 2016-06-22 15:42:36]

This is an attempt to fix the long standing bug in chrome. This is actually a
benign race, which is only caused by a test.

[mtklein, 2016-06-22 15:45:42]

lgtm

[herb_g, 2016-06-22 15:50:59]

The CQ bit was checked by herb@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-06-22 15:51:04]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2082423002/1

[reed1, 2016-06-22 16:05:35]

reed@google.com changed reviewers:
+ reed@google.com

[reed1, 2016-06-22 16:05:36]

https://codereview.chromium.org/2082423002/diff/1/src/core/SkGlyphCache_Globa...
File src/core/SkGlyphCache_Globals.h (right):

https://codereview.chromium.org/2082423002/diff/1/src/core/SkGlyphCache_Globa...
src/core/SkGlyphCache_Globals.h:45: mutable SkSpinlock     fLock;
make private?

[commit-bot: I haz the power, 2016-06-22 16:08:26]

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

[commit-bot: I haz the power, 2016-06-22 16:08:27]

Dry run: This issue passed the CQ dry run.

[herb_g, 2016-06-22 17:18:15]

The CQ bit was checked by herb@google.com

[commit-bot: I haz the power, 2016-06-22 17:18:22]

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

[commit-bot: I haz the power, 2016-06-22 17:19:18]

Description was changed from

==========
Add guards around access to font cache global fields.

BUG=589439
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2082423002
==========

to

==========
Add guards around access to font cache global fields.

BUG=589439
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2082423002

Committed:
https://skia.googlesource.com/skia/+/632d004677de66a985d05c25a25e25272c740ede
==========

[commit-bot: I haz the power, 2016-06-22 17:19:19]

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