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

chrome/browser/prefs/command_line_pref_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_PREFS_COMMAND_LINE_PREF_STORE_H_
 #define CHROME_BROWSER_PREFS_COMMAND_LINE_PREF_STORE_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "base/command_line.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/values.h"
 #include "chrome/browser/prefs/value_map_pref_store.h"
 
 // This PrefStore keeps track of preferences set by command-line switches,
 // such as proxy settings.
 class CommandLinePrefStore : public ValueMapPrefStore {
  public:
   explicit CommandLinePrefStore(const CommandLine* command_line);
+
+ protected:
   virtual ~CommandLinePrefStore();
 
- protected:
   // Logs a message and returns false if the proxy switches are
   // self-contradictory. Protected so it can be used in unit testing.
   bool ValidateProxySwitches();
 
  private:
+  friend class TestCommandLinePrefStore;
+
   struct StringSwitchToPreferenceMapEntry {
     const char* switch_name;
     const char* preference_path;
   };
 
   struct IntegerSwitchToPreferenceMapEntry {
     const char* switch_name;
     const char* preference_path;
   };
 
@@ skipping 23 matching lines @@
 
   // Mappings of command line switches to prefs.
   static const BooleanSwitchToPreferenceMapEntry boolean_switch_map_[];
   static const StringSwitchToPreferenceMapEntry string_switch_map_[];
   static const IntegerSwitchToPreferenceMapEntry integer_switch_map_[];
 
   DISALLOW_COPY_AND_ASSIGN(CommandLinePrefStore);
 };
 
 #endif  // CHROME_BROWSER_PREFS_COMMAND_LINE_PREF_STORE_H_