RefCounted types should not have public destructors, chrome/ remaining parts

chrome/service/cloud_print/print_system_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/service/cloud_print/print_system.h"
 
 #include <objidl.h>
 #include <winspool.h>
 #include <xpsprint.h>
 
@@ skipping 272 matching lines @@
     }
 
     // PrintSystemWatcherWin::Delegate implementation.
     virtual void OnPrinterAdded() OVERRIDE {
       delegate_->OnPrinterAdded();
     }
     virtual void OnPrinterDeleted() OVERRIDE {}
     virtual void OnPrinterChanged() OVERRIDE {}
     virtual void OnJobChanged() OVERRIDE {}
 
+   protected:
+    virtual ~PrintServerWatcherWin() {}
+
    private:
     PrintSystem::PrintServerWatcher::Delegate* delegate_;
     PrintSystemWatcherWin watcher_;
 
     DISALLOW_COPY_AND_ASSIGN(PrintServerWatcherWin);
   };
 
   class PrinterWatcherWin
       : public PrintSystem::PrinterWatcher,
         public PrintSystemWatcherWinDelegate {
@@ skipping 28 matching lines @@
     virtual void OnPrinterDeleted() OVERRIDE {
       delegate_->OnPrinterDeleted();
     }
     virtual void OnPrinterChanged() OVERRIDE {
       delegate_->OnPrinterChanged();
     }
     virtual void OnJobChanged() OVERRIDE {
       delegate_->OnJobChanged();
     }
 
+   protected:
+    virtual ~PrinterWatcherWin() {}
+
    private:
     std::string printer_name_;
     PrintSystem::PrinterWatcher::Delegate* delegate_;
     PrintSystemWatcherWin watcher_;
 
     DISALLOW_COPY_AND_ASSIGN(PrinterWatcherWin);
   };
 
   class JobSpoolerWin : public PrintSystem::JobSpooler {
    public:
@@ skipping 10 matching lines @@
       // TODO(gene): add tags handling.
       scoped_refptr<printing::PrintBackend> print_backend(
           printing::PrintBackend::CreateInstance(NULL));
       child_process_logging::ScopedPrinterInfoSetter prn_info(
           print_backend->GetPrinterDriverInfo(printer_name));
       return core_->Spool(print_ticket, print_data_file_path,
                           print_data_mime_type, printer_name, job_title,
                           delegate);
     }
 
+   protected:
+    virtual ~JobSpoolerWin() {}
+
    private:
     // We use a Core class because we want a separate RefCountedThreadSafe
     // implementation for ServiceUtilityProcessHost::Client.
     class Core : public ServiceUtilityProcessHost::Client,
                  public base::win::ObjectWatcher::Delegate {
      public:
       Core()
           : last_page_printed_(-1),
             job_id_(-1),
             delegate_(NULL),
@@ skipping 508 matching lines @@
   RpcStringFree(reinterpret_cast<RPC_WSTR *>(&proxy_id_as_string));
   return ret;
 }
 
 scoped_refptr<PrintSystem> PrintSystem::CreateInstance(
     const base::DictionaryValue* print_system_settings) {
   return new PrintSystemWin;
 }
 
 }  // namespace cloud_print