Timeline

Oct 21, 2020:

11:43 PM Changeset in webkit [268855] by clopez@igalia.com

• 4 edits
  3 adds in trunk/Tools

[JHbuild] Add patch to wpebackend-fdo to fix the build on Ubuntu-18.04
​https://bugs.webkit.org/show_bug.cgi?id=218068

Reviewed by Adrian Perez de Castro.

On r268591 I updated the version of wpebackend-fdo to the last stable 1.8
but this version is failing to build on Ubuntu-18.04 because of the cmake version.
Backport a patch to fix this.

• gtk/jhbuild.modules:
• gtk/patches/wpebackend-fdo-cmake-buildfix-3.10.patch: Added.
• jhbuild/jhbuild-minimal.modules:
• jhbuild/patches/wpebackend-fdo-cmake-buildfix-3.10.patch: Added.
• wpe/jhbuild.modules:
• wpe/patches/wpebackend-fdo-cmake-buildfix-3.10.patch: Added.

10:24 PM Changeset in webkit [268854] by rniwa@webkit.org

• 4 edits
  1 add in trunk/Source/WebKit

Move function definitions in JSIPCBinding.h into cpp file
​https://bugs.webkit.org/show_bug.cgi?id=218065

Reviewed by Wenson Hsieh.

Added JSIPCBinding.cpp and moved the code from JSIPCBinding.h to JSIPCBinding.cpp.

No new tests since there is no behavior change.

• Platform/IPC/JSIPCBinding.cpp: Added.

(IPC::jsValueForDecodedStringArgumentValue): Moved from .h.
(IPC::jsValueForDecodedArgumentValue<String>): Ditto.
(IPC::jsValueForDecodedArgumentValue<URL>): Ditto.
(IPC::jsValueForDecodedArgumentValue<RegistrableDomain>): Ditto.
(IPC::jsValueForDecodedNumericArgumentValue): Ditto.
(IPC::jsValueForDecodedArgumentValue<double>): Ditto.
(IPC::jsValueForDecodedArgumentValue<float>): Ditto.
(IPC::jsValueForDecodedArgumentValue<int8_t>): Ditto.
(IPC::jsValueForDecodedArgumentValue<int16_t>): Ditto.
(IPC::jsValueForDecodedArgumentValue<int32_t>): Ditto.
(IPC::jsValueForDecodedArgumentValue<int64_t>): Ditto.
(IPC::jsValueForDecodedArgumentValue<uint8_t>): Ditto.
(IPC::jsValueForDecodedArgumentValue<uint16_t>): Ditto.
(IPC::jsValueForDecodedArgumentValue<uint32_t>): Ditto.
(IPC::jsValueForDecodedArgumentValue<uint64_t>): Ditto.
(IPC::jsValueForDecodedArgumentRect): Ditto.
(IPC::jsValueForDecodedArgumentValue<IntRect>): Ditto.
(IPC::jsValueForDecodedArgumentValue<FloatRect>): Ditto.

• Platform/IPC/JSIPCBinding.h:

(IPC::jsValueForDecodedArgumentValue):

• Sources.txt:
• WebKit.xcodeproj/project.pbxproj:

10:24 PM Changeset in webkit [268853] by achristensen@apple.com

• 12 edits in trunk

Update and pass new URL web platform tests
​https://bugs.webkit.org/show_bug.cgi?id=218056

Reviewed by Tim Horton.

LayoutTests/imported/w3c:

• web-platform-tests/url/a-element-expected.txt:
• web-platform-tests/url/a-element-xhtml-expected.txt:
• web-platform-tests/url/resources/setters_tests.json:
• web-platform-tests/url/resources/urltestdata.json:
• web-platform-tests/url/url-constructor-expected.txt:
• web-platform-tests/url/url-searchparams.any.js:
• web-platform-tests/url/url-setters-expected.txt:

Source/WebCore:

Don't add an extra forward slash at the beginning when setting the path of a file URL
that starts with a backslash, which we turn into a forward slash when parsing file URL paths.
This conveniently involves removing duplicate code.

This matches the behavior of Chrome and the URL spec.
Covered by newly passing web platform tests.

• html/URLDecomposition.cpp:

(WebCore::URLDecomposition::setPathname):

Source/WTF:

• wtf/URL.cpp:

(WTF::URL::setPath):

9:50 PM Changeset in webkit [268852] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebCore

Replace O(n^{2) algorithm from r268709 with O(n) algorithm
​https://bugs.webkit.org/show_bug.cgi?id=218062}

Patch by Alex Christensen <​achristensen@webkit.org> on 2020-10-21
Reviewed by Alexey Shvayka.

r268709 introduced a Vector::findMatching call inside a loop that populates the Vector.
This causes very slow construction of URLSearchParams with large dictionaries with 16-bit keys.
To speed things up, keep a HashMap of the 16-bit strings we have already inserted to their index in the Vector
so we don't need to search the Vector.

• bindings/js/JSDOMConvertRecord.h:

7:18 PM Changeset in webkit [268851] by Karl Rackler

• 2 edits in trunk/LayoutTests

REGRESSION: [ Big Sur ] fast/text/international/complex-character-based-fallback.html is a constant failure
rdar://70556068

Unreviewed test gardening.

• platform/mac/TestExpectations:

6:12 PM Changeset in webkit [268850] by clopez@igalia.com

• 2 edits in trunk/Source/WebCore

REGRESSION(r268798): Fix build without ENABLE_LAYOUT_FORMATTING_CONTEXT
​https://bugs.webkit.org/show_bug.cgi?id=218017

Unreviewed build-fix.

• rendering/RenderBox.cpp:

(WebCore::RenderBox::styleDidChange): Add missing ifdef guards.

5:46 PM Changeset in webkit [268849] by weinig@apple.com

• 6 edits in trunk/Source/WebCore

Remove use of in-makefile grepping of FEATURE_AND_PLATFORM_DEFINES
​https://bugs.webkit.org/show_bug.cgi?id=218001

Reviewed by Darin Adler.

Remove use of the $(findstring NEEDLE,$(FEATURE_AND_PLATFORM_DEFINES)) idiom in
DerivedSources.make to pave the way to passing FEATURE_AND_PLATFORM_DEFINES as a
file to scripts, rather than always on the command line, which is quite noisy.

• DerivedSources.make:

Remove conditionalized additions to ADDITIONAL_BINDING_IDLS and just always add
the concrete set, relying on the existing extended attributes in the IDLs instead.
It's also not necessary to ever add the touch related IDL files in WebCore here
as no users of DerivedSources.make use those.

Also removes conditional additions to USER_AGENT_STYLE_SHEETS and instead adds
the appropriate #ifdefs to the css files themselves which are already getting
preprocessed appropriately.

• css/fullscreen.css:
• css/mathml.css:
• css/mediaControls.css:
• html/shadow/mac/imageControlsMac.css:

Put required #ifdefs into the css files themselves, rather than embedding the
logic in the makefile.

5:41 PM Changeset in webkit [268848] by rniwa@webkit.org

• 7 edits in trunk

IPC testing API should have the capability to observe messages being sent and received
​https://bugs.webkit.org/show_bug.cgi?id=217870

Reviewed by Darin Adler.

Source/WebKit:

Added IPC.addIncomingMessageListener and IPC.addOutgoingMessageListener which allows JavaScript
to observe IPC messages being sent or received by WebContent process. We use the generated code
added in r268503 to decode the IPC arguments.

Tests: TestWebKitAPI.IPCTestingAPI.CanInterceptAlert

TestWebKitAPI.IPCTestingAPI.CanInterceptHasStorageAccess
TestWebKitAPI.IPCTestingAPI.CanInterceptFindString

• Platform/IPC/Connection.cpp:

(IPC::Connection::sendMessage): Added the code to invoke MessageObserver::willSendMessage.
Also remove any stale MessageObserver as neded.
(IPC::Connection::addMessageObserver): Added.
(IPC::Connection::dispatchMessage): Added the code to invoke MessageObserver::didReceiveMessage.
Also remove any stale MessageObserver as neded.

• Platform/IPC/Connection.h:

(IPC::Connection::MessageObserver): Added. A pure virtual interface for observing IPC messages.

• Platform/IPC/JSIPCBinding.h:

(IPC::jsValueForDecodedStringArgumentValue): Extracted from jsValueForDecodedArgumentValue<String>.
Now takes the type name as an argument.
(IPC::jsValueForDecodedArgumentValue<URL>): Use "URL" as the type name.
(IPC::jsValueForDecodedArgumentValue<RegistrableDomain>): Use "RegistrableDomain" as the type name.
(IPC::jsValueForDecodedArgumentValue<OptionSet<U>>): Added. Specializations for OptionSet<U>

• WebProcess/WebPage/IPCTestingAPI.cpp:

(WebKit::IPCTestingAPI::JSMessageListener): Added. Implements IPC::MessageObserver.
(WebKit::IPCTestingAPI::JSIPC::staticFunctions):
(WebKit::IPCTestingAPI::createTypeError): Moved.
(WebKit::IPCTestingAPI::JSIPC::addMessageListener): Added.
(WebKit::IPCTestingAPI::JSIPC::addIncomingMessageListener): Added.
(WebKit::IPCTestingAPI::JSIPC::addOutgoingMessageListener): Added.
(WebKit::IPCTestingAPI::JSMessageListener::JSMessageListener): Added.
(WebKit::IPCTestingAPI::JSMessageListener::didReceiveMessage): Added.
(WebKit::IPCTestingAPI::JSMessageListener::willSendMessage): Added.
(WebKit::IPCTestingAPI::JSMessageListener::jsDescriptionFromDecoder): Added.

Tools:

Added tests to intercept IPC messages sent and received by WebContent process.

• TestWebKitAPI/Tests/WebKitCocoa/IPCTestingAPI.mm:

(IPCTestingAPI.CanInterceptAlert):
(IPCTestingAPI.CanInterceptHasStorageAccess):
(IPCTestingAPI.CanInterceptFindString):

5:40 PM Changeset in webkit [268847] by Megan Gardner

• 4 edits
  2 adds in trunk

Stop gap patch fix for regression in r267329.
​https://bugs.webkit.org/show_bug.cgi?id=218020
Source/WebCore:

<rdar://problem/69542459>

Reviewed by Darin Adler.

The refactoring in ​https://bugs.webkit.org/show_bug.cgi?id=216739 caused selections to
expand in both directions after a double-click and drag to expand on mac. This behavior
was not observed on iOS. This removes the error-prone calculation.
Note that this bug does not occur if only testing immediatly moving the drag point to the
end of the selection. Dragging across all the words as a real user would needs to be emulated.

Test: editing/selection/double-click-and-drag-over-anchor-to-select.html

• editing/VisibleSelection.cpp:

(WebCore::VisibleSelection::validate):

LayoutTests:

Reviewed by Darin Adler.

• editing/selection/double-click-and-drag-over-anchor-to-select-expected.txt: Added.
• editing/selection/double-click-and-drag-over-anchor-to-select.html: Added.
• resources/ui-helper.js:

(window.UIHelper.doubleClickAtMouseDown):
(window.UIHelper.mouseUp):
(window.UIHelper.dragMouseAcrossElement):
(window.UIHelper.doubleClickElementMouseDown):

4:36 PM Changeset in webkit [268846] by Karl Rackler

• 2 edits in trunk/LayoutTests

REGRESSION(iOS 14): imported/w3c/web-platform-tests/mathml/relations/html5-tree/dynamic-childlist-001.html is a constant failure
​https://bugs.webkit.org/show_bug.cgi?id=217268

Unreviewed test gardening.

• platform/ios-simulator-wk2/TestExpectations:

4:28 PM Changeset in webkit [268845] by Karl Rackler

• 2 edits in trunk/LayoutTests

REGRESSION (r267531): [ Big Sur iOS 14 ] imported/w3c/web-platform-tests/mathml/relations/html5-tree/dynamic-childlist-001.htm is a constant failure
rdar://70550210

Unreviewed test gardening.

• platform/mac/TestExpectations:

4:26 PM Changeset in webkit [268844] by Jonathan Bedard

• 4 edits in trunk/Tools

[webkitpy] Use webkitcorepy's autoinstaller for beautifulsoup
​https://bugs.webkit.org/show_bug.cgi?id=218049
<rdar://problem/70542684>

Reviewed by Aakash Jain.

• Scripts/webkitpy/init.py:
• Scripts/webkitpy/thirdparty/BeautifulSoup.py:
• Scripts/webkitpy/thirdparty/init.py:

(AutoinstallImportHook.find_module): Remove _install_beatifulsoup.
(AutoinstallImportHook._install_pytest_timeout): Deleted.
(AutoinstallImportHook._install_pytest): Deleted.
(AutoinstallImportHook._install_beautifulsoup): Deleted.

4:09 PM Changeset in webkit [268843] by commit-queue@webkit.org

• 5 edits in trunk

Implement ISO-2022-JP encoder step 3
​https://bugs.webkit.org/show_bug.cgi?id=218046

Patch by Alex Christensen <​achristensen@webkit.org> on 2020-10-21
Reviewed by Darin Adler.

LayoutTests/imported/w3c:

• web-platform-tests/encoding/iso-2022-jp-encoder-expected.txt:
• web-platform-tests/encoding/iso-2022-jp-encoder.html:

Source/WebCore:

​https://encoding.spec.whatwg.org/#iso-2022-jp-encoder step 3 was put in a few years ago and recently implemented by Firefox.
Covered by updated and newly passing web platform tests.

• platform/text/TextCodecCJK.cpp:

(WebCore::iso2022JPEncode):

3:41 PM Changeset in webkit [268842] by Aditya Keerthi

• 2 edits in trunk/Source/WebKit

Remove unused UIDocumentMenuViewController SPI declarations
​https://bugs.webkit.org/show_bug.cgi?id=218053

Reviewed by Tim Horton.

UIDocumentMenuViewController is deprecated and its last
use was removed in r258092.

• Platform/spi/ios/UIKitSPI.h:

3:38 PM Changeset in webkit [268841] by Chris Dumez

• 2 edits in trunk/Source/WebCore

Address post-landing review feedback from Sam Weinig for r268820.

• Modules/webaudio/AudioSummingJunction.cpp:

(WebCore::AudioSummingJunction::maximumNumberOfChannels const):

3:22 PM Changeset in webkit [268840] by Karl Rackler

• 2 edits in trunk/LayoutTests

REGRESSION(iOS 14): imported/w3c/web-platform-tests/mathml/relations/css-styling/padding-border-margin/border-002.html is a constant failure
​https://bugs.webkit.org/show_bug.cgi?id=217268

Unreviewed test gardening.

• platform/ios-simulator-wk2/TestExpectations:

3:17 PM Changeset in webkit [268839] by Karl Rackler

• 2 edits in trunk/LayoutTests

REGRESSION (r267531): [ Big Sur iOS 14 ] imported/w3c/web-platform-tests/mathml/relations/css-styling/padding-border-margin/border-002.html is a constant failure
rdar://70546330

Unreviewed test gardening.

• platform/mac/TestExpectations:

3:16 PM Changeset in webkit [268838] by Alan Coon

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

Cherry-pick r268570. rdar://problem/70541887

v2: WebKit::XPCServiceEventHandler block should call exit() on the main thread
<​https://webkit.org/b/217773>
<rdar://problem/70275659>

Reviewed by Geoff Garen.

• Shared/EntryPointUtilities/Cocoa/XPCService/XPCServiceMain.mm: (WebKit::XPCServiceEventHandler):
• Use NSRunLoop instead of libdispatch to run code on the main thread since libdispatch can randomly pick a new main thread when exit() is called on the original main thread.

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

3:16 PM Changeset in webkit [268837] by Alan Coon

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

Cherry-pick r268398. rdar://problem/70541902

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):

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

3:16 PM Changeset in webkit [268836] by Alan Coon

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

Cherry-pick r268712. rdar://problem/70541891

Fix crash in RenderLayerBacking::updateClippingStackLayerGeometry()
​https://bugs.webkit.org/show_bug.cgi?id=217940
<rdar://problem/70316952>

Reviewed by Tim Horton.

Crash data suggest that entry.clipData.clippingLayer (which is a WeakRef<RenderLayer>)
can be null, so check it.

• rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::updateClippingStackLayerGeometry):

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

3:16 PM Changeset in webkit [268835] by Alan Coon

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

Cherry-pick r268701. rdar://problem/70541911

Fix possible crash in GraphicsLayerCA::computeVisibleAndCoverageRect()
​https://bugs.webkit.org/show_bug.cgi?id=217930
<rdar://problem/70316943>

Reviewed by Tim Horton.

If we made a m_overflowControlsHostLayerAncestorClippingStack, make sure we unparent
its layers when tearing down the RenderLayerBacking, and when we determine that we
longer need a m_ancestorClippingStack (having a m_overflowControlsHostLayerAncestorClippingStack
implies that we have a m_ancestorClippingStack).

• rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::destroyGraphicsLayers): (WebCore::RenderLayerBacking::updateAncestorClipping):

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

3:15 PM Changeset in webkit [268834] by Alan Coon

• 4 edits in branches/safari-610-branch

Cherry-pick r268602. rdar://problem/70541712

[WebAuthn] Remove the "alg" field in the attestation statement
​https://bugs.webkit.org/show_bug.cgi?id=217720
<rdar://problem/70349734>

Reviewed by Brent Fulgham.

Source/WebKit:

This old field was used in an unreleased old attestation statement format and is not used by
the final attestation statement format that will be added to the spec:
​https://github.com/w3c/webauthn/pull/1491.

Therefore, remove it to resolve confusion.

Covered by existing tests.

• UIProcess/WebAuthentication/Cocoa/LocalAuthenticator.mm: (WebKit::LocalAuthenticator::continueMakeCredentialAfterAttested):

LayoutTests:

• http/wpt/webauthn/public-key-credential-create-success-local.https.html:

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

3:15 PM Changeset in webkit [268833] by Alan Coon

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

Cherry-pick r268507. rdar://problem/70541950

