SkNonCopyable should be used with private inheritance.

src/gpu/effects/GrTextureStripAtlas.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 GrTextureStripAtlas_DEFINED
 #define GrTextureStripAtlas_DEFINED
 
@@ skipping 61 matching lines @@
 
 private:
 
     // Key to indicate an atlas row without any meaningful data stored in it
     const static uint32_t kEmptyAtlasRowKey = 0xffffffff;
 
     /**
      * The state of a single row in our cache, next/prev pointers allow these to be chained
      * together to represent LRU status
      */
-    struct AtlasRow : public SkNoncopyable {
+    struct AtlasRow : SkNoncopyable {
         AtlasRow() : fKey(kEmptyAtlasRowKey), fLocks(0), fNext(NULL), fPrev(NULL) { }
         // GenerationID of the bitmap that is represented by this row, 0xffffffff means "empty"
         uint32_t fKey;
         // How many times this has been locked (0 == unlocked)
         int32_t fLocks;
         // We maintain an LRU linked list between unlocked nodes with these pointers
         AtlasRow* fNext;
         AtlasRow* fPrev;
     };
 
@@ skipping 92 matching lines @@
                                                       const AtlasHashKey& key) {
     return entry.fKey == key;
 }
 
 inline bool GrTextureStripAtlas::AtlasHashKey::LessThan(const AtlasEntry& entry,
                                                         const AtlasHashKey& key) {
     return entry.fKey < key;
 }
 
 #endif