OOPIF: Continuous resizing of content while scrolling |
||||||
Issue descriptionThe size provided in ResizeParams to an OOPIF's RenderWidget is in DIP (device independent pixels) and is a function of the scroll position of the page because it uses ScaleToEnclosingRect. This can cause scrollbars to show up and disappear and causes blink to churn through unnecessary relayout eating up battery while scrolling. The size of an OOPIF should be stable as scrolling happens.
,
Jan 23 2018
I should add that this only happens on high DPI.
,
Feb 5 2018
Friendly ping. Do we have a lead on how to fix, or anyone who may be able to help? Ken/Fady: Can you also clarify whether there's a functional problem due to this bug, or if it's mainly a performance issue?
,
Feb 7 2018
Feel free to re-add Component if the more specific is helpful later.
,
Feb 13 2018
Fady: To clarify, does this mean that the main frame is scrolling, and the OOPIF is getting continuous resize messages because of rounding changes in the frame rect size in the parent's layout? Do you know how this doesn't happen for local frames?
,
Feb 27 2018
+fsamuel@
,
Mar 2 2018
GPU triage: ping, what is the next step here?
,
Mar 2 2018
Ohh I actually think I fixed this one a while ago. https://chromium-review.googlesource.com/c/chromium/src/+/891663 The physical backing size is now a function of the "local frame size". The frame rect is no longer used. I'm marking this as FIXED. Yaay!
,
Mar 2 2018
[Auto-generated comment by a script] We noticed that this issue is targeted for M-66; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-66 label, otherwise remove Merge-TBD label. Thanks.
,
Mar 7 2018
Is there any merge need to M66 or M65? Is yes, pls request a merge otherwise remove "Merge-TBD" label. Thank you.
,
Mar 8 2018
I don't think there is anything further to do. The fix is on the 66 branch, and this isn't high enough priority to warrant a merge to 65.
,
Mar 8 2018
Removing "Merge-TBD" label per comment #11. Thank you. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by creis@chromium.org
, Jan 23 2018Components: Internals>Compositing>Scroll
Labels: -Pri-3 M-66 OS-Chrome OS-Linux OS-Mac OS-Windows Pri-2
Owner: kenrb@chromium.org
Status: Assigned (was: Untriaged)