Rename MediaStreamVideoSource:GetCurrentFormatImpl() to GetCurrentFormat()

content/renderer/media/mock_media_stream_video_source.cc

 // Copyright 2014 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 "content/renderer/media/mock_media_stream_video_source.h"
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/location.h"
 
@@ skipping 49 matching lines @@
     const VideoCaptureDeliverFrameCB& frame_callback) {
   DCHECK(frame_callback_.is_null());
   attempted_to_start_ = true;
   frame_callback_ = frame_callback;
 }
 
 void MockMediaStreamVideoSource::StopSourceImpl() {
 }
 
 base::Optional<media::VideoCaptureFormat>
-MockMediaStreamVideoSource::GetCurrentFormatImpl() const {
+MockMediaStreamVideoSource::GetCurrentFormat() const {
   return base::Optional<media::VideoCaptureFormat>(format_);

[hbos_chromium, 2017/07/04 14:34:29]

We DCHECK with the sequence_checker_, it's protected.

[Guido Urdaneta, 2017/07/05 15:28:25]

There is no sequence_checker_ in this class. Did you mean something else?

[hbos_chromium, 2017/07/05 15:33:30]

There's SEQUENCE_CHECKER(sequence_checker_) in MediaStreamVideoSource's
protected section.
https://cs.chromium.org/chromium/src/content/renderer/media/media_stream_vide...

[Guido Urdaneta, 2017/07/05 16:03:46]

Done. Also filed crbug.com/739393 to add sequence checks to MSVCS and its
internal classes.

 }
 
 void MockMediaStreamVideoSource::DeliverVideoFrame(
     const scoped_refptr<media::VideoFrame>& frame) {
   DCHECK(!frame_callback_.is_null());
   io_task_runner()->PostTask(
       FROM_HERE, base::Bind(frame_callback_, frame, base::TimeTicks()));
 }
 
 }  // namespace content