RefCounted types should not have public destructors, printing/ and ui/

printing/backend/print_backend_chromeos.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 "printing/backend/print_backend.h"
 
 #include "base/logging.h"
 
 namespace printing {
 
 // Provides a stubbed out PrintBackend implementation for use on ChromeOS.
 class PrintBackendChromeOS : public PrintBackend {
  public:
   PrintBackendChromeOS();
-  virtual ~PrintBackendChromeOS() {}
 
   // PrintBackend implementation.
-  virtual bool EnumeratePrinters(PrinterList* printer_list);
+  virtual bool EnumeratePrinters(PrinterList* printer_list) OVERRIDE;
+  virtual std::string GetDefaultPrinterName() OVERRIDE;
+  virtual bool GetPrinterCapsAndDefaults(
+      const std::string& printer_name,
+      PrinterCapsAndDefaults* printer_info) OVERRIDE;
+  virtual std::string GetPrinterDriverInfo(
+      const std::string& printer_name) OVERRIDE;
+  virtual bool IsValidPrinter(const std::string& printer_name) OVERRIDE;
 
-  virtual std::string GetDefaultPrinterName();
-
-  virtual bool GetPrinterCapsAndDefaults(const std::string& printer_name,
-                                         PrinterCapsAndDefaults* printer_info);
-
-  virtual std::string GetPrinterDriverInfo(const std::string& printer_name);
-
-  virtual bool IsValidPrinter(const std::string& printer_name);
-
- private:
+ protected:
+  virtual ~PrintBackendChromeOS() {}
 };
 
 PrintBackendChromeOS::PrintBackendChromeOS() {}
 
 bool PrintBackendChromeOS::EnumeratePrinters(PrinterList* printer_list) {
   return true;
 }
 
 
 
@@ skipping 19 matching lines @@
   return true;
 }
 
 scoped_refptr<PrintBackend> PrintBackend::CreateInstance(
     const base::DictionaryValue* print_backend_settings) {
   return new PrintBackendChromeOS();
 }
 
 }  // namespace printing