SkNonCopyable should be used with private inheritance.

include/gpu/GrContextFactory.h

 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef GrContextFactory_DEFINED
 #define GrContextFactory_DEFINED
 
@@ skipping 10 matching lines @@
 #include "GrContext.h"
 #include "SkTArray.h"
 
 /**
  * This is a simple class that is useful in test apps that use different
  * GrContexts backed by different types of GL contexts. It manages creating the
  * GL context and a GrContext that uses it. The GL/Gr contexts persist until the
  * factory is destroyed (though the caller can always grab a ref on the returned
  * Gr and GL contexts to make them outlive the factory).
  */
-class GrContextFactory : public SkNoncopyable {
+class GrContextFactory : SkNoncopyable {
 public:
     /**
      * Types of GL contexts supported. For historical and testing reasons the native GrContext will
      * not use "GL_NV_path_rendering" even when the driver supports it. There is a separate context
      * type that does not remove NVPR support and which will fail when the driver does not support
      * the extension.
      */
     enum GLContextType {
       kNative_GLContextType,
 #if SK_ANGLE
@@ skipping 146 matching lines @@
 private:
     struct GPUContext {
         GLContextType             fType;
         SkGLContextHelper*        fGLContext;
         GrContext*                fGrContext;
     };
     SkTArray<GPUContext, true> fContexts;
 };
 
 #endif