Grant independence to SkBaseMutex on Windows.

src/ports/SkMutex_win.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_win_DEFINED
 #define SkMutex_win_DEFINED
 
@@ skipping 13 matching lines @@
 #ifdef WIN32_IS_MEAN_WAS_LOCALLY_DEFINED
 #  undef WIN32_IS_MEAN_WAS_LOCALLY_DEFINED
 #  undef WIN32_LEAN_AND_MEAN
 #endif
 #ifdef NOMINMAX_WAS_LOCALLY_DEFINED
 #  undef NOMINMAX_WAS_LOCALLY_DEFINED
 #  undef NOMINMAX
 #endif
 
 // On Windows, SkBaseMutex and SkMutex are the same thing,
-// we can't easily get rid of static initializers.
-class SkMutex {
+// we can't easily get rid of static initializers. However,
+// we preserve the same inheritance pattern as other platforms
+// so that we can forward-declare cleanly.
+struct SkBaseMutex {

[mtklein, 2014/06/30 22:24:01]

Don't you have to do it the other way around?  SkBaseMutex has now lost its
acquire() and release() methods, among other things.

struct SkBaseMutex {
  ... real implementation ...
};
class SkMutex : public SkBaseMutex {};

ought to work?

+
+};
+
+class SkMutex : public SkBaseMutex {
 public:
     SkMutex() {
         InitializeCriticalSection(&fStorage);
         SkDEBUGCODE(fOwner = 0;)
     }
 
     ~SkMutex() {
         SkASSERT(0 == fOwner);
         DeleteCriticalSection(&fStorage);
     }
@@ skipping 14 matching lines @@
     }
 
 private:
     SkMutex(const SkMutex&);
     SkMutex& operator=(const SkMutex&);
 
     CRITICAL_SECTION fStorage;
     SkDEBUGCODE(DWORD fOwner;)
 };
 
-typedef SkMutex SkBaseMutex;
-
 // Windows currently provides no documented means of POD initializing a CRITICAL_SECTION.
 #define SK_DECLARE_STATIC_MUTEX(name) static SkBaseMutex name
 #define SK_DECLARE_GLOBAL_MUTEX(name) SkBaseMutex name
 
 #endif