Directly composited images take too much memory
Reported by
slavoroi...@gmail.com,
Apr 25 2016
|
||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.75 Safari/537.36 Example URL: https://jsfiddle.net/7abx92ke/1/ Steps to reproduce the problem: 1. Open the link : https://jsfiddle.net/7abx92ke/1/ 2. Zoom out 3. The images don't render properly. What is the expected behavior? The images should render properly. What went wrong? Every time we add a 3d (matrix3d translateZ and cet...) transform for an img element and zoom out it doesn't render the images fully. It works for explorer and firefox! Does it occur on multiple sites: N/A Is it a problem with a plugin? No Did this work before? N/A Does this work in other browsers? Yes Chrome version: 50.0.2661.75 Channel: n/a OS Version: 7 Flash Version: Shockwave Flash 21.0 r0 http://stackoverflow.com/questions/36689508/high-quality-images-rendering-in-chrome
,
Apr 26 2016
,
Apr 26 2016
I don't think it's the same issue as ours, we don't have a blur problem but a rendering one.I tried to degrade to chrome 45 and it didn't work for example.
,
Apr 26 2016
Thank you for providing more feedback. Adding requester "tkonchada@chromium.org" for another review and adding "Needs-Review" label for tracking. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Apr 26 2016
This is a severe issue. Makes it impossible to heavily use CSS properties, such as translate3d matrix3d rotate3d, that relies on GPU action. And it's a shame because they can be used to accomplish great things.
,
Apr 26 2016
The browser is at 100% GPU memory usage. The image concerns ourjsfiddle example. We used the chrome rendering settings. Thanks!
,
Apr 26 2016
And it happens on Mac, Linux, Windows 7/10 (newest chrome versions).
,
Apr 26 2016
Lets see if a bisect reveals anything.
,
Apr 26 2016
Able to reproduce the issue on Win7/64 bit, Mac OSX [10.11.4] & Linux/Ubuntu 14.04 - 50.0.2661.87 This is observed when the zoom out level is around 67% or less. It seems like not a regression issue as this is seen on M47 build Version 47.0.2526.111 as well. Please find the attached screen shot.
,
Apr 26 2016
,
Apr 26 2016
These are directly composited images, which are forced to raster at the full resolution of the image. It may be the time to kill these things. But probably in conjunction with a change to re-raster on every frame on scale change.
,
Apr 27 2016
As per comment #12 this seems to be a non regression issue. Hence removing the Needs-Bisect label. Please tag the label if required.
,
Apr 29 2016
Vlad, is this sufficiently awesome now with your work?
,
Apr 29 2016
No, this is suffering from oom tiles. Image decodes don't really have anything to do with this, it's just trying to display all of that content (with composited images) will cause us to be out of (tile) memory, which is why checkerboards remain there permanently (we also don't have direct picture rendering with the delegated compositor). The action plan here I believe is to do a heuristic that might do several steps of raster scales so that in this specific situation we'd use as much memory as with non composited images.
,
May 2 2016
Experiencing the issue too, with even as few as 4-5 JPEGs (though large ones, about 1MB each). In my instance, the container on which I place the images also enables zoom and pan abilities, and using these freezes the entire Chrome process. [Chrome 40.0.2214.91 on Linux Mint]
,
May 3 2016
,
May 15 2016
After a few tests we tried a div with a background-image (which is not a great workaround) and it worked on a small app: http://jsfiddle.net/7abx92ke/5/ (You can play with the width and height). But, on our app, we have our zoom (scene viewer) so we can further zoom out and it still kills the browser, and in other browsers it doesn't. Maybe our extra example will help you.
,
May 17 2016
,
Jun 3 2016
Hi, Is there any update on the issue in hand? or a workaround? Thanks for any feedback.
,
Jun 14 2016
Hi there, Any news from April?
,
Jun 14 2016
,
Jun 16 2016
Any update? A fix is really needed
,
Jun 17 2016
No update, sorry.
,
Jun 19 2016
Hi, Could you please provide us some information what are you going to do with this bug? Are you going to fix it soon?
,
Jun 24 2016
,
Jun 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c05804da22bca99f807b9198465dbe1318dbfa76 commit c05804da22bca99f807b9198465dbe1318dbfa76 Author: vmpstr <vmpstr@chromium.org> Date: Mon Jun 27 23:48:50 2016 cc: Change directly composited image raster source scale occasionally. This patch changes the behavior of directly composited images in the following way: - The scale starts at 1.f. - It is halved if the ideal source scale is less than half of the raster scale. - It is doubled if the ideal source scale is greater than the raster scale. R=enne, chrishtr, danakj BUG= 606287 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review-Url: https://codereview.chromium.org/2100483003 Cr-Commit-Position: refs/heads/master@{#402333} [modify] https://crrev.com/c05804da22bca99f807b9198465dbe1318dbfa76/cc/layers/picture_layer_impl.cc [modify] https://crrev.com/c05804da22bca99f807b9198465dbe1318dbfa76/cc/layers/picture_layer_impl_unittest.cc
,
Jun 28 2016
Thank you very much, when can we test it?
,
Jun 28 2016
This should be in either today's or tomorrow's canary builds. Please update when you can verify.
,
Jun 29 2016
We verified the issue and the performance and stability has definitely improved. The GPU Memory used dropped significantly. Still, we are seeing a difference in performance compared to Firefox which appears to render faster: this is mostly visible in our application but also in the above fiddle. Thank you for attending this issue. When is this canary version (53.0.2780.0) expected to be in a stable version?
,
Aug 11 2016
Hi, Can we please get an update about the expected date in which this fix will be deployed to a stable release? Thanks,
,
Aug 11 2016
This should make it to stable within a few weeks. You can see the current releases on omahaproxy.appspot.com |
||||||||||||||
►
Sign in to add a comment |
||||||||||||||
Comment 1 by tkonch...@chromium.org
, Apr 26 2016Labels: Needs-Feedback