Remove unnecessary image downloading handler of notifications.

ui/message_center/notifier_settings.h

 // Copyright (c) 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.
 
 #ifndef UI_MESSAGE_CENTER_NOTIFIER_SETTINGS_H_
 #define UI_MESSAGE_CENTER_NOTIFIER_SETTINGS_H_
 
 #include <string>
 
 #include "base/gtest_prod_util.h"
 #include "base/strings/string16.h"
 #include "ui/gfx/image/image.h"
 #include "ui/message_center/message_center_export.h"
 #include "url/gurl.h"
 
+class MessageCenterNotificationsTest;
 class MessageCenterTrayBridgeTest;
+class StubNotificationUIManager;
 
 namespace ash {
 class WebNotificationTrayTest;
 }
 
 namespace message_center {
 namespace test {
 class MessagePopupCollectionTest;
 }
 
+class MessageCenterNotificationManagerTest;
 class NotifierSettingsDelegate;
 class NotifierSettingsProvider;
 
 // Brings up the settings dialog and returns a weak reference to the delegate,
 // which is typically the view. If the dialog already exists, it is brought to
 // the front, otherwise it is created.
 MESSAGE_CENTER_EXPORT NotifierSettingsDelegate* ShowSettings(
     NotifierSettingsProvider* provider,
     gfx::NativeView context);
 
@@ skipping 21 matching lines @@
   std::string id;
 
   // The URL pattern of the notifer.
   GURL url;
 
   // The identifier of the profile where the notification is created. This is
   // used for ChromeOS multi-profile support and can be empty.
   std::string profile_id;
 
  private:
+  friend class ::MessageCenterNotificationsTest;

[dewittj, 2014/09/19 00:17:53]

wonder if sorting should put all the global-namespace tests first?

[Jun Mukai, 2014/09/19 00:22:29]

sorted by the namespace.  I think it's clearer in the order of same namespace ->
global namespace -> alphabetical namespace order

+  friend class MessageCenterNotificationManagerTest;
   friend class ::MessageCenterTrayBridgeTest;
   friend class MessageCenterTrayTest;
   friend class test::MessagePopupCollectionTest;
   friend class NotificationControllerTest;
   friend class PopupCollectionTest;
+  friend class ::StubNotificationUIManager;
   friend class TrayViewControllerTest;
   friend class ash::WebNotificationTrayTest;
   FRIEND_TEST_ALL_PREFIXES(PopupControllerTest, Creation);
   FRIEND_TEST_ALL_PREFIXES(NotificationListTest, UnreadCountNoNegative);
   FRIEND_TEST_ALL_PREFIXES(NotificationListTest, TestHasNotificationOfType);
 
   // The default constructor which doesn't specify the notifier. Used for tests.
   NotifierId();
 };
 
@@ skipping 107 matching lines @@
 
   // Called upon request for more information about a particular notifier.
   virtual void OnNotifierAdvancedSettingsRequested(
       const NotifierId& notifier_id,
       const std::string* notification_id) = 0;
 };
 
 }  // namespace message_center
 
 #endif  // UI_MESSAGE_CENTER_NOTIFIER_SETTINGS_H_