[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:

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

3:15 PM Changeset in webkit [268832] by Alan Coon

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

Cherry-pick r268473. rdar://problem/70541789

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.

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

3:15 PM Changeset in webkit [268831] by Alan Coon

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

Cherry-pick r268084. rdar://problem/70541921

AX: Expose lineRangeForPosition for iOS Accessibility code
​https://bugs.webkit.org/show_bug.cgi?id=217256
<rdar://problem/67478962>

Reviewed by Zalan Bujtas.

Expose the existing lineRangeForPosition method for iOS accessibility.

• accessibility/ios/WebAccessibilityObjectWrapperIOS.mm: (-[WebAccessibilityObjectWrapper lineMarkersForMarker:]):

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

3:15 PM Changeset in webkit [268830] by Alan Coon

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

Cherry-pick r266787. rdar://problem/70541715

AccessibilityMenuList and MenuListPopup notifications need to be posted asynchronously.
​https://bugs.webkit.org/show_bug.cgi?id=216309
<rdar://problem/68108824>

Reviewed by Chris Fleizach.

MenuList notifications were posted synchronously which triggers a DOM
layout and style update in the middle of an ongoing DOM mutation update.
This is unnecessary and, furthermore, causes crashes since the DOM
layout update cannot be re-entrant. This change makes these
notifications asynchronous.

• accessibility/AccessibilityMenuList.cpp: (WebCore::AccessibilityMenuList::didUpdateActiveOption):
• accessibility/AccessibilityMenuListPopup.cpp: (WebCore::AccessibilityMenuListPopup::didUpdateActiveOption):

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

3:04 PM Changeset in webkit [268829] by Alan Coon

• 1 copy in tags/Safari-611.1.3.5

Tag Safari-611.1.3.5.

3:03 PM Changeset in webkit [268828] by Alan Coon

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

Versioning.

WebKit-7611.1.3.5

2:37 PM Changeset in webkit [268827] by clopez@igalia.com

• 3 edits in trunk/Tools

update-webkitgtk-libs and run-webkit-tests fail silently if flatpak has problems updating
​https://bugs.webkit.org/show_bug.cgi?id=218047

Reviewed by Philippe Normand.

This scripts were returning zero status when flatpak failed to
update in some cases. Fix them to return a non-zero status.

• Scripts/update-webkit-flatpak: Add missing sys.exit call with the return code.
• flatpak/flatpakutils.py:

(WebkitFlatpak.run_in_sandbox): In python false is zero when casted to int.

2:33 PM Changeset in webkit [268826] by Chris Dumez

• 9 edits in trunk/Source/WebCore

Update FFTFrame::realData() / imagData() to return an AudioFloatArray
​https://bugs.webkit.org/show_bug.cgi?id=218048

Reviewed by Philippe Normand.

Update FFTFrame::realData() / imagData() to return an AudioFloatArray instead of a float*.
It is risky to expose a float* since the caller has no way to know the actual size of the
array. By returning an AudioFloatArray, the caller can query AudioFloatArray::size() to
know the actual size. Note that it is not safe to assume that the size of these arrays
is FFTFrame::fftSize() because this is not true for ports using GStreamer.

• Modules/webaudio/PeriodicWave.cpp:

(WebCore::PeriodicWave::createBandLimitedTables):

• Modules/webaudio/RealtimeAnalyser.cpp:

(WebCore::RealtimeAnalyser::doFFTAnalysisIfNecessary):

• platform/audio/FFTFrame.cpp:

(WebCore::FFTFrame::interpolateFrequencyComponents):
(WebCore::FFTFrame::scaleFFT):
(WebCore::FFTFrame::multiply):
(WebCore::FFTFrame::extractAverageGroupDelay):
(WebCore::FFTFrame::addConstantGroupDelay):
(WebCore::FFTFrame::print):

• platform/audio/FFTFrame.h:

(WebCore::FFTFrame::realData):
(WebCore::FFTFrame::imagData):
(WebCore::FFTFrame::realData const):
(WebCore::FFTFrame::imagData const):

• platform/audio/FFTFrameStub.cpp:

(WebCore::FFTFrame::realData const): Deleted.
(WebCore::FFTFrame::imagData const): Deleted.

• platform/audio/gstreamer/FFTFrameGStreamer.cpp:

(WebCore::FFTFrame::FFTFrame):
(WebCore::FFTFrame::realData const): Deleted.
(WebCore::FFTFrame::imagData const): Deleted.

• platform/audio/mac/FFTFrameMac.cpp:

(WebCore::FFTFrame::FFTFrame):
(WebCore::FFTFrame::doFFT):
(WebCore::FFTFrame::realData const): Deleted.
(WebCore::FFTFrame::imagData const): Deleted.

2:24 PM Changeset in webkit [268825] by Alan Bujtas

• 3 edits in trunk/Source/WebCore

[LFC][Integration] Add content dependent vertical line snapping
​https://bugs.webkit.org/show_bug.cgi?id=218034

Reviewed by Antti Koivisto.

This patch ensures that IFC integration line snapping matches the legacy line layout snapping behavior (e.g. images prevent line snapping).

• layout/inlineformatting/InlineLineBox.h:

(WebCore::Layout::LineBox::InlineLevelBox::isAtomicInlineLevelBox const):

• layout/integration/LayoutIntegrationLineLayout.cpp:

(WebCore::LayoutIntegration::LineLayout::constructContent):

2:23 PM Changeset in webkit [268824] by Alan Coon

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

Versioning.

WebKit-7610.3.5

2:21 PM Changeset in webkit [268823] by Alan Bujtas

• 2 edits in trunk/Source/WebCore

[LFC][IFC] Apply font line-spacing to <br> inline level box
​https://bugs.webkit.org/show_bug.cgi?id=218043

Reviewed by Antti Koivisto.

The inline level box initiated by <br> should be vertically sized the same way as the root inline box is.
(e.g. <div>root text<br></div> <- line break's inline level box should match the layout bounds of the root inline box).

• layout/inlineformatting/InlineFormattingContextGeometry.cpp:

(WebCore::Layout::LineBoxBuilder::isRootInlineBox const):
(WebCore::Layout::LineBoxBuilder::isRootBox const):
(WebCore::Layout::LineBoxBuilder::setVerticalGeometryForInlineBox const):
(WebCore::Layout::LineBoxBuilder::constructInlineLevelBoxes):

2:15 PM Changeset in webkit [268822] by Chris Dumez

• 13 edits in trunk/Source/WebCore

Share more code between WorkerGlobalScope and WorkletGlobalScope
​https://bugs.webkit.org/show_bug.cgi?id=217995

Reviewed by Darin Adler.

Share more code between WorkerGlobalScope and WorkletGlobalScope by
moving shared logic to WorkerOrWorkletGlobalScope.

No new tests, no Web-facing behavior change.

• Modules/webaudio/AudioWorkletGlobalScope.cpp:

(WebCore::AudioWorkletGlobalScope::prepareForDestruction):

• Modules/webaudio/AudioWorkletGlobalScope.h:
• Modules/webaudio/AudioWorkletThread.cpp:

(WebCore::AudioWorkletThread::stop):

• workers/WorkerEventLoop.cpp:

(WebCore::WorkerEventLoop::create):
(WebCore::WorkerEventLoop::WorkerEventLoop):

• workers/WorkerEventLoop.h:
• workers/WorkerGlobalScope.cpp:

(WebCore::WorkerGlobalScope::WorkerGlobalScope):
(WebCore::WorkerGlobalScope::prepareForDestruction):
(WebCore::WorkerGlobalScope::removeAllEventListeners):
(WebCore::WorkerGlobalScope::suspend):
(WebCore::WorkerGlobalScope::resume):
(WebCore::WorkerGlobalScope::close):
(WebCore::WorkerGlobalScope::wrapCryptoKey):
(WebCore::WorkerGlobalScope::unwrapCryptoKey):
(WebCore::WorkerGlobalScope::thread const):

• workers/WorkerGlobalScope.h:
• workers/WorkerOrWorkletGlobalScope.cpp:

(WebCore::WorkerOrWorkletGlobalScope::WorkerOrWorkletGlobalScope):
(WebCore::WorkerOrWorkletGlobalScope::prepareForDestruction):
(WebCore::WorkerOrWorkletGlobalScope::disableEval):
(WebCore::WorkerOrWorkletGlobalScope::disableWebAssembly):
(WebCore::WorkerOrWorkletGlobalScope::isJSExecutionForbidden const):
(WebCore::WorkerOrWorkletGlobalScope::eventLoop):
(WebCore::WorkerOrWorkletGlobalScope::isContextThread const):
(WebCore::WorkerOrWorkletGlobalScope::postTask):

• workers/WorkerOrWorkletGlobalScope.h:

(WebCore::WorkerOrWorkletGlobalScope::isClosing const):
(WebCore::WorkerOrWorkletGlobalScope::workerOrWorkletThread const):
(WebCore::WorkerOrWorkletGlobalScope::markAsClosing):

• workers/WorkerThread.cpp:

(WebCore::WorkerThread::stop):

• worklets/WorkletGlobalScope.cpp:

(WebCore::WorkletGlobalScope::WorkletGlobalScope):
(WebCore::WorkletGlobalScope::prepareForDestruction):

• worklets/WorkletGlobalScope.h:

1:40 PM Changeset in webkit [268821] by Alan Bujtas

• 2 edits in trunk/Source/WebCore

[LFC][IFC] Line may be able to fit some more content when the current candidate is trimmable
​https://bugs.webkit.org/show_bug.cgi?id=218044

Reviewed by Antti Koivisto.

This patch addresses the cases when the candidate content is partially/fully trimmable and
after putting the content on the line, there's still enough room to accommodate some more content.

• layout/inlineformatting/InlineContentBreaker.cpp:

(WebCore::Layout::InlineContentBreaker::processOverflowingContent const):

1:27 PM Changeset in webkit [268820] by Chris Dumez

• 6 edits in trunk/Source/WebCore

Add addOutput() / removeOutput() utility functions to AudioSummingJunction
​https://bugs.webkit.org/show_bug.cgi?id=218045

Reviewed by Eric Carlson.

Add addOutput() / removeOutput() utility functions to AudioSummingJunction to add
or remove outputs from m_outputs and abstract away the call to changedOutputs().
It was awkward that subclasses were modifying m_outputs directly and had to
explicitly call changedOutputs() whenever they did.

No new tests, no web-facing behavior change.

• Modules/webaudio/AudioNode.cpp:

(WebCore::AudioNode::updateChannelsForInputs):

• Modules/webaudio/AudioNodeInput.cpp:

(WebCore::AudioNodeInput::connect):
(WebCore::AudioNodeInput::disconnect):
(WebCore::AudioNodeInput::disable):
(WebCore::AudioNodeInput::enable):
(WebCore::AudioNodeInput::numberOfChannels const):

• Modules/webaudio/AudioParam.cpp:

(WebCore::AudioParam::connect):
(WebCore::AudioParam::disconnect):

• Modules/webaudio/AudioSummingJunction.cpp:

(WebCore::AudioSummingJunction::markRenderingStateAsDirty):
(WebCore::AudioSummingJunction::addOutput):
(WebCore::AudioSummingJunction::removeOutput):
(WebCore::AudioSummingJunction::maximumNumberOfChannels const):
(WebCore::AudioSummingJunction::changedOutputs): Deleted.

• Modules/webaudio/AudioSummingJunction.h:

1:05 PM Changeset in webkit [268819] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebCore

Add lazy image loading to feature status page
​https://bugs.webkit.org/show_bug.cgi?id=208225

Patch by Rob Buis <​rbuis@igalia.com> on 2020-10-21
Reviewed by Simon Fraser.

Add status entry for lazy image loading.

• features.json:

12:34 PM Changeset in webkit [268818] by Antti Koivisto

• 2 edits in trunk/Source/WebCore

[LFC][Integration] Update style of contained layout boxes
​https://bugs.webkit.org/show_bug.cgi?id=218017
<rdar://problem/70531938>

Reviewed by Zalan Bujtas.

Followup fix.

• layout/integration/LayoutIntegrationLineLayout.cpp:

(WebCore::LayoutIntegration::LineLayout::containing):

12:29 PM Changeset in webkit [268817] by Alan Bujtas

• 2 edits in trunk/Source/WebCore

[LFC][Integration] Remove redundant vertical position snapping in the iterator interface
​https://bugs.webkit.org/show_bug.cgi?id=218039

Reviewed by Antti Koivisto.

Snapping is done during display content construction.

• layout/integration/LayoutIntegrationRunIteratorModernPath.h:

(WebCore::LayoutIntegration::RunIteratorModernPath::rect const):
(WebCore::LayoutIntegration::verticallyRoundedRect): Deleted.

12:21 PM Changeset in webkit [268816] by Peng Liu

• 2 edits in trunk/Source/WebCore

A video element may fail to enter picture-in-picture from fullscreen
​https://bugs.webkit.org/show_bug.cgi?id=217999

Reviewed by Eric Carlson.

When a video element is entering picture-in-picture from fullscreen,
WebKit should only fire the webkitendfullscreenEvent event, but should not
request the player in the UI process to exit fullscreen(picture-in-picture).
So the condition to decide sending the exit fullscreen request is wrong because
HTMLMediaElement::didBecomeFullscreenElement(), which sets m_waitingToEnterFullscreen
to false, might be called before dispatching the webkitendfullscreenEvent event.

• html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::dispatchEvent): Fix the condition.
(WebCore::HTMLMediaElement::exitFullscreen): Set fullscreen mode to VideoFullscreenModeNone.

12:19 PM Changeset in webkit [268815] by Alan Bujtas

• 4 edits in trunk/Source/WebCore

[LFC][Integration] Ascent and descent are rounded to integral position
​https://bugs.webkit.org/show_bug.cgi?id=218036

Reviewed by Antti Koivisto.

In this patch, we round ascent/descent vertical position values to integral to match legacy behavior.

• layout/inlineformatting/InlineFormattingContextGeometry.cpp:

(WebCore::Layout::LineBoxBuilder::constructInlineLevelBoxes):

• layout/inlineformatting/InlineLineBox.cpp:

(WebCore::Layout::LineBox::InlineLevelBox::setBaseline):
(WebCore::Layout::LineBox::InlineLevelBox::setDescent):
(WebCore::Layout::LineBox::InlineLevelBox::setLayoutBounds):

• layout/inlineformatting/InlineLineBox.h:

(WebCore::Layout::LineBox::InlineLevelBox::setBaseline): Deleted.
(WebCore::Layout::LineBox::InlineLevelBox::setDescent): Deleted.
(WebCore::Layout::LineBox::InlineLevelBox::setLayoutBounds): Deleted.

12:14 PM Changeset in webkit [268814] by Chris Dumez

• 4 edits
  6 adds in trunk

MessagePort & MessageEvent should be exposed to AudioWorklets
​https://bugs.webkit.org/show_bug.cgi?id=218037

Reviewed by Darin Adler.

Source/WebCore:

MessagePort & MessageEvent should be exposed to AudioWorklets:

• ​https://html.spec.whatwg.org/#the-messageevent-interface
• ​https://html.spec.whatwg.org/#message-ports

Test: http/wpt/webaudio/the-audio-api/the-audioworklet-interface/exposed-properties.https.html

• dom/MessageEvent.idl:
• dom/MessagePort.idl:

LayoutTests:

Add layout test coverage.

• http/wpt/webaudio/the-audio-api/the-audioworklet-interface/exposed-properties.https-expected.txt: Added.
• http/wpt/webaudio/the-audio-api/the-audioworklet-interface/exposed-properties.https.html: Added.
• http/wpt/webaudio/the-audio-api/the-audioworklet-interface/processors/exposed-properties-processor.js: Added.

(ExposedPropertiesTestProcessor.prototype.process):
(ExposedPropertiesTestProcessor):

11:43 AM Changeset in webkit [268813] by Devin Rousso

• 5 edits in trunk/Source/WebKitLegacy/mac

[macOS WK1] provide a way for Apple-internal clients to disable link preview
​https://bugs.webkit.org/show_bug.cgi?id=217984
<rdar://problem/70500022>

Reviewed by Tim Horton.

• WebView/WebViewPrivate.h:
• WebView/WebView.mm:

(-[WebView _allowsLinkPreview]): Added.
(-[WebView _setAllowsLinkPreview:]): Added.

• WebView/WebImmediateActionController.h:
• WebView/WebImmediateActionController.mm:

(-[WebImmediateActionController isEnabled]): Added.
(-[WebImmediateActionController setEnabled:]): Added.
(-[WebImmediateActionController _cancelImmediateAction]):

11:38 AM Changeset in webkit [268812] by Chris Dumez

• 22 edits in trunk

Make sure WebAudio API throws exceptions with useful error messages
​https://bugs.webkit.org/show_bug.cgi?id=218033

Reviewed by Darin Adler.

LayoutTests/imported/w3c:

Rebaseline existing tests now that exception messages have been improved.

• web-platform-tests/webaudio/the-audio-api/the-constantsourcenode-interface/constant-source-basic-expected.txt:
• web-platform-tests/webaudio/the-audio-api/the-convolvernode-interface/convolver-channels-expected.txt:
• web-platform-tests/webaudio/the-audio-api/the-convolvernode-interface/ctor-convolver-expected.txt:
• web-platform-tests/webaudio/the-audio-api/the-delaynode-interface/delaynode-maxdelaylimit-expected.txt:

Source/WebCore:

Make sure WebAudio API throws exceptions with useful error messages to be
more developer friendly.

No new tests, rebaselined existing tests.

• Modules/webaudio/AudioContext.cpp:

(WebCore::AudioContext::create):

• Modules/webaudio/AudioScheduledSourceNode.cpp:

(WebCore::AudioScheduledSourceNode::startLater):

• Modules/webaudio/AudioWorkletGlobalScope.cpp:

(WebCore::AudioWorkletGlobalScope::registerProcessor):

• Modules/webaudio/BaseAudioContext.cpp:

(WebCore::BaseAudioContext::createScriptProcessor):
(WebCore::BaseAudioContext::finishedRendering):

• Modules/webaudio/ConvolverNode.cpp:

(WebCore::ConvolverNode::setBuffer):

• Modules/webaudio/DefaultAudioDestinationNode.cpp:

(WebCore::DefaultAudioDestinationNode::startRendering):

• Modules/webaudio/DelayNode.cpp:

(WebCore::DelayNode::create):

• Modules/webaudio/OfflineAudioContext.cpp:

(WebCore::OfflineAudioContext::create):

• Modules/webaudio/OfflineAudioDestinationNode.cpp:

(WebCore::OfflineAudioDestinationNode::startRendering):

LayoutTests:

Rebaseline existing tests now that exception messages have been improved.

• webaudio/Oscillator/oscillator-basic-expected.txt:
• webaudio/ScriptProcessor/scriptprocessornode-expected.txt:
• webaudio/convolver-channels-expected.txt:
• webaudio/convolver-setBuffer-different-samplerate-expected.txt:
• webaudio/delaynode-maxdelaylimit-expected.txt:
• webaudio/dom-exceptions-expected.txt:

11:33 AM Changeset in webkit [268811] by Lauro Moura

• 5 edits in trunk

webkitpy: Check for duplicated keys in json expectation files
​https://bugs.webkit.org/show_bug.cgi?id=218032

Reviewed by Carlos Alberto Lopez Perez.

Tools:

When JSON contains repeated keys, Python only uses the last key/value
pair in the resulting dict, without warnings. This may cause some
expectations to be ignored, like what r267323 fixed.

This change also makes MockTestExpectations use the same loading
validation as the actual expectation instead of calling json.loads
directly.

• Scripts/webkitpy/common/test_expectations.py:

(check_repeated_keys): Added.
(TestExpectations.init): Use helper function.
(TestExpectations._load_expectation_string): Moved validation here.

• Scripts/webkitpy/common/test_expectations_unittest.py:

(MockTestExpectations.init): Use helper function.
(test_repeated_keys): Added test when there are repeated keys.

WebDriverTests:

• TestExpectations.json: Fix duplicated test key.

11:21 AM Changeset in webkit [268810] by Karl Rackler

• 2 edits in trunk/LayoutTests

REGRESSION: [ Win10 wk1 ews ] svg/clip-path/clip-path-invalid.svg is a flaky crash
​https://bugs.webkit.org/show_bug.cgi?id=218040

Unreviewed test gardening.

• platform/win/TestExpectations:

11:10 AM Changeset in webkit [268809] by graouts@webkit.org

• 3 edits
  2 adds in trunk

REGRESSION (r263729): transform transition doesn't restart
​https://bugs.webkit.org/show_bug.cgi?id=218011
Source/WebCore:

Reviewed by Dean Jackson.
<rdar://problem/70035130>

In the case of accelerated animations, style for the targeted element is not updated while the animation is
in flight since the animation is performed by Core Animation. This means that by the time the "transitionend"
event is fired, a transition has not yet performed style updates for the element that would have already been
performed for a software animations and thus the updates to the completed and running transition maps performed
under AnimationTimeline::updateCSSTransitionsForStyleableAndProperty() may be out of sync with the assumptions
made in DocumentTimelinesController::updateAnimationsAndSendEvents() that a transition newly marked as finished
should be added to the list of completed transitions.

Indeed, in the newly-added test, if a style update is forced during the "transitionend" event listener before also
clearing the transition styles, two style updates, and thus two calls to updateCSSTransitionsForStyleableAndProperty(),
will have been performed by the time updateAnimationsAndSendEvents() runs again and collects completed transitions,
during which time the transition in question will have been removed from the list of running transitions and added
to the list of completed transitions (first update) and subsequently removed from the list of completed transitions
(second update). At this point, setting styles that would start a new transition for this property will not yield a
transition since we won't be able to satisfy the requirement that the new target value does not match that of the
completed transition.

In this change we stop assuming that just because updateAnimationsAndSendEvents() sees a transition as finished for
the first time it implies that the transition is still considered a running transition. As such we only add a transition
to the list of completed transitions should it be in the list of running transitions.

Test: webanimations/transition-restart-after-style-recalc-during-transitionend.html

• animation/DocumentTimeline.cpp:

(WebCore::DocumentTimeline::transitionDidComplete):

LayoutTests:

<rdar://problem/70035130>

Reviewed by Dean Jackson.

Add a new test that checks that forcing a style update during dispatch of the "transitionend" event
does not prevent the completed transition from starting again if reset.

• webanimations/transition-restart-after-style-recalc-during-transitionend-expected.txt: Added.
• webanimations/transition-restart-after-style-recalc-during-transitionend.html: Added.

11:09 AM Changeset in webkit [268808] by graouts@webkit.org

• 5 edits in trunk/Source/WebCore

Rename hasRunningTransitionsForProperty() and hasCompletedTransitionsForProperty() to be singular
​https://bugs.webkit.org/show_bug.cgi?id=218014

Reviewed by Geoffrey Garen.

Since there is only running or completed transition per property, the name of these methods was misleading.

• animation/AnimationTimeline.cpp:

(WebCore::AnimationTimeline::updateCSSTransitionsForStyleableAndProperty):

• dom/Element.cpp:

(WebCore::Element::hasCompletedTransitionForProperty const):
(WebCore::Element::hasRunningTransitionForProperty const):
(WebCore::Element::hasCompletedTransitionsForProperty const): Deleted.
(WebCore::Element::hasRunningTransitionsForProperty const): Deleted.

• dom/Element.h:
• style/Styleable.h:

(WebCore::Styleable::hasCompletedTransitionForProperty const):
(WebCore::Styleable::hasRunningTransitionForProperty const):
(WebCore::Styleable::hasCompletedTransitionsForProperty const): Deleted.
(WebCore::Styleable::hasRunningTransitionsForProperty const): Deleted.

11:07 AM Changeset in webkit [268807] by graouts@webkit.org

• 3 edits in trunk/LayoutTests

[WK1] webanimations/accelerated-overlapping-transform-animations.html is a failure
​https://bugs.webkit.org/show_bug.cgi?id=217997
<rdar://problem/70505533>

Reviewed by Dean Jackson.

On WK1 UIHelper.ensureStablePresentationUpdate() does not guarantee frames have elapsed so we
wait a couple of animation frames in that configuration to ensure an accelerated animation has
been applied.

• platform/mac-wk1/TestExpectations:
• webanimations/accelerated-overlapping-transform-animations.html:

11:07 AM Changeset in webkit [268806] by Antti Koivisto

• 4 edits in trunk/Source/WebCore

[LFC][Integration] Use LineLayout::containing() in more places
​https://bugs.webkit.org/show_bug.cgi?id=218029

Reviewed by Zalan Bujtas.

• layout/integration/LayoutIntegrationRunIterator.cpp:

(WebCore::LayoutIntegration::firstTextRunFor):
(WebCore::LayoutIntegration::runFor):
(WebCore::LayoutIntegration::lineLayoutSystemFlowForRenderer): Deleted.

• rendering/RenderText.cpp:

(WebCore::RenderText::usesComplexLineLayoutPath const):
(WebCore::RenderText::layoutFormattingContextLineLayout const): Deleted.

• rendering/RenderText.h:

10:14 AM Changeset in webkit [268805] by youenn@apple.com

• 21 edits
  1 copy
  5 adds in trunk

WebRTC VP9 Decoder should be able to use VTB
​https://bugs.webkit.org/show_bug.cgi?id=217811

Reviewed by Eric Carlson.

Source/ThirdParty/libwebrtc:

Add RTCVideoDecoderVTBVP9 as a VP9 decoder using the VTB API.
Enable this code path by default if VP9 is enabled and hardware VP9 decoding is supported.
Allow to override this setting for testing purposes.

Add SPI header files in case they are not available.

• Configurations/libwebrtc.iOS.exp:
• Configurations/libwebrtc.iOSsim.exp:
• Configurations/libwebrtc.mac.exp:
• Source/webrtc/sdk/WebKit/CMBaseObjectSPI.h: Added.
• Source/webrtc/sdk/WebKit/VTVideoDecoderSPI.h: Added.
• Source/webrtc/sdk/WebKit/WebKitDecoder.mm:

(webrtc::createWebKitDecoderFactory):

• Source/webrtc/sdk/WebKit/WebKitUtilities.h:
• Source/webrtc/sdk/WebKit/WebKitVP9Decoder.cpp:

(webrtc::createWebKitVP9Decoder):

• Source/webrtc/sdk/objc/api/video_codec/RTCWrappedNativeVideoDecoder.mm:

(-[RTCWrappedNativeVideoDecoder implementationName]):

• Source/webrtc/sdk/objc/components/video_codec/RTCDefaultVideoDecoderFactory.h:
• Source/webrtc/sdk/objc/components/video_codec/RTCDefaultVideoDecoderFactory.m:

(-[RTCDefaultVideoDecoderFactory initWithH265:vp9:vp9VTB:]):
(-[RTCDefaultVideoDecoderFactory createDecoder:]):

• Source/webrtc/sdk/objc/components/video_codec/RTCVideoDecoderVTBVP9.h: Added.
• Source/webrtc/sdk/objc/components/video_codec/RTCVideoDecoderVTBVP9.mm: Added.

(RTCFrameDecodeParams::RTCFrameDecodeParams):
(VP9BufferToCMSampleBuffer):
(vp9DecompressionOutputCallback):
(-[RTCVideoDecoderVTBVP9 dealloc]):
(-[RTCVideoDecoderVTBVP9 startDecodeWithNumberOfCores:]):
(-[RTCVideoDecoderVTBVP9 decode:missingFrames:codecSpecificInfo:renderTimeMs:]):
(-[RTCVideoDecoderVTBVP9 decodeData:size:timeStamp:]):
(-[RTCVideoDecoderVTBVP9 setCallback:]):
(-[RTCVideoDecoderVTBVP9 setError:]):
(-[RTCVideoDecoderVTBVP9 releaseDecoder]):
(-[RTCVideoDecoderVTBVP9 resetDecompressionSession]):
(-[RTCVideoDecoderVTBVP9 configureDecompressionSession]):
(-[RTCVideoDecoderVTBVP9 destroyDecompressionSession]):
(-[RTCVideoDecoderVTBVP9 setVideoFormat:]):
(-[RTCVideoDecoderVTBVP9 implementationName]):

• Source/webrtc/sdk/objc/native/src/objc_video_decoder_factory.mm:

(webrtc::ObjCVideoDecoderFactory::CreateVideoDecoder):

• libwebrtc.xcodeproj/project.pbxproj:

Source/WebCore:

Add testing support to enable VP9 decoding through VTB for WebRTC.

Test: webrtc/vp9-vtb.html

• platform/mediastream/libwebrtc/LibWebRTCProvider.h:
• platform/mediastream/libwebrtc/LibWebRTCProviderCocoa.cpp:

(WebCore::LibWebRTCProviderCocoa::createDecoderFactory):

• testing/Internals.cpp:

(WebCore::Internals::setWebRTCVP9VTBSupport):

• testing/Internals.h:
• testing/Internals.idl:

LayoutTests:

• TestExpectations:
• platform/mac-wk1/TestExpectations:
• platform/mac-wk2/TestExpectations:
• webrtc/vp9-vtb-expected.txt: Added.
• webrtc/vp9-vtb.html: Added.

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

• 2 edits in trunk/Source/WebCore

[LFC][Integration] Disable image map content
​https://bugs.webkit.org/show_bug.cgi?id=218030

Reviewed by Antti Koivisto.

Image maps are not supported yet.

• layout/integration/LayoutIntegrationCoverage.cpp:

(WebCore::LayoutIntegration::canUseForLineLayoutWithReason):

10:02 AM Changeset in webkit [268803] by Hector Lopez

• 2 edits in trunk/LayoutTests

Add test expectation
​https://bugs.webkit.org/show_bug.cgi?id=217997

Unreviewed test gardening.

• platform/win/TestExpectations:

9:54 AM Changeset in webkit [268802] by Alan Coon

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

Cherry-pick r268771. rdar://problem/70532973

REGRESSION (r268483): Map jumps around while zooming on windy.com, strava.com
​https://bugs.webkit.org/show_bug.cgi?id=217987
<rdar://problem/70418548>

Reviewed by Simon Fraser.

Source/WebCore:

When several animations targetting the same property and the same layer are overlapping, we used to
always override the previous animations. With r268483 we started maintaining all active animations
and let them run, potentially with additivity if the animation could be broken into several animations
each targeting a given transform operation.

On top of that, with r268615 and the support for accelerated animation of individual CSS transform
properties (translate, scale and rotate), all transform-related animations were made additive.

This meant that we would always run active animations targeting "transform" in a way where they would be
additive rather than being replaced.

Any animation targeting "transform" will yield one or several accelerated animations, and the first of this
animation set will always have a 0 index. So now, when we compile a list of transform animations in
GraphicsLayerCA::updateAnimations(), we reset that list any time we encounted an animation with a 0 index,
ensuring only the top-most transform animation is applied.

We also fix an issue where we didn't account for the possibility that a single KeyframeEffect could yield
several transform animations with the same name in pauseAnimation() and removeAnimation(). We now pause or
remove all animations with the provided name.

Test: webanimations/accelerated-overlapping-transform-animations.html

• platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::pauseAnimation): (WebCore::GraphicsLayerCA::removeAnimation): (WebCore::GraphicsLayerCA::updateAnimations):

