Skip to content

Commit d1effdf

Browse files
author
Tommy Martino
committed
[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 <[email protected]> Reviewed-by: Evan Stade <[email protected]> Commit-Queue: Fabio Tirelo <[email protected]> Cr-Original-Commit-Position: refs/heads/master@{#577373}(cherry picked from commit a134926) Reviewed-on: https://chromium-review.googlesource.com/1150320 Reviewed-by: Tommy Martino <[email protected]> Cr-Commit-Position: refs/branch-heads/3497@{#73} Cr-Branched-From: 271eaf5-refs/heads/master@{#576753}
1 parent b022902 commit d1effdf

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

chrome/browser/ui/views/autofill/autofill_popup_view_native_views.cc

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -424,6 +424,8 @@ void AutofillPopupSeparatorView::CreateContent() {
424424
/*bottom=*/0,
425425
/*right=*/0));
426426
AddChildView(separator);
427+
428+
SetBackground(CreateBackground());
427429
}
428430

429431
void AutofillPopupSeparatorView::RefreshStyle() {

0 commit comments

Comments
 (0)