Bug 218291

Summary: [Testing] Remove requirement of adding new SPI for each preference that needs testing (WebKitLegacy Windows)
Product: WebKit Reporter: Sam Weinig <sam>
Component: New BugsAssignee: Sam Weinig <sam>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, Hironori.Fujii, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
ews-feeder: commit-queue-
Patch
none
Patch
none
Patch
none
Patch
none
Patch none

Description Sam Weinig 2020-10-28 09:57:15 PDT
[Testing] Remove requirement of adding new SPI for each preference that needs testing (WebKitLegacy Windows)
Comment 1 Sam Weinig 2020-10-28 09:57:32 PDT Comment hidden (obsolete)
Comment 2 Sam Weinig 2020-10-28 10:10:17 PDT Comment hidden (obsolete)
Comment 3 Sam Weinig 2020-10-28 11:25:08 PDT Comment hidden (obsolete)
Comment 4 Sam Weinig 2020-10-28 12:27:10 PDT
Created attachment 412559 [details]
Patch
Comment 5 Sam Weinig 2020-10-28 13:53:03 PDT
Brent, any idea why the Apple Windows bot has this error:

error C2039: 'setBoolPreferenceForTesting': is not a member of 'IWebPreferencesPrivate8.

Is there something else I need to do to add functions to IWebPreferencesPrivate8 other than what I am doing?
Comment 6 Brent Fulgham 2020-10-28 15:33:03 PDT
I don't see any problems with your code. I suspect we need to force a clean build because of some dependency issue in the Windows build.
Comment 7 Brent Fulgham 2020-10-28 15:42:51 PDT
Comment on attachment 412559 [details]
Patch

r=me. The Windows build failure is not because of this change, but is a symptom of a dependency problem in the IDL generation on Windows. We may need to force a clean build once this lands.
Comment 8 Sam Weinig 2020-10-28 15:54:54 PDT
Created attachment 412580 [details]
Patch
Comment 9 Sam Weinig 2020-10-28 19:02:48 PDT
Created attachment 412600 [details]
Patch
Comment 10 EWS 2020-10-29 11:18:15 PDT
Committed r269157: <https://trac.webkit.org/changeset/269157>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 412600 [details].
Comment 11 Radar WebKit Bug Importer 2020-10-29 11:19:21 PDT
<rdar://problem/70817654>
Comment 12 Fujii Hironori 2020-12-24 17:59:49 PST
Filed: Bug 220145 – [WinCairo][DumpRenderTree] Some JS tests are timing out only in Debug builds since r269157