LayoutTests:

Add a new test that checks that only the last of two overlapping "transform" animations is applied.

• platform/mac-wk1/TestExpectations:
• webanimations/accelerated-overlapping-transform-animations-expected.html: Added.
• webanimations/accelerated-overlapping-transform-animations.html: Added.

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

9:40 AM Changeset in webkit [268801] by Chris Dumez

• 2 edits in trunk/Source/WebCore

Simplify ScriptExecutionContext::vm()
​https://bugs.webkit.org/show_bug.cgi?id=218028

Reviewed by Darin Adler.

• dom/ScriptExecutionContext.cpp:

(WebCore::ScriptExecutionContext::vm):

9:39 AM Changeset in webkit [268800] by Darin Adler

• 8 edits
  2 adds in trunk

REGRESSION(r266295): Range allows start and end containers to belong to different trees
​https://bugs.webkit.org/show_bug.cgi?id=217895

Reviewed by Ryosuke Niwa.

Source/WebCore:

Test: fast/dom/Range/ranges-across-trees.html

• dom/BoundaryPoint.h: Added treeOrder<TreeType>.

• dom/Node.cpp:

(WebCore::parent<Tree>): Added.
(WebCore::parent<ComposedTree>): Added.
(WebCore::depth): Changed into a template that takes TreeType.
(WebCore::commonInclusiveAncestorAndChildren): Ditto.
(WebCore::commonInclusiveAncestor): Changed to explicitly use ComposedTree to preserve
the current behavior, but likely will return later to make this a template and have it
us the normal tree by default.
(WebCore::treeOrder): Changed into a template that takes TreeType.
(WebCore::documentOrder): Call treeOrder<ComposedTree> to preserve the current behavior.
Likely will delete this later after changing callers to use treeOrder.

• dom/Node.h: Added Tree, ShadowIncludingTree, and ComposedTree. Added parent and

treeOrder function templates. TreeType currently is a set of classes but they could
also be objects of another type. Maybe an enumeration named TreeType instead?

• dom/Range.cpp:

(WebCore::Range::setStart): Use treeOrder instead of documentOrder to use the normal
tree instead of the composed tree.
(WebCore::Range::setEnd): Ditto.
(WebCore::Range::isPointInRange): Use isPointInRange<Tree> instead of isPointInRange
to use the normal tree instead of the composed tree.
(WebCore::Range::comparePoint const): Use treeOrder instead of documentOrder to use
the normal tree instead of the composed tree.
(WebCore::Range::compareNode const): Ditto.
(WebCore::Range::compareBoundaryPoints const): Ditto.
(WebCore::Range::intersectsNode const): Use intersects<Tree> instead of isPointInRange
to use the normal tree instead of the composed tree.

• dom/SimpleRange.cpp:

(WebCore::treeOrder): Changed into a template that takes TreeType.
(WebCore::documentOrder): Call treeOrder<ComposedTree> to preserve the current behavior.
Likely will delete this later after changing callers to use treeOrder.
(WebCore::isPointInRange): Changed into a template that takes TreeType.
For now the default tree type is still ComposedTree, but will change that later.
(WebCore::intersects): Ditto.
(WebCore::contains<Tree>): Added.
(WebCore::contains<ComposedTree>): Added.

• dom/SimpleRange.h: Added isPointInRange, intersects, and treeOrder function templates.

LayoutTests:

• fast/dom/Range/ranges-across-trees.html: Added.

9:36 AM Changeset in webkit [268799] by Hector Lopez

• 2 edits in trunk/LayoutTests

REGRESSION(r268476): [ macOS ] tiled-drawing/scrolling/non-fast-region/handlers-in-iframes.html is a flaky failure
​https://bugs.webkit.org/show_bug.cgi?id=218031

Unreviewed test gardening.

• platform/mac/TestExpectations:

9:21 AM Changeset in webkit [268798] by Antti Koivisto

• 11 edits in trunk/Source/WebCore

[LFC][Integration] Update style of contained layout boxes
​https://bugs.webkit.org/show_bug.cgi?id=218017

Reviewed by Zalan Bujtas.

Update layout box style on style change as needed. This fixes at least fast/replaced/max-width-percent.html with
image support enabled.

The patch also contains refactoring to make BoxTree non-const and moves the style update code there.

• layout/integration/LayoutIntegrationBoxTree.cpp:

(WebCore::LayoutIntegration::BoxTree::BoxTree):
(WebCore::LayoutIntegration::BoxTree::buildTree):
(WebCore::LayoutIntegration::BoxTree::updateStyle):
(WebCore::LayoutIntegration::BoxTree::layoutBoxForRenderer):
(WebCore::LayoutIntegration::BoxTree::layoutBoxForRenderer const):
(WebCore::LayoutIntegration::BoxTree::rendererForLayoutBox):
(WebCore::LayoutIntegration::BoxTree::rendererForLayoutBox const):

• layout/integration/LayoutIntegrationBoxTree.h:

(WebCore::LayoutIntegration::BoxTree::flow const):
(WebCore::LayoutIntegration::BoxTree::flow):

• layout/integration/LayoutIntegrationInlineContent.cpp:

(WebCore::LayoutIntegration::InlineContent::rendererForLayoutBox const):

• layout/integration/LayoutIntegrationInlineContent.h:
• layout/integration/LayoutIntegrationLineIteratorModernPath.h:

(WebCore::LayoutIntegration::LineIteratorModernPath::logicalStartRunWithNode const):
(WebCore::LayoutIntegration::LineIteratorModernPath::logicalEndRunWithNode const):

• layout/integration/LayoutIntegrationLineLayout.cpp:

(WebCore::LayoutIntegration::LineLayout::LineLayout):
(WebCore::LayoutIntegration::LineLayout::containing):
(WebCore::LayoutIntegration::LineLayout::updateReplacedDimensions):
(WebCore::LayoutIntegration::LineLayout::updateStyle):
(WebCore::LayoutIntegration::LineLayout::constructContent):
(WebCore::LayoutIntegration::LineLayout::textRunsFor const):
(WebCore::LayoutIntegration::LineLayout::runFor const):
(WebCore::LayoutIntegration::LineLayout::rendererForLayoutBox const):
(WebCore::LayoutIntegration::LineLayout::paint):
(WebCore::LayoutIntegration::LineLayout::hitTest):

• layout/integration/LayoutIntegrationLineLayout.h:
• layout/integration/LayoutIntegrationRunIteratorModernPath.h:

