Bug 240486

Summary:

Don't synchronously flush when destroying RemoteImageBufferProxy

Product:

WebKit

Reporter:

Matt Woodrow <mattwoodrow>

Component:

Layout and Rendering

Assignee:

Matt Woodrow <mattwoodrow>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bfulgham, sabouhallawa, simon.fraser, webkit-bug-importer, zalan

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	sabouhallawa: review+, ews-feeder: commit-queue-
Patch for landing	mattwoodrow: commit-queue+
Patch for landing	none

Description Matt Woodrow 2022-05-16 16:03:40 PDT

<rdar://90952989>

I don't think there's a need to synchronously flush from destruction, we should be able to just flush without waiting.

This hopefully improves PLT performance when navigating between pages (haven't managed to get a successful AB test on the bots).

Comment 1 Matt Woodrow 2022-05-16 16:06:58 PDT

Created attachment 459463 [details]
Patch

Comment 2 Matt Woodrow 2022-05-17 14:08:06 PDT

Created attachment 459514 [details]
Patch for landing

Comment 3 Matt Woodrow 2022-05-17 14:17:09 PDT

Created attachment 459515 [details]
Patch for landing

Comment 4 EWS 2022-05-17 15:43:10 PDT

Committed r294355 (250662@main): <https://commits.webkit.org/250662@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 459515 [details].