Markus Stange [:mstange] Assignee
	Description • 9 years ago

If you scroll the testcase with paint flashing on, you'll see regular invalidations of the whole page because the SingleTiledContentClient is resized as different amounts of text are put into the display port.

We're not using tiles here because the loop inside ContainerState::GetLayerCreationHint stops at mContainerAnimatedGeometryRoot.

	Markus Stange [:mstange] Assignee
	Comment 1 • 9 years ago

Hmm, using tiles here doesn't prevent the invalidations.

	Markus Stange [:mstange] Assignee
	Comment 3 • 9 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=0e2affd8a39f

	Markus Stange [:mstange] Assignee
	Comment 4 • 9 years ago

The invalidations are actually a new manifestation of bug 1164227 (invalid region simplification crosses unchanged stuff in the middle) and I'll file a new bug about them.

	Markus Stange [:mstange] Assignee
	Comment 5 • 9 years ago

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

	Markus Stange [:mstange] Assignee
	Comment 6 • 9 years ago

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

	Matt Woodrow (:mattwoodrow)
	Comment 7 • 9 years ago

Comment on attachment 8703140 [details]
MozReview Request: Bug 1221094 - Don't mutate local mParameters with stuff that we only want to pass somewhere else. r?mattwoodrow

https://reviewboard.mozilla.org/r/29279/#review26945

	Matt Woodrow (:mattwoodrow)
	Comment 8 • 9 years ago

Comment on attachment 8703141 [details]
MozReview Request: Bug 1221094 - Detect scrollability (for using tiled layers) through container layers. r?mattwoodrow

https://reviewboard.mozilla.org/r/29281/#review26947

	Markus Stange [:mstange] Assignee
	Comment 9 • 9 years ago

This causes layout/reftests/bugs/1078262-1.html to fail on B2G:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=b37065fb708c&selectedJob=15281327

It looks like there are black seams between the tiles in the layer for #inner, which doesn't make any sense to me.

	Pulsebot
	Comment 10 • 8 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/6d40b86b8cc1
https://hg.mozilla.org/integration/mozilla-inbound/rev/70f69c94da8c
https://hg.mozilla.org/integration/mozilla-inbound/rev/d5d0a9a548fe

	Carsten Book [:Tomcat]
	Comment 11 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/6d40b86b8cc1
https://hg.mozilla.org/mozilla-central/rev/70f69c94da8c
https://hg.mozilla.org/mozilla-central/rev/d5d0a9a548fe