1026271 - Remove the limitation related to the absolutely positioned frames in async scrolling

Status: VERIFIED FIXED

(Core :: Graphics, defect, P2)

Description

[Sandip Kamat]

Remove the limitation related to the absolutely positioned frames in async scrolling

Description: Nested scrolling frames sometimes have to clip an absolutely positioned items they do not contain.  APZ does not properly handle this scenario, which means that we have to fall back into sync scrolling in those cases.

Comment 1

[Kartikaya Gupta (email:[e-mail])]

Can you provide a test case for what you're describing? How is this different from bug 967844?

Comment 2

[Ivan Tsay (:ITsay)]

Create dependency to graphic features meta bug used for Forefox OS 2.1.

Comment 3

[Kartikaya Gupta (email:[e-mail])]

Based on the email thread this is just a bug for product management purposes. The implementation is entirely covered by 967844.

Comment 4

[No-Jun Park [:njpark]]

Testing Perspective:
- Use https://bug1011639.bugzilla.mozilla.org/attachment.cgi?id=8424102 to toggle APZC
- Offline discussion with milan to detect APZC scrolling easily, & create more real-life example cases

Comment 5

[No-Jun Park [:njpark]]

Testing Perspective:
- Investigate use of nsIDOMWindowUtils.getCompositorAPZTestData()  in bug 961289 to detect APZC, if the test app gives access to nsIDOMWindowUtils.
- Investigate use of apz.printtree pref and counting active APZC in the log (Bug 958596)

Comment 6

[Kartikaya Gupta (email:[e-mail])]

Bug 1035917 was the thing that added a message to logcat when we fall back to sync scrolling. I also filed bug 1053992 to add a visual indicator for this.

Comment 7

[Milan Sreckovic [:milan] (needinfo for best results)]

This is not actionable (though there may be some action after bug 967844 is resolved), but assigning anyway.

Comment 8

[No-Jun Park [:njpark]]

moztrap link:
https://moztrap.mozilla.org/manage/case/14506/

Comment 9

[Milan Sreckovic [:milan] (needinfo for best results)]

All the code for this was handled in bug 967844 which is in 2.1.  Not sure what 2.1-feature-slip+ means, but I'll leave it there, and just close this bug.

Comment 10

[Jason Smith [:jsmith]]

(In reply to Milan Sreckovic [:milan] from comment #9)
> All the code for this was handled in bug 967844 which is in 2.1.  Not sure
> what 2.1-feature-slip+ means, but I'll leave it there, and just close this
> bug.

I was just tagging features that are landing post FL. If bug 967844 did land before merge, then I'll remove the flag.

Comment 11

[Jason Smith [:jsmith]]

bug 1064840 has landed now, which means this should be possible to verify now.

Comment 12

[No-Jun Park [:njpark]]

Verified with https://bug1011639.bugzilla.mozilla.org/attachment.cgi?id=8424102 and http://people.mozilla.org/~bballo/multi-layer-apz.html  Also other often-used sites/apps scrolled properly without the warning indicator.