Pres Context::Is Root Content Document Cross Process instead of Is Root Content Document for places where we call ns Layout Utils::Compute Scroll Metadata
(Core :: Layout, defect, P3)
(Reporter: hiro, Assigned: hiro)
(Blocks 1 open bug)
I am pretty sure this is the right thing to do, but I don't have enough experiences/knowledge to write auto-mated tests for this case.
What I am currently supposing is that FrameMetrics.mIsRootContent is incorrectly set to true in the case of cross-process iframe root scroller, then mZoomAnimationId is set for the iframe's APZC, thus the code which is supposed to be called only for the real root APZC is going to be proccessed, so maybe we can write a test for this particular case.
Anyways, in bug 1586144 I am going to add a function call which has an assertion that checks IsRootContentDocumentCrossProcess in
if (aIsRootContent) block in nsLayoutUtils::ComputeScrollMetaData, so I want to fix it prior to the bug.
4 years ago
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/2d6b09a782bb Use nsPresContext::IsRootContentDocumentCrossProcess instead of IsRootContentDocument for places where we call nsLayoutUtils::ComputeScrollMetadata. r=botond
4 years ago