RefCounted types should not have public destructors, content/browser part 1

content/browser/device_orientation/device_orientation_browsertest.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 "base/command_line.h"
 #include "base/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
@@ skipping 11 matching lines @@
         added_observer_(false),
         removed_observer_(false) {}
 
   virtual void AddObserver(Observer* observer) {
     added_observer_ = true;
     observer->OnOrientationUpdate(orientation_);
   }
   virtual void RemoveObserver(Observer* observer) {
     removed_observer_ = true;
   }
+
   Orientation orientation_;
   bool added_observer_;
   bool removed_observer_;
+
+ private:
+  virtual ~MockProvider() {}
 };
 
 class DeviceOrientationBrowserTest : public InProcessBrowserTest {
  public:
   // From InProcessBrowserTest.
   virtual void SetUpCommandLine(CommandLine* command_line) {
     EXPECT_TRUE(!command_line->HasSwitch(switches::kDisableDeviceOrientation));
   }
 
   GURL testUrl(const FilePath::CharType* filename) {
@@ skipping 17 matching lines @@
                                                             2);
 
   // Check that the page got the event it expected and that the provider
   // saw requests for adding and removing an observer.
   EXPECT_EQ("pass", browser()->GetSelectedWebContents()->GetURL().ref());
   EXPECT_TRUE(provider->added_observer_);
   EXPECT_TRUE(provider->removed_observer_);
 }
 
 } //  namespace device_orientation