Add DownloadStatus metric to FaviconHandler

components/favicon/core/favicon_handler_unittest.cc

 // Copyright (c) 2012 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 "components/favicon/core/favicon_handler.h"
 
 #include <stddef.h>
 
 #include <map>
 #include <memory>
@@ skipping 24 matching lines @@
 using favicon_base::TOUCH_ICON;
 using favicon_base::TOUCH_PRECOMPOSED_ICON;
 using testing::Assign;
 using testing::ElementsAre;
 using testing::InSequence;
 using testing::Invoke;
 using testing::IsEmpty;
 using testing::Return;
 using testing::_;
 
+using DownloadOutcome = FaviconHandler::DownloadOutcome;
 using IntVector = std::vector<int>;
 using URLVector = std::vector<GURL>;
 using BitmapVector = std::vector<SkBitmap>;
 using SizeVector = std::vector<gfx::Size>;
 
 MATCHER_P2(ImageSizeIs, width, height, "") {
   *result_listener << "where size is " << arg.Width() << "x" << arg.Height();
   return arg.Size() == gfx::Size(width, height);
 }
 
@@ skipping 1010 matching lines @@
 
   EXPECT_THAT(
       histogram_tester.GetAllSamples("Favicons.DownloadAttempts.Favicons"),
       ElementsAre(base::Bucket(/*sample=*/1, /*expected_count=*/1)));
   EXPECT_THAT(
       histogram_tester.GetAllSamples("Favicons.DownloadAttempts.LargeIcons"),
       IsEmpty());
   EXPECT_THAT(
       histogram_tester.GetAllSamples("Favicons.DownloadAttempts.TouchIcons"),
       IsEmpty());
+  EXPECT_THAT(histogram_tester.GetAllSamples("Favicons.DownloadOutcome"),
+              ElementsAre(base::Bucket(DownloadOutcome::SUCCEEDED,
+                                       /*expected_count=*/1)));
 }
 
 TEST_F(FaviconHandlerTest, TestRecordSingleLargeIconDownloadAttempt) {
   base::HistogramTester histogram_tester;
 
   RunHandlerWithCandidates(FaviconDriverObserver::NON_TOUCH_LARGEST,
                            {FaviconURL(kIconURL64x64, FAVICON, kEmptySizes)});
 
   EXPECT_THAT(
       histogram_tester.GetAllSamples("Favicons.DownloadAttempts.Favicons"),
       IsEmpty());
   EXPECT_THAT(
       histogram_tester.GetAllSamples("Favicons.DownloadAttempts.LargeIcons"),
       ElementsAre(base::Bucket(/*sample=*/1, /*expected_count=*/1)));
   EXPECT_THAT(
       histogram_tester.GetAllSamples("Favicons.DownloadAttempts.TouchIcons"),
       IsEmpty());
+  EXPECT_THAT(histogram_tester.GetAllSamples("Favicons.DownloadOutcome"),
+              ElementsAre(base::Bucket(DownloadOutcome::SUCCEEDED,
+                                       /*expected_count=*/1)));
 }
 
 TEST_F(FaviconHandlerTest, TestRecordSingleTouchIconDownloadAttempt) {
   base::HistogramTester histogram_tester;
   RunHandlerWithCandidates(
       FaviconDriverObserver::TOUCH_LARGEST,
       {FaviconURL(kIconURL64x64, TOUCH_ICON, kEmptySizes)});
 
   EXPECT_THAT(
       histogram_tester.GetAllSamples("Favicons.DownloadAttempts.LargeIcons"),
       IsEmpty());
   EXPECT_THAT(
       histogram_tester.GetAllSamples("Favicons.DownloadAttempts.Favicons"),
       IsEmpty());
   EXPECT_THAT(
       histogram_tester.GetAllSamples("Favicons.DownloadAttempts.TouchIcons"),
       ElementsAre(base::Bucket(/*sample=*/1, /*expected_count=*/1)));
+  EXPECT_THAT(histogram_tester.GetAllSamples("Favicons.DownloadOutcome"),
+              ElementsAre(base::Bucket(DownloadOutcome::SUCCEEDED,
+                                       /*expected_count=*/1)));
 }
 
 TEST_F(FaviconHandlerTest, TestRecordDownloadAttemptsFinishedByCache) {
   const GURL kIconURL1024x1024("http://www.google.com/a-404-ing-icon");
   base::HistogramTester histogram_tester;
   favicon_service_.fake()->Store(
       GURL("http://so.de"), kIconURL64x64,
       CreateRawBitmapResult(kIconURL64x64, FAVICON, /*expired=*/false, 64));
 
   RunHandlerWithCandidates(
       FaviconDriverObserver::NON_TOUCH_LARGEST,
       {FaviconURL(kIconURL1024x1024, FAVICON, {gfx::Size(1024, 1024)}),
        FaviconURL(kIconURL12x12, FAVICON, {gfx::Size(12, 12)}),
        FaviconURL(kIconURL64x64, FAVICON, {gfx::Size(64, 64)})});
 
   // Should try only the first (receive 404) and get second icon from cache.
   EXPECT_THAT(delegate_.downloads(), ElementsAre(kIconURL1024x1024));
 
   EXPECT_THAT(
       histogram_tester.GetAllSamples("Favicons.DownloadAttempts.LargeIcons"),
       ElementsAre(base::Bucket(/*sample=*/1, /*expected_count=*/1)));
   EXPECT_THAT(
       histogram_tester.GetAllSamples("Favicons.DownloadAttempts.Favicons"),
       IsEmpty());
   EXPECT_THAT(
       histogram_tester.GetAllSamples("Favicons.DownloadAttempts.TouchIcons"),
       IsEmpty());
 }
 
+TEST_F(FaviconHandlerTest, TestRecordFailingDownloadAttempt) {
+  base::HistogramTester histogram_tester;
+  const GURL k404IconURL("http://www.google.com/404.png");
+
+  delegate_.fake_downloader().AddError(k404IconURL, 404);
+
+  EXPECT_CALL(favicon_service_, UnableToDownloadFavicon(k404IconURL));
+
+  RunHandlerWithSimpleFaviconCandidates({k404IconURL});
+
+  EXPECT_THAT(
+      histogram_tester.GetAllSamples("Favicons.DownloadOutcome"),
+      ElementsAre(base::Bucket(DownloadOutcome::FAILED, /*expected_count=*/1)));
+}
+
+TEST_F(FaviconHandlerTest, TestRecordSkippedDownloadForKnownFailingUrl) {
+  base::HistogramTester histogram_tester;
+  const GURL k404IconURL("http://www.google.com/404.png");
+
+  ON_CALL(favicon_service_, WasUnableToDownloadFavicon(k404IconURL))
+      .WillByDefault(Return(true));
+
+  RunHandlerWithSimpleFaviconCandidates({k404IconURL});
+
+  EXPECT_THAT(histogram_tester.GetAllSamples("Favicons.DownloadOutcome"),
+              ElementsAre(base::Bucket(DownloadOutcome::SKIPPED,
+                                       /*expected_count=*/1)));
+}
+
 }  // namespace
 }  // namespace favicon