RefCounted types should not have public destructors, chrome/browser/ part 5

chrome/browser/password_manager/password_store.h

 // Copyright (c) 2012 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 CHROME_BROWSER_PASSWORD_MANAGER_PASSWORD_STORE_H_
 #define CHROME_BROWSER_PASSWORD_MANAGER_PASSWORD_STORE_H_
 #pragma once
 
 #include <vector>
 
@@ skipping 44 matching lines @@
   // TODO(scr) If we can convert vector<PasswordForm*> to
   // ScopedVector<PasswordForm>, then we can move the following class to merely
   // a typedef. At the moment, a subclass of CancelableRequest1 is required to
   // provide a destructor, which cleans up after canceled requests by deleting
   // vector elements.
   class GetLoginsRequest
       : public CancelableRequest1<GetLoginsCallback,
                                   std::vector<webkit::forms::PasswordForm*> > {
    public:
     explicit GetLoginsRequest(const GetLoginsCallback& callback);
+
+   protected:
     virtual ~GetLoginsRequest();
 
    private:
     DISALLOW_COPY_AND_ASSIGN(GetLoginsRequest);
   };
 
   // An interface used to notify clients (observers) of this object that data in
   // the password store has changed. Register the observer via
   // PasswordStore::SetObserver.
   class Observer {
@@ skipping 135 matching lines @@
   // may have been changed.
   void NotifyLoginsChanged();
 
   // The observers.
   ObserverList<Observer> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(PasswordStore);
 };
 
 #endif  // CHROME_BROWSER_PASSWORD_MANAGER_PASSWORD_STORE_H_