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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 10821035: wip 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
« no previous file with comments | « chrome/browser/ui/cocoa/history_menu_bridge.mm ('k') | chrome/browser/ui/intents/web_intent_picker_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 d5d0f542dac3037a9f7cb751b88df49429b48958..5ddfee57c8177ab1d9a523304519a2cf07613a6b 100644
--- a/chrome/browser/ui/cocoa/history_menu_bridge_unittest.mm
+++ b/chrome/browser/ui/cocoa/history_menu_bridge_unittest.mm
@@ -375,8 +375,8 @@ TEST_F(HistoryMenuBridgeTest, GotFaviconData) {
history::FaviconData favicon;
favicon.known_icon = true;
favicon.bitmap_data = new base::RefCountedBytes(raw);
- favicon.pixel_size = gfx::kFaviconSize;
- favicon.scale_factor = ui::SCALE_FACTOR_100P;
+ favicon.pixel_size = gfx::Size(gfx::kFaviconSize, gfx::kFaviconSize);
+ //favicon.scale_factor = ui::SCALE_FACTOR_100P;
favicon.expired = false;
favicon.icon_url = GURL();
favicon.icon_type = history::FAVICON;
« no previous file with comments | « chrome/browser/ui/cocoa/history_menu_bridge.mm ('k') | chrome/browser/ui/intents/web_intent_picker_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698