Issue metadata
Sign in to add a comment
|
Text on web site is going corrupted and missing on scrolling the page |
||||||||||||||||||||||
Issue descriptionApplication Version (from "Chrome Settings > About Chrome"): 60.0.3110.0 Android Build Number (from "Android Settings > About Phone/Tablet"): 7.0.0 Device: Moto g4 Plus Steps to reproduce: 1) Launch chrome and complete FRE 2) Open any website with text article (eg., m.timesofindia.com) 3) Browse through any article and try to scroll slowly Observed behavior: Text is corrupted Expected behavior: Text should not get corrupted Frequency: 5/5 <number of times you were able to reproduce> Additional comments:
,
May 25 2017
Note: 1) Good Build: 60.0.3109.0 Bad Build: 60.0.3110.0 2) Bisect range: https://chromium.googlesource.com/chromium/src/+log/60.0.3109.0..60.0.3110.0?pretty=fuller&n=10000 3) Please find logs and video @ http://go/chrome-androidlogs1/6/726226 4) Unable to find the exact culprit as an exception "We don't have enough builds to bisect" is getting thrown when tried to bisect builds 5) This issue is observed only on N and O devices not observed on other devices
,
May 25 2017
,
May 25 2017
,
May 25 2017
,
May 25 2017
,
May 25 2017
,
May 25 2017
I *think* this is only happens when gpu raster is enabled. Someone can double check by setting --disable-gpu-rasterization
,
May 25 2017
Confirmed, only reproes with GPU raster.
,
May 25 2017
Not see this issue after disable gpu-rasterization
,
May 25 2017
Still repro on ToT r474709
,
May 25 2017
Bisected to this skia roll: https://chromium-review.googlesource.com/513374 Then this skia CL: https://skia-review.googlesource.com/17824
,
May 25 2017
still reverts cleanly and fixes this. revert first, ask questions later?
,
May 25 2017
oh already reverted
,
May 25 2017
Yep. I will need to merge the revert to the Skia branch for M60, which has already been created.
,
May 25 2017
The following revision refers to this bug: https://skia.googlesource.com/skia/+/1c214313248a4b5a69af14539608c54fb67c2bf8 commit 1c214313248a4b5a69af14539608c54fb67c2bf8 Author: Brian Salomon <bsalomon@google.com> Date: Thu May 25 20:03:07 2017 Revert "Clear atlas textures at creation in Chrome" This reverts commit 8785df15fe5a57651597d138d3c5aea0ffe3f661. Reason for revert: Chrome bug Bug: chromium:726226 Original change's description: > Clear atlas textures at creation in Chrome > > Bug: chromium:656320 > Change-Id: Ia65274aa733f199be188579821e745920493aefc > Reviewed-on: https://skia-review.googlesource.com/17824 > Reviewed-by: Robert Phillips <robertphillips@google.com> > Commit-Queue: Brian Salomon <bsalomon@google.com> > TBR=bsalomon@google.com,robertphillips@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: chromium:656320 Change-Id: Ibc4dd7f184866b58481f2dc7e7e88da9169e7adc Reviewed-on: https://skia-review.googlesource.com/17988 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com> [modify] https://crrev.com/1c214313248a4b5a69af14539608c54fb67c2bf8/include/gpu/GrCaps.h [modify] https://crrev.com/1c214313248a4b5a69af14539608c54fb67c2bf8/src/gpu/GrDrawOpAtlas.cpp [modify] https://crrev.com/1c214313248a4b5a69af14539608c54fb67c2bf8/src/gpu/gl/GrGLCaps.cpp [modify] https://crrev.com/1c214313248a4b5a69af14539608c54fb67c2bf8/src/gpu/GrCaps.cpp
,
May 25 2017
The following revision refers to this bug: https://skia.googlesource.com/skia/+/89ca3a3b0210355a40bc0bc0ffd0104f91b67997 commit 89ca3a3b0210355a40bc0bc0ffd0104f91b67997 Author: Brian Salomon <bsalomon@google.com> Date: Thu May 25 20:28:48 2017 Revert "Clear atlas textures at creation in Chrome" This reverts commit 8785df15fe5a57651597d138d3c5aea0ffe3f661. Reason for revert: Chrome bug Bug: chromium:726226 Original change's description: > Clear atlas textures at creation in Chrome > > Bug: chromium:656320 > Change-Id: Ia65274aa733f199be188579821e745920493aefc > Reviewed-on: https://skia-review.googlesource.com/17824 > Reviewed-by: Robert Phillips <robertphillips@google.com> > Commit-Queue: Brian Salomon <bsalomon@google.com> > TBR=bsalomon@google.com,robertphillips@google.com Bug: chromium:656320 No-Tree-Checks: true No-Try: true No-Presubmit: true Change-Id: Ibc4dd7f184866b58481f2dc7e7e88da9169e7adc Reviewed-On: https://skia-review.googlesource.com/17988 Reviewed-By: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-on: https://skia-review.googlesource.com/17991 [modify] https://crrev.com/89ca3a3b0210355a40bc0bc0ffd0104f91b67997/include/gpu/GrCaps.h [modify] https://crrev.com/89ca3a3b0210355a40bc0bc0ffd0104f91b67997/src/gpu/GrDrawOpAtlas.cpp [modify] https://crrev.com/89ca3a3b0210355a40bc0bc0ffd0104f91b67997/src/gpu/gl/GrGLCaps.cpp [modify] https://crrev.com/89ca3a3b0210355a40bc0bc0ffd0104f91b67997/src/gpu/GrCaps.cpp
,
May 26 2017
This is reverted on Skia master and Skia M60. However, the Skia autoroller is paused until Chrome branches for M60. At that point the M60 branch of Chrome will pick up the M60 branch of Skia and get the fix. Also, the autoroller will restart and Chrome master will get the fix. I've been able to reproduce this locally on several phones with different Adreno GPUs but not on my linux desktop or phones with GPUs from different vendors. I'm starting to wonder if this is a driver issue.
,
May 26 2017
Issue 726587 has been merged into this issue.
,
May 26 2017
Issue 726357 has been merged into this issue.
,
May 26 2017
This appears to be an Adreno driver bug when we clear the single channel texture used for the grayscale glyph atlas. I've made the following discoveries: Inserting a glFlush() after the clear fixes it. Oddly this still reproduces even when glColorMask is set to all GL_FALSE before glClear(GL_COLOR_BUFFER_BIT) is called. I don't seem to be able to reproduce this in standalone Skia testing, just in Chrome. Perhaps there is something done by the command buffer or gpu process that is necessary for the repro.
,
May 26 2017
Issue 726756 has been merged into this issue.
,
May 26 2017
Branch 3112 is the M60 candidate (we'll still need explicit confirmation but I expect it shortly). I pinned that branch to the m60 Skia branch and kicked a build, should deploy in ~2-3 hours.
,
May 26 2017
I checked all the duplicate bug reports and they're on various Adreno 4xx and Adreno 5xx. I'd suggest adding a GPU-service-side workaround targeted to those GPU ranges before the reland.
,
May 26 2017
,
May 26 2017
,
May 26 2017
,
May 26 2017
60.0.3112.3 is rolling out to canary now and will fix this.
,
May 28 2017
,
May 29 2017
This issue is not reproducible on all devices on latest 60.0.3112.6, hence verified
,
May 29 2017
Issue 726802 has been merged into this issue.
,
May 29 2017
Issue 727027 has been merged into this issue.
,
May 29 2017
Issue 726528 has been merged into this issue. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by acindhe@chromium.org
, May 25 2017