(WebCore::LayoutIntegration::RunIteratorModernPath::renderer const):

• rendering/RenderBlockFlow.cpp:

(WebCore::RenderBlockFlow::styleDidChange):

• rendering/RenderBox.cpp:

(WebCore::RenderBox::styleDidChange):

Update the line layout style.

9:15 AM Changeset in webkit [268797] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebCore

Un-needed casts in Screen::{width,height}()
​https://bugs.webkit.org/show_bug.cgi?id=218013

Patch by Philippe Normand <​pnormand@igalia.com> on 2020-10-21
Reviewed by Darin Adler.

• page/Screen.cpp:

(WebCore::Screen::height const): There is no need to convert from float to long and then
unsigned.
(WebCore::Screen::width const): Ditto.

9:01 AM Changeset in webkit [268796] by commit-queue@webkit.org

• 5 edits in trunk

Don't crash when deallocating WKWebView during TLS handshake
​https://bugs.webkit.org/show_bug.cgi?id=218025
<rdar://problem/70225969>

Patch by Alex Christensen <​achristensen@webkit.org> on 2020-10-21
Reviewed by Tim Horton.

Source/WebKit:

NetworkProcessProxy::didReceiveAuthenticationChallenge would sometimes dereference an unchecked
Optional<SecurityOriginData> which would result in a null dereference crash. Also, sometimes
Connection::initializeSendSource would assert because it was trying to set up a cancel handler for
a send port that had not been successfully set up yet. I added a test that reproduces both of these
issues most of the time.

• Platform/IPC/cocoa/ConnectionCocoa.mm:

(IPC::Connection::initializeSendSource):

• UIProcess/Network/NetworkProcessProxy.cpp:

(WebKit::NetworkProcessProxy::didReceiveAuthenticationChallenge):

Tools:

• TestWebKitAPI/Tests/WebKitCocoa/Challenge.mm:

(TEST):

8:04 AM Changeset in webkit [268795] by weinig@apple.com

• 7 edits in trunk/Tools

Cleanup DumpRenderTree in preparation for supporting arbitrary test header commands
​https://bugs.webkit.org/show_bug.cgi?id=217962

Reviewed by Darin Adler.

• Moves DumpRenderTree's TestOptions to be fully backed by TestFeatures like WebKitTestRunnner, allowing future changes to utilize any WebPreference once SPI is available from WebKitLegacy.

• Removes use of using namespace std; from DumpRenderTree.mm, which is not something we normally do and made the code a bit confusing.

• Move some random setting of preferences to centralized reset functions.

• DumpRenderTree/TestOptions.cpp:
• DumpRenderTree/TestOptions.h:
• DumpRenderTree/mac/DumpRenderTree.mm:
• DumpRenderTree/mac/TestRunnerMac.mm:
• DumpRenderTree/mac/UIDelegate.mm:
• DumpRenderTree/win/DumpRenderTree.cpp:

7:06 AM Changeset in webkit [268794] by caitp@igalia.com

• 26 edits
  14 adds in trunk

[JSC] support op_get_private_name in DFG and FTL
​https://bugs.webkit.org/show_bug.cgi?id=214861

Reviewed by Filip Pizlo.

JSTests:

• microbenchmarks/class-fields-private/monomorphic-get-private-field.js: Added.
• microbenchmarks/class-fields-private/polymorphic-get-private-field.js: Added.
• stress/dfg-get-private-name-by-id-generic.js: Added.
• stress/dfg-get-private-name-by-id-osr-bad-identifier.js: Added.
• stress/dfg-get-private-name-by-id.js: Added.
• stress/dfg-get-private-name-by-offset-osr-bad-identifier.js: Added.
• stress/dfg-get-private-name-by-offset-osr-bad-structure.js: Added.
• stress/dfg-get-private-name-by-offset.js: Added.
• stress/dfg-get-private-name-by-val-generic.js: Added.
• stress/ftl-get-private-name-by-id.js: Added.
• stress/ftl-get-private-name-by-offset-multi.js: Added.
• stress/get-private-name-with-constant-ident.js: Added.
• stress/get-private-name-with-constant-symbol.js: Added.
• stress/get-private-name-with-different-symbol.js: Added.

Source/JavaScriptCore:

Adds DFG/FTL support for op_get_private_name.

During DFG bytecode parsing, we will attempt, if deemed possible by
the information available, to output a GetByOffset operation. If a
single private field identifier is used in all cases (the common case),
but there are too many structure variants, a GetPrivateNameById
operation is emitted instead. Failing that, the GetPrivateName
operation is produced, which produces a GetByVal IC like in the
baseline JIT.

In FTL, GetPrivateNameByID can be reduced to [Multi]GetByOffset in the
DFGConstantFoldingPhase, or a GetByID IC when lowering to B3.

• bytecode/GetByStatus.cpp:

(JSC::GetByStatus::computeFromLLInt):

• bytecode/StructureStubInfo.h:

(JSC::appropriateOptimizingGetByIdFunction):
(JSC::appropriateGenericGetByIdFunction):

• dfg/DFGAbstractInterpreterInlines.h:

(JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects):

• dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::simplifyGetByStatus):
(JSC::DFG::ByteCodeParser::handleGetById):
(JSC::DFG::ByteCodeParser::handleGetPrivateNameById):
(JSC::DFG::ByteCodeParser::parseBlock):

• dfg/DFGClobberize.h:

(JSC::DFG::clobberize):

• dfg/DFGConstantFoldingPhase.cpp:

(JSC::DFG::ConstantFoldingPhase::foldConstants):

• dfg/DFGDoesGC.cpp:

(JSC::DFG::doesGC):

• dfg/DFGFixupPhase.cpp:

(JSC::DFG::FixupPhase::fixupNode):

• dfg/DFGNode.h:

(JSC::DFG::Node::convertToGetByOffset):
(JSC::DFG::Node::convertToMultiGetByOffset):
(JSC::DFG::Node::hasCacheableIdentifier):
(JSC::DFG::Node::hasHeapPrediction):

• dfg/DFGNodeType.h:
• dfg/DFGPredictionPropagationPhase.cpp:
• dfg/DFGSafeToExecute.h:

(JSC::DFG::safeToExecute):

• dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compileGetPrivateName):
(JSC::DFG::SpeculativeJIT::compileGetPrivateNameByVal):
(JSC::DFG::SpeculativeJIT::compileGetPrivateNameById):

• dfg/DFGSpeculativeJIT.h:
• dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::compile):

• dfg/DFGSpeculativeJIT64.cpp:

(JSC::DFG::SpeculativeJIT::compile):

• ftl/FTLCapabilities.cpp:

(JSC::FTL::canCompile):

• ftl/FTLLowerDFGToB3.cpp:

(JSC::FTL::DFG::LowerDFGToB3::compileNode):
(JSC::FTL::DFG::LowerDFGToB3::getPrivateName):
(JSC::FTL::DFG::LowerDFGToB3::compileGetPrivateName):
(JSC::FTL::DFG::LowerDFGToB3::compileGetPrivateNameById):

• jit/ICStats.h:
• jit/JITOperations.cpp:

(JSC::getPrivateName):
(JSC::JSC_DEFINE_JIT_OPERATION):

• jit/JITOperations.h:
• jit/Repatch.cpp:

(JSC::appropriateOptimizingGetByFunction):
(JSC::appropriateGetByFunction):
(JSC::tryCacheGetBy):

• jit/Repatch.h:
• runtime/OptionsList.h:

5:36 AM Changeset in webkit [268793] by Carlos Garcia Campos

• 23 edits in trunk

WebDriver: add support for wheel actions
​https://bugs.webkit.org/show_bug.cgi?id=217174

Reviewed by Brian Burg.

.:

Enable WEBDRIVER_WHEEL_INTERACTIONS for GTK and WPE ports.

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

Source/WebDriver:

Handle wheel actions.

• Actions.h:
• Session.cpp:

(WebDriver::automationSourceType): Handle InputSource::Type::Wheel.
(WebDriver::Session::performActions): Handle Action::Type::Wheel.

• WebDriverService.cpp:

(WebDriver::processKeyAction): Assert if Action::Subtype::Scroll.
(WebDriver::processPointerMoveAction): Move this code to a helper to be used by both pointer move and scroll actions.
(WebDriver::processPointerAction): Use processPointerMoveAction().
(WebDriver::processWheelAction): Call processPointerMoveAction() and process the scroll delta too.
(WebDriver::processInputActionSequence): Handle InputSource::Type::Wheel.

Source/WebKit:

• UIProcess/Automation/Automation.json: Add scroll delta to action state.
• UIProcess/Automation/SimulatedInputDispatcher.cpp:

(WebKit::SimulatedInputSourceState::emptyStateForSourceType): Initialize scrollDelta for wheel actions.
(WebKit::SimulatedInputDispatcher::transitionInputSourceToState): Handle SimulatedInputSourceType::Wheel.

• UIProcess/Automation/SimulatedInputDispatcher.h:
• UIProcess/Automation/WebAutomationSession.cpp:

(WebKit::WebAutomationSession::WebAutomationSession): Add SimulatedInputSourceType::Wheel.
(WebKit::WebAutomationSession::terminate): Handle pending wheel events.
(WebKit::WebAutomationSession::willShowJavaScriptDialog): Ditto.
(WebKit::WebAutomationSession::wheelEventsFlushedForPage): Ditto.
(WebKit::WebAutomationSession::willClosePage): Ditto.
(WebKit::WebAutomationSession::isSimulatingUserInteraction const): Return true if there are pending wheel events too.
(WebKit::WebAutomationSession::simulateWheelInteraction): Handle the wheel action.
(WebKit::simulatedInputSourceTypeFromProtocolSourceType): Handle Inspector::Protocol::Automation::InputSourceType::Wheel.
(WebKit::WebAutomationSession::performInteractionSequence): Initialize the scroll delta for wheel action.

• UIProcess/Automation/WebAutomationSession.h:
• UIProcess/Automation/gtk/WebAutomationSessionGtk.cpp:

(WebKit::WebAutomationSession::platformSimulateWheelInteraction): Synthesize a wheel event.

• UIProcess/Automation/wpe/WebAutomationSessionWPE.cpp:

(WebKit::WebAutomationSession::platformSimulateWheelInteraction): Ditto.

• UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::isProcessingWheelEvents const): Return whether page has pending wheel events.
(WebKit::WebPageProxy::didReceiveEvent): Notify automation that pending wheel events have been processed.

• UIProcess/WebPageProxy.h:
• config.h:

Tools:

Add webdriver-wheel-interactions option.

• Scripts/webkitperl/FeatureList.pm:

WebDriverTests:

Remove expectations for wheel actions test.

• TestExpectations.json:

5:26 AM Changeset in webkit [268792] by svillar@igalia.com

• 8 edits in trunk

[css-flexbox] flex-basis not animatable
​https://bugs.webkit.org/show_bug.cgi?id=180435
LayoutTests/imported/w3c:

<rdar://problem/59363784>

Reviewed by Antoine Quint.

• web-platform-tests/css/css-flexbox/animation/flex-basis-composition-expected.txt: Replaced FAIL expectations with PASS.
• web-platform-tests/css/css-flexbox/animation/flex-basis-interpolation-expected.txt: Ditto.
• web-platform-tests/web-animations/animation-model/animation-types/accumulation-per-property-001-expected.txt: Ditto.
• web-platform-tests/web-animations/animation-model/animation-types/addition-per-property-001-expected.txt: Ditto.
• web-platform-tests/web-animations/animation-model/animation-types/interpolation-per-property-001-expected.txt: Ditto.

Source/WebCore:

Reviewed by Antoine Quint.

Make flex-basis property animatable as per spec. This allows us to pass ~100 flexbox animation subtests.

• animation/CSSPropertyAnimation.cpp:

(WebCore::CSSPropertyAnimationWrapperMap::CSSPropertyAnimationWrapperMap): Mark flex-basis as animatable.

4:32 AM Changeset in webkit [268791] by youenn@apple.com

• 4 edits in trunk

[iOS] Disable audio capture in the background for non Safari applications
​https://bugs.webkit.org/show_bug.cgi?id=217948
<rdar://problem/70241557>

Reviewed by Eric Carlson.

Source/WebCore:

We do not have good OS support when capturing audio in a WebProcess for a backgrounded application.
Until we have proper support, it seems best to mute audio capture when being backgrounded.
Manually tested.

• platform/mediastream/mac/RealtimeMediaSourceCenterMac.mm:

(WebCore::RealtimeMediaSourceCenter::shouldInterruptAudioOnPageVisibilityChange):

LayoutTests:

• platform/ios/mediastream/video-muted-in-background-tab.html:

By default, audio capture will be muted in WTR if page goes to background.
Update the test to explicitly request for audio capture to continue while in background.

3:50 AM Changeset in webkit [268790] by Philippe Normand

• 3 edits in trunk/Source/WebCore

[GStreamer] Create video sink only for video player
​https://bugs.webkit.org/show_bug.cgi?id=217974

Reviewed by Xabier Rodriguez-Calvar.

There's no need for a video sink when the media player is created for an <audio> element.

• platform/graphics/gstreamer/GStreamerCommon.cpp:

(WebCore::initializeGStreamerAndRegisterWebKitElements): Downrank the GL video sink, as we
create it on-demand and not through autovideosink anyway.

• platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:

(WebCore::MediaPlayerPrivateGStreamer::createGSTPlayBin): Avoid creating useless video sink.

1:17 AM Changeset in webkit [268789] by Diego Pino Garcia

• 3 edits in trunk/LayoutTests

[GLIB][GTK] Unreviewed test gardening. Update test expectations.

Gardened several audioworklet test that are flaky in GTK. Also skip
non WPT MediaSession tests in GLIB.

• platform/glib/TestExpectations:
• platform/gtk/TestExpectations:

1:02 AM Changeset in webkit [268788] by Antti Koivisto

• 2 edits
  2 adds in trunk/LayoutTests/imported/w3c

Import web-platform-tests/css/selectors/is-where-basic.html
​https://bugs.webkit.org/show_bug.cgi?id=218009

Reviewed by Tim Horton.

One more :is/:where test.

• web-platform-tests/css/selectors/is-where-basic-expected.txt: Added.
• web-platform-tests/css/selectors/is-where-basic.html: Added.
• web-platform-tests/css/selectors/w3c-import.log:

12:32 AM Changeset in webkit [268787] by Lauro Moura

• 2 edits in trunk/Tools

webkitpy: Fix webdriver logging message
​https://bugs.webkit.org/show_bug.cgi?id=218006

Reviewed by Carlos Garcia Campos.

• Scripts/webkitpy/webdriver_tests/webdriver_test_runner.py:

(WebDriverTestRunner.process_results): Use the test name instead of
the dir name.

Oct 20, 2020:

11:25 PM Changeset in webkit [268786] by Devin Rousso

• 2 edits in trunk/Source/WebInspectorUI

Web Inspector: REGRESSION(r266074): Uncaught Exception: undefined is not an object (evaluating 'this._allListenersBreakpoint.disabled')
​https://bugs.webkit.org/show_bug.cgi?id=217992

Reviewed by Joseph Pecoraro.

r266074 changed it such that _allListenersBreakpoint doesn't always exist, so check that
it exists before getting a property from it.

• UserInterface/Controllers/DOMDebuggerManager.js:

(WI.DOMDebuggerManager.prototype.listenerBreakpointForEventName):

11:09 PM Changeset in webkit [268785] by Diego Pino Garcia

• 3 edits in trunk/LayoutTests

[GLIB][GTK] Unreviewwed test gardening. Update test expectations.

Added more flaky crash failures in GTK and updated MediaSession
expectations for GLIB.

• platform/glib/TestExpectations:
• platform/gtk/TestExpectations:

9:25 PM Changeset in webkit [268784] by Lauro Moura

• 2 edits in trunk/WebDriverTests

[WebDriver] Gardening WPE no browsing context failures

Unreviewed test gardening.

These failures are related to the lack of multi window support in
current WPE MiniBrowser code.

After this, a number of unexpected passes will appear due to
test_no_browsing_context failing in the teardown step and pytest
recording two runs (the actual test XPASSes but the teardown XFAILs).

• TestExpectations.json:

9:00 PM Changeset in webkit [268783] by sbarati@apple.com

• 6 edits
  1 add in trunk

Don't OSR exit to bc#0 for FTL argument type checks during loop OSR entry
​https://bugs.webkit.org/show_bug.cgi?id=217925
<rdar://problem/70369407>

Reviewed by Michael Saboff and Tadeu Zagallo.

JSTests:

• stress/ftl-osr-entry-should-not-exit-to-bc-zero.js: Added.

Source/JavaScriptCore:

When the FTL was emitting type checks for the named arguments of a function,
it was always emitting these type checks with an exit origin of bc#0. It was
doing this even if we were an OSR entry compilation! This meant that type
checks for arguments that failed during loop OSR entry would incorrectly exit
back to bc#0.

This patch fixes this by having the OSR entry runtime code validate the
argument types before OSR entering. The current OSR entry compiled code in
the FTL is designed to only allow exiting after all ExtractOSREntryLocal and
MovHints have executed, so it is simpler to put the type checks in the runtime
instead of the compiled code.

This patch also makes it so we do exponential backoff when failing to OSR
enter. This is needed due to insufficient profiling where we never properly
profile the type of arguments. Before this, we'd OSR exit in the FTL code
itself, which does exponential backoff when recompiling. This patch builds
this same exponential backoff in for when we fail to OSR enter enough times
to give up on the OSR entry compilation.

• ftl/FTLForOSREntryJITCode.h:
• ftl/FTLLowerDFGToB3.cpp:

(JSC::FTL::DFG::LowerDFGToB3::lower):

• ftl/FTLOSREntry.cpp:

(JSC::FTL::prepareOSREntry):

8:33 PM Changeset in webkit [268782] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebCore

Release assert in ~Node due to render element of pseudo element not getting removed in time
​https://bugs.webkit.org/show_bug.cgi?id=217996

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

When attaching a shadow root to an element, remember to tear down renderers not only if
one exists on the element, but also if it uses display: contents.

• dom/Element.cpp:

(WebCore::Element::addShadowRoot):

7:25 PM Changeset in webkit [268781] by Hector Lopez

• 2 edits in trunk/LayoutTests

Test expectation adjustment.
​https://bugs.webkit.org/show_bug.cgi?id=217620

Unreiviewed test gardening.

• platform/mac/TestExpectations:

6:40 PM Changeset in webkit [268780] by sihui_liu@apple.com

