Refactor FaviconDriver::OnFaviconAvailable()

chrome/browser/task_management/providers/web_contents/renderer_task.cc

 // Copyright 2015 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/task_management/providers/web_contents/renderer_task.h"
 
 #include "base/i18n/rtl.h"
 #include "base/numerics/safe_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 128 matching lines @@
 }
 
 bool RendererTask::ReportsWebCacheStats() const {
   return true;
 }
 
 blink::WebCache::ResourceTypeStats RendererTask::GetWebCacheStats() const {
   return webcache_stats_;
 }
 
-void RendererTask::OnFaviconAvailable(const gfx::Image& image) {
-}
-
 void RendererTask::OnFaviconUpdated(favicon::FaviconDriver* favicon_driver,
-                                    bool icon_url_changed) {
-  UpdateFavicon();
+                                    NotificationIconType notification_icon_type,
+                                    const GURL& icon_url,
+                                    bool icon_url_changed,
+                                    const gfx::Image& image) {
+  if (notification_icon_type == NON_TOUCH_16_DIP)
+    UpdateFavicon();
 }
 
 // static
 base::string16 RendererTask::GetTitleFromWebContents(
     content::WebContents* web_contents) {
   DCHECK(web_contents);
   base::string16 title = web_contents->GetTitle();
   if (title.empty()) {
     GURL url = web_contents->GetURL();
     title = base::UTF8ToUTF16(url.spec());
@@ skipping 53 matching lines @@
     if (is_incognito)
       message_id = IDS_TASK_MANAGER_EXTENSION_INCOGNITO_PREFIX;
     else
       message_id = IDS_TASK_MANAGER_EXTENSION_PREFIX;
   }
 
   return l10n_util::GetStringFUTF16(message_id, title);
 }
 
 }  // namespace task_management