Timeline

Oct 14, 2020:

11:11 PM Changeset in webkit [268515] by Kocsen Chung

• 3 edits in branches/safari-610.2.11.51-branch/Source/WebKit

Cherry-pick r268376. rdar://problem/70312257

Cursor window is transparent - but not hidden - when cursor: none is specified
​https://bugs.webkit.org/show_bug.cgi?id=217639
<rdar://problem/70006186>

Reviewed by Simon Fraser.

• Platform/spi/mac/AppKitSPI.h:
• UIProcess/mac/PageClientImplMac.mm: (WebKit::PageClientImpl::setCursor): Adopt new AppKit SPI to actually hide the cursor when we set it to the "none" shape. This hiding lives until the next time the app-global cursor shape is changed by any client.] We'll re-set it the next time we set the "none" shape.

This is important, because on some platforms, hiding the cursor is a performance improvement
over simply making it transparent.

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268376 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11:11 PM Changeset in webkit [268514] by Kocsen Chung

• 5 edits in branches/safari-610.2.11.51-branch/Source/WebCore

Cherry-pick r268308. rdar://problem/70321631

[Mac] AirPlay menu does not show up when AirPlay button is clicked
​https://bugs.webkit.org/show_bug.cgi?id=217536
<rdar://problem/69973777>

Reviewed by Eric Carlson.

Source/WebCore:

PAL tries to soft-link the AVRoutePickerView from the AVFoundation framework, rather than the (correct)
AVKit framework. Remove AVRoutePickerView from AVFoundationSoftLink.h and use the already declared version
inside AVRoutePickerViewTargetPicker.mm.

• platform/graphics/avfoundation/objc/AVRoutePickerViewTargetPicker.mm: (WebCore::AVRoutePickerViewTargetPicker::isAvailable):

Source/WebCore/PAL:

• pal/cocoa/AVFoundationSoftLink.h:
• pal/cocoa/AVFoundationSoftLink.mm:

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268308 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11:11 PM Changeset in webkit [268513] by Kocsen Chung

• 2 edits in branches/safari-610.2.11.51-branch/Source/WebKit

Cherry-pick r268377. rdar://problem/70320656

Unreviewed build fix after r268367.

• Configurations/WebKit.xcconfig: Corrected the definition of WK_PDFKIT_LDFLAGS_maccatalyst.

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268377 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11:11 PM Changeset in webkit [268512] by Kocsen Chung

• 4 edits in branches/safari-610.2.11.51-branch/Source

Cherry-pick r268367. rdar://problem/70320656

[macCatalyst] Enable WKPDFView
​https://bugs.webkit.org/show_bug.cgi?id=217403
<rdar://problem/48217791>

Reviewed by Tim Horton.

Source/WebKit:

• Configurations/WebKit.xcconfig: Linked PDFKit.framework when building for the Catalyst variant on Big Sur or later.

Source/WTF:

• wtf/PlatformEnableCocoa.h: Enabled WKPDFView when building for the Catalyst variant on Big Sur or later.

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268367 268f45cc-cd09-0410-ab3c-d52691b4dbfc

10:48 PM Changeset in webkit [268511] by Kocsen Chung

• 3 edits in branches/safari-610.2.11.1-branch/Source/WebKit

Cherry-pick r268376. rdar://problem/70312205

Cursor window is transparent - but not hidden - when cursor: none is specified
​https://bugs.webkit.org/show_bug.cgi?id=217639
<rdar://problem/70006186>

Reviewed by Simon Fraser.

• Platform/spi/mac/AppKitSPI.h:
• UIProcess/mac/PageClientImplMac.mm: (WebKit::PageClientImpl::setCursor): Adopt new AppKit SPI to actually hide the cursor when we set it to the "none" shape. This hiding lives until the next time the app-global cursor shape is changed by any client.] We'll re-set it the next time we set the "none" shape.

This is important, because on some platforms, hiding the cursor is a performance improvement
over simply making it transparent.

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268376 268f45cc-cd09-0410-ab3c-d52691b4dbfc

10:46 PM Changeset in webkit [268510] by Kocsen Chung

• 8 edits in branches/safari-610.2.11.1-branch/Source

Versioning.

WebKit-7610.2.11.1.3

10:01 PM Changeset in webkit [268509] by Kocsen Chung

• 8 edits in branches/safari-610.2.11.51-branch/Source

Versioning.

WebKit-7610.2.11.51.2

8:28 PM Changeset in webkit [268508] by Fujii Hironori

• 3 edits in trunk/LayoutTests

[WinCairo] Unreviewed test gardening

• platform/wincairo/TestExpectations:
• platform/wincairo/fast/css/word-space-extra-expected.txt:

8:24 PM Changeset in webkit [268507] by Brent Fulgham

• 2 edits in trunk/Source/WebKit

[iOS] Allow additional sysctl reads needed by image decoding
​https://bugs.webkit.org/show_bug.cgi?id=217733
<rdar://problem/68649171>

Reviewed by Per Arne Vollan.

Telemetry on iOS 14 shows that we are hitting some sandbox violations during image decoding. We should
grant access to hw.byteorder, hw.cachelinesize_compat, and hw.vectorunit. We should silence warnings
about hw.cpufrequency_compat since it is not needed in web-facing use cases.

• Resources/SandboxProfiles/ios/com.apple.WebKit.WebContent.sb:

7:29 PM Changeset in webkit [268506] by Chris Dumez

• 4 edits in trunk/Source/WebCore

Rename VectorMath::multiplyThenAddScalar() for clarity
​https://bugs.webkit.org/show_bug.cgi?id=217744

Reviewed by Darin Adler.

Rename VectorMath::multiplyThenAddScalar() to VectorMath::multiplyByScalarThenAddToOutput()
for clarity. The previous name was a bit confusing and hopefully the new name makes it
clearer what the function actually does. If not, I also improved the comment next to the
function.

• platform/audio/AudioBus.cpp:

(WebCore::AudioBus::speakersSumFromByDownMixing):

• platform/audio/VectorMath.cpp:

(WebCore::VectorMath::multiplyByScalarThenAddToOutput):

• platform/audio/VectorMath.h:

7:23 PM Changeset in webkit [268505] by Chris Dumez

• 4 edits in trunk/Source/WebCore

Vectorize downmixing in AudioBus::createByMixingToMono()
​https://bugs.webkit.org/show_bug.cgi?id=217743

Reviewed by Darin Adler.

Vectorize downmixing in AudioBus::createByMixingToMono().

No new tests, no Web-facing behavior change.

• platform/audio/AudioBus.cpp:

(WebCore::AudioBus::createByMixingToMono):

• platform/audio/VectorMath.cpp:

(WebCore::VectorMath::addVectorsThenMultiplyByScalar):

• platform/audio/VectorMath.h:

6:43 PM Changeset in webkit [268504] by rniwa@webkit.org

• 2 edits in trunk/Source/WebKit

Unreviewed build fix for non-Cocoa platforms & release builds.

• Platform/IPC/Connection.cpp:

(IPC::Connection::dispatchSyncMessage):

5:33 PM Changeset in webkit [268503] by rniwa@webkit.org

• 15 edits
  2 adds in trunk

Source/WebKit:
IPC testing JS API should expose a reply and describe the list of arguments for each message
​https://bugs.webkit.org/show_bug.cgi?id=217565

Reviewed by Geoffrey Garen.

This patch makes IPC.sendMessage and IPC.sendSyncMessage decode the reply. IPC.sendSyncMessage now returns
a dictionary with two keys: "buffer" and "arguments", the first of which returns ArrayBuffer of the decoded
message reply and the second of which is an array of decoded arguments where there was an appropriate JS
binding code existed for the argument type. IPC.sendMessage now returns a Promise which can either resolve
with the same dictionary as IPC.sendSyncMessage or reject when the decoding fails.

In addition, this patch exposes a dictionary describing each IPC message argument's type and parameter name.

In order to add these two functionalities, this patch adds a new step in generate-message-receiver.py to
generate MessageArgumentDescriptions.cpp, which contains functions which know how to decode arguments of
any IPC message and create a JS dictionary describing it as well as functions that return descriptions of
arguments or reply arguments.

Finally, this patch adds encoders for a few types found to be very common after r268239 had been landed.

Tests: TestWebKitAPI.IPCTestingAPI.DecodesReplyArgumentsForPrompt

TestWebKitAPI.IPCTestingAPI.DecodesReplyArgumentsForAsyncMessage
TestWebKitAPI.IPCTestingAPI.DescribesArguments

• CMakeLists.txt:
• DerivedSources-output.xcfilelist:
• DerivedSources.make:
• Platform/IPC/Decoder.h:

(IPC::Decoder::buffer const):

• Platform/IPC/JSIPCBinding.h: Added.

(jsValueForDecodedArgumentValue): Added. A template function to construct a JS value for a given C++ value.
(jsValueForDecodedNumericArgumentValue): Added. A helper for constructing a JS value for numeric values.
(jsValueForDecodedArgumentRect): Added. Ditto for IntRect and FloatRect.
(DecodedArgumentJSValueConverter): Added. A helper class to construct JS values for a tuple of values using
partial template specializations.
(DecodedArgumentJSValueConverter::convert): Added.
(jsValueForArgumentTuple): Added. A helper to construct a JS array for the decoded IPC arguments.
(jsValueForDecodedArguments): Added.

• Platform/IPC/MessageArgumentDescriptions.h: Added.

(IPC::ArgumentDescription): Added.

• Scripts/generate-message-receiver.py:

(main): Generate MessageArgumentDescriptions.cpp.

• Scripts/webkit/messages.py:

(headers_for_type): Removed the special case for PaymentMethodUpdate now that it's in its own header. Also
added made webrtc::WebKitEncodedFrameInfo include LibWebRTCEnumTraits.h as it uses webrtc::VideoFrameType.
(collect_header_conditions_for_receiver): Extracted from generate_message_handler.
(generate_header_includes_from_conditions): Ditto.
(generate_message_handler):
(generate_js_value_conversion_function): Added.
(generate_js_argument_descriptions): Added.
(generate_message_argument_description_implementation): Added.

• Shared/ApplePay/ApplePayPaymentSetupFeaturesWebKit.h: Fixed a bug that we were not forward declaring NSArray.
• SourcesCocoa.txt: Added MessageArgumentDescriptions.cpp as a non-unified cpp file as its size is around 1MB.
• WebKit.xcodeproj/project.pbxproj:
• WebProcess/WebPage/IPCTestingAPI.cpp:

(WebKit::IPCTestingAPI::JSIPC::staticValues): Added the session ID and page ID as static variables.
(WebKit::IPCTestingAPI::encodePointType): Added.
(WebKit::IPCTestingAPI::encodeRectType): Fixed the bug was that this code wasn't checking for any exceptions.
(WebKit::IPCTestingAPI::encodeNumericType): Renamed from encodeIntegralType since this function is now used
to encode double and float, not just integral types.
(WebKit::IPCTestingAPI::encodeArgument): Added the support for IntPoint, FloatPoint, URL, RegistrableDomain,
double, and float all of which turned out to be in the top 20 most common types.
(WebKit::IPCTestingAPI::jsResultFromReplyDecoder): Added.
(WebKit::IPCTestingAPI::JSIPC::sendMessage): Added the code to return Promise when there is a reply and resolve
it with the JS object describing the decoded argument. We use messageReplyArgumentDescriptions to figure out
whether there is a reply or not.
(WebKit::IPCTestingAPI::JSIPC::sendSyncMessage): Decode the reply and return a JS object which describes it.
(WebKit::IPCTestingAPI::JSIPC::frameID): Renamed from frameIdentifier to be consistent.
(WebKit::IPCTestingAPI::JSIPC::pageID): Added.
(WebKit::IPCTestingAPI::JSIPC::sessionID): Added.
(WebKit::IPCTestingAPI::createJSArrayForArgumentDescriptions): Added.
(WebKit::IPCTestingAPI::JSIPC::messages): Added the code to generate descriptions for arguments.

Source/WTF:
Unreviewed build fix. Disable IPC testing API on non-Cocoa platforms.

• wtf/PlatformEnable.h:

Tools:
IPC testing JS API should expose a reply and describe the list of arguments for each message
​https://bugs.webkit.org/show_bug.cgi?id=217565

Reviewed by Geoffrey Garen.

Added tests for decoding replies for sync and async messages and one for argument descriptions.

• TestWebKitAPI/Tests/WebKitCocoa/IPCTestingAPI.mm:

(-[IPCTestingAPIDelegate webView:runJavaScriptTextInputPanelWithPrompt:defaultText:initiatedByFrame:completionHandler:]):
(-[IPCTestingAPIDelegate _webView:webContentProcessDidTerminateWithReason:]):
(IPCTestingAPI.DecodesReplyArgumentsForPrompt):
(IPCTestingAPI.DecodesReplyArgumentsForAsyncMessage):
(IPCTestingAPI.DescribesArguments):

5:24 PM Changeset in webkit [268502] by James Darpinian

• 3 edits
  7 adds in trunk

Unity RubysAdventure demo does not render properly if WebGL2 is enabled.
​https://bugs.webkit.org/show_bug.cgi?id=215846

Reviewed by Kenneth Russell.

Source/WebCore:

uniform*v srcLength parameter needs to be divided by the uniform
component size before passing to OpenGL's count parameter.

Test: webgl/conformance2/uniforms/gl-uniform-arrays-sub-source.html

• html/canvas/WebGL2RenderingContext.cpp:

(WebCore::WebGL2RenderingContext::uniform2uiv):
(WebCore::WebGL2RenderingContext::uniform3uiv):
(WebCore::WebGL2RenderingContext::uniform4uiv):
(WebCore::WebGL2RenderingContext::uniformMatrix2x3fv):
(WebCore::WebGL2RenderingContext::uniformMatrix3x2fv):
(WebCore::WebGL2RenderingContext::uniformMatrix2x4fv):
(WebCore::WebGL2RenderingContext::uniformMatrix4x2fv):
(WebCore::WebGL2RenderingContext::uniformMatrix3x4fv):
(WebCore::WebGL2RenderingContext::uniformMatrix4x3fv):
(WebCore::WebGL2RenderingContext::uniform2fv):
(WebCore::WebGL2RenderingContext::uniform3fv):
(WebCore::WebGL2RenderingContext::uniform4fv):
(WebCore::WebGL2RenderingContext::uniform2iv):
(WebCore::WebGL2RenderingContext::uniform3iv):
(WebCore::WebGL2RenderingContext::uniform4iv):
(WebCore::WebGL2RenderingContext::uniformMatrix2fv):
(WebCore::WebGL2RenderingContext::uniformMatrix3fv):
(WebCore::WebGL2RenderingContext::uniformMatrix4fv):

LayoutTests:

• webgl/conformance2/uniforms/gl-uniform-arrays-sub-source-expected.txt: Added.
• webgl/conformance2/uniforms/gl-uniform-arrays-sub-source.html: Added.
• webgl/resources/webgl_test_files/conformance2/uniforms/gl-uniform-arrays-sub-source.html: Added.

5:11 PM Changeset in webkit [268501] by Ross Kirsling

• 547 edits
  8 copies
  773 adds
  69 deletes in trunk/JSTests

Update test262 (2020.10.14)
​https://bugs.webkit.org/show_bug.cgi?id=217736

Reviewed by Yusuke Suzuki.

• test262/config.yaml:
• test262/expectations.yaml:
• test262/harness/:
• test262/latest-changes-summary.txt:
• test262/test/:
• test262/test262-Revision.txt:

5:09 PM Changeset in webkit [268500] by Chris Dumez

• 3 edits in trunk/Source/WebKit

REGRESSION(r268423) SHOULD NEVER BE REACHED: GPUConnectionToWebProcess::didReceiveMessage(IPC::Connection &, IPC::Decoder &)
​https://bugs.webkit.org/show_bug.cgi?id=217730
<rdar://problem/70308112>

Reviewed by Geoffrey Garen.

r268423 added support for registering IPC WorkQueueMessageReceivers & ThreadMessageReceivers with
a given identifier. When receiving an IPC message, we would look up the key [ReceiverName, ID] in
the receiver map and pass the message to this receiver.
This introduced a crash because of an existing ThreadMessageReceiver (RemoteAudioMediaStreamTrackRenderer)
was registering itself as a message receiver without an identifier (ID=0). However, the IPC senders
on GPU process side would pass a non-0 identifier when sending the IPC. As a result, we would fail to
look up the receiver in the map when receiving those IPC messages.

While existing code should probably be fixed so that senders and receivers agree on on identifier,
I have decided to fix the issue in this patch at IPC::Connection level. When receiving an IPC
message, we first check if there is a "global" receiver in the map by looking up key
[ReceiverName, 0]. If there is, we pass the IPC to this one. If there isn't, we look up the key
[ReceiverName, ID] in the map and use the more specific receiver.

The reason I chose to fix it this way is that the code in MessageReceiverMap (which is used for
receivers that are not using a WorkQueue or a Thread) behaves in this fashion. I believe it is
good to have consistency between all kinds of receivers.

No new tests, covered by existing test that is crashing on the bots.

• Platform/IPC/Connection.cpp:

(IPC::Connection::threadMessageReceiver):
(IPC::Connection::workQueueMessageReceiver):
(IPC::Connection::dispatchMessageToWorkQueueReceiver):
(IPC::Connection::dispatchMessageToThreadReceiver):

• Platform/IPC/Connection.h:

4:55 PM Changeset in webkit [268499] by Simon Fraser

• 3 edits in trunk/Source/WebKit

Scrolls in the passive handler region should be reported as handled promptly
​https://bugs.webkit.org/show_bug.cgi?id=217718

Reviewed by Tim Horton.

The UI process will enqueue wheel events to send later if the WebContent process doesn't
promptly respond that it has handled the previous event. When scrolling in the passive
event region, we want the WebContent process to appear responsive, so respond to the UI
process when we've scrolled (which is before the event has been dispatched for DOM events).

Hard to test because CPU-constrained bots don't send wheel events in a predictable way.

• WebProcess/WebPage/EventDispatcher.cpp:

(WebKit::EventDispatcher::wheelEvent):

• WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::wheelEvent):

4:21 PM Changeset in webkit [268498] by Hector Lopez

• 10 edits
  1 delete in trunk/Tools

Unreviewed, reverting r268474.

Revision is causing a number of test failures across iOS and
macOS

Reverted changeset:

"[Testing] Support configuring any preference from test
headers for WebKitTestRunner"
​https://bugs.webkit.org/show_bug.cgi?id=217645
​https://trac.webkit.org/changeset/268474

4:14 PM Changeset in webkit [268497] by rniwa@webkit.org

• 8 edits in trunk

Enabling IPC testing API should prevent the termination of WebContent process which sends an invalid IPC
​https://bugs.webkit.org/show_bug.cgi?id=217698

Reviewed by Geoffrey Garen.

Source/WebKit:

A part of this was landed in r268431 but this patch formally disables UI process' default behavior
to terminate a Web process upon receiving an invalid message from it.

Tests: IPCTestingAPI.CanSendInvalidAsyncMessageWithoutTermination

IPCTestingAPI.CanSendInvalidMessageWithoutTermination

• Platform/IPC/Connection.cpp:

(IPC::Connection::dispatchSyncMessage): Disable the debug assertion if the IPC testing API is enabled.

• Platform/IPC/Connection.h:

(IPC::Connection::setIgnoreInvalidMessageForTesting): Added.
(IPC::Connection::ignoreInvalidMessageForTesting const): Added.
(IPC::Connection::m_ignoreInvalidMessageForTesting): Added.

• UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::WebPageProxy): Set the flag on WebProcessProxy to trigger the behavior.
(WebKit::WebPageProxy::launchProcess): Ditto.

• UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::didReceiveInvalidMessage): Use the flag on Connection instead of reaching
out to the default page group.
(WebKit::WebProcessProxy::setIgnoreInvalidMessageForTesting): Added. Remember the fact we've enabled IPC
testing API in a member variable and propagate the flag to Connection if a Web process is already running.
(WebKit::WebProcessProxy::didFinishLaunching): Propagte the flag to the newly launched Web process.

• UIProcess/WebProcessProxy.h:

(WebKit::WebProcessProxy::m_ignoreInvalidMessageForTesting): Added.

Tools:

Added regression tests.

• TestWebKitAPI/Tests/WebKitCocoa/IPCTestingAPI.mm:

(IPCTestingAPI.CanSendInvalidAsyncMessageWithoutTermination):
(IPCTestingAPI.CanSendInvalidMessageWithoutTermination):

4:08 PM Changeset in webkit [268496] by aakash_jain@apple.com

• 4 edits in trunk/Tools

[build.webkit.org] Add python 3 support - part 1
​https://bugs.webkit.org/show_bug.cgi?id=217727

Reviewed by Jonathan Bedard.

• BuildSlaveSupport/build.webkit.org-config/steps.py:

(UploadBuiltProduct.init): Changed 0644 to 0o644.
(UploadTestResults.init): Ditto.

• BuildSlaveSupport/build.webkit.org-config/steps_unittest.py: Removed unused imports.
• BuildSlaveSupport/build.webkit.org-config/run-buildbot-test.py: Used .format() instead of % string formatting.

3:52 PM Changeset in webkit [268495] by weinig@apple.com

• 9 edits
  2 adds in trunk/Tools

[Testing] Generate test header key map for DumpRenderTree
​https://bugs.webkit.org/show_bug.cgi?id=217723

Reviewed by Tim Horton.

Copy test header command type generation from WebKitTestRunner to DumpRenderTree, to
begin support for using any WebPreference from DumpRenderTree. This does not actually
change any behavior yet, just updates the key mapping.

• DumpRenderTree/CMakeLists.txt:
• DumpRenderTree/DerivedSources-input.xcfilelist:
• DumpRenderTree/DerivedSources-output.xcfilelist:
• DumpRenderTree/DerivedSources.make:
• DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj:
• DumpRenderTree/Scripts/PreferencesTemplates: Added.
• DumpRenderTree/Scripts/PreferencesTemplates/TestOptionsGeneratedKeys.h.erb: Added.
• DumpRenderTree/TestOptions.cpp:

(WTR::TestOptions::keyTypeMapping):

• DumpRenderTree/TestOptions.h:
• DumpRenderTree/mac/Configurations/Base.xcconfig:

3:47 PM Changeset in webkit [268494] by Andres Gonzalez

• 6 edits
  2 adds in trunk

Support AXCoreObject::elementPath for text elements.
​https://bugs.webkit.org/show_bug.cgi?id=215206

Reviewed by Chris Fleizach.

Source/WebCore:

Test: accessibility/text-element-path.html

Text elements now support returning a Path in those cases where the
element consists of the last part of a line and the beginning of the
next line. Heuristics were added to avoid to compute the element Path for those cases in which the bounding rectangle is suitable.

• accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::supportsPath const):
(WebCore::AccessibilityRenderObject::elementPath const):

LayoutTests:

• accessibility/mac/bounds-for-range-expected.txt:

Since elementPath is now supported for text elements,
AccessibilityUIEelement::attributesOfLinkedUIElements also returns the
element path description.

• accessibility/mac/internal-link-anchors-expected.txt: Ditto.
• accessibility/text-element-path-expected.txt: Added.
• accessibility/text-element-path.html: Added.

3:25 PM Changeset in webkit [268493] by Alan Coon

• 10 edits in branches/safari-610-branch/Source

Cherry-pick r267203. rdar://problem/70310545

Rename ShouldAllowExternalSchemes to ShouldAllowExternalSchemesButNotAppLinks to clarify what this enum value is for.
​https://bugs.webkit.org/show_bug.cgi?id=216651

Patch by Hoa Dinh <​dvh@apple.com> on 2020-09-17
Reviewed by Alex Christensen.

Rename ShouldAllowExternalSchemes to ShouldAllowExternalSchemesButNotAppLinks and reflects the change where it's used.

Source/WebCore:

Source/WebCore:

• loader/FrameLoaderTypes.h:
• page/ContextMenuController.cpp: (WebCore::ContextMenuController::contextMenuItemSelected):

Source/WebKit:

Source/WebKit:

• Shared/SessionState.cpp: (WebKit::isValidEnum):
• UIProcess/API/APINavigationAction.h:
• UIProcess/API/glib/WebKitWebViewSessionState.cpp: (toExternalURLsPolicy): (toWebCoreExternalURLsPolicy):
• UIProcess/WebPageProxy.h:
• UIProcess/mac/LegacySessionStateCoding.cpp: (WebKit::decodeSessionHistoryEntry):
• WebProcess/InjectedBundle/InjectedBundleNavigationAction.cpp: (WebKit::InjectedBundleNavigationAction::InjectedBundleNavigationAction):

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@267203 268f45cc-cd09-0410-ab3c-d52691b4dbfc

