Split atomic and mutex implementations and make inlinable.

src/ports/SkMutex_none.h

Index: src/ports/SkMutex_none.h
===================================================================
--- src/ports/SkMutex_none.h	(revision 0)
+++ src/ports/SkMutex_none.h	(working copy)
@@ -0,0 +1,30 @@
+/*
+ * 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_none_DEFINED
+#define SkMutex_none_DEFINED
+
+/** Non-mutex mutex for uniprocessor systems. */
+
+struct SkBaseMutex {
+    void acquire() { }
+    void release() { }
+};
+
+class SkMutex : public SkBaseMutex, SkNoncopyable {
+public:
+    SkMutex() { }
+    ~SkMutex() { }
+};
+
+// Using POD-style initialization prevents the generation of a static initializer.
+#define SK_DECLARE_STATIC_MUTEX(name) static SkBaseMutex name = { }

[djsollen, 2013/07/25 16:00:34]

do you need the '{ }' here?

[bungeman-skia, 2013/09/26 17:16:23]

Hmmmm... I suppose this was just here to 0 initialize the struct (and to
parallel the construction in the _posix version). However, seeing as there is no
state in this struct, it doesn't seem like it should make a difference. On the
other hand, having the braces here also helps to prevent mis-use, it pretty much
forces the user to put a ';' after it.

+
+// Special case used when the static mutex must be available globally.
+#define SK_DECLARE_GLOBAL_MUTEX(name) SkBaseMutex name = { }
+
+#endif