New issue
Advanced search Search tips

Issue 887231 link

Starred by 1 user
Status: Verified
Owner:
rbpotter@chromium.org
Closed: Sep 24
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: [Print Preview]'Out of bounds page' error message disappears after clicking on it.

Reported by dchau...@etouch.net, Sep 20 
Chrome Version: 71.0.3557.0 (Official Build) Revision	409ad95fbafba7ca90dc799c9a0be24d3abb549f-refs/branch-heads/3557@{#1} (32/64-bit)
OS: Windows(7,8,8.1,10), Linux(14.04 LTs).

1. Launch Chrome and give print command on any webpage.
2. Type any number (For Ex: 54) in 'Pages' text-box so that 'Out of bounds page' error message appears.
3. Now click on "Out of bounds page reference. limit is 1" error message and observe.

Actual: 'Out of bounds page' error message disappears after clicking on it.
Expected: 'Out of bounds page' error message should not disappear after clicking on it.

This is a regression issue, broken in M-71 series, will soon update other info.

Kindly review the attached screen-cast for reference.

Thank you.
Actual behavior.mp4
325 KB View Download
Expected behavior.mp4
480 KB View Download

Comment 1 by dchau...@etouch.net, Sep 20

Labels: hasbisect-per-revision OS-Mac
Owner: rbasuvula@chromium.org
Status: Assigned (was: Unconfirmed)
Below is manual regression range:

Good build: 71.0.3556.0 (Revision: 592302)
Bad build: 71.0.3557.0 (Revision: 592637)

Using the 'per-revision' script providing the bisect result:

You are probably looking for a change made after 592628 (known good), but no later than 592629 (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/ed308de938fdfb207110ce1d1cd196a6159fe3eb..e6113ed7a9644d287e512d78a17a08be0c7e2818

Suspecting: https://chromium.googlesource.com/chromium/src/+/e6113ed7a9644d287e512d78a17a08be0c7e2818

@rbpotter: 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.

NOTE: This issue is also reproducible on Mac(10.12.6, 10.13.1, 10.13.6, 10.14) OS.

Thank you.

Comment 2 by dchau...@etouch.net, Sep 20

Owner: rbpotter@chromium.org

Comment 3 by rbasuvula@chromium.org, Sep 20

Labels: ReleaseBlock-Stable
As this is a recent regression adding release blocker label for this issue.Please reduce priority or remove if not the case.

Thank You!
Project Member

Comment 4 by bugdroid1@chromium.org, Sep 22 

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

commit 4756a5ebdcd7ced17fe797a74da76725e11f9811
Author: rbpotter <rbpotter@chromium.org>
Date: Sat Sep 22 00:42:38 2018

Print Preview Refresh: Fix some pages bugs

Splitting out fixes for bugs from change to dropdown, in case we decide
not to keep dropdown change. Addresses some focus issues, and removes
use of auto-validate. Auto-validate does not work well for this section
as some errors (e.g. out of range) cannot be caught by the auto
validation. Having 2 different sets of logic for setting the input
invalid (auto-validate + JS validation) causes some bugs.

Bug:  887231 ,  886591 ,  886542 ,  885094 ,  886560 
Change-Id: I053899718a735fa8bcc23df27d36b23d2c26e0b0
Reviewed-on: https://chromium-review.googlesource.com/1238868
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#593394}
[modify] https://crrev.com/4756a5ebdcd7ced17fe797a74da76725e11f9811/chrome/browser/resources/print_preview/new/input_behavior.js
[modify] https://crrev.com/4756a5ebdcd7ced17fe797a74da76725e11f9811/chrome/browser/resources/print_preview/new/pages_settings.html
[modify] https://crrev.com/4756a5ebdcd7ced17fe797a74da76725e11f9811/chrome/browser/resources/print_preview/new/pages_settings.js

Comment 5 by dchau...@etouch.net, Sep 24

Labels: TE-Verified-M71 TE-Verified-71.0.3559.0
Updated:
Tested this issue on Windows (7, 8, 8.1, 10), Mac(10.12.6, 10.13.1, 10.13.6, 10.14) and Linux(14.04 LTS) machines using latest Canary build #71.0.3559.0 and the issue is fixed. Hence adding TE-Verified labels. 

Kindly review an attached screen-cast for reference.

Thank you..!
Fixed_behavior.mp4
409 KB View Download

Comment 6 by rbpotter@chromium.org, Sep 24

Status: Verified (was: Assigned)
►

Sign in to add a comment