Add EXPECT_CALL to fix warning during unit_test run for Hats

chrome/browser/chromeos/hats/hats_notification_controller_unittest.cc

 // Copyright 2016 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/chromeos/hats/hats_notification_controller.h"
 
 #include "base/run_loop.h"
 #include "base/strings/string_split.h"
 #include "chrome/browser/notifications/message_center_notification_manager.h"
 #include "chrome/browser/notifications/notification.h"
@@ skipping 105 matching lines @@
     g_browser_process->notification_ui_manager()->StartShutdown();
     // The notifications may be deleted async.
     base::RunLoop loop;
     loop.RunUntilIdle();
     profile_manager_.reset();
     network_portal_detector::InitializeForTesting(nullptr);
     BrowserWithTestWindowTest::TearDown();
   }
 
   scoped_refptr<HatsNotificationController> InstantiateHatsController() {
-    MockImageFetcher* mock_image_fetcher_ = new MockImageFetcher;
+    mock_image_fetcher_ = new MockImageFetcher;
     // The initialization will fail since the function IsNewDevice() will return
     // true.
     scoped_refptr<HatsNotificationController> hats_notification_controller =
         new HatsNotificationController(&profile_, mock_image_fetcher_);
 
     // HatsController::IsNewDevice() is run on a blocking thread.
     content::RunAllBlockingPoolTasksUntilIdle();
 
     // Send a callback to the observer to simulate internet connectivity is
     // present on device.
@@ skipping 33 matching lines @@
           hats_notification_controller->OnImageFetched(
               HatsNotificationController::kImageFetcher2xId, icon_1x,
               image_fetcher::RequestMetadata());
         }));
 
     return hats_notification_controller;
   }
 
   TestingProfile profile_;
   StrictMock<MockNetworkPortalDetector> mock_network_portal_detector_;
+  MockImageFetcher* mock_image_fetcher_;
 
  private:
   std::unique_ptr<TestingProfileManager> profile_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(HatsNotificationControllerTest);
 };
 
 TEST_F(HatsNotificationControllerTest, NewDevice_ShouldNotShowNotification) {
   int64_t initial_timestamp = base::Time::Now().ToInternalValue();
   PrefService* pref_service = profile_.GetPrefs();
@@ skipping 29 matching lines @@
   // NetworkPortalDetector to detect internet connectivity.
   EXPECT_CALL(mock_network_portal_detector_,
               AddAndFireObserver(hats_notification_controller.get()))
       .Times(1);
   // Observer is removed if an internet connection is detected. It is called
   // a second time when hats_notification_controller is destroyed.
   EXPECT_CALL(mock_network_portal_detector_,
               RemoveObserver(hats_notification_controller.get()))
       .Times(2);
 
+  EXPECT_CALL(*mock_image_fetcher_,
+              StartOrQueueNetworkRequest(
+                  HatsNotificationController::kImageFetcher1xId,
+                  GURL(HatsNotificationController::kGoogleIcon1xUrl), _))
+      .Times(1);
+  EXPECT_CALL(*mock_image_fetcher_,
+              StartOrQueueNetworkRequest(
+                  HatsNotificationController::kImageFetcher2xId,
+                  GURL(HatsNotificationController::kGoogleIcon2xUrl), _))
+      .Times(1);
+
   hats_notification_controller->Initialize(false);
 
   // Finally check if notification was launched to confirm initialization.
   const Notification* notification =
       g_browser_process->notification_ui_manager()->FindById(
           HatsNotificationController::kDelegateId, &profile_);
   EXPECT_TRUE(notification != nullptr);
 }
 
 TEST_F(HatsNotificationControllerTest, NoInternet_DoNotShowNotification) {
@@ skipping 45 matching lines @@
   ASSERT_TRUE(base::Time::FromInternalValue(new_timestamp) >
               base::Time::FromInternalValue(now_timestamp));
 
   // Destructor for HatsController removes self from observer list.
   EXPECT_CALL(mock_network_portal_detector_,
               RemoveObserver(hats_notification_controller.get()))
       .Times(1);
 }
 
 }  // namespace chromeos