• 4 edits in trunk/Source

Add stubs for SpeechRecognition
​https://bugs.webkit.org/show_bug.cgi?id=217780
<rdar://problem/70350727>

Source/WebCore:

Unreviewed build fix after r268762.

• Modules/speech/SpeechRecognitionResultList.h:

Source/WTF:

Unreviewed build fix after r268762, which sets ENABLE_SPEECH_SYNTHESIS by mistake.

• wtf/PlatformEnable.h:

6:24 PM Changeset in webkit [268779] by Alan Coon

• 1 copy in tags/Safari-611.1.3.4

Tag Safari-611.1.3.4.

6:14 PM Changeset in webkit [268778] by Hector Lopez

• 2 edits in trunk/LayoutTests

Test expectation adjustment
​https://bugs.webkit.org/show_bug.cgi?id=217669

Unreviewed test gardening.

• platform/mac/TestExpectations:

6:05 PM Changeset in webkit [268777] by Karl Rackler

• 1 edit
  1 add in trunk/LayoutTests

Rebaseline test for Big Sur after the changes in r268750
​https://bugs.webkit.org/show_bug.cgi?id=209813

Unreviewed test gardening.

• platform/mac-bigsur/imported/w3c/web-platform-tests/mathml/relations/css-styling/ignored-properties-001-expected.txt: Added.

5:07 PM Changeset in webkit [268776] by Russell Epstein

• 1 copy in tags/Safari-610.2.11.51.4

Tag Safari-610.2.11.51.4.

5:00 PM Changeset in webkit [268775] by Chris Dumez

• 26 edits
  1 copy
  1 move
  5 deletes in trunk/Source/WebCore

Merge WorkerScriptController and WorkletScriptController into WorkerOrWorkletScriptController
​https://bugs.webkit.org/show_bug.cgi?id=217980

Reviewed by Darin Adler.

Merge WorkerScriptController and WorkletScriptController into WorkerOrWorkletScriptController
to share more code between workers and worklets.

No new tests, no Web-facing behavior change.

• Modules/webaudio/AudioWorkletThread.cpp:

(WebCore::AudioWorkletThread::workletThread):

• Sources.txt:
• WebCore.xcodeproj/project.pbxproj:
• bindings/js/JSDOMGlobalObject.cpp:

(WebCore::JSDOMGlobalObject::clearDOMGuardedObjects):
(WebCore::toJSDOMGlobalObject):

• bindings/js/JSDOMGlobalObject.h:
• bindings/js/JSDOMWindowBase.cpp:

(WebCore::JSDOMWindowBase::JSDOMWindowBase):
(WebCore::JSDOMWindowBase::finishCreation):
(WebCore::JSDOMWindowBase::proxy const):

• bindings/js/JSDOMWindowBase.h:
• bindings/js/JSWorkerGlobalScopeBase.cpp:

(WebCore::toJS):

• bindings/js/JSWorkerGlobalScopeBase.h:

(WebCore::JSWorkerGlobalScopeBase::wrapped const):

• bindings/js/JSWorkletGlobalScopeBase.cpp:

(WebCore::toJS):

• bindings/js/JSWorkletGlobalScopeBase.h:

(WebCore::JSWorkletGlobalScopeBase::wrapped const):

• bindings/js/ScheduledAction.cpp:

(WebCore::ScheduledAction::execute):

• bindings/js/ScriptState.cpp:
• bindings/js/WorkerScriptController.cpp: Removed.
• bindings/js/WorkerScriptController.h: Removed.
• dom/ScriptExecutionContext.cpp:
• testing/js/WebCoreTestSupport.cpp:

(WebCoreTestSupport::setupNewlyCreatedServiceWorker):

• workers/DedicatedWorkerGlobalScope.h:

(isType):

• workers/WorkerConsoleClient.cpp:

(WebCore::WorkerConsoleClient::WorkerConsoleClient):
(WebCore::WorkerConsoleClient::messageWithTypeAndLevel):
(WebCore::WorkerConsoleClient::count):
(WebCore::WorkerConsoleClient::countReset):
(WebCore::WorkerConsoleClient::time):
(WebCore::WorkerConsoleClient::timeLog):
(WebCore::WorkerConsoleClient::timeEnd):

• workers/WorkerConsoleClient.h:
• workers/WorkerGlobalScope.cpp:

(WebCore::WorkerGlobalScope::WorkerGlobalScope):
(WebCore::WorkerGlobalScope::disableEval):
(WebCore::WorkerGlobalScope::disableWebAssembly):
(WebCore::WorkerGlobalScope::importScripts):
(WebCore::WorkerGlobalScope::isJSExecutionForbidden const):

• workers/WorkerGlobalScope.h:
• workers/WorkerOrWorkletGlobalScope.cpp: Copied from Source/WebCore/workers/WorkerOrWorkletScriptController.h.

(WebCore::WorkerOrWorkletGlobalScope::WorkerOrWorkletGlobalScope):

• workers/WorkerOrWorkletGlobalScope.h:

(WebCore::WorkerOrWorkletGlobalScope::script const):
(WebCore::WorkerOrWorkletGlobalScope::clearScript):

• workers/WorkerOrWorkletScriptController.cpp: Renamed from Source/WebCore/worklets/WorkletScriptController.cpp.

(WebCore::WorkerOrWorkletScriptController::WorkerOrWorkletScriptController):
(WebCore::WorkerOrWorkletScriptController::~WorkerOrWorkletScriptController):
(WebCore::WorkerOrWorkletScriptController::attachDebugger):
(WebCore::WorkerOrWorkletScriptController::detachDebugger):
(WebCore::WorkerOrWorkletScriptController::forbidExecution):
(WebCore::WorkerOrWorkletScriptController::isExecutionForbidden const):
(WebCore::WorkerOrWorkletScriptController::scheduleExecutionTermination):
(WebCore::WorkerOrWorkletScriptController::isTerminatingExecution const):
(WebCore::WorkerOrWorkletScriptController::releaseHeapAccess):
(WebCore::WorkerOrWorkletScriptController::acquireHeapAccess):
(WebCore::WorkerOrWorkletScriptController::addTimerSetNotification):
(WebCore::WorkerOrWorkletScriptController::removeTimerSetNotification):
(WebCore::WorkerOrWorkletScriptController::setException):
(WebCore::WorkerOrWorkletScriptController::disableEval):
(WebCore::WorkerOrWorkletScriptController::disableWebAssembly):
(WebCore::WorkerOrWorkletScriptController::evaluate):
(WebCore::WorkerOrWorkletScriptController::initScriptWithSubclass):
(WebCore::WorkerOrWorkletScriptController::initScript):

• workers/WorkerOrWorkletScriptController.h:

(WebCore::WorkerOrWorkletScriptController::globalScopeWrapper):
(WebCore::WorkerOrWorkletScriptController::vm):
(WebCore::WorkerOrWorkletScriptController::globalScope const):
(WebCore::WorkerOrWorkletScriptController::initScriptIfNeeded):

• workers/WorkerThread.cpp:

(WebCore::WorkerThread::workerThread):

• worklets/WorkletConsoleClient.cpp: Removed.
• worklets/WorkletConsoleClient.h: Removed.
• worklets/WorkletGlobalScope.cpp:

(WebCore::WorkletGlobalScope::WorkletGlobalScope):
(WebCore::WorkletGlobalScope::~WorkletGlobalScope):
(WebCore::WorkletGlobalScope::prepareForDestruction):
(WebCore::WorkletGlobalScope::evaluate):
(WebCore::WorkletGlobalScope::isJSExecutionForbidden const):
(WebCore::WorkletGlobalScope::disableEval):
(WebCore::WorkletGlobalScope::disableWebAssembly):
(WebCore::WorkletGlobalScope::notifyFinished):

• worklets/WorkletGlobalScope.h:
• worklets/WorkletScriptController.h: Removed.

4:58 PM Changeset in webkit [268774] by Megan Gardner

• 29 edits
  6 moves in trunk

Rename HighlightMap to HighlightRegister and HighlightRangeGroup to Highlight to match current spec
​https://bugs.webkit.org/show_bug.cgi?id=217919

Reviewed by Ryosuke Niwa.

No new tests, no new behavior, rename only.
​https://drafts.csswg.org/css-highlight-api-1/

• DerivedSources-input.xcfilelist:
• DerivedSources-output.xcfilelist:
• DerivedSources.make:
• Modules/highlight/Highlight.cpp: Renamed from Source/WebCore/Modules/highlight/HighlightRangeGroup.cpp.

(WebCore::Highlight::Highlight):
(WebCore::Highlight::create):
(WebCore::Highlight::initializeSetLike):
(WebCore::Highlight::removeFromSetLike):
(WebCore::Highlight::clearFromSetLike):
(WebCore::Highlight::addToSetLike):

• Modules/highlight/Highlight.h: Renamed from Source/WebCore/Modules/highlight/HighlightRangeGroup.h.
• Modules/highlight/Highlight.idl: Renamed from Source/WebCore/Modules/highlight/HighlightRangeGroup.idl.
• Modules/highlight/HighlightRegister.cpp: Renamed from Source/WebCore/Modules/highlight/HighlightMap.cpp.

(WebCore::HighlightRegister::initializeMapLike):
(WebCore::HighlightRegister::setFromMapLike):
(WebCore::HighlightRegister::clear):
(WebCore::HighlightRegister::remove):

• Modules/highlight/HighlightRegister.h: Renamed from Source/WebCore/Modules/highlight/HighlightMap.h.

(WebCore::HighlightRegister::create):
(WebCore::HighlightRegister::map const):

• Modules/highlight/HighlightRegister.idl: Renamed from Source/WebCore/Modules/highlight/HighlightMap.idl.
• Sources.txt:
• WebCore.xcodeproj/project.pbxproj:
• bindings/js/WebCoreBuiltinNames.h:
• css/DOMCSSNamespace.cpp:

(WebCore::DOMCSSNamespace::highlights):

• css/DOMCSSNamespace.h:
• css/DOMCSSNamespace.idl:
• dom/Document.cpp:

(WebCore::Document::commonTeardown):
(WebCore::Document::highlightRegister):
(WebCore::Document::updateHighlightPositions):
(WebCore::Document::highlightMap): Deleted.

• dom/Document.h:
• rendering/HighlightData.cpp:
• rendering/InlineTextBox.cpp:

(WebCore::InlineTextBox::collectMarkedTextsForHighlights const):

• rendering/RenderReplaced.cpp:

(WebCore::RenderReplaced::calculateHighlightColor const):

• rendering/SelectionRangeData.cpp:

4:33 PM Changeset in webkit [268773] by msaboff@apple.com

• 15 edits in trunk

[JSC] Update RegExp UCD to version 13.0
​https://bugs.webkit.org/show_bug.cgi?id=217975

Reviewed by Yusuke Suzuki.

JSTests:

Updated test expectations for tests that now pass based on the updating to UCD 13.0.

• test262/expectations.yaml:

Source/JavaScriptCore:

UCD 13.0 data files and an update to the generated file's copyright.

• ucd/CaseFolding.txt:
• ucd/DerivedBinaryProperties.txt:
• ucd/DerivedCoreProperties.txt:
• ucd/DerivedNormalizationProps.txt:
• ucd/PropList.txt:
• ucd/PropertyAliases.txt:
• ucd/PropertyValueAliases.txt:
• ucd/ScriptExtensions.txt:
• ucd/Scripts.txt:
• ucd/UnicodeData.txt:
• ucd/emoji-data.txt:
• yarr/generateYarrUnicodePropertyTables.py:

4:14 PM Changeset in webkit [268772] by aakash_jain@apple.com

• 2 edits in trunk/Tools

report-non-inclusive-language should do exact match for directory names to ignore
​https://bugs.webkit.org/show_bug.cgi?id=217993

Reviewed by Darin Adler.

• Scripts/report-non-inclusive-language:

3:58 PM Changeset in webkit [268771] by graouts@webkit.org

• 4 edits
  2 adds in trunk

REGRESSION (r268483): Map jumps around while zooming on windy.com, strava.com
​https://bugs.webkit.org/show_bug.cgi?id=217987
<rdar://problem/70418548>

Reviewed by Simon Fraser.

Source/WebCore:

When several animations targetting the same property and the same layer are overlapping, we used to
always override the previous animations. With r268483 we started maintaining all active animations
and let them run, potentially with additivity if the animation could be broken into several animations
each targeting a given transform operation.

On top of that, with r268615 and the support for accelerated animation of individual CSS transform
properties (translate, scale and rotate), all transform-related animations were made additive.

This meant that we would always run active animations targeting "transform" in a way where they would be
additive rather than being replaced.

Any animation targeting "transform" will yield one or several accelerated animations, and the first of this
animation set will always have a 0 index. So now, when we compile a list of transform animations in
GraphicsLayerCA::updateAnimations(), we reset that list any time we encounted an animation with a 0 index,
ensuring only the top-most transform animation is applied.

We also fix an issue where we didn't account for the possibility that a single KeyframeEffect could yield
several transform animations with the same name in pauseAnimation() and removeAnimation(). We now pause or
remove all animations with the provided name.

Test: webanimations/accelerated-overlapping-transform-animations.html

• platform/graphics/ca/GraphicsLayerCA.cpp:

(WebCore::GraphicsLayerCA::pauseAnimation):
(WebCore::GraphicsLayerCA::removeAnimation):
(WebCore::GraphicsLayerCA::updateAnimations):

LayoutTests:

Add a new test that checks that only the last of two overlapping "transform" animations is applied.

• platform/mac-wk1/TestExpectations:
• webanimations/accelerated-overlapping-transform-animations-expected.html: Added.
• webanimations/accelerated-overlapping-transform-animations.html: Added.

3:47 PM Changeset in webkit [268770] by Truitt Savell

• 2 edits in trunk/LayoutTests

imported/w3c/web-platform-tests/webaudio/the-audio-api/the-audioworklet-interface/audioworkletnode-output-channel-count.https.html is a flaky failure
​https://bugs.webkit.org/show_bug.cgi?id=217994

Unreviewed test gardening.

• platform/mac/TestExpectations:

3:41 PM Changeset in webkit [268769] by jer.noble@apple.com

• 10 edits
  5 adds in trunk

[media-session] Basic support for MediaSession.setPositionState() and MediaSession.setActionHandler()
​https://bugs.webkit.org/show_bug.cgi?id=217963

Reviewed by Eric Carlson.

Source/WebCore:

Tests: media/media-session/mock-actionHandlers.html

media/media-session/mock-currentPosition.html

Add basic support for setPositionState() and currentTime calculations of the Media Session
standard, basic support for setActionHandler(), and Internals methods to query the internal
state of both those APIs from layout tests.

• Modules/mediasession/MediaSession.cpp:

(WebCore::MediaSession::setMetadata):
(WebCore::MediaSession::setPlaybackState):
(WebCore::MediaSession::setActionHandler):
(WebCore::MediaSession::handlerForAction const):
(WebCore::MediaSession::setPositionState):
(WebCore::MediaSession::currentPosition const):

• Modules/mediasession/MediaSession.h:

(WebCore::MediaSession::playbackState const):

• Modules/mediasession/MediaSession.idl:
• Modules/mediasession/MediaSessionAction.h:
• Modules/mediasession/MediaSessionActionDetails.idl:
• testing/Internals.cpp:

(WebCore::Internals::currentMediaSessionPosition):
(WebCore::Internals::sendMediaSessionAction):

• testing/Internals.h:
• testing/Internals.idl:

LayoutTests:

• media/media-session/mock-actionHandlers-expected.txt: Added.
• media/media-session/mock-actionHandlers.html: Added.
• media/media-session/mock-currentPosition-expected.txt: Added.
• media/media-session/mock-currentPosition.html: Added.

3:33 PM Changeset in webkit [268768] by aakash_jain@apple.com

• 2 edits in trunk/Tools

[build.webkit.org] Rename slavename to workername in steps.py
​https://bugs.webkit.org/show_bug.cgi?id=217988

Reviewed by Dewei Zhu.

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

3:25 PM Changeset in webkit [268767] by Diego Pino Garcia

• 3 edits in trunk/LayoutTests

[GLIB][GTK] Unreviewed test gardening. Gardened several flaky crash failures.

• platform/glib/TestExpectations:
• platform/gtk/TestExpectations:

3:20 PM Changeset in webkit [268766] by don.olmstead@sony.com

• 5 edits in trunk/Source/WebCore

[WebGPU] Increase portability of GPUBindGroup
​https://bugs.webkit.org/show_bug.cgi?id=217978

Reviewed by Myles C. Maxfield.

Remove more uses of USE(METAL) around the GPUBindGroup related constructs to increase
portability. Introduce a platform specific type for an offset into the GPU Buffer.

No new tests. No change in behavior.

• platform/graphics/gpu/GPUBindGroup.h:

(WebCore::GPUBindGroup::argumentBuffer const):

• platform/graphics/gpu/GPUBindGroupAllocator.h:

(WebCore::GPUBindGroupAllocator::argumentBuffer const):

• platform/graphics/gpu/cocoa/GPUBindGroupAllocatorMetal.mm:
• platform/graphics/gpu/cocoa/GPUPlatformTypesMetal.h:

3:12 PM Changeset in webkit [268765] by Diego Pino Garcia

• 2 edits in trunk/LayoutTests

[GLIB] Unreviewed test gardening. Mark several WPT Mediasession tests as failure after r268735.

• platform/glib/TestExpectations:

3:06 PM Changeset in webkit [268764] by don.olmstead@sony.com

• 5 edits in trunk/Source/WebCore

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

Unreviewed build fix.

No new tests. No change in behavior.

• bindings/js/JSEventCustom.cpp:

(WebCore::toJS):

• display/DisplayTreeBuilder.h:
• layout/integration/LayoutIntegrationLineLayout.cpp:
• workers/WorkerOrWorkletScriptController.h:

3:01 PM Changeset in webkit [268763] by aakash_jain@apple.com

• 2 edits in trunk/Tools

Make report-non-inclusive-language ignore autoinstalled directories and certain file types (follow-up fix)
​https://bugs.webkit.org/show_bug.cgi?id=217972

Reviewed by Darin Adler.

• Scripts/report-non-inclusive-language: Rename IGNORE_DIRECTORIES to IGNORE_DIRECTORIES_CONTAINING.

2:54 PM Changeset in webkit [268762] by sihui_liu@apple.com

• 24 edits
  20 adds in trunk

Add stubs for SpeechRecognition
​https://bugs.webkit.org/show_bug.cgi?id=217780
<rdar://problem/70350727>

Reviewed by Youenn Fablet.

