commit 1731c9cb4c4ea693ce836720dc64bed32cf6fe94
author asapersson <asapersson@webrtc.org> Wed Nov 30 00:29:09 2016 -0800
committer Commit bot <commit-bot@chromium.org> Wed Nov 30 08:29:17 2016 +0000
tree 0abc2736142577c25795c722359c31c0cc7c9440
parent 84e56d576806635c966093d5421c5d04c9b90746

Use swap instead of copy in RtcHistogram::GetAndReset.

BUG=webrtc:5283

Review-Url: https://codereview.webrtc.org/2268323002
Cr-Commit-Position: refs/heads/master@{#15311}

webrtc/system_wrappers/source/metrics_default_unittest.cc

diff --git a/webrtc/system_wrappers/source/metrics_default_unittest.cc b/webrtc/system_wrappers/source/metrics_default_unittest.cc
index c2eefe5..0414bc7 100644
--- a/webrtc/system_wrappers/source/metrics_default_unittest.cc
+++ b/webrtc/system_wrappers/source/metrics_default_unittest.cc
@@ -134,6 +134,16 @@
   EXPECT_EQ(1, NumEvents("Histogram1", 4, histograms));
   EXPECT_EQ(2, NumEvents("Histogram1", 5, histograms));
   EXPECT_EQ(1, NumEvents("Histogram2", 10, histograms));
+
+  // Add samples after reset.
+  metrics::GetAndReset(&histograms);
+  EXPECT_EQ(0u, histograms.size());
+  RTC_HISTOGRAM_PERCENTAGE("Histogram1", 50);
+  RTC_HISTOGRAM_PERCENTAGE("Histogram2", 8);
+  EXPECT_EQ(1, metrics::NumSamples("Histogram1"));
+  EXPECT_EQ(1, metrics::NumSamples("Histogram2"));
+  EXPECT_EQ(1, metrics::NumEvents("Histogram1", 50));
+  EXPECT_EQ(1, metrics::NumEvents("Histogram2", 8));
 }
 
 TEST_F(MetricsDefaultTest, TestMinMaxBucket) {