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

content/renderer/media/capture_video_decoder_unittest.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/bind.h"
 #include "content/renderer/media/capture_video_decoder.h"
 #include "content/renderer/media/video_capture_impl.h"
 #include "content/renderer/media/video_capture_impl_manager.h"
 #include "media/base/filters.h"
 #include "media/base/limits.h"
@@ skipping 34 matching lines @@
   decoder->OnStopped(vc_impl);
 }
 
 class MockVideoCaptureImpl : public VideoCaptureImpl {
  public:
   MockVideoCaptureImpl(const media::VideoCaptureSessionId id,
                        scoped_refptr<base::MessageLoopProxy> ml_proxy,
                        VideoCaptureMessageFilter* filter)
       : VideoCaptureImpl(id, ml_proxy, filter) {
   }
-  virtual ~MockVideoCaptureImpl() {}
 
   MOCK_METHOD2(StartCapture,
                void(media::VideoCapture::EventHandler* handler,
                     const media::VideoCaptureCapability& capability));
   MOCK_METHOD1(StopCapture, void(media::VideoCapture::EventHandler* handler));
   MOCK_METHOD1(FeedBuffer, void(scoped_refptr<VideoFrameBuffer> buffer));
 
  private:
   DISALLOW_COPY_AND_ASSIGN(MockVideoCaptureImpl);
 };
 
 class MockVideoCaptureImplManager : public VideoCaptureImplManager {
  public:
   MockVideoCaptureImplManager() {}
-  virtual ~MockVideoCaptureImplManager() {}
 
   MOCK_METHOD2(AddDevice,
                media::VideoCapture*(media::VideoCaptureSessionId id,
                    media::VideoCapture::EventHandler* handler));
   MOCK_METHOD2(RemoveDevice,
                void(media::VideoCaptureSessionId id,
                    media::VideoCapture::EventHandler* handler));
 
+ protected:
+  virtual ~MockVideoCaptureImplManager() {}
+
  private:
   DISALLOW_COPY_AND_ASSIGN(MockVideoCaptureImplManager);
 };
 
 class CaptureVideoDecoderTest : public ::testing::Test {
  protected:
   CaptureVideoDecoderTest() {
     message_loop_.reset(new MessageLoop(MessageLoop::TYPE_IO));
     message_loop_proxy_ =
         base::MessageLoopProxy::current().get();
@@ skipping 119 matching lines @@
   params.session_id = kVideoStreamId;
 
   decoder_->OnDeviceInfoReceived(vc_impl_.get(), params);
   message_loop_->RunAllPending();
 
   EXPECT_EQ(expected_size.width(), decoder_->natural_size().width());
   EXPECT_EQ(expected_size.height(), decoder_->natural_size().height());
 
   Stop();
 }