Cleanup desktop NTP metrics recording

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 "base/metrics/histogram_macros.h"
 #include "base/metrics/statistics_recorder.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/common/search/ntp_logging_events.h"
@@ skipping 63 matching lines @@
 
 }  // namespace
 
 class NTPUserDataLoggerTest : public testing::Test {
   content::TestBrowserThreadBundle thread_bundle_;
 };
 
 TEST_F(NTPUserDataLoggerTest, TestNumberOfTiles) {
   base::StatisticsRecorder::Initialize();
 
-  // Enusure non-zero statistics.
+  // Ensure non-zero statistics.
   TestNTPUserDataLogger logger;
   logger.ntp_url_ = GURL("chrome://newtab/");
 
   base::TimeDelta delta = base::TimeDelta::FromMilliseconds(0);
 
   for (int i = 0; i < 8; ++i)
-    logger.LogEvent(NTP_TILE, delta);
-  logger.LogEvent(NTP_SERVER_SIDE_SUGGESTION, delta);
+    logger.LogEvent(NTP_SERVER_SIDE_SUGGESTION, delta);
 
   logger.LogEvent(NTP_ALL_TILES_LOADED, delta);
 
   EXPECT_EQ(1, GetTotalCount("NewTabPage.NumberOfTiles"));
   EXPECT_EQ(1, GetBinCount("NewTabPage.NumberOfTiles", 8));
 
   // Statistics should be reset to 0, so we should not log anything else.
   logger.LogEvent(NTP_ALL_TILES_LOADED, delta);
   EXPECT_EQ(1, GetTotalCount("NewTabPage.NumberOfTiles"));
 
@@ skipping 98 matching lines @@
 
   // Navigations always increase.
   logger.LogMostVisitedNavigation(1, NTPLoggingTileSource::SERVER);
   logger.LogMostVisitedNavigation(2, NTPLoggingTileSource::CLIENT);
   logger.LogMostVisitedNavigation(3, NTPLoggingTileSource::SERVER);
   logger.LogMostVisitedNavigation(4, NTPLoggingTileSource::CLIENT);
   EXPECT_THAT(TotalNavigations({1, 2, 3, 4}), ElementsAre(2, 2, 2, 2));
   EXPECT_THAT(ServerNavigations({1, 2, 3, 4}), ElementsAre(2, 1, 1, 0));
   EXPECT_THAT(ClientNavigations({1, 2, 3, 4}), ElementsAre(0, 1, 1, 2));
 }