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

Side by Side Diff: components/history/core/browser/history_service.h

Issue 2721363002: Extend LargeIconService to fetch missing favicons from a Google server (Closed)
Patch Set: Addressed comments. 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_HISTORY_CORE_BROWSER_HISTORY_SERVICE_H_ 5 #ifndef COMPONENTS_HISTORY_CORE_BROWSER_HISTORY_SERVICE_H_
6 #define COMPONENTS_HISTORY_CORE_BROWSER_HISTORY_SERVICE_H_ 6 #define COMPONENTS_HISTORY_CORE_BROWSER_HISTORY_SERVICE_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 743 matching lines...) Expand 10 before | Expand all | Expand 10 after
754 // |page_url| for |icon_type|. 754 // |page_url| for |icon_type|.
755 // Use MergeFavicon() if |bitmaps| is incomplete, and favicon bitmaps in the 755 // Use MergeFavicon() if |bitmaps| is incomplete, and favicon bitmaps in the
756 // database should be preserved if possible. For instance, favicon bitmaps 756 // database should be preserved if possible. For instance, favicon bitmaps
757 // from sync are 1x only. MergeFavicon() is used to avoid deleting the 2x 757 // from sync are 1x only. MergeFavicon() is used to avoid deleting the 2x
758 // favicon bitmap if it is present in the history backend. 758 // favicon bitmap if it is present in the history backend.
759 void SetFavicons(const GURL& page_url, 759 void SetFavicons(const GURL& page_url,
760 favicon_base::IconType icon_type, 760 favicon_base::IconType icon_type,
761 const GURL& icon_url, 761 const GURL& icon_url,
762 const std::vector<SkBitmap>& bitmaps); 762 const std::vector<SkBitmap>& bitmaps);
763 763
764 // Same as SetFavicons with two differences: a) the favicon will be initially
765 // set as expired; b) it will be a no-op if there is an existing cached
766 // favicon for *any* type for |page_url|.
767 void SetExpiredFaviconsIfNoneKnown(const GURL& page_url,
768 favicon_base::IconType icon_type,
769 const GURL& icon_url,
770 const std::vector<SkBitmap>& bitmaps);
771
764 // Used by the FaviconService to mark the favicon for the page as being out 772 // Used by the FaviconService to mark the favicon for the page as being out
765 // of date. 773 // of date.
766 void SetFaviconsOutOfDateForPage(const GURL& page_url); 774 void SetFaviconsOutOfDateForPage(const GURL& page_url);
767 775
768 // Used by the FaviconService for importing many favicons for many pages at 776 // Used by the FaviconService for importing many favicons for many pages at
769 // once. The pages must exist, any favicon sets for unknown pages will be 777 // once. The pages must exist, any favicon sets for unknown pages will be
770 // discarded. Existing favicons will not be overwritten. 778 // discarded. Existing favicons will not be overwritten.
771 void SetImportedFavicons( 779 void SetImportedFavicons(
772 const favicon_base::FaviconUsageDataList& favicon_usage); 780 const favicon_base::FaviconUsageDataList& favicon_usage);
773 781
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
836 844
837 // All vended weak pointers are invalidated in Cleanup(). 845 // All vended weak pointers are invalidated in Cleanup().
838 base::WeakPtrFactory<HistoryService> weak_ptr_factory_; 846 base::WeakPtrFactory<HistoryService> weak_ptr_factory_;
839 847
840 DISALLOW_COPY_AND_ASSIGN(HistoryService); 848 DISALLOW_COPY_AND_ASSIGN(HistoryService);
841 }; 849 };
842 850
843 } // namespace history 851 } // namespace history
844 852
845 #endif // COMPONENTS_HISTORY_CORE_BROWSER_HISTORY_SERVICE_H_ 853 #endif // COMPONENTS_HISTORY_CORE_BROWSER_HISTORY_SERVICE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698