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

Issue 2658123003: Fix missing borders on Autofill popups on Windows High DPI (Closed)

Created:
3 years, 10 months ago by Stephen Chennney
Modified:
3 years, 9 months ago
Reviewers:
mattp
CC:
chromium-reviews, rouslan+autofill_chromium.org, rogerm+autofillwatch_chromium.org, sebsg+autofillwatch_chromium.org, mathp+autofillwatch_chromium.org, vabr+watchlistautofill_chromium.org, estade+watch_chromium.org, chrishtr, Roger McFarlane (Chromium)
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Fix missing borders on Autofill popups on Windows High DPI When the Widnows 10 display setting are for 125% or 150% zoom, the borders on autofill entries are missing on some sides. This is due to a mismatch int he rounding methods used to convert from floating point rectangles to integers. This patch switches the popup layout code to use Nearest rect rounding rather than Enclosing rect, to more closely match the behavior of Display::SetScaleAndBounds. R=mattp@chromium.org BUG=685867

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M chrome/browser/ui/autofill/autofill_popup_layout_model.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 7 (5 generated)
Stephen Chennney
PTAL. Running it through a CQ dry run. I'm fine if we decide not to ...
3 years, 10 months ago (2017-01-27 21:27:51 UTC) #3
Stephen Chennney
3 years, 10 months ago (2017-01-27 21:30:03 UTC) #4
On 2017/01/27 21:27:51, Stephen Chennney wrote:
> PTAL. Running it through a CQ dry run.
> 
> I'm fine if we decide not to land this and put more work into a better fix.
> There should also be some testing of some form. However, my work day is done
and
> I'll be away from a Windows 10 machine next week, so I figure it's worth
putting
> this up for others to build on (or just land if that's OK with everyone).

This does not fix the 175% case, as expected because the layer rounds down but
my change will still round up in that case.

Powered by Google App Engine
This is Rietveld 408576698