<rdar://problem/19821583>
Created attachment 255441 [details] Patch
Comment on attachment 255441 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=255441&action=review > Source/WebCore/page/FrameView.cpp:2496 > + // Turn off speculative tiling for a brief moment after a FrameView appears on screen. > + // Note that adjustTiledBackingCoverage() kicks the (500ms) timer to re-enable it. > + m_speculativeTilingEnabled = false; > + m_wasScrolledByUser = false; > + adjustTiledBackingCoverage(); Do you want to do this for non-main FrameViews?
(In reply to comment #2) > Comment on attachment 255441 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=255441&action=review > > > Source/WebCore/page/FrameView.cpp:2496 > > + // Turn off speculative tiling for a brief moment after a FrameView appears on screen. > > + // Note that adjustTiledBackingCoverage() kicks the (500ms) timer to re-enable it. > > + m_speculativeTilingEnabled = false; > > + m_wasScrolledByUser = false; > > + adjustTiledBackingCoverage(); > > Do you want to do this for non-main FrameViews? Ah, good point. We shouldn't do this for non-main FrameViews.
Created attachment 255445 [details] Patch Only do this for the main FrameView.
Comment on attachment 255445 [details] Patch Clearing flags on attachment: 255445 Committed r185897: <http://trac.webkit.org/changeset/185897>
All reviewed patches have been landed. Closing bug.