SkNonCopyable should be used with private inheritance.

include/core/SkTInternalLList.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 SkTInternalLList_DEFINED
 #define SkTInternalLList_DEFINED
 
@@ skipping 20 matching lines @@
 #define SK_DECLARE_INTERNAL_LLIST_INTERFACE(ClassName)              \
     friend class SkTInternalLList<ClassName>;                       \
     /* back pointer to the owning list - for debugging */           \
     SkDEBUGCODE(SkPtrWrapper<SkTInternalLList<ClassName> > fList;)  \
     SkPtrWrapper<ClassName> fPrev;                                  \
     SkPtrWrapper<ClassName> fNext
 
 /**
  * This class implements a templated internal doubly linked list data structure.
  */
-template <class T> class SkTInternalLList : public SkNoncopyable {
+template <class T> class SkTInternalLList : SkNoncopyable {
 public:
     SkTInternalLList()
         : fHead(NULL)
         , fTail(NULL) {
     }
 
     void remove(T* entry) {
         SkASSERT(NULL != fHead && NULL != fTail);
         SkASSERT(this->isInList(entry));
 
@@ skipping 211 matching lines @@
 #endif // SK_DEBUG
 
 private:
     T* fHead;
     T* fTail;
 
     typedef SkNoncopyable INHERITED;
 };
 
 #endif