Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(320)

Side by Side Diff: components/favicon/core/favicon_service_impl.h

Issue 2721363002: Extend LargeIconService to fetch missing favicons from a Google server (Closed)
Patch Set: Add fallback_opts=TYPE Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_FAVICON_CORE_FAVICON_SERVICE_IMPL_H_ 5 #ifndef COMPONENTS_FAVICON_CORE_FAVICON_SERVICE_IMPL_H_
6 #define COMPONENTS_FAVICON_CORE_FAVICON_SERVICE_IMPL_H_ 6 #define COMPONENTS_FAVICON_CORE_FAVICON_SERVICE_IMPL_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
95 const favicon_base::FaviconUsageDataList& favicon_usage) override; 95 const favicon_base::FaviconUsageDataList& favicon_usage) override;
96 void MergeFavicon(const GURL& page_url, 96 void MergeFavicon(const GURL& page_url,
97 const GURL& icon_url, 97 const GURL& icon_url,
98 favicon_base::IconType icon_type, 98 favicon_base::IconType icon_type,
99 scoped_refptr<base::RefCountedMemory> bitmap_data, 99 scoped_refptr<base::RefCountedMemory> bitmap_data,
100 const gfx::Size& pixel_size) override; 100 const gfx::Size& pixel_size) override;
101 void SetFavicons(const GURL& page_url, 101 void SetFavicons(const GURL& page_url,
102 const GURL& icon_url, 102 const GURL& icon_url,
103 favicon_base::IconType icon_type, 103 favicon_base::IconType icon_type,
104 const gfx::Image& image) override; 104 const gfx::Image& image) override;
105 void SetLastResortFavicons(const GURL& page_url,
106 const GURL& icon_url,
107 favicon_base::IconType icon_type,
108 const gfx::Image& image,
109 base::Callback<void(bool)> callback) override;
105 void UnableToDownloadFavicon(const GURL& icon_url) override; 110 void UnableToDownloadFavicon(const GURL& icon_url) override;
106 bool WasUnableToDownloadFavicon(const GURL& icon_url) const override; 111 bool WasUnableToDownloadFavicon(const GURL& icon_url) const override;
107 void ClearUnableToDownloadFavicons() override; 112 void ClearUnableToDownloadFavicons() override;
108 113
109 private: 114 private:
110 typedef uint32_t MissingFaviconURLHash; 115 typedef uint32_t MissingFaviconURLHash;
111 116
112 // Helper function for GetFaviconImageForPageURL(), GetRawFaviconForPageURL() 117 // Helper function for GetFaviconImageForPageURL(), GetRawFaviconForPageURL()
113 // and GetFaviconForPageURL(). 118 // and GetFaviconForPageURL().
114 base::CancelableTaskTracker::TaskId GetFaviconForPageURLImpl( 119 base::CancelableTaskTracker::TaskId GetFaviconForPageURLImpl(
(...skipping 26 matching lines...) Expand all
141 base::hash_set<MissingFaviconURLHash> missing_favicon_urls_; 146 base::hash_set<MissingFaviconURLHash> missing_favicon_urls_;
142 std::unique_ptr<FaviconClient> favicon_client_; 147 std::unique_ptr<FaviconClient> favicon_client_;
143 history::HistoryService* history_service_; 148 history::HistoryService* history_service_;
144 149
145 DISALLOW_COPY_AND_ASSIGN(FaviconServiceImpl); 150 DISALLOW_COPY_AND_ASSIGN(FaviconServiceImpl);
146 }; 151 };
147 152
148 } // namespace favicon 153 } // namespace favicon
149 154
150 #endif // COMPONENTS_FAVICON_CORE_FAVICON_SERVICE_IMPL_H_ 155 #endif // COMPONENTS_FAVICON_CORE_FAVICON_SERVICE_IMPL_H_
OLDNEW
« no previous file with comments | « components/favicon/core/favicon_service.h ('k') | components/favicon/core/favicon_service_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698