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

Unified Diff: chrome/browser/ui/cocoa/history_menu_bridge_unittest.mm

Issue 10802066: Adds support for saving favicon size into history database. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 5 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/ui/cocoa/history_menu_bridge_unittest.mm
diff --git a/chrome/browser/ui/cocoa/history_menu_bridge_unittest.mm b/chrome/browser/ui/cocoa/history_menu_bridge_unittest.mm
index a143e434420ed0a74408fd01388f6b759968b4bb..3d7db7aef5056f912c110d1adeb0198331d7d911 100644
--- a/chrome/browser/ui/cocoa/history_menu_bridge_unittest.mm
+++ b/chrome/browser/ui/cocoa/history_menu_bridge_unittest.mm
@@ -20,6 +20,7 @@
#import "testing/gtest_mac.h"
#include "third_party/skia/include/core/SkBitmap.h"
#include "ui/gfx/codec/png_codec.h"
+#include "ui/gfx/favicon_size.h"
namespace {
@@ -372,7 +373,8 @@ TEST_F(HistoryMenuBridgeTest, GotFaviconData) {
// Pretend to be called back.
history::FaviconData favicon;
favicon.known_icon = true;
- favicon.image_data = new base::RefCountedBytes(raw);
+ favicon.bitmap_data = new base::RefCountedBytes(raw);
+ favicon.requested_size = gfx::Size(gfx::kFaviconSize, gfx::kFaviconSize);
favicon.expired = false;
favicon.icon_url = GURL();
favicon.icon_type = history::FAVICON;

Powered by Google App Engine
This is Rietveld 408576698