LayoutTests/imported/w3c:

• web-platform-tests/speech-api/SpeechRecognition-basics.https-expected.txt:
• web-platform-tests/speech-api/idlharness.window-expected.txt:

Source/WebCore:

Add bindings code and a feature flag for SpeechRecognition.
Spec: ​https://wicg.github.io/speech-api/#speechreco-section.

This patch includes all interfaces of SpeechRecognition, except SpeechGrammar and SpeechGrammarList. We have not
decided what grammar format and types we are going to support, or how we are going to support it if it is not
defined in exisiting speech recognition service. So let's add those interfaces after we figure this out.

No new tests. Rebaseline existins tests.

• CMakeLists.txt:
• DerivedSources-input.xcfilelist:
• DerivedSources-output.xcfilelist:
• DerivedSources.make:
• Modules/speech/SpeechRecognition.cpp: Added.

(WebCore::SpeechRecognition::create):
(WebCore::SpeechRecognition::SpeechRecognition):
(WebCore::SpeechRecognition::suspend):
(WebCore::SpeechRecognition::startRecognition):
(WebCore::SpeechRecognition::stopRecognition):
(WebCore::SpeechRecognition::abortRecognition):
(WebCore::SpeechRecognition::activeDOMObjectName const):

• Modules/speech/SpeechRecognition.h: Added.

(WebCore::SpeechRecognition::lang const):
(WebCore::SpeechRecognition::setLang):
(WebCore::SpeechRecognition::continuous const):
(WebCore::SpeechRecognition::setContinuous):
(WebCore::SpeechRecognition::interimResults const):
(WebCore::SpeechRecognition::setInterimResults):
(WebCore::SpeechRecognition::maxAlternatives const):
(WebCore::SpeechRecognition::setMaxAlternatives):

• Modules/speech/SpeechRecognition.idl: Added.
• Modules/speech/SpeechRecognitionAlternative.cpp: Added.

(WebCore::SpeechRecognitionAlternative::create):
(WebCore::SpeechRecognitionAlternative::SpeechRecognitionAlternative):

• Modules/speech/SpeechRecognitionAlternative.h: Added.
• Modules/speech/SpeechRecognitionAlternative.idl: Added.
• Modules/speech/SpeechRecognitionErrorCode.h: Added.
• Modules/speech/SpeechRecognitionErrorCode.idl: Added.
• Modules/speech/SpeechRecognitionErrorEvent.cpp: Added.

(WebCore::SpeechRecognitionErrorEvent::create):
(WebCore::SpeechRecognitionErrorEvent::SpeechRecognitionErrorEvent):
(WebCore::SpeechRecognitionErrorEvent::eventInterface const):

• Modules/speech/SpeechRecognitionErrorEvent.h: Added.
• Modules/speech/SpeechRecognitionErrorEvent.idl: Added.
• Modules/speech/SpeechRecognitionEvent.cpp: Added.

(WebCore::SpeechRecognitionEvent::create):
(WebCore::SpeechRecognitionEvent::SpeechRecognitionEvent):
(WebCore::SpeechRecognitionEvent::eventInterface const):

• Modules/speech/SpeechRecognitionEvent.h: Added.
• Modules/speech/SpeechRecognitionEvent.idl: Added.
• Modules/speech/SpeechRecognitionResult.cpp: Added.

(WebCore::SpeechRecognitionResult::create):
(WebCore::SpeechRecognitionResult::SpeechRecognitionResult):
(WebCore::SpeechRecognitionResult::item const):

• Modules/speech/SpeechRecognitionResult.h: Added.
• Modules/speech/SpeechRecognitionResult.idl: Added.
• Modules/speech/SpeechRecognitionResultList.cpp: Added.

(WebCore::SpeechRecognitionResultList::create):
(WebCore::SpeechRecognitionResultList::item const):
(WebCore::SpeechRecognitionResultList::add):
(WebCore::SpeechRecognitionResultList::SpeechRecognitionResultList):

• Modules/speech/SpeechRecognitionResultList.h: Added.
• Modules/speech/SpeechRecognitionResultList.idl: Added.
• Sources.txt:
• WebCore.xcodeproj/project.pbxproj:
• bindings/js/WebCoreBuiltinNames.h:
• dom/EventNames.in:
• dom/EventTargetFactory.in:

Source/WebKitLegacy/mac:

• WebView/WebPreferenceKeysPrivate.h:
• WebView/WebPreferences.mm:

(-[WebPreferences _speechRecognitionEnabled]):
(-[WebPreferences _setSpeechRecognitionEnabled:]):

• WebView/WebPreferencesPrivate.h:

Source/WTF:

• Scripts/Preferences/WebPreferencesExperimental.yaml:
• wtf/PlatformEnable.h:

Tools:

• DumpRenderTree/mac/DumpRenderTree.mm:

(enableExperimentalFeatures):

LayoutTests:

• platform/mac/TestExpectations:

2:40 PM Changeset in webkit [268761] by Jonathan Bedard

• 5 edits in trunk/Tools

[webkitpy] Use allowlist and blocklist
​https://bugs.webkit.org/show_bug.cgi?id=217985
<rdar://problem/70499499>

Reviewed by Aakash Jain.

• Scripts/webkitpy/pylintrc:
• Scripts/webkitpy/style/checkers/cpp.py:

(check_namespace_indentation):

• Scripts/webkitpy/w3c/common.py:

(is_basename_skipped):

• flatpak/flatpakutils.py:

(WebkitFlatpak.setup_gstbuild):

2:23 PM Changeset in webkit [268760] by Ross Kirsling

• 15 edits
  1 move in trunk

[JSC] Rename item() to at() and move it behind a flag
​https://bugs.webkit.org/show_bug.cgi?id=217942

Reviewed by Yusuke Suzuki.

JSTests:

• stress/at-method.js: Renamed from JSTests/stress/item-method.js.
• test262/config.yaml: Add skips until the feature is renamed.

Source/JavaScriptCore:

{Array, %TypedArray%}.prototype.item is official web-incompatible,
but it is expected to be renamed to at instead of being scrapped entirely:
​https://github.com/tc39/proposal-item-method/issues/34

This patch performs the renaming, but does so behind a runtime flag since this has yet to achieve consensus.

• builtins/ArrayPrototype.js:

(at):
(item): Deleted.

• builtins/TypedArrayPrototype.js:

(at):
(item): Deleted.

• runtime/ArrayPrototype.cpp:

(JSC::ArrayPrototype::finishCreation):

• runtime/JSTypedArrayViewPrototype.cpp:

(JSC::JSTypedArrayViewPrototype::finishCreation):

• runtime/OptionsList.h:

LayoutTests:

• inspector/model/remote-object-get-properties-expected.txt:
• js/array-unscopables-properties-expected.txt:
• js/Object-getOwnPropertyNames-expected.txt:
• js/script-tests/Object-getOwnPropertyNames.js:
• js/script-tests/array-unscopables-properties.js:

2:00 PM Changeset in webkit [268759] by Diego Pino Garcia

• 3 edits in trunk/LayoutTests

[GLIB][GTK] Unreviewed test gardening. Gardened several flaky crash tests.

• platform/glib/TestExpectations:
• platform/gtk/TestExpectations:

1:42 PM Changeset in webkit [268758] by Peng Liu

• 5 edits in trunk/Source/WebKit

[Media in GPU Process] Some WebAudio layout tests generate strange noises
​https://bugs.webkit.org/show_bug.cgi?id=217921

Reviewed by Eric Carlson.

RemoteAudioDestination::render() should not return noErr unless we can provide the requested
samples to the provided AudioBufferList. Otherwise, the audio output unit of CoreAudio will output
the samples in the AudioBufferList, which might be invalid data at the beginning of a rendering.
We have observed that happens in some layout tests and some WebAudio example pages.

Currently, RemoteAudioDestination::render() always returns noErr in the render thread (immediately),
but the AudioBufferList (ioData) is updated in the main thread (later). This patch fixes that by only
setting the bounds of CARingBuffer in the completion handler of sendWithAsyncReply() in the main thread,
and fetching AudioBuffer(s) from the CARingBuffer in the render thread. Also, RemoteAudioDestination
tracks the progress of fetching, so RemoteAudioDestinationProxy does not need to send startFrame
and numberOfFramesToRender to RemoteAudioDestination in response to a buffer request.

• GPUProcess/media/RemoteAudioDestinationManager.cpp:

(WebKit::RemoteAudioDestination::audioSamplesStorageChanged):
(WebKit::RemoteAudioDestination::render): Only return noErr if the function renders the requested
sample successfully.

• WebProcess/GPU/media/RemoteAudioDestinationProxy.cpp:

(WebKit::RemoteAudioDestinationProxy::requestBuffer): Remove unused parameters.
(WebKit::RemoteAudioDestinationProxy::renderOnRenderingThead): Ditto.

• WebProcess/GPU/media/RemoteAudioDestinationProxy.h: Ditto.
• WebProcess/GPU/media/RemoteAudioDestinationProxy.messages.in: Ditto.

1:21 PM Changeset in webkit [268757] by don.olmstead@sony.com

• 2 edits in trunk/Source/WebCore

Fix non-c-typedef-for-linkage warning in SocketStreamHandle
​https://bugs.webkit.org/show_bug.cgi?id=217979

Reviewed by Alex Christensen.

The warning was because the anonymous type was not C-compatible due to a default member
initializer, so remove the typedef and name the struct.

No new tests. No change in behavior.

• platform/network/SocketStreamHandle.h:

1:11 PM Changeset in webkit [268756] by Alan Coon

• 11 edits
  2 deletes in branches/safari-611.1.4-branch/Source

Revert r268386. rdar://problem/70497386

1:10 PM Changeset in webkit [268755] by Alan Coon

• 16 edits
  2 deletes in branches/safari-611.1.4-branch/Source/WebCore

Revert r268578. rdar://problem/70497386

1:10 PM Changeset in webkit [268754] by Alan Coon

• 7 edits in branches/safari-611.1.4-branch

Cherry-pick r268746. rdar://problem/70497736

REGRESSION(r268615): some accelerated transform tests are failing
​https://bugs.webkit.org/show_bug.cgi?id=217851
<rdar://problem/70394402>

Reviewed by Dean Jackson.

Source/WebCore:

When we added support for accelerated individual transform properties animations, we added
the notion of base transform animations which are used to set the base value of any
transform-related property that is not animated.

Those animations were defined to start as early as possible, assuming that a very small value
after 0s was as early as possible. However, it's possible that other animations start with a
negative time if they have a delay or are seeked, if the value returned by CACurrentMediaTime()
is smaller than that delay. This means that if the machine had been booted for less time than
an accelerated animation's delay, the base transform animation wouldn't overlap.

We now ensure that those base transform animations start as early as the earliest animation
that is being committed in a call to GraphicsLayerCA::updateAnimations().

• platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::updateAnimations): (WebCore::GraphicsLayerCA::setAnimationOnLayer):
• platform/graphics/ca/GraphicsLayerCA.h: (WebCore::GraphicsLayerCA::LayerPropertyAnimation::computedBeginTime const):

LayoutTests:

Remove flaky epectations for affected tests and skip tests on Windows where failures remain.

• TestExpectations:
• platform/mac/TestExpectations:
• platform/win/TestExpectations:

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

1:10 PM Changeset in webkit [268753] by Alan Coon

• 4 edits in branches/safari-611.1.4-branch/Source/WebCore

Revert r268446. rdar://problem/70497386

12:13 PM Changeset in webkit [268752] by aakash_jain@apple.com

• 2 edits in trunk/Tools

Make report-non-inclusive-language ignore autoinstalled directories and certain file types
​https://bugs.webkit.org/show_bug.cgi?id=217972

Reviewed by Beth Dakin.

• Scripts/report-non-inclusive-language: startswith and endswith also takes a tuple of suffixes to

look for, reference: ​https://docs.python.org/3/library/stdtypes.html#str.endswith

11:51 AM Changeset in webkit [268751] by commit-queue@webkit.org

• 2 edits in trunk

[GTK] Move ENABLE_ASYNC_SCROLLING build option to right place in OptionsGTK.cmake
​https://bugs.webkit.org/show_bug.cgi?id=217977

Unreviewed. No functional changes.

Patch by Michael Catanzaro <Michael Catanzaro> on 2020-10-20

• Source/cmake/OptionsGTK.cmake:

10:56 AM Changeset in webkit [268750] by Truitt Savell

• 1 edit
  14 deletes in trunk/LayoutTests

Remove expectations from big-sur sense it is not needed after r268683
​https://bugs.webkit.org/show_bug.cgi?id=209813

Unreviewed test gardening.

• platform/mac-bigsur/editing/selection/3690703-2-expected.txt: Removed.
• platform/mac-bigsur/editing/selection/3690703-expected.txt: Removed.
• platform/mac-bigsur/editing/selection/3690719-expected.txt: Removed.
• platform/mac-bigsur/fast/block/float/float-avoidance-expected.txt: Removed.
• platform/mac-bigsur/fast/css/rtl-ordering-expected.txt: Removed.
• platform/mac-bigsur/fast/forms/basic-inputs-expected.txt: Removed.
• platform/mac-bigsur/fast/forms/input-appearance-spinbutton-expected.txt: Removed.
• platform/mac-bigsur/fast/forms/option-text-clip-expected.txt: Removed.
• platform/mac-bigsur/fast/forms/select-change-listbox-to-popup-expected.txt: Removed.
• platform/mac-bigsur/fast/forms/textfield-outline-expected.txt: Removed.
• platform/mac-bigsur/imported/w3c/web-platform-tests/mathml/relations/css-styling/ignored-properties-001-expected.txt: Removed.
• platform/mac-bigsur/media/video-display-toggle-expected.txt: Removed.
• platform/mac-bigsur/media/video-volume-slider-expected.txt: Removed.
• platform/mac-bigsur/tables/mozilla/bugs/bug30692-expected.txt: Removed.

10:53 AM Changeset in webkit [268749] by aakash_jain@apple.com

• 3 edits in trunk/Tools

[build.webkit.org] Use builder tags instead of category for latest buildbot
​https://bugs.webkit.org/show_bug.cgi?id=217958

Reviewed by Jonathan Bedard.

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

(loadBuilderConfig):
(PlatformSpecificScheduler.filter):
(getInvalidTags):
(getValidTags):
(getTagsForBuilder):

• BuildSlaveSupport/build.webkit.org-config/loadConfig_unittest.py: Added unit-tests, copied from EWS code.

(TagsForBuilderTest):
(TagsForBuilderTest.verifyTags):
(TagsForBuilderTest.test_getTagsForBuilder):
(TagsForBuilderTest.test_tags_type):
(TagsForBuilderTest.test_getInvalidTags):

10:48 AM Changeset in webkit [268748] by Chris Dumez

• 2 edits in trunk/LayoutTests

Unreviewed, mark imported/w3c/web-platform-tests/webaudio/the-audio-api/the-mediaelementaudiosourcenode-interface/no-cors.https.html as failing on mac-wk1.

This test was unskipped recently but only seems to pass on WebKit2.

• platform/mac-wk1/TestExpectations:

10:36 AM Changeset in webkit [268747] by Alan Coon

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

Versioning.

WebKit-7611.1.3.4

10:34 AM Changeset in webkit [268746] by graouts@webkit.org

• 7 edits in trunk

REGRESSION(r268615): some accelerated transform tests are failing
​https://bugs.webkit.org/show_bug.cgi?id=217851
<rdar://problem/70394402>

Reviewed by Dean Jackson.

Source/WebCore:

When we added support for accelerated individual transform properties animations, we added
the notion of base transform animations which are used to set the base value of any
transform-related property that is not animated.

Those animations were defined to start as early as possible, assuming that a very small value
after 0s was as early as possible. However, it's possible that other animations start with a
negative time if they have a delay or are seeked, if the value returned by CACurrentMediaTime()
is smaller than that delay. This means that if the machine had been booted for less time than
an accelerated animation's delay, the base transform animation wouldn't overlap.

We now ensure that those base transform animations start as early as the earliest animation
that is being committed in a call to GraphicsLayerCA::updateAnimations().

• platform/graphics/ca/GraphicsLayerCA.cpp:

(WebCore::GraphicsLayerCA::updateAnimations):
(WebCore::GraphicsLayerCA::setAnimationOnLayer):

• platform/graphics/ca/GraphicsLayerCA.h:

(WebCore::GraphicsLayerCA::LayerPropertyAnimation::computedBeginTime const):

LayoutTests:

Remove flaky epectations for affected tests and skip tests on Windows where failures remain.

• TestExpectations:
• platform/mac/TestExpectations:
• platform/win/TestExpectations:

10:24 AM Changeset in webkit [268745] by commit-queue@webkit.org

• 13 edits
  1 copy in trunk

Introduce Selection specific mixin of GlobalEventHandlers
​https://bugs.webkit.org/show_bug.cgi?id=217248

Patch by Rob Buis <​rbuis@igalia.com> on 2020-10-20
Reviewed by Sam Weinig.

Source/WebCore:

Introduce Selection specific mixin of GlobalEventHandlers [1] by
splitting out the interface from Document+Selection.idl. This
also means the HTMLBodyElement specific selectionchange IDL is
superflous and can be removed.

Behavior matches Chrome and Firefox.

Test: fast/dom/event-handler-attributes.html

[1] ​https://w3c.github.io/selection-api/#dom-globaleventhandlers

• CMakeLists.txt:
• DerivedSources-input.xcfilelist:
• DerivedSources-output.xcfilelist:
• DerivedSources.make:
• WebCore.xcodeproj/project.pbxproj:
• dom/Document+Selection.idl:
• dom/Element.idl:
• dom/GlobalEventHandlers+Selection.idl: Copied from Source/WebCore/dom/Document+Selection.idl.

LayoutTests:

Adapt test to GlobalEventHandlers change.

• fast/dom/event-attribute-availability-expected.txt:
• fast/dom/event-handler-attributes-expected.txt:
• fast/dom/event-handler-attributes.html:

10:18 AM Changeset in webkit [268744] by Alan Coon

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

Cherry-pick r268712. rdar://problem/70488953

Fix crash in RenderLayerBacking::updateClippingStackLayerGeometry()
​https://bugs.webkit.org/show_bug.cgi?id=217940
<rdar://problem/70316952>

Reviewed by Tim Horton.

Crash data suggest that entry.clipData.clippingLayer (which is a WeakRef<RenderLayer>)
can be null, so check it.

• rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::updateClippingStackLayerGeometry):

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

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

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

Cherry-pick r268701. rdar://problem/70488900

