Bug 225714

Summary: [MotionMark] Temporarily remove the focus test
Product: WebKit Reporter: Myles C. Maxfield <mmaxfield>
Component: New BugsAssignee: Myles C. Maxfield <mmaxfield>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, ews-watchlist, ggaren, jbedard, rniwa, sabouhallawa, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch ggaren: review+

Description Myles C. Maxfield 2021-05-12 13:35:29 PDT
[MotionMark] Temporarily remove the focus test
Comment 1 Myles C. Maxfield 2021-05-12 16:06:12 PDT
Created attachment 428426 [details]
Patch
Comment 2 Myles C. Maxfield 2021-05-12 16:10:48 PDT
Created attachment 428428 [details]
Patch
Comment 3 Geoffrey Garen 2021-05-12 16:13:29 PDT
Comment on attachment 428428 [details]
Patch

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

r=me

> PerformanceTests/ChangeLog:10
> +        runloop and the compositor's runloop. Not only that, but the variance test seems to affect

variance test => variable cost of the focus test

> PerformanceTests/ChangeLog:13
> +        This patch temporarily removes the focus test, just so we can get a stronger signal of WebKit's

Data below seems to indicate that we'll get a stronger signal in all browsers.
Comment 4 Myles C. Maxfield 2021-05-12 16:20:00 PDT
Committed r277406 (237656@main): <https://commits.webkit.org/237656@main>
Comment 5 Radar WebKit Bug Importer 2021-05-12 16:20:19 PDT
<rdar://problem/77938125>