3:19 PM Changeset in webkit [268492] by pvollan@apple.com

• 6 edits in trunk/Source/WebKit

[macOS] Issue sandbox extension to additional icon service when attachment element is enabled.
​https://bugs.webkit.org/show_bug.cgi?id=217706
<rdar://problem/70291100>

Reviewed by Brent Fulgham.

Issue sandbox extension to 'com.apple.iconservices.store' when attachment element is enabled on macOS.

• Shared/WebPageCreationParameters.cpp:

(WebKit::WebPageCreationParameters::encode const):
(WebKit::WebPageCreationParameters::decode):

• Shared/WebPageCreationParameters.h:
• Shared/WebProcessCreationParameters.cpp:

(WebKit::WebProcessCreationParameters::encode const):
(WebKit::WebProcessCreationParameters::decode):

• Shared/WebProcessCreationParameters.h:
• UIProcess/WebPageProxy.cpp:

(WebKit::attachmentElementServices):
(WebKit::WebPageProxy::creationParameters):

• WebProcess/WebPage/WebPage.cpp:
• WebProcess/com.apple.WebProcess.sb.in:

3:13 PM Changeset in webkit [268491] by Hector Lopez

• 2 edits in trunk/LayoutTests

Rebaseline editing/editable-region/iframe.html result
​https://bugs.webkit.org/show_bug.cgi?id=158439

Unreviewed test gardening.

• editing/editable-region/iframe-expected.txt:

3:10 PM Changeset in webkit [268490] by ap@apple.com

• 3 edits
  2 deletes in trunk/Tools

Remove WebKitLauncher
​https://bugs.webkit.org/show_bug.cgi?id=217729

Reviewed by Darin Adler.

This is part of long defunct nightly build infrastructure.

• .gitattributes:
• BuildSlaveSupport/build-launcher-app: Removed.
• Scripts/webkitpy/style/checkers/sdkvariant.py:
• WebKitLauncher: Removed.

2:48 PM Changeset in webkit [268489] by Alexey Shvayka

• 15 edits
  1 add in trunk

Use @putByValDirect instead of Array.prototype.@push in built-ins
​https://bugs.webkit.org/show_bug.cgi?id=175432

Reviewed by Yusuke Suzuki.

JSTests:

• stress/array-push-intrinsic.js: Added.
• test262/expectations.yaml: Mark 2 test cases as passing.

Source/JavaScriptCore:

Before this patch, Array.prototype.@push was used to implement List spec type,
stacks / queues, and in place of CreateDataProperty [1].
It's undesirably observable since elements are pushed using Set,
affecting indexed properties on prototypes [2].

This change introduces @arrayPush() intrinsic to use with lists / stacks / queues.
@arrayPush() should only be used with JSArray receivers because "length" isn't
incremented. Unlike Array.prototype.@push, it doesn't grow arrays beyond UINT_MAX,
which is OK for current use cases. Object.entries microbenchmark is neutral.

Despite Array.prototype.@shift also performing Set, it's safe to use with
non-sparse receivers.

[1]: ​https://tc39.es/ecma262/#sec-createarrayfromlist (step 4.a)
[2]: ​https://tc39.es/ecma262/#sec-array.prototype.push (step 5.a)

• builtins/ArrayPrototype.js:

(globalPrivate.sortBucketSort):

• builtins/ObjectConstructor.js:

(entries):

• builtins/RegExpPrototype.js:

(globalPrivate.matchSlow):
(Symbol.replace):
(Symbol.split):

• builtins/TypedArrayPrototype.js:

(filter):

• bytecode/BytecodeIntrinsicRegistry.h:
• bytecompiler/NodesCodegen.cpp:

(JSC::BytecodeIntrinsicNode::emit_intrinsic_arrayPush):

• runtime/ArrayPrototype.cpp:

(JSC::ArrayPrototype::finishCreation):

Source/WebCore:

Use @arrayPush() intrinsic instead of Array.prototype.@push.

No new tests, no behavior change.

• Modules/streams/ReadableByteStreamInternals.js:

(readableByteStreamControllerPull):
(readableByteStreamControllerEnqueueChunk):
(readableByteStreamControllerPullInto):
(readableStreamAddReadIntoRequest):

• Modules/streams/ReadableStreamInternals.js:

(readableStreamAddReadRequest):

• Modules/streams/StreamInternals.js:

(enqueueValueWithSize):

• Modules/streams/WritableStreamInternals.js:

(writableStreamAddWriteRequest):

2:19 PM Changeset in webkit [268488] by Chris Dumez

• 4 edits in trunk/LayoutTests

REGRESSION (r267744): [ Debug WK2 ] fast/css-custom-paint/out-of-memory-while-adding-worklet-module.html is a constant timeout
​https://bugs.webkit.org/show_bug.cgi?id=217549

Reviewed by Sam Weinig.

After r267744, Worklet.addModule() returns a promise and no longer throws exceptions.
Update the test accordingly to expect that a promise gets rejected with an out of
memory error, instead of expecting an exception to be thrown.

• fast/css-custom-paint/out-of-memory-while-adding-worklet-module-expected.txt:
• fast/css-custom-paint/script-tests/out-of-memory-while-adding-worklet-module.js:

1:55 PM Changeset in webkit [268487] by Megan Gardner

• 1 edit
  1 move in trunk/LayoutTests

Paint CSS highlights over images
​https://bugs.webkit.org/show_bug.cgi?id=217417

Reviewed by Tim Horton.

• http/wpt/css/css-highlight-api/highlight-image-expected-mismatched.html: Removed.
• http/wpt/css/css-highlight-api/highlight-image-expected-mismatch.html: Added.

1:52 PM Changeset in webkit [268486] by commit-queue@webkit.org

• 14 edits
  2 deletes in trunk

Unreviewed, reverting r268431 and r268443.
​https://bugs.webkit.org/show_bug.cgi?id=217728

Broke debug builds of GTK, WinCairo and Playstation ports

Reverted changesets:

"IPC testing JS API should expose a reply and describe the
list of arguments for each message"
​https://bugs.webkit.org/show_bug.cgi?id=217565
​https://trac.webkit.org/changeset/268431

"IPC testing JS API should expose a reply and describe the
list of arguments for each message"
​https://bugs.webkit.org/show_bug.cgi?id=217565
​https://trac.webkit.org/changeset/268443

1:46 PM Changeset in webkit [268485] by don.olmstead@sony.com

• 19 edits in trunk/Source

Non-unified build fixes, mid October 2020
​https://bugs.webkit.org/show_bug.cgi?id=217721

Reviewed by Yusuke Suzuki.

Source/JavaScriptCore:

• API/JSContextRef.cpp:
• dfg/DFGDoesGCCheck.cpp:
• llint/LLIntExceptions.cpp:
• llint/LLIntExceptions.h:
• llint/LLIntThunks.h:

Source/WebCore:

• Modules/webgpu/WebGPUShaderModule.h:
• html/canvas/EXTTextureCompressionRGTC.cpp:
• layout/floats/FloatingContext.cpp:
• layout/floats/FloatingContext.h:
• layout/inlineformatting/InlineFormattingContextGeometry.cpp:
• layout/inlineformatting/InlineLine.cpp:
• platform/graphics/gpu/GPUDevice.cpp:
• rendering/RenderReplaced.cpp:
• workers/WorkerRunLoop.cpp:

Source/WebKit:

• NetworkProcess/Downloads/DownloadManager.cpp:
• UIProcess/Downloads/DownloadProxyMap.cpp:

1:42 PM Changeset in webkit [268484] by Megan Gardner

• 12 edits
  1 copy
  1 add in trunk/Source/WebCore

Refactor HighlightData
​https://bugs.webkit.org/show_bug.cgi?id=217711

Reviewed by Darin Adler.

HighlightData was renamed when it was used for more that selections, but
more than half the file was only supposed to be used to deal with the current
selection, so I've split it back out into a more reasonable set of files.
Also made offsets no longer optional, as there doesn't seem to be a need.
Extended the HighlightData setRenderRange into InlineTextBox, as it's a better
way to calculate that does not use deprecated functions.

No behavior change.

• Headers.cmake:
• Sources.txt:
• WebCore.xcodeproj/project.pbxproj:
• editing/FrameSelection.cpp:

(WebCore::FrameSelection::setNeedsSelectionUpdate):
(WebCore::DragCaretController::nodeWillBeRemoved):
(WebCore::FrameSelection::respondToNodeModification):
(WebCore::FrameSelection::willBeRemovedFromFrame):
(WebCore::FrameSelection::focusedOrActiveStateChanged):
(WebCore::FrameSelection::updateAppearance):

• page/FrameView.cpp:

(WebCore::FrameView::paintContentsForSnapshot):

• platform/DragImage.cpp:

(WebCore::ScopedFrameSelectionState::~ScopedFrameSelectionState):
(WebCore::createDragImageForRange):

• rendering/HighlightData.cpp:

(WebCore::RenderRangeIterator::RenderRangeIterator):
(WebCore::RenderRangeIterator::current const):
(WebCore::RenderRangeIterator::next):
(WebCore::RenderRangeIterator::checkForSpanner):
(WebCore::rendererAfterOffset):
(WebCore::HighlightData::highlightStateForRenderer):
(): Deleted.
(WebCore::isValidRendererForSelection): Deleted.
(WebCore::containingBlockBelowView): Deleted.
(WebCore::collectSelectionData): Deleted.
(WebCore::HighlightData::HighlightData): Deleted.
(WebCore::HighlightData::setSelection): Deleted.
(WebCore::HighlightData::clearSelection): Deleted.
(WebCore::HighlightData::repaintSelection const): Deleted.
(WebCore::HighlightData::collectBounds const): Deleted.
(WebCore::HighlightData::applySelection): Deleted.

• rendering/HighlightData.h:

(WebCore::RenderRange::RenderRange):
(WebCore::RenderRange::start const):
(WebCore::RenderRange::end const):
(WebCore::RenderRange::startOffset const):
(WebCore::RenderRange::endOffset const):
(WebCore::RenderRange::operator== const):
(WebCore::HighlightData::startOffset const):
(WebCore::HighlightData::endOffset const):
(WebCore::HighlightData::RenderRange::RenderRange): Deleted.
(WebCore::HighlightData::RenderRange::start const): Deleted.
(WebCore::HighlightData::RenderRange::end const): Deleted.
(WebCore::HighlightData::RenderRange::startOffset const): Deleted.
(WebCore::HighlightData::RenderRange::endOffset const): Deleted.
(WebCore::HighlightData::RenderRange::operator== const): Deleted.
(WebCore::HighlightData::bounds const): Deleted.
(WebCore::HighlightData::boundsClippedToVisibleContent const): Deleted.

• rendering/InlineTextBox.cpp:

(WebCore::InlineTextBox::collectMarkedTextsForHighlights const):

• rendering/RenderReplaced.cpp:

(WebCore::RenderReplaced::calculateHighlightColor const):

• rendering/RenderView.h:
• rendering/SelectionRangeData.cpp: Copied from Source/WebCore/rendering/HighlightData.cpp.

(WebCore::rendererAfterOffset):
(WebCore::isValidRendererForSelection):
(WebCore::containingBlockBelowView):
(WebCore::collectSelectionData):
(WebCore::SelectionRangeData::SelectionData):
(WebCore::SelectionRangeData::set):
(WebCore::SelectionRangeData::clear):
(WebCore::SelectionRangeData::repaint const):
(WebCore::SelectionRangeData::collectBounds const):
(WebCore::SelectionRangeData::apply):

• rendering/SelectionRangeData.h: Added.

(WebCore::SelectionRangeData::bounds const):
(WebCore::SelectionRangeData::boundsClippedToVisibleContent const):

1:37 PM Changeset in webkit [268483] by graouts@webkit.org

• 3 edits in trunk/Source/WebCore

Simplify management of LayerPropertyAnimation instances in GraphicsLayerCA
​https://bugs.webkit.org/show_bug.cgi?id=217707
<rdar://problem/70291140>

Reviewed by Dean Jackson.

The management of LayerPropertyAnimation instances in GraphicsLayerCA spread
those animations between different data structures. We now simplify this to
be a single Vector<LayerPropertyAnimation> with three new properties.

The m_playState property indicates:

• whether the animation has been added to the list but awaiting the next commit

to be played (PlayPending)

• whether the animation has been committed and playing (Playing)
• whether the animation has been marked to be paused but awaiting the next commit

to be paused (PausePending)

• whether the animation has been committed and paused (Paused)

The m_pendingRemoval property indicates whether the animation has been marked for
removal upon the next commit.

The m_beginTime property indicates the time at which the animation was first added,
ensuring that the animation can be removed and re-added while preserving the same
begin time.

As such, when GraphicsLayerCA::updateAnimations() is called we remove all animations
from the layer, filter m_animations to only have animations that are not marked
as pending removal, and add the now-current list of animations to the layer.

No new test since this change is refactoring only and should not have any change
in behavior.

• platform/graphics/ca/GraphicsLayerCA.cpp:

(WebCore::GraphicsLayerCA::moveOrCopyAnimations):
(WebCore::GraphicsLayerCA::animationIsRunning const):
(WebCore::animationCanBeAccelerated):
(WebCore::GraphicsLayerCA::pauseAnimation):
(WebCore::GraphicsLayerCA::removeAnimation):
(WebCore::GraphicsLayerCA::platformCALayerAnimationStarted):
(WebCore::GraphicsLayerCA::updateAnimations):
(WebCore::GraphicsLayerCA::isRunningTransformAnimation const):
(WebCore::GraphicsLayerCA::setAnimationOnLayer):
(WebCore::GraphicsLayerCA::removeCAAnimationFromLayer):
(WebCore::GraphicsLayerCA::pauseCAAnimationOnLayer):
(WebCore::GraphicsLayerCA::createAnimationFromKeyframes):
(WebCore::GraphicsLayerCA::appendToUncommittedAnimations):
(WebCore::GraphicsLayerCA::createTransformAnimationsFromKeyframes):
(WebCore::animationIdentifier): Deleted.
(WebCore::GraphicsLayerCA::animationCanBeAccelerated const): Deleted.
(WebCore::GraphicsLayerCA::addProcessingActionForAnimation): Deleted.
(WebCore::GraphicsLayerCA::ensureLayerAnimations): Deleted.

• platform/graphics/ca/GraphicsLayerCA.h:

(WebCore::GraphicsLayerCA::hasAnimations const):
(WebCore::GraphicsLayerCA::LayerPropertyAnimation::animationIdentifier const):
(WebCore::GraphicsLayerCA::moveAnimations):
(WebCore::GraphicsLayerCA::copyAnimations):
(WebCore::GraphicsLayerCA::animationIsRunning const): Deleted.
(WebCore::GraphicsLayerCA::AnimationProcessingAction::AnimationProcessingAction): Deleted.

1:27 PM Changeset in webkit [268482] by Devin Rousso

• 3 edits in trunk/Source/WebKit

REGRESSION (r268384): ASSERTION FAILED: _startCount > 1 in -[WKMouseDeviceObserver stop]
​https://bugs.webkit.org/show_bug.cgi?id=217684
<rdar://problem/70272646>

Reviewed by Tim Horton.

The ASSERT was incorrect in that it's possible for _startCount to be equal to 1. Also
add a check to make sure _startCount doesn't overflow.

• UIProcess/ios/WKMouseDeviceObserver.mm:

(-[WKMouseDeviceObserver stop]):

• UIProcess/ios/WKStylusDeviceObserver.mm:

(-[WKStylusDeviceObserver stop]):

1:20 PM Changeset in webkit [268481] by Alan Coon

• 6 edits in branches/safari-610-branch/Source

Cherry-pick r268003. rdar://problem/70267725

Make NetscapePlugInStreamLoaderClient a weak pointer on NetscapePlugInStreamLoader
<​https://webkit.org/b/217237>
<rdar://problem/69776714>

Reviewed by Alex Christensen.

Source/WebCore:

Could not write a test since reproducing a crash requires
rapidly reloading a URL to a PDF at random points during the
previous load.

Make NetscapePlugInStreamLoaderClient a WeakPtr on
NetscapePlugInStreamLoader, and add nullptr checks for
NetscapePlugInStreamLoader.m_client as needed.

• loader/NetscapePlugInStreamLoader.cpp: (WebCore::NetscapePlugInStreamLoader::NetscapePlugInStreamLoader): (WebCore::NetscapePlugInStreamLoader::willSendRequest): (WebCore::NetscapePlugInStreamLoader::didReceiveResponse): (WebCore::NetscapePlugInStreamLoader::didReceiveDataOrBuffer): (WebCore::NetscapePlugInStreamLoader::didFinishLoading): (WebCore::NetscapePlugInStreamLoader::didFail): (WebCore::NetscapePlugInStreamLoader::willCancel):
• loader/NetscapePlugInStreamLoader.h:

Source/WebKit:

Making NetscapePlugInStreamLoaderClient inherit from
CanMakeWeakPtr<> caused PDFPlugin to fail to compile because one
of its parent classes already inherited from CanMakeWeakPtr<>.
To fix this, extract a PDFPluginStreamLoaderClient class from
PDFPlugin and link the lifetime of PDFPluginStreamLoaderClient
to PDFPlugin.

After moving the NetscapePlugInStreamLoaderClient virtual
methods from PDFPlugin to PDFPluginStreamLoaderClient, add some
simple methods to PDFPlugin so that PDFPluginStreamLoaderClient
doesn't access the instance variables directly. Also make use
of these new methods in other parts of PDFPlugin.

• WebProcess/Plugins/PDF/PDFPlugin.h: (WebKit::PDFPlugin::documentFinishedLoading): Add. (WebKit::PDFPlugin::identifierForLoader): Add. (WebKit::PDFPlugin::removeOutstandingByteRangeRequest): Add. (WebKit::PDFPlugin::PDFPluginStreamLoaderClient): Add class.
• WebProcess/Plugins/PDF/PDFPlugin.mm: (WebKit::PDFPlugin::PDFPlugin): (WebKit::PDFPlugin::receivedNonLinearizedPDFSentinel): (WebKit::PDFPlugin::getResourceBytesAtPosition): (WebKit::PDFPlugin::PDFPluginStreamLoaderClient::willSendRequest): Move from PDFPlugin. (WebKit::PDFPlugin::PDFPluginStreamLoaderClient::didReceiveResponse): Ditto. (WebKit::PDFPlugin::PDFPluginStreamLoaderClient::didReceiveData): Ditto. (WebKit::PDFPlugin::PDFPluginStreamLoaderClient::didFail): Ditto. (WebKit::PDFPlugin::PDFPluginStreamLoaderClient::didFinishLoading): Ditto. (WebKit::PDFPlugin::byteRangeRequestForLoader): (WebKit::PDFPlugin::forgetLoader): (WebKit::PDFPlugin::willSendRequest): Move to PDFPluginStreamLoaderClient. (WebKit::PDFPlugin::didReceiveResponse): Ditto. (WebKit::PDFPlugin::didReceiveData): Ditto. (WebKit::PDFPlugin::didFail): Ditto. (WebKit::PDFPlugin::didFinishLoading): Ditto.

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268003 268f45cc-cd09-0410-ab3c-d52691b4dbfc

1:20 PM Changeset in webkit [268480] by Alan Coon

• 4 edits in branches/safari-610-branch/Source

Cherry-pick r267116. rdar://problem/70267318

Integrator's note: Source/WebCore/platform/graphics/cocoa/FontCascadeCocoa.mm was altered instead of
Source/WebCore/platform/graphics/coretext/FontCascadeCoreText.cpp.

[macOS Big Sur] CGFontRenderingGetFontSmoothingDisabled() is no longer useful
​https://bugs.webkit.org/show_bug.cgi?id=216588
<rdar://problem/68657748>

Reviewed by Simon Fraser.

Source/WebCore:

No new tests because there is no behavior change.

• platform/graphics/coretext/FontCascadeCoreText.cpp: (WebCore::FontCascade::isSubpixelAntialiasingAvailable):

Source/WTF:

We can eliminate WebKit's use of it to eventually phase it out entirely.

• wtf/PlatformHave.h:

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@267116 268f45cc-cd09-0410-ab3c-d52691b4dbfc

1:00 PM Changeset in webkit [268479] by achristensen@apple.com

• 13 edits in trunk

Disallow ports in file URLs
​https://bugs.webkit.org/show_bug.cgi?id=217252

Reviewed by Darin Adler.

LayoutTests/imported/w3c:

• web-platform-tests/url/a-element-expected.txt:
• web-platform-tests/url/a-element-xhtml-expected.txt:
• web-platform-tests/url/failure-expected.txt:
• web-platform-tests/url/url-constructor-expected.txt:
• web-platform-tests/url/url-setters-expected.txt:

Source/WTF:

This matches Chrome and the URL specification.
Covered by newly passing web platform tests.

• wtf/URLParser.cpp:

(WTF::URLParser::parsePort):

Tools:

• TestWebKitAPI/Tests/WTF/URLParser.cpp:

(TestWebKitAPI::TEST_F):

LayoutTests:

• fast/loader/file-URL-with-port-number-expected.txt:
• fast/loader/file-URL-with-port-number.html:

Update test to reflect this new behavior. The test was added in r24484 and
based on discussion in ​https://github.com/whatwg/url/issues/548 I've decided to try it and see if we can make this change.

12:46 PM Changeset in webkit [268478] by Alexey Shvayka

• 2 edits in trunk/Tools

Unreviewed. Update my status to "reviewer" and add WebIDL expertise.

• Scripts/webkitpy/common/config/contributors.json:

11:56 AM Changeset in webkit [268477] by youenn@apple.com

• 7 edits in trunk

MediaRecorder .stop should not throw in Inactive state
​https://bugs.webkit.org/show_bug.cgi?id=217705

Reviewed by Eric Carlson.

LayoutTests/imported/w3c:

fix-217705

• web-platform-tests/mediacapture-record/MediaRecorder-stop-expected.txt:
• web-platform-tests/mediacapture-record/MediaRecorder-stop.html:

Source/WebCore:

Update according spec changes in ​https://github.com/w3c/mediacapture-record/pull/158.
stop no longer throws if media recorder is inactive.
Covered by updated test.

• Modules/mediarecorder/MediaRecorder.cpp:

(WebCore::MediaRecorder::stopRecording):

• Modules/mediarecorder/MediaRecorder.h:
• Modules/mediarecorder/MediaRecorder.idl:

11:06 AM Changeset in webkit [268476] by Simon Fraser

• 37 edits
  4 adds in trunk

Allow passive mouse wheel event listeners to not force synchronous scrolling
​https://bugs.webkit.org/show_bug.cgi?id=158439
<rdar://problem/28265360>

Reviewed by Tim Horton.
Source/WebCore:

Turn on ENABLE_WHEEL_EVENT_REGIONS for macOS, and hit-test the passive and active regions
in ScrollingTree::determineWheelEventProcessin() to determine whether scrolling needs to
happen on the main thread, communicating via WheelEventProcessingSteps.

Now that we use EventRegion to store wheel event region information, we no longer need
to consult wheel event targets im ScrollingCoordinator::absoluteEventTrackingRegionsForFrame(),
but the non-fast region does still contain rectangles for non-fast-scrollable overflow
if async overflow scrolling is disabled.

Frame::invalidateContentEventRegionsIfNeeded() needs a fix to ensure that when wheel
event handlers in non-fast-scrollable subframes change we invalidate event regions.

RenderLayerBacking::updateEventRegion() needs a fix so that the event region for the root
(for handlers registered on the document or window) covers the entire root layer; painting
code only covered the RenderView, which is too small (and root background painting is complex).

Finally, when the processing steps given to EventHandler::handleWheelEvent() do not
include MainThreadForScrolling, then we shouldn't scroll here. The check in
EventHandler::handleWheelEvent() applies to frames, and the one in EventHandler::defaultWheelEventHandler()
applies to overflow:scroll.

Test: fast/scrolling/mac/slow-scrolling-overflow.html

• page/EventHandler.cpp:

(WebCore::EventHandler::handleWheelEvent):
(WebCore::EventHandler::defaultWheelEventHandler):

• page/Frame.cpp:

(WebCore::Frame::invalidateContentEventRegionsIfNeeded):

• page/scrolling/ScrollingCoordinator.cpp:

(WebCore::ScrollingCoordinator::absoluteEventTrackingRegionsForFrame const):

• page/scrolling/ScrollingTree.cpp:

(WebCore::ScrollingTree::determineWheelEventProcessing):

Source/WTF:

Define ENABLE_WHEEL_EVENT_REGIONS for macOS.

