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

Issue 2786953002: Render focus-selection-textarea at constant font-size to avoid flakiness (Closed)

Created:
3 years, 8 months ago by hugoh_UTC2
Modified:
3 years, 8 months ago
Reviewers:
Finnur
CC:
chromium-reviews, blink-reviews, imcheng
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Render focus-selection-textarea at constant font-size to avoid flakiness When crrev.com/453211 updated FreeType on Linux, some glyphs got bigger due to the new antialiasing and hinting settings. Bigger glyphs made focus-selection-textarea.html layout differently. With the new layout, scripted mouse clicks no longer hit the right spot. After the manual rebaselining in crrev.com/453211, the expected result on Linux started to include FAIL whereas other platforms still expected PASS. This CL makes also Linux PASS. BUG=706119, 274030 TEST=python third_party/WebKit/Tools/Scripts/run-webkit-tests -t\ PcDebug LayoutTests/fast/forms/focus-selection-textarea.html Review-Url: https://codereview.chromium.org/2786953002 Cr-Commit-Position: refs/heads/master@{#460732} Committed: https://chromium.googlesource.com/chromium/src/+/3afa26135e229e0f7fb81328a2bf4dcf966f90af

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -10 lines) Patch
M third_party/WebKit/LayoutTests/TestExpectations View 1 chunk +1 line, -3 lines 0 comments Download
M third_party/WebKit/LayoutTests/fast/forms/focus-selection-textarea.html View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/LayoutTests/platform/linux/fast/forms/focus-selection-textarea-expected.txt View 1 chunk +6 lines, -6 lines 0 comments Download

Messages

Total messages: 11 (5 generated)
hugoh_UTC2
finnur@ PTAL Hopefully the rebaselining will fix crbug.com/706119 (unless the Windows environment itself is flaky ...
3 years, 8 months ago (2017-03-30 10:15:19 UTC) #2
drott
Hi, thanks for fixing this and sorry if I missed this in the large rebaseline. ...
3 years, 8 months ago (2017-03-30 10:34:21 UTC) #3
hugoh_UTC2
On 2017/03/30 10:34:21, drott wrote: > Hi, thanks for fixing this and sorry if I ...
3 years, 8 months ago (2017-03-30 10:40:21 UTC) #4
Finnur
LGTM
3 years, 8 months ago (2017-03-30 11:04:43 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2786953002/1
3 years, 8 months ago (2017-03-30 11:08:20 UTC) #8
commit-bot: I haz the power
3 years, 8 months ago (2017-03-30 12:29:57 UTC) #11
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/3afa26135e229e0f7fb81328a2bf...

Powered by Google App Engine
This is Rietveld 408576698