1263347 - Updating the display port margins should schedule a paint if the critical displayport changes

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

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

I've noticed a lot of running into low-res content on Fennec, sometimes even getting that in a perma-low-res state. With the minimap it's clear that the critical displayport is not moving with the viewport properly, and I suspect it's because I'm at the edge of the content where the full displayport isn't moving. The code in nsLayoutUtils::SetDisplayPortMarginsForElement should check for changes in the critical dp as well as the regular dp when deciding whether or not to schedule a paint.

Comment 1

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

Attachment: MozReview Request: Bug 1263347 - When checking if displayport changes should schedule a paint, make sure to use the proper displayport. r?mstange

Review commit: https://reviewboard.mozilla.org/r/47241/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/47241/

Comment 2

[Markus Stange [:mstange]]

Comment on attachment 8742505 [details]
MozReview Request: Bug 1263347 - When checking if displayport changes should schedule a paint, make sure to use the proper displayport. r?mstange

https://reviewboard.mozilla.org/r/47241/#review43929

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/552d5832785a

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/552d5832785a