• wtf/PlatformEnableCocoa.h:

LayoutTests:

Change tests that relied on dumping the non-fast scrollable rects for wheel handlers
to dump layer trees with event regions instead. They also need to wait for a
rendering update, since wheel event regions are updated once per frame.

slow-scrolling-overflow.html tests that an overflow:scroll that's forced into slow
scrolling because of background-attachment:fixed still scrolls.

• fast/scrolling/mac/slow-scrolling-overflow-expected.txt: Added.
• fast/scrolling/mac/slow-scrolling-overflow.html: Added.
• fast/scrolling/mac/wheel-event-listener-region-inside-overflow-scroll-clipped-out-expected.txt:
• fast/scrolling/mac/wheel-event-listener-region-inside-overflow-scroll-expected.txt:
• platform/mac-wk2/TestExpectations:
• tiled-drawing/scrolling/non-fast-region/fixed-div-in-scrollable-page.html:
• tiled-drawing/scrolling/non-fast-region/handlers-in-iframes-expected.txt:
• tiled-drawing/scrolling/non-fast-region/handlers-in-iframes.html:
• tiled-drawing/scrolling/non-fast-region/non-fast-scrollable-region-hide-show-iframe-expected.txt:
• tiled-drawing/scrolling/non-fast-region/non-fast-scrollable-region-hide-show-iframe.html:
• tiled-drawing/scrolling/non-fast-region/resources/wheel-handler-region-helper.js:

(nonFastScrollableRects):
(dumpNonFastScrollableRects):
(async dumpRegionAndNotifyDone):
(rectsAsString): Deleted.
(dumpRegion): Deleted.

• tiled-drawing/scrolling/non-fast-region/top-content-inset-expected.txt:
• tiled-drawing/scrolling/non-fast-region/top-content-inset-header-expected.txt:
• tiled-drawing/scrolling/non-fast-region/top-content-inset-header.html:
• tiled-drawing/scrolling/non-fast-region/top-content-inset.html:
• tiled-drawing/scrolling/non-fast-region/wheel-event-plugin.html:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-fixed-child-expected.txt:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-fixed-child.html:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-in-columns-expected.txt:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-in-columns.html:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-in-overflow-scroll-expected.txt:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-in-overflow-scroll.html:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-inside-fixed-expected.txt:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-inside-fixed.html:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-on-document-expected.txt:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-on-document.html:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-on-fixed-expected.txt:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-on-fixed.html:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-region-basic-expected.txt:
• tiled-drawing/scrolling/non-fast-region/wheel-handler-region-basic.html:

11:03 AM Changeset in webkit [268475] by Keith Rollin

• 2 edits in trunk/Tools

'make debug' fails on the repository root if ccache is installed on mac after r262147
​https://bugs.webkit.org/show_bug.cgi?id=212469
<rdar://problem/70278783>

Reviewed by David Kilzer.

ccache is frequently installed in /usr/local/bin. However, when
running under XCBuild, $PATH does not include /usr/local/bin. This
leaves in a situation where the check for the existence of ccache in
Makefile.shared succeeds, but the use of ccache in ccache-wrapper
fails. To address this, look for ccache in a few well-known places
rather than just relying on $PATH. If it still can't be found, fall
back to compiling normally without ccache.

• ccache/ccache-wrapper:

10:56 AM Changeset in webkit [268474] by weinig@apple.com

• 10 edits
  2 adds in trunk/Tools

[Testing] Support configuring any preference from test headers for WebKitTestRunner
​https://bugs.webkit.org/show_bug.cgi?id=217645

Reviewed by Tim Horton.

Support using any preference defined in any of the WebPreference*.yaml configuration files
as a test header command rather than limiting it to a hard coded subset by generating the
list of supported commands and their types from the yaml files themselves.

This currently only works for WebKitTestRunner, but will be made to work with DumpRenderTree
in subsequent changes.

• WebKitTestRunner/CMakeLists.txt:
• WebKitTestRunner/Configurations/Base.xcconfig:
• WebKitTestRunner/DerivedSources-input.xcfilelist:
• WebKitTestRunner/DerivedSources-output.xcfilelist:
• WebKitTestRunner/DerivedSources.make:
• WebKitTestRunner/WebKitTestRunner.xcodeproj/project.pbxproj:
• WebKitTestRunner/Scripts/PreferencesTemplates: Added.
• WebKitTestRunner/Scripts/PreferencesTemplates/TestOptionsGeneratedKeys.h.erb: Added.

Add generation of TestOptionsGeneratedKeys.h from the WebPreference*.yaml using
the shared GeneratePreferences.rb script in WTF.

• WebKitTestRunner/TestController.cpp:

(WTR::TestController::resetPreferencesToConsistentValues):
Move preference setting to the bottom of the file to allow any preference
to be overriden. Also adds in support for double, uint32_t, and string preferences
though none of those are currently being used.

• WebKitTestRunner/TestOptions.cpp:

(WTR::TestOptions::keyTypeMapping):
Use generated macro GENERATED_WEB_PREFERENCE_KEY_TYPE_MAPPINGS rather than hardcoding
all the preferences types.

• WebKitTestRunner/TestOptions.h:

(WTR::TestOptions::boolWebPreferenceFeatures const):
(WTR::TestOptions::doubleWebPreferenceFeatures const):
(WTR::TestOptions::uint32WebPreferenceFeatures const):
(WTR::TestOptions::stringWebPreferenceFeatures const):
Expose accessors for preferences values for use in setting the actual preference
values. Rather than setting all preferences, we now only set preferences that have
been explicitly requested, using the default value for any that have not.

10:50 AM Changeset in webkit [268473] by Devin Rousso

• 10 edits in trunk/Source

Web Inspector: REGRESSION(r267148): new Big Sur styles are not used
​https://bugs.webkit.org/show_bug.cgi?id=217682
<rdar://problem/70269030>

Reviewed by Brian Burg.

r267148 changed navigator.userAgent from 11_0 to 10_15_7. This breaks Web Inspector
styling, which relied on this value to know what OS version it was running on in order to
know which styles to use.

Source/WebCore:

• inspector/InspectorFrontendHost.idl:
• inspector/InspectorFrontendHost.h:
• inspector/InspectorFrontendHost.cpp:

(WebCore::InspectorFrontendHost::platformVersionName const): Added.
Derive a lowercased kekab-cased platform version name from __MAC_OS_X_VERSION_MIN_REQUIRED.

Source/WebInspectorUI:

• UserInterface/Base/Platform.js:
• UserInterface/Views/Variables.css:

(body.mac-platform): Added.
(body.mac-platform:not(.sierra, .high-sierra)): Deleted.
Use InspectorFrontendHost.platformVersionName instead of navigator.userAgent. Remove
unnecessary platform version names and other unused data.

• UserInterface/Base/Main.js:

(WI.contentLoaded):
(WI.undockedTitleAreaHeight): Added.

• UserInterface/Views/Popover.js:

(WI.Popover.prototype._update):
Provide a way to get the undocked title area height for use in JavaScript calculations.

• UserInterface/Controllers/HARBuilder.js:

(WI.HARBuilder.creator):
The WI.Platform.version.build hasn't changed in a long time, so just remove it.

10:44 AM Changeset in webkit [268472] by svillar@igalia.com

• 2 edits in trunk/Source/WebKit

[GLIB] Fix linking error for systems not providing <sys/memfd.h>
​https://bugs.webkit.org/show_bug.cgi?id=217514

Reviewed by Carlos Garcia Campos.

• UIProcess/Launcher/glib/BubblewrapLauncher.cpp: Unconditionally check for <sys/mman.h> which should always exist instead

of <sys/memfd.h>. Also define the fallback with syscall() only for linux systems (as it requires <linux/memfd.h>).

10:30 AM Changeset in webkit [268471] by stephan.szabo@sony.com

• 3 edits in trunk/Tools

builtins-generator-tests and bindings-generation-tests are reporting "Failed to install configparser-4.0.2!" since r268267 on WinCairo Buildbot
​https://bugs.webkit.org/show_bug.cgi?id=217586

Reviewed by Jonathan Bedard.

Pass proxy environment variables to setup.py so wincairo bots
which need the proxy set can reach pypi.org.

• Scripts/libraries/webkitcorepy/webkitcorepy/init.py:
• Scripts/libraries/webkitcorepy/webkitcorepy/autoinstall.py:

10:20 AM Changeset in webkit [268470] by Alan Coon

• 3 edits in branches/safari-610-branch/Source/WebCore

Apply patch. rdar://problem/70267314

10:17 AM Changeset in webkit [268469] by Alan Coon

• 34 edits in branches/safari-610-branch

Cherry-pick r268371. rdar://problem/70267767

[macOS] Workaround for MAC_OS_X_VERSION_MAJOR incorrectly including minor version when building
with Xcode 12 on macOS Big Sur SUs
​https://bugs.webkit.org/show_bug.cgi?id=217602
rdar://70194453

Patch by Luming Yin <​luming_yin@apple.com> on 2020-10-12
Reviewed by Darin Adler.

The previous workaround turns out to be ineffective because we can't set the value of
TARGET_MAC_OS_X_VERSION_MAJOR based on a previous value of itself. Introduce a new
variable TARGET_MAC_OS_X_VERSION_MAJOR to determine whether we need to explicitly
adjust MAC_OS_X_VERSION_MAJOR to 110000.

PerformanceTests:

• DecoderTest/Configurations/DebugRelease.xcconfig:
• MediaTime/Configurations/DebugRelease.xcconfig:

Source/bmalloc:

• Configurations/DebugRelease.xcconfig:

Source/JavaScriptCore:

• Configurations/DebugRelease.xcconfig:

Source/ThirdParty:

• gtest/xcode/Config/General.xcconfig:

Source/ThirdParty/ANGLE:

• Configurations/DebugRelease.xcconfig:

Source/ThirdParty/libwebrtc:

• Configurations/DebugRelease.xcconfig:

Source/WebCore:

• Configurations/DebugRelease.xcconfig:

Source/WebCore/PAL:

• Configurations/DebugRelease.xcconfig:

Source/WebInspectorUI:

• Configurations/DebugRelease.xcconfig:

Source/WebKit:

• Configurations/DebugRelease.xcconfig:

Source/WebKitLegacy/mac:

• Configurations/DebugRelease.xcconfig:

Source/WTF:

• Configurations/DebugRelease.xcconfig:

Tools:

• ContentExtensionTester/Configurations/DebugRelease.xcconfig:
• DumpRenderTree/mac/Configurations/DebugRelease.xcconfig:
• ImageDiff/cg/Configurations/DebugRelease.xcconfig:
• MiniBrowser/Configurations/DebugRelease.xcconfig:
• TestWebKitAPI/Configurations/DebugRelease.xcconfig:
• WebEditingTester/Configurations/DebugRelease.xcconfig:
• WebKitTestRunner/Configurations/DebugRelease.xcconfig:
• lldb/lldbWebKitTester/Configurations/DebugRelease.xcconfig:

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268371 268f45cc-cd09-0410-ab3c-d52691b4dbfc

10:17 AM Changeset in webkit [268468] by Alan Coon

• 34 edits in branches/safari-610-branch

Cherry-pick r268353. rdar://problem/70267767

[macOS] Workaround for MAC_OS_X_VERSION_MAJOR incorrectly including minor version when building
with Xcode 12 on macOS Big Sur SUs
​https://bugs.webkit.org/show_bug.cgi?id=217602
rdar://70194453

Patch by Luming Yin <​luming_yin@apple.com> on 2020-10-12
Reviewed by Darin Adler.

