DescriptionRevert of Delete CompositingUpdateAfterLayout (https://codereview.chromium.org/323653002/)
Reason for revert:
Triggers ASSERT when running browser_test CastStreamingApiTestWithPixelOutput.RtpStreamError
Original issue's description:
> Delete CompositingUpdateAfterLayout
>
> Rather than using a different update type, we can just mark the root as needing
> its properties updated, which will cause us to update the entire tree. This CL
> alone isn't a huge win, but it will let us remove
> RenderLayerCompositor::m_compositingLayersNeedRebuild in the next CL.
>
> Also, move the call to enableCompositingModeIfNeeded into the new
> RenderLayerCompositor::didLayout function. This makes the sequence look a bit
> silly (because we mark the dirty bit only to clear it immediately), but that's
> what the code did before (albeit in a more roundabout way).
>
> BUG=332248
>
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175738
TBR=esprehn@chromium.org,abarth@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=332248
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175742
Patch Set 1 #
Created: 6 years, 6 months ago
(Patch set is too large to download)
Messages
Total messages: 3 (0 generated)
|