New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 782608 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 779998
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression : Blue focus ring appears chopped on text field in chrome://supervised-user-internals/

Reported by vineetha...@etouch.net, Nov 8 2017

Issue description

Chrome version: 64.0.3262.0 d4f91d118274c0f0583e7a6f196be6c48da4acc9-refs/heads/master@{#514703}(32/64-bit)
OS: Windows (7,8,10),Linux (14.04 LTS),Mac(10.12.6, 10.13.2).

Steps to reproduce:
1. Launch Chrome. Navigate to chrome://supervised-user-internals/.
2. Try to get focus on "Try a URL" field and observe.

Actual Result: Blue focus ring on the "Try a URL" text field appears chopped.
Expected Result: Blue focus ring on the "Try a URL" text field should not be chopped.

This is Regression Issue broken in M-64 and using the per-revision bisect providing the bisect results,
Good Build: 64.0.3249.0((Revision: 511319)
Bad Build: 64.0.3250.0((Revision: 511680)

You are probably looking for a change made after 511471 (known good), but no later than 511472 (first known bad).
CHANGELOG URL:
The script might not always return single CL as suspect as some perf builds might get missing due to failure.

https://chromium.googlesource.com/chromium/src/+log/a6ba03d35d8e187799e9e3b63c98eff490018b71..2cff87c380b22ade36aa4d85a6f983387559178c

Suspect : https://chromium.googlesource.com/chromium/src/+/2cff87c380b22ade36aa4d85a6f983387559178c

@chrishtr : Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.

Thank You

 
ActualVideo.mp4
623 KB View Download
ExpectedVideo.mp4
558 KB View Download
Description: Show this description
Reduced testcase attached. In the presence of multicl, the invalidation rect
is off by a few pixels.
test.html
66 bytes View Download
Mergedinto: 779998
Status: Duplicate (was: Assigned)
Cc: chrishtr@chromium.org
 Issue 780418  has been merged into this issue.
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 9 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7b689930b0918450948c72447fec65f98e7eaa9a

commit 7b689930b0918450948c72447fec65f98e7eaa9a
Author: Chris Harrelson <chrishtr@chromium.org>
Date: Thu Nov 09 20:33:06 2017

Use BorderBoxRect when computing fragment paint offsets and
column fragmentation.

Visual rects may not have an origin at the (0, 0) point of the
object, which makes them inappropriate for paint offsets.
They are also not appropriate for fragmentation, because visual
overflow should extend beyond column bounds rather than
causing elements to paint into multiple fragments.

Note that fragment clips may still cut off visual overflow,
which is a bug, but now visual overflow will not extend across
columns.

Bug:  782608 ,  774699 ,  779998 
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I23feab957c018c124f59b5b9b84644709c466e03
Reviewed-on: https://chromium-review.googlesource.com/759233
Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Chris Harrelson <chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515269}
[modify] https://crrev.com/7b689930b0918450948c72447fec65f98e7eaa9a/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.cpp
[modify] https://crrev.com/7b689930b0918450948c72447fec65f98e7eaa9a/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilderTest.cpp

Sign in to add a comment