Due to a bug in Xcode (rdar://70185899), Xcode 12.0 and Xcode 12.1 Beta incorrectly includes the
minor release number in MAC_OS_X_VERSION_MAJOR, which causes Debug and Release builds of WebKit
to be misconfigured when building on macOS Big Sur SUs, leading to webpages failing to load.

To work around the Xcode bug, when the MAC_OS_X_VERSION_MAJOR includes the minor version number,
drop the minor version number by explicitly setting TARGET_MAC_OS_X_VERSION_MAJOR to 110000.

Note: This change should be reverted after <rdar://70185899> is resolved.

PerformanceTests:

• DecoderTest/Configurations/DebugRelease.xcconfig:
• MediaTime/Configurations/DebugRelease.xcconfig:

Source/bmalloc:

• Configurations/DebugRelease.xcconfig:

Source/JavaScriptCore:

• Configurations/DebugRelease.xcconfig:

Source/ThirdParty:

• gtest/xcode/Config/General.xcconfig:

Source/ThirdParty/ANGLE:

• Configurations/DebugRelease.xcconfig:

Source/ThirdParty/libwebrtc:

• Configurations/DebugRelease.xcconfig:

Source/WebCore:

• Configurations/DebugRelease.xcconfig:

Source/WebCore/PAL:

• Configurations/DebugRelease.xcconfig:

Source/WebInspectorUI:

• Configurations/DebugRelease.xcconfig:

Source/WebKit:

• Configurations/DebugRelease.xcconfig:

Source/WebKitLegacy/mac:

• Configurations/DebugRelease.xcconfig:

Source/WTF:

• Configurations/DebugRelease.xcconfig:

Tools:

• ContentExtensionTester/Configurations/DebugRelease.xcconfig:
• DumpRenderTree/mac/Configurations/DebugRelease.xcconfig:
• ImageDiff/cg/Configurations/DebugRelease.xcconfig:
• MiniBrowser/Configurations/DebugRelease.xcconfig:
• TestWebKitAPI/Configurations/DebugRelease.xcconfig:
• WebEditingTester/Configurations/DebugRelease.xcconfig:
• WebKitTestRunner/Configurations/DebugRelease.xcconfig:
• lldb/lldbWebKitTester/Configurations/DebugRelease.xcconfig:

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268353 268f45cc-cd09-0410-ab3c-d52691b4dbfc

10:17 AM Changeset in webkit [268467] by Alan Coon

• 9 edits in branches/safari-610-branch

Cherry-pick r268331. rdar://problem/70267393

[macOS] Harden more build configurations against macOS versions 11 and higher as well as inclusion of patch versions
​https://bugs.webkit.org/show_bug.cgi?id=217595

Reviewed by Alexey Proskuryakov.

PerformanceTests:

• MediaTime/Configurations/Base.xcconfig: Updated to same idiom used elsewhere.

Tools:

• ContentExtensionTester/Configurations/Base.xcconfig: Updated to same idiom used elsewhere.
• DumpRenderTree/mac/Configurations/Base.xcconfig: Ditto.
• ImageDiff/cg/Configurations/Base.xcconfig: Ditto.
• WebKitTestRunner/Configurations/Base.xcconfig: Ditto.
• lldb/lldbWebKitTester/Configurations/Base.xcconfig: Ditto.

• Scripts/webkitpy/common/config/contributors.json: Added contributor Luming Yin, an engineer who works on Safari at Apple who has been contributing build fixes like these.

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268331 268f45cc-cd09-0410-ab3c-d52691b4dbfc

10:17 AM Changeset in webkit [268466] by Alan Coon

• 4 edits in branches/safari-610-branch

Cherry-pick r268206. rdar://problem/70267322

Presidential Executive Order pages not accessible with Safari.
​https://bugs.webkit.org/show_bug.cgi?id=217415
<rdar://problem/69922416>

Reviewed by Chris Fleizach and Simon Fraser.

Follow up to the previous change for this bug per Simon Fraser's comment.
Check not only the visibility and opacity of the object in question but
also of its ancestors.
Expanded the test accessibility/aria-modal.html to include the case
where the visibility of the modal dialog is determined by the opacity of
its parent object.

• accessibility/AXObjectCache.cpp: (WebCore::AXObjectCache::isNodeVisible const):

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268206 268f45cc-cd09-0410-ab3c-d52691b4dbfc

10:17 AM Changeset in webkit [268465] by Alan Coon

• 4 edits
  2 adds in branches/safari-610-branch

Cherry-pick r268162. rdar://problem/70267741

REGRESSION: Safari unable to load PDF in <embed> (docs.legalconnect.com)
​https://bugs.webkit.org/show_bug.cgi?id=217451
<rdar://problem/69767043>

Reviewed by Alex Christensen.

Source/WebCore:

Test: fast/replaced/pdf-as-embed-with-no-mime-type-is-not-blank.html

If plugins are enabled, we'll always let the request go through, and WebKit will
guess that files with PDFPlugin-handled extensions should instantiate PDFPlugin,
even if no other plugins are available.

However, if plugins are disabled, requestPlugin() will early return if the explicitly
specified MIME type is not handled by an application plugin (even though the downstream
WebKit code would have happily instantiated an application plugin for us).

Application plugins shouldn't depend on the plugin enablement setting.
To fix this, have SubframeLoader guess the MIME type if not explicitly specified
(matching WebKit's behavior), and allow the request if it matches an application plugin.

• loader/SubframeLoader.cpp: (WebCore::findPluginMIMETypeFromURL): Improve this previously logging-only function to use the lastPathComponent of the URL instead of randomly looking at the end of the URL, to ignore query strings and fragments when looking for the file extension.

(WebCore::FrameLoader::SubframeLoader::requestPlugin):
Make use of findPluginMIMETypeFromURL to guess the MIME type if it's not
explicitly specified. If the guessed MIME type is one that is handled
by application plugins, allow the request to go out to WebKit (which
may then instantiate a PDFPlugin, for example).

(WebCore::logPluginRequest):
(WebCore::FrameLoader::SubframeLoader::requestObject):
(WebCore::FrameLoader::SubframeLoader::createJavaAppletWidget):
Pass the URL instead of stringifying it, so we can lastPathComponent as above.

LayoutTests:

• fast/replaced/pdf-as-embed-with-no-mime-type-is-not-blank-expected-mismatch.html: Added.
• fast/replaced/pdf-as-embed-with-no-mime-type-is-not-blank.html: Added. Add a test ensuring that <embed> with no specified MIME type still renders the PDF.

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268162 268f45cc-cd09-0410-ab3c-d52691b4dbfc

10:17 AM Changeset in webkit [268464] by Alan Coon

• 5 edits in branches/safari-610-branch

Cherry-pick r268117. rdar://problem/70267698

Presidential Executive Order pages not accessible with Safari.
​https://bugs.webkit.org/show_bug.cgi?id=217415

Reviewed by Chris Fleizach.

Source/WebCore:

Test: accessibility/aria-modal.html.

These pages have a dialog modal element of the form

<div class="popover" role="dialog" aria-modal="true">

with style visible and a display value different from none. This makes
WebCore/accessibility to treat it as if the dialog were always visible,
and thus do not convey the content of the page outside the dialog to
VoiceOver. The author controls the visibility of the dialog using the
style opacity. This patch adds the check for the opacity to determine
whether an element is visible or not.

• accessibility/AXObjectCache.cpp: (WebCore::AXObjectCache::isNodeVisible const):

LayoutTests:

Expanded this test to cover the case whether the modal dialog visibility
is controlled by its opacity. Cleaned up JS code and made it work in
isolated tree mode.

• accessibility/aria-modal-expected.txt:
• accessibility/aria-modal.html:

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268117 268f45cc-cd09-0410-ab3c-d52691b4dbfc

10:17 AM Changeset in webkit [268463] by Alan Coon

• 3 edits
  2 adds in branches/safari-610-branch/Source/WebKit

Cherry-pick r267352. rdar://problem/70267163

WKWebView Swift overlay has mis-annotated nullability for evaluateJavaScript
<​http://webkit.org/b/216198>
<rdar://problem/68035950>

Reviewed by Darin Adler.

Due to a mistranslation of evaluateJavaScript, we are vending an API which does not expect
to receive nil as a valid result value. This change fixes the crash, but does not yet fix
the API to have the correct signature. That will come in a later patch.

To fix the crash, we need to produce a valid Result<Any, Error> to pass back to clients.
Fortunately, instead of inventing something clever, we can just use nil. It's valid to box
optional values into Any, and clients can technically retrieve them with the right dynamic cast
as well. Since client code must be using dynamic casting to convert the result Any to a usable
type, and because in the case where we now return a new value at runtime we would have previously
crashed, this shouldn't have any binary compatibility impact either.

To better validate these changes, I also add new unit tests for the conversion of JavaScript
results into Swift values, including a test for the deprecated API.

• SwiftOverlay/SwiftOverlay/ObjectiveCBlockConversions.swift: Added. For clarity I'm factoring helper methods into a single namespace, as it also makes a nice place to document their expectations. (ObjectiveCBlockConversion.exclusive.exclusive(_:)): This is renamed from the free function,

makeResultHandler(_:). It still has the same fatalError (now precondition) as before, but
hopefully a better name to clarify that it expects exactly-one value.

(ObjectiveCBlockConversions.boxingNilAsAnyForCompatibility(_:)): This is a variant of exclusive(_:)

that makes the tradeoff of boxing any nil values as Any to avoid crashing. This is still safe,
since as mentioned our clients will need to cast the value they recieve to do anything with it,
and since the deprecated API expects Any, no one could have been successfully comparing it
to nil today anyways.

• SwiftOverlay/Tests/JavaScriptToSwiftTypeConversions.swift: Added. (JavaScriptToSwiftConversions.setUp): Construct a new web view, and add it to a window so that it is

in an expected state. I'm using about:blank as the URL, since page content doesn't matter for
these tests and I want the web content to be ready immediately.

(JavaScriptToSwiftConversions.tearDown): Just perform some window cleanup.
(JavaScriptToSwiftConversions.evaluateJavaScript(_:andExpect:)): Helper method to evaluate script and

check its result. I'm using String.debugDescription because it escapes quotes and special characters
which makes the readout easier to parse.

(JavaScriptToSwiftConversions.testNull): JavaScript's null is actually mapped to NSNull, not nil.
(JavaScriptToSwiftConversions.testInteger): Some standard type coercion tests. The underlying value for

all number types should be NSNumber, so this is actually check against a float or integer type
without issue.

(JavaScriptToSwiftConversions.testDecimal): Ditto.
(JavaScriptToSwiftConversions.testBoolean): Ditto.
(JavaScriptToSwiftConversions.testString): Ditto.
(JavaScriptToSwiftConversions.testArray): Ditto.
(JavaScriptToSwiftConversions.testDictionary): Ditto, only you can't evaluate an object literal directly

so I need to store it in a temporary location first.

(JavaScriptToSwiftConversions.testUndefined): Test our boxing of nil. The exact value matters less than

not crashing at all.

• SwiftOverlay/WebKitSwiftOverlay.xcodeproj/project.pbxproj: Added new files to project. I kept the

new test file and helper files within the SwiftOverlay group, because they do not contribute any API.

• UIProcess/API/Cocoa/WebKitSwiftOverlay.swift: (WKWebView.callAsyncJavaScript(_:arguments:in:in:completionHandler:)): Switch to a conversion which does

not trap on nil.

(WKWebView.createPDF(_:completionHandler:)): Updated to use new helper method name.
(WKWebView.createWebArchiveData(_:)): Ditto.
(WKWebView.evaluateJavaScript(_:in:in:completionHandler:Error:)): See above.
(makeResultHandler(_:): This has been subsumed by ObjCBlockConversions.

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@267352 268f45cc-cd09-0410-ab3c-d52691b4dbfc

10:17 AM Changeset in webkit [268462] by Alan Coon

• 3 edits in branches/safari-610-branch/Source/WebKit

Cherry-pick r267619. rdar://problem/70296953

Add -[WKWebView _loadRequest:shouldOpenExternalURLsPolicy:] to provide a way to allow app link
​https://bugs.webkit.org/show_bug.cgi?id=216977

Patch by Hoa Dinh <​dvh@apple.com> on 2020-09-25
Reviewed by Alex Christensen.

The method -[WKWebView _loadRequest:shouldOpenExternalURLs:] would only allow opening external or not.
It wasn't providing the ability to allow external URL but not app links.
This patch adds -[WKWebView _loadRequest:shouldOpenExternalURLsPolicy:]. The parameters value for
shouldOpenExternalURLsPolicy has the same meaning than WebCore::ShouldOpenExternalURLsPolicy.

• UIProcess/API/Cocoa/WKWebView.mm: (-[WKWebView _loadRequest:shouldOpenExternalURLsPolicy:]):
• UIProcess/API/Cocoa/WKWebViewPrivate.h: Added enum _WKShouldOpenExternalURLsPolicy,

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@267619 268f45cc-cd09-0410-ab3c-d52691b4dbfc

10:05 AM Changeset in webkit [268461] by Alan Coon

• 8 edits in branches/safari-610-branch/Source

Versioning.

WebKit-7610.3.4

9:19 AM Changeset in webkit [268460] by Ryan Haddad

• 2 edits in trunk/Tools

Temporarily reduce the number of iOS simulators booted during testing
​https://bugs.webkit.org/show_bug.cgi?id=217681

Reviewed by Jonathan Bedard.

To work around unexpected timouts with iOS 14 (webkit.org/b/217392),
increase the estimated memory cost for booting iOS simulators.

• Scripts/webkitpy/xcode/simulated_device.py:

(SimulatedDeviceManager): Remove an obsolete FIXME and change the
estimated memory cost for a simulator to 8GB, which reduces the number
of booted simulators in our CI from 10 to 8. This change reduced the
number of unexpected timeouts seen on our bots from ~20 to 2, and came
at a penalty of < 5 minutes to layout test duration.

8:47 AM Changeset in webkit [268459] by Chris Dumez

• 2 edits in trunk/Source/WTF

Enable AudioWorklet by default
​https://bugs.webkit.org/show_bug.cgi?id=217708

Reviewed by Eric Carlson.

Enable AudioWorklet by default on platforms where modern/unprefixed WebAudio
is enabled.

• Scripts/Preferences/WebPreferencesExperimental.yaml:

8:28 AM Changeset in webkit [268458] by Kate Cheney

• 41 edits
  4 deletes in trunk

Remove ResourceLoadStatistics telemetry
​https://bugs.webkit.org/show_bug.cgi?id=217665
<rdar://problem/69748988>

Reviewed by Sam Weinig.

Source/WebKit:

Telemetry calculation in ITP is unused, very complicated, a potential
privacy risk, and it causes flaky tests. We should remove it.

No new tests. Existing ITP tests will check for no behavior change.

• NetworkProcess/Classifier/ResourceLoadStatisticsDatabaseStore.cpp:

(WebKit::ResourceLoadStatisticsDatabaseStore::ResourceLoadStatisticsDatabaseStore):
(WebKit::makeMedianWithUIQuery): Deleted.
(WebKit::buildQueryStartAndEnd): Deleted.
(WebKit::makeMedianWithoutUIQuery): Deleted.
(WebKit::getMedianOfPrevalentResourcesWithUserInteraction): Deleted.
(WebKit::ResourceLoadStatisticsDatabaseStore::getNumberOfPrevalentResources const): Deleted.
(WebKit::ResourceLoadStatisticsDatabaseStore::getNumberOfPrevalentResourcesWithUI const): Deleted.
(WebKit::ResourceLoadStatisticsDatabaseStore::getTopPrevelentResourceDaysSinceUI const): Deleted.
(WebKit::getMedianOfPrevalentResourceWithoutUserInteraction): Deleted.
(WebKit::getNumberOfPrevalentResourcesInTopResources): Deleted.
(WebKit::makeStatisticQuery): Deleted.
(WebKit::ResourceLoadStatisticsDatabaseStore::getNumberOfPrevalentResourcesWithoutUI const): Deleted.
(WebKit::ResourceLoadStatisticsDatabaseStore::calculateTelemetryData const): Deleted.
(WebKit::ResourceLoadStatisticsDatabaseStore::calculateAndSubmitTelemetry const): Deleted.

• NetworkProcess/Classifier/ResourceLoadStatisticsDatabaseStore.h:
• NetworkProcess/Classifier/ResourceLoadStatisticsMemoryStore.cpp:

(WebKit::ResourceLoadStatisticsMemoryStore::ResourceLoadStatisticsMemoryStore):
(WebKit::ResourceLoadStatisticsMemoryStore::calculateAndSubmitTelemetry const): Deleted.

• NetworkProcess/Classifier/ResourceLoadStatisticsMemoryStore.h:
• NetworkProcess/Classifier/ResourceLoadStatisticsStore.cpp:

(WebKit::ResourceLoadStatisticsStore::setShouldSubmitTelemetry): Deleted.

• NetworkProcess/Classifier/ResourceLoadStatisticsStore.h:
• NetworkProcess/Classifier/WebResourceLoadStatisticsStore.cpp:

(WebKit::WebResourceLoadStatisticsStore::performDailyTasks):
(WebKit::WebResourceLoadStatisticsStore::setShouldSubmitTelemetry): Deleted.
(WebKit::WebResourceLoadStatisticsStore::submitTelemetry): Deleted.
(WebKit::WebResourceLoadStatisticsStore::notifyPageStatisticsTelemetryFinished const): Deleted.

• NetworkProcess/Classifier/WebResourceLoadStatisticsStore.h:
• NetworkProcess/Classifier/WebResourceLoadStatisticsTelemetry.cpp: Removed.
• NetworkProcess/Classifier/WebResourceLoadStatisticsTelemetry.h: Removed.
• NetworkProcess/NetworkProcess.cpp:

(WebKit::NetworkProcess::submitTelemetry): Deleted.

• NetworkProcess/NetworkProcess.h:
• NetworkProcess/NetworkProcess.messages.in:
• NetworkProcess/NetworkSession.cpp:

(WebKit::NetworkSession::notifyPageStatisticsTelemetryFinished): Deleted.

• NetworkProcess/NetworkSession.h:
• SourcesCocoa.txt:
• UIProcess/API/C/WKWebsiteDataStoreRef.cpp:

(WKWebsiteDataStoreStatisticsSubmitTelemetry): Deleted.

• UIProcess/API/C/WKWebsiteDataStoreRef.h:
• UIProcess/API/Cocoa/WKWebsiteDataStore.mm:
• UIProcess/Network/NetworkProcessProxy.cpp:

(WebKit::NetworkProcessProxy::submitTelemetry): Deleted.
(WebKit::NetworkProcessProxy::notifyResourceLoadStatisticsTelemetryFinished): Deleted.

• UIProcess/Network/NetworkProcessProxy.h:
• UIProcess/Network/NetworkProcessProxy.messages.in:
• UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::notifyPageStatisticsTelemetryFinished): Deleted.

• UIProcess/WebProcessProxy.h:
• UIProcess/WebsiteData/WebsiteDataStore.cpp:

(WebKit::WebsiteDataStore::submitTelemetry): Deleted.

• UIProcess/WebsiteData/WebsiteDataStore.h:
• WebKit.xcodeproj/project.pbxproj:
• PlatformFTW.cmake:
• SourcesGTK.txt:
• PlatformPlayStation.cmake:
• PlatformWin.cmake:
• SourcesWPE.txt:

Tools:

Telemetry calculation in ITP is unused, very complicated, a potential
privacy risk, and it causes flaky tests. We should remove it.

• WebKitTestRunner/InjectedBundle/Bindings/TestRunner.idl:
• WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:

(WTR::InjectedBundle::didReceiveMessageToPage):

• WebKitTestRunner/InjectedBundle/TestRunner.cpp:

(WTR::TestRunner::installStatisticsDidRunTelemetryCallback): Deleted.
(WTR::TestRunner::statisticsDidRunTelemetryCallback): Deleted.
(WTR::TestRunner::statisticsSubmitTelemetry): Deleted.

• WebKitTestRunner/InjectedBundle/TestRunner.h:
• WebKitTestRunner/TestController.cpp:

(WTR::TestController::statisticsSubmitTelemetry): Deleted.

• WebKitTestRunner/TestController.h:
• WebKitTestRunner/TestInvocation.cpp:

(WTR::TestInvocation::didReceiveSynchronousMessageFromInjectedBundle):

LayoutTests:

Remove layout tests.

• http/tests/resourceLoadStatistics/telemetry-generation-expected.txt: Removed.
• http/tests/resourceLoadStatistics/telemetry-generation.html: Removed.
• platform/wk2/TestExpectations:

8:24 AM Changeset in webkit [268457] by Kate Cheney

• 9 edits
  1 delete in trunk

Remove _WKWebViewCategory from WKWebViewConfiguration
​https://bugs.webkit.org/show_bug.cgi?id=217679
<rdar://problem/70260337>

Reviewed by Sam Weinig.

Source/WebKit:

This property is not used in any meaningful way. Its original purpose
was to categorize WKWebViews for app-bound domains, but this is
now done using the isNavigatingToAppBoundDomain property on
WebPageProxy. We should remove it.

No new tests, behavior confirmed by existing tests.

• Shared/WebViewCategory.h: Removed.
• UIProcess/API/APIPageConfiguration.cpp:

(API::PageConfiguration::copy const):

• UIProcess/API/APIPageConfiguration.h:

(API::PageConfiguration::webViewCategory const): Deleted.
(API::PageConfiguration::setWebViewCategory): Deleted.

• UIProcess/API/Cocoa/WKWebViewConfiguration.mm:

(toWebKitWebViewCategory): Deleted.
(toWKWebViewCategory): Deleted.
(-[WKWebViewConfiguration _webViewCategory]): Deleted.
(-[WKWebViewConfiguration _setWebViewCategory:]): Deleted.

• UIProcess/API/Cocoa/WKWebViewConfigurationPrivate.h:
• UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::setIsNavigatingToAppBoundDomainAndCheckIfPermitted):

• WebKit.xcodeproj/project.pbxproj:

Tools:

• TestWebKitAPI/Tests/WebKitCocoa/InAppBrowserPrivacy.mm:

7:00 AM Changeset in webkit [268456] by Alan Bujtas

• 11 edits in trunk/Source/WebCore

[LFC] Use the term overridden instead of override to indicate priority values
​https://bugs.webkit.org/show_bug.cgi?id=217657

Reviewed by Antti Koivisto.

The OverriddenHorizontal/Vertical structures are used to pass in priority values (e.g. use the "max-height" value instead of the computed "height" value) to geometry computation functions.

• layout/FormattingContext.h:
• layout/FormattingContextGeometry.cpp:

(WebCore::Layout::FormattingContext::Geometry::outOfFlowNonReplacedVerticalGeometry const):
(WebCore::Layout::FormattingContext::Geometry::outOfFlowNonReplacedHorizontalGeometry):
(WebCore::Layout::FormattingContext::Geometry::outOfFlowReplacedVerticalGeometry const):
(WebCore::Layout::FormattingContext::Geometry::outOfFlowReplacedHorizontalGeometry):
(WebCore::Layout::FormattingContext::Geometry::complicatedCases const):
(WebCore::Layout::FormattingContext::Geometry::floatingNonReplacedWidthAndMargin):
(WebCore::Layout::FormattingContext::Geometry::floatingReplacedHeightAndMargin const):
(WebCore::Layout::FormattingContext::Geometry::floatingReplacedWidthAndMargin):
(WebCore::Layout::FormattingContext::Geometry::outOfFlowVerticalGeometry const):
(WebCore::Layout::FormattingContext::Geometry::outOfFlowHorizontalGeometry):
(WebCore::Layout::FormattingContext::Geometry::floatingHeightAndMargin const):
(WebCore::Layout::FormattingContext::Geometry::floatingWidthAndMargin):
(WebCore::Layout::FormattingContext::Geometry::inlineReplacedHeightAndMargin const):
(WebCore::Layout::FormattingContext::Geometry::inlineReplacedWidthAndMargin):

• layout/LayoutUnits.h:
• layout/blockformatting/BlockFormattingContext.h:
• layout/blockformatting/BlockFormattingContextGeometry.cpp:

(WebCore::Layout::BlockFormattingContext::Geometry::inFlowNonReplacedHeightAndMargin):
(WebCore::Layout::BlockFormattingContext::Geometry::inFlowNonReplacedWidthAndMargin):
(WebCore::Layout::BlockFormattingContext::Geometry::inFlowReplacedWidthAndMargin):
(WebCore::Layout::BlockFormattingContext::Geometry::inFlowHeightAndMargin):
(WebCore::Layout::BlockFormattingContext::Geometry::inFlowWidthAndMargin):

• layout/blockformatting/tablewrapper/TableWrapperBlockFormattingContext.cpp:

(WebCore::Layout::TableWrapperBlockFormattingContext::computeWidthAndMarginForTableBox):
(WebCore::Layout::TableWrapperBlockFormattingContext::computeHeightAndMarginForTableBox):

• layout/blockformatting/tablewrapper/TableWrapperBlockFormattingContext.h:
• layout/blockformatting/tablewrapper/TableWrapperBlockFormattingContextQuirks.cpp:

(WebCore::Layout::TableWrapperBlockFormattingContext::Quirks::overriddenTableHeight const):
(WebCore::Layout::TableWrapperBlockFormattingContext::Quirks::overrideTableHeight const): Deleted.

• layout/inlineformatting/InlineFormattingContext.h:
• layout/inlineformatting/InlineFormattingContextGeometry.cpp:

(WebCore::Layout::InlineFormattingContext::Geometry::inlineBlockWidthAndMargin):
(WebCore::Layout::InlineFormattingContext::Geometry::inlineBlockHeightAndMargin const):

6:54 AM Changeset in webkit [268455] by aakash_jain@apple.com

• 2 edits in trunk/Tools

Rename slave to bot for gtk-linux bot names
​https://bugs.webkit.org/show_bug.cgi?id=217702

Reviewed by Carlos Alberto Lopez Perez.

• BuildSlaveSupport/build.webkit.org-config/config.json:

6:29 AM Changeset in webkit [268454] by Andres Gonzalez

• 16 edits in trunk

[ macOS ] accessibility/mac/select-element-selection-with-optgroups.html is a flaky failure
​https://bugs.webkit.org/show_bug.cgi?id=215649
<rdar://problem/67406148>

Reviewed by Chris Fleizach.

Source/WebCore:

Test: accessibility/mac/select-element-selection-with-optgroups.html.

Fix this test by using Promises and waiting for the correct events. No
longer flaky.
Also fixed the test in isolated tree mode, for which several changes detailed below were necessary.

• accessibility/AXObjectCache.cpp:

(WebCore::AXObjectCache::objectsForIDs const):
Added this utility method to retrieve a Vector of AX objects given their IDs.

• accessibility/AXObjectCache.h:
• accessibility/AccessibilityListBox.cpp:

(WebCore::AccessibilityListBox::canSetSelectedChildren const):
(WebCore::AccessibilityListBox::setSelectedChildren):
(WebCore::AccessibilityListBox::canSetSelectedChildrenAttribute const):
Became canSetSelectedChildren since Attribute was unnecessary and
misleading.

• accessibility/AccessibilityListBox.h:
• accessibility/AccessibilityObject.h:
• accessibility/AccessibilityObjectInterface.h:

(WebCore::AXCoreObject::setSelectedChildren): Exposes this functionality
in the base class so that it is accessible for AX and isolated objects.
(WebCore::AXCoreObject::isNativeListBox const): Deleted.

• accessibility/isolatedtree/AXIsolatedObject.cpp:

(WebCore::AXIsolatedObject::initializeAttributeData):
(WebCore::AXIsolatedObject::setSelectedChildren):
Implements this functionality by forwarding to the main thread and
retrieving the correspodning AX objects.
(WebCore::AXIsolatedObject::isAccessibilityListBoxInstance const):

• accessibility/isolatedtree/AXIsolatedObject.h:
• accessibility/isolatedtree/AXIsolatedTree.cpp:

(WebCore::AXIsolatedTree::objectsForIDs const):
(WebCore::AXIsolatedTree::idsForObjects const):

• accessibility/isolatedtree/AXIsolatedTree.h:
• accessibility/mac/WebAccessibilityObjectWrapperMac.mm:

(-[WebAccessibilityObjectWrapper accessibilityIsAttributeSettable:]):
(-[WebAccessibilityObjectWrapper _accessibilitySetValue:forAttribute:]):
Instead of special casing AccessibilityListBox class, it allows to set
selected children for all objects that canSetSelectedChildren.
Eliminates the downcast that does not work for isolated objects.

LayoutTests:

Uses Promises to make test deterministic. Also waits for the correct event
since selectedChildrenCount is not the right indication that the
expected update happened.

• accessibility/mac/select-element-selection-with-optgroups-expected.txt:
• accessibility/mac/select-element-selection-with-optgroups.html:
• platform/mac/TestExpectations: No longer flaky.

5:06 AM Changeset in webkit [268453] by zandobersek@gmail.com

• 32 edits
  5 deletes in trunk

Remove ACCELERATED_2D_CANVAS build flags and guarded code
​https://bugs.webkit.org/show_bug.cgi?id=217603

Reviewed by Carlos Garcia Campos.

.:

Yank the CMake ENABLE_ACCELERATED_2D_CANVAS option now that it won't
have any effect. FindCairoGL.cmake can also be removed.

• Source/cmake/FindCairoGL.cmake: Removed.
• Source/cmake/OptionsFTW.cmake:
• Source/cmake/OptionsGTK.cmake:
• Source/cmake/OptionsWPE.cmake:
• Source/cmake/OptionsWin.cmake:
• Source/cmake/WebKitFeatures.cmake:
• Source/cmake/tools/vsprops/FeatureDefines.props:
• Source/cmake/tools/vsprops/FeatureDefinesCairo.props:

Source/WebCore:

Remove code guarded by the ACCELERATED_2D_CANVAS build guard, along
with related build system cruft. This feature was only implemented for
Cairo, and even then was using an experimental GL-oriented backend of
Cairo that was not providing necessarily better performance or
experience and is not seeing further development.

• PlatformMac.cmake:
• SourcesCocoa.txt:
• WebCore.xcodeproj/project.pbxproj:
• html/HTMLCanvasElement.cpp:

(WebCore::HTMLCanvasElement::createContext2d):
(WebCore::HTMLCanvasElement::createContextBitmapRenderer):
(WebCore::HTMLCanvasElement::shouldAccelerate const):
(WebCore::HTMLCanvasElement::createImageBuffer const):

• html/ImageBitmap.cpp:
• html/canvas/CanvasRenderingContext2DBase.cpp:

(WebCore::CanvasRenderingContext2DBase::isAccelerated const):
(WebCore::CanvasRenderingContext2DBase::drawImage):
(WebCore::CanvasRenderingContext2DBase::createPattern):
(WebCore::CanvasRenderingContext2DBase::didDraw):
(WebCore::CanvasRenderingContext2DBase::platformLayer const): Deleted.

• html/canvas/CanvasRenderingContext2DBase.h:
• html/canvas/ImageBitmapRenderingContext.cpp:
• platform/Cairo.cmake:
• platform/SourcesCairo.txt:
• platform/graphics/GLContext.h:
• platform/graphics/PlatformImageBuffer.h:
• platform/graphics/cairo/CairoUtilities.cpp:

(WebCore::cairoSurfaceSize):

• platform/graphics/cairo/ImageBufferCairoGLSurfaceBackend.cpp: Removed.
• platform/graphics/cairo/ImageBufferCairoGLSurfaceBackend.h: Removed.
• platform/graphics/egl/GLContextEGL.cpp:

(WebCore::GLContextEGL::~GLContextEGL):
(WebCore::GLContextEGL::cairoDevice): Deleted.

• platform/graphics/egl/GLContextEGL.h:
• platform/graphics/glx/GLContextGLX.cpp:

(WebCore::GLContextGLX::~GLContextGLX):
(WebCore::GLContextGLX::cairoDevice): Deleted.

• platform/graphics/glx/GLContextGLX.h:
• platform/graphics/gpu/Texture.cpp: Removed.
• platform/graphics/gpu/Texture.h: Removed.
• platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:

(WebCore::MediaPlayerPrivateGStreamer::nativeImageForCurrentTime):

• rendering/RenderLayerBacking.cpp:

(WebCore::canvasCompositingStrategy):
(WebCore::RenderLayerBacking::updateConfiguration):
(WebCore::RenderLayerBacking::containsPaintedContent const):
(WebCore::RenderLayerBacking::contentChanged):

Source/WTF:

• wtf/PlatformEnable.h:

Remove the fallback ENABLE_ACCELERATED_2D_CANVAS definition.

Tools:

• Scripts/webkitperl/FeatureList.pm:

Remove the ACCELERATED_2D_CANVAS feature flag.

5:00 AM WebKitGTK/2.30.x edited by Adrian Perez de Castro

(diff)

4:58 AM Changeset in webkit [268452] by Adrian Perez de Castro

• 2 edits in releases/WebKitGTK/webkit-2.30/Source/WebCore

Merge r268390 - [GTK] Make gamepads visible after their first input is received
​https://bugs.webkit.org/show_bug.cgi?id=217494

Reviewed by Carlos Garcia Campos.

Turns out this was mostly working already, with a small caveat: Sometimes there would
be a timing issue if m_inputNotificationTimer would be fired before having a chance
of setting the m_initialGamepadsConnected flag, and in that case the early return
would prevent dispatchPlatformGamepadInputActivity() from ever being called. This
re-arms the timer to be fired again in the next event loop iteration if the flag was
unset, so it will be checked again after it has been toggled. This same approach is
used by HIDGamepadProvider in the Mac port, for example.

• platform/gamepad/manette/ManetteGamepadProvider.cpp:

(WebCore::ManetteGamepadProvider::inputNotificationTimerFired): Re-arm timer if
m_initialGamepadsConnected was not yet set.

4:58 AM Changeset in webkit [268451] by Adrian Perez de Castro

• 11 edits in releases/WebKitGTK/webkit-2.30/Source

Merge r268312 - [GTK] Build broken with ENABLE_GAMEPAD enabled

​https://bugs.webkit.org/show_bug.cgi?id=217226

Reviewed by Darin Adler.

Source/WebCore:

No new tests needed.

• Headers.cmake: List SharedGamepadValue.h in WebCore_PRIVATE_FRAMEWORK_HEADERS, which

was missing.

• platform/gamepad/EmptyGamepadProvider.h: Mark class as final, because the destructor

is final itself.

• platform/gamepad/manette/ManetteGamepad.cpp:

(WebCore::ManetteGamepad::ManetteGamepad): Adapt to use SharedGamepadValue.
(WebCore::ManetteGamepad::buttonPressedOrReleased): Ditto.
(WebCore::ManetteGamepad::absoluteAxisChanged): Ditto.

• platform/gamepad/manette/ManetteGamepad.h: Ditto.
• platform/gamepad/manette/ManetteGamepadProvider.cpp: Rename m_connectionDelayTimer

to m_initialGamepadsConnectedTimer, which better reflects the usage and also makes the
provider a bit more similar to HIDGamepadProvider (from the Mac port). Arrange to use
the EventMakesGamepadsVisible flag.
(WebCore::ManetteGamepadProvider::ManetteGamepadProvider): Remove the loop that iterates
over the pre-detected gamepads because they are being added too early before any client
has had the chance to be attached.
(WebCore::ManetteGamepadProvider::startMonitoringGamepads): The loop that iterates and
adds pre-detected gamepads is moved here, once there is at least a client. This solves
hitting an assertion later on when UIGamepadProvider checks that the number of gamepads
it has been notified of matches the amount tracked by ManetteGamepadProvider.
(WebCore::ManetteGamepadProvider::stopMonitoringGamepads):
(WebCore::ManetteGamepadProvider::deviceConnected):
(WebCore::ManetteGamepadProvider::deviceDisconnected):
(WebCore::ManetteGamepadProvider::initialGamepadsConnectedTimerFired):
(WebCore::ManetteGamepadProvider::inputNotificationTimerFired):

• platform/gamepad/manette/ManetteGamepadProvider.h: Ditto.

Source/WebKit:

• UIProcess/Gamepad/UIGamepadProvider.h: Mark class as final, because the destructor is

final itself.

• UIProcess/WebProcessPool.cpp:

(WebKit::WebProcessPool::numberOfConnectedGamepadsForTesting): Reorganize PLATFORM(MAC)
guards to use GameControllerGamepadProvider only on macOS, as it is unavailable on other
ports.

• WebProcess/Gamepad/WebGamepadProvider.h: Mark class as final, because the destructor

is final itself.

4:51 AM Changeset in webkit [268450] by Karl Rackler

• 2 edits in trunk/LayoutTests

Unreviewed, reverting r268441.

This reverts commit r268441 made in error.

Reverted changeset:

"REGRESSION (iOS 14): [ iOS wk2 ] 12 2d.gradient.radial WPT
tests are a flaky failure"
​https://bugs.webkit.org/show_bug.cgi?id=217690
​https://trac.webkit.org/changeset/268441

3:38 AM Changeset in webkit [268449] by jh718.park@samsung.com

• 2 edits in trunk/Source/WebKit

Unreviewed. Fix the build warning below since r268312.
warning: control reaches end of non-void function [-Wreturn-type]

No new tests, no new behaviors.

• UIProcess/WebProcessPool.cpp:

(WebKit::WebProcessPool::numberOfConnectedGamepadsForTesting):

3:03 AM Changeset in webkit [268448] by jh718.park@samsung.com

• 2 edits in trunk/Source/WebKit

Unreviewed. Fix the build warning below since r222233.
warning: redundant move in return statement [-Wredundant-move]

No new tests, no new behaviors.

• Shared/Gamepad/GamepadData.cpp:

(WebKit::GamepadData::decode):

2:33 AM Changeset in webkit [268447] by Diego Pino Garcia

• 2 edits in trunk/Tools

[webkitpy] Allow passing subtests to GLIB API testrunner
​https://bugs.webkit.org/show_bug.cgi?id=217466

Reviewed by Carlos Garcia Campos.

GLIB API testrunner didn't allow passing a list of subtests to run.
That was possible, however, when executing a GLIB API binary directly
using knob -p. The patch adds the same option to GLIB API testrunner
and passes down the list of subtests accordingly.

• glib/api_test_runner.py:

(TestRunner._run_test_glib):
(TestRunner._run_test):
(TestRunner.run_tests):
(add_options):

2:18 AM Changeset in webkit [268446] by commit-queue@webkit.org

• 4 edits in trunk/Source/WebCore

REGRESSION: fast/canvas/webgl/context-attributes-alpha.html fails
​https://bugs.webkit.org/show_bug.cgi?id=217697

Patch by Kimmo Kinnunen <​kkinnunen@apple.com> on 2020-10-14
Reviewed by Antti Koivisto.

The original work in "Cocoa: Make WebGLLayer not dependent on
GraphicsContextGLOpenGL" was missing the hunk to initialize
WebGLLayer contentsOpaque. Neither the EWS nor the local testing
used the test case.

No new tests, tested by
fast/canvas/webgl/context-attributes-alpha.html.

• platform/graphics/cocoa/GraphicsContextGLOpenGLCocoa.mm:

(WebCore::GraphicsContextGLOpenGL::GraphicsContextGLOpenGL):

• platform/graphics/cocoa/WebGLLayer.h:
• platform/graphics/cocoa/WebGLLayer.mm:

(-[WebGLLayer initWithClient:devicePixelRatio:contentsOpaque:]):
(-[WebGLLayer initWithClient:devicePixelRatio:]): Deleted.

1:09 AM Changeset in webkit [268445] by Fujii Hironori

• 2 edits in trunk/LayoutTests

[WinCairo] Unreviewed test gardening

• platform/wincairo/TestExpectations:

12:17 AM Changeset in webkit [268444] by graouts@webkit.org

• 3 edits
  6 adds in trunk

transform-origin is applied on top of invidivual CSS transform properties on composited layers
​https://bugs.webkit.org/show_bug.cgi?id=217661
<rdar://problem/70259960>

Reviewed by Simon Fraser.

Source/WebCore:

Ensure that transform origin is only applied when the RenderStyle::applyTransform() call site would
opt into it. This caused the call under RenderLayerBacking::updateTransform() to apply the transform
origin even though the resulting CALayer already relies on the anchorPoint property.

Tests: transforms/2d/rotate-composited.html

transforms/2d/scale-composited.html
transforms/2d/translate-composited.html

• rendering/style/RenderStyle.cpp:

(WebCore::RenderStyle::applyTransform const):
(WebCore::requireTransformOrigin): Deleted.

LayoutTests:

Add new tests checking that individual CSS properties render the same as the single transform property
when applied to a composited element.

• transforms/2d/rotate-composited-expected.html: Added.
• transforms/2d/rotate-composited.html: Added.
• transforms/2d/scale-composited-expected.html: Added.
• transforms/2d/scale-composited.html: Added.
• transforms/2d/translate-composited-expected.html: Added.
• transforms/2d/translate-composited.html: Added.

Oct 13, 2020:

8:20 PM Changeset in webkit [268443] by rniwa@webkit.org

• 3 edits in trunk

IPC testing JS API should expose a reply and describe the list of arguments for each message
​https://bugs.webkit.org/show_bug.cgi?id=217565

Fixed the tests.

• TestWebKitAPI/Tests/WebKitCocoa/IPCTestingAPI.mm:

8:03 PM Changeset in webkit [268442] by sbarati@apple.com

• 3 edits
  1 add in trunk

JSObject::getPropertyNames should have a stack overflow check
​https://bugs.webkit.org/show_bug.cgi?id=217677
<rdar://problem/69286436>

Reviewed by Tadeu Zagallo.

JSTests:

• stress/get-property-names-stack-overflow.js: Added.

Source/JavaScriptCore:

A prototype chain can be long enough where the recursion causes a stack
overflow. The attached test case uses $vm to mimic such a prototype chain
by using JSProxy to make a cyclic prototype chain. But the same works by
making a prototype chain very long, but the test case takes too long to
run to justify landing. For posterity, the alternate test case is:

`
function makeLongProtoChain(length) {

let object = /foo/;
for (let i = 0; i < length; ++i) {

next = /foo/;
next.proto = object;
object = next;

}
return object;

}

let o = makeLongProtoChain(30000);
for (let q in o) {}
`

• runtime/JSObject.cpp:

(JSC::JSObject::getPropertyNames):

6:30 PM Changeset in webkit [268441] by Karl Rackler

• 2 edits in trunk/LayoutTests

REGRESSION (iOS 14): [ iOS wk2 ] 12 2d.gradient.radial WPT tests are a flaky failure
​https://bugs.webkit.org/show_bug.cgi?id=217690

Unreviewed test gardening.

• platform/ios-simulator-wk2/TestExpectations:

6:10 PM Changeset in webkit [268440] by Jonathan Bedard

• 3 edits in trunk/Tools

[webkitscmpy] Add git-webkit find (Follow up fix)
​https://bugs.webkit.org/show_bug.cgi?id=217534
<rdar://problem/70152431>

Unreviewed follow up fix.

• Scripts/libraries/webkitcorepy/webkitcorepy/init.py: Version bump.
• Scripts/libraries/webkitscmpy/setup.py: "program" was not a directory in the landed change.

5:43 PM Changeset in webkit [268439] by Alan Coon

• 4 edits in branches/safari-610.2.11.0-branch

Revert r268206. rdar://problem/70258965

5:28 PM Changeset in webkit [268438] by Hector Lopez

• 2 edits in trunk/LayoutTests

[ macOS ] inspector/page/overrideSetting-ITPDebugModeEnabled.html is a flaky failure
​https://bugs.webkit.org/show_bug.cgi?id=217686

Unreviewed test gardening.

• platform/mac-wk2/TestExpectations:

5:23 PM Changeset in webkit [268437] by Karl Rackler

• 2 edits in trunk/LayoutTests

REGRESSION (iOS 14): [ iOS wk2 ] accessibility tests are a flaky timeout
​https://bugs.webkit.org/show_bug.cgi?id=217687

Unreviewed test gardening.

• platform/ios-simulator-wk2/TestExpectations:

5:19 PM Changeset in webkit [268436] by Megan Gardner

• 6 edits
  3 adds in trunk

Paint CSS highlights over images
​https://bugs.webkit.org/show_bug.cgi?id=217417
Source/WebCore:

<rdar://problem/58459054>

Reviewed by Darin Adler.

Add support to paint CSS pseudo highlights over images that are in the
range of the defined highlight.

Test: http/wpt/css/css-highlight-api/highlight-image.html

• rendering/HighlightData.cpp:

(WebCore::HighlightData::highlightStateForRenderer):

• rendering/HighlightData.h:
• rendering/RenderReplaced.cpp:

(WebCore::RenderReplaced::potentialHighlightColor const):
(WebCore::RenderReplaced::paint):

• rendering/RenderReplaced.h:

LayoutTests:

Reviewed by Darin Adler.

• http/wpt/css/css-highlight-api/highlight-image-expected-mismatched.html: Added.
• http/wpt/css/css-highlight-api/highlight-image.html: Added.

5:16 PM commit-identifiers edited by Jonathan Bedard

(diff)

5:14 PM Changeset in webkit [268435] by weinig@apple.com

• 5 edits in trunk/Source/WebCore

[WebIDL] Add support for identifiers with dashes ('-')
​https://bugs.webkit.org/show_bug.cgi?id=217667

Reviewed by Darin Adler.

To support CSSStyleDeclaration.idl correctly, we need to add support for dashes ('-')
in WebIDL identifers.

• bindings/scripts/IDLParser.pm:

Update identifer regular expression to match current spec which allows for dashes ('-').

• bindings/scripts/CodeGeneratorJS.pm:

(MangleAttributeOrFunctionName):
Helper to centralize mangling required to construct valid c++ identifiers from the WebIDL
identifiers, since now WebIDL identifiers can contain dashes ('-'), which is not valid in
c++. For now, we use a very simple mangling which replaces all dashes ('-') with the string
"_dash_".

(GetAttributeGetterName):
(GetAttributeSetterName):
(GetFunctionName):
Clarify and cleanup logic a bit and call the new mangler.

(GenerateImplementation):
GetAttributeGetterName will now always use the $className, so this caller needs to actually
pass the right value in. Previously, the $generatorName it passed in was not actually used.
Also removes some duplicate definitions of $interfaceName.

• bindings/scripts/test/JS/JSTestObj.cpp:
• bindings/scripts/test/TestObj.idl:

Adds tests for various combinations of underscores and dashes in identifiers.

4:57 PM Changeset in webkit [268434] by Alan Coon

• 1 copy in tags/Safari-610.2.11.1.2

Tag Safari-610.2.11.1.2.

4:53 PM Changeset in webkit [268433] by Jonathan Bedard

• 9 edits
  2 copies
  2 adds in trunk/Tools

[webkitscmpy] Add git-webkit find
​https://bugs.webkit.org/show_bug.cgi?id=217534
<rdar://problem/70152431>

Reviewed by Dewei Zhu.

• Scripts/git-webkit: Added.
• Scripts/libraries/webkitcorepy/webkitcorepy/init.py: Bump version.
• Scripts/libraries/webkitcorepy/webkitcorepy/arguments.py:

(NoAction.call):
(CountAction): Create an action to increment or decrement a variable by a value.
(CallbackAction): Create an action which will trigger a callback when set.
(LoggingGroup): Create a group which allows the user to change logging behavior.

• Scripts/libraries/webkitscmpy/git-webkit: Added.
• Scripts/libraries/webkitscmpy/setup.py: Add webkitscmpy.program and git-webkit.
• Scripts/libraries/webkitscmpy/webkitscmpy/init.py: Bump version.
• Scripts/libraries/webkitscmpy/webkitscmpy/commit.py:

(Commit): Make the 12 character hash based on a class variable.

• Scripts/libraries/webkitscmpy/webkitscmpy/local/git.py:

(Git.default_branch): Fallback to 'master' or 'main' if the remote isn't available.
(Git.find): Use git to match branches and tags to a hash instead of trying to do it ourselves.

• Scripts/libraries/webkitscmpy/webkitscmpy/local/scm.py:

(Scm.find): Add function to convert generic string arguments to a commit, and then find that commit in
the repository.

• Scripts/libraries/webkitscmpy/webkitscmpy/local/svn.py:

(Svn): Support a few different SVN binary paths.

• Scripts/libraries/webkitscmpy/webkitscmpy/mocks/local/git.py: Add rev-parse branch/tag/HEAD -> hash conversion.
• Scripts/libraries/webkitscmpy/webkitscmpy/program.py: Added.

(Command): Base class for all commands.
(Find):
(Find.parser): Configure parser for 'find' sub-program.
(Find.main): Given an identifier, revision, hash or branch name, find and print the matching commit from
(main): Entry point to git-<project> scripts.

• Scripts/libraries/webkitscmpy/webkitscmpy/test/find_unittest.py: Added.

(TestFind):
(TestFind.test_basic_git):
(TestFind.test_basic_git_svn):
(TestFind.test_basic_svn):
(TestFind.test_branch_tilde):
(TestFind.test_identifier_git):
(TestFind.test_identifier_git_svn):
(TestFind.test_identifier_svn):
(TestFind.test_hash):
(TestFind.test_revision_svn):
(TestFind.test_revision_git_svn):
(TestFind.test_standard):

4:28 PM Changeset in webkit [268432] by commit-queue@webkit.org

• 3 edits
  2 adds in trunk

Null dereference in PDFPlugin::snapshot()
​https://bugs.webkit.org/show_bug.cgi?id=217668

Patch by Julian Gonzalez <​julian_a_gonzalez@apple.com> on 2020-10-13
Reviewed by Ryosuke Niwa.

Source/WebKit:

Add a null pointer check in PDFPlugin's snapshot method
to prevent a crash if the backing store's size
isn't sufficient for the ShareableBitmap.

Test: plugins/pdf-plugin-null-onloaddeddata.html

• WebProcess/Plugins/PDF/PDFPlugin.mm:

(WebKit::PDFPlugin::snapshot):

LayoutTests:

Add new pass-if-no-crash test to catch crash in PDFPlugin
snapshotting.

• plugins/pdf-plugin-null-onloaddeddata-expected.txt: Added.
• plugins/pdf-plugin-null-onloaddeddata.html: Added.

4:20 PM Changeset in webkit [268431] by rniwa@webkit.org

• 13 edits
  2 adds in trunk

IPC testing JS API should expose a reply and describe the list of arguments for each message
​https://bugs.webkit.org/show_bug.cgi?id=217565

Reviewed by Geoffrey Garen.

Source/WebKit:

This patch makes IPC.sendMessage and IPC.sendSyncMessage decode the reply. IPC.sendSyncMessage now returns
a dictionary with two keys: "buffer" and "arguments", the first of which returns ArrayBuffer of the decoded
message reply and the second of which is an array of decoded arguments where there was an appropriate JS
binding code existed for the argument type. IPC.sendMessage now returns a Promise which can either resolve
with the same dictionary as IPC.sendSyncMessage or reject when the decoding fails.

In addition, this patch exposes a dictionary describing each IPC message argument's type and parameter name.

In order to add these two functionalities, this patch adds a new step in generate-message-receiver.py to
generate MessageArgumentDescriptions.cpp, which contains functions which know how to decode arguments of
any IPC message and create a JS dictionary describing it as well as functions that return descriptions of
arguments or reply arguments.

Finally, this patch adds encoders for a few types found to be very common after r268239 had been landed.

Tests: TestWebKitAPI.IPCTestingAPI.DecodesReplyArgumentsForPrompt

TestWebKitAPI.IPCTestingAPI.DecodesReplyArgumentsForAsyncMessage
TestWebKitAPI.IPCTestingAPI.DescribesArguments

• CMakeLists.txt:
• DerivedSources-output.xcfilelist:
• DerivedSources.make:
• Platform/IPC/Decoder.h:

(IPC::Decoder::buffer const):

• Platform/IPC/JSIPCBinding.h: Added.

(jsValueForDecodedArgumentValue): Added. A template function to construct a JS value for a given C++ value.
(jsValueForDecodedNumericArgumentValue): Added. A helper for constructing a JS value for numeric values.
(jsValueForDecodedArgumentRect): Added. Ditto for IntRect and FloatRect.
(DecodedArgumentJSValueConverter): Added. A helper class to construct JS values for a tuple of values using
partial template specializations.
(DecodedArgumentJSValueConverter::convert): Added.
(jsValueForArgumentTuple): Added. A helper to construct a JS array for the decoded IPC arguments.
(jsValueForDecodedArguments): Added.

• Platform/IPC/MessageArgumentDescriptions.h: Added.

(IPC::ArgumentDescription): Added.

• Scripts/generate-message-receiver.py:

(main): Generate MessageArgumentDescriptions.cpp.

• Scripts/webkit/messages.py:

(headers_for_type): Removed the special case for PaymentMethodUpdate now that it's in its own header. Also
added made webrtc::WebKitEncodedFrameInfo include LibWebRTCEnumTraits.h as it uses webrtc::VideoFrameType.
(collect_header_conditions_for_receiver): Extracted from generate_message_handler.
(generate_header_includes_from_conditions): Ditto.
(generate_message_handler):
(generate_js_value_conversion_function): Added.
(generate_js_argument_descriptions): Added.
(generate_message_argument_description_implementation): Added.

• Shared/ApplePay/ApplePayPaymentSetupFeaturesWebKit.h: Fixed a bug that we were not forward declaring NSArray.
• SourcesCocoa.txt: Added MessageArgumentDescriptions.cpp as a non-unified cpp file as its size is around 1MB.
• WebKit.xcodeproj/project.pbxproj:
• WebProcess/WebPage/IPCTestingAPI.cpp:

(WebKit::IPCTestingAPI::JSIPC::staticValues): Added the session ID and page ID as static variables.
(WebKit::IPCTestingAPI::encodePointType): Added.
(WebKit::IPCTestingAPI::encodeRectType): Fixed the bug was that this code wasn't checking for any exceptions.
(WebKit::IPCTestingAPI::encodeNumericType): Renamed from encodeIntegralType since this function is now used
to encode double and float, not just integral types.
(WebKit::IPCTestingAPI::encodeArgument): Added the support for IntPoint, FloatPoint, URL, RegistrableDomain,
double, and float all of which turned out to be in the top 20 most common types.
(WebKit::IPCTestingAPI::jsResultFromReplyDecoder): Added.
(WebKit::IPCTestingAPI::JSIPC::sendMessage): Added the code to return Promise when there is a reply and resolve
it with the JS object describing the decoded argument. We use messageReplyArgumentDescriptions to figure out
whether there is a reply or not.
(WebKit::IPCTestingAPI::JSIPC::sendSyncMessage): Decode the reply and return a JS object which describes it.
(WebKit::IPCTestingAPI::JSIPC::frameID): Renamed from frameIdentifier to be consistent.
(WebKit::IPCTestingAPI::JSIPC::pageID): Added.
(WebKit::IPCTestingAPI::JSIPC::sessionID): Added.
(WebKit::IPCTestingAPI::createJSArrayForArgumentDescriptions): Added.
(WebKit::IPCTestingAPI::JSIPC::messages): Added the code to generate descriptions for arguments.

Tools:

Added tests for decoding replies for sync and async messages and one for argument descriptions.

• TestWebKitAPI/Tests/WebKitCocoa/IPCTestingAPI.mm:

(-[IPCTestingAPIDelegate webView:runJavaScriptTextInputPanelWithPrompt:defaultText:initiatedByFrame:completionHandler:]):
(-[IPCTestingAPIDelegate _webView:webContentProcessDidTerminateWithReason:]):
(IPCTestingAPI.DecodesReplyArgumentsForPrompt):
(IPCTestingAPI.DecodesReplyArgumentsForAsyncMessage):
(IPCTestingAPI.DescribesArguments):

4:01 PM Changeset in webkit [268430] by Alan Coon

• 1 copy in tags/Safari-611.1.3.2

Tag Safari-611.1.3.2.

3:49 PM Changeset in webkit [268429] by aakash_jain@apple.com

• 5 edits in trunk/Tools

[build.webkit.org] Rename slave to worker in config.json
​https://bugs.webkit.org/show_bug.cgi?id=217672

Reviewed by Dewei Zhu.

• BuildSlaveSupport/build.webkit.org-config/config.json: Renamed slaves to workers and slavenames to workernames.

Also deleted unused entry for test-slave.

• BuildSlaveSupport/build.webkit.org-config/loadConfig.py: Renamed slave to worker.
• BuildSlaveSupport/build.webkit.org-config/loadConfig_unittest.py: Ditto.
• BuildSlaveSupport/build.webkit.org-config/make_passwords_json.py: Ditto.

(create_mock_worker_passwords_dict):

3:49 PM Changeset in webkit [268428] by commit-queue@webkit.org

• 7 edits in trunk

Expose a property on WKWebView to know if it's being inspected.
​https://bugs.webkit.org/show_bug.cgi?id=217629
rdar://70073369

Patch by Ellie Epskamp-Hunt <​eepskamphunt@apple.com> on 2020-10-13
Reviewed by Brian Burg.

Source/WebKit:

Move an existing property from WKWebViewPrivateForTesting to WKWebViewPrivate so Safari can
know when a web view is being inspected.

• UIProcess/API/Cocoa/WKWebView.mm:

(-[WKWebView _isBeingInspected]):

• UIProcess/API/Cocoa/WKWebViewPrivate.h:
• UIProcess/API/Cocoa/WKWebViewPrivateForTesting.h:
• UIProcess/API/Cocoa/WKWebViewTesting.mm:

(-[WKWebView _hasInspectorFrontend]): Deleted.

Tools:

Update the property name in existing tests.

• TestWebKitAPI/Tests/WebKitCocoa/WKInspectorDelegate.mm:

(TEST):

3:46 PM Changeset in webkit [268427] by Devin Rousso

• 2 edits in trunk/Source/WebInspectorUI

Web Inspector: Sources: tab has wrong icon when paused if added after pause
​https://bugs.webkit.org/show_bug.cgi?id=217671

Reviewed by Brian Burg.

• UserInterface/Views/SourcesTabContentView.js:

(WI.SourcesTabContentView.tabInfo):

3:46 PM Changeset in webkit [268426] by Chris Dumez

• 2 edits in trunk/Source/WebCore

Simplify AudioNode::checkNumberOfChannelsForInput()
​https://bugs.webkit.org/show_bug.cgi?id=217674

Reviewed by Geoffrey Garen.

Simplify AudioNode::checkNumberOfChannelsForInput() by simply calling
updateInternalBus() on the input in release. We only iterate m_inputs
in debug to make sure this is an input associated with this node.

• Modules/webaudio/AudioNode.cpp:

(WebCore::AudioNode::checkNumberOfChannelsForInput):

3:39 PM Changeset in webkit [268425] by Russell Epstein

• 1 copy in tags/Safari-610.3.3

Tag Safari-610.3.3.

3:16 PM Changeset in webkit [268424] by Chris Dumez

• 4 edits in trunk

It should be possible to create an AudioWorkletNode for a context that has finished rendering
​https://bugs.webkit.org/show_bug.cgi?id=217666

Reviewed by Eric Carlson.

LayoutTests/imported/w3c:

Rebaseline WPT test that is now passing.

• web-platform-tests/webaudio/the-audio-api/the-audioworklet-interface/suspended-context-messageport.https-expected.txt:

Source/WebCore:

It should be possible to create an AudioWorkletNode for a context that has finished rendering.
We were previously throwing an exception when trying to create an AudioWorkletNode for a context
whose state is "closed" (OfflineAudioContext's state becomes closed as soon as rendering
finishes). This was causing us to fail a WPT test and did not match Blink's behavior.

No new tests, rebaselined existing test.

• Modules/webaudio/AudioWorkletNode.cpp:

(WebCore::AudioWorkletNode::create):

2:53 PM Changeset in webkit [268423] by Chris Dumez

• 21 edits in trunk

WebAudio tests are crashing in debug when enabling the GPU process
​https://bugs.webkit.org/show_bug.cgi?id=217663

Reviewed by Geoff Garen.

Source/WebCore:

No new tests, unskipped existing tests.

• WebCore.xcodeproj/project.pbxproj:
• platform/audio/PushPullFIFO.h:

Export PushPullFIFO so that it can be used at WebKit layer.

Source/WebKit:

WebAudio tests were crashing in debug when enabling the GPU process because it did audio
processing on the WebContent process's main thread. To address the issue, I made
RemoteAudioDestinationProxy a ThreadMessageReceiver so that it receives IPC on an audio
thread instead of the main thread. IPC messages are processed directly on the AudioWorklet
thread when active or on an audio thread constructed by RemoteAudioDestinationProxy
otherwise.

• GPUProcess/media/RemoteAudioDestinationManager.cpp:

(WebKit::RemoteAudioDestination::RemoteAudioDestination):
Use a PushPullFIFO structure in render() to avoid hanging the audio rendering thread
on a semaphore. Hanging the rendering thread was terrible for performance and was also
a source of deadlock since the underlying framework is holding a lock while render() is
called. We could process a RemoteAudioDestination::CreateAudioDestination sync IPC on the
main thread and deadlock on that lock.

• GPUProcess/webrtc/LibWebRTCCodecsProxy.h:
• GPUProcess/webrtc/RemoteAudioMediaStreamTrackRendererManager.h:
• GPUProcess/webrtc/RemoteSampleBufferDisplayLayerManager.h:
• NetworkProcess/IndexedDB/WebIDBServer.h:
• NetworkProcess/webrtc/NetworkRTCProvider.h:
• WebProcess/Network/webrtc/LibWebRTCNetwork.h:
• WebProcess/cocoa/RemoteCaptureSampleManager.h:

Use ThreadMessageReceiverRefCounted instead of ThreadMessageReceiver since those classes
do not provide their own RefCounting.

• Platform/IPC/Connection.cpp:

(IPC::Connection::addWorkQueueMessageReceiver):
(IPC::Connection::removeWorkQueueMessageReceiver):
(IPC::Connection::addThreadMessageReceiver):
(IPC::Connection::removeThreadMessageReceiver):
(IPC::Connection::processIncomingMessage):
(IPC::Connection::dispatchMessageToWorkQueueReceiver):
(IPC::Connection::dispatchMessageToThreadReceiver):

• Platform/IPC/Connection.h:

(IPC::Connection::ThreadMessageReceiver::ref):
(IPC::Connection::ThreadMessageReceiver::deref):

• Add support for passing a destinationID when registering a WorkQueueMessageReceiver or a ThreadMessageReceiver, similarly to regular MessageReceivers. This was needed here since The GPUProcess sends IPC messages to the RemoteAudioDestinationProxy with a given destinationID and since RemoteAudioDestinationProxy is now a ThreadMessageReceiver.
• Stop having ThreadMessageReceiver subclass ThreadSafeRefCounted since RemoteAudioDestinationProxy already subclasses ThreadSafeRefCounted indirectly. A new ThreadMessageReceiverRefCounted class was added for convenience for existing code that relied on its refcounting.

• WebProcess/GPU/media/RemoteAudioDestinationProxy.cpp:

(WebKit::RemoteAudioDestinationProxy::RemoteAudioDestinationProxy):
(WebKit::RemoteAudioDestinationProxy::~RemoteAudioDestinationProxy):
(WebKit::RemoteAudioDestinationProxy::start):
(WebKit::RemoteAudioDestinationProxy::stop):
(WebKit::RemoteAudioDestinationProxy::renderBuffer):
(WebKit::RemoteAudioDestinationProxy::didChangeIsPlaying):
(WebKit::RemoteAudioDestinationProxy::dispatchToThread):

• WebProcess/GPU/media/RemoteAudioDestinationProxy.h:

Use a PushPullFIFO container in render() to avoid handing the audio rendering thread on
a semaphore while the Render IPC is getting processed by the WebProcess.

Source/WTF:

• wtf/CrossThreadQueue.h:

(WTF::CrossThreadQueue<DataType>::waitForMessage):
If CrossThreadQueue::kill() gets called while another thread is waiting on a
CrossThreadQueue::waitForMessage() call, make it so that waitForMessage()
returns a default-constructed DataType instead of crashing trying to
dequeue (since the queue is empty).

LayoutTests:

Unskip webaudio tests when the GPU process is enabled.

• gpu-process/TestExpectations:

2:41 PM Changeset in webkit [268422] by Karl Rackler

• 2 edits in trunk/LayoutTests

REGRESSION (iOS 14): [ iOS wk2 ] service-workers-spinning tests are flaky failures
​https://bugs.webkit.org/show_bug.cgi?id=217669

Unreviewed test gardening.

• platform/ios-simulator-wk2/TestExpectations:

2:38 PM Changeset in webkit [268421] by pvollan@apple.com

• 2 edits in trunk/Source/WebKit

[iOS 14] Hang in -[WKUserDefaults _notifyObserversOfChangeFromValuesForKeys:toValuesForKeys:]
​https://bugs.webkit.org/show_bug.cgi?id=217183
<rdar://problem/69916673>

Reviewed by Geoffrey Garen.

Dispatch this work on a non-main thread, in order to avoid blocking the main thread.

• UIProcess/Cocoa/PreferenceObserver.mm:

(-[WKUserDefaults _notifyObserversOfChangeFromValuesForKeys:toValuesForKeys:]):

2:25 PM Changeset in webkit [268420] by Keith Rollin

• 28 edits in trunk

Remove leftover MACOSX_DEPLOYMENT_TARGET_macosx support
​https://bugs.webkit.org/show_bug.cgi?id=217649
<rdar://problem/70236877>

Reviewed by Darin Adler.

Bug 42796 introduced MACOSX_DEPLOYMENT_TARGET_<PLATFORM> as "support
for compiling WebKit against iOS SDKs". Support for the iOS part of
this feature was later removed in several changes, including Bug
139212, Bug 139463 and Bug 144762. However, vestiges have remained for
five or six years in the form of MACOSX_DEPLOYMENT_TARGET_macosx. The
inclusion of the platform in MACOSX_DEPLOYMENT_TARGET is no longer
needed and can be removed.

This changes brings most projects in conformance with other projects
that don't support including the platform in MACOSX_DEPLOYMENT_TARGET,
including WebEditingTester, gtest, WebKitTestRunner, MiniBrowser, and
TestWebKitAPI.

Along the way, remove a couple of left-over references to macOS 10.16,
and a couple of places where [sdk=macosx*] was still being used.

With this change, initialization of MACOSX_DEPLOYMENT_TARGET should be
consistent across all projects, with two exceptions: WebKitLauncher
(which hardcodes it to 10.12) and libwebrtc's copy of googletest
(which hardcodes it to 10.4). The reasons for these hard-coded values
is not apparent, so leave them be.

PerformanceTests:

• DecoderTest/Configurations/DebugRelease.xcconfig:
• MediaTime/Configurations/DebugRelease.xcconfig:

Source/bmalloc:

• Configurations/DebugRelease.xcconfig:

Source/JavaScriptCore:

• Configurations/DebugRelease.xcconfig:

Source/ThirdParty/ANGLE:

• Configurations/DebugRelease.xcconfig:

Source/ThirdParty/libwebrtc:

• Configurations/DebugRelease.xcconfig:

Source/WebCore:

No new tests -- no changed functionality.

• Configurations/DebugRelease.xcconfig:

Source/WebCore/PAL:

• Configurations/DebugRelease.xcconfig:

Source/WebInspectorUI:

• Configurations/DebugRelease.xcconfig:

Source/WebKit:

• Configurations/DebugRelease.xcconfig:

Source/WebKitLegacy/mac:

• Configurations/DebugRelease.xcconfig:

Source/WTF:

• Configurations/DebugRelease.xcconfig:

Tools:

• ContentExtensionTester/Configurations/DebugRelease.xcconfig:
• DumpRenderTree/mac/Configurations/DebugRelease.xcconfig:
• ImageDiff/cg/Configurations/DebugRelease.xcconfig:
• lldb/lldbWebKitTester/Configurations/DebugRelease.xcconfig:

2:09 PM Changeset in webkit [268419] by Russell Epstein

• 2 edits in branches/safari-610-branch/Source/WebKit

Cherry-pick r268381. rdar://problem/70260409

Fix the macCatalyst build ("reference to 'Rect' is ambiguous")
​https://bugs.webkit.org/show_bug.cgi?id=217642
<rdar://problem/70224629>

Reviewed by Wenson Hsieh.

• UIProcess/ios/forms/WKDateTimeInputControl.mm: (-[WKDateTimePicker initWithView:datePickerMode:]): (-[WKDateTimePicker shouldPresentGregorianCalendar:]): (-[WKDateTimePicker controlBeginEditing]): (-[WKDateTimeInputControl initWithView:]): An earlier unified source's using namespace WebCore inside namespace WebKit and then WKDateTimeInputControl's using namespace WebKit in the global namespace resulted in once-again conflicting definitions of Rect between CSS code and MacTypes.h.

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268381 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2:06 PM Changeset in webkit [268418] by Russell Epstein

• 8 edits in branches/safari-610-branch/Source

Versioning.

WebKit-7610.3.3

1:53 PM Changeset in webkit [268417] by Simon Fraser

• 19 edits in trunk/Source

Plumb OptionSet<WheelEventProcessingSteps> through the wheel event handling code
​https://bugs.webkit.org/show_bug.cgi?id=217660

Reviewed by Tim Horton.

Source/WebCore:

Async scrolling in passive wheel event regions will require that the main thread event
handling code knows that the scrolling thread did the scrolling. To propagate this
information, plumb OptionSet<WheelEventProcessingSteps> through.

No behavior change.

• page/EventHandler.cpp:

(WebCore::EventHandler::handleWheelEvent):
(WebCore::EventHandler::passWheelEventToWidget):

• page/EventHandler.h:
• page/ios/EventHandlerIOS.mm:

(WebCore::EventHandler::wheelEvent):
(WebCore::EventHandler::passWheelEventToWidget):

• page/mac/EventHandlerMac.mm:

(WebCore::EventHandler::wheelEvent):
(WebCore::EventHandler::passWheelEventToWidget):

• page/scrolling/ScrollingTree.cpp:

(WebCore::ScrollingTree::determineWheelEventProcessing):
(WebCore::ScrollingTree::handleWheelEvent):

• page/scrolling/ScrollingTree.h:

(WebCore::WheelEventHandlingResult::handled):
(WebCore::WheelEventHandlingResult::unhandled):
(WebCore::ScrollingTree::handleWheelEvent):

• page/scrolling/ThreadedScrollingTree.cpp:

(WebCore::ThreadedScrollingTree::handleWheelEvent):

• page/scrolling/ThreadedScrollingTree.h:
• platform/PlatformWheelEvent.cpp:

(WebCore::operator<<):

• replay/UserInputBridge.cpp:

(WebCore::UserInputBridge::handleWheelEvent):

• replay/UserInputBridge.h:

Source/WebKit:

Async scrolling in passive wheel event regions will require that the main thread event
handling code knows that the scrolling thread did the scrolling. To propagate this
information, plumb OptionSet<WheelEventProcessingSteps> through.

No behavior change.

• WebProcess/WebPage/EventDispatcher.cpp:

(WebKit::EventDispatcher::wheelEvent):
(WebKit::EventDispatcher::dispatchWheelEventViaMainThread):
(WebKit::EventDispatcher::dispatchWheelEvent):

• WebProcess/WebPage/EventDispatcher.h:
• WebProcess/WebPage/WebPage.cpp:

(WebKit::handleWheelEvent):
(WebKit::WebPage::wheelEvent):

• WebProcess/WebPage/WebPage.h:

Source/WebKitLegacy/win:

• WebView.cpp:

(WebView::mouseWheel):

1:15 PM Changeset in webkit [268416] by Alan Coon

• 4 edits in branches/safari-610.2.11.1-branch

Revert r268206. rdar://problem/70258921

1:06 PM Changeset in webkit [268415] by Fujii Hironori

• 5 edits in trunk

[WinCairo][MediaFoundation] Many test cases are timing out by waiting for events
​https://bugs.webkit.org/show_bug.cgi?id=217463

Reviewed by Don Olmstead.

Source/WebCore:

The ended event wasn't dispatched because MediaFoundation winds
back currentTime to 0 at the end of play back. The currentTime
should keep the position. Added m_sessionEnded to return the
duration in currentTime() if m_sessionEnded is true.

currentTime() didn't work for audio elements because m_presenter is
available only for videos. Use IMFMediaSession::GetClock instead.

seek() started play back even if it is paused. Added m_seeking to
remember it is seeking and pause the session in onSessionStarted if
m_seeking and m_paused are true.

Update NetworkState properly. Return NetworkState::FormatError on
MF_E_TOPO_CODEC_NOT_FOUND, NetworkState::DecodeError on MEError
event.

MEError event with MF_E_NOT_INITIALIZED status was dispatch if
SetVideoPosition is called while the session is not started. Deffer
calling SetVideoPosition to onSessionStarted.

• platform/graphics/win/MediaPlayerPrivateMediaFoundation.cpp:

(WebCore::MediaPlayerPrivateMediaFoundation::MediaPlayerPrivateMediaFoundation):
(WebCore::MediaPlayerPrivateMediaFoundation::play):
(WebCore::MediaPlayerPrivateMediaFoundation::seeking const):
(WebCore::MediaPlayerPrivateMediaFoundation::seek):
(WebCore::MediaPlayerPrivateMediaFoundation::currentTime const):
(WebCore::MediaPlayerPrivateMediaFoundation::buffered const):
(WebCore::MediaPlayerPrivateMediaFoundation::setSize):
(WebCore::MediaPlayerPrivateMediaFoundation::endCreatedMediaSource):
(WebCore::MediaPlayerPrivateMediaFoundation::endGetEvent):
(WebCore::MediaPlayerPrivateMediaFoundation::updateReadyState):
(WebCore::MediaPlayerPrivateMediaFoundation::onTopologySet):
(WebCore::MediaPlayerPrivateMediaFoundation::onSessionStarted):
(WebCore::MediaPlayerPrivateMediaFoundation::onSessionEnded):
(WebCore::MediaPlayerPrivateMediaFoundation::CustomVideoPresenter::currentTime): Deleted.

• platform/graphics/win/MediaPlayerPrivateMediaFoundation.h:

LayoutTests:

• platform/wincairo/TestExpectations: Unskip media tests. Marked

failing tests.

1:02 PM Changeset in webkit [268414] by Chris Dumez

• 22 edits in trunk

'parameters' argument to AudioWorkletProcessor.process() is not spec-compliant
​https://bugs.webkit.org/show_bug.cgi?id=217637

Reviewed by Darin Adler.

LayoutTests/imported/w3c:

Rebaseline WPT tests that are now passing.

• web-platform-tests/webaudio/the-audio-api/the-audioparam-interface/k-rate-audioworklet-connections.https-expected.txt:
• web-platform-tests/webaudio/the-audio-api/the-audioparam-interface/k-rate-audioworklet.https-expected.txt:
• web-platform-tests/webaudio/the-audio-api/the-audioworklet-interface/audioworklet-audioparam-size.https-expected.txt:
• web-platform-tests/webaudio/the-audio-api/the-audioworklet-interface/audioworklet-audioparam.https-expected.txt:
• web-platform-tests/webaudio/the-audio-api/the-audioworklet-interface/audioworkletnode-channel-count.https-expected.txt:

Source/WebCore:

'parameters' argument to AudioWorkletProcessor.process() is not spec-compliant:

• ​https://www.w3.org/TR/webaudio/#dom-audioworkletprocessor-process-inputs-outputs-parameters-parameters

Its type should not be a JS Map, as evidenced by how WPT tests use it. We now pass a generic JS Object,
as Blink is doing.

Also, as per the specification, if no automation is scheduled during the render quantum, the array may
have length 1 with the array element being the constant value of the AudioParam for the render quantum.

No new tests, rebaselined existing tests.

• Modules/webaudio/AudioParam.cpp:

(WebCore::AudioParam::hasSampleAccurateValues):

• Modules/webaudio/AudioParam.h:
• Modules/webaudio/AudioParamTimeline.cpp:

(WebCore::AudioParamTimeline::hasValues const):

• Modules/webaudio/AudioParamTimeline.h:

This is an optimization to avoid calculating sample accurate values if the first timeline event is
still in the future. At some point, we should consider improving the logic to do so when all events
are in the past, similarly to what Blink is doing.

• Modules/webaudio/AudioWorkletNode.cpp:

(WebCore::AudioWorkletNode::initializeAudioParameters):
Do not initialize memory for the AudioFloatArrays of each AudioParam yet since we won't know until
process() is called if size will be 1 or renderQuantumSize and since the size can change dynamically.

(WebCore::AudioWorkletNode::process):
As per the specification, if there is no timeline event, we pass an array that has length 1 and whose
element is the constant value.

• Modules/webaudio/AudioWorkletProcessor.cpp:

(WebCore::constructFrozenKeyValueObject):
(WebCore::AudioWorkletProcessor::process):
Instead of passing the parameters as a JS map, we now use a simple frozen object. We also make sure
the Float32Array we pass JS has length 1 if the input array constains a constant value. This behavior
is consistent with what Blink is doing.

• Modules/webaudio/DelayDSPKernel.cpp:

(WebCore::DelayDSPKernel::DelayDSPKernel):

• Modules/webaudio/RealtimeAnalyser.cpp:

(WebCore::RealtimeAnalyser::setFftSize):

• platform/audio/AudioArray.h:

(WebCore::AudioArray::AudioArray):
(WebCore::AudioArray::resize):
(WebCore::AudioArray::at):
(WebCore::AudioArray::zero):
(WebCore::AudioArray::zeroRange):
(WebCore::AudioArray::copyToRange):
(WebCore::AudioArray::containsConstantValue const):

• platform/audio/Biquad.cpp:

(WebCore::Biquad::Biquad):

• platform/audio/ReverbConvolverStage.cpp:

(WebCore::ReverbConvolverStage::ReverbConvolverStage):

• platform/audio/ios/AudioFileReaderIOS.cpp:

(WebCore::AudioFileReader::createBus):

• platform/audio/mac/AudioFileReaderMac.cpp:

(WebCore::AudioFileReader::createBus):
Rename AudioArray::allocate() to AudioArray::resize() for clarity since it can be called multiple times
after construction of the AudioArray. Make it a no-op if the new size is the same as the old one.
Also add a utility function to check if the array constains a constant value.

12:35 PM Changeset in webkit [268413] by Alan Coon

• 4 edits in branches/safari-610.2-branch/Source/WebCore/dom

Cherry-pick f9437de2cd47. rdar://problem/69101091

Fix the build after cherry-picking r266909.

12:35 PM Changeset in webkit [268412] by Alan Coon

• 8 edits
  2 adds in branches/safari-610.2-branch

Cherry-pick r266909. rdar://problem/69101091

Source/WebCore:
Text replacements at the beginning of a second line are replaced too early
​https://bugs.webkit.org/show_bug.cgi?id=216327
<rdar://problem/68170353>

Reviewed by Darin Adler.

In the changes in r258871, using SimpleRanges instead of Range causing some side effects
when the replacements at the beginning of lines. The ranges that we are counting are backwards
and the return characters are being counted instead of being ignored. There is almost
certainly a better fix than this, but this patch restores the original logic that
was present when Range was being used, until a better fix can be worked out.

Test: editing/spelling/text-replacement-first-word-second-line.html

• editing/Editor.cpp: (WebCore::Editor::markAndReplaceFor):
• editing/TextCheckingHelper.cpp: (WebCore::TextCheckingParagraph::automaticReplacementStart const): (WebCore::TextCheckingParagraph::automaticReplacementLength const):
• editing/TextCheckingHelper.h:

LayoutTests:
Overlapping text replacements at the beginning of a line are replaced too early
​https://bugs.webkit.org/show_bug.cgi?id=216327

Reviewed by Darin Adler.

• editing/spelling/text-replacement-first-word-second-line-expected.txt: Added.
• editing/spelling/text-replacement-first-word-second-line.html: Added.

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@266909 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12:32 PM Changeset in webkit [268411] by Alan Coon

• 2 edits in branches/safari-611.1.3-branch/Source/WebKit

Cherry-pick r268401. rdar://problem/70259990

Allow legacy custom scheme registration on non-main threads
​https://bugs.webkit.org/show_bug.cgi?id=217656
<rdar://problem/70228036>

Patch by Alex Christensen <​achristensen@webkit.org> on 2020-10-13
Reviewed by Brady Eidson.

r267802 was quite necessary, but it exposed an internal client calling our API on a background thread,
which crashes now. Make it not crash. There is a race condition in their registration, but there already was.

• UIProcess/API/Cocoa/WKBrowsingContextController.mm: (+[WKBrowsingContextController registerSchemeForCustomProtocol:]): (+[WKBrowsingContextController unregisterSchemeForCustomProtocol:]):

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268401 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12:27 PM Changeset in webkit [268410] by Alan Coon

• 8 edits in branches/safari-611.1.3-branch/Source

Versioning.

WebKit-7611.1.3.2

12:22 PM Changeset in webkit [268409] by Alan Coon

• 5 edits in branches/safari-610.2.11.1-branch/Source/WebCore

Cherry-pick r268308. rdar://problem/70259013

[Mac] AirPlay menu does not show up when AirPlay button is clicked
​https://bugs.webkit.org/show_bug.cgi?id=217536
<rdar://problem/69973777>

Reviewed by Eric Carlson.

Source/WebCore:

PAL tries to soft-link the AVRoutePickerView from the AVFoundation framework, rather than the (correct)
AVKit framework. Remove AVRoutePickerView from AVFoundationSoftLink.h and use the already declared version
inside AVRoutePickerViewTargetPicker.mm.

• platform/graphics/avfoundation/objc/AVRoutePickerViewTargetPicker.mm: (WebCore::AVRoutePickerViewTargetPicker::isAvailable):

Source/WebCore/PAL:

• pal/cocoa/AVFoundationSoftLink.h:
• pal/cocoa/AVFoundationSoftLink.mm:

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268308 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12:21 PM Changeset in webkit [268408] by Alan Coon

• 4 edits in branches/safari-610.2.11.1-branch

Cherry-pick r268206. rdar://problem/70258921

Presidential Executive Order pages not accessible with Safari.
​https://bugs.webkit.org/show_bug.cgi?id=217415
<rdar://problem/69922416>

Reviewed by Chris Fleizach and Simon Fraser.

Follow up to the previous change for this bug per Simon Fraser's comment.
Check not only the visibility and opacity of the object in question but
also of its ancestors.
Expanded the test accessibility/aria-modal.html to include the case
where the visibility of the modal dialog is determined by the opacity of
its parent object.

• accessibility/AXObjectCache.cpp: (WebCore::AXObjectCache::isNodeVisible const):

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268206 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12:19 PM Changeset in webkit [268407] by Alan Coon

• 8 edits in branches/safari-610.2.11.1-branch/Source

Versioning.

WebKit-7610.2.11.1.2

12:12 PM Changeset in webkit [268406] by Alan Coon

• 4 edits in branches/safari-610.2.11.0-branch

Cherry-pick r268206. rdar://problem/70258965

Presidential Executive Order pages not accessible with Safari.
​https://bugs.webkit.org/show_bug.cgi?id=217415
<rdar://problem/69922416>

Reviewed by Chris Fleizach and Simon Fraser.

Follow up to the previous change for this bug per Simon Fraser's comment.
Check not only the visibility and opacity of the object in question but
also of its ancestors.
Expanded the test accessibility/aria-modal.html to include the case
where the visibility of the modal dialog is determined by the opacity of
its parent object.

• accessibility/AXObjectCache.cpp: (WebCore::AXObjectCache::isNodeVisible const):

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@268206 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12:11 PM Changeset in webkit [268405] by Truitt Savell

• 2 edits in trunk/LayoutTests

REGRESSION: [ Mac ] inspector/audit/basic-async.htm is flaky timeout
​https://bugs.webkit.org/show_bug.cgi?id=217620

Unreviewed test gardening.

• platform/mac/TestExpectations:

12:08 PM Changeset in webkit [268404] by Alan Coon

• 8 edits in branches/safari-610.2.11.0-branch/Source

Versioning.

WebKit-7610.2.11.0.2

12:02 PM Changeset in webkit [268403] by Truitt Savell

• 2 edits in trunk/LayoutTests

REGRESSION: [ Mac iOS ] fast/css-custom-paint/out-of-memory-while-adding-worklet-module.html is flaky timeout
​https://bugs.webkit.org/show_bug.cgi?id=217618

Unreviewed test Gardening.

• TestExpectations:

11:43 AM Changeset in webkit [268402] by Lauro Moura

• 3 edits in trunk/Tools

webkitpy: Log and report webdriver harness errors
​https://bugs.webkit.org/show_bug.cgi?id=217659

Reviewed by Jonathan Bedard.

• Scripts/webkitpy/webdriver_tests/webdriver_test_runner.py:

(WebDriverTestRunner.process_results): Report failed harness tests,
not just subtests.

• Scripts/webkitpy/webdriver_tests/webdriver_test_runner_w3c.py:

(WebDriverTestRunnerW3C.run): Print to error log when a test run
fails, not just subtests.

11:42 AM Changeset in webkit [268401] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebKit

Allow legacy custom scheme registration on non-main threads
​https://bugs.webkit.org/show_bug.cgi?id=217656
<rdar://problem/70228036>

Patch by Alex Christensen <​achristensen@webkit.org> on 2020-10-13
Reviewed by Brady Eidson.

r267802 was quite necessary, but it exposed an internal client calling our API on a background thread,
which crashes now. Make it not crash. There is a race condition in their registration, but there already was.

• UIProcess/API/Cocoa/WKBrowsingContextController.mm:

(+[WKBrowsingContextController registerSchemeForCustomProtocol:]):
(+[WKBrowsingContextController unregisterSchemeForCustomProtocol:]):

10:16 AM Changeset in webkit [268400] by weinig@apple.com

• 525 edits in trunk

[Testing] Rename test header commands to match names in WebPreferences*.yaml
​https://bugs.webkit.org/show_bug.cgi?id=217654

Reviewed by Simon Fraser.

Tools:

Renames all the test header commands (e.g. useAcceleratedDrawing) to match name in
WebPreferences*.yaml files (e.g. AcceleratedDrawingEnabled).

Also splits up TestFeatures into test runner specific features (e.g. runSingly) that
don't have a WebPreferences counterpart, and WebPreferences specific features. This
will allow future patches to support arbritrary WebPreferences more easily.

• DumpRenderTree/TestOptions.cpp:
• DumpRenderTree/TestOptions.h:
• DumpRenderTree/mac/DumpRenderTree.mm:

(setWebPreferencesForTestOptions):

• TestRunnerShared/TestFeatures.cpp:
• TestRunnerShared/TestFeatures.h:
• WebKitTestRunner/TestController.cpp:
• WebKitTestRunner/TestOptions.cpp:
• WebKitTestRunner/TestOptions.h:
• WebKitTestRunner/cocoa/TestControllerCocoa.mm:
• WebKitTestRunner/gtk/TestControllerGtk.cpp:
• WebKitTestRunner/mac/TestControllerMac.mm:
• WebKitTestRunner/wpe/TestControllerWPE.cpp:

LayoutTests:

Update test header commands to match names in the WebPreferences*.yaml configuration files.

• accessibility/attachment-element.html:
• accessibility/content-changed-notification-causes-crash.html:
• accessibility/contenteditable-table-check-causes-crash.html:
• accessibility/duplicate-axrenderobject-crash.html:
• accessibility/ios-simulator/attributed-string-for-range.html:
• accessibility/ios-simulator/has-touch-event-listener-with-shadow.html:
• accessibility/ios-simulator/media-with-aria-label.html:
• accessibility/mac/attachment-element-replacement-character.html:
• accessibility/mac/video-tag-hit-test.html:
• accessibility/mac/video-volume-slider-accessibility.html:
• accessibility/media-element.html:
• accessibility/media-emits-object-replacement.html:
• accessibility/media-with-aria-label.html:
• animations/resume-after-page-cache.html:
• compositing/accelerated-layers-after-back.html:
• compositing/iframes/page-cache-layer-tree.html:
• compositing/page-cache-back-crash.html:
• compositing/show-composited-iframe-on-back-button.html:
• compositing/video/poster-expected.html:
• compositing/video/poster.html:
• css3/color-filters/color-filter-animation.html:
• css3/color-filters/color-filter-apple-invert-lightness.html:
• css3/color-filters/color-filter-backgrounds-borders.html:
• css3/color-filters/color-filter-box-shadow.html:
• css3/color-filters/color-filter-brightness.html:
• css3/color-filters/color-filter-caret-color.html:
• css3/color-filters/color-filter-color-property-list-item.html:
• css3/color-filters/color-filter-color-property.html:
• css3/color-filters/color-filter-color-text-decorations.html:
• css3/color-filters/color-filter-column-rule.html:
• css3/color-filters/color-filter-composition-underline-color-expected.html:
• css3/color-filters/color-filter-composition-underline-color.html:
• css3/color-filters/color-filter-contrast.html:
• css3/color-filters/color-filter-current-color.html:
• css3/color-filters/color-filter-filter-list.html:
• css3/color-filters/color-filter-gradients.html:
• css3/color-filters/color-filter-grayscale.html:
• css3/color-filters/color-filter-hue-rotate.html:
• css3/color-filters/color-filter-ignore-semantic-expected.html:
• css3/color-filters/color-filter-ignore-semantic.html:
• css3/color-filters/color-filter-inherits.html:
• css3/color-filters/color-filter-invert.html:
• css3/color-filters/color-filter-opacity.html:
• css3/color-filters/color-filter-outline.html:
• css3/color-filters/color-filter-parsing.html:
• css3/color-filters/color-filter-saturate.html:
• css3/color-filters/color-filter-sepia.html:
• css3/color-filters/color-filter-text-decoration-shadow.html:
• css3/color-filters/color-filter-text-emphasis.html:
• css3/color-filters/color-filter-text-shadow.html:
• css3/color-filters/color-filter-text-stroke.html:
• css3/color-filters/svg/color-filter-inline-svg.html:
• editing/async-clipboard/clipboard-change-data-while-reading.html:
• editing/async-clipboard/clipboard-change-data-while-writing.html:
• editing/async-clipboard/clipboard-do-not-read-text-from-platform-if-text-changes.html:
• editing/async-clipboard/clipboard-read-text-from-platform.html:
• editing/async-clipboard/clipboard-read-text-same-origin.html:
• editing/async-clipboard/clipboard-read-while-pasting.html:
• editing/async-clipboard/sanitize-when-reading-markup.html:
• editing/execCommand/set-backColor-with-color-filter-from-scripts.html:
• editing/execCommand/set-foreColor-with-color-filter-from-scripts.html:
• editing/mac/attributed-string/attrib-string-colors-with-color-filter.html:
• editing/mac/attributed-string/attrib-string-range-with-color-filter.html:
• editing/mac/attributed-string/attribute-string-for-copy-with-color-filter.html:
• editing/mac/attributed-string/attributed-string-for-typing-with-color-filter.html:
• editing/mac/input/unconfirmed-text-navigation-with-page-cache.html:
• editing/pasteboard/copy-paste-attachment.html:
• editing/pasteboard/dom-paste/dom-paste-confirmation.html:
• editing/pasteboard/dom-paste/dom-paste-consecutive-confirmations.html:
• editing/pasteboard/dom-paste/dom-paste-rejection.html:
• editing/pasteboard/dom-paste/dom-paste-requires-user-gesture.html:
• editing/pasteboard/dom-paste/dom-paste-same-origin.html:
• editing/pasteboard/drag-file-promises-to-editable-element-as-URLs.html:
• editing/pasteboard/drag-file-promises-to-editable-element-as-attachment.html:
• editing/pasteboard/drag-files-to-editable-element-as-URLs.html:
• editing/pasteboard/drag-files-to-editable-element-as-attachment.html:
• editing/pasteboard/entries-api/DirectoryEntry-getFile-back-forward-cache.html:
• editing/pasteboard/entries-api/DirectoryReader-readEntries-back-forward-cache.html:
• editing/pasteboard/paste-dark-mode-color-filtered.html:
• editing/selection/selection-change-in-disconnected-frame-crash.html:
• editing/style/inverse-color-filter.html:
• editing/style/set-backColor-with-color-filter.html:
• editing/style/set-foreColor-with-color-filter.html:
• fast/attachment/attachment-action-expected.html:
• fast/attachment/attachment-action.html:
• fast/attachment/attachment-border-should-stay-inside-attachment.html:
• fast/attachment/attachment-borderless-expected-mismatch.html:
• fast/attachment/attachment-borderless.html:
• fast/attachment/attachment-default-icon-expected.html:
• fast/attachment/attachment-default-icon.html:
• fast/attachment/attachment-dom.html:
• fast/attachment/attachment-dynamic-type.html:
• fast/attachment/attachment-folder-icon-expected.html:
• fast/attachment/attachment-folder-icon.html:
• fast/attachment/attachment-icon-from-file-extension-expected.html:
• fast/attachment/attachment-icon-from-file-extension.html:
• fast/attachment/attachment-label-highlight.html:
• fast/attachment/attachment-progress.html:
• fast/attachment/attachment-rendering.html:
• fast/attachment/attachment-respects-css-size.html:
• fast/attachment/attachment-select-on-click-inside-user-select-all.html:
• fast/attachment/attachment-select-on-click.html:
• fast/attachment/attachment-subtitle-resize.html:
• fast/attachment/attachment-subtitle.html:
• fast/attachment/attachment-thumbnail-preserves-aspect-ratio.html:
• fast/attachment/attachment-title-with-rtl-expected.html:
• fast/attachment/attachment-title-with-rtl.html:
• fast/attachment/attachment-title.html:
• fast/attachment/attachment-type-attribute-expected.html:
• fast/attachment/attachment-type-attribute.html:
• fast/attachment/attachment-uti-expected.html:
• fast/attachment/attachment-uti.html:
• fast/attachment/attachment-wrapping-action-expected-mismatch.html:
• fast/attachment/attachment-wrapping-action.html:
• fast/block/float/float-not-removed-from-first-letter.html:
• fast/block/float/split-inline-sibling-of-float-crash.html:
• fast/canvas/2d.currentPoint.html:
• fast/canvas/2d.getPath.modification.html:
• fast/canvas/2d.getPath.newobject.html:
• fast/canvas/2d.setPath.html:
• fast/canvas/webgl/canvas-webgl-page-cache.html:
• fast/css/fontloader-page-cache.html:
• fast/css/has-attachment.html:
• fast/css/pseudo-required-optional-unapplied.html:
• fast/dom/DeviceMotion/no-page-cache.html:
• fast/dom/DeviceOrientation/event-after-navigation.html:
• fast/dom/DeviceOrientation/no-page-cache.html:
• fast/dom/HTMLFormElement/associated-elements-after-index-assertion-fail1.html:
• fast/dom/HTMLKeygenElement/keygen-keytype.html:
• fast/dom/HTMLKeygenElement/keygen.html:
• fast/dom/Window/timer-resume-on-navigation-back.html:
• fast/dom/adopt-node-crash-2.html:
• fast/dom/attribute-set-before-element-in-tree.html:
• fast/dom/domstring-attribute-reflection.html:
• fast/dom/set-custom-validity-with-too-few-arguments.html:
• fast/dom/tabindex-defaults.html:
• fast/dom/wrapper-classes.html:
• fast/dynamic/insertAdjacentHTML-allowed-parents.html:
• fast/events/ios/do-not-show-keyboard-when-focusing-after-blur.html:
• fast/events/ios/pdf-modifer-key-down-crash.html:
• fast/events/ios/rotation/do-not-shrink-to-fit-content-after-rotation.html:
• fast/events/onunload-back-to-page-cache.html:
• fast/events/pagehide-timeout.html:
• fast/events/pagehide-xhr-open.html:
• fast/events/pageshow-pagehide-on-back-cached-with-frames.html:
• fast/events/pageshow-pagehide-on-back-cached.html:
• fast/events/suspend-timers.html:
• fast/files/file-reader-back-forward-cache.html:
• fast/flexbox/crash-button-keygen.html:
• fast/forms/ValidityState-001.html:
• fast/forms/autocomplete-off-with-default-value-does-not-clear.html:
• fast/forms/autofocus-keygen.html:
• fast/forms/checkValidity-001.html:
• fast/forms/fieldset/fieldset-elements-htmlcollection.html:
• fast/forms/fieldset/fieldset-elements.html:
• fast/forms/form-associated-element-crash3.html:
• fast/forms/form-attribute.html:
• fast/forms/form-collection-elements-order.html:
• fast/forms/form-collection-elements.html:
• fast/forms/form-collection-radio-node-list.html:
• fast/forms/ios/no-scrolling-when-moving-focus-between-adjacent-fields.html:
• fast/forms/label/labelable-elements.html:
• fast/forms/label/labels-add-htmlFor-label.html:
• fast/forms/label/labels-add-parent-label.html:
• fast/forms/label/labels-change-htmlFor-attribute.html:
• fast/forms/label/labels-multiple-sibling-labels.html:
• fast/forms/label/labels-parent-and-sibling-labels.html:
• fast/forms/label/labels-remove-htmlFor-attribute.html:
• fast/forms/label/labels-remove-htmlFor-label.html:
• fast/forms/label/labels-remove-parent-label.html:
• fast/forms/large-parts.html:
• fast/forms/scrollheight-with-mutation-crash.html:
• fast/forms/setCustomValidity-arguments.html:
• fast/forms/setCustomValidity-existence.html:
• fast/forms/setCustomValidity-null-parameter.html:
• fast/forms/state-restore-skip-stateless.html:
• fast/forms/update-form-owner-in-moved-subtree-assertion-failure.html:
• fast/forms/validationMessage.html:
• fast/forms/willvalidate.html:
• fast/frames/frame-crash-with-page-cache.html:
• fast/frames/frame-unload-navigate-and-setTimeout-assert-fail.html:
• fast/frames/page-hide-document-open.html:
• fast/frames/restoring-page-cache-should-not-run-scripts-via-style-update.html:
• fast/frames/restoring-page-cache-should-not-run-scripts.html:
• fast/harness/page-cache-crash-on-data-urls.html:
• fast/harness/use-page-cache.html:
• fast/hidpi/video-controls-in-hidpi.html:
• fast/history/back-forward-cache-visibility-state.html:
• fast/history/back-from-page-with-focused-iframe.html:
• fast/history/go-back-to-iframe-with-plugin.html:
• fast/history/go-back-to-object-subframe.html:
• fast/history/history-back-while-pdf-in-pagecache.html:
• fast/history/multiple-back-forward-navigations.html:
• fast/history/page-cache-MessagePort-pending-message.html:
• fast/history/page-cache-active-fetch-request-blobReadAsBlob.html:
• fast/history/page-cache-active-fetch-request-blobReadAsReadableStream.html:
• fast/history/page-cache-active-fetch-request-blobReadAsText.html:
• fast/history/page-cache-active-fetch-response-blobReadAsBlob.html:
• fast/history/page-cache-after-window-open.html:
• fast/history/page-cache-back-navigation-crash.html:
• fast/history/page-cache-clearing.html:
• fast/history/page-cache-closed-audiocontext.html:
• fast/history/page-cache-createImageBitmap.html:
• fast/history/page-cache-createObjectURL-using-open-panel.html:
• fast/history/page-cache-createObjectURL.html:
• fast/history/page-cache-destroy-document.html:
• fast/history/page-cache-element-state-focused.html:
• fast/history/page-cache-execute-script-during-restore.html:
• fast/history/page-cache-geolocation-active-oneshot.html:
• fast/history/page-cache-geolocation-active-watcher.html:
• fast/history/page-cache-geolocation.html:
• fast/history/page-cache-gpu-canvas-context.html:
• fast/history/page-cache-iframe-js-url.html:
• fast/history/page-cache-indexed-closed-db.html:
• fast/history/page-cache-indexed-opened-db.html:
• fast/history/page-cache-media-recorder.html:
• fast/history/page-cache-media-source-closed-2.html:
• fast/history/page-cache-media-source-closed.html:
• fast/history/page-cache-media-source-opened.html:
• fast/history/page-cache-navigate-during-restore.html:
• fast/history/page-cache-notification-showing.html:
• fast/history/page-cache-notification-suspendable.html:
• fast/history/page-cache-removed-source-buffer.html:
• fast/history/page-cache-running-audiocontext.html:
• fast/history/page-cache-subframes-with-provisional-load.html:
• fast/history/page-cache-suspended-audiocontext.html:
• fast/history/page-cache-webdatabase-no-transaction-db.html:
• fast/history/page-cache-webdatabase-pending-transaction.html:
• fast/history/page-cache-with-opener.html:
• fast/history/pagehide-remove-iframe-crash.html:
• fast/history/resources/page-cache-window-with-iframe.html:
• fast/history/resources/page-cache-window-with-opener.html:
• fast/history/timed-refresh-in-cached-frame.html:
• fast/html/clone-keygen.html:
• fast/html/keygen.html:
• fast/html/marquee-reparent-check.html:
• fast/html/menuitem-element.html:
• fast/images/animated-gif-restored-from-bfcache.html:
• fast/invalid/residual-style-expected.html:
• fast/invalid/residual-style.html:
• fast/layers/video-layer.html:
• fast/loader/data-url-load-denied.html:
• fast/loader/frames-with-unload-handlers-in-page-cache.html:
• fast/loader/image-in-page-cache.html:
• fast/loader/input-element-page-cache-crash.html:
• fast/loader/navigate-with-new-target-after-back-forward-navigation.html:
• fast/loader/navigate-with-post-to-new-target-after-back-forward-navigation.html:
• fast/loader/scroll-position-restored-on-back.html:
• fast/loader/stateobjects/no-popstate-when-back-to-stateless-entry-with-page-cache.html:
• fast/loader/stateobjects/popstate-fires-with-page-cache.html:
• fast/loader/unschedule-relayout-after-unload.html:
• fast/loader/window-properties-restored-from-page-cache.html:
• fast/mediastream/MediaStream-video-element-video-tracks-disabled.html:
• fast/mediastream/RTCPeerConnection-page-cache.html:
• fast/mediastream/captureInGPUProcess.html:
• fast/mediastream/media-stream-track-interrupted.html:
• fast/multicol/span/positioned-child-not-removed-crash.html:
• fast/overflow/horizontal-scroll-after-back.html:
• fast/ruby/float-object-doesnt-crash.html:
• fast/sandbox/ios/sandbox-mach-lookup-attachment-element.html:
• fast/scrolling/iframe-scrollable-after-back.html:
• fast/scrolling/ios/scroll-events-back-forward-after-pageshow.html:
• fast/scrolling/ios/scroll-events-back-forward.html:
• fast/scrolling/ios/touch-scroll-back-forward.html:
• fast/scrolling/mac/scrollbars/overflow-in-iframe-overlay-scrollbar-hovered.html:
• fast/scrolling/mac/scrollbars/overflow-in-iframe-overlay-scrollbar-reveal.html:
• fast/scrolling/mac/scrollbars/overflow-overlay-scrollbar-hovered.html:
• fast/scrolling/mac/scrollbars/overflow-overlay-scrollbar-reveal.html:
• fast/scrolling/mac/scrollbars/overlay-scrollbar-hovered.html:
• fast/scrolling/mac/scrollbars/overlay-scrollbar-reveal.html:
• fast/scrolling/mac/scrollbars/overlay-scrollbar-state.html:
• fast/scrolling/mac/scrollbars/select-overlay-scrollbar-hovered.html:
• fast/scrolling/mac/scrollbars/select-overlay-scrollbar-reveal.html:
• fast/scrolling/overflow-scrollable-after-back.html:
• fast/scrolling/page-cache-back-overflow-scroll-restore.html:
• fast/scrolling/rtl-scrollbars-animation-property.html:
• fast/shadow-dom/keygen-shadow-manipulation.html:
• fast/table/crash-splitColumn-2.html:
• fast/table/multiple-captions-crash3-expected.html:
• fast/table/multiple-captions-crash3.html:
• fast/text-autosizing/ios/text-autosizing-after-back.html:
• fast/viewport/ios/ignore-minimum-device-width-for-page-with-viewport-device-width.html:
• fast/viewport/ios/minimum-scale-after-changing-view-scale.html:
• fast/viewport/ios/resources/go-back.html:
• fast/viewport/ios/shrink-to-fit-content-constant-width.html:
• fast/viewport/ios/shrink-to-fit-content-large-constant-width.html:
• fast/viewport/ios/shrink-to-fit-content-large-width-breakpoint.html:
• fast/viewport/ios/shrink-to-fit-content-no-viewport.html:
• fast/viewport/ios/shrink-to-fit-content-responsive-viewport-with-horizontal-overflow.html:
• fast/viewport/ios/shrink-to-fit-content-temporary-overflow.html:
• fast/viewport/ios/shrink-to-fit-for-page-without-viewport-meta.html:
• fast/viewport/ios/shrink-to-fit-large-content-width.html:
• fast/viewport/ios/use-minimum-device-width-for-page-without-viewport-meta.html:
• fast/viewport/ios/viewport-shrink-to-fit-on-new-navigation.html:
• fast/viewport/viewport-128.html:
• fast/workers/worker-page-cache.html:
• fullscreen/video-controls-drag.html:
• fullscreen/video-controls-override.html:
• fullscreen/video-controls-rtl.html:
• fullscreen/video-controls-timeline.html:
• http/tests/cache-storage/page-cache-domcache-pending-promise.html:
• http/tests/cache-storage/page-cache-domcachestorage-pending-promise.html:
• http/tests/cache/display-image-unset-allows-cached-image-load.html:
• http/tests/eventsource/eventsource-page-cache-connected.html:
• http/tests/eventsource/eventsource-page-cache-connecting.html:
• http/tests/loading/main-resource-delegates-on-back-navigation.html:
• http/tests/loading/unfinished-load-back-to-cached-page-callbacks.html:
• http/tests/loading/unfinished-main-resource-back-to-cached-page-callbacks.html:
• http/tests/media/hls/video-controls-live-stream.html:
• http/tests/media/hls/video-duration-accessibility.html:
• http/tests/media/reload-after-dialog.html:
• http/tests/media/video-auth-with-allowCrossOriginSubresourcesToAskForCredentials.html:
• http/tests/misc/resources/check-keygen-post.php:
• http/tests/misc/submit-post-keygen.html:
• http/tests/navigation/go-back-to-error-page.html:
• http/tests/navigation/https-in-page-cache.html:
• http/tests/navigation/https-no-store-subframe-in-page-cache.html:
• http/tests/navigation/image-load-in-pagehide-handler.html:
• http/tests/navigation/page-cache-failed-xhr.html:
• http/tests/navigation/page-cache-fontfaceset.html:
• http/tests/navigation/page-cache-fragment-referrer.html:
• http/tests/navigation/page-cache-getUserMedia-pending-promise.html:
• http/tests/navigation/page-cache-iframe-no-current-historyItem.html:
• http/tests/navigation/page-cache-iframe-provisional-load-async-delegates.html:
• http/tests/navigation/page-cache-iframe-provisional-load.html:
• http/tests/navigation/page-cache-mediakeysession.html:
• http/tests/navigation/page-cache-mediastream.html:
• http/tests/navigation/page-cache-pending-image-load-cache-partition.html:
• http/tests/navigation/page-cache-pending-image-load.html:
• http/tests/navigation/page-cache-pending-load.html:
• http/tests/navigation/page-cache-pending-ping-load-cross-origin.html:
• http/tests/navigation/page-cache-pending-ping-load-same-origin.html:
• http/tests/navigation/page-cache-requestAnimationFrame.html:
• http/tests/navigation/page-cache-video.html:
• http/tests/navigation/page-cache-xhr-in-loading-iframe.html:
• http/tests/navigation/page-cache-xhr-in-pagehide.html:
• http/tests/navigation/page-cache-xhr.html:
• http/tests/navigation/resources/https-in-page-cache-3.html:
• http/tests/navigation/subframe-pagehide-handler-starts-load.html:
• http/tests/navigation/subframe-pagehide-handler-starts-load2.html:
• http/tests/paymentrequest/page-cache-closed-payment-request.https.html:
• http/tests/paymentrequest/page-cache-completed-payment-response.https.html:
• http/tests/paymentrequest/page-cache-created-payment-request.https.html:
• http/tests/paymentrequest/page-cache-created-payment-response.https.html:
• http/tests/paymentrequest/page-cache-interactive-payment-request.https.html:
• http/tests/paymentrequest/page-cache-retried-payment-response.https.html:
• http/tests/security/contentSecurityPolicy/userAgentShadowDOM/default-src-object-data-url-allowed.html:
• http/tests/security/contentSecurityPolicy/userAgentShadowDOM/default-src-object-data-url-blocked.html:
• http/tests/security/contentSecurityPolicy/userAgentShadowDOM/default-src-object-data-url-blocked2.html:
• http/tests/security/contentSecurityPolicy/userAgentShadowDOM/video-controls-allowed.html:
• http/tests/security/credentials-iframes-allowCrossOriginSubresourcesToAskForCredentials.html:
• http/tests/security/cross-origin-window-open-insert-script.html:
• http/tests/security/cross-origin-window-open-javascript-url.html:
• http/tests/security/mixedContent/insecure-basic-auth-image-allowCrossOriginSubresourcesToAskForCredentials.https.html:
• http/tests/security/mixedContent/insecure-image-redirects-to-basic-auth-secure-image-allowCrossOriginSubresourcesToAskForCredentials.html:
• http/tests/security/mixedContent/secure-redirect-to-insecure-redirect-to-basic-auth-secure-image-allowCrossOriginSubresourcesToAskForCredentials.https.html:
• http/tests/security/mixedContent/secure-redirect-to-secure-redirect-to-basic-auth-insecure-image-allowCrossOriginSubresourcesToAskForCredentials.https.html:
• http/tests/security/mixedContent/secure-redirect-to-secure-redirect-to-basic-auth-secure-image-allowCrossOriginSubresourcesToAskForCredentials.https.html:
• http/tests/security/navigate-when-restoring-cached-page.html:
• http/tests/security/xss-DENIED-click-and-form-submission-from-inactive-domwindow.html:
• http/tests/security/xss-DENIED-script-inject-into-inactive-window.html:
• http/tests/security/xss-DENIED-script-inject-into-inactive-window2-pson.html:
• http/tests/security/xss-DENIED-script-inject-into-inactive-window2.html:
• http/tests/security/xss-DENIED-script-inject-into-inactive-window3.html:
• http/tests/ssl/applepay/page-cache-active-apple-pay-session.html:
• http/tests/ssl/applepay/page-cache-inactive-apple-pay-session.html:
• http/tests/websocket/tests/hybi/closed-when-entering-page-cache.html:
• http/tests/websocket/tests/hybi/stop-on-resume-in-error-handler.html:
• http/tests/workers/service/client-added-to-clients-when-restored-from-page-cache.html:
• http/tests/workers/service/client-removed-from-clients-while-in-page-cache.html:
• http/tests/workers/service/page-cache-service-worker-pending-promise.https.html:
• http/tests/workers/service/page-caching.html:
• http/wpt/beacon/support/sendBeacon-onpagehide-window.html:
• http/wpt/fetch/resources/fetch-in-pagehide-window.html:
• http/wpt/notifications/constructor-basic-bfcache.html:
• http/wpt/service-workers/mac/throttleable.https.html:
• http/wpt/service-workers/service-worker-spinning-activate.https.html:
• http/wpt/service-workers/service-worker-spinning-fetch.https.html:
• http/wpt/service-workers/service-worker-spinning-install.https.html:
• http/wpt/service-workers/service-worker-spinning-message.https.html:
• imported/blink/fast/dom/shadow/shadowdom-for-keygen-only-shadow-expected.html:
• imported/blink/fast/dom/shadow/shadowdom-for-keygen-only-shadow.html:
• imported/blink/fast/text/international-iteration-simple-text.html:
• inspector/css/pseudo-element-matches.html:
• intersection-observer/intersection-observer-entry-interface.html:
• intersection-observer/intersection-observer-interface.html:
• intersection-observer/root-element-deleted.html:
• intersection-observer/root-element-moved.html:
• intersection-observer/target-deleted.html:
• js/dom/regress-157246.html:
• loader/go-back-cached-main-resource.html:
• loader/go-back-to-different-window-size.html:
• media/accessibility-closed-captions-has-aria-owns.html:
• media/adopt-node-crash.html:
• media/airplay-allows-buffering.html:
• media/airplay-autoplay.html:
• media/audio-as-video-fullscreen.html:
• media/audio-controls-do-not-fade-out.html:
• media/audio-controls-rendering.html:
• media/audio-controls-timeline-in-media-document.html:
• media/audio-delete-while-slider-thumb-clicked.html:
• media/audio-repaint.html:
• media/click-placeholder-not-pausing.html:
• media/click-volume-bar-not-pausing.html:
• media/controls-after-reload.html:
• media/controls-drag-timebar.html:
• media/controls-right-click-on-timebar.html:
• media/controls-strict.html:
• media/controls-styling.html:
• media/controls-without-preload.html:
• media/controls/airplay-controls.html:
• media/controls/airplay-picker.html:
• media/controls/basic.html:
• media/controls/elementOrder.html:
• media/controls/forced-tracks-only.html:
• media/controls/fullscreen-button-inline-layout.html:
• media/controls/inline-elements-dropoff-order.html:
• media/controls/picture-in-picture.html:
• media/controls/pip-placeholder-without-video-controls.html:
• media/controls/showControlsButton.html:
• media/controls/statusDisplay.html:
• media/controls/statusDisplayBad.html:
• media/controls/track-menu.html:
• media/crash-closing-page-with-media-as-plugin-fallback.html:
• media/mac/controls-panel-not-clipped-out-expected.html:
• media/mac/controls-panel-not-clipped-out.html:
• media/media-captions-no-controls.html:
• media/media-controller-drag-crash.html:
• media/media-controls-accessibility.html:
• media/media-controls-cancel-events.html:
• media/media-controls-clone.html:
• media/media-controls-drag-timeline-set-controls-property.html:
• media/media-controls-timeline-updates-after-playing.html:
• media/media-controls-timeline-updates-when-hovered.html:
• media/media-controls-timeline-updates.html:
• media/media-document-audio-controls-visible.html:
• media/media-document-audio-repaint.html:
• media/media-fullscreen-loop-inline.html:
• media/media-fullscreen-pause-inline.html:
• media/media-fullscreen-return-to-inline.html:
• media/media-source/media-source-page-cache.html:
• media/nodesFromRect-shadowContent.html:
• media/progress-events-generated-correctly.html:
• media/require-user-gesture-to-load-video.html:
• media/restore-from-page-cache.html:
• media/tab-focus-inside-media-elements.html:
• media/track/track-cue-rendering-horizontal.html:
• media/track/track-cue-rendering-snap-to-lines-not-set.html:
• media/track/track-cue-rendering-vertical.html:
• media/track/track-cues-cuechange.html:
• media/track/track-cues-enter-exit.html:
• media/track/track-in-band-duplicate-tracks-when-source-changes.html:
• media/track/track-manual-mode.html:
• media/video-click-dblckick-standalone.html:
• media/video-controls-audiotracks-trackmenu.html:
• media/video-controls-captions-trackmenu-localized.html:
• media/video-controls-captions-trackmenu-only-captions-descriptions-and-subtitles.html:
• media/video-controls-captions-trackmenu-sorted.html:
• media/video-controls-captions-trackmenu.html:
• media/video-controls-drop-and-restore-timeline.html:
• media/video-controls-in-media-document.html:
• media/video-controls-no-display-with-text-track.html:
• media/video-controls-rendering.html:
• media/video-controls-show-on-kb-or-ax-event.html:
• media/video-controls-toggling.html:
• media/video-controls-transformed.html:
• media/video-controls-visible-audio-only.html:
• media/video-controls-visible-exiting-fullscreen.html:
• media/video-controls-zoomed.html:
• media/video-display-toggle.html:
• media/video-empty-source.html:
• media/video-fullscreen-only-controls.html:
• media/video-fullscreen-only-playback.html:
• media/video-initially-hidden-volume-slider-up.html:
• media/video-no-audio.html:
• media/video-play-audio-require-user-gesture.html:
• media/video-play-require-user-gesture.html:
• media/video-trackmenu-selection.html:
• media/video-volume-slider-drag.html:
• media/video-volume-slider.html:
• media/video-zoom-controls.html:
• media/volume-bar-empty-when-muted.html:
• platform/ios/ios/fixed/fixed-back-forward.html:
• platform/ios/media/video-play-glyph-composited-outside-overflow-scrolling-touch-container.html:
• platform/ios/mediastream/audio-muted-in-background-tab-gpu-process.html:
• platform/ios/mediastream/audio-muted-in-background-tab.html:
• platform/ios/mediastream/getUserMedia-single-capture-gpu-process.html:
• platform/ios/mediastream/getUserMedia-single-capture.html:
• platform/ipad/fast/viewport/meta-viewport-ignored.html:
• platform/ipad/fast/viewport/viewport-overriden-by-minimum-effective-width-if-ignore-meta-viewport.html:
• plugins/crash-restoring-plugin-page-from-page-cache.html:
• plugins/frameset-with-plugin-frame.html:
• plugins/netscape-plugin-page-cache-works.html:
• requestidlecallback/requestidlecallback-in-page-cache.html:
• storage/indexeddb/IDBRequest-page-cache.html:
• storage/indexeddb/IDBTransaction-page-cache.html:
• svg/dom/css-animate-input-foucs-crash.html:
• swipe/basic-cached-back-swipe.html:
• swipe/main-frame-pinning-requirement.html:
• swipe/pushState-cached-back-swipe.html:
• swipe/pushState-programmatic-back-while-swiping-crash.html:
• swipe/pushstate-with-manual-scrollrestoration.html:
• tiled-drawing/scrolling/null-parent-back-crash.html:
• tiled-drawing/tiled-drawing-scroll-position-page-cache-restoration.html:
• webanimations/animation-page-cache.html:
• webrtc/datachannel/datachannel-page-cache-send.html:
• webrtc/datachannel/datachannel-page-cache.html:
• webrtc/peerconnection-new-candidate-page-cache.html:
• webrtc/peerconnection-page-cache-long.html:
• webrtc/peerconnection-page-cache.html:
• webrtc/video-rotation-no-cvo.html:

10:13 AM WebKitGTK/2.30.x edited by Michael Catanzaro

(diff)

10:10 AM Changeset in webkit [268399] by Alan Bujtas

• 4 edits in trunk/Source/WebCore

[LFC][IFC] Introduce InlineLevelBox::LineBreakBox and GenericInlineLevelBox
​https://bugs.webkit.org/show_bug.cgi?id=217646

Reviewed by Antti Koivisto.

This is in preparation for adding support for ​https://www.w3.org/TR/css-inline-3/#layout-bounds.

• layout/inlineformatting/InlineFormattingContextGeometry.cpp:

(WebCore::Layout::LineBoxBuilder::constructInlineLevelBoxes): Cleaning up a bit and moving adjustVerticalGeometryForNonEmptyInlineBox
to a dedicated function. constructInlineLevelBoxes was getting a bit too busy.

• layout/inlineformatting/InlineLineBox.cpp:

(WebCore::Layout::LineBox::InlineLevelBox::InlineLevelBox):

• layout/inlineformatting/InlineLineBox.h:

(WebCore::Layout::LineBox::InlineLevelBox::createRootInlineBox):
(WebCore::Layout::LineBox::InlineLevelBox::createAtomicInlineLevelBox):
(WebCore::Layout::LineBox::InlineLevelBox::createInlineBox):
(WebCore::Layout::LineBox::InlineLevelBox::createLineBreakBox):
(WebCore::Layout::LineBox::InlineLevelBox::createGenericInlineLevelBox):

10:06 AM Changeset in webkit [268398] by youenn@apple.com

• 7 edits in trunk/Source/WebCore

Webcam video from navigator.mediaDevices.getUserMedia() to 2D canvas fails on Safari on iPhone
​https://bugs.webkit.org/show_bug.cgi?id=217578
<rdar://problem/70183875>

Reviewed by Eric Carlson.

It is unneeded for MediaStream video backends to enqueue samples to the display layer if the video element is hidden.
In iOS, the samples may never be flushed which might cause capture failing after enqueuing too many camera samples.
To avoid that, we no longer enqueue samples to the display layer when the player is not visible.
In case of canvas painting, other backends need to be made visible for canvas painting to work.
For MediaStream backend, we do not need that since we always keep the last sample.
For that reason, we keep the backend as not visible even if canvas happens.
We do so by introducing a setVisibleForCanvas that is a no-op for MediaStream backend and similar to setVisible for other backends.
For good measure, we now flush the MediaStream display layer whenever visibility changed.

Manually tested.

• html/HTMLVideoElement.cpp:

(WebCore::HTMLVideoElement::paintCurrentFrameInContext):

• platform/graphics/MediaPlayer.cpp:

(WebCore::MediaPlayer::setVisibleForCanvas):

• platform/graphics/MediaPlayer.h:
• platform/graphics/MediaPlayerPrivate.h:

(WebCore::MediaPlayerPrivateInterface::setVisibleForCanvas):

• platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaStreamAVFObjC.h:
• platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaStreamAVFObjC.mm:

(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::enqueueVideoSample):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::setVisible):
(WebCore::MediaPlayerPrivateMediaStreamAVFObjC::setVisibleForCanvas):

