SkNonCopyable should be used with private inheritance.

src/gpu/GrOrderedSet.h

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef GrOrderedSet_DEFINED
 #define GrOrderedSet_DEFINED
 
 #include "GrRedBlackTree.h"
 
 template <typename T, typename C = GrLess<T> >
-class GrOrderedSet : public SkNoncopyable {
+class GrOrderedSet : SkNoncopyable {
 public:
     /**
      * Creates an empty set
      */
     GrOrderedSet() : fComp() {}
     ~GrOrderedSet() {}
 
     class Iter;
 
     /**
@@ skipping 120 matching lines @@
 }
 
 template <typename T, typename C>
 void GrOrderedSet<T,C>::remove(const typename GrOrderedSet<T,C>::Iter& iter) {
     if (this->end() != iter) {
         fRBTree.remove(iter.getTreeIter());
     }
 }
 
 #endif