Issue metadata
Sign in to add a comment
|
Regression:Unwanted black line is seen after opening 'Language drop down list' in "support.google.com"
Reported by
adha...@etouch.net,
Dec 15 2016
|
||||||||||||||||||||||
Issue description57.0.2952.0 (Official Build)199daadb512fd39c2c8c3e56f536acb7eb941fb4-refs/heads/master@{#438707} (32/64-bit) OS:Windows(7,8,10) What steps will reproduce the problem? (1)Launch chrome and navigate to https://support.google.com/chrome/?p=help&ctx=menu#topic=3227046 (2)Scroll at the bottom of the page and click on 'Language drop down list'(Kindly refer the video) (3)Observe the scroll bar for 'Language drop down list'. Actual:Unwanted black line is seen after opening 'Language drop down list'. Expected:No such unwanted black line should be seen after opening 'Language drop down list'. This is a Regression issue broken in M-57,will soon update other info. Good build:57.0.2950.0 Bad build:57.0.2951.0 Note:Note: This is Windows specific issue seen on Screen resolution 1280x1024.
,
Dec 15 2016
This is likely caused by my change, as it changes where the background contents are painted. There is likely some underlying bug when painting into the composited scrolling layer. Unfortunately I am unable to reproduce on my home Windows desktop on 57.0.2952.1 , with the screen resolution set to 1280x1024. I will attempt to locate a Windows machine in the office and try again today.
,
Dec 15 2016
Seems like this might have the same underlying cause as issue 664904 .
,
Dec 15 2016
,
Dec 15 2016
I can't repro on either Win 10 or Win 7 with 57.0.2952.0. That's at 2560x1600.
,
Dec 15 2016
But I can repro at 1280x1024 and in fact if you set the browser window size to 1282 (and probably 1280) it also reproduces. It may reproduce on other platforms at that size too. Neither of the other "Unwanted black line" bugs reproduce at that resolution.
,
Dec 15 2016
Yes, I managed to reproduce a similar or identical issue on Linux 57.0.2953.0 (Developer Build) (64-bit), by launching with: ./out/Default/chrome --window-size="1282x1024" https://support.google.com/chrome/?p=help&ctx=menu#topic=3227046 See attached image. I will now check that reverting the revert fixes this issue, so that we have a known path out of the regression :).
,
Dec 15 2016
Confirmed that reverting c765a85995920f79d3132e06a2c146316b823c3f gets rid of the cyan line in my reproduction. I assume there is a painting bug when the background is being painted to the composited scrolling contents layer, unsure why it would be cyan though. Given that we are fast approaching holiday season, I don't expect to have time to look at the underlying cause. Unless anyone else wants to take this over (speak up soon :)), later today I will revert c765a85995920f79d3132e06a2c146316b823c3f and re-open the original http://crbug.com/646464 with additional details from this bug.
,
Dec 15 2016
The cyan color is used in a debug build to indicate unpainted area of a layer that is supposed to be fully opaque.
,
Dec 15 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4baddb7eb1745ba1f39697e89e303805f90efe16 commit 4baddb7eb1745ba1f39697e89e303805f90efe16 Author: smcgruer <smcgruer@chromium.org> Date: Thu Dec 15 19:48:37 2016 Revert of Revert "Disable local background equivalence when we have a box shadow due to painting bug." (patchset #5 id:80001 of https://codereview.chromium.org/2561183002/ ) Reason for revert: Caused a regression in drawing backgrounds when the window was sized around 1280x1024. Regression seen in at least Windows and Linux - see http://crbug.com/674429 BUG= 674429 Original issue's description: > Revert "Disable local background equivalence when we have a box shadow due to painting bug." > > This reverts commit a12e7a002f85301bf013b45e6e8bb61455304a76. > > BUG= 646464 > > TEST=Visit http://output.jsbin.com/vucefi, confirm that both boxes have > box shadows and the first box has sub-pixel AA text > > Committed: https://crrev.com/c765a85995920f79d3132e06a2c146316b823c3f > Cr-Commit-Position: refs/heads/master@{#438020} TBR=wangxianzhu@chromium.org,chrishtr@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG= 646464 Review-Url: https://codereview.chromium.org/2576423002 Cr-Commit-Position: refs/heads/master@{#438897} [modify] https://crrev.com/4baddb7eb1745ba1f39697e89e303805f90efe16/third_party/WebKit/LayoutTests/compositing/overflow/scrollbar-layer-placement-expected.txt [add] https://crrev.com/4baddb7eb1745ba1f39697e89e303805f90efe16/third_party/WebKit/LayoutTests/platform/android/compositing/overflow/scrollbar-layer-placement-expected.txt [add] https://crrev.com/4baddb7eb1745ba1f39697e89e303805f90efe16/third_party/WebKit/LayoutTests/platform/mac-mac10.10/virtual/prefer_compositing_to_lcd_text/compositing/overflow/scrollbar-layer-placement-expected.txt [add] https://crrev.com/4baddb7eb1745ba1f39697e89e303805f90efe16/third_party/WebKit/LayoutTests/platform/mac-retina/virtual/prefer_compositing_to_lcd_text/compositing/overflow/scrollbar-layer-placement-expected.txt [add] https://crrev.com/4baddb7eb1745ba1f39697e89e303805f90efe16/third_party/WebKit/LayoutTests/platform/win/virtual/prefer_compositing_to_lcd_text/compositing/overflow/scrollbar-layer-placement-expected.txt [modify] https://crrev.com/4baddb7eb1745ba1f39697e89e303805f90efe16/third_party/WebKit/Source/core/layout/LayoutBoxModelObject.cpp
,
Dec 15 2016
,
Jan 3 2017
Tested the issue on Latest Dev# 57.0.2970.0 on Windows and issue is no more reproducible. Hence adding TE-Verified Labels. Note: Adding Screenshot for further reference. Thank You.
,
Jan 3 2017
Forgot to attach screenshot. Thank You. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by hdodda@chromium.org
, Dec 15 2016Labels: hasbisect-per-revision ReleaseBlock-Stable
Owner: smcgruer@chromium.org
Status: Assigned (was: Unconfirmed)