Chris Lord [:cwiiis] Assignee
	Description • 11 years ago

So, looks like I forgot to change UpdateSubFrame in bug 907743, so the change only applied for the root frame and not for sub-frames (which is good enough for the b2g browser, but significantly less useful when APZC is used outside of that use-case).

Patch incoming.

	Chris Lord [:cwiiis] Assignee
	Comment 1 • 11 years ago

I realise that it would be a smaller delta to take a const FrameMetrics into the function, but I thought it'd be good for it to operate in the same way as the root version, and that we may end up needing it to update the FrameMetrics at some point. Feel free to disagree though and I'll change it :)

	Chris Lord [:cwiiis] Assignee
	Comment 2 • 11 years ago

Comment on attachment 8341139 [details] [diff] [review]
Align sub-frame display ports to tile boundaries

Actually, ignore this for now, the results aren't correct - think I'm making a bad assumption about the scaling...

	Chris Lord [:cwiiis] Assignee
	Comment 3 • 11 years ago

Here's the correct patch. It fixes two bugs in the MaybeAlignAndClampToDisplayPort function:

1- It used mZoom instead of LayersPixelsPerCSSPixel() (which didn't matter on the root frame)
2- It used ClampRect instead of Intersect (which meant the display-port wouldn't be tile-aligned at the very top and near-bottom of the document)

Although the Settings app has some bad behaviour (some of which is the fault of the platform, some of which I think is Gaia's fault), it does what I expect now, and the Contacts app shows pretty decent behaviour.

	Botond Ballo [:botond]
	Comment 4 • 11 years ago

Comment on attachment 8341715 [details] [diff] [review]
Align sub-frame display ports to tile boundaries (v2)

Review of attachment 8341715 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good!

	Chris Lord [:cwiiis] Assignee
	Comment 5 • 11 years ago

Pushed to inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/0ae343996d36

	Ryan VanderMeulen [:RyanVM]
	Comment 6 • 11 years ago

https://hg.mozilla.org/mozilla-central/rev/0ae343996d36