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

Unified Diff: webrtc/video/video_quality_test.cc

Issue 2286633002: Delete IsZeroSize method. (Closed)
Patch Set: Created 4 years, 4 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 51160c91c6a2e15b23f8b4c90d536e58eba8478f..5b6ad7373835a16e247fcf21f37775f4b3999707 100644
--- a/webrtc/video/video_quality_test.cc
+++ b/webrtc/video/video_quality_test.cc
@@ -606,12 +606,8 @@ class VideoAnalyzer : public PacketReceiver,
void PerformFrameComparison(const FrameComparison& comparison) {
// Perform expensive psnr and ssim calculations while not holding lock.
- double psnr = -1.0;
- double ssim = -1.0;
- if (!comparison.reference.IsZeroSize()) {
stefan-webrtc 2016/09/05 14:45:25 I'm pretty sure this was needed when I added this
- psnr = I420PSNR(&comparison.reference, &comparison.render);
- ssim = I420SSIM(&comparison.reference, &comparison.render);
- }
+ double psnr = I420PSNR(&comparison.reference, &comparison.render);
+ double ssim = I420SSIM(&comparison.reference, &comparison.render);
int64_t input_time_ms = comparison.reference.ntp_time_ms();

Powered by Google App Engine
This is Rietveld 408576698