Fix thread unsafe mutex initialization.

src/ports/SkMutex_pthread.h

 /*
  * Copyright 2013 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkMutex_pthread_DEFINED
 #define SkMutex_pthread_DEFINED
 
@@ skipping 71 matching lines @@
         default:
             SkDebugf("pthread error [%d] unknown\n", status);
             break;
         }
     }
 };
 
 #define SK_BASE_MUTEX_INIT { PTHREAD_MUTEX_INITIALIZER, SkDEBUGCODE(0) }
 
 // Using POD-style initialization prevents the generation of a static initializer.
-#define SK_DECLARE_STATIC_MUTEX(name) static SkBaseMutex name = SK_BASE_MUTEX_INIT
-
-// Special case used when the static mutex must be available globally.
-#define SK_DECLARE_GLOBAL_MUTEX(name) SkBaseMutex name = SK_BASE_MUTEX_INIT
+#define SK_DECLARE_STATIC_MUTEX(name) static void SK_MACRO_APPEND_LINE(name)(); static SkBaseMutex name = SK_BASE_MUTEX_INIT

[mtklein, 2014/07/25 12:35:33]

Add a comment about the function declaration here too?

[bungeman-skia, 2014/07/25 14:40:40]

Hmmm... this one is actually safe (well, in practice though not according to
spec). Since this is POD all existing compilers will just compile time init such
a thing. Maybe the comment should go in SkThread.h, though that is a bit far
from the actual 'definition' of this macro.

 
 #endif