From a1349268f945805bcc749dd5b292a77802d5f753 Mon Sep 17 00:00:00 2001 From: Tommy Martino Date: Tue, 24 Jul 2018 00:36:07 +0000 Subject: [PATCH] [Autofill Views] Draw separator background Currently, we don't draw the background on the separator. This is fine on Win and Mac, because by default we just see white anyway. But on Linux (specifically GTK+) we draw gray by default, giving an ugly gray splotch. Change-Id: I6f6d61d7ade28d30a659858a9a83bf0506aa3637 Bug: 866637 Reviewed-on: https://chromium-review.googlesource.com/1147158 Reviewed-by: Fabio Tirelo Reviewed-by: Evan Stade Commit-Queue: Fabio Tirelo Cr-Commit-Position: refs/heads/master@{#577373} --- .../ui/views/autofill/autofill_popup_view_native_views.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/chrome/browser/ui/views/autofill/autofill_popup_view_native_views.cc b/chrome/browser/ui/views/autofill/autofill_popup_view_native_views.cc index 9de17f6835191..15c6bf9b8e625 100644 --- a/chrome/browser/ui/views/autofill/autofill_popup_view_native_views.cc +++ b/chrome/browser/ui/views/autofill/autofill_popup_view_native_views.cc @@ -424,6 +424,8 @@ void AutofillPopupSeparatorView::CreateContent() { /*bottom=*/0, /*right=*/0)); AddChildView(separator); + + SetBackground(CreateBackground()); } void AutofillPopupSeparatorView::RefreshStyle() {