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

Side by Side Diff: chrome/browser/history/thumbnail_database.h

Issue 11830007: Avoid sending notifications when the bitmap data in history has not changed (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Split the changes to MergeFavicon() and SetFavicons() into two CLs as only the changes to MergeFavi… Created 7 years, 11 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 | Annotate | Revision Log
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 CHROME_BROWSER_HISTORY_THUMBNAIL_DATABASE_H_ 5 #ifndef CHROME_BROWSER_HISTORY_THUMBNAIL_DATABASE_H_
6 #define CHROME_BROWSER_HISTORY_THUMBNAIL_DATABASE_H_ 6 #define CHROME_BROWSER_HISTORY_THUMBNAIL_DATABASE_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/gtest_prod_util.h" 10 #include "base/gtest_prod_util.h"
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
135 base::Time time, 135 base::Time time,
136 const gfx::Size& pixel_size); 136 const gfx::Size& pixel_size);
137 137
138 // Sets the bitmap data and the last updated time for the favicon bitmap at 138 // Sets the bitmap data and the last updated time for the favicon bitmap at
139 // |bitmap_id|. 139 // |bitmap_id|.
140 // Returns true if successful. 140 // Returns true if successful.
141 bool SetFaviconBitmap(FaviconBitmapID bitmap_id, 141 bool SetFaviconBitmap(FaviconBitmapID bitmap_id,
142 scoped_refptr<base::RefCountedMemory> bitmap_data, 142 scoped_refptr<base::RefCountedMemory> bitmap_data,
143 base::Time time); 143 base::Time time);
144 144
145 // Sets the last updated time for the favicon bitmap at |bitmap_id|.
146 // Returns true if successful.
147 bool SetFaviconBitmapLastUpdateTime(FaviconBitmapID bitmap_id,
148 base::Time time);
149
145 // Deletes the favicon bitmaps for the favicon with with |icon_id|. 150 // Deletes the favicon bitmaps for the favicon with with |icon_id|.
146 // Returns true if successful. 151 // Returns true if successful.
147 bool DeleteFaviconBitmapsForFavicon(FaviconID icon_id); 152 bool DeleteFaviconBitmapsForFavicon(FaviconID icon_id);
148 153
149 // Deletes the favicon bitmap with |bitmap_id|. 154 // Deletes the favicon bitmap with |bitmap_id|.
150 // Returns true if successful. 155 // Returns true if successful.
151 bool DeleteFaviconBitmap(FaviconBitmapID bitmap_id); 156 bool DeleteFaviconBitmap(FaviconBitmapID bitmap_id);
152 157
153 // Favicons ------------------------------------------------------------------ 158 // Favicons ------------------------------------------------------------------
154 159
(...skipping 261 matching lines...) Expand 10 before | Expand all | Expand 10 after
416 const HistoryPublisher* history_publisher_; 421 const HistoryPublisher* history_publisher_;
417 422
418 // True if migration to TopSites has been done and the thumbnails 423 // True if migration to TopSites has been done and the thumbnails
419 // table should not be used. 424 // table should not be used.
420 bool use_top_sites_; 425 bool use_top_sites_;
421 }; 426 };
422 427
423 } // namespace history 428 } // namespace history
424 429
425 #endif // CHROME_BROWSER_HISTORY_THUMBNAIL_DATABASE_H_ 430 #endif // CHROME_BROWSER_HISTORY_THUMBNAIL_DATABASE_H_
OLDNEW
« no previous file with comments | « chrome/browser/history/history_backend_unittest.cc ('k') | chrome/browser/history/thumbnail_database.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698