New issue
Advanced search Search tips

Issue 886542 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: Error message keeps refreshing for every wrong number of entry in pages text field on print preview.

Reported by pranjali...@etouch.net, Sep 19 
Chrome version :71.0.3555.0 (Official Build) 6c42d21566c861ad6a7d4adb5ac47bdd3f4c473d-refs/branch-heads/3555@{#1}(32/64 bit)

OS:  Win(7,8,8.1,10) , Mac(10.12.6 , 10.13.1 , 10.13.6 , 10.14)  and Linux(14.04 LTS) OS

What steps will reproduce the problem?
1.Launch chrome and open NTP.
2.Give print command and enter wrong digits in pages text field.
3.Observe.

Actual  : Error message keeps refreshing for every wrong number of entry in pages text field on print preview.
Expected: Error message should not keep refreshing for every wrong number of entry in pages text field on print preview.

This is a regression issue broken in ‘M-71’ and below is the 'Chromium bisect' information:
Good Build: 71.0.3551.0 (Revision: 590851)
Bad Build : 71.0.3552.2 (Revision: 591210)

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

You are probably looking for a change made after 591108 (known good), but no later than 591109 (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/974aa7b559b4361d7410479796becb0b933dc6a4..e73e92b7fbf4206ffdb20282ac73bcbed7cf910e

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

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

Kindly review the attached screen-cast for reference.

Thank you.
Actaul Result.mp4
566 KB View Download
Expected Result.mp4
593 KB View Download
Project Member

Comment 1 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 2 by pranjali...@etouch.net, Sep 24

Labels: TE-Verified-M71 TE-Verified-71.0.3559.0
Update:

Rechecked the above issue on Win(7,8,8.1,10) , Mac(10.12.6 , 10.13.1 , 10.13.6 , 10.14)  and Linux(14.04 LTS) OS using latest canary build#71.0.3559.0 and issue is fixed.

Kindly refer attached screen cast.

Thank You..
Canary Behaviour.mp4
491 KB View Download

Comment 3 by rbpotter@chromium.org, Sep 24

Status: Verified (was: Assigned)
►

Sign in to add a comment