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

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

Issue 2510373003: Cleanup: Remove "gray text" logic from Omnibox (Closed)
Patch Set: review2 Created 4 years, 1 month 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/autocomplete_text_field_unittest.mm
diff --git a/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field_unittest.mm b/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field_unittest.mm
index 4ca3df8b6b17ce594febe736897fd28f6713e9b5..c21c1b61ebed5c2e9ae423282c8ade8e9f2bd4ed 100644
--- a/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field_unittest.mm
+++ b/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field_unittest.mm
@@ -243,15 +243,6 @@ TEST_F(AutocompleteTextFieldTest, Display) {
[field_ display];
}
-// Test setting gray text, mostly to ensure nothing leaks or crashes.
-TEST_F(AutocompleteTextFieldTest, GrayText) {
- [field_ display];
- EXPECT_FALSE([field_ needsDisplay]);
- [field_ setGrayTextAutocompletion:@"foo" textColor:[NSColor redColor]];
- EXPECT_TRUE([field_ needsDisplay]);
- [field_ display];
-}
-
TEST_F(AutocompleteTextFieldObserverTest, FlagsChanged) {
InSequence dummy; // Call mock in exactly the order specified.

Powered by Google App Engine
This is Rietveld 408576698