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

Unified Diff: chrome/browser/ui/cocoa/location_bar/ev_bubble_decoration_unittest.mm

Issue 12042002: Alternate NTP: Add search token to omnibox (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address review comments 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/location_bar/ev_bubble_decoration_unittest.mm
diff --git a/chrome/browser/ui/cocoa/location_bar/ev_bubble_decoration_unittest.mm b/chrome/browser/ui/cocoa/location_bar/ev_bubble_decoration_unittest.mm
index 0506a722b87c5397569efdc971544e8f3af8a3be..331c20ed4213693752f7312c69c42dee42779261 100644
--- a/chrome/browser/ui/cocoa/location_bar/ev_bubble_decoration_unittest.mm
+++ b/chrome/browser/ui/cocoa/location_bar/ev_bubble_decoration_unittest.mm
@@ -35,21 +35,21 @@ TEST_F(EVBubbleDecorationTest, MiddleElide) {
decoration_.SetFullLabel(kLongString);
// Lots of space, decoration not omitted.
- EXPECT_NE(decoration_.GetWidthForSpace(kWide),
+ EXPECT_NE(decoration_.GetWidthForSpace(kWide, 0),
LocationBarDecoration::kOmittedWidth);
// If the available space is of the same magnitude as the required
// space, the decoration doesn't eat it all up.
- const CGFloat long_width = decoration_.GetWidthForSpace(kWide);
- EXPECT_NE(decoration_.GetWidthForSpace(long_width + 20.0),
+ const CGFloat long_width = decoration_.GetWidthForSpace(kWide, 0);
+ EXPECT_NE(decoration_.GetWidthForSpace(long_width + 20.0, 0),
LocationBarDecoration::kOmittedWidth);
- EXPECT_LT(decoration_.GetWidthForSpace(long_width + 20.0), long_width);
+ EXPECT_LT(decoration_.GetWidthForSpace(long_width + 20.0, 0), long_width);
// If there is very little space, the decoration is still relatively
// big.
- EXPECT_NE(decoration_.GetWidthForSpace(kNarrow),
+ EXPECT_NE(decoration_.GetWidthForSpace(kNarrow, 0),
LocationBarDecoration::kOmittedWidth);
- EXPECT_GT(decoration_.GetWidthForSpace(kNarrow), kMinimumWidth);
+ EXPECT_GT(decoration_.GetWidthForSpace(kNarrow, 0), kMinimumWidth);
}
} // namespace
« no previous file with comments | « chrome/browser/ui/cocoa/location_bar/ev_bubble_decoration.mm ('k') | chrome/browser/ui/cocoa/location_bar/image_decoration.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698