Issue metadata
Sign in to add a comment
|
Regression: Copies count doesn't decreases according to click on print preview page.
Reported by
dchau...@etouch.net,
Dec 15 2017
|
||||||||||||||||||||||
Issue descriptionChrome Version: 65.0.3294.5 (Official Build) 682d479193fcc856b8a99b7250773802b3f631ff-refs/branch-heads/3294@{#7} 32/64-bit. OS: Windows(7,8,10). What steps will reproduce the problem? 1. Launch chrome, go to NTP and give print command. 2. Enter number 20 or any number in 'Copies' text-box. 3. Now multiple times click on down arrow to decrease the copies count and observe. Copies count doesn't decreases as per click. Copies count should decrease as per click. NOTE: 1. This issue is also seen in 'Scale' text-box. 2. This issue is not seen on Mac and Linux OS. This is a Windows specific regression issue, broken in M-65 series, below is manual regression range. Good build: 65.0.3293.0 Bad build: 65.0.3294.0 Kindly review the attached screen-cast for reference.
,
Dec 15 2017
Adding RB Label as this is a recent Regression. Please remove if not required. Thank You.
,
Dec 15 2017
Confirmed this issue is from my CL. reverting.
,
Dec 15 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ffe75f56318e0e3c2f29ee0a1156c85f83dfad2e commit ffe75f56318e0e3c2f29ee0a1156c85f83dfad2e Author: Jianpeng Chao <chaopeng@chromium.org> Date: Fri Dec 15 19:22:35 2017 Revert "Fix Windows touchpad lagging in first tab" This reverts commit 12dca8659ca7a447e813d0d25f930cf7932b87fd. Reason for revert: crbug.com/795223 Original change's description: > Fix Windows touchpad lagging in first tab > > Currently the receiving of WM_MOUSEWHEEL messages lags when we open a > URL on a new window until we resize. The issue was introduced by > r384698. That CL change wait for MOUSE_EVENT to wait for SENDMESSAGE > in MessagePumpForUI::WaitForWork to optimize the GPU thread. But the > GPU thread does not use MessagePumpForUI anymore so it should be OK to > just remove that code. > > The updated version of Chrome was tested to make sure that the hangs > that were occasionally seen when using chrome://tracing are still gone. > > After this CL chaopeng@ need to monitor to make sure that the crash > rate not increase. > > Bug: 713907 , 596190 > Change-Id: I7545a867de4851acd4e57c9b8c7a3dd05def1dd8 > Reviewed-on: https://chromium-review.googlesource.com/809829 > Commit-Queue: Jianpeng Chao <chaopeng@chromium.org> > Reviewed-by: Bruce Dawson <brucedawson@chromium.org> > Reviewed-by: Lei Zhang <thestig@chromium.org> > Cr-Commit-Position: refs/heads/master@{#523689} TBR=thestig@chromium.org,brucedawson@chromium.org,chaopeng@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 713907 , 596190, 795223 Change-Id: I3f53fd77e7a6545cd214444cac46709cff0142c9 Reviewed-on: https://chromium-review.googlesource.com/830174 Commit-Queue: Jianpeng Chao <chaopeng@chromium.org> Reviewed-by: Jianpeng Chao <chaopeng@chromium.org> Cr-Commit-Position: refs/heads/master@{#524428} [modify] https://crrev.com/ffe75f56318e0e3c2f29ee0a1156c85f83dfad2e/base/message_loop/message_pump_win.cc
,
Dec 15 2017
I check spyxx with my CL, the mouse button down/up are route to correct window.
,
Dec 18 2017
Update:- Tested this issue on Windows (7,8,10) machines using latest Chrome canary build# 65.0.3298.0 and fix is working as expected i.e. Copies count decreases/increases properly according to the click. Hence adding TE Verified labels. Please find the attached screen-cast for reference. Thanks..!
,
Jan 6 2018
,
Jan 24 2018
The new approach of 713907, 596190 does not effect print page. Close this issue. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by dchau...@etouch.net
, Dec 15 2017Owner: chaopeng@chromium.org
Status: Assigned (was: Unconfirmed)