Skip to content

Commit

Permalink
[Autofill Views] Draw separator background
Browse files Browse the repository at this point in the history
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 <ftirelo@chromium.org>
Reviewed-by: Evan Stade <estade@chromium.org>
Commit-Queue: Fabio Tirelo <ftirelo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#577373}
  • Loading branch information
Tommy Martino authored and Commit Bot committed Jul 24, 2018
1 parent 6973776 commit a134926
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -424,6 +424,8 @@ void AutofillPopupSeparatorView::CreateContent() {
/*bottom=*/0,
/*right=*/0));
AddChildView(separator);

SetBackground(CreateBackground());
}

void AutofillPopupSeparatorView::RefreshStyle() {
Expand Down

0 comments on commit a134926

Please sign in to comment.