Desktop NTP: Add a UMA metric NewTabPage.TilesReceivedTime

chrome/browser/ui/webui/ntp/ntp_user_data_logger_unittest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/webui/ntp/ntp_user_data_logger.h"
 
 #include <memory>
 #include <string>
 #include <vector>
 
 #include "base/metrics/histogram.h"
 #include "base/metrics/statistics_recorder.h"
 #include "base/test/histogram_tester.h"
 #include "chrome/common/search/ntp_logging_events.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using base::Bucket;
 using ntp_tiles::NTPTileSource;
 using testing::ElementsAre;
 using testing::IsEmpty;
+using testing::SizeIs;
 
 namespace {
 
 using Sample = base::HistogramBase::Sample;
 using Samples = std::vector<Sample>;
 
 class TestNTPUserDataLogger : public NTPUserDataLogger {
  public:
   TestNTPUserDataLogger() : NTPUserDataLogger(nullptr) {}
   ~TestNTPUserDataLogger() override {}
@@ skipping 172 matching lines @@
   EXPECT_THAT(
       histogram_tester.GetAllSamples("NewTabPage.MostVisited"),
       ElementsAre(Bucket(0, 2), Bucket(1, 2), Bucket(2, 2), Bucket(3, 2)));
   EXPECT_THAT(
       histogram_tester.GetAllSamples("NewTabPage.MostVisited.server"),
       ElementsAre(Bucket(0, 2), Bucket(1, 1), Bucket(2, 1)));
   EXPECT_THAT(
       histogram_tester.GetAllSamples("NewTabPage.MostVisited.client"),
       ElementsAre(Bucket(1, 1), Bucket(2, 1), Bucket(3, 2)));
 }
+
+TEST(NTPUserDataLoggerTest, TestLoadTime) {
+  base::StatisticsRecorder::Initialize();
+
+  base::HistogramTester histogram_tester;
+
+  TestNTPUserDataLogger logger;
+  logger.ntp_url_ = GURL("chrome://newtab/");
+
+  base::TimeDelta delta_tiles_received = base::TimeDelta::FromMilliseconds(10);
+  base::TimeDelta delta_tiles_loaded = base::TimeDelta::FromMilliseconds(100);
+
+  // Send the ALL_TILES_RECEIVED event.
+  logger.LogEvent(NTP_ALL_TILES_RECEIVED, delta_tiles_received);
+
+  // The actual histograms are emitted only after the ALL_TILES_LOADED event, so
+  // at this point everything should still be empty.

[sfiera, 2017/02/16 14:35:33]

Is this something we actually care about testing? I would be totally OK with it
if TilesReceivedTime were recorded already at this point.

[Marc Treib, 2017/02/16 15:13:09]

Good point. Removed.

+  EXPECT_THAT(histogram_tester.GetAllSamples("NewTabPage.TilesReceivedTime"),
+              IsEmpty());
+  EXPECT_THAT(histogram_tester.GetAllSamples("NewTabPage.LoadTime"), IsEmpty());
+
+  // Log a TOP_SITES impression (for the .MostVisited vs .MostLikely split in
+  // the LoadTime histogram).
+  logger.LogMostVisitedImpression(0, NTPTileSource::TOP_SITES);
+
+  // Send the ALL_TILES_LOADED event, this should trigger emitting histograms.
+  logger.LogEvent(NTP_ALL_TILES_LOADED, delta_tiles_loaded);
+
+  EXPECT_THAT(histogram_tester.GetAllSamples("NewTabPage.TilesReceivedTime"),
+              SizeIs(1));
+  EXPECT_THAT(histogram_tester.GetAllSamples(
+                  "NewTabPage.TilesReceivedTime.MostVisited"),
+              SizeIs(1));
+  EXPECT_THAT(
+      histogram_tester.GetAllSamples("NewTabPage.TilesReceivedTime.MostLikely"),
+      IsEmpty());
+  EXPECT_THAT(histogram_tester.GetAllSamples("NewTabPage.LoadTime"), SizeIs(1));
+  EXPECT_THAT(histogram_tester.GetAllSamples("NewTabPage.LoadTime.MostVisited"),
+              SizeIs(1));
+  EXPECT_THAT(histogram_tester.GetAllSamples("NewTabPage.LoadTime.MostLikely"),
+              IsEmpty());
+
+  histogram_tester.ExpectTimeBucketCount("NewTabPage.TilesReceivedTime",
+                                         delta_tiles_received, 1);
+  histogram_tester.ExpectTimeBucketCount(
+      "NewTabPage.TilesReceivedTime.MostVisited", delta_tiles_received, 1);
+  histogram_tester.ExpectTimeBucketCount("NewTabPage.LoadTime",
+                                         delta_tiles_loaded, 1);
+  histogram_tester.ExpectTimeBucketCount("NewTabPage.LoadTime.MostVisited",
+                                         delta_tiles_loaded, 1);
+
+  // We should not log again for the same NTP.
+  logger.LogEvent(NTP_ALL_TILES_RECEIVED, delta_tiles_received);
+  logger.LogEvent(NTP_ALL_TILES_LOADED, delta_tiles_loaded);
+  histogram_tester.ExpectTimeBucketCount("NewTabPage.TilesReceivedTime",
+                                         delta_tiles_received, 1);
+  histogram_tester.ExpectTimeBucketCount("NewTabPage.LoadTime",
+                                         delta_tiles_loaded, 1);
+
+  // After navigating away from the NTP and back, we record again.
+  logger.NavigatedFromURLToURL(GURL("chrome://newtab/"),
+                               GURL("http://chromium.org"));
+  logger.NavigatedFromURLToURL(GURL("http://chromium.org"),
+                               GURL("chrome://newtab/"));
+
+  // This time, log a SUGGESTIONS_SERVICE impression, so the load time will end
+  // up in .MostLikely.
+  logger.LogMostVisitedImpression(0, NTPTileSource::SUGGESTIONS_SERVICE);
+
+  base::TimeDelta delta_tiles_received2 = base::TimeDelta::FromMilliseconds(50);
+  base::TimeDelta delta_tiles_loaded2 = base::TimeDelta::FromMilliseconds(500);
+  logger.LogEvent(NTP_ALL_TILES_RECEIVED, delta_tiles_received2);
+  logger.LogEvent(NTP_ALL_TILES_LOADED, delta_tiles_loaded2);
+
+  EXPECT_THAT(histogram_tester.GetAllSamples("NewTabPage.TilesReceivedTime"),
+              SizeIs(2));
+  EXPECT_THAT(histogram_tester.GetAllSamples(
+                  "NewTabPage.TilesReceivedTime.MostVisited"),
+              SizeIs(1));
+  EXPECT_THAT(
+      histogram_tester.GetAllSamples("NewTabPage.TilesReceivedTime.MostLikely"),
+      SizeIs(1));
+  EXPECT_THAT(histogram_tester.GetAllSamples("NewTabPage.LoadTime"), SizeIs(2));
+  EXPECT_THAT(histogram_tester.GetAllSamples("NewTabPage.LoadTime.MostVisited"),
+              SizeIs(1));
+  EXPECT_THAT(histogram_tester.GetAllSamples("NewTabPage.LoadTime.MostLikely"),
+              SizeIs(1));
+
+  histogram_tester.ExpectTimeBucketCount("NewTabPage.TilesReceivedTime",
+                                         delta_tiles_received2, 1);
+  histogram_tester.ExpectTimeBucketCount(
+      "NewTabPage.TilesReceivedTime.MostLikely", delta_tiles_received2, 1);
+  histogram_tester.ExpectTimeBucketCount("NewTabPage.LoadTime",
+                                         delta_tiles_loaded2, 1);
+  histogram_tester.ExpectTimeBucketCount("NewTabPage.LoadTime.MostLikely",
+                                         delta_tiles_loaded2, 1);
+}