Remove use of deprecated methods to cricket::VideoCapturer in WebrtcVideoCapturerAdapter

remoting/protocol/webrtc_video_capturer_adapter.cc

 // Copyright 2015 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/protocol/webrtc_video_capturer_adapter.h"
 
 #include <utility>
 
 #include "third_party/libyuv/include/libyuv/convert.h"
 #include "third_party/webrtc/media/engine/webrtcvideoframe.h"
@@ skipping 53 matching lines @@
   desktop_capturer_->Start(this);
 
   capture_timer_.reset(new base::RepeatingTimer());
   capture_timer_->Start(FROM_HERE,
                         base::TimeDelta::FromSeconds(1) / kFramesPerSec, this,
                         &WebrtcVideoCapturerAdapter::CaptureNextFrame);
 
   return cricket::CS_RUNNING;
 }
 
-// Similar to the base class implementation with some important differences:
-// 1. Does not call either Stop() or Start(), as those would affect the state of
-// |desktop_capturer_|.
-// 2. Does not support unpausing after stopping the capturer. It is unclear
-// if that flow needs to be supported.
-bool WebrtcVideoCapturerAdapter::Pause(bool pause) {
+bool WebrtcVideoCapturerAdapter::PauseCapturer(bool pause) {
   DCHECK(thread_checker_.CalledOnValidThread());
 
   if (pause) {
-    if (capture_state() == cricket::CS_PAUSED)
+    if (paused_)
       return true;
 
     bool running = capture_state() == cricket::CS_STARTING ||
                    capture_state() == cricket::CS_RUNNING;
 
     DCHECK_EQ(running, IsRunning());
 
     if (!running) {
       LOG(ERROR)
           << "Cannot pause WebrtcVideoCapturerAdapter.";
       return false;
     }
 
-    // Stop |capture_timer_| and set capture state to cricket::CS_PAUSED.
     capture_timer_->Stop();
-    SetCaptureState(cricket::CS_PAUSED);
+    paused_ = true;
 
     VLOG(1) << "WebrtcVideoCapturerAdapter paused.";
 
     return true;
   } else {  // Unpausing.
-    if (capture_state() != cricket::CS_PAUSED || !GetCaptureFormat() ||
-        !capture_timer_) {
+    if (!paused_ || !GetCaptureFormat() || !capture_timer_) {
       LOG(ERROR) << "Cannot unpause WebrtcVideoCapturerAdapter.";
       return false;
     }
 
-    // Restart |capture_timer_| and set capture state to cricket::CS_RUNNING;
     capture_timer_->Start(FROM_HERE,
                           base::TimeDelta::FromMicroseconds(
                               GetCaptureFormat()->interval /
                               (base::Time::kNanosecondsPerMicrosecond)),
                           this,
                           &WebrtcVideoCapturerAdapter::CaptureNextFrame);
-    SetCaptureState(cricket::CS_RUNNING);
+    paused_ = false;
 
     VLOG(1) << "WebrtcVideoCapturerAdapter unpaused.";
   }
   return true;
 }
 
 void WebrtcVideoCapturerAdapter::Stop() {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK_NE(capture_state(), cricket::CS_STOPPED);
 
@@ skipping 87 matching lines @@
             left * webrtc::DesktopFrame::kBytesPerPixel,
         frame->stride(),
         yuv_frame_->GetYPlane() + yuv_frame_->GetYPitch() * top + left,
         yuv_frame_->GetYPitch(),
         yuv_frame_->GetUPlane() + yuv_frame_->GetUPitch() * top / 2 + left / 2,
         yuv_frame_->GetUPitch(),
         yuv_frame_->GetVPlane() + yuv_frame_->GetVPitch() * top / 2 + left / 2,
         yuv_frame_->GetVPitch(), width, height);
   }
 
-  SignalVideoFrame(this, yuv_frame_.get());
+  OnFrame(this, yuv_frame_.get());
 }
 
 void WebrtcVideoCapturerAdapter::CaptureNextFrame() {
   DCHECK(thread_checker_.CalledOnValidThread());
 
   if (capture_pending_)
     return;
   capture_pending_ = true;
   desktop_capturer_->Capture(webrtc::DesktopRegion());
 }
 
 }  // namespace protocol
 }  // namespace remoting