ShareableBitmap::makeCGImageCopy() should have a null check for when createGraphicsContext() could return a nil.
<rdar://problem/53572444>
It's also possible that m_data of GraphicsContext to be null. Tracking this issue in the bug 200185.
Created attachment 375002 [details] Fixes the bug
Comment on attachment 375002 [details] Fixes the bug What about convertCGImageToBitmap()?
(In reply to Simon Fraser (smfr) from comment #4) > Comment on attachment 375002 [details] > Fixes the bug > > What about convertCGImageToBitmap()? Sure, I'll add the check there too.
Created attachment 375004 [details] Patch for landing
Comment on attachment 375002 [details] Fixes the bug I think you should make ShareableBitmap::createGraphicsContext() return nullptr and fix the ~10 call sites that we care about.
Comment on attachment 375004 [details] Patch for landing I think you should make ShareableBitmap::createGraphicsContext() return nullptr and fix the ~10 call sites that we care about.
(In reply to Simon Fraser (smfr) from comment #8) > Comment on attachment 375004 [details] > Patch for landing > > I think you should make ShareableBitmap::createGraphicsContext() return > nullptr and fix the ~10 call sites that we care about. That's done in https://bugs.webkit.org/show_bug.cgi?id=200185