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

Unified Diff: chrome/browser/ui/views/autofill/autofill_popup_view_views.cc

Issue 2496683003: Http Bad: Add a PopupItemId to identify http warning message (Closed)
Patch Set: update 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/views/autofill/autofill_popup_view_views.cc
diff --git a/chrome/browser/ui/views/autofill/autofill_popup_view_views.cc b/chrome/browser/ui/views/autofill/autofill_popup_view_views.cc
index 4831ed76649608fcfd8b14888a77897f16d5f8d2..b3ce678f84b03b47d62cb491a4ef1fa3a95eec14 100644
--- a/chrome/browser/ui/views/autofill/autofill_popup_view_views.cc
+++ b/chrome/browser/ui/views/autofill/autofill_popup_view_views.cc
@@ -81,8 +81,8 @@ void AutofillPopupViewViews::DrawAutofillEntry(gfx::Canvas* canvas,
canvas->DrawStringRectWithFlags(
msw 2016/11/17 18:49:39 This view should probably use a Label, but that's
lshang 2016/11/18 00:03:19 Thanks, maybe I'll do it in the next CL.
controller_->GetElidedValueAt(index),
controller_->layout_model().GetValueFontListForRow(index),
- controller_->IsWarning(index) ? kLabelTextColor : kValueTextColor,
- value_rect, text_align);
+ controller_->layout_model().GetValueFontColorForRow(index), value_rect,
+ text_align);
// Use this to figure out where all the other Autofill items should be placed.
int x_align_left =

Powered by Google App Engine
This is Rietveld 408576698