Rename MediaStreamVideoSource:GetCurrentFormatImpl() to GetCurrentFormat()

content/renderer/media/mock_media_stream_video_source.cc

Index: content/renderer/media/mock_media_stream_video_source.cc
diff --git a/content/renderer/media/mock_media_stream_video_source.cc b/content/renderer/media/mock_media_stream_video_source.cc
index 379e430d0268129f57c2f6c7723af513df52d9a4..980c819ede76a98c9ec495f9bd36ca753fe1d839 100644
--- a/content/renderer/media/mock_media_stream_video_source.cc
+++ b/content/renderer/media/mock_media_stream_video_source.cc
@@ -67,7 +67,7 @@ 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.

 }