9:56 AM Changeset in webkit [268397] by Peng Liu

• 2 edits in trunk/Tools

Clean up the API tests of CARingBuffer
​https://bugs.webkit.org/show_bug.cgi?id=217633

Reviewed by Simon Fraser.

Remove unused private functions and enable the tests for iOS.

• TestWebKitAPI/Tests/WebCore/CARingBuffer.cpp:

(TestWebKitAPI::CARingBufferTest::audioBufferListSizeForStream): Deleted.
(TestWebKitAPI::CARingBufferTest::configureBufferListForStream): Deleted.

9:13 AM Changeset in webkit [268396] by youenn@apple.com

• 4 edits in trunk/Source/WebCore

Improve computation of default audio input and output devices
​https://bugs.webkit.org/show_bug.cgi?id=217652
<rdar://problem/69857857>

Reviewed by Eric Carlson.

Add support for computation of the default audio output device.
Fix a bug in the computation of the default device on the first call to refreshAudioCaptureDevices.
Refactor the code as follows to make it easier to read:

• for each case we want to refresh devices, compute a Vector of audio devices, first microphones, then speakers.
• make sure to put in that list the default microphone first in the list and the default speaker just after the last microphone in the list.
• check whether the list is equal to the current list of audio devices. If it is equal, abort.
• update microphone and speaker lists and send notification of device change.

