Bug 208614

Summary:

[First paint] Check for VNE state as the image content arrives

Product:

WebKit

Reporter:

zalan <zalan>

Component:

Layout and Rendering

Assignee:

zalan <zalan>

Status:

NEW ---

Severity:

Normal

CC:

bfulgham, eric.carlson, esprehn+autocc, ews-watchlist, glenn, jer.noble, kondapallykalyan, pdr, philipj, sergio, simon.fraser, webkit-bug-importer, zalan

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none

Description zalan 2020-03-04 16:41:59 PST

Image content does not necessarily trigger layout and definitely won't trigger tree building -> no VNE check.

Comment 1 Radar WebKit Bug Importer 2020-03-04 16:42:25 PST

<rdar://problem/60063261>

Comment 2 zalan 2020-03-04 17:27:57 PST

Created attachment 392517 [details]
Patch

Comment 3 Simon Fraser (smfr) 2020-03-04 18:11:15 PST

Comment on attachment 392517 [details]
Patch

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

> Source/WebCore/rendering/RenderVideo.cpp:106
> +        incrementVisuallyNonEmptyPixelCountIfNeeded(roundedIntSize(size), !needsLayout());

Ew. Do we really want to do this?