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

Issue 680009 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression : Page number tool tip is seen misplaced in print preview of any web page.

Reported by yfulgaon...@etouch.net, Jan 11 2017

Issue description

Chrome Version : 57.0.2978.0 e2867aed46b3f00f3735d310144d6ef638f8be1a-refs/heads/master@{#442756} 32/64 bit
OS : Mac(10.12.1, 10.11.6, 10.12), Windows(7,8,10)

What steps will reproduce the problem?
1. Launch chrome, navigate to chrome://terms and hit Cmd+P.
2. Observe the page number tool tip in print preview.

Actual : Page number tool tip is seen misplaced in print preview.
Expected : Tool tip should not be misplaced.

This is a regression issue broken in ‘M-57’, below is the Manual Regression range and will soon update other info.
Good build : 57.0.2977.0
Bad build : 57.0.2978.0

Note : Issue is seen in print preview of any large web page.
 
Actual_tooltip.mov
6.6 MB Download
Expected_tooltip.mov
4.7 MB Download

Comment 1 by hdodda@chromium.org, Jan 11 2017

Cc: hdodda@chromium.org
Labels: hasbisect-per-revision
Owner: napper@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build : 57.0.2977.0 (revision : 442447)
Bad build : 57.0.2978.0 (revision : 442756)

You are probably looking for a change made after 442499 (known good), but no later than 442500 (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/42019ad33415b5f25bd14433a315b3084ce2cdd9..d8b62ea53246ab528786b9a8d312d0a3733e46e9

From the CL above, assigning the issue to the concern owner 

@napper - Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Review-Url: https://codereview.chromium.org/2628503002

Thanks!

Comment 2 by hdodda@chromium.org, Jan 11 2017

Labels: OS-Linux
Issue is also seen on Linux OS.
Labels: ReleaseBlock-Stable
Adding RB Label as this is a recent Regression. Please remove if not required.
Thank You.

Comment 4 by napper@chromium.org, Jan 11 2017

Cc: sashab@chromium.org

Comment 5 by napper@chromium.org, Jan 11 2017

I will revert the CL and investigate the issue.

Comment 6 by napper@chromium.org, Jan 11 2017

Cc: napper@chromium.org
 Issue 680038  has been merged into this issue.
Project Member

Comment 7 by bugdroid1@chromium.org, Jan 12 2017

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

commit 19df4a657a408440594776000848d096feffe5d7
Author: napper <napper@chromium.org>
Date: Thu Jan 12 01:09:52 2017

Revert of Made text-align property independent. (patchset #2 id:20001 of https://codereview.chromium.org/2628503002/ )

Reason for revert:
Misaligned text elements found in manual testing.

BUG= 680009 

Original issue's description:
> Made text-align property independent.
>
> Made text-align property independent. Also updated
> independent-inheritance-fast-path.html to include textAlign.
>
> Tested using independent-inheritance-fast-path.html.
>
> BUG= 628043 
>
> Review-Url: https://codereview.chromium.org/2628503002
> Cr-Commit-Position: refs/heads/master@{#442500}
> Committed: https://chromium.googlesource.com/chromium/src/+/d8b62ea53246ab528786b9a8d312d0a3733e46e9

TBR=sashab@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 628043 

Review-Url: https://codereview.chromium.org/2627833003
Cr-Commit-Position: refs/heads/master@{#443101}

[modify] https://crrev.com/19df4a657a408440594776000848d096feffe5d7/third_party/WebKit/LayoutTests/fast/css/invalidation/independent-inheritance-fast-path.html
[modify] https://crrev.com/19df4a657a408440594776000848d096feffe5d7/third_party/WebKit/Source/core/css/CSSProperties.in

Comment 8 by napper@chromium.org, Jan 12 2017

The CL has been reverted. I investigate what caused the regression.
Labels: TE-Verified-57.0.2979.2 TE-Verified-57.0.2979.0 TE-Verified-57
Tested the issue on windows 7, Mac 10.12.2, Linux Ubuntu 14.04 using chrome version-57.0.2979.0 & Windows clang build#57.0.2979.2 with the steps mentioned in comment #0.

Observed Page number tool tip displayed properly in print preview screen.

Please find the attached screen cast for the same.

Adding TE-Verified labels.

Thank you.
680009.mp4
7.4 MB View Download
Status: Verified (was: Assigned)

Sign in to add a comment