Refcounted types should not have public destructors, remoting/ edition

remoting/host/screen_recorder.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 "remoting/host/screen_recorder.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/callback.h"
@@ skipping 34 matching lines @@
       encoder_stopped_(false),
       max_recordings_(kMaxRecordings),
       recordings_(0),
       frame_skipped_(false),
       sequence_number_(0) {
   DCHECK(capture_loop_);
   DCHECK(encode_loop_);
   DCHECK(network_loop_);
 }
 
-ScreenRecorder::~ScreenRecorder() {
-}
-
 // Public methods --------------------------------------------------------------
 
 void ScreenRecorder::Start() {
   capture_loop_->PostTask(
       FROM_HERE, base::Bind(&ScreenRecorder::DoStart, this));
 }
 
 void ScreenRecorder::Stop(const base::Closure& done_task) {
   if (MessageLoop::current() != capture_loop_) {
     capture_loop_->PostTask(FROM_HERE, base::Bind(
@@ skipping 37 matching lines @@
   if (MessageLoop::current() != capture_loop_) {
     capture_loop_->PostTask(
         FROM_HERE, base::Bind(&ScreenRecorder::UpdateSequenceNumber,
                               this, sequence_number));
     return;
   }
 
   sequence_number_ = sequence_number;
 }
 
-// Private accessors -----------------------------------------------------------
+// Private methods -----------------------------------------------------------
+
+ScreenRecorder::~ScreenRecorder() {
+}
 
 Capturer* ScreenRecorder::capturer() {
   DCHECK_EQ(capture_loop_, MessageLoop::current());
   DCHECK(capturer_);
   return capturer_;
 }
 
 Encoder* ScreenRecorder::encoder() {
   DCHECK_EQ(encode_loop_, MessageLoop::current());
   DCHECK(encoder_.get());
@@ skipping 205 matching lines @@
     packet->set_encode_time_ms(encode_time_ms);
     scheduler_.RecordEncodeTime(encode_time);
   }
 
   network_loop_->PostTask(
       FROM_HERE, base::Bind(&ScreenRecorder::DoSendVideoPacket, this,
                             base::Passed(packet.Pass())));
 }
 
 }  // namespace remoting