commit | 086eea93b1f8253420c233c33cdabc0e74b08091 | [log] [tgz] |
---|---|---|
author | reed <reed@google.com> | Wed May 04 17:12:46 2016 -0700 |
committer | Commit bot <commit-bot@chromium.org> | Wed May 04 17:12:46 2016 -0700 |
tree | 34a4f19d22263c547d2a53734caf179797fb9b53 | |
parent | 5e56cfd3fa1041dbb83899844fb92fa9a2ef1009 [diff] [blame] |
Revert of SK_DECLARE_STATIC_MUTEX -> static SkMutex (patchset #1 id:1 of https://codereview.chromium.org/1948193002/ ) Reason for revert: broken the Mac and Linux builders, e.g.: https://build.chromium.org/p/chromium/builders/Mac/builds/15151 https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/19052 Original issue's description: > SK_DECLARE_STATIC_MUTEX -> static SkMutex > > There's no need to use a macro to declare static SkMutexes any more > (and there's likewise no need to restrict them to global scope). > > BUG=skia: > GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1948193002 > > Committed: https://skia.googlesource.com/skia/+/5e56cfd3fa1041dbb83899844fb92fa9a2ef1009 TBR=mtklein@google.com,mtklein@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review-Url: https://codereview.chromium.org/1945353003
diff --git a/tools/sk_tool_utils_font.cpp b/tools/sk_tool_utils_font.cpp index 67f86d1..adbfa16 100644 --- a/tools/sk_tool_utils_font.cpp +++ b/tools/sk_tool_utils_font.cpp
@@ -27,7 +27,7 @@ } } -static SkMutex gTestFontMutex; +SK_DECLARE_STATIC_MUTEX(gTestFontMutex); SkTypeface* create_font(const char* name, SkTypeface::Style style) { SkTestFontData* fontData = nullptr;