[Icons NTP] Refactor large_icon_source to extract the logic shared between desktop and Android to f…

components/favicon_base/favicon_types.h

 // Copyright 2014 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 COMPONENTS_FAVICON_BASE_FAVICON_TYPES_H_
 #define COMPONENTS_FAVICON_BASE_FAVICON_TYPES_H_
 
+#include <base/memory/scoped_ptr.h>

[pkotwicz, 2015/04/21 21:41:05]

Is there a reason for the weird include syntax for scoped_ptr.h

[beaudoin, 2015/04/21 23:02:05]

Nope. Changed.

Done.

+
 #include "base/memory/ref_counted_memory.h"
 #include "ui/gfx/geometry/size.h"
 #include "ui/gfx/image/image.h"
 #include "url/gurl.h"
 
 namespace favicon_base {
 
+struct FallbackIconStyle;
+
 typedef int64 FaviconID;
 
 // Defines the icon types. They are also stored in icon_type field of favicons
 // table.
 // The values of the IconTypes are used to select the priority in which favicon
 // data is returned in HistoryBackend and ThumbnailDatabase. Data for the
 // largest IconType takes priority if data for multiple IconTypes is available.
 enum IconType {
   INVALID_ICON = 0x0,
   FAVICON = 1 << 0,
@@ skipping 40 matching lines @@
   GURL icon_url;
 
   // The icon type of the containing favicon.
   IconType icon_type;
 };
 
 // Define type with same structure as FaviconRawBitmapResult for passing data to
 // HistoryBackend::SetFavicons().
 typedef FaviconRawBitmapResult FaviconRawBitmapData;
 
+// Result returned when accessing a large icon, which is either a bitmap if
+// one was available; or the style of the fallback icon otherwise.

[pkotwicz, 2015/04/21 21:41:05]

How about: "Result returned by LargeIconService::GetLargeIconOrFallbackStyle().
Contains either the bitmap data if the favicon database has a sufficiently large
favicon bitmap and the style of the fallback icon otherwise."

[beaudoin, 2015/04/21 23:02:05]

Done.

+struct LargeIconResult {
+  LargeIconResult();
+  ~LargeIconResult();
+
+  // The bitmap of the large icon if available.

[pkotwicz, 2015/04/21 21:41:05]

How about: "The bitmap from the favicon database if the database has a
sufficiently large one."

[beaudoin, 2015/04/21 23:02:05]

Done.

+  FaviconRawBitmapResult bitmap;
+
+  // The fallback icon style if a large icon isn't available. This uses the
+  // dominant color of a smaller icon as the background if available.
+  scoped_ptr<FallbackIconStyle> fallback_icon_style;
+};
+
 }  // namespace favicon_base
 
 #endif  // COMPONENTS_FAVICON_BASE_FAVICON_TYPES_H_