Minor refactoring to make CoreAudioCaptureDevice isAlive a private method.

Manually tested.

• platform/mediastream/mac/CoreAudioCaptureDevice.cpp:

(WebCore::CoreAudioCaptureDevice::CoreAudioCaptureDevice):

• platform/mediastream/mac/CoreAudioCaptureDevice.h:
• platform/mediastream/mac/CoreAudioCaptureDeviceManager.cpp:

(WebCore::getDefaultCaptureOutputDevice):
(WebCore::CoreAudioCaptureDeviceManager::coreAudioCaptureDevices):
(WebCore::CoreAudioCaptureDeviceManager::refreshAudioCaptureDevices):

9:10 AM Changeset in webkit [268395] by Lauro Moura

• 2 edits in trunk/Tools

REGRESSION(r268267) [webkitpy] Add missing deps for pytest when running webdriver tests
​https://bugs.webkit.org/show_bug.cgi?id=217650

Reviewed by Jonathan Bedard.

scandir (needed as os.scandir was added only in py 3.5)

Also fixed more_itertools autoinstall declaration so the package
name, with underscore, is the name of the webkitcorepy package and the
dash is used for the pypi package name.

With these changes the script does not error anymore, although there
seems to be some issues with the executor when running tests. They
happen to only start and stop the webserver but not actually being
run. To be investigated in a separate bug.

