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

Unified Diff: webrtc/video/video_quality_test.cc

Issue 1814763002: Cleanup of webrtc::VideoRenderer (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Use a more specific DEPS rule. Created 4 years, 9 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: webrtc/video/video_quality_test.cc
diff --git a/webrtc/video/video_quality_test.cc b/webrtc/video/video_quality_test.cc
index fb07e88a729ac3048e37c1afb62858d6c04ddb58..74cf8162701984918df533d069f331be91030e31 100644
--- a/webrtc/video/video_quality_test.cc
+++ b/webrtc/video/video_quality_test.cc
@@ -199,8 +199,7 @@ class VideoAnalyzer : public PacketReceiver,
encoded_frame_size_.AddSample(frame.length_);
}
- void RenderFrame(const VideoFrame& video_frame,
- int time_to_render_ms) override {
+ void OnFrame(const VideoFrame& video_frame) override {
int64_t render_time_ms =
Clock::GetRealTimeClock()->CurrentNtpInMilliseconds();
@@ -240,8 +239,6 @@ class VideoAnalyzer : public PacketReceiver,
last_rendered_frame_ = video_frame;
}
- bool IsTextureSupported() const override { return false; }
-
void Wait() {
// Frame comparisons can be very expensive. Wait for test to be done, but
// at time-out check if frames_processed is going up. If so, give it more

Powered by Google App Engine
This is Rietveld 408576698