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

Unified Diff: chrome/browser/history/android/android_provider_backend_unittest.cc

Issue 10802066: Adds support for saving favicon size into history database. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Updated comment for History::SetFavicons Created 8 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/history/android/android_provider_backend_unittest.cc
diff --git a/chrome/browser/history/android/android_provider_backend_unittest.cc b/chrome/browser/history/android/android_provider_backend_unittest.cc
index 1fbcc37309dadbcc36470f06c5bd955a1d8d5585..cc535a90ccba0af929191b32cd0497b90604700d 100644
--- a/chrome/browser/history/android/android_provider_backend_unittest.cc
+++ b/chrome/browser/history/android/android_provider_backend_unittest.cc
@@ -248,10 +248,20 @@ TEST_F(AndroidProviderBackendTest, UpdateTables) {
// Set favicon to url2.
std::vector<unsigned char> data;
data.push_back('1');
- history_backend->SetFavicon(url2,
- GURL(),
- new base::RefCountedBytes(data),
- FAVICON);
+ FaviconBitmapData bitmap_data_element;
+ bitmap_data_element.bitmap_data = new base::RefCountedBytes(data);
+ bitmap_data_element.pixel_size = gfx::Size();
+ bitmap_data_element.icon_url = GURL();
+ std::vector<FaviconBitmapData> favicon_bitmap_data;
+ favicon_bitmap_data.push_back(bitmap_data_element);
+
+ IconURLSizesMap icon_url_sizes;
+ icon_url_sizes[GURL()] = GetDefaultFaviconSizes();
+
+ history_backend->SetFavicons(url2,
+ FAVICON,
+ favicon_bitmap_data,
+ icon_url_sizes);
history_backend->Closing();
}

Powered by Google App Engine
This is Rietveld 408576698