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:

[Sergey Ulanov, 2016/03/23 18:08:03]

remove this comment please.

[perkj_chrome, 2016/03/23 20:11:45]

Done.

 // 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)

[Sergey Ulanov, 2016/03/23 18:08:03]

do we still have capture_state() in CapturerAdapter()? If yes, is PAUSED state
not deprecated? Should we have paused_ flag in this class?

[perkj_chrome, 2016/03/23 20:11:45]

I should remove PAUSED as well. Thanks for pointing it out. 

       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.";
@@ skipping 122 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