Chromium Code Reviews| Index: webrtc/video/end_to_end_tests.cc | 
| diff --git a/webrtc/video/end_to_end_tests.cc b/webrtc/video/end_to_end_tests.cc | 
| index 106b5137123100c11b18d6ae3bf066075349b562..ea9128abe5995851968ad34412c6003895125f14 100644 | 
| --- a/webrtc/video/end_to_end_tests.cc | 
| +++ b/webrtc/video/end_to_end_tests.cc | 
| @@ -2641,7 +2641,8 @@ void EndToEndTest::VerifyHistogramStats(bool use_rtx, | 
| std::string video_prefix = | 
| screenshare ? "WebRTC.Video.Screenshare." : "WebRTC.Video."; | 
| - | 
| + std::string video_prefix_sliced = | 
| + screenshare ? "WebRTC.Video.Screenshare.S0." : "WebRTC.Video."; | 
| // Verify that stats have been updated once. | 
| EXPECT_EQ(2, metrics::NumSamples("WebRTC.Call.LifetimeInSeconds")); | 
| EXPECT_EQ(1, metrics::NumSamples( | 
| @@ -2703,10 +2704,13 @@ void EndToEndTest::VerifyHistogramStats(bool use_rtx, | 
| EXPECT_EQ(1, metrics::NumSamples("WebRTC.Video.CurrentDelayInMs")); | 
| EXPECT_EQ(1, metrics::NumSamples("WebRTC.Video.OnewayDelayInMs")); | 
| - EXPECT_EQ(1, metrics::NumSamples(video_prefix + "EndToEndDelayInMs")); | 
| - EXPECT_EQ(1, metrics::NumSamples(video_prefix + "EndToEndDelayMaxInMs")); | 
| - EXPECT_EQ(1, metrics::NumSamples(video_prefix + "InterframeDelayInMs")); | 
| - EXPECT_EQ(1, metrics::NumSamples(video_prefix + "InterframeDelayMaxInMs")); | 
| + EXPECT_EQ(1, metrics::NumSamples(video_prefix_sliced + "EndToEndDelayInMs")); | 
| 
 
ilnik
2017/07/26 14:15:23
Need to fix this test. S0 comes at the very end of
 
 | 
| + EXPECT_EQ(1, | 
| + metrics::NumSamples(video_prefix_sliced + "EndToEndDelayMaxInMs")); | 
| + EXPECT_EQ(1, | 
| + metrics::NumSamples(video_prefix_sliced + "InterframeDelayInMs")); | 
| + EXPECT_EQ( | 
| + 1, metrics::NumSamples(video_prefix_sliced + "InterframeDelayMaxInMs")); | 
| EXPECT_EQ(1, metrics::NumSamples("WebRTC.Video.RenderSqrtPixelsPerSecond")); |