The blitContents() call removed by this patch used to be part of scheduleZoomAboutPoint(). Its goal was to display a preview of the zoomed contents, primarily when auto-zoomed after rotation. Nested inside a pair of screen suspend/resume calls, it has been a pointless no-op for a while.
Antonio's recent change to remove scheduleZoomAboutPoint() and call zoomAboutPoint() from setViewportSize() directly (the only call site) obsoletes the call completely. The zoomAboutPoint() call itself will cause a re-render and blit right away, so we don't care about any preview. zoomAboutPoint() will also take care of the necessary screen/backingstore suspension.
The result is a vastly simplified block of code.
Further simplification seems feasible, but is not exactly logically equivalent so I have to check with other people on how exactly they are meant to function. In order to minimize the risk of unintended regressions, this patch leaves out those sensitive pieces.
Created attachment 164454 [details]
Comment on attachment 164454 [details]
Clearing flags on attachment: 164454
Committed r128817: <http://trac.webkit.org/changeset/128817>
All reviewed patches have been landed. Closing bug.