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+

Myles C. Maxfield
Reported 2021-05-12 13:35:29 PDT
[MotionMark] Temporarily remove the focus test
Attachments
Patch (2.43 KB, patch)
2021-05-12 16:06 PDT, Myles C. Maxfield 		no flags
DetailsFormatted DiffDiff
Patch (2.37 KB, patch)
2021-05-12 16:10 PDT, Myles C. Maxfield 		ggaren: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Myles C. Maxfield
Comment 1 2021-05-12 16:06:12 PDT
Created attachment 428426 [details] Patch
Myles C. Maxfield
Comment 2 2021-05-12 16:10:48 PDT
Created attachment 428428 [details] Patch
Geoffrey Garen
Comment 3 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.
Myles C. Maxfield
Comment 4 2021-05-12 16:20:00 PDT
Committed r277406 (237656@main): <https://commits.webkit.org/237656@main>
Radar WebKit Bug Importer
Comment 5 2021-05-12 16:20:19 PDT
<rdar://problem/77938125>
Note You need to log in before you can comment on or make changes to this bug.