[webkit-reviews] review granted: [Bug 219492] ASSERTION FAILED: isMainThread() in WTF::Optional<IntSize> &WebCore::surfaceMaximumSize() : [Attachment 415336] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Dec 3 13:51:05 PST 2020

Ryosuke Niwa <rniwa at webkit.org> has granted Tim Horton <thorton at apple.com>'s
request for review:
Bug 219492: ASSERTION FAILED: isMainThread() in WTF::Optional<IntSize>

Attachment 415336: Patch


--- Comment #6 from Ryosuke Niwa <rniwa at webkit.org> ---
Comment on attachment 415336
  --> https://bugs.webkit.org/attachment.cgi?id=415336

View in context: https://bugs.webkit.org/attachment.cgi?id=415336&action=review

> Source/WebCore/platform/graphics/cocoa/IOSurface.mm:253
> -    surfaceMaximumSize() = size;
> +    surfaceMaximumSize().store(size);

Can we assert that new size is not empty?

> Source/WebCore/platform/graphics/cocoa/IOSurface.mm:260
> +	   auto computedSize = computeMaximumSurfaceSize();

We should consider asserting that this won't happen when GPU process is enabled
and we're in the web content process using ProcessPrivilege.

More information about the webkit-reviews mailing list