SkNonCopyable should be used with private inheritance.

include/gpu/GrCoordTransform.h

Index: include/gpu/GrCoordTransform.h
diff --git a/include/gpu/GrCoordTransform.h b/include/gpu/GrCoordTransform.h
index 14e2620e44b1eef6fe8743f6d4a74c8f90942e0f..f266577ec62b8d027f307b594a1ab1e574b59402 100644
--- a/include/gpu/GrCoordTransform.h
+++ b/include/gpu/GrCoordTransform.h
@@ -39,7 +39,7 @@ enum GrCoordSet {
  * position). GrEffects just define these transformations, and the framework does the rest of the
  * work to make the transformed coordinates available in their fragment shader.
  */
-class GrCoordTransform : public SkNoncopyable {
+class GrCoordTransform : SkNoncopyable {
 public:
     GrCoordTransform() { SkDEBUGCODE(fInEffect = false); }