• Scripts/webkitpy/init.py:

8:39 AM Changeset in webkit [268394] by commit-queue@webkit.org

• 3 edits in trunk/Source/WebCore

[SOUP] webkit_web_view_get_https_status() broken with service workers
​https://bugs.webkit.org/show_bug.cgi?id=216038

Patch by Michael Catanzaro <Michael Catanzaro> on 2020-10-13
Reviewed by Carlos Garcia Campos.

This implements CertificateInfo::isolatedCopy for libsoup ports. This is impossible to do
completely, because we cannot copy the private key portion of the GTlsCertificate, because
it is a write-only property, because it might be backed by a hardware token and therefore
really impossible to write software to get it. If we were to implement g_tls_certificate_copy()
in GLib -- which I will probably do eventually, because I need this outside WebKit as well --
then GTlsCertificate implementations could copy the private key or PKCS#11 handle or
whatever.

But anyway, let not perfect be the enemy of the good. We only need this for service workers
currently. (Probably that's all we'll *ever* need it for.) So we are only working with
server certificates, and the private portion of the certificate is guaranteed to not exist
(because servers don't send us their private keys), and we can just forget about it. As
long as nobody tries to copy a client certificate in the future -- where we really would
need the private key portion of the certificate -- this will be perfectly fine.

Actually copying the certificate is kind of annoying, because a chain of certificates is
represented by having the main (server) certificate keep a reference to its issuer, which is
not referenced anywhere else, so we have to reconstruct the chain in reverse order starting
from the final certificate, working back towards the server cert. Fun. Let's also be
careful to construct a completely new GByteArray rather than expecting the GTlsCertificate
implementation to copy it for us. Because GTlsCertificate is implemented by an extension
point and applications and system administrators can -- and do -- implement their own,
implementations could do anything, including keep a reference to the GByteArray that we
pass in.

It would be nice to have a test for this, but writing tests is hard. Also, I don't really
want to learn what service workers are. :)

