Add enum-based UMA reporting to automatic redirect logic of Offline Pages.

chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.cc

Index: chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.cc
diff --git a/chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.cc b/chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.cc
index ba76871c70dda2ef3b1bfafef54dbe70a5862245..ee2b5c8a7fdca4602054711dca3bd22afbd52669 100644
--- a/chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.cc
+++ b/chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.cc
@@ -36,10 +36,7 @@ namespace {
 const GURL kTestPageUrl("http://test.org/page1");
 const ClientId kTestClientId = ClientId(kBookmarkNamespace, "1234");
 const int64_t kTestFileSize = 876543LL;
-const char kBadNetworkHistogram[] = "OfflinePages.ShowOfflinePageOnBadNetwork";
-const char kRedirectToOfflineHistogram[] =
-    "OfflinePages.RedirectToOfflineCount";
-const char kRedirectToOnlineHistogram[] = "OfflinePages.RedirectToOnlineCount";
+const char kRedirectResultHistogram[] = "OfflinePages.RedirectResult";
 const char kTabId[] = "42";
 
 class TestNetworkChangeNotifier : public net::NetworkChangeNotifier {
@@ -227,9 +224,10 @@ TEST_F(OfflinePageTabHelperTest, SwitchToOnlineFromOfflineOnNetwork) {
   RunUntilIdle();
   // Redirection will be done immediately on navigation start.
   EXPECT_EQ(online_url(), controller().GetPendingEntry()->GetURL());
-  histograms().ExpectTotalCount(kBadNetworkHistogram, 0);
-  histograms().ExpectTotalCount(kRedirectToOfflineHistogram, 0);
-  histograms().ExpectTotalCount(kRedirectToOnlineHistogram, 1);
+  histograms().ExpectUniqueSample(
+      kRedirectResultHistogram,
+      OfflinePageTabHelper::RedirectResult::CONNECTED_NETWORK,
+      1);
 }
 
 TEST_F(OfflinePageTabHelperTest, SwitchToOfflineFromOnlineOnNoNetwork) {
@@ -240,9 +238,10 @@ TEST_F(OfflinePageTabHelperTest, SwitchToOfflineFromOnlineOnNoNetwork) {
   RunUntilIdle();
   // Redirection will be done immediately on navigation start.
   EXPECT_EQ(offline_url(), controller().GetPendingEntry()->GetURL());
-  histograms().ExpectTotalCount(kBadNetworkHistogram, 0);
-  histograms().ExpectTotalCount(kRedirectToOfflineHistogram, 1);
-  histograms().ExpectTotalCount(kRedirectToOnlineHistogram, 0);
+  histograms().ExpectUniqueSample(
+      kRedirectResultHistogram,
+      OfflinePageTabHelper::RedirectResult::DISCONNECTED_NETWORK,
+      1);
 }
 
 TEST_F(OfflinePageTabHelperTest, TestCurrentOfflinePage) {
@@ -275,10 +274,10 @@ TEST_F(OfflinePageTabHelperTest, SwitchToOfflineFromOnlineOnError) {
   FailLoad(online_url());
   EXPECT_EQ(offline_url(), controller().GetPendingEntry()->GetURL());
 
-  histograms().ExpectBucketCount(kBadNetworkHistogram, false, 0);
-  histograms().ExpectBucketCount(kBadNetworkHistogram, true, 1);
-  histograms().ExpectTotalCount(kRedirectToOfflineHistogram, 1);
-  histograms().ExpectTotalCount(kRedirectToOnlineHistogram, 0);
+  histograms().ExpectUniqueSample(
+      kRedirectResultHistogram,
+      OfflinePageTabHelper::RedirectResult::FLAKY_NETWORK,
+      1);
 }
 
 TEST_F(OfflinePageTabHelperTest, NewNavigationCancelsPendingRedirects) {
@@ -301,9 +300,11 @@ TEST_F(OfflinePageTabHelperTest, NewNavigationCancelsPendingRedirects) {
   // |unsaved_url|.
   EXPECT_EQ(unsaved_url, controller().GetPendingEntry()->GetURL());
 
-  histograms().ExpectTotalCount(kBadNetworkHistogram, 0);
-  histograms().ExpectTotalCount(kRedirectToOfflineHistogram, 0);
-  histograms().ExpectTotalCount(kRedirectToOnlineHistogram, 0);
+  // Should report attempt of redirect, but the page not found.
+  histograms().ExpectUniqueSample(
+      kRedirectResultHistogram,
+      OfflinePageTabHelper::RedirectResult::DISCONNECTED_NETWORK_NOT_FOUND,
+      1);
 }
 
 // This test saves 3 pages (one in setup and 2 in test). The most appropriate