The following four values are only used in one instance of ImageBuffer::create(): RenderingMode::DisplayListAccelerated, RenderingMode::DisplayListUnaccelerated, RenderingMode::RemoteAccelerated, RenderingMode::RemoteUnaccelerated Many places in the code convert from RenderingMode to ShouldAccelerate and vice versa. Also one of the WebChromeClient::createImageBuffer() always returns null since it does not override its virtual function in the base class ChromeClient. So RenderingMode is converted back to be a bool enum as it was before and ShouldAccelerate will be removed. No conversion is needed anymore. And only one instance of ImageBuffer::create() will be handling creating the remote and display list ImageBuffers.
Created attachment 412474 [details] Patch
Created attachment 412479 [details] Patch
Committed r269084: <https://trac.webkit.org/changeset/269084> All reviewed patches have been landed. Closing bug and clearing flags on attachment 412479 [details].
<rdar://problem/70747549>