Issue metadata
Sign in to add a comment
|
Chrome window move to different monitor to address-bar, bookmark-bar areas be black-out
Reported by
naruc...@gmail.com,
Dec 21 2017
|
||||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0 Steps to reproduce the problem: 1. Move chrome browser window to different monitor What is the expected behavior? No black out What went wrong? Black out Did this work before? Yes 63.0.3239.84 Chrome version: <Copy from: 'about:version'> Channel: stable OS Version: 10.0 Flash Version:
,
Dec 21 2017
,
Dec 22 2017
Unable to check the issue on reported chrome version 63.0.3239.84 using Windows 10 with the below mentioned steps. 1. Launched Chrome Using Windows 10 2. Connected Dual monitor cable to the cpu of Win 10 OS As the second monitor fails to respond/Going to sleep mode we coudn't check. Tried by having Win7 as Main system too, same issue is seen. @Inhouse: Could some one from TE-HYD team have a look into this issue. Hence adding label TE-NeedsTriageFromHYD. Thanks!
,
Dec 22 2017
Sorry for confuse this post. 63.0.3239.84 is not happen this bug. Exactly start from 63.0.3239.108 to black filled top area of tab.
,
Dec 27 2017
Able to reproduce the issue on Windows 10 with chrome Stable #63.0.3239.108 , Beta #64.0.3282.39, Dev # 65.0.3298.3 & Canary #65.0.3304.0 Issue broken between M64 & M65 Bisect Info: =========== Good build : 64.0.3282.0, Revision Range - 520840 Bad build : 65.0.3285.0, Revision Range - 521571 Manual Change Log between Good & Bad builds ====================================== https://chromium.googlesource.com/chromium/src/+log/64.0.3282.0..65.0.3285.0?pretty=fuller&n=10000 The suspecting Change Log is : ----------- https://chromium.googlesource.com/chromium/src/+/a9634625b03962f7fdb714153a6fb8564da4084b Note: 1. Unable to run per-revision bisect script, since it is breaking on Windows 10 machine. 2. Not able to repro on Ubuntu 14.04 & Mac 10.12.6 ccameron@- Could you please look into this issue, if it's related to your change? if not could you please help us to reassign this issue to the right owner.
,
Dec 30 2017
I can confirm that I also see the problem with Chrome 63.0.3239.108 (Official Build)(64-bit) on Windows 10 1709 (build 16299.125).
,
Jan 3 2018
The owner of this issue is ooo , hence adding the reviewer of the file as per comment #5 for more updates on this issue. Thanks!
,
Jan 9 2018
This issue is marked as RB-Stable for M64, could any one let us know is there any latest update available on this issue? Thanks!
,
Jan 9 2018
I will take a look at this next week when I have access to my Windows machine. In the mean time, please navigate to about:gpu in Canary on an affected machine print the result as a PDF, and attach that.
,
Jan 10 2018
As per comment #9, attaching the GPU of affected windows 10 machine on chrome canary #65.0.3316.0
,
Jan 16 2018
ccameron@, Attached GPU details as per C#9,Could you please take a look as it is marked as stable blocker. Thanks..!
,
Jan 16 2018
We're not planning any further M63 releases. abdulsyed@ to evaluate for M64.
,
Jan 18 2018
Thanks, I'll try to reproduce this
,
Jan 18 2018
,
Jan 18 2018
,
Jan 18 2018
,
Jan 18 2018
Hmm, seeing lots of dupes of this, but still can't repro :/
,
Jan 19 2018
windows 10 machine on chrome canary 65.0.3325.0
,
Jan 22 2018
I think that this is a damage tracking bug. A side-effect of changing the color space on the ui::Compositor is that we call cc::Layer::SetNeedsDisplay on every layer in the tree. That side-effect is making the layers not appear black. Digging in a bit more here...
,
Jan 22 2018
Forcing a re-display of the full window every frame results in lots of black flickering ... current_frame()->root_damage_rect = gfx::Rect(device_viewport_size); We call this whenever reshape is called, which happens for output color space changes, among many other things...
,
Jan 22 2018
,
Jan 22 2018
,
Jan 22 2018
Actually, I'll do a separate fix for this -- that bug is tricky.
,
Jan 24 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4f96f1f0fbf78ea9b6d6ccb0e9f6ec995497a0c3 commit 4f96f1f0fbf78ea9b6d6ccb0e9f6ec995497a0c3 Author: Christopher Cameron <ccameron@chromium.org> Date: Wed Jan 24 07:19:34 2018 Work around black flashing in ui::Compositor The root cause for this bug is crbug.com/804430, where we optimize away required quads because we aren't aware of a reshape. The fix for that bug will be subtle, and this needs merges, so just lie and say that we changed the UI color space even if it's still always just sRGB. Bug: 796828 Change-Id: I23929eb92a780020f67f003985823b01a77901e3 Reviewed-on: https://chromium-review.googlesource.com/879395 Reviewed-by: weiliangc <weiliangc@chromium.org> Commit-Queue: ccameron <ccameron@chromium.org> Cr-Commit-Position: refs/heads/master@{#531470} [modify] https://crrev.com/4f96f1f0fbf78ea9b6d6ccb0e9f6ec995497a0c3/ui/compositor/compositor.cc
,
Jan 25 2018
Verified the fix on Windows10 using Chrome version #66.0.3331.0 as per the comment #0. Attaching screen cast for reference. Observed that the movement of window from one monitor to other is clear with out any black-out Hence, the fix is working as expected. Adding the verified labels. Thanks...!!
,
Jan 25 2018
Adding 65 merge request
,
Jan 26 2018
Your change meets the bar and is auto-approved for M65. Please go ahead and merge the CL to branch 3325 manually. Please contact milestone owner if you have questions. Owners: cmasso@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jan 26 2018
Pls merge your change to M65 branch 3325 ASAP so we can pick it up for next M65 dev release. Thank you.
,
Jan 29 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/49614d7b9e7d8dc3b0a4c4b8c576a3e7753ff859 commit 49614d7b9e7d8dc3b0a4c4b8c576a3e7753ff859 Author: Christopher Cameron <ccameron@chromium.org> Date: Mon Jan 29 18:52:48 2018 Work around black flashing in ui::Compositor The root cause for this bug is crbug.com/804430, where we optimize away required quads because we aren't aware of a reshape. The fix for that bug will be subtle, and this needs merges, so just lie and say that we changed the UI color space even if it's still always just sRGB. TBR=ccameron@chromium.org (cherry picked from commit 4f96f1f0fbf78ea9b6d6ccb0e9f6ec995497a0c3) Bug: 796828 Change-Id: I23929eb92a780020f67f003985823b01a77901e3 Reviewed-on: https://chromium-review.googlesource.com/879395 Reviewed-by: weiliangc <weiliangc@chromium.org> Commit-Queue: ccameron <ccameron@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#531470} Reviewed-on: https://chromium-review.googlesource.com/891683 Reviewed-by: ccameron <ccameron@chromium.org> Cr-Commit-Position: refs/branch-heads/3325@{#147} Cr-Branched-From: bc084a8b5afa3744a74927344e304c02ae54189f-refs/heads/master@{#530369} [modify] https://crrev.com/49614d7b9e7d8dc3b0a4c4b8c576a3e7753ff859/ui/compositor/compositor.cc
,
Jan 30 2018
,
Jan 30 2018
Verified the fix on Windows 10 using Chrome version #65.0.3325.31 as per the comment #0. Attaching screen cast for reference. Observed that the movement of window from one monitor to other is clear with out any black-out Hence, the fix is working as expected. Adding the verified labels. Thanks...!! |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by naruc...@gmail.com
, Dec 21 20179.5 MB
9.5 MB Download