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

chrome/browser/password_manager/password_store_win.cc

 // 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.
 
 #include "chrome/browser/password_manager/password_store_win.h"
 
 #include <map>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 21 matching lines @@
   // We hold a copy of the |form| used in GetLoginsImpl as a pointer.  If the
   // form is not set (is NULL), then we are not a GetLogins request.
   void SetLoginsRequestForm(const PasswordForm& form) {
     form_.reset(new PasswordForm(form));
   }
   PasswordForm* form() const {
     return form_.get();
   }
   bool IsLoginsRequest() const { return !!form_.get(); }
 
+ protected:
+  virtual ~FormGetLoginsRequest() {}
+
  private:
   scoped_ptr<PasswordForm> form_;
 };
 
 }  // namespace
 
 // Handles requests to WebDataService.
 class PasswordStoreWin::DBHandler : public WebDataServiceConsumer {
  public:
   DBHandler(WebDataService* web_data_service,
@@ skipping 151 matching lines @@
     PasswordStore::ForwardLoginsResult(request);
   }
 }
 
 void PasswordStoreWin::GetLoginsImpl(GetLoginsRequest* request,
                                      const PasswordForm& form) {
   static_cast<FormGetLoginsRequest*>(request)->SetLoginsRequestForm(form);
 
   PasswordStoreDefault::GetLoginsImpl(request, form);
 }