PendingImageBitmap should not prevent entering the back/forward cache.
Created attachment 380227 [details] Patch
<rdar://problem/56036482>
Comment on attachment 380227 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=380227&action=review > LayoutTests/TestExpectations:265 > +fast/history/page-cache-createImageBitmap.html [ DumpJSConsoleLogInStdErr ] Weird, why is this an option here instead of in the <!-- webkit-test-runner --> block I wonder
Comment on attachment 380227 [details] Patch Clearing flags on attachment: 380227 Committed r250782: <https://trac.webkit.org/changeset/250782>
All reviewed patches have been landed. Closing bug.
(In reply to Tim Horton from comment #3) > Comment on attachment 380227 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=380227&action=review > > > LayoutTests/TestExpectations:265 > > +fast/history/page-cache-createImageBitmap.html [ DumpJSConsoleLogInStdErr ] > > Weird, why is this an option here instead of in the <!-- webkit-test-runner > --> block I wonder Do we have such a <!-- webkit-test-runner option?
(In reply to Chris Dumez from comment #6) > (In reply to Tim Horton from comment #3) > > Comment on attachment 380227 [details] > > Patch > > > > View in context: > > https://bugs.webkit.org/attachment.cgi?id=380227&action=review > > > > > LayoutTests/TestExpectations:265 > > > +fast/history/page-cache-createImageBitmap.html [ DumpJSConsoleLogInStdErr ] > > > > Weird, why is this an option here instead of in the <!-- webkit-test-runner > > --> block I wonder > > Do we have such a <!-- webkit-test-runner option? I don't know! This wasn't so much a complaint about your patch as a "huh it's weird we do this that way".
(In reply to Tim Horton from comment #7) > (In reply to Chris Dumez from comment #6) > > (In reply to Tim Horton from comment #3) > > > Comment on attachment 380227 [details] > > > Patch > > > > > > View in context: > > > https://bugs.webkit.org/attachment.cgi?id=380227&action=review > > > > > > > LayoutTests/TestExpectations:265 > > > > +fast/history/page-cache-createImageBitmap.html [ DumpJSConsoleLogInStdErr ] > > > > > > Weird, why is this an option here instead of in the <!-- webkit-test-runner > > > --> block I wonder > > > > Do we have such a <!-- webkit-test-runner option? > > I don't know! This wasn't so much a complaint about your patch as a "huh > it's weird we do this that way". Historically, we've had to use this a lot of Web-platform-tests, which we cannot modify. I think this feature was actually added for WPT.