Drive-by fix: also remove explicit from a constructor that doesn't need it.

• platform/network/soup/CertificateInfo.h:

(WebCore::CertificateInfo::isolatedCopy const):

• platform/network/soup/CertificateInfoSoup.cpp:

(WebCore::createCertificate):
(WebCore::CertificateInfo::isolatedCopy const):

7:38 AM Changeset in webkit [268393] by Alan Bujtas

• 4 edits
  2 adds in trunk

[LFC][IFC] Remove fully collapsed trailing run
​https://bugs.webkit.org/show_bug.cgi?id=217653

Reviewed by Antti Koivisto.

Source/WebCore:

We don't attach fully collapsible runs to the line.
When Line::appendText sees an inline item that would generate such run
(e.g. <div>foobar <span> <span></div> - the inline text item inside the <span> would generate fully collapsed run)
it simply ignores it and will not generate a run for it.

In this patch when the trailing trimmable run is fully collapsed, we should not only collapse it,
but also remove it from the line.

Test: fast/layoutformattingcontext/replaced-and-trailing-trimmable-content.html

• layout/inlineformatting/InlineLine.cpp:

(WebCore::Layout::Line::TrimmableTrailingContent::addFullyTrimmableContent):
(WebCore::Layout::Line::TrimmableTrailingContent::addPartiallyTrimmableContent):
(WebCore::Layout::Line::TrimmableTrailingContent::remove):

• layout/inlineformatting/InlineLine.h:

(WebCore::Layout::Line::TrimmableTrailingContent::isEmpty const):
(WebCore::Layout::Line::TrimmableTrailingContent::reset):

LayoutTests:

• fast/layoutformattingcontext/replaced-and-trailing-trimmable-content-expected.html: Added.
• fast/layoutformattingcontext/replaced-and-trailing-trimmable-content.html: Added.

5:10 AM WebKitGTK/2.30.x edited by Philippe Normand

(diff)

5:08 AM Changeset in webkit [268392] by Philippe Normand

• 5 edits in trunk

[GStreamer] Crash in WebCore::GStreamerRegistryScanner::isAVC1CodecSupported
​https://bugs.webkit.org/show_bug.cgi?id=217647

Reviewed by Xabier Rodriguez-Calvar.

Source/WebCore:

The registry scanner was assuming the avc1 codec field is always of the form
avc1.codecprofile, which is wrong.

Covered by test: media/media-can-play-mpeg4-video.html

• platform/graphics/gstreamer/GStreamerRegistryScanner.cpp:

(WebCore::GStreamerRegistryScanner::isAVC1CodecSupported const): Check for '.' presence and
fallback to unconstrained check if no profile or level information was extracted.

LayoutTests:

• media/media-can-play-mpeg4-video-expected.txt:
• media/media-can-play-mpeg4-video.html: Add test for "avc1" codec.

2:24 AM Changeset in webkit [268391] by Adrian Perez de Castro

• 7 edits in trunk

[GTK] Add support for gamepad tests
​https://bugs.webkit.org/show_bug.cgi?id=98937

Reviewed by Carlos Garcia Campos.

Change PLATFORM(MAC) guards to ENABLE(GAMEPAD) around the support code used for
testing gamepad support in WKTR, and update GTK port test expectations accordingly.

Tools:

• WebKitTestRunner/InjectedBundle/TestRunner.cpp: Change guards to ENABLE(GAMEPAD).
• WebKitTestRunner/TestController.cpp: Ditto.

(WTR::TestController::initialize):

• WebKitTestRunner/TestInvocation.cpp: Ditto.

(WTR::TestInvocation::didReceiveSynchronousMessageFromInjectedBundle):

LayoutTests:

• platform/glib/TestExpectations: Remove skipped gamepad tests.
• platform/wpe/TestExpectations: Skip gamepad tests for the WPE port.

2:20 AM Changeset in webkit [268390] by Adrian Perez de Castro

• 2 edits in trunk/Source/WebCore

[GTK] Make gamepads visible after their first input is received
​https://bugs.webkit.org/show_bug.cgi?id=217494

Reviewed by Carlos Garcia Campos.

Turns out this was mostly working already, with a small caveat: Sometimes there would
be a timing issue if m_inputNotificationTimer would be fired before having a chance
of setting the m_initialGamepadsConnected flag, and in that case the early return
would prevent dispatchPlatformGamepadInputActivity() from ever being called. This
re-arms the timer to be fired again in the next event loop iteration if the flag was
unset, so it will be checked again after it has been toggled. This same approach is
used by HIDGamepadProvider in the Mac port, for example.

• platform/gamepad/manette/ManetteGamepadProvider.cpp:

(WebCore::ManetteGamepadProvider::inputNotificationTimerFired): Re-arm timer if
m_initialGamepadsConnected was not yet set.

2:15 AM Changeset in webkit [268389] by Adrian Perez de Castro

• 2 edits in trunk

[GTK] Set ENABLE_GAMEPAD=ON for developer builds
​https://bugs.webkit.org/show_bug.cgi?id=217493

Reviewed by Carlos Garcia Campos.

• Source/cmake/OptionsGTK.cmake: Make the default value of ENABLE_GAMEPAD

depend on ENABLE_EXPERIMENTAL_FEATURES.

1:17 AM Changeset in webkit [268388] by commit-queue@webkit.org

• 3 edits in trunk

[WPE][GTK] ENABLE_SHAREABLE_RESOURCE should be PRIVATE build option
​https://bugs.webkit.org/show_bug.cgi?id=217592

Patch by Michael Catanzaro <Michael Catanzaro> on 2020-10-13
Reviewed by Carlos Garcia Campos.

• Source/cmake/OptionsGTK.cmake:
• Source/cmake/OptionsWPE.cmake:

12:44 AM Changeset in webkit [268387] by Diego Pino Garcia

• 2 edits in trunk/Tools

Unreviewed GLIB gardening. Remove several tests that were only failing in EWS API-GTK.

• TestWebKitAPI/glib/TestExpectations.json:

12:16 AM Changeset in webkit [268386] by commit-queue@webkit.org

• 11 edits
  1 copy
  1 add in trunk/Source

Cocoa: Make WebGLLayer not dependent on GraphicsContextGLOpenGL
​https://bugs.webkit.org/show_bug.cgi?id=217212
<rdar://problem/69876022>

Patch by Kimmo Kinnunen <​kkinnunen@apple.com> on 2020-10-13
Reviewed by Dean Jackson.

Source/WebCore:

WebGLLayer was needlessly using GraphicsContextGLOpenGL.
This is problematic because WebGLLayer should work with
upcoming remote GraphicsContextGL implementation.

The prepare callgraph was:

GCGLOpenGL -> WebGLLayer -> GCGLOpenGL

Refactor it to be:

GCGLOpenGL -> WebGLLayer

Move the back buffer ownership to the GraphicsContextGLOpenGL.
Make the front buffer ownership explicit in WebGLLayer.
Move the EGL bindings ownerships of all buffers to
GraphicsContextGLOpenGL.

Make the WebGLLayer not use EGL or OpenGL, it does not
need and cannot use it as not all of its clients use OpenGL
(i.e. the above mentioned remote use-case).

Improves the memory usage by not allocating front buffers
unless needed. In case the canvas does not present, will
not allocate front buffers at all.

Improves error handling of the allocations and EGL bindings.

No new tests, a refactor.

• WebCore.xcodeproj/project.pbxproj:
• platform/graphics/angle/GraphicsContextGLANGLE.cpp:

(WebCore::GraphicsContextGLOpenGL::reshapeFBOs):
(WebCore::GraphicsContextGLOpenGL::prepareTexture):
(WebCore::GraphicsContextGLOpenGL::prepareTextureImpl):

• platform/graphics/angle/GraphicsContextGLANGLEUtilities.h: Added.

(WebCore::ScopedRestoreTextureBinding::ScopedRestoreTextureBinding):
(WebCore::ScopedRestoreTextureBinding::~ScopedRestoreTextureBinding):

• platform/graphics/cocoa/GraphicsContextGLOpenGLCocoa.mm:

(WebCore::GraphicsContextGLOpenGL::GraphicsContextGLOpenGL):
(WebCore::GraphicsContextGLOpenGL::~GraphicsContextGLOpenGL):
(WebCore::GraphicsContextGLOpenGL::makeContextCurrent):
(WebCore::GraphicsContextGLOpenGL::reshapeDisplayBufferBacking):
(WebCore::GraphicsContextGLOpenGL::bindDisplayBufferBacking):
(WebCore::GraphicsContextGLOpenGL::prepareForDisplay):
(WebCore::GraphicsContextGLOpenGL::didDisplay):

• platform/graphics/cocoa/WebGLLayer.h:
• platform/graphics/cocoa/WebGLLayer.mm:

(-[WebGLLayer initWithClient:devicePixelRatio:]):
(-[WebGLLayer copyImageSnapshotWithColorSpace:]):
(-[WebGLLayer recycleBuffer]):
(-[WebGLLayer prepareForDisplayWithContents:]):
(-[WebGLLayer display]):
(-[WebGLLayer detachClient]):

• platform/graphics/cocoa/WebGLLayerClient.h: Copied from Source/WebCore/platform/graphics/cocoa/WebGLLayer.h.

(WebCore::WebGLLayerClient::~WebGLLayerClient):

• platform/graphics/opengl/GraphicsContextGLOpenGL.h:
• platform/graphics/opengl/GraphicsContextGLOpenGLBase.cpp:

(WebCore::GraphicsContextGLOpenGL::reshapeFBOs):

Source/WebKit:

Removed unneeded inclusions of GraphicsContextGLOpenGL.h. The
file is now using non-public headers.

• WebProcess/WebPage/WebPage.cpp:
• WebProcess/WebPage/mac/WebPageMac.mm: