Make icon_messages use the IPC macros for defining the structs and the serialization code, and mo...

chrome/browser/favicon_helper.cc

 // Copyright (c) 2011 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/favicon_helper.h"
 
 #include "build/build_config.h"
 
 #include <vector>
 
 #include "base/callback.h"
 #include "base/memory/ref_counted_memory.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
 #include "chrome/browser/profiles/profile.h"
-#include "chrome/common/render_messages.h"
+#include "chrome/common/icon_messages.h"
 #include "content/browser/renderer_host/render_view_host.h"
 #include "content/browser/tab_contents/navigation_controller.h"
 #include "content/browser/tab_contents/navigation_entry.h"
 #include "content/browser/tab_contents/tab_contents_delegate.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "skia/ext/image_operations.h"
 #include "ui/gfx/codec/png_codec.h"
 
+namespace {
+
+// Returns history::IconType the given icon_type corresponds to.
+history::IconType ToHistoryIconType(FaviconURL::IconType icon_type) {
+  switch (icon_type) {
+    case FaviconURL::FAVICON:
+      return history::FAVICON;
+    case FaviconURL::TOUCH_ICON:
+      return history::TOUCH_ICON;
+    case FaviconURL::TOUCH_PRECOMPOSED_ICON:
+      return history::TOUCH_PRECOMPOSED_ICON;
+    case FaviconURL::INVALID_ICON:
+      return history::INVALID_ICON;
+  }
+  NOTREACHED();
+  // Shouldn't reach here, just make compiler happy.
+  return history::INVALID_ICON;
+}
+
+bool DoUrlAndIconMatch(const FaviconURL& favicon_url,
+                       const GURL& url,
+                       history::IconType icon_type) {
+  return favicon_url.icon_url == url &&
+      favicon_url.icon_type == static_cast<FaviconURL::IconType>(icon_type);
+}
+
+}  // namespace
+
 FaviconHelper::DownloadRequest::DownloadRequest()
     : callback(NULL),
       icon_type(history::INVALID_ICON) {
 }
 
 FaviconHelper::DownloadRequest::DownloadRequest(const GURL& url,
                                                 const GURL& image_url,
                                                 ImageDownloadCallback* callback,
                                                 history::IconType icon_type)
     : url(url),
@@ skipping 118 matching lines @@
   // TODO(davemoore) Should clear on empty url. Currently we ignore it.
   // This appears to be what FF does as well.
   // No URL was added.
   if (!got_favicon_url_update)
     return;
 
   if (!GetFaviconService())
     return;
 
   // For FAVICON.
-  if (current_candidate()->icon_type == ::FAVICON) {
+  if (current_candidate()->icon_type == FaviconURL::FAVICON) {
     if (!favicon_expired_ && entry->favicon().is_valid() &&
-        do_url_and_icon_match(*current_candidate(), entry->favicon().url(),
-                              history::FAVICON))
+        DoUrlAndIconMatch(*current_candidate(), entry->favicon().url(),
+                          history::FAVICON))
       return;
 
     entry->favicon().set_url(current_candidate()->icon_url);
   } else if (!favicon_expired_ && got_favicon_from_history_ &&
               history_icon_.is_valid() &&
-              do_url_and_icon_match(*current_candidate(),
+              DoUrlAndIconMatch(
+                  *current_candidate(),
                   history_icon_.icon_url, history_icon_.icon_type)) {
     return;
   }
 
   if (got_favicon_from_history_)
     DownloadFaviconOrAskHistory(entry->url(), current_candidate()->icon_url,
         ToHistoryIconType(current_candidate()->icon_type));
 }
 
 NavigationEntry* FaviconHelper::GetEntry() {
@@ skipping 49 matching lines @@
 
 bool FaviconHelper::ShouldSaveFavicon(const GURL& url) {
   if (!tab_contents()->profile()->IsOffTheRecord())
     return true;
 
   // Otherwise store the favicon if the page is bookmarked.
   BookmarkModel* bookmark_model = tab_contents()->profile()->GetBookmarkModel();
   return bookmark_model && bookmark_model->IsBookmarked(url);
 }
 
-history::IconType FaviconHelper::ToHistoryIconType(IconType icon_type) {
-  switch (icon_type) {
-    case ::FAVICON:
-      return history::FAVICON;
-    case ::TOUCH_ICON:
-      return history::TOUCH_ICON;
-    case ::TOUCH_PRECOMPOSED_ICON:
-      return history::TOUCH_PRECOMPOSED_ICON;
-    case ::INVALID_ICON:
-      return history::INVALID_ICON;
-  }
-  NOTREACHED();
-  // Shouldn't reach here, just make compiler happy.
-  return history::INVALID_ICON;
-}
-
 bool FaviconHelper::OnMessageReceived(const IPC::Message& message) {
   bool message_handled = true;
   IPC_BEGIN_MESSAGE_MAP(FaviconHelper, message)
-    IPC_MESSAGE_HANDLER(ViewHostMsg_DidDownloadFavicon, OnDidDownloadFavicon)
+    IPC_MESSAGE_HANDLER(IconHostMsg_DidDownloadFavicon, OnDidDownloadFavicon)
     IPC_MESSAGE_UNHANDLED(message_handled = false)
   IPC_END_MESSAGE_MAP()
   return message_handled;
 }
 
 void FaviconHelper::OnDidDownloadFavicon(int id,
                                          const GURL& image_url,
                                          bool errored,
                                          const SkBitmap& image) {
   DownloadRequests::iterator i = download_requests_.find(id);
   if (i == download_requests_.end()) {
     // Currently TabContents notifies us of ANY downloads so that it is
     // possible to get here.
     return;
   }
 
   if (i->second.callback) {
     i->second.callback->Run(id, errored, image);
   } else if (current_candidate() &&
-             do_url_and_icon_match(*current_candidate(), image_url,
-                                   i->second.icon_type)) {
+             DoUrlAndIconMatch(*current_candidate(), image_url,
+                               i->second.icon_type)) {
     // The downloaded icon is still valid when there is no FaviconURL update
     // during the downloading.
     if (!errored) {
       SetFavicon(i->second.url, image_url, image, i->second.icon_type);
     } else if (GetEntry() && ++current_url_index_ < urls_.size()) {
       // Copies all candidate except first one and notifies the FaviconHelper,
       // so the next candidate can be processed.
       std::vector<FaviconURL> new_candidates(++urls_.begin(), urls_.end());
       OnUpdateFaviconURL(0, new_candidates);
     }
   }
   download_requests_.erase(i);
 }
 
 void FaviconHelper::OnFaviconDataForInitialURL(
     FaviconService::Handle handle,
     history::FaviconData favicon) {
   NavigationEntry* entry = GetEntry();
   if (!entry)
     return;
 
   got_favicon_from_history_ = true;
   history_icon_ = favicon;
 
   favicon_expired_ = (favicon.known_icon && favicon.expired);
 
   if (favicon.known_icon && favicon.icon_type == history::FAVICON &&
       !entry->favicon().is_valid() &&
-      (!current_candidate() || do_url_and_icon_match(*current_candidate(),
-                                  favicon.icon_url, favicon.icon_type))) {
+      (!current_candidate() ||
+       DoUrlAndIconMatch(
+           *current_candidate(), favicon.icon_url, favicon.icon_type))) {
     // The db knows the favicon (although it may be out of date) and the entry
     // doesn't have an icon. Set the favicon now, and if the favicon turns out
     // to be expired (or the wrong url) we'll fetch later on. This way the
     // user doesn't see a flash of the default favicon.
     entry->favicon().set_url(favicon.icon_url);
     if (favicon.is_valid())
       UpdateFavicon(entry, favicon.image_data);
     entry->favicon().set_is_valid(true);
   }
 
   if (favicon.known_icon && !favicon.expired) {
-    if (current_candidate() && !do_url_and_icon_match(*current_candidate(),
-                                   favicon.icon_url, favicon.icon_type)) {
+    if (current_candidate() &&
+        !DoUrlAndIconMatch(
+             *current_candidate(), favicon.icon_url, favicon.icon_type)) {
       // Mapping in the database is wrong. DownloadFavIconOrAskHistory will
       // update the mapping for this url and download the favicon if we don't
       // already have it.
       DownloadFaviconOrAskHistory(entry->url(), current_candidate()->icon_url,
           static_cast<history::IconType>(current_candidate()->icon_type));
     }
   } else if (current_candidate()) {
     // We know the official url for the favicon, by either don't have the
     // favicon or its expired. Continue on to DownloadFaviconOrAskHistory to
     // either download or check history again.
@@ skipping 49 matching lines @@
       UpdateFavicon(entry, favicon.image_data);
     }
     if (!favicon.known_icon || favicon.expired) {
       // We don't know the favicon, or it is out of date. Request the current
       // one.
       ScheduleDownload(entry->url(), entry->favicon().url(),
                        preferred_icon_size(),
                        history::FAVICON, NULL);
     }
   } else if (current_candidate() && (!favicon.known_icon || favicon.expired ||
-      !(do_url_and_icon_match(*current_candidate(), favicon.icon_url,
-                              favicon.icon_type)))) {
+      !(DoUrlAndIconMatch(
+            *current_candidate(), favicon.icon_url, favicon.icon_type)))) {
     // We don't know the favicon, it is out of date or its type is not same as
     // one got from page. Request the current one.
     ScheduleDownload(entry->url(), current_candidate()->icon_url,
         preferred_icon_size(),
         ToHistoryIconType(current_candidate()->icon_type), NULL);
   }
   history_icon_ = favicon;
 }
 
 int FaviconHelper::ScheduleDownload(const GURL& url,
@@ skipping 16 matching lines @@
   int width = image.width();
   int height = image.height();
   if (width > 0 && height > 0) {
     calc_favicon_target_size(&width, &height);
     return skia::ImageOperations::Resize(
           image, skia::ImageOperations::RESIZE_LANCZOS3,
           width, height);
   }
   return image;
 }