SkNonCopyable should be used with private inheritance.

include/core/SkTypes.h

Index: include/core/SkTypes.h
diff --git a/include/core/SkTypes.h b/include/core/SkTypes.h
index 9b376041066fe2ecffd964cc175e9d7a7ad64ae8..8bfb4c357a50f671101b34b2b0c4f99d20c1ff23 100644
--- a/include/core/SkTypes.h
+++ b/include/core/SkTypes.h
@@ -486,7 +486,7 @@ private:
  *  the lifetime of the block, so the caller must not call sk_free() or delete
  *  on the block, unless detach() was called.
  */
-class SkAutoMalloc : public SkNoncopyable {
+class SkAutoMalloc : SkNoncopyable {
 public:
     explicit SkAutoMalloc(size_t size = 0) {
         fPtr = size ? sk_malloc_throw(size) : NULL;