Fix possible crash in GraphicsLayerCA::computeVisibleAndCoverageRect()
​https://bugs.webkit.org/show_bug.cgi?id=217930
<rdar://problem/70316943>

Reviewed by Tim Horton.

If we made a m_overflowControlsHostLayerAncestorClippingStack, make sure we unparent
its layers when tearing down the RenderLayerBacking, and when we determine that we
longer need a m_ancestorClippingStack (having a m_overflowControlsHostLayerAncestorClippingStack
implies that we have a m_ancestorClippingStack).

• rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::destroyGraphicsLayers): (WebCore::RenderLayerBacking::updateAncestorClipping):

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

10:16 AM Changeset in webkit [268742] by Alan Coon

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

Versioning.

WebKit-7610.2.11.51.4

10:07 AM Changeset in webkit [268741] by Antti Koivisto

• 44 edits
  10 deletes in trunk

Implement <forgiving-selector-list> for :is/:where
​https://bugs.webkit.org/show_bug.cgi?id=217814
<rdar://problem/70384483>

Reviewed by Sam Weinig.

LayoutTests/imported/w3c:

• web-platform-tests/css/selectors/is-where-error-recovery-expected.txt:
• web-platform-tests/css/selectors/is-where-not-expected.txt:

Source/WebCore:

The spec now says :is/:where should parse as a forgiving selector list: ​https://drafts.csswg.org/selectors/#matches

• contentextensions/ContentExtensionParser.cpp:

(WebCore::ContentExtensions::isValidCSSSelector):

• css/CSSPageRule.cpp:

(WebCore::CSSPageRule::setSelectorText):

• css/CSSSelectorList.h:

(WebCore::CSSSelectorList::isEmpty const):
(WebCore::CSSSelectorList::isValid const): Deleted.

An empty CSSSelectorList is now valid.
Represent invalid CSSSelectorList with Optional where needed.

• css/CSSStyleRule.cpp:

(WebCore::CSSStyleRule::setSelectorText):

• css/parser/CSSParser.cpp:

(WebCore::CSSParser::parseSelector):

• css/parser/CSSParser.h:
• css/parser/CSSParserImpl.cpp:

(WebCore::CSSParserImpl::consumePageRule):
(WebCore::CSSParserImpl::consumeStyleRule):

• css/parser/CSSSelectorParser.cpp:

(WebCore::parseCSSSelector):
(WebCore::CSSSelectorParser::consumeComplexSelectorList):
(WebCore::CSSSelectorParser::consumeComplexForgivingSelectorList):

Add function for consuming <forgiving-selector-list>.

(WebCore::CSSSelectorParser::consumePseudo):

Use it for :is/:where.

• css/parser/CSSSelectorParser.h:
• dom/SelectorQuery.cpp:
• inspector/InspectorStyleSheet.cpp:

(WebCore::isValidSelectorListString):

• inspector/agents/InspectorDOMAgent.cpp:

(WebCore::InspectorDOMAgent::highlightSelector):

• style/RuleSet.cpp:

(WebCore::Style::RuleSet::addStyleRule):

LayoutTests:

Update some selector parsing tests to log the results instead of PASS/FAIL as forgiving
parsing allows many combinations involving :is/:where.

Remove a few repetetive tests that don't add anything.

• fast/css/parsing-css-attribute-case-insensitive-value-4-expected.txt: Removed.
• fast/css/parsing-css-attribute-case-insensitive-value-4.html: Removed.
• fast/css/parsing-css-is-5-expected.txt:
• fast/css/parsing-css-is-5.html:
• fast/css/parsing-css-is-6-expected.txt:
• fast/css/parsing-css-is-6.html:
• fast/css/parsing-css-is-7-expected.txt:
• fast/css/parsing-css-is-7.html:
• fast/css/parsing-css-is-8-expected.txt:
• fast/css/parsing-css-is-8.html:
• fast/css/parsing-css-matches-5-expected.txt: Removed.
• fast/css/parsing-css-matches-5.html: Removed.
• fast/css/parsing-css-matches-6-expected.txt: Removed.
• fast/css/parsing-css-matches-6.html: Removed.
• fast/css/parsing-css-matches-7-expected.txt: Removed.
• fast/css/parsing-css-matches-7.html: Removed.
• fast/css/parsing-css-matches-8-expected.txt: Removed.
• fast/css/parsing-css-matches-8.html: Removed.
• fast/css/parsing-css-not-5-expected.txt:
• fast/css/parsing-css-not-5.html:
• fast/css/parsing-css-not-6-expected.txt:
• fast/css/parsing-css-not-6.html:
• fast/css/parsing-css-not-7-expected.txt:
• fast/css/parsing-css-not-7.html:
• fast/css/parsing-css-not-8-expected.txt:
• fast/css/parsing-css-not-8.html:
• fast/css/parsing-css-not-9-expected.txt:
• fast/css/parsing-css-not-9.html:
• fast/css/parsing-css-nth-child-of-4-expected.txt:
• fast/css/parsing-css-nth-child-of-4.html:
• fast/css/parsing-css-nth-last-child-of-4-expected.txt:
• fast/css/parsing-css-nth-last-child-of-4.html:
• fast/selectors/invalid-functional-pseudo-class-expected.txt:
• fast/selectors/invalid-functional-pseudo-class.html:
• fast/selectors/pseudo-element-in-is-where-expected.html:
• fast/selectors/pseudo-element-in-is-where.html:

10:05 AM Changeset in webkit [268740] by commit-queue@webkit.org

• 2 edits in trunk/Tools

TestOptionsGeneratedKeys.h:30:52: warning: backslash-newline at end of file
​https://bugs.webkit.org/show_bug.cgi?id=217957

Patch by Michael Catanzaro <Michael Catanzaro> on 2020-10-20
Reviewed by Sam Weinig.

I guess this is bad because something else from another header could be on the next line,
depending on the order that headers get included into the source file. So let's add a blank
line to the end of the file.

• WebKitTestRunner/Scripts/PreferencesTemplates/TestOptionsGeneratedKeys.h.erb:

10:04 AM Changeset in webkit [268739] by Diego Pino Garcia

• 3 edits in trunk/LayoutTests

[GLIB][GTK] Unreviewed test gardening. Update test expectations after r268732.

Added some of the latest flaky failures that have been happening in WebKitGTK
post-commit bot.

• platform/glib/TestExpectations:
• platform/gtk/TestExpectations:

9:53 AM Changeset in webkit [268738] by Hector Lopez

• 2 edits in trunk/LayoutTests

Test expectation adjustment
​https://bugs.webkit.org/show_bug.cgi?id=215324

Unreviewed test gardening.

• platform/ios-wk2/TestExpectations:

9:41 AM Changeset in webkit [268737] by Hector Lopez

• 2 edits in trunk/LayoutTests

Adjust test expectation for imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-no-sink.https.html
​https://bugs.webkit.org/show_bug.cgi?id=217263

Unreviewed test gardening.

• platform/ios-wk2/TestExpectations:

9:25 AM Changeset in webkit [268736] by Chris Dumez

• 9 edits
  3 deletes in trunk/Source

Drop legacy code using AssertionServices
​https://bugs.webkit.org/show_bug.cgi?id=217787
<rdar://problem/70160942>

Reviewed by Darin Adler.

Drop legacy code using AssertionServices as this framework is deprecated in favor of
RunningBoard. All our code has been ported to RunningBoard already.

• Configurations/WebKit.xcconfig:
• Platform/spi/ios/AssertionServicesSPI.h: Removed.
• Shared/Cocoa/ProcessTaskStateObserver.h: Removed.
• Shared/Cocoa/ProcessTaskStateObserver.mm: Removed.
• UIProcess/ApplicationStateTracker.mm:
• UIProcess/ProcessAssertion.h:
• UIProcess/ios/PageClientImplIOS.mm:
• UIProcess/ios/ProcessAssertionIOS.mm:

(-[WKProcessAssertionBackgroundTaskManager init]):
(-[WKProcessAssertionBackgroundTaskManager _hasBackgroundTask]):
(-[WKProcessAssertionBackgroundTaskManager _updateBackgroundTask]):
(-[WKProcessAssertionBackgroundTaskManager assertion:didInvalidateWithError:]):
(-[WKProcessAssertionBackgroundTaskManager _releaseBackgroundTask]):

• WebKit.xcodeproj/project.pbxproj:
• WebProcess/cocoa/WebProcessCocoa.mm:

9:20 AM Changeset in webkit [268735] by jer.noble@apple.com

• 17 edits
  42 adds in trunk

Add skeleton implementation of Media Session API
​https://bugs.webkit.org/show_bug.cgi?id=217797
LayoutTests/imported/w3c:

Reviewed by Darin Adler.

• resources/import-expectations.json:
• resources/resource-files.json:
• web-platform-tests/mediasession/META.yml: Added.
• web-platform-tests/mediasession/README.md: Added.
• web-platform-tests/mediasession/helper/artwork-generator.html: Added.
• web-platform-tests/mediasession/helper/w3c-import.log: Added.
• web-platform-tests/mediasession/idlharness.window-expected.txt: Added.
• web-platform-tests/mediasession/idlharness.window.html: Added.
• web-platform-tests/mediasession/idlharness.window.js: Added.
• web-platform-tests/mediasession/mediametadata-expected.txt: Added.
• web-platform-tests/mediasession/mediametadata.html: Added.
• web-platform-tests/mediasession/playbackstate-expected.txt: Added.
• web-platform-tests/mediasession/playbackstate.html: Added.
• web-platform-tests/mediasession/positionstate-expected.txt: Added.
• web-platform-tests/mediasession/positionstate.html: Added.
• web-platform-tests/mediasession/setactionhandler-expected.txt: Added.
• web-platform-tests/mediasession/setactionhandler.html: Added.
• web-platform-tests/mediasession/w3c-import.log: Added.

Source/WebCore:

Reviewed by Darin Adler.

Tests: imported/w3c/web-platform-tests/mediasession/idlharness.window.html

imported/w3c/web-platform-tests/mediasession/mediametadata.html
imported/w3c/web-platform-tests/mediasession/playbackstate.html
imported/w3c/web-platform-tests/mediasession/positionstate.html
imported/w3c/web-platform-tests/mediasession/setactionhandler.html

Add an initial, no-op, implementation of the Media Session API.

Note, the WPT test results include a failing test of Object.isFrozen(); the bindings
generator does not currently support generating a FrozenArray of dictionary values.

• DerivedSources-input.xcfilelist:
• DerivedSources-output.xcfilelist:
• DerivedSources.make:
• Modules/mediasession/MediaImage.h: Added.
• Modules/mediasession/MediaImage.idl: Added.
• Modules/mediasession/MediaMetadata.h: Added.

(WebCore::MediaMetadata::create):
(WebCore::MediaMetadata::MediaMetadata):
(WebCore::MediaMetadata::setMediaSession):
(WebCore::MediaMetadata::title const):
(WebCore::MediaMetadata::setTitle):
(WebCore::MediaMetadata::artist const):
(WebCore::MediaMetadata::setArtist):
(WebCore::MediaMetadata::album const):
(WebCore::MediaMetadata::setAlbum):
(WebCore::MediaMetadata::artwork const):
(WebCore::MediaMetadata::setArtwork):
(WebCore::MediaMetadata::metadataUpdated):

• Modules/mediasession/MediaMetadata.idl: Added.
• Modules/mediasession/MediaMetadataInit.h: Added.
• Modules/mediasession/MediaMetadataInit.idl: Added.
• Modules/mediasession/MediaPositionState.h: Added.
• Modules/mediasession/MediaPositionState.idl: Added.
• Modules/mediasession/MediaSession.cpp: Added.

(WebCore::MediaSession::create):
(WebCore::MediaSession::MediaSession):
(WebCore::MediaSession::setMetadata):
(WebCore::MediaSession::setPlaybackState):
(WebCore::MediaSession::setActionHandler):
(WebCore::MediaSession::setPositionState):
(WebCore::MediaSession::metadataUpdated):

• Modules/mediasession/MediaSession.h: Added.

(WebCore::MediaSession::metadata const):
(WebCore::MediaSession::playbackState const):

• Modules/mediasession/MediaSession.idl: Added.
• Modules/mediasession/MediaSessionAction.h: Added.
• Modules/mediasession/MediaSessionAction.idl: Added.
• Modules/mediasession/MediaSessionActionDetails.h: Added.
• Modules/mediasession/MediaSessionActionDetails.idl: Added.
• Modules/mediasession/MediaSessionActionHandler.h: Added.
• Modules/mediasession/MediaSessionActionHandler.idl: Added.
• Modules/mediasession/MediaSessionPlaybackState.h: Added.
• Modules/mediasession/MediaSessionPlaybackState.idl: Added.
• Modules/mediasession/Navigator+MediaSession.idl: Added.
• Modules/mediasession/NavigatorMediaSession.cpp: Added.

(WebCore::NavigatorMediaSession::NavigatorMediaSession):
(WebCore::NavigatorMediaSession::mediaSession):
(WebCore::NavigatorMediaSession::from):
(WebCore::NavigatorMediaSession::supplementName):

• Modules/mediasession/NavigatorMediaSession.h: Added.
• Sources.txt:
• WebCore.xcodeproj/project.pbxproj:

Source/WTF:

Reviewed by Darin Adler.

• Scripts/Preferences/WebPreferencesExperimental.yaml:
• wtf/PlatformEnable.h:
• wtf/PlatformEnableCocoa.h:

Tools:

<rdar://problem/70367487>

Reviewed by Darin Adler.

• DumpRenderTree/mac/DumpRenderTree.mm:

(enableExperimentalFeatures):

8:59 AM Changeset in webkit [268734] by Chris Dumez

• 2 edits in trunk/LayoutTests

Unreviewed, unskip imported/w3c/web-platform-tests/webaudio/the-audio-api/the-mediaelementaudiosourcenode-interface/no-cors.https.html.

The test is no longer timing out.

• TestExpectations:

8:50 AM Changeset in webkit [268733] by Nikita Vasilyev

• 2 edits in trunk/Source/WebInspectorUI

REGRESSION(r268691): Web Inspector: WI.Sidebar.Event.CollapsedStateChanged is undefined
​https://bugs.webkit.org/show_bug.cgi?id=217939

Reviewed by Brian Burg.

r268691 added an event listener for CollapsedStateChanged, which is undefined.

• UserInterface/Base/Main.js:

8:48 AM Changeset in webkit [268732] by Philippe Normand

• 2 edits in trunk/Source/WebCore

Unreviewed, temporary workaround for bug #217959

• platform/audio/gstreamer/AudioDestinationGStreamer.cpp:

(WebCore::maximumNumberOfOutputChannels): Return a hard-coded value until the device monitor issue is fixed.

8:42 AM Changeset in webkit [268731] by aakash_jain@apple.com

• 3 edits in trunk/Tools

[ews] Remove blacklist terminology
​https://bugs.webkit.org/show_bug.cgi?id=217953

Reviewed by Jonathan Bedard.

• BuildSlaveSupport/ews-build/loadConfig.py:

(getInvalidTags):
(getValidTags):

• BuildSlaveSupport/ews-build/loadConfig_unittest.py:

(TagsForBuilderTest.test_getInvalidTags):

8:37 AM Changeset in webkit [268730] by Devin Rousso

• 2 edits in trunk/Source/WebCore

Web Inspector: REGRESSION(r260076): crash under InspectorInstrumentation::willApplyKeyframeEffect
​https://bugs.webkit.org/show_bug.cgi?id=217936

Reviewed by Brian Burg.

Speculative fix by null-checking targetElementOrPseudoElement() before using it. Before
r260076 this was not necessary as m_target is null-checked earlier in the function, but
after it can now return nullptr depending on the value of m_pseudoId.

• animation/KeyframeEffect.cpp:

(WebCore::KeyframeEffect::apply):

8:17 AM Changeset in webkit [268729] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebCore

Unused parameter warning in GraphicsContextCairo.cpp
​https://bugs.webkit.org/show_bug.cgi?id=217954

Unreviewed.

Patch by Michael Catanzaro <Michael Catanzaro> on 2020-10-20

• platform/graphics/cairo/GraphicsContextCairo.cpp:

(WebCore::GraphicsContext::platformDrawNativeImage):

8:02 AM Changeset in webkit [268728] by Chris Dumez

• 15 edits in trunk/Source/WebCore

Drop explicit calls to BaseAudioContext::refNode()
​https://bugs.webkit.org/show_bug.cgi?id=217926

Reviewed by Eric Carlson.

Drop explicit calls to BaseAudioContext::refNode(). Instead we now rely on the
sourceNodeWillBeginPlayback() signal to ref the source AudioNode. This simplifies
the code a bit. I also did some renaming for clarity, so be explicit about the
fact we are dealing with *source* nodes here.

No new tests, no Web-facing behavior change.

• Modules/webaudio/AudioBufferSourceNode.cpp:

(WebCore::AudioBufferSourceNode::create):
(WebCore::AudioBufferSourceNode::startPlaying):

• Modules/webaudio/AudioContext.cpp:

(WebCore::AudioContext::sourceNodeWillBeginPlayback):

• Modules/webaudio/AudioContext.h:
• Modules/webaudio/AudioScheduledSourceNode.cpp:

(WebCore::AudioScheduledSourceNode::startLater):
(WebCore::AudioScheduledSourceNode::finish):

• Modules/webaudio/AudioWorkletNode.cpp:

(WebCore::AudioWorkletNode::create):
(WebCore::AudioWorkletNode::didFinishProcessingOnRenderingThread):
Only call sourceNodeWillBeginPlayback() / sourceNodeDidFinishPlayback()
when the AudioWorklet is actually a *source* node, meaning that it has
outputs. This behavior is consistent with Blink.

• Modules/webaudio/BaseAudioContext.cpp:

(WebCore::BaseAudioContext::~BaseAudioContext):
(WebCore::BaseAudioContext::uninitialize):
(WebCore::BaseAudioContext::createScriptProcessor):
(WebCore::BaseAudioContext::derefFinishedSourceNodes):
(WebCore::BaseAudioContext::refSourceNode):
(WebCore::BaseAudioContext::derefSourceNode):
(WebCore::BaseAudioContext::derefUnfinishedSourceNodes):
(WebCore::BaseAudioContext::sourceNodeWillBeginPlayback):
(WebCore::BaseAudioContext::sourceNodeDidFinishPlayback):

• Modules/webaudio/BaseAudioContext.h:
• Modules/webaudio/ConstantSourceNode.cpp:

(WebCore::ConstantSourceNode::create):

