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

Unified Diff: chrome/browser/cocoa/location_bar/autocomplete_text_field_unittest.mm

Issue 3046029: [Mac] First pass at final sizing of toolbar items. (Closed) Base URL: git://codf21.jail/chromium.git
Patch Set: Don't disable unit tests for realz, sorry. Created 10 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/cocoa/location_bar/autocomplete_text_field_unittest.mm
diff --git a/chrome/browser/cocoa/location_bar/autocomplete_text_field_unittest.mm b/chrome/browser/cocoa/location_bar/autocomplete_text_field_unittest.mm
index 341058c1880b2abe0d4c92f390aa7fad2ad3586f..29beb38e71947a4ab16d9d8ff1c69f9ad772038d 100644
--- a/chrome/browser/cocoa/location_bar/autocomplete_text_field_unittest.mm
+++ b/chrome/browser/cocoa/location_bar/autocomplete_text_field_unittest.mm
@@ -427,7 +427,7 @@ TEST_F(AutocompleteTextFieldTest, ClickBorderSelectsAll) {
TEST_F(AutocompleteTextFieldTest, ClickSelectsAll) {
EXPECT_FALSE([field_ currentEditor]);
- const NSPoint point(NSMakePoint(20.0, 5.0));
+ const NSPoint point = NSMakePoint(20.0, NSMidY([field_ bounds]));
NSEvent* downEvent(Event(field_, point, NSLeftMouseDown));
NSEvent* upEvent(Event(field_, point, NSLeftMouseUp));
[NSApp postEvent:upEvent atStart:YES];
@@ -468,7 +468,7 @@ TEST_F(AutocompleteTextFieldTest, ClickDragSelectsText) {
TEST_F(AutocompleteTextFieldTest, DoubleClickSelectsWord) {
EXPECT_FALSE([field_ currentEditor]);
- const NSPoint point(NSMakePoint(20.0, 5.0));
+ const NSPoint point = NSMakePoint(20.0, NSMidY([field_ bounds]));
NSEvent* downEvent(Event(field_, point, NSLeftMouseDown, 1));
NSEvent* upEvent(Event(field_, point, NSLeftMouseUp, 1));
NSEvent* downEvent2(Event(field_, point, NSLeftMouseDown, 2));

Powered by Google App Engine
This is Rietveld 408576698