Summary: | [Win] fast/shadow-dom/fullscreen-in-slot-fullscreenElement.html is flaky. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Per Arne Vollan <pvollan> | ||||
Component: | Tools / Tests | Assignee: | Per Arne Vollan <pvollan> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | achristensen, bfulgham, lforschler, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Per Arne Vollan
2017-05-30 05:55:11 PDT
Created attachment 311500 [details]
Patch
Comment on attachment 311500 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=311500&action=review R=me but please make a new IWebViewPrivate5 interface to avoid breaking existing COM interface users. > Source/WebKit/win/Interfaces/IWebViewPrivate.idl:350 > + HRESULT exitFullscreenIfNeeded(); Sadly, we need to make an IWebViewPrivate5 here, because IWebViewPrivate4 already shipped to users, and we will break them if we modify the COM interface. (In reply to Brent Fulgham from comment #2) > Comment on attachment 311500 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=311500&action=review > > R=me but please make a new IWebViewPrivate5 interface to avoid breaking > existing COM interface users. > > > Source/WebKit/win/Interfaces/IWebViewPrivate.idl:350 > > + HRESULT exitFullscreenIfNeeded(); > > Sadly, we need to make an IWebViewPrivate5 here, because IWebViewPrivate4 > already shipped to users, and we will break them if we modify the COM > interface. Thanks for reviewing! I will make an IWebViewPrivate5 interface before landing. Committed <https://trac.webkit.org/changeset/217557/webkit> |