• Modules/webaudio/MediaElementAudioSourceNode.cpp:

(WebCore::MediaElementAudioSourceNode::create):

• Modules/webaudio/MediaStreamAudioSourceNode.cpp:

(WebCore::MediaStreamAudioSourceNode::create):

• Modules/webaudio/OscillatorNode.cpp:

(WebCore::OscillatorNode::create):

• Modules/webaudio/ScriptProcessorNode.cpp:

(WebCore::ScriptProcessorNode::eventListenersDidChange):
(WebCore::ScriptProcessorNode::virtualHasPendingActivity const):

• Modules/webaudio/ScriptProcessorNode.h:

Use ActiveDOMObject logic to keep the script wrapper alive instead of relying on the
AudioContext to do so. We keep the wrapper alive as long as the context's state is
not closed and as long as there is an audioprocess event listener registered. This
behavior is consistent with what Blink does.

• Modules/webaudio/WebKitAudioContext.cpp:

(WebCore::WebKitAudioContext::createWebKitOscillator):
(WebCore::WebKitAudioContext::createWebKitBufferSource):

6:19 AM Changeset in webkit [268727] by Philippe Normand

• 7 edits in trunk

[GStreamer] REGRESSION(r266559): imported/w3c/web-platform-tests/webaudio/the-audio-api/the-destinationnode-interface/destination.html is failing
​https://bugs.webkit.org/show_bug.cgi?id=216171

Reviewed by Xabier Rodriguez-Calvar.

Source/WebCore:

The AudioDestination now probes the audio sink to find the maximum number of output channels
it supports. The AudioDestination is also now able to render >2 channels from the AudioBus.
The AudioProvider also handles more than 2 channels. This patch also includes various code
cleanups as these files were not updated much during the past few years.

Covered by existing webaudio tests.

• platform/audio/gstreamer/AudioDestinationGStreamer.cpp:

(WebCore::initializeDebugCategory):
(WebCore::maximumNumberOfOutputChannels):
(WebCore::AudioDestination::create):
(WebCore::AudioDestination::maxChannelCount):
(WebCore::AudioDestinationGStreamer::AudioDestinationGStreamer):
(WebCore::AudioDestinationGStreamer::~AudioDestinationGStreamer):
(WebCore::AudioDestinationGStreamer::handleMessage):
(WebCore::AudioDestinationGStreamer::start):
(WebCore::AudioDestinationGStreamer::stop):

• platform/audio/gstreamer/AudioDestinationGStreamer.h:
• platform/audio/gstreamer/AudioSourceProviderGStreamer.cpp:

(WebCore::initializeDebugCategory):
(WebCore::copyGStreamerBuffersToAudioChannel):
(WebCore::AudioSourceProviderGStreamer::AudioSourceProviderGStreamer):
(WebCore::AudioSourceProviderGStreamer::~AudioSourceProviderGStreamer):
(WebCore::AudioSourceProviderGStreamer::provideInput):
(WebCore::AudioSourceProviderGStreamer::handleSample):
(WebCore::AudioSourceProviderGStreamer::setClient):
(WebCore::AudioSourceProviderGStreamer::handleNewDeinterleavePad):
(WebCore::AudioSourceProviderGStreamer::handleRemovedDeinterleavePad):
(WebCore::AudioSourceProviderGStreamer::deinterleavePadsConfigured):
(WebCore::AudioSourceProviderGStreamer::clearAdapters):

• platform/audio/gstreamer/AudioSourceProviderGStreamer.h:

LayoutTests:

• platform/glib/TestExpectations: Unflag tests.

imported/w3c/web-platform-tests/webaudio/the-audio-api/the-mediaelementaudiosourcenode-interface/cors-check.https.html
is still flaky but flagged already as such in the generic TestExpectations.

5:19 AM Changeset in webkit [268726] by svillar@igalia.com

• 7 edits in trunk

[css-flexbox] order property should be animatable
​https://bugs.webkit.org/show_bug.cgi?id=209876

Reviewed by Antoine Quint.

LayoutTests/imported/w3c:

Updated ~150 FAIL expectations to PASS.

• web-platform-tests/css/css-flexbox/animation/order-interpolation-expected.txt: Updated expectations.
• web-platform-tests/web-animations/animation-model/animation-types/accumulation-per-property-002-expected.txt: Ditto.
• web-platform-tests/web-animations/animation-model/animation-types/addition-per-property-002-expected.txt: Ditto.
• web-platform-tests/web-animations/animation-model/animation-types/interpolation-per-property-002-expected.txt: Ditto.

Source/WebCore:

Make order property animatable as per spec. This allows us to pass ~150 flexbox animation subtests.

• animation/CSSPropertyAnimation.cpp:

(WebCore::CSSPropertyAnimationWrapperMap::CSSPropertyAnimationWrapperMap): Mark property as animatable.

4:53 AM Changeset in webkit [268725] by Adrian Perez de Castro

• 2 edits in trunk

[GTK] Implement HTML5 Gamepad API
​https://bugs.webkit.org/show_bug.cgi?id=133847

Reviewed by Michael Catanzaro.

Enable the gamepad support by default for the GTK port. The option is marked PUBLIC
because libmanette is not available in the current Debian "old stable" (Buster), which
we need to support until September 2021.

• Source/cmake/OptionsGTK.cmake: Set ENABLE_GAMEPAD to PUBLIC and ON by default.

4:02 AM Changeset in webkit [268724] by Philippe Normand

• 2 edits in trunk/Tools

Unreviewed, partial revert of r268721.

• buildstream/elements/sdk/libmanette.bst: libevdev is actually a requirement for libmanette.

3:58 AM Changeset in webkit [268723] by Carlos Garcia Campos

• 5 edits in trunk

WebDriver: start a https server too for tests
​https://bugs.webkit.org/show_bug.cgi?id=217223
<rdar://problem/70136595>

Reviewed by Adrian Perez de Castro.

Tools:

Handle HTTPS configuration in WebDriver WPT server.

• Scripts/webkitpy/webdriver_tests/webdriver_w3c_executor.py:

(WebDriverW3CExecutor.init): Set https configuration from server.

• Scripts/webkitpy/webdriver_tests/webdriver_w3c_web_server.py:

(WebDriverW3CWebServer._wait_for_server): Also wait for HTTPS port.
(WebDriverW3CWebServer.start): Write config file and get also https port.
(WebDriverW3CWebServer.http_port): Renamed port as http_port.
(WebDriverW3CWebServer.https_port): Added to return the https port.
(WebDriverW3CWebServer.port): Deleted.

WebDriverTests:

• imported/w3c/config.json: Add HTTPs configuration.

3:56 AM Changeset in webkit [268722] by commit-queue@webkit.org

• 5 edits in trunk/Tools

[Flatpak SDK] Update rr for AMD Ryzen support
​https://bugs.webkit.org/show_bug.cgi?id=217474

Patch by Philippe Normand <​pnormand@igalia.com> on 2020-10-20
Reviewed by Adrian Perez de Castro.

• buildstream/Pipfile: Remove Python 3.8 hard requirement.
• buildstream/Pipfile.lock:
• buildstream/elements/freedesktop-sdk.bst: Bump to latest version of the 20.08 FDO junction.
• buildstream/elements/sdk/rr.bst: Bump to current git master.

3:53 AM Changeset in webkit [268721] by Adrian Perez de Castro

• 2 edits in trunk/Tools

[GTK] Gamepad events do not occur in the flatpak shell.
​https://bugs.webkit.org/show_bug.cgi?id=212453

Reviewed by Michael Catanzaro.

If the support for gudev is disabled in libmanette, it falls back to monitoring the
files under /dev/input by itself without trying to use udev at all. This works fine
because the Flatpak SDK environment has access to all of /dev.

• buildstream/elements/sdk/libmanette.bst: Pass -Dgudev=disabled to Meson, and remove

the libgudev dependency from the element.

2:42 AM Changeset in webkit [268720] by Diego Pino Garcia

• 3 edits in trunk/LayoutTests

[GLIB][GTK] Unreviewed test gardening. Update test expectations after r268715.

• platform/glib/TestExpectations:
• platform/gtk/TestExpectations:

2:27 AM Changeset in webkit [268719] by weinig@apple.com

• 21 edits in trunk

Tweak WebPreferences*.yaml "exposed" key to only indicate that the key should not be changeable by the frontend
​https://bugs.webkit.org/show_bug.cgi?id=217918

Reviewed by Darin Adler.

Source/WebKit:

Update to use new "exposed" prefix bindings and use default value explicitly
when updating settings for non-exposed preferences.

• Scripts/PreferencesTemplates/WebPageUpdatePreferences.cpp.erb:
• Scripts/PreferencesTemplates/WebPreferencesDefinitions.h.erb:
• Scripts/PreferencesTemplates/WebPreferencesExperimentalFeatures.cpp.erb:
• Scripts/PreferencesTemplates/WebPreferencesGetterSetters.cpp.erb:
• Scripts/PreferencesTemplates/WebPreferencesInternalDebugFeatures.cpp.erb:
• Scripts/PreferencesTemplates/WebPreferencesKeys.cpp.erb:
• Scripts/PreferencesTemplates/WebPreferencesKeys.h.erb:
• Scripts/PreferencesTemplates/WebPreferencesStoreDefaultsMap.cpp.erb:

Source/WebKitLegacy/mac:

Update to use new "exposed" prefix bindings and use default value explicitly
when updating settings for non-exposed preferences.

• Scripts/PreferencesTemplates/WebPreferencesDefinitions.h.erb:
• Scripts/PreferencesTemplates/WebPreferencesExperimentalFeatures.mm.erb:
• Scripts/PreferencesTemplates/WebPreferencesInternalFeatures.mm.erb:
• Scripts/PreferencesTemplates/WebViewPreferencesChangedGenerated.mm.erb:

Source/WTF:

• Scripts/GeneratePreferences.rb:

Tweak how the "exposed" key is interpreted to extend to keys with default values
for the current frontend (that should continue to be set on Settings for instance)
but that should not respect the key being passed in. For instance, the key
"AsyncFrameScrollingEnabled" has default values for all front ends, since we need
to set it to false in Settings when building WebKitLegacy, but is only exposed to
WebKit, so it won't be in WebKitLegacy's -[WebPreferences internalFeatures] array
and won't do anything if passed to -[WebPreferences _setBoolPreferenceForTestingWithValue:forKey:].

• Scripts/Preferences/WebPreferences.yaml:

Replace now incorrect uses of exposed with temporary key "webKitLegacyBinding" to indicate
that these keys should be valid, but currently use a custom binding in WebKitLegacy.

• Scripts/Preferences/WebPreferencesInternal.yaml:

Only expose AsyncFrameScrollingEnabled and AsyncOverflowScrollingEnabled to WebKit.
This maintains the behavior that these keys are not valid keys as test header commands
when run through DumpRenderTree.

Tools:

Update to use new "exposed" prefix bindings and use default value explicitly
when updating settings for non-exposed preferences.

• DumpRenderTree/Scripts/PreferencesTemplates/TestOptionsGeneratedKeys.h.erb:
• WebKitTestRunner/Scripts/PreferencesTemplates/TestOptionsGeneratedKeys.h.erb:

2:19 AM Changeset in webkit [268718] by svillar@igalia.com

• 9 edits
  2 deletes in trunk

[css-flexbox] flex-shrink property should be animatable
​https://bugs.webkit.org/show_bug.cgi?id=209874

Reviewed by Manuel Rego Casasnovas.

LayoutTests/imported/w3c:

Added new expectations. Replaced ~100 FAIL subtests with PASS expectations.

• web-platform-tests/css/css-flexbox/animation/flex-shrink-interpolation-expected.txt:
• web-platform-tests/web-animations/animation-model/animation-types/accumulation-per-property-001-expected.txt:
• web-platform-tests/web-animations/animation-model/animation-types/addition-per-property-001-expected.txt:
• web-platform-tests/web-animations/animation-model/animation-types/interpolation-per-property-001-expected.txt:

Source/WebCore:

Make flew-shrink animatable as per spec. Also clamp its minimum value to 0 as it cannot be a negative number.

This allows us to pass a bunch of flexbox animation subtests.

• animation/CSSPropertyAnimation.cpp:

(WebCore::CSSPropertyAnimationWrapperMap::CSSPropertyAnimationWrapperMap): Mark flex-shrink as animatable.

• rendering/style/RenderStyle.h:

(WebCore::RenderStyle::setFlexShrink): Clamp minimum value to 0.

LayoutTests:

Removed GTK's and WPE's platform expectations for some web-animation tests as both ports could make use of the cross platform ones.

• platform/gtk/imported/w3c/web-platform-tests/web-animations/animation-model/animation-types/accumulation-per-property-expected.txt: Removed.
• platform/gtk/imported/w3c/web-platform-tests/web-animations/animation-model/animation-types/addition-per-property-expected.txt: Removed.
• platform/wpe/imported/w3c/web-platform-tests/web-animations/animation-model/animation-types/accumulation-per-property-expected.txt: Removed.
• platform/wpe/imported/w3c/web-platform-tests/web-animations/animation-model/animation-types/addition-per-property-expected.txt: Removed.

2:14 AM Changeset in webkit [268717] by Carlos Garcia Campos

• 12 edits in trunk

WebDriver: add support for right variations of virtual keys
​https://bugs.webkit.org/show_bug.cgi?id=217819

Reviewed by Brian Burg.

Source/WebDriver:

The browser is currently receiving the virtual keys already normalized, but WebDriver expects the key code to be
the actual one, not the normalized one. We need to send the raw virtual keys to the browser and do the
normalization there, but use the raw one when synthesizing the events.

Fixes: imported/w3c/webdriver/tests/perform_actions/key_events.py::test_modifier_key_sends_correct_events[\ue009-CONTROL]

imported/w3c/webdriver/tests/perform_actions/key_events.py::test_modifier_key_sends_correct_events[\ue008-SHIFT]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_modifier_key_sends_correct_events[\ue052-R_ALT]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_special_key_sends_keydown[CONTROL-expected5]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_special_key_sends_keydown[R_ALT-expected48]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_special_key_sends_keydown[R_ARROWDOWN-expected49]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_special_key_sends_keydown[R_ARROWLEFT-expected50]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_special_key_sends_keydown[R_ARROWRIGHT-expected51]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_special_key_sends_keydown[R_ARROWUP-expected52]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_special_key_sends_keydown[R_DELETE-expected54]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_special_key_sends_keydown[R_END-expected55]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_special_key_sends_keydown[R_HOME-expected56]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_special_key_sends_keydown[R_INSERT-expected57]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_special_key_sends_keydown[R_PAGEDOWN-expected59]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_special_key_sends_keydown[R_PAGEUP-expected60]
imported/w3c/webdriver/tests/perform_actions/key_events.py::test_special_key_sends_keydown[SHIFT-expected64]

• Session.cpp:

(WebDriver::Session::virtualKeyForKey): Do not normalize the virtual keys.

Source/WebKit:

Instead of using a HashSet the pressed virtual keys are now stored in a HashMap where the key is the normalized
virtual key and the value is the raw virtual key.

• UIProcess/Automation/Automation.json:
• UIProcess/Automation/SimulatedInputDispatcher.cpp:

(WebKit::SimulatedInputDispatcher::transitionInputSourceToState):

• UIProcess/Automation/SimulatedInputDispatcher.h:
• UIProcess/Automation/WebAutomationSession.cpp:

(WebKit::normalizedVirtualKey):
(WebKit::WebAutomationSession::performInteractionSequence):

• UIProcess/Automation/gtk/WebAutomationSessionGtk.cpp:

(WebKit::keyCodeForVirtualKey):
(WebKit::modifiersForKeyCode):

• UIProcess/Automation/mac/WebAutomationSessionMac.mm:

(WebKit::keyCodeForVirtualKey):

• UIProcess/Automation/wpe/WebAutomationSessionWPE.cpp:

(WebKit::keyCodeForVirtualKey):
(WebKit::modifiersForKeyCode):

WebDriverTests:

Remove expectations for tests that are now passing.

• TestExpectations.json:

1:00 AM Changeset in webkit [268716] by commit-queue@webkit.org

• 16 edits
  4 adds in trunk

Web Inspector: Add setScreenSizeOverride API to the Page agent
​https://bugs.webkit.org/show_bug.cgi?id=213242

Source/JavaScriptCore:

Patch by Philippe Normand <​pnormand@igalia.com> and Pavel Feldman <​pavel.feldman@gmail.com> on 2020-10-20
Reviewed by Devin Rousso.

• inspector/protocol/Page.json: Add a new setScreenSizeOverride API in the Page agent.

Source/WebCore:

Patch by Philippe Normand <​pnormand@igalia.com> and Pavel Feldman <​pavel.feldman@gmail.com> on 2020-10-20
Reviewed by Devin Rousso.

Allow the WebInspector Page agent to override the Screen size for media queries.

Test: inspector/page/setScreenSizeOverride.html

• css/MediaQueryEvaluator.cpp: Use the Frame::screenSize() method to query the screen dimensions.

(WebCore::deviceAspectRatioEvaluate):
(WebCore::deviceHeightEvaluate):
(WebCore::deviceWidthEvaluate):

• inspector/agents/InspectorPageAgent.cpp:

(WebCore::InspectorPageAgent::setScreenSizeOverride): Implementation of the new API hooking
into the WebCore::Frame::setOverrideScreenSize() method.

• inspector/agents/InspectorPageAgent.h:
• page/Frame.cpp:

(WebCore::Frame::screenSize const): New entry point to query the screen size, this takes into account the
optional overriden screen size.
(WebCore::Frame::setOverrideScreenSize): API used by the Page agent to fake a new screen size.

• page/Frame.h:
• page/Page.cpp:
• page/Screen.cpp: Use the Frame::screenSize() method to query the screen dimensions.

(WebCore::Screen::height const):
(WebCore::Screen::width const):

Source/WebInspectorUI:

Patch by Philippe Normand <​pnormand@igalia.com> on 2020-10-20
Reviewed by Devin Rousso.

• Localizations/en.lproj/localizedStrings.js:
• UserInterface/Base/Main.js:

(WI.loaded): Add a new drop-down menu allowing to override the device screen size.

LayoutTests:

Patch by Philippe Normand <​pnormand@igalia.com> and Pavel Feldman <​pavel.feldman@gmail.com> on 2020-10-20
Reviewed by Devin Rousso.

• TestExpectations:
• inspector/page/setScreenSizeOverride.html: Added.
• platform/glib/TestExpectations:
• platform/glib/inspector/page/setScreenSizeOverride-expected.txt: Added.