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

Issue 719341 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner: ----
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression
Team-Security-UX



Sign in to add a comment

Regression:Spacing issue is seen in Timed out page

Project Member Reported by keerthan...@techmahindra.com, May 8 2017

Issue description

Chrome Version: 60.0.3093.0
OS: Ubuntu 14.04, Windows

What steps will reproduce the problem?
(1)Launch Chrome and navigate to https://dhsdfga.com/ 
(2)Wait till the page to load and observe the spacing between ERR_TIMED_OUT and Reload button.


Actual:Spacing is more between ERR_TIMED_OUT and Reload button.
Expected:Spacing should be proper between ERR_TIMED_OUT and Reload button.

This is a Regression issue seen in M-60

Manual Bisect Info:
======================
Good Build:60.0.3090.0
Bad Build: 60.0.3091.0 
 
ReloadActual.png
136 KB View Download
ReloadExpected.png
132 KB View Download
Description: Show this description

Comment 2 Deleted

Issue is seen with a resolution of 1366*768
Labels: -Needs-Bisect ReleaseBlock-Stable hasbisect OS-Mac
Owner: ntfschr@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce this issue on Mac OS 10.12.4 using chrome latest dev #60.0.3093.0.

Unable to perform per revision bisect results, builds are not getting invoked for the given good and bad ranges. So performed chromium bisect and updating information below.

Bisect Information:
--------------------
You are probably looking for a change made after 469773 (known good), but no later than 469779 (first known bad).

Change Log URL:
https://chromium.googlesource.com/chromium/src/+log/df1ffb753c87b529bca2cc1f42dbd42977955e8f..ea259a34782be0b2cc32d3ced99c845d53d838e2

From the above change log suspecting below change
https://chromium.googlesource.com/chromium/src/+/b6089c6b8758bcca1d154e53d0ba6cd0b1f880b0

ntfschr@ - 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.

Thanks!
Does this break any automated tests? If so, those tests should be updated to be compatible with both the desktop and mobile layouts for interstitials.

If this is just a manual test, is there an actual problem? The point of the CL was to switch to error page mobile layout for more window sizes/resolutions (whenever the desktop layout doesn't fit). Unless there's a serious concern, this is WAI.
Cc: -brajkumar@chromium.org ntfschr@chromium.org
Components: UI>Browser>Interstitials
Owner: brajkumar@chromium.org
This was an intentional change, so I'm going to close as WAI unless there's a real issue I'm overlooking.
Labels: -ReleaseBlock-Stable
Owner: ----
Status: WontFix (was: Assigned)
As per comment #6 confirmed this is an intentional changes from dev, So marking it as wont fix please feel free to reopen this issue in case if it's a valid one.

Thanks!

Sign in to add a comment