Generalize InterfacePtrSet for [Associated]InterfacePtr.

mojo/public/cpp/bindings/interface_ptr_set.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MOJO_PUBLIC_CPP_BINDINGS_INTERFACE_PTR_SET_H_
 #define MOJO_PUBLIC_CPP_BINDINGS_INTERFACE_PTR_SET_H_
 
 #include <utility>
 #include <vector>
 
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
+#include "mojo/public/cpp/bindings/associated_interface_ptr.h"
 #include "mojo/public/cpp/bindings/interface_ptr.h"
 
 namespace mojo {
 
-template <typename Interface>
-class InterfacePtrSet {
+template <typename Interface, template <typename> class Ptr>
+class PtrSet {

[yzshen1, 2016/04/08 20:52:56]

Please put this in mojo::internal namespace.

[msw, 2016/04/08 21:12:14]

Done.

  public:
-  InterfacePtrSet() {}
-  ~InterfacePtrSet() { CloseAll(); }
+  PtrSet() {}
+  ~PtrSet() { CloseAll(); }
 
-  void AddInterfacePtr(InterfacePtr<Interface> ptr) {
+  void AddPtr(Ptr<Interface> ptr) {
     auto weak_interface_ptr = new Element(std::move(ptr));
     ptrs_.push_back(weak_interface_ptr->GetWeakPtr());
-    ClearNullInterfacePtrs();
+    ClearNullPtrs();
   }
 
   template <typename FunctionType>
   void ForAllPtrs(FunctionType function) {
     for (const auto& it : ptrs_) {
       if (it)
         function(it->get());
     }
-    ClearNullInterfacePtrs();
+    ClearNullPtrs();
   }
 
   void CloseAll() {
     for (const auto& it : ptrs_) {
       if (it)
         it->Close();
     }
     ptrs_.clear();
   }
 
  private:
   class Element {
    public:
-    explicit Element(InterfacePtr<Interface> ptr)
+    explicit Element(Ptr<Interface> ptr)
         : ptr_(std::move(ptr)), weak_ptr_factory_(this) {
       ptr_.set_connection_error_handler([this]() { delete this; });
     }
     ~Element() {}
 
     void Close() { ptr_.reset(); }
 
     Interface* get() { return ptr_.get(); }
 
     base::WeakPtr<Element> GetWeakPtr() {
       return weak_ptr_factory_.GetWeakPtr();
     }
 
    private:
-    InterfacePtr<Interface> ptr_;
+    Ptr<Interface> ptr_;
     base::WeakPtrFactory<Element> weak_ptr_factory_;
 
     DISALLOW_COPY_AND_ASSIGN(Element);
   };
 
-  void ClearNullInterfacePtrs() {
+  void ClearNullPtrs() {
     ptrs_.erase(std::remove_if(ptrs_.begin(), ptrs_.end(),
                                [](const base::WeakPtr<Element>& p) {
                                  return p.get() == nullptr;
                                }),
                 ptrs_.end());
   }
 
   std::vector<base::WeakPtr<Element>> ptrs_;
 };
 
+template <typename Interface>
+using InterfacePtrSet = PtrSet<Interface, InterfacePtr>;
+
+template <typename Interface>
+using AssociatedInterfacePtrSet = PtrSet<Interface, AssociatedInterfacePtr>;
+
 }  // namespace mojo
 
 #endif  // MOJO_PUBLIC_CPP_BINDINGS_INTERFACE_PTR_SET_H_