Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(895)

Unified Diff: content/renderer/media/media_stream_video_capturer_source.h

Issue 2970823003: Rename MediaStreamVideoSource:GetCurrentFormatImpl() to GetCurrentFormat() (Closed)
Patch Set: Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/media/media_stream_video_capturer_source.h
diff --git a/content/renderer/media/media_stream_video_capturer_source.h b/content/renderer/media/media_stream_video_capturer_source.h
index a153fdb8de38dce72171b3c908c0b31d628cc400..f4fa6067b584f97812c85e10775574448ce8b092 100644
--- a/content/renderer/media/media_stream_video_capturer_source.h
+++ b/content/renderer/media/media_stream_video_capturer_source.h
@@ -51,8 +51,7 @@ class CONTENT_EXPORT MediaStreamVideoCapturerSource
void StartSourceImpl(
const VideoCaptureDeliverFrameCB& frame_callback) override;
void StopSourceImpl() override;
- base::Optional<media::VideoCaptureFormat> GetCurrentFormatImpl()
- const override;
+ base::Optional<media::VideoCaptureFormat> GetCurrentFormat() const override;
hbos_chromium 2017/07/04 14:34:29 nit: Shouldn't this be moved to the public section
Guido Urdaneta 2017/07/05 15:28:25 no need.
// RenderFrameObserver implementation.
void OnDestruct() final {}

Powered by Google App Engine
This is Rietveld 408576698