Timeline
Aug 31, 2020:
- 11:56 PM Changeset in webkit [266387] by
-
- 6 edits2 adds in trunk
Ahom text is still busted
https://bugs.webkit.org/show_bug.cgi?id=216024
<rdar://problem/60571257>
Reviewed by Zalan Bujtas.
Source/WebCore:
r259533 didn't actually solve the problem becasue I made a typo.
Luckily, iOS 14 and macOS Big Sur include a font with which this patch can be tested!
So this patch actually fixes it for real.
Test: fast/text/ahom.html
- platform/graphics/FontCascade.cpp:
(WebCore::FontCascade::characterRangeCodePath):
LayoutTests:
- fast/text/ahom-expected-mismatch.html: Added.
- fast/text/ahom.html: Added.
- platform/mac/TestExpectations:
- platform/win/TestExpectations:
- platform/ios-13/TestExpectations:
- 10:19 PM Changeset in webkit [266386] by
-
- 19 edits1 delete in trunk/LayoutTests
Unreviewed, reverting r266378 and r266381.
https://bugs.webkit.org/show_bug.cgi?id=216029
rebaselined tests for new OS without leaving old baselines
behind
Reverted changesets:
"[ BigSur+ ] media/media-source/media-source-webm.html"
https://trac.webkit.org/changeset/266378
"[ BigSur+ ] platform/mac/media/media-source/is-type-
supported-vp9-codec-check.html"
https://trac.webkit.org/changeset/266381
- 9:08 PM Changeset in webkit [266385] by
-
- 2 edits in trunk/WebDriverTests
[WPE][WebDriver] Gardening a few more failures
Unreviewed test gardening.
- TestExpectations.json:
- 8:59 PM Changeset in webkit [266384] by
-
- 6 edits in trunk/Source/WebKit
Webpages flash when switching between windows
https://bugs.webkit.org/show_bug.cgi?id=216012
Reviewed by Darin Adler.
Based on patches from Tim Horton. To avoid flash, we need to make sure view to be selected updates its content
before UI process commits its transaction. We did this by making UI process block until receiving a reply
DidUpdateActivityState from web process. However, web process did not make sure the reply would be sent after
normal rendering update and corresponding transasction commit. Instead, it flushed transactions in progress and
replied. To fix this, now we make web process reply in transaction commit handler after rendering update.
In the switching case, view to be unselected will detach from root layer in its web process, which makes its
content empty. This change is independent from the UI process commit, so we want this to happen after UI process
commits (which submits the view hierachy change). Otherwise, empty content(white flash) on the unselected view
will be displayed. To fix this, we let UI process send the activity state update message in transaction commit
handler.
- UIProcess/Cocoa/WebPageProxyCocoa.mm:
(WebKit::WebPageProxy::scheduleActivityStateUpdate):
- UIProcess/WebPageProxy.cpp:
(WebKit::m_limitsNavigationsToAppBoundDomains):
(WebKit::WebPageProxy::setSuppressVisibilityUpdates):
(WebKit::WebPageProxy::activityStateDidChange):
(WebKit::WebPageProxy::dispatchActivityStateChange):
- UIProcess/WebPageProxy.h:
- WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.h:
- WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm:
(WebKit::TiledCoreAnimationDrawingArea::TiledCoreAnimationDrawingArea):
(WebKit::TiledCoreAnimationDrawingArea::handleActivityStateChangeCallbacksIfNeeded):
(WebKit::TiledCoreAnimationDrawingArea::updateRendering):
(WebKit::TiledCoreAnimationDrawingArea::activityStateDidChange):
(WebKit::TiledCoreAnimationDrawingArea::didUpdateActivityStateTimerFired): Deleted.
- 7:45 PM Changeset in webkit [266383] by
-
- 3 edits2 adds in trunk
Missing exception check while handling the onbeforeunload event.
https://bugs.webkit.org/show_bug.cgi?id=216015
<rdar://problem/66324934>
Reviewed by Saam Barati.
Source/WebCore:
Test: js/dom/missing-exception-check-while-handling-onbeforeunload.html
- bindings/js/JSEventListener.cpp:
(WebCore::JSEventListener::handleEvent):
LayoutTests:
- js/dom/missing-exception-check-while-handling-onbeforeunload-expected.txt: Added.
- js/dom/missing-exception-check-while-handling-onbeforeunload.html: Added.
- 6:18 PM Changeset in webkit [266382] by
-
- 2 edits in trunk/LayoutTests
[ BigSur+ ] platform/mac/media/mediacapabilities/vp9-decodingInfo-sw.html
rdar://65188503
Unreviewed test gardening.
- platform/mac-wk1/TestExpectations:
- 6:10 PM Changeset in webkit [266381] by
-
- 1 edit2 adds in trunk/LayoutTests
Rebaseline after macOS build progression.
Add for Big-Sur
Unreviewed test gardening.
- platform/mac/fast/forms/date/date-input-rendering-basic-expected.txt: Added.
- 5:48 PM Changeset in webkit [266380] by
-
- 2 edits in trunk/LayoutTests
[ BigSur+ ] platform/mac/media/media-source/is-type-supported-vp9-codec-check.html
rdar://65188503
Unreviewed test gardening.
- platform/mac-wk1/TestExpectations:
- 5:04 PM Changeset in webkit [266379] by
-
- 2 edits in trunk/LayoutTests
[ BigSur+ ] media/media-source/media-source-webm.html is a constant failure
rdar://problem/66487888
Unreviewed test gardening.
- platform/mac-wk1/TestExpectations:
- 5:01 PM Changeset in webkit [266378] by
-
- 19 edits in trunk/LayoutTests
Rebaseline after macOS build progression.
Add for Big-Sur
Unreviewed test gardening.
- platform/mac-bigsur/fast/block/margin-collapse/103-expected.txt:
- platform/mac/fast/css/continuationCrash-expected.txt:
- platform/mac/fast/css/text-overflow-input-expected.txt:
- platform/mac/fast/forms/button-positioned-expected.txt:
- platform/mac/fast/forms/button-sizes-expected.txt:
- platform/mac/fast/forms/input-button-sizes-expected.txt:
- platform/mac/fast/forms/input-disabled-color-expected.txt:
- platform/mac/fast/forms/input-readonly-dimmed-expected.txt:
- platform/mac/fast/forms/input-text-word-wrap-expected.txt:
- platform/mac/fast/forms/listbox-bidi-align-expected.txt:
- platform/mac/fast/forms/plaintext-mode-2-expected.txt:
- platform/mac/fast/forms/select-change-popup-to-listbox-expected.txt:
- platform/mac/fast/forms/select-selected-expected.txt:
- platform/mac/fast/forms/select/optgroup-rendering-expected.txt:
- platform/mac/fast/parser/document-write-option-expected.txt:
- platform/mac/fast/text/international/pop-up-button-text-alignment-and-direction-expected.txt:
- platform/mac/tables/mozilla/bugs/bug2479-3-expected.txt:
- platform/mac/tables/mozilla/bugs/bug33855-expected.txt:
- 4:36 PM Changeset in webkit [266377] by
-
- 8 edits in branches/safari-610.1.28.1-branch/Source
Versioning.
WebKit-7610.1.28.1.6
- 4:36 PM Changeset in webkit [266376] by
-
- 2 edits in trunk/LayoutTests
Unreviewed, reverting r266374.
Revision set up incorrect expectation
Reverted changeset:
"[ BigSur+ ] media/media-source/media-source-webm.html"
https://trac.webkit.org/changeset/266374
- 4:34 PM Changeset in webkit [266375] by
-
- 2 edits in trunk/Source/WebCore
Avoid computing metadata for idempotent text autosizing on macOS
https://bugs.webkit.org/show_bug.cgi?id=216011
Reviewed by Darin Adler.
Avoids an unnecessary call to
adjustForTextAutosizing
during style resolution on macOS, where idempotent text
autosizing is disabled. This function call populates theAutosizeStatus
fields onRenderStyle
that are used
by the idempotent text autosizing heuristic. This function call appears consistently in traces taken while
running the Multiply subtest of MotionMark, if only for a relatively small number of samples. While
AutosizeStatus::computeStatus
is designed to be very cheap, it ends up being called around 1.6 million times
over the course of the subtest.
- style/StyleAdjuster.cpp:
(WebCore::Style::Adjuster::adjust const):
- 4:24 PM Changeset in webkit [266374] by
-
- 2 edits in trunk/LayoutTests
[ BigSur+ ] media/media-source/media-source-webm.html
rdar://66487888
Unreviewed test gardening.
- platform/mac/TestExpectations:
- 4:19 PM Changeset in webkit [266373] by
-
- 2 edits in trunk/Source/WebCore
Unreviewed. Remove the unused variable build warning since r238538.
No new tests, no new behaviors.
- html/HTMLImageElement.cpp:
(WebCore::HTMLImageElement::copyNonAttributePropertiesFromElement):
- 4:16 PM Changeset in webkit [266372] by
-
- 5 edits in trunk/Tools
[webkitcorepy] Register another package in the WebKit repository
https://bugs.webkit.org/show_bug.cgi?id=216013
<rdar://problem/68097461>
Reviewed by Dewei Zhu.
- Scripts/libraries/webkitcorepy/webkitcorepy/init.py: Bump version.
- Scripts/libraries/webkitcorepy/webkitcorepy/autoinstall.py:
(AutoInstall.register): If a package is also checked into the webkit repository,
we prefer the checked in version to the auto-installed version.
- Scripts/webkitpy/init.py: Add webkitscmpy dependency.
- Scripts/webkitpy/thirdparty/init.py:
(AutoinstallImportHook._ensure_autoinstalled_dir_is_in_sys_path): Handle case where the autoinstalled
directory is no the first in the Python path.
- 3:16 PM Changeset in webkit [266371] by
-
- 2 edits in trunk/LayoutTests
[ macOS ] Tests expectations changed as test passing but expected to fail
https://bugs.webkit.org/show_bug.cgi?id=215786
Unreviewed test gardening.
- platform/mac/TestExpectations: correction
- 3:04 PM Changeset in webkit [266370] by
-
- 2 edits in trunk/LayoutTests
[ Mac wk2 ] platform/mac/webrtc/captureCanvas-webrtc-software-encoder.html
rdar://problem/66492801
Unreviewed test gardening.
- platform/mac-wk2/TestExpectations:
- 2:18 PM Changeset in webkit [266369] by
-
- 2 edits in trunk/Source/WebCore
Unreviewed build fix after http://trac.webkit.org/changeset/266364/webkit.
- DerivedSources-input.xcfilelist:
- DerivedSources-output.xcfilelist:
Regenerated these files.
- 2:14 PM Changeset in webkit [266368] by
-
- 5 edits in trunk/LayoutTests
[GLIB] Gardening some failures and flakies
Unreviewed test gardening.
- platform/glib/TestExpectations:
- platform/gtk-wayland/TestExpectations:
- platform/gtk/TestExpectations:
- platform/wpe/TestExpectations:
- 1:25 PM Changeset in webkit [266367] by
-
- 4 edits in trunk/LayoutTests
Update imported/w3c/web-platform-tests/webrtc/RTCPeerConnection-mandatory-getStats.https.html so that it no longer times out if first test fails
https://bugs.webkit.org/show_bug.cgi?id=216009
Reviewed by Darin Adler.
LayoutTests/imported/w3c:
- web-platform-tests/webrtc/RTCPeerConnection-mandatory-getStats.https-expected.txt:
- web-platform-tests/webrtc/RTCPeerConnection-mandatory-getStats.https.html:
Wrap internal asserts in first test in its own test wrapper.
Migrate all promise tests to simple synchronous tests.
LayoutTests:
- 12:54 PM Changeset in webkit [266366] by
-
- 4 edits in trunk/LayoutTests
Unreviewed, address flakiness of imported/w3c/web-platform-tests/webaudio/the-audio-api/the-analysernode-interface/test-analyser-minimum.html
LayoutTests/imported/w3c:
- web-platform-tests/webaudio/the-audio-api/the-analysernode-interface/test-analyser-minimum-expected.txt:
LayoutTests:
- 12:37 PM Changeset in webkit [266365] by
-
- 2 edits in trunk/LayoutTests
https://bugs.webkit.org/show_bug.cgi?id=215909
Removed test expectations
Unreviewed test gardening.
- platform/wincairo/TestExpectations:
- 11:59 AM Changeset in webkit [266364] by
-
- 13 edits3 adds in trunk/Source
Implement WEBGL_compressed_texture_s3tc_srgb extension
https://bugs.webkit.org/show_bug.cgi?id=215973
Patch by James Darpinian <James Darpinian> on 2020-08-31
Reviewed by Dean Jackson.
Source/ThirdParty/ANGLE:
- src/libANGLE/renderer/gl/formatutilsgl.cpp:
(rx::nativegl::ExtAndVersionOrExt):
(rx::nativegl::BuildInternalFormatInfoMap):
Source/WebCore:
Tested by webgl/2.0.0/conformance/extensions/webgl-compressed-texture-s3tc-srgb.html
- CMakeLists.txt:
- DerivedSources.make:
- Sources.txt:
- WebCore.xcodeproj/project.pbxproj:
- bindings/js/JSDOMConvertWebGL.cpp:
(WebCore::convertToJSValue):
- html/canvas/WebGLCompressedTextureS3TCsRGB.cpp: Added.
(WebCore::WebGLCompressedTextureS3TCsRGB::WebGLCompressedTextureS3TCsRGB):
(WebCore::WebGLCompressedTextureS3TCsRGB::getName const):
(WebCore::WebGLCompressedTextureS3TCsRGB::supported):
- html/canvas/WebGLCompressedTextureS3TCsRGB.h: Added.
- html/canvas/WebGLCompressedTextureS3TCsRGB.idl: Added.
- html/canvas/WebGLExtension.h:
- html/canvas/WebGLRenderingContext.cpp:
(WebCore::WebGLRenderingContext::getExtension):
(WebCore::WebGLRenderingContext::getSupportedExtensions):
- html/canvas/WebGLRenderingContextBase.cpp:
(WebCore::WebGLRenderingContextBase::extensionIsEnabled):
(WebCore::WebGLRenderingContextBase::validateCompressedTexFuncData):
(WebCore::WebGLRenderingContextBase::validateCompressedTexDimensions):
(WebCore::WebGLRenderingContextBase::validateCompressedTexSubDimensions):
- html/canvas/WebGLRenderingContextBase.h:
- platform/graphics/ExtensionsGL.h:
- 11:51 AM Changeset in webkit [266363] by
-
- 3 edits in trunk/Source/WebCore
Wheel event region code is not fully disabled
https://bugs.webkit.org/show_bug.cgi?id=216008
<rdar://problem/68086333>
Reviewed by Simon Fraser.
- rendering/RenderBlock.cpp:
(WebCore::RenderBlock::paintObject):
- rendering/RenderLayerBacking.cpp:
(WebCore::RenderLayerBacking::maintainsEventRegion const):
Use the relevant ENABLE flag rather than !PLATFORM_IOS to compile out the code.
- 11:28 AM Changeset in webkit [266362] by
-
- 3 edits2 adds in trunk
WebGL goes in a bad state where glContext.createProgram() returns null
https://bugs.webkit.org/show_bug.cgi?id=215844
Patch by James Darpinian <James Darpinian> on 2020-08-31
Reviewed by Kenneth Russell.
Added test webgl/2.0.0/conformance/canvas/render-after-resize-test.html
- platform/graphics/angle/GraphicsContextGLANGLE.cpp:
(WebCore::GraphicsContextGLOpenGL::reshape):
- 11:23 AM Changeset in webkit [266361] by
-
- 5 edits in trunk/Source/WebKit
Upstream support for image picker confirmation and downscaling
https://bugs.webkit.org/show_bug.cgi?id=215997
Reviewed by Anders Carlsson.
- UIProcess/ios/WKContentViewInteraction.h:
- UIProcess/ios/WKContentViewInteraction.mm:
(-[WKContentView _showRunOpenPanel:frameInfo:resultListener:]):
- UIProcess/ios/forms/WKFileUploadPanel.mm:
(-[WKFileUploadPanel _showPhotoPickerWithSourceType:]):
(-[WKFileUploadPanel _configureImagePicker:]): Deleted.
- 11:16 AM Changeset in webkit [266360] by
-
- 103 edits in trunk/Source
Many settings in RuntimeEnabledSettings should autogenerated Settings: Part 1
https://bugs.webkit.org/show_bug.cgi?id=215981
Reviewed by Darin Adler.
This moves an initial chunk of RuntimeEnabledSettings settings to Settings.yaml, focusing
on settings that don't require substantial refactoring or new features to work in Settings.yaml.
Source/WebCore:
The one "substantial" change to Settings generation was to switch the conditional key, to require
the complete conditional form (e.g. require ENABLE(FOO) rather than FOO), which matches
WebPreferences.yaml in WebKit, and is more versitile allowing non-ENABLE based conditionals.
- Modules/credentialmanagement/BasicCredential.idl:
- Modules/credentialmanagement/CredentialsContainer.idl:
- Modules/credentialmanagement/NavigatorCredentials.idl:
- Modules/encryptedmedia/MediaKeyMessageEvent.idl:
- Modules/encryptedmedia/MediaKeySession.idl:
- Modules/encryptedmedia/MediaKeyStatusMap.idl:
- Modules/encryptedmedia/MediaKeySystemAccess.idl:
- Modules/encryptedmedia/MediaKeys.idl:
- Modules/encryptedmedia/NavigatorEME.idl:
- Modules/encryptedmedia/legacy/WebKitMediaKeyMessageEvent.idl:
- Modules/encryptedmedia/legacy/WebKitMediaKeyNeededEvent.idl:
- Modules/encryptedmedia/legacy/WebKitMediaKeySession.idl:
- Modules/encryptedmedia/legacy/WebKitMediaKeys.idl:
- Modules/gamepad/Gamepad.idl:
- Modules/gamepad/GamepadButton.idl:
- Modules/gamepad/GamepadEvent.idl:
- Modules/gamepad/NavigatorGamepad.idl:
- Modules/mediarecorder/BlobEvent.idl:
- Modules/mediarecorder/MediaRecorder.idl:
- Modules/mediarecorder/MediaRecorderErrorEvent.idl:
- Modules/mediastream/MediaDevices.idl:
- Modules/mediastream/MediaStreamTrack.cpp:
(WebCore::MediaStreamTrack::updateCaptureAccordingToMutedState):
- Modules/mediastream/NavigatorMediaDevices.idl:
- Modules/webauthn/AuthenticatorAssertionResponse.idl:
- Modules/webauthn/AuthenticatorAttestationResponse.idl:
- Modules/webauthn/AuthenticatorResponse.idl:
- Modules/webauthn/PublicKeyCredential.cpp:
(WebCore::PublicKeyCredential::isUserVerifyingPlatformAuthenticatorAvailable):
- Modules/webauthn/PublicKeyCredential.idl:
- Scripts/SettingsTemplates/InternalSettingsGenerated.cpp.erb:
- Scripts/SettingsTemplates/InternalSettingsGenerated.h.erb:
- Scripts/SettingsTemplates/Settings.cpp.erb:
- Scripts/SettingsTemplates/Settings.h.erb:
- WebCore.xcodeproj/project.pbxproj:
- css/DOMCSSCustomPropertyDescriptor.idl:
- css/DOMCSSRegisterCustomProperty.idl:
- dom/BeforeLoadEvent.idl:
- dom/CustomElementRegistry.idl:
- dom/DataTransfer.idl:
- dom/DataTransferItem.idl:
- dom/DataTransferItemList.idl:
- dom/Document.cpp:
(WebCore::createUpgradeCandidateElement):
- dom/Document.idl:
- dom/Element.idl:
- dom/InputEvent.idl:
- dom/Node.cpp:
(WebCore::Node::dispatchBeforeLoadEvent):
- dom/StaticRange.idl:
- editing/Editor.cpp:
(WebCore::Editor::registerCustomUndoStep):
- html/HTMLAnchorElement.cpp:
(WebCore::HTMLAnchorElement::isSystemPreviewLink):
(WebCore::HTMLAnchorElement::parseAdClickAttribution const):
(WebCore::HTMLAnchorElement::handleClick):
(WebCore::HTMLAnchorElement::referrerPolicy const):
- html/HTMLAnchorElement.idl:
- html/HTMLAreaElement.idl:
- html/HTMLButtonElement.idl:
- html/HTMLFieldSetElement.idl:
- html/HTMLFormElement.idl:
- html/HTMLIFrameElement.cpp:
(WebCore::HTMLIFrameElement::referrerPolicy const):
- html/HTMLIFrameElement.idl:
- html/HTMLImageElement.cpp:
(WebCore::HTMLImageElement::attributeChanged):
(WebCore::HTMLImageElement::isSystemPreviewImage const):
(WebCore::HTMLImageElement::referrerPolicy const):
- html/HTMLImageElement.idl:
- html/HTMLInputElement.idl:
- html/HTMLKeygenElement.idl:
- html/HTMLLinkElement.cpp:
(WebCore::HTMLLinkElement::as const):
(WebCore::HTMLLinkElement::referrerPolicy const):
- html/HTMLLinkElement.idl:
- html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::updateShouldContinueAfterNeedKey):
(WebCore::HTMLMediaElement::mediaPlayerKeyNeeded):
(WebCore::HTMLMediaElement::webkitSetMediaKeys):
(WebCore::HTMLMediaElement::keyAdded):
(WebCore::HTMLMediaElement::mediaPlayerInitializationDataEncountered):
- html/HTMLMediaElement.idl:
- html/HTMLMetaElement.cpp:
(WebCore::HTMLMetaElement::process):
- html/HTMLObjectElement.idl:
- html/HTMLOutputElement.idl:
- html/HTMLPictureElement.cpp:
(WebCore::HTMLPictureElement::isSystemPreviewImage):
- html/HTMLScriptElement.cpp:
(WebCore::HTMLScriptElement::referrerPolicy const):
- html/HTMLScriptElement.idl:
- html/HTMLSelectElement.idl:
- html/HTMLTextAreaElement.idl:
- html/LinkRelAttribute.cpp:
(WebCore::LinkRelAttribute::LinkRelAttribute):
(WebCore::LinkRelAttribute::isSupported):
- html/MediaEncryptedEvent.idl:
- html/WebKitMediaKeyError.idl:
- html/parser/HTMLPreloadScanner.cpp:
(WebCore::TokenPreloadScanner::StartTagScanner::StartTagScanner):
(WebCore::TokenPreloadScanner::StartTagScanner::processAttributes):
(WebCore::TokenPreloadScanner::StartTagScanner::createPreloadRequest):
(WebCore::TokenPreloadScanner::StartTagScanner::processAttribute):
(WebCore::TokenPreloadScanner::StartTagScanner::resourceType const):
(WebCore::TokenPreloadScanner::scan):
- loader/ImageLoader.cpp:
(WebCore::ImageLoader::updateFromElement):
- loader/LinkLoader.cpp:
(WebCore::LinkLoader::resourceTypeFromAsAttribute):
(WebCore::createLinkPreloadResourceClient):
(WebCore::LinkLoader::isSupportedType):
(WebCore::LinkLoader::preloadIfNeeded):
(WebCore::LinkLoader::prefetchIfNeeded):
- loader/LinkLoader.h:
- page/DOMWindow.idl:
- page/IntersectionObserver.idl:
- page/IntersectionObserverEntry.idl:
- page/Navigator.cpp:
(WebCore::Navigator::canShare):
(WebCore::Navigator::share):
- page/Navigator.h:
- page/NavigatorShare.idl:
- page/RuntimeEnabledFeatures.cpp:
(WebCore::RuntimeEnabledFeatures::RuntimeEnabledFeatures):
(WebCore::RuntimeEnabledFeatures::touchEventsEnabled const): Deleted.
- page/RuntimeEnabledFeatures.h:
(WebCore::RuntimeEnabledFeatures::setMouseEventsSimulationEnabled):
(WebCore::RuntimeEnabledFeatures::setBlankAnchorTargetImpliesNoOpenerEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::blankAnchorTargetImpliesNoOpenerEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setLinkPreloadEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::linkPreloadEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setLinkPrefetchEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::linkPrefetchEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setMediaPreloadingEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::mediaPreloadingEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setInputEventsEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::inputEventsEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setInteractiveFormValidationEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::interactiveFormValidationEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setCustomElementsEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::customElementsEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setDataTransferItemsEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::dataTransferItemsEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setWebShareEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::webShareEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setWebAuthenticationEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::webAuthenticationEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setWebAuthenticationLocalAuthenticatorEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::webAuthenticationLocalAuthenticatorEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setDisabledAdaptationsMetaTagEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::disabledAdaptationsMetaTagEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setCSSCustomPropertiesAndValuesEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::cssCustomPropertiesAndValuesEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setSyntheticEditingCommandsEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::syntheticEditingCommandsEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::mediaRecorderEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setMediaRecorderEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::mediaDevicesEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setMediaDevicesEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::mediaStreamEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setMediaStreamEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::screenCaptureEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setScreenCaptureEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::setGamepadsEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::gamepadsEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setDownloadAttributeEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::downloadAttributeEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setIntersectionObserverEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::intersectionObserverEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setUndoManagerAPIEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::undoManagerAPIEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setEncryptedMediaAPIEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::encryptedMediaAPIEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setLegacyEncryptedMediaAPIEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::legacyEncryptedMediaAPIEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setSystemPreviewEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::systemPreviewEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setLegacyBeforeLoadEventEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::legacyBeforeLoadEventEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::adClickAttributionEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setAdClickAttributionEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::setTouchEventsEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::referrerPolicyAttributeEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setReferrerPolicyAttributeEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::interruptAudioOnPageVisibilityChangeEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setInterruptAudioOnPageVisibilityChangeEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::linkPreloadResponsiveImagesEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setLinkPreloadResponsiveImagesEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::setLazyImageLoadingEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::lazyImageLoadingEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setIsInAppBrowserPrivacyEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::isInAppBrowserPrivacyEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setNeedsInAppBrowserPrivacyQuirks): Deleted.
(WebCore::RuntimeEnabledFeatures::needsInAppBrowserPrivacyQuirks const): Deleted.
(WebCore::RuntimeEnabledFeatures::setWebShareFileAPIEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::webShareFileAPIEnabled const): Deleted.
- page/Settings.yaml:
- page/UndoItem.idl:
- page/UndoManager.idl:
- rendering/RenderImage.cpp:
(WebCore::RenderImage::paintIntoRect):
- testing/InternalSettings.cpp:
(WebCore::InternalSettings::Backup::Backup):
(WebCore::InternalSettings::Backup::restoreTo):
(WebCore::InternalSettings::setScreenCaptureEnabled): Deleted.
- testing/InternalSettings.h:
- testing/InternalSettings.idl:
- testing/Internals.cpp:
(WebCore::Internals::resetToConsistentState):
(WebCore::Internals::setShouldInterruptAudioOnPageVisibilityChange):
(WebCore::Internals::addPrefetchLoadEventListener):
Source/WebKit:
In WebKit, this means mostly removing now unnecessary use of the webcoreBinding key in
WebPreferences.yaml and removing now superfluous cases in InjectedBundle::overrideBoolPreferenceForTestRunner.
- Shared/WebPreferences.yaml:
Removing now unnecessary use of the webcoreBinding key and fix indentation of a few cases.
- WebProcess/InjectedBundle/InjectedBundle.cpp:
(WebKit::InjectedBundle::overrideBoolPreferenceForTestRunner):
Remove now superfluous cases, as each boolean Settings based setting gets handled automatically.
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::updatePreferences):
Remove explicit setting of setInteractiveFormValidationEnabled now that we don't have the
RuntimeEnabledSettings variant being set by the generator.
Source/WebKitLegacy/mac:
- WebView/WebView.mm:
(-[WebView _preferencesChanged:]):
Update changed settings to use Settings rather than RuntimeEnabledSettings.
Source/WebKitLegacy/win:
- WebView.cpp:
(WebView::notifyPreferencesChanged):
Update changed settings to use Settings rather than RuntimeEnabledSettings.
- 10:43 AM Changeset in webkit [266359] by
-
- 21 edits in trunk/Source/JavaScriptCore
[JSC] StructureStubInfo / CallLinkInfo / ByValInfo should set CodeOrigin or BytecodeIndex at construction
https://bugs.webkit.org/show_bug.cgi?id=215987
<rdar://problem/66370323>
Reviewed by Mark Lam.
We had race condition during construction of StructureStubInfo and CodeOrigin field setting.
- The thread creates StructureStubInfo by calling CodeBlock::addStubInfo. This is guarded by the lock. But at this point we are not setting StructureStubInfo::codeOrigin.
- Then (1)'s thread attempts to set StructureStubInfo::codeOrigin. But at this point, it is not guarded by the lock.
- Before (2) is executed, DFG ByteCodeParser calls CodeBlock::getICStatusMap. It creates HashMap<CodeOrigin, StructureStubInfo*>.
- Since StructureStubInfo*'s codeOrigin is not configured yet, (3) sees invalid CodeOrigin. And storing invalid CodeOrigin as a HashMap key is not correct.
We should configure CodeOrigin at construction of StructureStubInfo, which is guarded by the lock. We have the same problem for CallLinkInfo and ByValInfo. This patch fixes them.
To reproduce this, we need to execute a script 2~ days repeatedly. So it is difficult to add a test.
- bytecode/AccessCase.cpp:
(JSC::AccessCase::generateImpl):
- bytecode/ByValInfo.h:
(JSC::ByValInfo::ByValInfo):
(JSC::ByValInfo::setUp):
- bytecode/CallLinkInfo.cpp:
(JSC::CallLinkInfo::CallLinkInfo):
- bytecode/CallLinkInfo.h:
(JSC::CallLinkInfo::setUpCall):
(JSC::CallLinkInfo::setCodeOrigin): Deleted.
- bytecode/CodeBlock.cpp:
(JSC::CodeBlock::addStubInfo):
(JSC::CodeBlock::addByValInfo):
(JSC::CodeBlock::addCallLinkInfo):
- bytecode/CodeBlock.h:
- bytecode/StructureStubInfo.cpp:
(JSC::StructureStubInfo::StructureStubInfo):
- bytecode/StructureStubInfo.h:
- dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::emitCall):
- dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::emitCall):
- ftl/FTLLowerDFGToB3.cpp:
(JSC::FTL::DFG::LowerDFGToB3::compileCallOrConstruct):
(JSC::FTL::DFG::LowerDFGToB3::compileDirectCallOrConstruct):
(JSC::FTL::DFG::LowerDFGToB3::compileTailCall):
(JSC::FTL::DFG::LowerDFGToB3::compileCallOrConstructVarargsSpread):
(JSC::FTL::DFG::LowerDFGToB3::compileCallOrConstructVarargs):
(JSC::FTL::DFG::LowerDFGToB3::compileCallEval):
- jit/JIT.cpp:
(JSC::JIT::link):
- jit/JITCall.cpp:
(JSC::JIT::compileCallEvalSlowCase):
(JSC::JIT::compileOpCall):
- jit/JITCall32_64.cpp:
(JSC::JIT::compileCallEvalSlowCase):
(JSC::JIT::compileOpCall):
- jit/JITInlineCacheGenerator.cpp:
(JSC::garbageStubInfo):
(JSC::JITInlineCacheGenerator::JITInlineCacheGenerator):
- jit/JITOpcodes.cpp:
(JSC::JIT::emit_op_has_indexed_property):
- jit/JITOpcodes32_64.cpp:
(JSC::JIT::emit_op_has_indexed_property):
- jit/JITPropertyAccess.cpp:
(JSC::JIT::emit_op_put_by_val):
- jit/JITPropertyAccess32_64.cpp:
(JSC::JIT::emit_op_put_by_val):
- wasm/js/WasmToJS.cpp:
(JSC::Wasm::wasmToJS):
- 10:22 AM Changeset in webkit [266358] by
-
- 56 edits in trunk
Unreviewed, reverting r266350.
Revision is causing a constant crash on both macOS and iOS
Reverted changeset:
"Convert runtime flag to setting for lazy image loading"
https://bugs.webkit.org/show_bug.cgi?id=215610
https://trac.webkit.org/changeset/266350
- 9:56 AM Changeset in webkit [266357] by
-
- 2 edits in trunk
[PlayStation] Build fix for intermittent copy errors
https://bugs.webkit.org/show_bug.cgi?id=215951
Reviewed by Ross Kirsling.
- Source/cmake/OptionsPlayStation.cmake:
When copying platform specific files into the binary
directory post build on executable targets, we'd
intermittently get copy errors in parallel builds.
- 9:47 AM Changeset in webkit [266356] by
-
- 3 edits80 copies22 adds1 delete in trunk/LayoutTests
Import css-conditional tests.
https://bugs.webkit.org/show_bug.cgi?id=216000
Reviewed by Darin Adler.
LayoutTests/imported/w3c:
- web-platform-tests/css/conditional/LICENSE: Renamed from LayoutTests/css3/conditional/w3c/LICENSE.
- web-platform-tests/css/conditional/META.yml: Added.
- web-platform-tests/css/conditional/at-media-whitespace-optional-001-expected.html: Added.
- web-platform-tests/css/conditional/at-media-whitespace-optional-001.html: Added.
- web-platform-tests/css/conditional/at-media-whitespace-optional-002-expected.html: Added.
- web-platform-tests/css/conditional/at-media-whitespace-optional-002.html: Added.
- web-platform-tests/css/conditional/at-supports-001-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-001.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-001.html.
- web-platform-tests/css/conditional/at-supports-002-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-002.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-002.html.
- web-platform-tests/css/conditional/at-supports-003-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-003.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-003.html.
- web-platform-tests/css/conditional/at-supports-004-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-004.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-004.html.
- web-platform-tests/css/conditional/at-supports-005-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-005.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-005.html.
- web-platform-tests/css/conditional/at-supports-006-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-006.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-006.html.
- web-platform-tests/css/conditional/at-supports-007-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-007.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-007.html.
- web-platform-tests/css/conditional/at-supports-008-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-008.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-008.html.
- web-platform-tests/css/conditional/at-supports-009-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-009.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-009.html.
- web-platform-tests/css/conditional/at-supports-010-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-010.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-010.html.
- web-platform-tests/css/conditional/at-supports-011-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-011.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-011.html.
- web-platform-tests/css/conditional/at-supports-012-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-012.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-012.html.
- web-platform-tests/css/conditional/at-supports-013-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-013.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-013.html.
- web-platform-tests/css/conditional/at-supports-014-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-014.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-014.html.
- web-platform-tests/css/conditional/at-supports-015-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-015.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-015.html.
- web-platform-tests/css/conditional/at-supports-016-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-016.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-016.html.
- web-platform-tests/css/conditional/at-supports-017-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-017.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-017.html.
- web-platform-tests/css/conditional/at-supports-018-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-018.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-018.html.
- web-platform-tests/css/conditional/at-supports-019-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-019.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-019.html.
- web-platform-tests/css/conditional/at-supports-020-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-020.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-020.html.
- web-platform-tests/css/conditional/at-supports-021-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-021.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-021.html.
- web-platform-tests/css/conditional/at-supports-022-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-022.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-022.html.
- web-platform-tests/css/conditional/at-supports-023-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-023.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-023.html.
- web-platform-tests/css/conditional/at-supports-024-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-024.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-024.html.
- web-platform-tests/css/conditional/at-supports-025-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-025.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-025.html.
- web-platform-tests/css/conditional/at-supports-026-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-026.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-026.html.
- web-platform-tests/css/conditional/at-supports-027-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-027.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-027.html.
- web-platform-tests/css/conditional/at-supports-028-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-028.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-028.html.
- web-platform-tests/css/conditional/at-supports-029-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-029.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-029.html.
- web-platform-tests/css/conditional/at-supports-030-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-030.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-030.html.
- web-platform-tests/css/conditional/at-supports-031-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-031.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-031.html.
- web-platform-tests/css/conditional/at-supports-032-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-032.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-032.html.
- web-platform-tests/css/conditional/at-supports-033-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-033.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-033.html.
- web-platform-tests/css/conditional/at-supports-034-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-034.html: Added.
- web-platform-tests/css/conditional/at-supports-035-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-035.html: Added.
- web-platform-tests/css/conditional/at-supports-036-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-036.html: Added.
- web-platform-tests/css/conditional/at-supports-037-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-037.html: Added.
- web-platform-tests/css/conditional/at-supports-038-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-038.html: Added.
- web-platform-tests/css/conditional/at-supports-039-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-039.html: Added.
- web-platform-tests/css/conditional/at-supports-040-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-040.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-040.html.
- web-platform-tests/css/conditional/at-supports-041-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-041.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-041.html.
- web-platform-tests/css/conditional/at-supports-042-expected.html: Copied from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-042.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-042.html.
- web-platform-tests/css/conditional/at-supports-043-expected.html: Renamed from LayoutTests/css3/conditional/w3c/at-supports-041-expected.html.
- web-platform-tests/css/conditional/at-supports-043.html: Added.
- web-platform-tests/css/conditional/idlharness-expected.txt: Added.
- web-platform-tests/css/conditional/idlharness.html: Added.
- web-platform-tests/css/conditional/js/001-expected.txt: Added.
- web-platform-tests/css/conditional/js/001.html: Added.
- web-platform-tests/css/conditional/reference/background-lime.html: Added.
- web-platform-tests/css/conditional/test_group_insertRule-expected.txt: Added.
- web-platform-tests/css/conditional/test_group_insertRule.html: Added.
LayoutTests:
- TestExpectations:
- css3/conditional/w3c/at-supports-001-expected.html: Removed.
- css3/conditional/w3c/at-supports-002-expected.html: Removed.
- css3/conditional/w3c/at-supports-003-expected.html: Removed.
- css3/conditional/w3c/at-supports-004-expected.html: Removed.
- css3/conditional/w3c/at-supports-005-expected.html: Removed.
- css3/conditional/w3c/at-supports-006-expected.html: Removed.
- css3/conditional/w3c/at-supports-007-expected.html: Removed.
- css3/conditional/w3c/at-supports-008-expected.html: Removed.
- css3/conditional/w3c/at-supports-009-expected.html: Removed.
- css3/conditional/w3c/at-supports-010-expected.html: Removed.
- css3/conditional/w3c/at-supports-011-expected.html: Removed.
- css3/conditional/w3c/at-supports-012-expected.html: Removed.
- css3/conditional/w3c/at-supports-013-expected.html: Removed.
- css3/conditional/w3c/at-supports-014-expected.html: Removed.
- css3/conditional/w3c/at-supports-015-expected.html: Removed.
- css3/conditional/w3c/at-supports-016-expected.html: Removed.
- css3/conditional/w3c/at-supports-017-expected.html: Removed.
- css3/conditional/w3c/at-supports-018-expected.html: Removed.
- css3/conditional/w3c/at-supports-019-expected.html: Removed.
- css3/conditional/w3c/at-supports-020-expected.html: Removed.
- css3/conditional/w3c/at-supports-021-expected.html: Removed.
- css3/conditional/w3c/at-supports-022-expected.html: Removed.
- css3/conditional/w3c/at-supports-023-expected.html: Removed.
- css3/conditional/w3c/at-supports-024-expected.html: Removed.
- css3/conditional/w3c/at-supports-025-expected.html: Removed.
- css3/conditional/w3c/at-supports-026-expected.html: Removed.
- css3/conditional/w3c/at-supports-027-expected.html: Removed.
- css3/conditional/w3c/at-supports-028-expected.html: Removed.
- css3/conditional/w3c/at-supports-029-expected.html: Removed.
- css3/conditional/w3c/at-supports-030-expected.html: Removed.
- css3/conditional/w3c/at-supports-031-expected.html: Removed.
- css3/conditional/w3c/at-supports-032-expected.html: Removed.
- css3/conditional/w3c/at-supports-033-expected.html: Removed.
- css3/conditional/w3c/at-supports-040-expected.html: Removed.
- css3/conditional/w3c/at-supports-042-expected.html: Removed.
- 9:13 AM Changeset in webkit [266355] by
-
- 4 edits in trunk/Source/WebCore
DeprecatedGlobalSettings::globalConstRedeclarationShouldThrow() is not actually a setting, so should not be in DeprecatedGlobalSettings
https://bugs.webkit.org/show_bug.cgi?id=215978
Reviewed by Darin Adler.
Move behavior quirk defined by DeprecatedGlobalSettings::globalConstRedeclarationShouldThrow() to its
only call site in CommonVM.cpp
- bindings/js/CommonVM.cpp:
(WebCore::globalConstRedeclarationShouldThrow):
(WebCore::commonVMSlow):
- page/DeprecatedGlobalSettings.cpp:
(WebCore::DeprecatedGlobalSettings::globalConstRedeclarationShouldThrow): Deleted.
- page/DeprecatedGlobalSettings.h:
- 8:51 AM Changeset in webkit [266354] by
-
- 20 edits1 add in trunk
Make Big5 encoder conform to the specification and behavior of Chrome and Firefox
https://bugs.webkit.org/show_bug.cgi?id=215983
Reviewed by Darin Adler.
LayoutTests/imported/w3c:
- web-platform-tests/encoding/big5-encoder-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-cn-big5-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-csbig5-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-errors-han-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-errors-misc-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-errors-pua-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-x-x-big5-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-href-errors-han-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-href-errors-misc-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-href-expected.txt:
Source/WebCore:
Covered by web platform tests.
- platform/text/EncodingTables.cpp:
- platform/text/EncodingTables.h:
- platform/text/TextCodecICU.cpp:
(WebCore::eucJPEncode):
(WebCore::big5Encode):
(WebCore::uncheckedAppendDecimal):
(WebCore::urlEncodedEntityUnencodableHandler):
(WebCore::entityUnencodableHandler):
(WebCore::questionMarkUnencodableHandler):
(WebCore::Function<void):
(WebCore::TextCodecICU::encode):
Source/WTF:
- WTF.xcodeproj/project.pbxproj:
- wtf/CMakeLists.txt:
- wtf/URLParser.cpp:
(WTF::CodePointIterator::CodePointIterator): Deleted.
(WTF::CodePointIterator::operator== const): Deleted.
(WTF::CodePointIterator::operator!= const): Deleted.
(WTF::CodePointIterator::atEnd const): Deleted.
(WTF::CodePointIterator::codeUnitsSince const): Deleted.
(WTF::CodePointIterator<LChar>::operator const): Deleted.
(WTF::CodePointIterator<LChar>::operator): Deleted.
(WTF::CodePointIterator<UChar>::operator const): Deleted.
(WTF::CodePointIterator<UChar>::operator): Deleted.
- wtf/text/CodePointIterator.h: Added.
(WTF::CodePointIterator::CodePointIterator):
(WTF::CodePointIterator::operator== const):
(WTF::CodePointIterator::operator!= const):
(WTF::CodePointIterator::atEnd const):
(WTF::CodePointIterator::codeUnitsSince const):
(WTF::CodePointIterator<LChar>::operator const):
(WTF::CodePointIterator<LChar>::operator):
(WTF::CodePointIterator<UChar>::operator const):
(WTF::CodePointIterator<UChar>::operator):
- 8:14 AM Changeset in webkit [266353] by
-
- 4 edits in trunk/Source/WebCore
Optimize the implementation of TransformationMatrix::rotate(double)
https://bugs.webkit.org/show_bug.cgi?id=215994
Reviewed by Tim Horton.
TransformationMatrix::rotate(angle) currently just calls TransformationMatrix::rotate3d(0, 0, 1, angle), which
has a fast path for the case where we're rotating about the z-axis. However, we can make this *slightly* faster
by omitting the hypotenuse computation, several floating point comparisons, and several assignments in this case
because we already know that we're just rotating about the z-axis.
This is a small (but measurable) improvement on the Multiply subtest of MotionMark, which applies a little over
3 million rotation transformations over the course of 30 seconds.
- platform/graphics/transforms/RotateTransformOperation.h:
Instead of calling
rotate3d
, just haverotate
directly callmultiply
with the following
TransformationMatrix
:
`
[[ cos(z) sin(z) 0 0 ]
[ -sin(z) cos(z) 0 0 ]
[ 0 0 1 0 ]
[ 0 0 0 1 ]]
`
...wherez
is the angle of rotation, in radians.
- platform/graphics/transforms/TransformationMatrix.cpp:
(WebCore::TransformationMatrix::rotate):
- platform/graphics/transforms/TransformationMatrix.h:
(WebCore::TransformationMatrix::rotate): Deleted.
- 8:12 AM Changeset in webkit [266352] by
-
- 1 edit4 adds in trunk/LayoutTests
Windows baselines after r266228 and r266273.
Add wincairo
Unreviewed test gardening.
- platform/wincairo/streams/reference-implementation/transform-stream-errors-expected.txt: Added.
- platform/wincairo/streams/reference-implementation/transform-stream-expected.txt: Added.
- 7:25 AM Changeset in webkit [266351] by
-
- 20 edits14 adds in trunk
[macOS] Date inputs should contain editable components
https://bugs.webkit.org/show_bug.cgi?id=215155
Reviewed by Devin Rousso.
Source/WebCore:
Currently, users are unable to edit the values in date and time input types.
To match system date controls, users should be able to select and edit
individual components of the input. For example, <input type="date"> should
contain an editable year, month, and day control.
Note that this functionality was present in WebKit, up until it was removed
in r150876. Previously known as "INPUT_MULTIPLE_FIELDS_UI", the feature was
removed as it was only used by Chromium port. However, much of the removed
logic remains useful and is resurfaced in this patch. Modifications were made
to update the code to follow current WebKit practices.
DateTimeEditElement is a wrapper element that contains individual fields.
The fields are all DateTimeFieldElements. Note that DateTimeFieldElements
are further specialized into DateTimeNumericFieldElement (for integral
values) and DateTimeSymbolicFieldElement (for symbolic values such as A.M
and P.M). DateTimeEditElement builds the control using the visitor pattern
and existing parsing logic contained in DateTimeFormat.
In order to reduce the patch size, this patch only sets up the rendering
and layout of multiple fields for <input type="date">. Upcoming patches will
add rendering for other date/time input types and introduce editability.
Test: fast/forms/date/date-pseudo-elements.html
- Sources.txt:
- WebCore.xcodeproj/project.pbxproj:
- css/html.css:
(input::-webkit-datetime-edit):
(input::-webkit-datetime-edit-fields-wrapper):
(input::-webkit-datetime-edit-year-field,):
(input::-webkit-datetime-edit-year-field:focus,):
(input::-webkit-datetime-edit-text):
- html/BaseChooserOnlyDateAndTimeInputType.cpp:
(WebCore::DateTimeFormatValidator::visitField):
(WebCore::DateTimeFormatValidator::validateFormat):
(WebCore::BaseChooserOnlyDateAndTimeInputType::createShadowSubtree):
(WebCore::BaseChooserOnlyDateAndTimeInputType::destroyShadowSubtree):
(WebCore::BaseChooserOnlyDateAndTimeInputType::updateInnerTextValue):
(WebCore::BaseChooserOnlyDateAndTimeInputType::valueForEditControl const):
(WebCore::BaseChooserOnlyDateAndTimeInputType::localeIdentifier const):
- html/BaseChooserOnlyDateAndTimeInputType.h:
- html/BaseDateAndTimeInputType.h:
- html/DateInputType.cpp:
(WebCore::DateInputType::isValidFormat const):
(WebCore::DateInputType::setupLayoutParameters const):
- html/DateInputType.h:
- html/DateTimeLocalInputType.cpp:
(WebCore::DateTimeLocalInputType::isValidFormat const):
(WebCore::DateTimeLocalInputType::setupLayoutParameters const):
- html/DateTimeLocalInputType.h:
- html/MonthInputType.cpp:
(WebCore::MonthInputType::isValidFormat const):
(WebCore::MonthInputType::setupLayoutParameters const):
- html/MonthInputType.h:
- html/TimeInputType.cpp:
(WebCore::TimeInputType::isValidFormat const):
(WebCore::TimeInputType::setupLayoutParameters const):
- html/TimeInputType.h:
- html/WeekInputType.cpp:
(WebCore::WeekInputType::isValidFormat const):
(WebCore::WeekInputType::setupLayoutParameters const):
- html/WeekInputType.h:
- html/shadow/DateTimeEditElement.cpp: Added.
(WebCore::DateTimeEditBuilder::DateTimeEditBuilder):
(WebCore::DateTimeEditBuilder::build):
(WebCore::DateTimeEditBuilder::visitField):
(WebCore::DateTimeEditBuilder::visitLiteral):
(WebCore::DateTimeEditElement::DateTimeEditElement):
(WebCore::DateTimeEditElement::fieldsWrapperElement const):
(WebCore::DateTimeEditElement::addField):
(WebCore::DateTimeEditElement::create):
(WebCore::DateTimeEditElement::layout):
(WebCore::DateTimeEditElement::localeIdentifier const):
(WebCore::DateTimeEditElement::resetFields):
(WebCore::DateTimeEditElement::setValueAsDate):
(WebCore::DateTimeEditElement::setEmptyValue):
(WebCore::DateTimeEditElement::value const):
- html/shadow/DateTimeEditElement.h: Added.
- html/shadow/DateTimeFieldElement.cpp: Added.
(WebCore::DateTimeFieldElement::DateTimeFieldElement):
(WebCore::DateTimeFieldElement::initialize):
(WebCore::DateTimeFieldElement::localeForOwner const):
(WebCore::DateTimeFieldElement::localeIdentifier const):
(WebCore::DateTimeFieldElement::updateVisibleValue):
(WebCore::DateTimeFieldElement::supportsFocus const):
- html/shadow/DateTimeFieldElement.h: Added.
- html/shadow/DateTimeFieldElements.cpp: Added.
(WebCore::DateTimeDayFieldElement::DateTimeDayFieldElement):
(WebCore::DateTimeDayFieldElement::create):
(WebCore::DateTimeDayFieldElement::setValueAsDate):
(WebCore::DateTimeMonthFieldElement::DateTimeMonthFieldElement):
(WebCore::DateTimeMonthFieldElement::create):
(WebCore::DateTimeMonthFieldElement::setValueAsDate):
(WebCore::DateTimeSymbolicMonthFieldElement::DateTimeSymbolicMonthFieldElement):
(WebCore::DateTimeSymbolicMonthFieldElement::create):
(WebCore::DateTimeSymbolicMonthFieldElement::setValueAsDate):
(WebCore::DateTimeYearFieldElement::DateTimeYearFieldElement):
(WebCore::DateTimeYearFieldElement::create):
(WebCore::DateTimeYearFieldElement::setValueAsDate):
- html/shadow/DateTimeFieldElements.h: Added.
- html/shadow/DateTimeNumericFieldElement.cpp: Added.
(WebCore::DateTimeNumericFieldElement::DateTimeNumericFieldElement):
(WebCore::DateTimeNumericFieldElement::formatValue const):
(WebCore::DateTimeNumericFieldElement::hasValue const):
(WebCore::DateTimeNumericFieldElement::initialize):
(WebCore::DateTimeNumericFieldElement::setEmptyValue):
(WebCore::DateTimeNumericFieldElement::setValueAsInteger):
(WebCore::DateTimeNumericFieldElement::value const):
(WebCore::DateTimeNumericFieldElement::valueAsInteger const):
(WebCore::DateTimeNumericFieldElement::visibleValue const):
- html/shadow/DateTimeNumericFieldElement.h: Added.
- html/shadow/DateTimeSymbolicFieldElement.cpp: Added.
(WebCore::makeVisibleEmptyValue):
(WebCore::DateTimeSymbolicFieldElement::DateTimeSymbolicFieldElement):
(WebCore::DateTimeSymbolicFieldElement::hasValue const):
(WebCore::DateTimeSymbolicFieldElement::initialize):
(WebCore::DateTimeSymbolicFieldElement::setEmptyValue):
(WebCore::DateTimeSymbolicFieldElement::setValueAsInteger):
(WebCore::DateTimeSymbolicFieldElement::value const):
(WebCore::DateTimeSymbolicFieldElement::valueAsInteger const):
(WebCore::DateTimeSymbolicFieldElement::visibleEmptyValue const):
(WebCore::DateTimeSymbolicFieldElement::visibleValue const):
- html/shadow/DateTimeSymbolicFieldElement.h: Added.
(WebCore::DateTimeSymbolicFieldElement::symbolsSize const):
LayoutTests:
Added a test to verify styling of pseudo elements within date inputs.
Rebaselined existing tests to match new appearance.
- fast/forms/date/date-pseudo-elements-expected.txt: Added.
- fast/forms/date/date-pseudo-elements.html: Added.
- platform/mac-wk2/fast/forms/date/date-input-rendering-basic-expected.txt:
- platform/mac-wk2/fast/forms/date/date-pseudo-elements-expected.txt: Added.
- platform/mac-wk2/fast/forms/time/time-appearance-basic-expected.txt: Added.
- platform/mac-wk2/fast/forms/time/time-input-rendering-basic-expected.txt:
- 4:51 AM Changeset in webkit [266350] by
-
- 56 edits in trunk
Convert runtime flag to setting for lazy image loading
https://bugs.webkit.org/show_bug.cgi?id=215610
Patch by Rob Buis <rbuis@igalia.com> on 2020-08-31
Reviewed by Youenn Fablet.
LayoutTests/imported/w3c:
Remove changes done to the lazy image loading tests as we enable the setting in testharnessreport.js.
The new expected results are improvements.
- web-platform-tests/html/semantics/embedded-content/the-img-element/below-viewport-image-loading-lazy-load-event.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/disconnected-image-loading-lazy.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-eager.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-base-url-2-expected.txt:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-base-url-expected.txt:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-below-viewport-dynamic.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-crossorigin-change.sub.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-in-cross-origin-iframe-001.sub-expected.txt:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-in-cross-origin-iframe-002.sub-expected.txt:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-in-script-disabled-iframe.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-in-viewport-dynamic.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-load-event.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-move-document.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-move-into-script-disabled-iframe.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-multicol.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-multiple-times-expected.txt:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-referrerpolicy-change.sub.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-relevant-mutations-expected.txt:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-srcset-expected.txt:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy-to-eager-expected.txt:
- web-platform-tests/html/semantics/embedded-content/the-img-element/image-loading-lazy.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/invisible-image.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/move-element-and-scroll.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/not-rendered-below-viewport-image-loading-lazy.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/not-rendered-image-loading-lazy.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/picture-loading-lazy.html:
- web-platform-tests/html/semantics/embedded-content/the-img-element/remove-element-and-scroll.html:
Source/WebCore:
Convert runtime flag to setting for lazy image loading.
- html/HTMLImageElement.idl:
- html/parser/HTMLPreloadScanner.cpp:
(WebCore::TokenPreloadScanner::StartTagScanner::processAttribute):
- loader/ImageLoader.cpp:
(WebCore::ImageLoader::updateFromElement):
- page/RuntimeEnabledFeatures.h:
(WebCore::RuntimeEnabledFeatures::setLazyImageLoadingEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::lazyImageLoadingEnabled const): Deleted.
- page/Settings.yaml:
Source/WebKit:
Remove lazy image loading runtime flag.
- Shared/WebPreferences.yaml:
- UIProcess/API/C/WKPreferences.cpp:
(WKPreferencesSetLazyImageLoadingEnabled): Deleted.
(WKPreferencesGetLazyImageLoadingEnabled): Deleted.
- UIProcess/API/C/WKPreferencesRefPrivate.h:
Tools:
Remove lazy image loading runtime flag.
- WebKitTestRunner/TestController.cpp:
(WTR::TestController::resetPreferencesToConsistentValues):
(WTR::updateTestOptionsFromTestHeader):
- WebKitTestRunner/TestOptions.h:
(WTR::TestOptions::hasSameInitializationOptions const):
LayoutTests:
Enable lazy image loading setting in resources/testharnessreport.js. Update test expectations
and adjust the tests in http/tests/lazyload/ to enable the setting. Fow WK1 list more tests
that should be skipped.
- TestExpectations:
- http/tests/lazyload/attribute.html:
- http/tests/lazyload/js-image.html:
- http/tests/lazyload/lazy-image-load-in-iframes-scripting-disabled.html:
- http/tests/lazyload/lazy-image-load-in-iframes-scripting-enabled.html:
- http/tests/lazyload/lazy.html:
- platform/gtk/imported/w3c/web-platform-tests/html/dom/idlharness.https-expected.txt:
- platform/ios-wk2/imported/w3c/web-platform-tests/html/dom/idlharness.https-expected.txt:
- platform/mac-wk1/TestExpectations:
- platform/mac-wk1/imported/w3c/web-platform-tests/html/dom/idlharness.https-expected.txt:
- platform/mac-wk2/imported/w3c/web-platform-tests/html/dom/idlharness.https-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/html/dom/idlharness.https-expected.txt:
- resources/testharnessreport.js:
- 2:45 AM Changeset in webkit [266349] by
-
- 2 edits in trunk/Tools
[Flatpak] SDK branch upgrade path is broken
https://bugs.webkit.org/show_bug.cgi?id=215927
Patch by Philippe Normand <pnormand@igalia.com> on 2020-08-31
Reviewed by Adrian Perez de Castro.
The development environment (flatpak install) needs to be performed before attempting to
rebuild toolchain archives. In most cases this wasn't an issue, but when the SDK needs to be
upgraded to a new ABI/branch, the previous version needs to be removed, that triggers a new
flatpak install
and toolchain archives update, which need then to be done in the right
order.
- flatpak/flatpakutils.py:
(WebkitFlatpak.main):
- 2:01 AM Changeset in webkit [266348] by
-
- 35 edits5 adds in trunk
Add support for TextDecoderStream
https://bugs.webkit.org/show_bug.cgi?id=215919
Reviewed by Alex Christensen.
LayoutTests/imported/w3c:
- web-platform-tests/encoding/streams/backpressure.any-expected.txt:
- web-platform-tests/encoding/streams/backpressure.any.worker-expected.txt:
- web-platform-tests/encoding/streams/decode-attributes.any-expected.txt:
- web-platform-tests/encoding/streams/decode-attributes.any.worker-expected.txt:
- web-platform-tests/encoding/streams/decode-bad-chunks.any-expected.txt:
- web-platform-tests/encoding/streams/decode-bad-chunks.any.worker-expected.txt:
- web-platform-tests/encoding/streams/decode-ignore-bom.any-expected.txt:
- web-platform-tests/encoding/streams/decode-ignore-bom.any.worker-expected.txt:
- web-platform-tests/encoding/streams/decode-incomplete-input.any-expected.txt:
- web-platform-tests/encoding/streams/decode-incomplete-input.any.worker-expected.txt:
- web-platform-tests/encoding/streams/decode-non-utf8.any-expected.txt:
- web-platform-tests/encoding/streams/decode-non-utf8.any.worker-expected.txt:
- web-platform-tests/encoding/streams/decode-split-character.any-expected.txt:
- web-platform-tests/encoding/streams/decode-split-character.any.worker-expected.txt:
- web-platform-tests/encoding/streams/decode-utf8.any-expected.txt:
- web-platform-tests/encoding/streams/decode-utf8.any.worker-expected.txt:
- web-platform-tests/encoding/streams/encode-bad-chunks.any-expected.txt:
- web-platform-tests/encoding/streams/encode-bad-chunks.any.worker-expected.txt:
- web-platform-tests/encoding/streams/readable-writable-properties.any-expected.txt:
- web-platform-tests/encoding/streams/readable-writable-properties.any.worker-expected.txt:
- web-platform-tests/encoding/streams/realms.window-expected.txt:
Source/WebCore:
Implement TextEncoderStream as a JS built-in.
It is enabled by transform stream runtime flag.
TextDecoderStream takes a TransformStream and TextDecoderStreamEncoder as private slots.
Add TextDecoderStreamEncoder to implement the actual UTF-8 encoding steps, as a wrapper around TextDecoder.
Covered by rebased tests.
- CMakeLists.txt:
- DerivedSources-input.xcfilelist:
- DerivedSources-output.xcfilelist:
- DerivedSources.make:
- Sources.txt:
- WebCore.xcodeproj/project.pbxproj:
- bindings/js/WebCoreBuiltinNames.h:
- dom/TextDecoderStream.idl: Added.
- dom/TextDecoderStream.js: Added.
(initializeTextDecoderStream):
(getter.encoding):
(getter.fatal):
(getter.ignoreBOM):
(getter.readable):
(getter.writable):
- dom/TextDecoderStreamDecoder.cpp: Added.
(WebCore::TextDecoderStreamDecoder::create):
(WebCore::TextDecoderStreamDecoder::TextDecoderStreamDecoder):
(WebCore::TextDecoderStreamDecoder::decode):
(WebCore::TextDecoderStreamDecoder::flush):
- dom/TextDecoderStreamDecoder.h: Added.
(WebCore::TextDecoderStreamDecoder::encoding const):
- dom/TextDecoderStreamDecoder.idl: Added.
LayoutTests:
Unskipping encoding tests except legacy-* tests which are big and slow.
- 1:58 AM Changeset in webkit [266347] by
-
- 2 edits in trunk/Source/WebKit
[Cocoa] Disable sniffing for WebSocket handshakes
https://bugs.webkit.org/show_bug.cgi?id=215932
Reviewed by Darin Adler.
Covered by http/tests/websocket/tests/hybi/handshake-error.html no longer timing out with NSURLSession WebSocket code path.
- NetworkProcess/cocoa/NetworkSessionCocoa.mm:
(WebKit::NetworkSessionCocoa::createWebSocketTask):
- 12:23 AM Changeset in webkit [266346] by
-
- 5 edits in trunk
[JSC] @defaultPromiseThen fast path should check species constructor
https://bugs.webkit.org/show_bug.cgi?id=215996
Reviewed by Ross Kirsling.
JSTests:
- test262/expectations.yaml:
Source/JavaScriptCore:
When executing @defaultPromiseThen fast path, we assumed that this execution is not observable.
This is wrong only for species constructor part: this @@species access & derived constructor calls
can be observable. In this patch,
- We extract part of Promise#then as @performPromiseThen, which corresponds to the spec's PerformPromiseThen.
- In promise fast path, we check @speciesConstructor is @Promise or @InternalPromise. If it is not, then we go to the slow path.
This fixes Promise#finally failures in test262.
- builtins/PromiseOperations.js:
(globalPrivate.promiseResolveThenableJobFast):
(globalPrivate.promiseResolveThenableJobWithoutPromiseFast):
(globalPrivate.promiseResolveThenableJobWithDerivedPromise):
(onFulfilled):
(onRejected):
(globalPrivate.performPromiseThen):
- builtins/PromisePrototype.js:
(then):
(onFulfilled): Deleted.
(onRejected): Deleted.
Aug 30, 2020:
- 10:18 PM Changeset in webkit [266345] by
-
- 2 edits in trunk/Source/WebKitLegacy/mac
Remove an unused addition point
- WebCoreSupport/WebFrameLoaderClient.mm:
- 9:51 PM Changeset in webkit [266344] by
-
- 9 edits in trunk/Source/WebCore
Make StyleRareNonInheritedData::mask and StyleBackgroundData::background DataRefs
https://bugs.webkit.org/show_bug.cgi?id=215942
Reviewed by Darin Adler.
A significant amount of time in MotionMark's Multiply subtest is spent underneath the copy constructor
StyleRareNonInheritedData
, and a significant amount of time underneath this copy constructor is spent copying
the rare non-inherited data's mask (aFillLayer
).
This
FillLayer
is currently inline data inStyleRareNonInheritedData
; to reduce the cost of copying rare
non-inherited data, we can instead make this aDataRef<FillLayer>
, such that copying rare data will only copy
the reference to theFillLayer
rather than theFillLayer
itself.
Upon mutating the
FillLayer
, we now useDataRef::access()
to ensure that theseFillLayer
s are copied
before writing. See below for more details.
- css/makeprop.pl:
(generateFillLayerPropertyInheritValueSetter):
(generateFillLayerPropertyValueSetter):
- rendering/style/FillLayer.cpp:
(WebCore::FillLayer::create):
(WebCore::FillLayer::FillLayer):
Deeply copy the
FillLayer
's linked list ofFillLayer
s. This preserves existing behavior, which currently
usesstd::unique_ptr
to store the pointer to the nextFillLayer
(and therefore, requires a deep copy of the
linked list when copyingFillLayer
).
In a future patch, we could probably further optimize this as well to behave in a copy-on-write way, by turning
theRefPtr<FillLayer> m_next;
into anOptional<DataRef<FillLayer>>
instead (and then useaccess()
to copy
the nextFillLayer
prior to writing).
(WebCore::FillLayer::~FillLayer):
(WebCore::FillLayer::operator=):
- rendering/style/FillLayer.h:
Make
FillLayer
ref-counted, and introduce newcreate
methods for constructingFillLayer
s. Use these
methods in several places where we currently call the constructors directly, viamakeUnique
.
(WebCore::FillLayer::copy const):
(WebCore::FillLayer::setNext):
- rendering/style/RenderStyle.h:
(WebCore::RenderStyle::hasBackgroundImage const):
(WebCore::RenderStyle::hasFixedBackgroundImage const):
(WebCore::RenderStyle::backgroundRepeatX const):
(WebCore::RenderStyle::backgroundRepeatY const):
(WebCore::RenderStyle::backgroundComposite const):
(WebCore::RenderStyle::backgroundAttachment const):
(WebCore::RenderStyle::backgroundClip const):
(WebCore::RenderStyle::backgroundOrigin const):
(WebCore::RenderStyle::backgroundXPosition const):
(WebCore::RenderStyle::backgroundYPosition const):
(WebCore::RenderStyle::backgroundSizeType const):
(WebCore::RenderStyle::backgroundSizeLength const):
(WebCore::RenderStyle::ensureBackgroundLayers):
(WebCore::RenderStyle::maskImage const):
(WebCore::RenderStyle::maskRepeatX const):
(WebCore::RenderStyle::maskRepeatY const):
(WebCore::RenderStyle::maskComposite const):
(WebCore::RenderStyle::maskClip const):
(WebCore::RenderStyle::maskOrigin const):
(WebCore::RenderStyle::maskXPosition const):
(WebCore::RenderStyle::maskYPosition const):
(WebCore::RenderStyle::maskSizeType const):
(WebCore::RenderStyle::maskSizeLength const):
(WebCore::RenderStyle::ensureMaskLayers):
(WebCore::RenderStyle::hasMask const):
(WebCore::RenderStyle::setBackgroundXPosition):
(WebCore::RenderStyle::setBackgroundYPosition):
(WebCore::RenderStyle::setBackgroundSize):
(WebCore::RenderStyle::setBackgroundSizeLength):
(WebCore::RenderStyle::clearBackgroundLayers):
(WebCore::RenderStyle::inheritBackgroundLayers):
(WebCore::RenderStyle::clearMaskLayers):
(WebCore::RenderStyle::inheritMaskLayers):
(WebCore::RenderStyle::setMaskImage):
(WebCore::RenderStyle::setMaskXPosition):
(WebCore::RenderStyle::setMaskYPosition):
(WebCore::RenderStyle::setMaskSize):
- rendering/style/StyleBackgroundData.cpp:
(WebCore::StyleBackgroundData::StyleBackgroundData):
(WebCore::StyleBackgroundData::dump const):
- rendering/style/StyleBackgroundData.h:
Since
FillLayer
is now ref-counted,StyleBackgroundData::background
needs to be aDataRef
as well.
- rendering/style/StyleRareNonInheritedData.cpp:
(WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData):
- rendering/style/StyleRareNonInheritedData.h:
- 9:37 PM Changeset in webkit [266343] by
-
- 2 edits in trunk/WebDriverTests
[WPE] Gardening some webdriver failures
Unreviewed test gardening.
- TestExpectations.json:
- 8:26 PM Changeset in webkit [266342] by
-
- 85 edits17 deletes in trunk
Remove unused editable image code
https://bugs.webkit.org/show_bug.cgi?id=215991
Reviewed by Simon Fraser.
Source/WebCore:
- Sources.txt:
- WebCore.xcodeproj/project.pbxproj:
- editing/EditAction.cpp:
(WebCore::undoRedoLabel):
- editing/EditAction.h:
- editing/Editor.cpp:
(WebCore::Editor::insertEditableImage): Deleted.
- editing/Editor.h:
- editing/EditorCommand.cpp:
(WebCore::createCommandMap):
(WebCore::executeInsertEditableImage): Deleted.
(WebCore::enabledInRichlyEditableTextWithEditableImagesEnabled): Deleted.
- editing/InsertEditableImageCommand.cpp: Removed.
- editing/InsertEditableImageCommand.h: Removed.
- en.lproj/Localizable.strings:
- html/HTMLAttributeNames.in:
- html/HTMLImageElement.cpp:
(WebCore::HTMLImageElement::parseAttribute):
(WebCore::HTMLImageElement::insertedIntoAncestor):
(WebCore::HTMLImageElement::removedFromAncestor):
(WebCore::HTMLImageElement::copyNonAttributePropertiesFromElement):
(WebCore::HTMLImageElement::supportsFocus const): Deleted.
(WebCore::HTMLImageElement::isFocusable const): Deleted.
(WebCore::HTMLImageElement::didFinishInsertingNode): Deleted.
(WebCore::HTMLImageElement::hasEditableImageAttribute const): Deleted.
(WebCore::HTMLImageElement::editableImageViewID const): Deleted.
(WebCore::HTMLImageElement::updateEditableImage): Deleted.
(WebCore::HTMLImageElement::defaultEventHandler): Deleted.
- html/HTMLImageElement.h:
- page/ChromeClient.h:
(WebCore::ChromeClient::associateEditableImageWithAttachment): Deleted.
(WebCore::ChromeClient::didCreateEditableImage): Deleted.
(WebCore::ChromeClient::didDestroyEditableImage): Deleted.
- page/EditableImageReference.cpp: Removed.
- page/EditableImageReference.h: Removed.
- page/Settings.yaml:
- platform/graphics/GraphicsLayer.cpp:
(WebCore::GraphicsLayer::nextEmbeddedViewID): Deleted.
- platform/graphics/GraphicsLayer.h:
(WebCore::GraphicsLayer::setContentsToSolidColor):
(WebCore::GraphicsLayer::setContentsToEmbeddedView): Deleted.
- platform/graphics/ca/GraphicsLayerCA.cpp:
(WebCore::GraphicsLayerCA::createPlatformCALayerForEmbeddedView): Deleted.
(WebCore::GraphicsLayerCA::setContentsToEmbeddedView): Deleted.
- platform/graphics/ca/GraphicsLayerCA.h:
- platform/graphics/ca/PlatformCALayer.cpp:
(WebCore::operator<<):
- platform/graphics/ca/PlatformCALayer.h:
- platform/graphics/ca/cocoa/PlatformCALayerCocoa.h:
- platform/graphics/ca/cocoa/PlatformCALayerCocoa.mm:
(WebCore::PlatformCALayerCocoa::PlatformCALayerCocoa):
(WebCore::PlatformCALayerCocoa::embeddedViewID const): Deleted.
- rendering/RenderImage.cpp:
(WebCore::RenderImage::isEditableImage const): Deleted.
(WebCore::RenderImage::requiresLayer const): Deleted.
- rendering/RenderImage.h:
- rendering/RenderLayer.cpp:
(WebCore::RenderLayer::shouldBeNormalFlowOnly const):
(WebCore::RenderLayer::calculateClipRects const):
- rendering/RenderLayerBacking.cpp:
(WebCore::RenderLayerBacking::updateConfiguration):
- rendering/RenderLayerCompositor.cpp:
(WebCore::RenderLayerCompositor::requiresCompositingLayer const):
(WebCore::RenderLayerCompositor::requiresOwnBackingStore const):
(WebCore::RenderLayerCompositor::reasonsForCompositing const):
(WebCore::RenderLayerCompositor::requiresCompositingForEditableImage const): Deleted.
- rendering/RenderLayerCompositor.h:
Source/WebKit:
- DerivedSources.make:
- Platform/spi/ios/PencilKitSPI.h: Removed.
- Shared/FocusedElementInformation.cpp:
(WebKit::FocusedElementInformation::encode const):
(WebKit::FocusedElementInformation::decode):
- Shared/FocusedElementInformation.h:
- Shared/RemoteLayerTree/RemoteLayerBackingStore.mm:
(WebKit::RemoteLayerBackingStore::drawInContext):
- Shared/RemoteLayerTree/RemoteLayerTreeTransaction.h:
- Shared/RemoteLayerTree/RemoteLayerTreeTransaction.mm:
(WebKit::RemoteLayerTreeTransaction::LayerCreationProperties::LayerCreationProperties):
(WebKit::RemoteLayerTreeTransaction::LayerCreationProperties::encode const):
(WebKit::RemoteLayerTreeTransaction::LayerCreationProperties::decode):
- Shared/WebPreferences.yaml:
- SourcesCocoa.txt:
- UIProcess/API/Cocoa/WKWebView.mm:
(-[WKWebView _setupPageConfiguration:]):
- UIProcess/API/Cocoa/WKWebViewConfiguration.mm:
(-[WKWebViewConfiguration init]):
(-[WKWebViewConfiguration copyWithZone:]):
(-[WKWebViewConfiguration _setEditableImagesEnabled:]): Deleted.
(-[WKWebViewConfiguration _editableImagesEnabled]): Deleted.
- UIProcess/API/Cocoa/WKWebViewConfigurationPrivate.h:
- UIProcess/API/ios/WKWebViewIOS.h:
- UIProcess/API/ios/WKWebViewIOS.mm:
(-[WKWebView _setupScrollAndContentViews]):
(-[WKWebView _stylusTapGestureShouldCreateEditableImage]): Deleted.
- UIProcess/PageClient.h:
(WebKit::PageClient::createDrawingView): Deleted.
- UIProcess/RemoteLayerTree/RemoteLayerTreeHost.h:
- UIProcess/RemoteLayerTree/RemoteLayerTreeHost.mm:
(WebKit::RemoteLayerTreeHost::layerWillBeRemoved):
(WebKit::RemoteLayerTreeHost::clearLayers):
(WebKit::RemoteLayerTreeHost::makeNode):
- UIProcess/RemoteLayerTree/ios/RemoteLayerTreeHostIOS.mm:
(WebKit::RemoteLayerTreeHost::makeNode):
(WebKit::RemoteLayerTreeHost::createEmbeddedView): Deleted.
- UIProcess/RemoteLayerTree/ios/RemoteLayerTreeViews.h:
- UIProcess/RemoteLayerTree/ios/RemoteLayerTreeViews.mm:
(-[WKEmbeddedView initWithEmbeddedViewID:]): Deleted.
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::didAttachToRunningProcess):
(WebKit::WebPageProxy::resetState):
(WebKit::WebPageProxy::willUpdateAttachmentAttributes):
- UIProcess/WebPageProxy.h:
(WebKit::WebPageProxy::editableImageController): Deleted.
- UIProcess/ios/EditableImageController.h: Removed.
- UIProcess/ios/EditableImageController.messages.in: Removed.
- UIProcess/ios/EditableImageController.mm: Removed.
- UIProcess/ios/PageClientImplIOS.h:
- UIProcess/ios/PageClientImplIOS.mm:
(WebKit::PageClientImpl::createDrawingView): Deleted.
- UIProcess/ios/PencilKitSoftLink.h: Removed.
- UIProcess/ios/PencilKitSoftLink.mm: Removed.
- UIProcess/ios/WKContentViewInteraction.h:
- UIProcess/ios/WKContentViewInteraction.mm:
(-[WKContentView setUpInteraction]):
(-[WKContentView cleanUpInteraction]):
(-[WKContentView _removeDefaultGestureRecognizers]):
(-[WKContentView _addDefaultGestureRecognizers]):
(-[WKContentView gestureRecognizerShouldBegin:]):
(-[WKContentView _elementDidFocus:userIsInteracting:blurPreviousNode:activityStateChanges:userObject:]):
(-[WKContentView _elementDidBlur]):
(-[WKContentView _stylusSingleTapRecognized:]): Deleted.
(-[WKContentView _drawingCoordinator]): Deleted.
- UIProcess/ios/WKDrawingCoordinator.h: Removed.
- UIProcess/ios/WKDrawingCoordinator.mm: Removed.
- UIProcess/ios/WKDrawingView.h: Removed.
- UIProcess/ios/WKDrawingView.mm: Removed.
- UIProcess/ios/WebPageProxyIOS.mm:
(WebKit::WebPageProxy::handleStylusSingleTapAtPoint): Deleted.
- UIProcess/ios/fullscreen/WKFullScreenWindowControllerIOS.mm:
- WebKit.xcodeproj/project.pbxproj:
- WebProcess/WebCoreSupport/WebChromeClient.h:
- WebProcess/WebCoreSupport/ios/WebChromeClientIOS.mm:
(WebKit::WebChromeClient::associateEditableImageWithAttachment): Deleted.
(WebKit::WebChromeClient::didCreateEditableImage): Deleted.
(WebKit::WebChromeClient::didDestroyEditableImage): Deleted.
- WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.cpp:
(WebKit::GraphicsLayerCARemote::createPlatformCALayerForEmbeddedView): Deleted.
- WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.h:
- WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.cpp:
(WebKit::PlatformCALayerRemote::createForEmbeddedView): Deleted.
(WebKit::PlatformCALayerRemote::embeddedViewID const): Deleted.
- WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.h:
- WebProcess/WebPage/RemoteLayerTree/RemoteLayerTreeContext.mm:
(WebKit::RemoteLayerTreeContext::layerDidEnterContext):
- WebProcess/WebPage/WebPage.h:
- WebProcess/WebPage/WebPage.messages.in:
- WebProcess/WebPage/ios/WebPageIOS.mm:
(WebKit::isAssistableElement):
(WebKit::WebPage::getFocusedElementInformation):
(WebKit::WebPage::handleStylusSingleTapAtPoint): Deleted.
Tools:
- TestRunnerShared/UIScriptContext/Bindings/UIScriptController.idl:
- TestRunnerShared/UIScriptContext/UIScriptController.h:
(WTR::UIScriptController::drawSquareInEditableImage): Deleted.
(WTR::UIScriptController::numberOfStrokesInEditableImage): Deleted.
- TestRunnerShared/spi/PencilKitTestSPI.h: Removed.
- TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
- TestWebKitAPI/Tests/WebKitCocoa/WKAttachmentTests.mm:
(webViewForTestingAttachments):
(TestWebKitAPI::forEachViewInHierarchy): Deleted.
(TestWebKitAPI::findEditableImageCanvas): Deleted.
(TestWebKitAPI::drawSquareInEditableImage): Deleted.
- TestWebKitAPI/ios/PencilKitTestSPI.h: Removed.
- WebKitTestRunner/TestController.cpp:
(WTR::updateTestOptionsFromTestHeader):
- WebKitTestRunner/TestOptions.h:
(WTR::TestOptions::hasSameInitializationOptions const):
- WebKitTestRunner/WebKitTestRunner.xcodeproj/project.pbxproj:
- WebKitTestRunner/cocoa/TestControllerCocoa.mm:
(WTR::TestController::platformCreateWebView):
- WebKitTestRunner/ios/UIScriptControllerIOS.h:
- WebKitTestRunner/ios/UIScriptControllerIOS.mm:
(WTR::UIScriptControllerIOS::findEditableImageCanvas const): Deleted.
(WTR::UIScriptControllerIOS::drawSquareInEditableImage): Deleted.
(WTR::UIScriptControllerIOS::numberOfStrokesInEditableImage): Deleted.
LayoutTests:
- editing/images/basic-editable-image-expected.txt: Removed.
- editing/images/basic-editable-image-from-execCommand-expected.txt: Removed.
- editing/images/basic-editable-image-from-execCommand.html: Removed.
- editing/images/basic-editable-image-with-gesture-expected.txt: Removed.
- editing/images/basic-editable-image-with-gesture.html: Removed.
- editing/images/basic-editable-image.html: Removed.
- editing/images/editable-image-creates-attachment-expected.txt: Removed.
- editing/images/editable-image-creates-attachment.html: Removed.
- editing/images/paste-editable-image-expected.txt: Removed.
- editing/images/paste-editable-image.html: Removed.
- editing/images/redo-insert-editable-image-maintains-strokes-expected.txt: Removed.
- editing/images/redo-insert-editable-image-maintains-strokes.html: Removed.
- editing/images/reparent-editable-image-maintains-strokes-expected.txt: Removed.
- editing/images/reparent-editable-image-maintains-strokes.html: Removed.
- editing/images/undo-insert-editable-image-expected.txt: Removed.
- editing/images/undo-insert-editable-image.html: Removed.
- resources/ui-helper.js:
(window.UIHelper.drawSquareInEditableImage): Deleted.
(window.UIHelper.numberOfStrokesInEditableImage): Deleted.
- 6:29 PM Changeset in webkit [266341] by
-
- 4 edits in trunk
[JSC] Use -2 for grouping options in IntlRelativeTimeFormat
https://bugs.webkit.org/show_bug.cgi?id=215984
Reviewed by Ross Kirsling.
JSTests:
- stress/intl-relativetimeformat.js:
Source/JavaScriptCore:
Several test262 tests are failing after ICU 67. This is because Intl.RelativeTimeFormat is not using locale-sensitive grouping option.
There are hidden option -2 for UNumberFormat. It is supported so long, but it is not explicitly documented. After ICU 68, it is exposed as a constant,
we should pass -2 to UNumberFormat's grouping options to use locale-sensitive grouping option here.
- runtime/IntlRelativeTimeFormat.cpp:
(JSC::IntlRelativeTimeFormat::initializeRelativeTimeFormat):
- 6:26 PM Changeset in webkit [266340] by
-
- 5 edits1 add in trunk
[JSC] async function cannot appear in single-statement context
https://bugs.webkit.org/show_bug.cgi?id=215993
Reviewed by Darin Adler.
JSTests:
- stress/async-function-lookahead.js: Added.
(testSyntax):
(testSyntaxError):
(testSyntax.false.async t):
(testSyntaxError.false.async t):
- test262/expectations.yaml:
Source/JavaScriptCore:
The following code is syntax error[1] because ExpressionStatement has
async [no LineTerminator here] function
lookahead.
if (false)
async function t() { }
[1]: https://tc39.es/ecma262/#sec-expression-statement
- parser/Parser.cpp:
(JSC::Parser<LexerType>::parseStatement):
(JSC::Parser<LexerType>::maybeParseAsyncFunctionDeclarationStatement): Deleted.
- parser/Parser.h:
- 6:20 PM Changeset in webkit [266339] by
-
- 6 edits1 copy2 adds in trunk/Source/WebCore
EnabledBySetting extended attributes on a partial interface do not get merged with EnabledBySetting extended attributes on properties of that partial interface
https://bugs.webkit.org/show_bug.cgi?id=215992
Reviewed by Darin Adler.
Add support for merging extended attributes that support conjunctions (like EnabledBySetting) when
they are used in both a partial wide context and on a member. This allows a case like:
[
EnabledBySettings=FirstSetting
] partial interface Foo {
[EnabledBySettings=SecondSetting] attribute DOMString bar;
};
to be identical to:
partial interface Foo {
[EnabledBySettings=FirstSetting&SecondSetting] attribute DOMString bar;
};
- bindings/scripts/CodeGenerator.pm:
(MergeExtendedAttributesFromSupplemental):
Factor out and rework merging logic. Only allow adding extended attributes that are allowed for the context
they are being merge into to be merged, and only allow extended attributes declared in IDLAttributes.json as
supporting conjunctions to be merged.
(ProcessSupplementalDependencies):
Call MergeExtendedAttributesFromSupplemental for each type of member.
(ExtendedAttributeContains):
Fix variable name, since this is used for all extended attributes, not just callWith.
- bindings/scripts/IDLAttributes.json:
Add supportsConjunction keys for extended attributes that support the conjunction (&) syntax and semantics.
- bindings/scripts/test/JS/JSTestEnabledBySetting.cpp:
(WebCore::JSTestEnabledBySettingPrototype::finishCreation):
(WebCore::jsTestEnabledBySettingEnabledByTwoSettingsAttributeGetter):
(WebCore::jsTestEnabledBySettingEnabledByTwoSettingsAttribute):
(WebCore::setJSTestEnabledBySettingEnabledByTwoSettingsAttributeSetter):
(WebCore::setJSTestEnabledBySettingEnabledByTwoSettingsAttribute):
(WebCore::jsTestEnabledBySettingSupplementalAttributeGetter):
(WebCore::jsTestEnabledBySettingSupplementalAttribute):
(WebCore::setJSTestEnabledBySettingSupplementalAttributeSetter):
(WebCore::setJSTestEnabledBySettingSupplementalAttribute):
- bindings/scripts/test/JS/JSTestEnabledBySettingSupplemental.cpp: Added.
- bindings/scripts/test/JS/JSTestEnabledBySettingSupplemental.h: Added.
- bindings/scripts/test/SupplementalDependencies.dep:
- bindings/scripts/test/TestEnabledBySetting.idl:
- bindings/scripts/test/TestEnabledBySettingSupplemental.idl: Added.
Add/update tests.
- 4:16 PM Changeset in webkit [266338] by
-
- 2 edits in trunk/Source/WebCore
REGRESSION (r266333): Mousewheel scrolling no longer works on WinCairo
https://bugs.webkit.org/show_bug.cgi?id=215990
Reviewed by Tim Horton.
Make sure we call processWheelEventForScrolling() when ENABLE(WHEEL_EVENT_LATCHING) is not
defined.
- page/EventHandler.cpp:
(WebCore::EventHandler::handleWheelEvent):
- 3:25 PM Changeset in webkit [266337] by
-
- 3 edits2 adds in trunk
Rubberbanding in an overflow:scroll can trigger page jiggles
https://bugs.webkit.org/show_bug.cgi?id=215629
<rdar://problem/67295797>
Reviewed by Tim Horton.
Source/WebCore:
When a scroller starts to rubberband, and is receiving momentum scroll events, it gets
into a state where events are ignored (to stop endless stretching). When in that
state, ScrollController::handleWheelEvent() needs to report that it handled the event,
to avoid the event bouncing to the scrolling thread and triggering page scrolls.
Test: fast/scrolling/mac/rubberband-overflow-in-wheel-region-root-jiggle.html
- platform/cocoa/ScrollController.mm:
(WebCore::ScrollController::handleWheelEvent):
LayoutTests:
- fast/scrolling/mac/rubberband-overflow-in-wheel-region-root-jiggle-expected.txt: Added.
- fast/scrolling/mac/rubberband-overflow-in-wheel-region-root-jiggle.html: Added.
- 3:09 PM Changeset in webkit [266336] by
-
- 3 edits2 adds in trunk
REGRESSION (r266262): Scroll latching fails when scrolling into a preventDefault() region
https://bugs.webkit.org/show_bug.cgi?id=215988
<rdar://problem/68034716>
Reviewed by Tim Horton.
Source/WebCore:
In r266262 ScrollingTreeLatchingController started to consult the "last handled wheel event"
time when deciding whether to use the latched node, but we didn't update this timestamp when
handling an event on the latched node, on the scrolling thread. So fix that.
Test: fast/scrolling/latching/latched-scroll-into-nonfast-region.html
- page/scrolling/ScrollingTree.cpp:
(WebCore::ScrollingTree::handleWheelEvent):
LayoutTests:
- fast/scrolling/latching/latched-scroll-into-nonfast-region-expected.txt: Added.
- fast/scrolling/latching/latched-scroll-into-nonfast-region.html: Added.
- 1:24 PM Changeset in webkit [266335] by
-
- 2 edits in trunk/LayoutTests
[WinCairo] Unreviewed test gardening
- platform/wincairo/TestExpectations:
- 12:45 PM Changeset in webkit [266334] by
-
- 1 edit1 add in trunk/LayoutTests
Add missing test result from r266333.
- fast/scrolling/latching/latched-scroll-remove-iframe-expected.txt: Added.
- 10:25 AM Changeset in webkit [266333] by
-
- 16 edits3 adds2 deletes in trunk
Rewrite main thread scroll latching logic
https://bugs.webkit.org/show_bug.cgi?id=215979
Reviewed by Tim Horton.
Source/WebCore:
The existing main thread wheel event handling and latching logic had a number of issues,
some of which were indicated via the FIXME comments added in r266016:
- It tracked scrollable containers as ContainerNodes rather than ScrollableAreas
- It fetched and used latched state from a different frame, causing an EventHandler to end up scrolling some unrelated frame.
- Overflow scrolling ignored latched state.
- The latching stack's purpose was unclear.
This patch fixes those issues. The design is as follow:
- Latching logic is moved into ScrollLatchingController, which is owned by Page.
- ScrollLatchingController owns a stack of FrameState.
- When receiving a wheel event, determineWheelEventTarget() identifies the target ScrollableArea, if any, for that frame only.
- As hit-testing descends into subframes, state is pushed onto ScrollLatchingController's state stack. Frames with potential scrollers have a non-null ScrollableArea in their state.
- The latched ScrollableArea is the top non-null ScrollableArea in this stack.
- EventHandler consults ScrollLatchingController for select, overflow and frame scrolling.
This change fixes an issue where scrolling over an overflow:scroll in the non-fast scrollable
region would rubber-band the overflow, rather than the main page (tested by the adjusted
fast/scrolling/mac/rubberband-overflow-in-wheel-region.html).
Tests: fast/scrolling/latching/latched-scroll-remove-iframe.html
- Sources.txt:
- WebCore.xcodeproj/project.pbxproj:
- dom/Element.cpp:
(WebCore::Element::removedFromAncestor):
- page/EventHandler.cpp:
(WebCore::EventHandler::EventHandler):
(WebCore::EventHandler::determineWheelEventTarget):
(WebCore::EventHandler::processWheelEventForScrolling):
(WebCore::EventHandler::platformCompletePlatformWidgetWheelEvent):
(WebCore::EventHandler::processWheelEventForScrollSnap):
(WebCore::EventHandler::completeWidgetWheelEvent):
(WebCore::EventHandler::handleWheelEvent):
(WebCore::EventHandler::clearLatchedState):
(WebCore::EventHandler::defaultWheelEventHandler):
(WebCore::EventHandler::clearLatchedStateTimerFired): Deleted.
(WebCore::EventHandler::clearOrScheduleClearingLatchedStateIfNeeded): Deleted.
- page/EventHandler.h:
- page/Page.cpp:
(WebCore::Page::startMonitoringWheelEvents):
(WebCore::Page::scrollLatchingController):
(WebCore::Page::scrollLatchingControllerIfExists):
(WebCore::Page::latchingState): Deleted.
(WebCore::Page::pushNewLatchingState): Deleted.
(WebCore::Page::resetLatchingState): Deleted.
(WebCore::Page::popLatchingState): Deleted.
(WebCore::Page::removeLatchingStateForTarget): Deleted.
- page/Page.h:
(WebCore::Page::latchingStateStack const): Deleted.
- page/mac/EventHandlerMac.mm:
(WebCore::EventHandler::determineWheelEventTarget):
(WebCore::EventHandler::processWheelEventForScrolling):
(WebCore::EventHandler::platformCompletePlatformWidgetWheelEvent):
(WebCore::EventHandler::processWheelEventForScrollSnap):
(WebCore::deltaIsPredominantlyVertical): Deleted.
(WebCore::scrolledToEdgeInDominantDirection): Deleted.
(WebCore::latchingIsLockedToPlatformFrame): Deleted.
(WebCore::latchingIsLockedToAncestorOfThisFrame): Deleted.
(WebCore::latchedToFrameOrBody): Deleted.
(WebCore::EventHandler::clearOrScheduleClearingLatchedStateIfNeeded): Deleted.
(WebCore::frameViewForLatchingState): Deleted.
- page/scrolling/ScrollLatchingController.cpp: Added.
(WebCore::ScrollLatchingController::ScrollLatchingController):
(WebCore::ScrollLatchingController::clear):
(WebCore::ScrollLatchingController::clearOrScheduleClearIfNeeded):
(WebCore::ScrollLatchingController::clearTimerFired):
(WebCore::ScrollLatchingController::receivedWheelEvent):
(WebCore::ScrollLatchingController::latchingAllowsScrollingInFrame const):
(WebCore::ScrollLatchingController::updateLatchingStateForFrame):
(WebCore::ScrollLatchingController::getLatchingStateForFrame const):
(WebCore::ScrollLatchingController::removeLatchingStateForTarget):
(WebCore::ScrollLatchingController::removeLatchingStateForFrame):
(WebCore::deltaIsPredominantlyVertical):
(WebCore::ScrollLatchingController::shouldLatchToScrollableArea const):
(WebCore::ScrollLatchingController::hasStateForFrame const):
(WebCore::ScrollLatchingController::stateForFrame):
(WebCore::ScrollLatchingController::stateForFrame const):
(WebCore::ScrollLatchingController::dump const):
(WebCore::operator<<):
- page/scrolling/ScrollLatchingController.h: Added.
(WebCore::ScrollLatchingController::cumulativeEventDelta const):
- page/scrolling/ScrollLatchingState.cpp: Removed.
- page/scrolling/ScrollLatchingState.h: Removed.
- page/scrolling/ScrollingTreeLatchingController.cpp:
(WebCore::ScrollingTreeLatchingController::receivedWheelEvent):
(WebCore::ScrollingTreeLatchingController::nodeDidHandleEvent):
- page/scrolling/ThreadedScrollingTree.cpp:
(WebCore::ThreadedScrollingTree::handleWheelEventAfterMainThread):
- page/scrolling/mac/ScrollingTreeScrollingNodeDelegateMac.mm:
(WebCore::ScrollingTreeScrollingNodeDelegateMac::allowsHorizontalStretching const):
(WebCore::ScrollingTreeScrollingNodeDelegateMac::allowsVerticalStretching const):
(WebCore::newGestureIsStarting): Deleted.
- platform/PlatformWheelEvent.h:
(WebCore::PlatformWheelEvent::isGestureStart const):
(WebCore::PlatformWheelEvent::isGestureContinuation const):
(WebCore::PlatformWheelEvent::shouldResetLatching const):
(WebCore::PlatformWheelEvent::isNonGestureEvent const):
(WebCore::PlatformWheelEvent::shouldConsiderLatching const): Deleted. Renamed to isGestureStart(). This class
should not prescribe latching behaviors.
LayoutTests:
Add a test for iframe unparenting in the middle of a latched scroll.
- fast/scrolling/latching/latched-scroll-remove-iframe.html: Added.
- fast/scrolling/latching/scroll-nested-iframe.html: 1000ms -> 0ms
- fast/scrolling/mac/rubberband-overflow-in-wheel-region.html: Test needs to latch the overflow by scrolling down then up.
- 2:53 AM Changeset in webkit [266332] by
-
- 26 edits5 adds in trunk
Add support for TextEncoderStream
https://bugs.webkit.org/show_bug.cgi?id=215916
Reviewed by Alex Christensen.
LayoutTests/imported/w3c:
- web-platform-tests/encoding/idlharness.any-expected.txt:
- web-platform-tests/encoding/idlharness.any.worker-expected.txt:
- web-platform-tests/encoding/streams/backpressure.any-expected.txt:
- web-platform-tests/encoding/streams/backpressure.any.worker-expected.txt:
- web-platform-tests/encoding/streams/encode-bad-chunks.any-expected.txt:
- web-platform-tests/encoding/streams/encode-bad-chunks.any.worker-expected.txt:
- web-platform-tests/encoding/streams/encode-utf8.any-expected.txt:
- web-platform-tests/encoding/streams/encode-utf8.any.worker-expected.txt:
- web-platform-tests/encoding/streams/readable-writable-properties.any-expected.txt:
- web-platform-tests/encoding/streams/readable-writable-properties.any.worker-expected.txt:
- web-platform-tests/encoding/streams/realms.window-expected.txt:
- web-platform-tests/service-workers/service-worker/fetch-event.https-expected.txt:
Source/WebCore:
Implement TextEncoderStream as a JS built-in.
It is controlled by transform stream runtime flag.
TextEncoderStream takes a TransformStream and TextEncoderStreamEncoder as private slots.
Add TextEncoderStreamEncoder to implement the actual UTF-8 encoding steps.
Covered by rebased tests.
- CMakeLists.txt:
- DerivedSources-input.xcfilelist:
- DerivedSources-output.xcfilelist:
- DerivedSources.make:
- Modules/streams/TransformStream.idl:
- Modules/streams/TransformStream.js:
(initializeTransformStream):
- Modules/streams/TransformStreamInternals.js:
(createTransformStream):
- Sources.txt:
- WebCore.xcodeproj/project.pbxproj:
- bindings/js/WebCoreBuiltinNames.h:
- dom/TextEncoderStream.idl: Added.
- dom/TextEncoderStream.js: Added.
(initializeTextEncoderStream):
(getter.encoding):
(getter.readable):
(getter.writable):
- dom/TextEncoderStreamEncoder.cpp: Added.
(WebCore::TextEncoderStreamEncoder::encode):
(WebCore::TextEncoderStreamEncoder::flush):
- dom/TextEncoderStreamEncoder.h: Added.
(WebCore::TextEncoderStreamEncoder::create):
- dom/TextEncoderStreamEncoder.idl: Added.
LayoutTests:
- 2:40 AM Changeset in webkit [266331] by
-
- 2 edits in trunk/Source/WTF
[GTK] REGRESSION(r150392) insufficient space allocation results in heap corruption
https://bugs.webkit.org/show_bug.cgi?id=215976
Patch by Jim Mason <jmason@ibinx.com> on 2020-08-30
Reviewed by Carlos Garcia Campos.
- wtf/glib/RunLoopGLib.cpp:
(WTF::RunLoop::TimerBase::TimerBase):
Aug 29, 2020:
- 6:47 PM Changeset in webkit [266330] by
-
- 64 edits2 adds in trunk
Remove NFC normalization when submitting forms and encoding URL queries and fix EUC-JP encoding
https://bugs.webkit.org/show_bug.cgi?id=215970
Reviewed by Darin Adler.
LayoutTests/imported/w3c:
- web-platform-tests/encoding/legacy-mb-japanese/euc-jp/eucjp-encode-form-cseucpkdfmtjapanese-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/euc-jp/eucjp-encode-form-errors-han-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/euc-jp/eucjp-encode-form-errors-misc-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/euc-jp/eucjp-encode-form-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/euc-jp/eucjp-encode-form-x-euc-jp-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/euc-jp/eucjp-encode-href-errors-han-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/euc-jp/eucjp-encode-href-errors-misc-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/euc-jp/eucjp-encode-href-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/iso-2022-jp/iso2022jp-encode-form-csiso2022jp-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/iso-2022-jp/iso2022jp-encode-form-errors-han-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/iso-2022-jp/iso2022jp-encode-form-errors-misc-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/iso-2022-jp/iso2022jp-encode-form-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/iso-2022-jp/iso2022jp-encode-href-errors-han-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/iso-2022-jp/iso2022jp-encode-href-errors-misc-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/iso-2022-jp/iso2022jp-encode-href-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/shift_jis/sjis-encode-form-csshiftjis-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/shift_jis/sjis-encode-form-errors-han-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/shift_jis/sjis-encode-form-errors-misc-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/shift_jis/sjis-encode-form-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/shift_jis/sjis-encode-form-ms932-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/shift_jis/sjis-encode-form-ms_kanji-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/shift_jis/sjis-encode-form-shift-jis-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/shift_jis/sjis-encode-form-sjis-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/shift_jis/sjis-encode-form-windows-31j-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/shift_jis/sjis-encode-form-x-sjis-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/shift_jis/sjis-encode-href-errors-han-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/shift_jis/sjis-encode-href-errors-misc-expected.txt:
- web-platform-tests/encoding/legacy-mb-japanese/shift_jis/sjis-encode-href-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-form-cseuckr-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-form-csksc56011987-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-form-errors-han-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-form-errors-misc-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-form-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-form-iso-ir-149-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-form-korean-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-form-ks_c_5601-1987-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-form-ks_c_5601-1989-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-form-ksc5601-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-form-ksc_5601-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-form-windows-949-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-href-errors-han-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-href-errors-misc-expected.txt:
- web-platform-tests/encoding/legacy-mb-korean/euc-kr/euckr-encode-href-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-big5-hkscs-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-cn-big5-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-csbig5-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-errors-han-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-errors-misc-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-form-x-x-big5-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-href-errors-han-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-href-errors-misc-expected.txt:
- web-platform-tests/encoding/legacy-mb-tchinese/big5/big5-encode-href-expected.txt:
Source/WebCore:
This fixes 3 things that were common between Chrome and Firefox but different in WebKit.
The first two are omitting NFC normalization when submitting forms and encoding URL queries
in documents with non-UTF-8 encodings. The third is a problem with our use of ICU's EUC-JP encoding,
which uses the euc-jp-2007 table. Chrome and Firefox use a slightly different table from
https://encoding.spec.whatwg.org/index-jis0208.txt which is not in ICU. To be compatible and pass many
web platform tests, I include the table here, which adds about 30kb to WebKit's binary size.
I intend to do the same with Big5, ISO-2022-JP, Shift_JIS, and EUC-KR.
I opened https://github.com/whatwg/encoding/issues/225 to resolve an ambiguity in the standard related to this.
- Sources.txt:
- WebCore.xcodeproj/project.pbxproj:
- platform/network/FormData.cpp:
(WebCore::normalizeStringData):
- platform/text/EncodingTables.cpp: Added.
- platform/text/EncodingTables.h: Added.
- platform/text/TextCodecICU.cpp:
(WebCore::eucJPEncode):
(WebCore::uncheckedAppendDecimal):
(WebCore::urlEncodedEntityUnencodableHandler):
(WebCore::entityUnencodableHandler):
(WebCore::questionMarkUnencodableHandler):
(WebCore::Function<void):
(WebCore::TextCodecICU::encode):
- platform/text/TextEncoding.cpp:
(WebCore::TextEncoding::encode const):
- platform/text/TextEncoding.h:
LayoutTests:
- fast/forms/form-data-encoding-2.html:
- fast/forms/form-data-encoding.html:
- 5:19 PM Changeset in webkit [266329] by
-
- 2 edits in trunk/Source/WebCore
[Cocoa] MediaSample JSON strings are created for logging to disabled channels
https://bugs.webkit.org/show_bug.cgi?id=215980
<rdar://problem/68004132>
Reviewed by Simon Fraser.
On platforms that USE(OS_LOG), DEBUG_LOG() evaluates its arguments even in production
builds. In SourceBufferPrivateAVFObjC::didProvideMediaDataForTrackID(), one DEBUG_LOG() call
involved creating a non-trivial JSON object describing a MediaSample then converting that to
a JSON string. This string was created even when debug logging was disabled at runtime.
Profiles showed this cost half a millisecond per second of video playback on netflix.com in
Safari on a MacBook Pro.
- platform/graphics/avfoundation/objc/SourceBufferPrivateAVFObjC.mm:
(WebCore::SourceBufferPrivateAVFObjC::didProvideMediaDataForTrackID): Rather than calling
MediaSample::toJSONString() directly, let the LogArgument<MediaSample> specialization call
toJSONString() iff the log channel is enabled.
- 5:05 PM Changeset in webkit [266328] by
-
- 5 edits in trunk/Source/WebCore
Move PerformanceMonitor related DeprecatedGlobalSettings to Settings
https://bugs.webkit.org/show_bug.cgi?id=215963
Reviewed by Anders Carlsson.
The PerformanceMonitor related uses of DeprecatedGlobalSettings can easily be moved to Settings, as PerformanceMonitor
has a Page reference.
- page/DeprecatedGlobalSettings.h:
(WebCore::DeprecatedGlobalSettings::isPostLoadCPUUsageMeasurementEnabled): Deleted.
(WebCore::DeprecatedGlobalSettings::isPostBackgroundingCPUUsageMeasurementEnabled): Deleted.
(WebCore::DeprecatedGlobalSettings::isPerActivityStateCPUUsageMeasurementEnabled): Deleted.
(WebCore::DeprecatedGlobalSettings::isPostLoadMemoryUsageMeasurementEnabled): Deleted.
(WebCore::DeprecatedGlobalSettings::isPostBackgroundingMemoryUsageMeasurementEnabled): Deleted.
- page/Settings.yaml:
- page/SettingsDefaultValues.h:
Moved settings from DeprecatedGlobalSettings.h to Settings.yaml. Default values encoded in SettingsDefaultValues.h
- page/PerformanceMonitor.cpp:
(WebCore::PerformanceMonitor::PerformanceMonitor):
(WebCore::PerformanceMonitor::didFinishLoad):
(WebCore::PerformanceMonitor::activityStateChanged):
Access the settings via m_page.settings(). Update constants to use constexpr.
- 1:44 PM Changeset in webkit [266327] by
-
- 4 edits1 add in trunk
[JSC]
let [
sequence cannot appear in ExpressionStatement context
https://bugs.webkit.org/show_bug.cgi?id=215977
Reviewed by Ross Kirsling.
JSTests:
- stress/let-and-open-bracket.js: Added.
(testSyntax):
(testSyntaxError):
- test262/expectations.yaml:
Source/JavaScriptCore:
Because of ambiguity between destructuring assignment and member access (let IDENTIFIER), ECMA262 does not allow
let [
sequence in ExpressionStatement context[1].
We should throw SyntaxError when we see something like this.
if (false)
let [ok] = [42];
[1]: https://tc39.es/ecma262/#sec-expression-statement
- parser/Parser.cpp:
(JSC::Parser<LexerType>::parseStatement):
- 12:06 PM Changeset in webkit [266326] by
-
- 4 edits in trunk
[JSC] for-of uses AssignmentExpression while for-in uses Expression
https://bugs.webkit.org/show_bug.cgi?id=215975
Reviewed by Ross Kirsling.
JSTests:
- test262/expectations.yaml:
Source/JavaScriptCore:
While for-in uses Expression, for-of and for-await-of use AssignmentExpression which does not accept comma-expression.
We should align our implementation to that.
for (LeftHandSideExpression in Expression) Statement
for (LeftHandSideExpression of AssignmentExpression) Statement
for await(LeftHandSideExpression of AssignmentExpression) Statement
- parser/Parser.cpp:
(JSC::Parser<LexerType>::parseForStatement):
- 10:19 AM Changeset in webkit [266325] by
-
- 4 edits in trunk/LayoutTests
[GLIB] Rebaseline webaudio tests after k-rate changes in r266319
Unreviewed test gardening.
- platform/glib/imported/w3c/web-platform-tests/webaudio/the-audio-api/the-audioparam-interface/k-rate-panner-connections-expected.txt:
- platform/glib/imported/w3c/web-platform-tests/webaudio/the-audio-api/the-audioparam-interface/k-rate-panner-expected.txt:
- platform/glib/imported/w3c/web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/panner-automation-basic-expected.txt:
- 1:38 AM Changeset in webkit [266324] by
-
- 9 edits1 add in trunk
[JSC] for-of / for-in left-hand-side target should be simple-assignment-target
https://bugs.webkit.org/show_bug.cgi?id=215969
Reviewed by Ross Kirsling.
JSTests:
- stress/for-of-in-non-reference.js: Added.
(testSyntax):
(testSyntaxError):
(testSyntaxError.t):
(testSyntaxError.async t):
(testSyntax.async t):
- test262/expectations.yaml:
Source/JavaScriptCore:
Left-hand-side of
for-in
,for-of
, andfor-await-of
should be simple assignment target[1]
if the target is not declaration and not destructuring pattern.
[1]: https://tc39.es/ecma262/#sec-for-in-and-for-of-statements-static-semantics-early-errors
- parser/Parser.cpp:
(JSC::Parser<LexerType>::parseForStatement):
- parser/SyntaxChecker.h:
(JSC::SyntaxChecker::createCommaExpr): Should return CommaExpr to align it to ASTBuilder.
(JSC::SyntaxChecker::appendToCommaExpr):
(JSC::SyntaxChecker::appendStatement):
(JSC::SyntaxChecker::combineCommaNodes): Deleted since it is not used.
LayoutTests:
Fix for-in / for-of parser behavior.
- js/basic-strict-mode-expected.txt:
- js/parser-syntax-check-expected.txt:
- js/script-tests/parser-syntax-check.js:
- 1:32 AM Changeset in webkit [266323] by
-
- 10 edits1 add in trunk
[JSC] Implement Intl.DateTimeFormat dayPeriod
https://bugs.webkit.org/show_bug.cgi?id=215839
Reviewed by Ross Kirsling.
JSTests:
- stress/intl-datetimeformat-day-period.js: Added.
(shouldBe):
(throw.new.Error):
- test262/config.yaml:
Source/JavaScriptCore:
This patch implements Intl.DateTimeFormat dayPeriod option[1]. We can use "narrow", "short", or "long" for dayPeriod,
and it determines how "AM" etc. is represented.
[1]: https://github.com/tc39/ecma402/pull/346
- builtins/DatePrototype.js:
(toLocaleString.toDateTimeOptionsAnyAll):
(toLocaleString):
(toLocaleTimeString.toDateTimeOptionsTimeTime):
(toLocaleTimeString):
- bytecode/BytecodeIntrinsicRegistry.cpp:
(JSC::BytecodeIntrinsicRegistry::BytecodeIntrinsicRegistry):
- bytecode/BytecodeIntrinsicRegistry.h:
- runtime/CommonIdentifiers.h:
- runtime/IntlDateTimeFormat.cpp:
(JSC::toDateTimeOptionsAnyDate):
(JSC::IntlDateTimeFormat::setFormatsFromPattern):
(JSC::IntlDateTimeFormat::initializeDateTimeFormat):
(JSC::IntlDateTimeFormat::dayPeriodString):
(JSC::IntlDateTimeFormat::resolvedOptions const):
- runtime/IntlDateTimeFormat.h:
- runtime/OptionsList.h:
- 12:11 AM Changeset in webkit [266322] by
-
- 6 edits1 add in trunk
[JSC] super property with new should be accepted
https://bugs.webkit.org/show_bug.cgi?id=215966
Reviewed by Ross Kirsling.
JSTests:
- stress/super-and-new.js: Added.
(testSyntaxError):
(shouldBe):
(test.A):
(test.A.prototype.get hey):
(test.A.prototype.get hey2):
(test.A.prototype.super):
(test.B.get super):
Source/JavaScriptCore:
While we should reject
new super
/new super()
, we should acceptnew super.property
.
https://tc39.es/ecma262/#prod-SuperProperty is a child production of https://tc39.es/ecma262/#prod-MemberExpression,
unlike https://tc39.es/ecma262/#prod-SuperCall. Sonew
should accept SuperProperty (e.g.super.xxx
).
- parser/Parser.cpp:
(JSC::Parser<LexerType>::parseMemberExpression):
LayoutTests:
- js/class-syntax-super-expected.txt:
- js/script-tests/class-syntax-super.js:
Aug 28, 2020:
- 11:57 PM BuildingCairoOnWindows edited by
- (diff)
- 10:13 PM Changeset in webkit [266321] by
-
- 24 edits in branches/safari-610.1.28.0-branch/LayoutTests/imported/w3c/web-platform-tests/webaudio
Cherry-pick r266315. rdar://problem/67367072
Rebaseline WebAudio test results
<rdar://problem/67367072>
- r265824 changed what error message is. (This is unavoidable since the kind of thrown error is changed (JS side error v.s. JS error thrown from C++), and this needs to be done to fix the reported webpage’s issue).
- This error is recorded as expect files in various WebAudio WPT test if the tested WebAudio constructors are not implemented.
- After that branch cut, a lot of WebAudio constructors are newly implemented, which removes this error messages from expect files.
- r265824 is landed after these WebAudio patches. So it does not include rebaseline for these WebAudio WPT tests (since it is no longer including that error messages).
- But in that branch, these WebAudio patches are not cherry-picked. This means that error message is still recorded in WPT expect files.
- So, we just need to update these expect files, or cherry-pick these WebAudio patches. And I think updating expect files is the way to go.
git-svn-id: https://svn.webkit.org/repository/webkit/branches/safari-610.1.28.1-branch@266315 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- 8:30 PM Changeset in webkit [266320] by
-
- 2 edits4 adds1 delete in trunk/LayoutTests
[WPE] Gardening and rebaselining
Unreviewed test gardening.
- platform/wpe/TestExpectations:
- platform/wpe/imported/w3c/web-platform-tests/html/editing/dnd/datastore/datatransfer-constructor-001-expected.txt:
Added with specific failure due to DRAG_SUPPORT not enabled.
- platform/wpe/imported/w3c/web-platform-tests/service-workers/service-worker/fetch-event-within-sw-manual.https-expected.txt: Removed.
- 7:32 PM Changeset in webkit [266319] by
-
- 26 edits in trunk
Update PannerNode to support k-rate automation of its AudioParams
https://bugs.webkit.org/show_bug.cgi?id=215901
Reviewed by Darin Adler.
LayoutTests/imported/w3c:
Rebaseline WPT tests now that more checks are passing.
- web-platform-tests/webaudio/the-audio-api/the-audioparam-interface/k-rate-panner-connections-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-audioparam-interface/k-rate-panner-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/automation-changes-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/panner-automation-basic-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/panner-automation-equalpower-stereo-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/panner-automation-position-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/panner-equalpower-stereo-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/test-pannernode-automation-expected.txt:
Source/WebCore:
Update PannerNode to support k-rate automation of its AudioParams.
This patch is based on Chromium's implementation and the following
change in particular:
No new tests, rebaselined existing tests.
- Modules/webaudio/AudioListener.cpp:
(WebCore::AudioListener::AudioListener):
(WebCore::AudioListener::hasSampleAccurateValues const):
(WebCore::AudioListener::shouldUseARate const):
(WebCore::AudioListener::updateValuesIfNeeded):
(WebCore::AudioListener::positionXValues):
(WebCore::AudioListener::positionYValues):
(WebCore::AudioListener::positionZValues):
(WebCore::AudioListener::forwardXValues):
(WebCore::AudioListener::forwardYValues):
(WebCore::AudioListener::forwardZValues):
(WebCore::AudioListener::upXValues):
(WebCore::AudioListener::upYValues):
(WebCore::AudioListener::upZValues):
Add functions to retrieve sample accurate values from the AudioListener's
AudioParams. Those values are cached for performance.
(WebCore::AudioListener::setPosition):
Call AudioParam::setValueAtTime(value, now) instead of
AudioParam::setValue(), as per:
(WebCore::AudioListener::setOrientation):
Call AudioParam::setValueAtTime(value, now) instead of
AudioParam::setValue(), as per:
- Modules/webaudio/AudioListener.h:
- Modules/webaudio/AudioListener.idl:
Update IDL to indicate that setPosition() / setOrientation() may now
throw exceptions.
- Modules/webaudio/AudioParam.cpp:
(WebCore::AudioParam::calculateFinalValues):
If we're not sample accurate, make sure we duplicate the first element of
|values| to other elements. Some WPT tests kept failing without this change.
- Modules/webaudio/AudioParam.h:
- Modules/webaudio/AudioParamTimeline.h:
(WebCore::AudioParamTimeline::hasValues const):
Add more constness.
- Modules/webaudio/PannerNode.cpp:
(WebCore::PannerNode::process):
(WebCore::PannerNode::processSampleAccurateValues):
Add support for k-rate automation of PannerNode's AudioParams.
(WebCore::PannerNode::hasSampleAccurateValues const):
(WebCore::PannerNode::shouldUseARate const):
(WebCore::PannerNode::setPanningModel):
(WebCore::PannerNode::setPosition):
Call AudioParam::setValueAtTime(value, now) instead of
AudioParam::setValue(), as per:
(WebCore::PannerNode::setOrientation):
Call AudioParam::setValueAtTime(value, now) instead of
AudioParam::setValue(), as per:
(WebCore::PannerNode::setDistanceModel):
(WebCore::PannerNode::setRefDistance):
(WebCore::PannerNode::setMaxDistance):
(WebCore::PannerNode::setRolloffFactor):
(WebCore::PannerNode::setConeOuterGain):
(WebCore::PannerNode::setConeOuterAngle):
(WebCore::PannerNode::setConeInnerAngle):
(WebCore::PannerNode::setChannelCount):
(WebCore::PannerNode::setChannelCountMode):
(WebCore::PannerNode::calculateAzimuthElevation):
(WebCore::PannerNode::azimuthElevation):
(WebCore::PannerNode::calculateDistanceConeGain):
(WebCore::PannerNode::distanceConeGain):
- Modules/webaudio/PannerNode.h:
- Modules/webaudio/PannerNode.idl:
- platform/audio/EqualPowerPanner.cpp:
(WebCore::EqualPowerPanner::calculateDesiredGain):
(WebCore::EqualPowerPanner::panWithSampleAccurateValues):
- platform/audio/EqualPowerPanner.h:
- platform/audio/HRTFPanner.cpp:
(WebCore::HRTFPanner::panWithSampleAccurateValues):
- platform/audio/HRTFPanner.h:
- platform/audio/Panner.h:
LayoutTests:
Unskip a few tests that should no longer be flaky since they are now passing.
- 7:26 PM Changeset in webkit [266318] by
-
- 6 edits in trunk
[JSC]
new import.meta()
is acceptable
https://bugs.webkit.org/show_bug.cgi?id=215915
Reviewed by Ross Kirsling.
JSTests:
- stress/modules-syntax-error.js:
- stress/modules-syntax.js:
- test262/expectations.yaml:
Source/JavaScriptCore:
new import.meta()
is valid in terms of syntax while it throws runtime error.
We should accept this code, whilenew import()
is not correct syntax.
- parser/Parser.cpp:
(JSC::Parser<LexerType>::parseMemberExpression):
- 6:39 PM Changeset in webkit [266317] by
-
- 40 edits2 adds in trunk
Web Inspector: Audit: should be able to create/edit imported audits
https://bugs.webkit.org/show_bug.cgi?id=215555
<rdar://problem/67255483>
Reviewed by Devin Rousso.
Source/WebInspectorUI:
Extend the existing audit edit mode (which previously only let audits be disabled/deleted)
to also allow other JSON properties of audits to be added/edited/removed, meaning that
audits can now be completely created/configured from within the Web Inspector UI. \(o.o)/
- UserInterface/Models/AuditTestBase.js:
(WI.AuditTestBase):
(WI.AuditTestBase.prototype.set name): Added.
(WI.AuditTestBase.prototype.set description): Added.
(WI.AuditTestBase.prototype.get supports): Added.
(WI.AuditTestBase.prototype.set supports): Added.
(WI.AuditTestBase.prototype.get setup): Added.
(WI.AuditTestBase.prototype.set setup): Added.
(WI.AuditTestBase.prototype.set disabled):
(WI.AuditTestBase.prototype.get editable): Added.
(WI.AuditTestBase.prototype.get default): Added.
(WI.AuditTestBase.prototype.markAsDefault): Added.
(WI.AuditTestBase.prototype.get topLevelTest): Added.
(WI.AuditTestBase.prototype.async runSetup): Added.
(WI.AuditTestBase.prototype.async start()):
(WI.AuditTestBase.prototype.stop()):
(WI.AuditTestBase.prototype.async clone): Added.
(WI.AuditTestBase.remove): Added.
(WI.AuditTestBase.saveIdentityToCookie):
(WI.AuditTestBase.toJSON):
(WI.AuditTestBase.prototype.determineIfSupported): Added.
(WI.AuditTestBase.prototype.updateSupported): Added.
(WI.AuditTestBase.prototype.updateResult): Added.
(WI.AuditTestBase.prototype.set supported): Deleted.
(WI.AuditTestBase.async setup): Deleted.
(WI.AuditTestBase.prototype.async setup): Deleted.
- UserInterface/Models/AuditTestGroup.js:
(WI.AuditTestGroup):
(WI.AuditTestGroup.prototype.addTest): Added.
(WI.AuditTestGroup.prototype.removeTest): Added.
(WI.AuditTestGroup.prototype.clearResult):
(WI.AuditTestGroup.prototype.async run):
(WI.AuditTestGroup.prototype.determineIfSupported): Added.
(WI.AuditTestGroup.prototype.updateSupported): Added.
(WI.AuditTestGroup.prototype.updateResult): Added.
(WI.AuditTestGroup.prototype._checkDisabled): Added.
(WI.AuditTestGroup.prototype._handleTestCompleted):
(WI.AuditTestGroup.prototype._handleTestDisabledChanged):
(WI.AuditTestGroup.prototype._handleTestSupportedChanged): Added.
(WI.AuditTestGroup.prototype._handleTestChanged): Added.
(WI.AuditTestGroup.prototype.get supported): Deleted.
(WI.AuditTestGroup.prototype.set supported): Deleted.
(WI.AuditTestGroup.prototype.get disabled): Deleted.
(WI.AuditTestGroup.prototype.set disabled): Deleted.
(WI.AuditTestGroup.prototype._updateResult): Deleted.
(WI.AuditTestGroup.prototype._updateResult): Deleted.
- UserInterface/Models/AuditTestCase.js:
(WI.AuditTestCase):
(WI.AuditTestCase.stringifyFunction): Added.
(WI.AuditTestCase.prototype.set test): Added.
(WI.AuditTestCase.prototype.async run):
Allow additional JSON keys to be changed in the UI:
name
description
supports
(ensure that changes recalculate whether the audit is actually supported)setup
- (groups)
tests
(via separate add and remove functions) - (test cases) the actual
test
function
These properties are expected to only change in edit mode.
- UserInterface/Models/AuditTestResultBase.js:
(WI.AuditTestResultBase.prototype.get disabled): Added.
(WI.AuditTestResultBase.prototype.get editable): Added.
Audit results are never disabled and not editable.
- UserInterface/Controllers/AuditManager.js:
(WI.AuditManager):
(WI.AuditManager.prototype.set editing):
(WI.AuditManager.async start):
(WI.AuditManager.prototype.async start):
(WI.AuditManager.prototype.stop):
(WI.AuditManager.prototype.async processJSON):
(WI.AuditManager.prototype.loadStoredTests):
(WI.AuditManager.prototype.async addTest): Added.
(WI.AuditManager.prototype.removeTest):
(WI.AuditManager.prototype._addDefaultTests):
(WI.AuditManager.prototype._addDefaultTests.removeWhitespace):
(WI.AuditManager.prototype._addTest): Deleted.
(WI.AuditManager.prototype._topLevelTestForTest): Deleted.
Add helper functions and events for when the audit manager running mode changes to update UI.
Also add another default test that demonstrates thesupports
JSON key.
- UserInterface/Views/AuditNavigationSidebarPanel.js:
(WI.AuditNavigationSidebarPanel.prototype.showDefaultContentView):
(WI.AuditNavigationSidebarPanel.prototype.willDismissPopover): Added.
(WI.AuditNavigationSidebarPanel.prototype.initialLayout):
(WI.AuditNavigationSidebarPanel.prototype.matchTreeElementAgainstCustomFilters):
(WI.AuditNavigationSidebarPanel.prototype._addTest):
(WI.AuditNavigationSidebarPanel.prototype._addResult):
(WI.AuditNavigationSidebarPanel.prototype._updateControlNavigationItems): Added.
(WI.AuditNavigationSidebarPanel.prototype._updateEditNavigationItems): Added.
(WI.AuditNavigationSidebarPanel.prototype._handleAuditManagerEditingChanged):
(WI.AuditNavigationSidebarPanel.prototype._handleAuditManagerRunningStateChanged): Added.
(WI.AuditNavigationSidebarPanel.prototype._handleAuditTestAdded):
(WI.AuditNavigationSidebarPanel.prototype._handleAuditTestCompleted):
(WI.AuditNavigationSidebarPanel.prototype._handleAuditTestRemoved):
(WI.AuditNavigationSidebarPanel.prototype._handleAuditTestScheduled):
(WI.AuditNavigationSidebarPanel.prototype._treeSelectionDidChange):
(WI.AuditNavigationSidebarPanel.prototype._handleStartStopButtonNavigationItemClicked):
(WI.AuditNavigationSidebarPanel.prototype._handleCreateButtonNavigationItemClicked): Added.
(WI.AuditNavigationSidebarPanel.prototype._updateStartStopButtonNavigationItemState): Deleted.
(WI.AuditNavigationSidebarPanel.prototype._updateEditButtonNavigationItemState): Deleted.
- UserInterface/Views/AuditNavigationSidebarPanel.css:
(.sidebar > .panel.navigation.audit .edit-audits:not(.disabled):active): Added.
(.sidebar > .panel.navigation.audit .edit-audits:not(.disabled).activated): Added.
(.sidebar > .panel.navigation.audit .edit-audits:not(.disabled).activated:active): Added.
(.sidebar > .panel.navigation.audit .edit-audits.disabled): Added.
(.content-view.audit .message-text-view .navigation-item-help:is(.start-editing-audits, .stop-editing-audits) .navigation-bar): Added.
(.content-view.tab.audit .content-view > .audit-version): Added.
(.content-view.tab.audit .content-view .reference-page-link-container): Added.
(body[dir=ltr] .content-view.tab.audit .content-view .reference-page-link-container): Added.
(body[dir=rtl] .content-view.tab.audit .content-view .reference-page-link-container): Added.
(.sidebar > .panel.navigation.audit > .content .edit-audits:not(.disabled):active): Deleted.
(.sidebar > .panel.navigation.audit > .content .edit-audits:not(.disabled).activated): Deleted.
(.sidebar > .panel.navigation.audit > .content .edit-audits:not(.disabled).activated:active): Deleted.
(.sidebar > .panel.navigation.audit > .content .edit-audits.disabled): Deleted.
(.finish-editing-audits-placeholder.message-text-view .navigation-item-help .navigation-bar): Deleted.
(.audit-version): Deleted.
Move the "Edit" toggle to be next to the filter bar so that it's always visible. Replace the
"[|>] Start" button with a "[+] Create" button when in edit mode, which adds a new audit at
the top-level.
- UserInterface/Views/AuditTestContentView.js:
(WI.AuditTestContentView):
(WI.AuditTestContentView.get navigationItems):
(WI.AuditTestContentView.get supportsSave):
(WI.AuditTestContentView.prototype.createNameElement): Added.
(WI.AuditTestContentView.prototype.createDescriptionElement): Added.
(WI.AuditTestContentView.prototype.createControlsTableElement): Added.
(WI.AuditTestContentView.prototype.layout):
(WI.AuditTestContentView.prototype.shown):
(WI.AuditTestContentView.prototype.hidden):
(WI.AuditTestContentView.prototype.handleResultChanged):
(WI.AuditTestContentView.prototype.showStoppingPlaceholder):
(WI.AuditTestContentView.prototype.showNoResultPlaceholder):
(WI.AuditTestContentView.prototype.showNoResultDataPlaceholder):
(WI.AuditTestContentView.prototype.showFilteredPlaceholder):
(WI.AuditTestContentView.prototype._updateExportNavigationItems): Added.
(WI.AuditTestContentView.prototype._updateSupportsInputState): Added.
(WI.AuditTestContentView.prototype._createSetupEditor): Added.
(WI.AuditTestContentView.prototype._handleEditorKeydown): Added.
(WI.AuditTestContentView.prototype._handleExportTestButtonNavigationItemClicked): Added.
(WI.AuditTestContentView.prototype._handleExportResultButtonNavigationItemClicked): Added.
(WI.AuditTestContentView.prototype._handleTestDisabledChanged): Added.
(WI.AuditTestContentView.prototype._handleTestSupportedChanged): Added.
(WI.AuditTestContentView.prototype._handleEditingChanged): Added.
(WI.AuditTestContentView.prototype._updateExportButtonNavigationItemState): Deleted.
(WI.AuditTestContentView.prototype._handleExportButtonNavigationItemClicked): Deleted.
- UserInterface/Views/AuditTestContentView.css:
(.content-view.audit-test:is(.unsupported, .disabled):not(.manager-editing)): Added.
(.content-view.audit-test.manager-editing .editor:not(:empty)): Added.
(.content-view.audit-test.manager-editing :is(.content-view.audit-test, header) .editor:not(:empty)): Added.
(.content-view.audit-test .CodeMirror): Added.
(.content-view.audit-test > header :is(.name, .description):not([contenteditable])): Added.
(.content-view.audit-test.manager-editing > header :is(.name, .description)[contenteditable]): Added.
(.content-view.audit-test.manager-editing > header .name[contenteditable]:empty): Added.
(.content-view.audit-test.manager-editing > header .name[contenteditable]:empty:before): Added.
(.content-view.audit-test.manager-editing > header .description[contenteditable]:empty:before): Added.
(.content-view.audit-test:not(.manager-editing) > header .description:empty): Added.
(.content-view.audit-test:not(.manager-editing) > header .description:empty, .content-view.audit-test:not(.manager-editing) > header table.controls): Added.
(.content-view.audit-test > header table.controls, .content-view.audit-test > header table.controls > tr > td): Added.
(.content-view.audit-test > header table.controls > tr > th): Added.
(.content-view.audit-test > header table.controls > tr.supports input[type="number"]): Added.
(.content-view.audit-test > header table.controls > tr.supports .warning): Added.
(.content-view.audit-test > header table.controls > tr.supports .warning:not(:empty)::before ): Added.
(.content-view.audit-test > header table.controls > tr.setup .editor): Added.
(.content-view.audit-test > section > .message-text-view > :is(progress, .indeterminate-progress-spinner)): Added.
(@media (prefers-color-scheme: dark) .content-view.audit-test > header table.controls > tr > th): Added.
Create helper functions for subclasses that simplify creating the editing UI. When in edit
mode, addcontenteditable
to thename
/description
and inputs forsupports
/setup
.
- UserInterface/Views/AuditTestGroupContentView.js:
(WI.AuditTestGroupContentView):
(WI.AuditTestGroupContentView.prototype.willDismissPopover): Added.
(WI.AuditTestGroupContentView.prototype.createControlsTableElement): Added.
(WI.AuditTestGroupContentView.prototype.initialLayout):
(WI.AuditTestGroupContentView.prototype.layout):
(WI.AuditTestGroupContentView.prototype.shown):
(WI.AuditTestGroupContentView.prototype.hidden):
(WI.AuditTestGroupContentView.prototype.showRunningPlaceholder):
(WI.AuditTestGroupContentView.prototype._updateClassList): Added.
(WI.AuditTestGroupContentView.prototype._updateLevelScopeBar):
(WI.AuditTestGroupContentView.prototype._addTest): Added.
(WI.AuditTestGroupContentView.prototype._handleTestGroupTestAdded): Added.
(WI.AuditTestGroupContentView.prototype._handleTestGroupTestRemoved): Added.
- UserInterface/Views/AuditTestGroupContentView.css:
(.content-view.audit-test-group > section > .audit-test-group > header): Added.
(.content-view.audit-test-group.contains-test-case > header):
(.content-view.audit-test-group > section > .audit-test-group.contains-test-case > header): Added.
(.content-view.audit-test-group.contains-test-case + .audit-test-group.contains-test-case, .content-view.audit-test-group + .content-view.audit-test-case): Added.
(.content-view.audit-test-group.contains-test-case + .audit-test-group.contains-test-case): Deleted.
When in edit mode, add buttons for removing the audit and adding a new sub-audit (using the
newWI.CreateAuditPopover
popover).
- UserInterface/Views/AuditTestCaseContentView.js:
(WI.AuditTestCaseContentView):
(WI.AuditTestCaseContentView.prototype.initialLayout):
(WI.AuditTestCaseContentView.prototype.layout):
(WI.AuditTestCaseContentView.prototype.showRunningPlaceholder):
- UserInterface/Views/AuditTestCaseContentView.css:
(.content-view-container > .content-view.audit-test-case): Added.
(.content-view-container > .content-view.audit-test-case > header):
(.content-view-container > .content-view.audit-test-case.manager-editing > header h1 > img): Added.
(.content-view-container > .content-view.audit-test-case > section > :not(.message-text-view, .editor):first-child): Added.
(.content-view-container > .content-view.audit-test-case > section): Added.
(.content-view-container > .content-view.audit-test-case > section, .content-view-container > .content-view.audit-test-case > section :is(.editor, .CodeMirror)): Added.
(.content-view.audit-test-case.manager-editing.disabled:not(.editable) > header h1 > img): Added.
(.content-view.audit-test-case > section > :not(.message-text-view, .editor)): Added.
(.content-view.audit-test-case > section > :not(.message-text-view, .editor):last-child): Added.
(.content-view.audit-test-case > section > :not(.message-text-view, .editor) + :not(.message-text-view, .editor)): Added.
(@media (prefers-color-scheme: dark) .content-view.audit-test-case.manager-editing > header h1 > img): Added.
(.content-view-container > .content-view.audit-test-case > section > :not(.message-text-view):first-child): Deleted.
(.content-view.audit-test-case > section > :not(.message-text-view)): Deleted.
(.content-view.audit-test-case > section > :not(.message-text-view):last-child): Deleted.
(.content-view.audit-test-case > section > :not(.message-text-view) + :not(.message-text-view)): Deleted.
(.content-view.audit-test-case > section .CodeMirror): Deleted.
When in edit mode, replace the icon with a (X) to remove the audit and show aCodeMirror
instance to allow editing the content.
- UserInterface/Views/AuditTreeElement.js:
(WI.AuditTreeElement):
(WI.AuditTreeElement.expandedSettingKey): Added.
(WI.AuditTreeElement.prototype.onattach):
(WI.AuditTreeElement.prototype.ondelete):
(WI.AuditTreeElement.prototype.canSelectOnMouseDown): Added.
(WI.AuditTreeElement.prototype.populateContextMenu):
(WI.AuditTreeElement.prototype._handleTestNameChanged): Added.
(WI.AuditTreeElement.prototype._handleTestSupportedChanged): Added.
(WI.AuditTreeElement.prototype._handleTestGroupTestAdded): Added.
- UserInterface/Views/AuditTreeElement.css:
(.tree-outline .item.audit:matches(.test-case, .test-group):not(.unsupported, .manager-active):hover > .status > img): Added.
(body:not(.window-inactive, .window-docked-inactive) .tree-outline:focus-within .item.audit:matches(.test-case, .test-group):not(.unsupported, .manager-active).selected:hover > .status > img, body:not(.window-inactive, .window-docked-inactive) .tree-outline:focus-within .item.audit.test-case.selected > .status > .indeterminate-progress-spinner, body:not(.window-inactive, .window-docked-inactive) .tree-outline:focus-within .item.audit.test-group.selected > .status > progress): Added.
(.tree-outline .item.audit:not(:hover) > .status > img.show-on-hover, .tree-outline .item.audit.test-group.expanded:not(.unsupported, .editing-audits):not(:hover) > .status): Added.
(.tree-outline .item.audit.manager-active > .status > img.show-on-hover, .tree-outline .item.audit.manager-active > .status > img.show-on-hover, .tree-outline .item.audit.test-group.expanded:not(.editing-audits):hover > .status > :not(img), .tree-outline .item.audit.test-group-result.expanded > .status, .tree-outline .item.audit.unsupported + .children .item.audit.unsupported > .status > img): Added.
(@media (prefers-color-scheme: dark) .tree-outline .item.audit:matches(.test-case, .test-group):not(.unsupported, .manager-active):hover > .status > img): Added.
(.tree-outline .item.audit:matches(.test-case, .test-group):not(.unsupported, .manager-active) > .status:hover > img): Deleted.
(.tree-outline .item.audit > .status:not(:hover) > img.show-on-hover, .tree-outline .item.audit.test-group.expanded:not(.unsupported, .editing-audits) > .status:not(:hover)): Deleted.
(.tree-outline .item.audit.manager-active > .status > img.show-on-hover, .tree-outline .item.audit.test-group.expanded:not(.editing-audits) > .status:hover > :not(img), .tree-outline .item.audit.test-group-result.expanded > .status, .tree-outline .item.audit.unsupported + .children .item.audit.unsupported > .status > img): Deleted.
Add context menu items for duplicating/deleting tests when in edit mode. Adjust the label
and disabled state of existing context menu items for clarity.
- UserInterface/Views/CreateAuditPopover.js: Added.
(WI.CreateAuditPopover):
(WI.CreateAuditPopover.prototype.get audit):
(WI.CreateAuditPopover.prototype.show):
(WI.CreateAuditPopover.prototype.dismiss.const.testFunction):
(WI.CreateAuditPopover.prototype.dismiss):
(WI.CreateAuditPopover.prototype._presentOverTargetElement):
- UserInterface/Views/CreateAuditPopover.css: Added.
(.popover .create-audit-content):
(.popover .create-audit-content > .editor-wrapper):
(.popover .create-audit-content > .editor-wrapper > .reference-page-link):
New popover for creating an audit:
[<select> of group or test case] [<input> for name] (?)
- UserInterface/Views/Main.css:
(.navigation-item-help > .navigation-bar):
(.message-text-view > .navigation-item-help + .navigation-item-help): Added.
Add styles for when multiple navigation help items are used in the same message text view.
- UserInterface/Views/Variables.css:
(:root):
(@media (prefers-color-scheme: dark) :root):
Add--filter-invert
to light mode too.
- UserInterface/Base/Utilities.js:
(HTMLInputElement.prototype.autosize):
- UserInterface/Views/CanvasOverviewContentView.js:
(WI.CanvasOverviewContentView):
(WI.CanvasOverviewContentView.prototype._updateRecordingAutoCaptureInputElementSize):
(WI.CanvasOverviewContentView.get recordingAutoCaptureInputMargin): Deleted.
- UserInterface/Views/CanvasOverviewContentView.css:
(.navigation-bar > .item.canvas-recording-auto-capture > label > input):
(.navigation-bar > .item.canvas-recording-auto-capture > label > input::-webkit-inner-spin-button): Deleted.
Create a helper function for autosizing an<input>
.
- UserInterface/Views/AuditTabContentView.js:
(WI.AuditTabContentView):
(WI.AuditTabContentView.prototype.initialLayout):
Remove the back/foward arrows as they can get into an inconsistent state when editing.
Drive-by: update the drop zone text for clarity.
- UserInterface/Views/SearchSidebarPanel.js:
(WI.SearchSidebarPanel.prototype.showDefaultContentView):
Drive-by: add period to help text.
- UserInterface/Main.html:
- Localizations/en.lproj/localizedStrings.js:
LayoutTests:
- inspector/model/auditTestGroup.html:
- inspector/model/auditTestGroup-expected.txt:
Add tests that check how
disabled
andsupported
(viasupports
) is propagated in groups.
- inspector/audit/run.html:
- inspector/audit/run-expected.txt:
Add tests for
WebInspectorAudit
beforeAudit.setup
is called.
- inspector/audit/manager-start-setup.html:
Renaming function call.
- inspector/audit/basic-async.html:
- inspector/audit/basic-boolean.html:
- inspector/audit/basic-debugger.html:
- inspector/audit/basic-error.html:
- inspector/audit/basic-object.html:
- inspector/audit/basic-promise.html:
- inspector/audit/basic-string.html:
- inspector/audit/basic-timeout.html:
- inspector/audit/run-resources.html:
Drive-by: remove
InspectorTest.debug()
.
- 6:36 PM Changeset in webkit [266316] by
-
- 2 edits in branches/safari-610.1.28.1-branch/LayoutTests/platform
Remove WebAudio Failure entries from TestExpectations
<rdar://problem/67367072>
Unreviewed, rebaselined after r266315.
- 6:31 PM Changeset in webkit [266315] by
-
- 24 edits in branches/safari-610.1.28.1-branch/LayoutTests/imported/w3c/web-platform-tests/webaudio
Rebaseline WebAudio test results
<rdar://problem/67367072>
- r265824 changed what error message is. (This is unavoidable since the kind of thrown error is changed (JS side error v.s. JS error thrown from C++), and this needs to be done to fix the reported webpage’s issue).
- This error is recorded as expect files in various WebAudio WPT test if the tested WebAudio constructors are not implemented.
- After that branch cut, a lot of WebAudio constructors are newly implemented, which removes this error messages from expect files.
- r265824 is landed after these WebAudio patches. So it does not include rebaseline for these WebAudio WPT tests (since it is no longer including that error messages).
- But in that branch, these WebAudio patches are not cherry-picked. This means that error message is still recorded in WPT expect files.
- So, we just need to update these expect files, or cherry-pick these WebAudio patches. And I think updating expect files is the way to go.
- 5:27 PM Changeset in webkit [266314] by
-
- 4 edits in trunk
Sites built with flambe.js don't work with trackpad on iPad
https://bugs.webkit.org/show_bug.cgi?id=215954
Reviewed by Wenson Hsieh.
Source/WebKit:
New API Test: iOSMouseSupport.MouseTimestampTimebase
- UIProcess/ios/WKMouseGestureRecognizer.mm:
(-[WKMouseGestureRecognizer createMouseEventWithType:]):
Use the timestamp property on UITouch instead of GSCurrentEventTimestamp().
This property is in the same units as mach_absolute_time/MonotonicTime.
Previously, treating a monotonic time as wall time resulted in it getting clamped to 0,
breaking sites that depend on it.
Tools:
- TestWebKitAPI/Tests/WebKitCocoa/iOSMouseSupport.mm:
(-[WKTestingHoverEvent locationInView:]):
(-[WKTestingTouch timestamp]):
(TEST):
- 5:21 PM Changeset in webkit [266313] by
-
- 2 edits in trunk/Source/WebCore
Debug event regions should paint the interactive region
https://bugs.webkit.org/show_bug.cgi?id=215955
Reviewed by Tim Horton.
Interactive regions (EventRegion's region()) were not ever shown on macOS, even though
we use them for async-scrolling hit-testing. Fix RenderLayerBacking::paintDebugOverlays()
to always paint them if wheel event handler, or touch-action regions are enabled.
- rendering/RenderLayerBacking.cpp:
(WebCore::RenderLayerBacking::paintDebugOverlays):
- 4:21 PM Changeset in webkit [266312] by
-
- 3 edits in trunk/Source/WebCore
[macOS] AirPlay device name is wrong when playing to multiple devices
https://bugs.webkit.org/show_bug.cgi?id=215952
<rdar://problem/66930799>
Reviewed by Jer Noble.
Tested manually because this requires a specific hardware setup.
- platform/graphics/avfoundation/MediaPlaybackTargetCocoa.mm:
(WebCore::MediaPlaybackTargetCocoa::deviceName const): If the AVOutputContext
supports multiple output devices, create the device name by concatenating the names
of all of the active output devices.
- 4:04 PM Changeset in webkit [266311] by
-
- 333 edits in trunk
[WebIDL] 'void' type is changing to 'undefined'
https://bugs.webkit.org/show_bug.cgi?id=215514
<rdar://problem/67566201>
Reviewed by Darin Adler.
Update IDLs and IDL parsing / code generation for the switch from 'void' to 'undefined'.
Source/WebCore:
Also updates the parser to better match current WebIDL grammar, removing the special
'ReturnType' construction and just using 'Type' instead.
- Modules/applepay/ApplePaySession.idl:
- Modules/async-clipboard/Clipboard.idl:
- Modules/cache/DOMCache.idl:
- Modules/credentialmanagement/CredentialsContainer.idl:
- Modules/encryptedmedia/MediaKeySession.h:
- Modules/encryptedmedia/MediaKeySession.idl:
- Modules/encryptedmedia/legacy/WebKitMediaKeySession.idl:
- Modules/entriesapi/ErrorCallback.idl:
- Modules/entriesapi/FileCallback.idl:
- Modules/entriesapi/FileSystemDirectoryEntry.idl:
- Modules/entriesapi/FileSystemDirectoryReader.idl:
- Modules/entriesapi/FileSystemEntriesCallback.idl:
- Modules/entriesapi/FileSystemEntry.idl:
- Modules/entriesapi/FileSystemEntryCallback.idl:
- Modules/entriesapi/FileSystemFileEntry.idl:
- Modules/fetch/FetchHeaders.idl:
- Modules/geolocation/Geolocation.idl:
- Modules/geolocation/PositionCallback.idl:
- Modules/geolocation/PositionErrorCallback.idl:
- Modules/indexeddb/IDBCursor.idl:
- Modules/indexeddb/IDBDatabase.idl:
- Modules/indexeddb/IDBObjectStore.idl:
- Modules/indexeddb/IDBTransaction.idl:
- Modules/mediacontrols/MediaControlsHost.idl:
- Modules/mediarecorder/MediaRecorder.idl:
- Modules/mediasession/MediaSession.idl:
- Modules/mediasource/MediaSource.idl:
- Modules/mediasource/SourceBuffer.idl:
- Modules/mediastream/CanvasCaptureMediaStreamTrack.idl:
- Modules/mediastream/MediaStream.idl:
- Modules/mediastream/MediaStreamTrack.idl:
- Modules/mediastream/NavigatorMediaDevices.idl:
- Modules/mediastream/RTCDTMFSender.idl:
- Modules/mediastream/RTCDataChannel.idl:
- Modules/mediastream/RTCPeerConnection.idl:
- Modules/mediastream/RTCRtpSender.idl:
- Modules/mediastream/RTCRtpTransceiver.idl:
- Modules/notifications/Notification.idl:
- Modules/notifications/NotificationPermissionCallback.idl:
- Modules/paymentrequest/MerchantValidationEvent.idl:
- Modules/paymentrequest/PaymentRequest.idl:
- Modules/paymentrequest/PaymentRequestUpdateEvent.idl:
- Modules/paymentrequest/PaymentResponse.idl:
- Modules/pictureinpicture/DocumentPictureInPicture.idl:
- Modules/plugins/QuickTimePluginReplacement.idl:
- Modules/quota/StorageErrorCallback.idl:
- Modules/quota/StorageInfo.idl:
- Modules/quota/StorageQuota.idl:
- Modules/quota/StorageQuotaCallback.idl:
- Modules/quota/StorageUsageCallback.idl:
- Modules/remoteplayback/RemotePlayback.idl:
- Modules/speech/SpeechSynthesis.idl:
- Modules/streams/ReadableByteStreamController.idl:
- Modules/streams/ReadableStreamBYOBReader.idl:
- Modules/streams/ReadableStreamBYOBRequest.idl:
- Modules/streams/ReadableStreamDefaultController.idl:
- Modules/streams/ReadableStreamDefaultReader.idl:
- Modules/streams/ReadableStreamSink.idl:
- Modules/streams/ReadableStreamSource.idl:
- Modules/streams/TransformStreamDefaultController.idl:
- Modules/streams/WritableStreamDefaultController.idl:
- Modules/streams/WritableStreamDefaultWriter.idl:
- Modules/webaudio/AnalyserNode.idl:
- Modules/webaudio/AudioBuffer.idl:
- Modules/webaudio/AudioBufferCallback.idl:
- Modules/webaudio/AudioBufferSourceNode.idl:
- Modules/webaudio/AudioContext.idl:
- Modules/webaudio/AudioListener.idl:
- Modules/webaudio/AudioNode.idl:
- Modules/webaudio/AudioScheduledSourceNode.idl:
- Modules/webaudio/BiquadFilterNode.idl:
- Modules/webaudio/IIRFilterNode.idl:
- Modules/webaudio/OfflineAudioContext.idl:
- Modules/webaudio/OscillatorNode.idl:
- Modules/webaudio/PannerNode.idl:
- Modules/webaudio/WebKitAudioBufferSourceNode.idl:
- Modules/webaudio/WebKitAudioContext.idl:
- Modules/webaudio/WebKitAudioListener.idl:
- Modules/webaudio/WebKitAudioPannerNode.idl:
- Modules/webaudio/WebKitOscillatorNode.idl:
- Modules/webdatabase/Database.idl:
- Modules/webdatabase/DatabaseCallback.idl:
- Modules/webdatabase/SQLStatementCallback.idl:
- Modules/webdatabase/SQLTransaction.idl:
- Modules/webdatabase/SQLTransactionCallback.idl:
- Modules/webdatabase/SQLTransactionErrorCallback.idl:
- Modules/webgpu/WebGPUBuffer.idl:
- Modules/webgpu/WebGPUCommandEncoder.idl:
- Modules/webgpu/WebGPUComputePassEncoder.idl:
- Modules/webgpu/WebGPUDeviceErrorScopes.idl:
- Modules/webgpu/WebGPUProgrammablePassEncoder.idl:
- Modules/webgpu/WebGPUQueue.idl:
- Modules/webgpu/WebGPURenderPassEncoder.idl:
- Modules/webgpu/WebGPUTexture.idl:
- Modules/websockets/WebSocket.idl:
- Modules/webxr/WebXRSession.idl:
- Modules/webxr/WebXRSystem.idl:
- Modules/webxr/XRFrameRequestCallback.idl:
- Scripts/SettingsTemplates/InternalSettingsGenerated.idl.erb:
- animation/AnimationEffect.idl:
- animation/KeyframeEffect.idl:
- animation/WebAnimation.idl:
- bindings/IDLTypes.h:
- bindings/js/DOMPromiseProxy.h:
- bindings/scripts/CodeGenerator.pm:
- bindings/scripts/CodeGeneratorJS.pm:
(NeedsExplicitPropagateExceptionCall):
(GenerateCallbackImplementationContent):
(GenerateImplementationFunctionCall):
(GetBaseIDLType):
- bindings/scripts/IDLParser.pm:
(parseCallbackRest):
(parseAttributeOrOperationForStringifierOrStatic):
(parseOperation):
(parseSpecialOperation):
(parseMapLikeProperties):
(parseSetLikeProperties):
(parseSingleType):
(parseUnionMemberType):
(parseDistinguishableType):
(parseConstType):
(parsePrimitiveType):
(parseNonAnyType): Deleted.
(parseReturnType): Deleted.
- bindings/scripts/test/JS/JSTestCallbackFunctionWithThisObject.cpp:
(WebCore::JSTestCallbackFunctionWithThisObject::handleEvent):
- bindings/scripts/test/JS/JSTestCallbackFunctionWithThisObject.h:
- bindings/scripts/test/JS/JSTestCallbackFunctionWithTypedefs.cpp:
(WebCore::JSTestCallbackFunctionWithTypedefs::handleEvent):
- bindings/scripts/test/JS/JSTestCallbackFunctionWithTypedefs.h:
- bindings/scripts/test/JS/JSTestCallbackInterface.cpp:
(WebCore::JSTestCallbackInterface::callbackWithNoParam):
(WebCore::JSTestCallbackInterface::callbackWithArrayParam):
(WebCore::JSTestCallbackInterface::callbackWithSerializedScriptValueParam):
(WebCore::JSTestCallbackInterface::callbackWithStringList):
(WebCore::JSTestCallbackInterface::callbackWithBoolean):
(WebCore::JSTestCallbackInterface::callbackRequiresThisToPass):
- bindings/scripts/test/JS/JSTestCallbackInterface.h:
- bindings/scripts/test/JS/JSTestIterable.cpp:
- bindings/scripts/test/JS/JSTestNode.cpp:
- bindings/scripts/test/JS/JSTestObj.cpp:
(WebCore::JSTestObjConstructor::construct):
(WebCore::JSTestObjPrototype::finishCreation):
(WebCore::jsTestObjTestReadOnlyVoidPromiseAttributeGetter):
(WebCore::jsTestObjPrototypeFunctionUndefinedMethodBody):
(WebCore::jsTestObjPrototypeFunctionUndefinedMethod):
(WebCore::jsTestObjPrototypeFunctionUndefinedMethodWithArgsBody):
(WebCore::jsTestObjPrototypeFunctionUndefinedMethodWithArgs):
(WebCore::jsTestObjPrototypeFunctionMethodWithOptionalPromiseBody):
(WebCore::jsTestObjPrototypeFunctionTestReturnsOwnPromiseAndPromiseProxyFunctionBody):
(WebCore::jsTestObjPrototypeFunctionVoidMethodBody): Deleted.
(WebCore::jsTestObjPrototypeFunctionVoidMethod): Deleted.
(WebCore::jsTestObjPrototypeFunctionVoidMethodWithArgsBody): Deleted.
(WebCore::jsTestObjPrototypeFunctionVoidMethodWithArgs): Deleted.
- bindings/scripts/test/JS/JSTestVoidCallbackFunction.cpp:
(WebCore::JSTestVoidCallbackFunction::handleEvent):
- bindings/scripts/test/JS/JSTestVoidCallbackFunction.h:
- bindings/scripts/test/TestCEReactions.idl:
- bindings/scripts/test/TestCallTracer.idl:
- bindings/scripts/test/TestCallbackFunctionWithThisObject.idl:
- bindings/scripts/test/TestCallbackFunctionWithTypedefs.idl:
- bindings/scripts/test/TestCallbackInterface.idl:
- bindings/scripts/test/TestDomainSecurity.idl:
- bindings/scripts/test/TestEnabledBySetting.idl:
- bindings/scripts/test/TestGlobalObject.idl:
- bindings/scripts/test/TestImplements.idl:
- bindings/scripts/test/TestIndexedSetterNoIdentifier.idl:
- bindings/scripts/test/TestIndexedSetterThrowingException.idl:
- bindings/scripts/test/TestIndexedSetterWithIdentifier.idl:
- bindings/scripts/test/TestJSBuiltinConstructor.idl:
- bindings/scripts/test/TestNamedAndIndexedSetterNoIdentifier.idl:
- bindings/scripts/test/TestNamedAndIndexedSetterThrowingException.idl:
- bindings/scripts/test/TestNamedAndIndexedSetterWithIdentifier.idl:
- bindings/scripts/test/TestNamedDeleterNoIdentifier.idl:
- bindings/scripts/test/TestNamedDeleterThrowingException.idl:
- bindings/scripts/test/TestNamedDeleterWithIdentifier.idl:
- bindings/scripts/test/TestNamedDeleterWithIndexedGetter.idl:
- bindings/scripts/test/TestNamedSetterNoIdentifier.idl:
- bindings/scripts/test/TestNamedSetterThrowingException.idl:
- bindings/scripts/test/TestNamedSetterWithIdentifier.idl:
- bindings/scripts/test/TestNamedSetterWithIndexedGetter.idl:
- bindings/scripts/test/TestNamedSetterWithIndexedGetterAndSetter.idl:
- bindings/scripts/test/TestNamedSetterWithOverrideBuiltins.idl:
- bindings/scripts/test/TestNamedSetterWithUnforgableProperties.idl:
- bindings/scripts/test/TestNamedSetterWithUnforgablePropertiesAndOverrideBuiltins.idl:
- bindings/scripts/test/TestNode.idl:
- bindings/scripts/test/TestObj.idl:
- bindings/scripts/test/TestOperationBase.idl:
- bindings/scripts/test/TestSerializedScriptValueInterface.idl:
- bindings/scripts/test/TestSupplemental.idl:
- bindings/scripts/test/TestTypedefs.idl:
- bindings/scripts/test/TestVoidCallbackFunction.idl:
- css/CSSKeyframesRule.idl:
- css/CSSMediaRule.idl:
- css/CSSPaintCallback.idl:
- css/CSSStyleDeclaration.idl:
- css/CSSStyleSheet.idl:
- css/CSSSupportsRule.idl:
- css/DOMCSSRegisterCustomProperty.idl:
- css/DeprecatedCSSOMPrimitiveValue.idl:
- css/FontFaceSet.idl:
- css/MediaList.idl:
- css/MediaQueryList.idl:
- css/WebKitCSSMatrix.idl:
- dom/AbortAlgorithm.idl:
- dom/AbortController.idl:
- dom/AbortSignal.idl:
- dom/CharacterData.idl:
- dom/ChildNode.idl:
- dom/CompositionEvent.idl:
- dom/CustomElementRegistry.idl:
- dom/CustomEvent.idl:
- dom/DOMStringMap.idl:
- dom/DataTransfer.idl:
- dom/DataTransferItem.idl:
- dom/DataTransferItemList.idl:
- dom/DeviceMotionEvent.idl:
- dom/DeviceOrientationEvent.idl:
- dom/Document.idl:
- dom/DocumentFullscreen.idl:
- dom/DocumentStorageAccess.idl:
- dom/Element.idl:
- dom/Event.idl:
- dom/EventListener.idl:
- dom/EventTarget.idl:
- dom/HashChangeEvent.idl:
- dom/IdleRequestCallback.idl:
- dom/KeyboardEvent.idl:
- dom/MessageEvent.idl:
- dom/MessagePort.idl:
- dom/MouseEvent.idl:
- dom/MutationCallback.idl:
- dom/MutationEvent.idl:
- dom/MutationObserver.idl:
- dom/Node.idl:
- dom/NodeIterator.idl:
- dom/ParentNode.idl:
- dom/Range.idl:
- dom/RequestAnimationFrameCallback.idl:
- dom/StringCallback.idl:
- dom/TextEvent.idl:
- dom/TouchEvent.idl:
- dom/UIEvent.idl:
- dom/WheelEvent.idl:
- fileapi/BlobCallback.idl:
- fileapi/FileReader.idl:
- html/DOMFormData.idl:
- html/DOMTokenList.idl:
- html/DOMURL.idl:
- html/HTMLButtonElement.idl:
- html/HTMLCanvasElement.idl:
- html/HTMLDialogElement.idl:
- html/HTMLElement.idl:
- html/HTMLFieldSetElement.idl:
- html/HTMLFormElement.idl:
- html/HTMLImageElement.idl:
- html/HTMLInputElement.idl:
- html/HTMLKeygenElement.idl:
- html/HTMLMarqueeElement.idl:
- html/HTMLMediaElement.idl:
- html/HTMLObjectElement.idl:
- html/HTMLOptionsCollection.idl:
- html/HTMLOrForeignElement.idl:
- html/HTMLOutputElement.idl:
- html/HTMLSelectElement.idl:
- html/HTMLTableElement.idl:
- html/HTMLTableRowElement.idl:
- html/HTMLTableSectionElement.idl:
- html/HTMLTextAreaElement.idl:
- html/HTMLVideoElement.idl:
- html/ImageBitmap.idl:
- html/MediaController.idl:
- html/URLSearchParams.idl:
- html/VoidCallback.idl:
- html/canvas/ANGLEInstancedArrays.idl:
- html/canvas/CanvasDrawImage.idl:
- html/canvas/CanvasDrawPath.idl:
- html/canvas/CanvasGradient.idl:
- html/canvas/CanvasImageData.idl:
- html/canvas/CanvasPath.idl:
- html/canvas/CanvasPathDrawingStyles.idl:
- html/canvas/CanvasPattern.idl:
- html/canvas/CanvasRect.idl:
- html/canvas/CanvasRenderingContext2D.idl:
- html/canvas/CanvasState.idl:
- html/canvas/CanvasText.idl:
- html/canvas/CanvasTransform.idl:
- html/canvas/CanvasUserInterface.idl:
- html/canvas/ImageBitmapRenderingContext.idl:
- html/canvas/OESVertexArrayObject.idl:
- html/canvas/OffscreenCanvasRenderingContext2D.idl:
- html/canvas/Path2D.idl:
- html/canvas/WebGL2RenderingContext.idl:
- html/canvas/WebGLDrawBuffers.idl:
- html/canvas/WebGLLoseContext.idl:
- html/canvas/WebGLRenderingContextBase.idl:
- html/track/TextTrack.idl:
- inspector/CommandLineAPIHost.idl:
- inspector/InspectorFrontendHost.idl:
- loader/appcache/DOMApplicationCache.idl:
- page/DOMSelection.idl:
- page/DOMWindow.idl:
- page/EventSource.idl:
- page/History.idl:
- page/IntersectionObserver.idl:
- page/IntersectionObserverCallback.idl:
- page/Location.idl:
- page/Navigator.idl:
- page/NavigatorIsLoggedIn.idl:
- page/NavigatorShare.idl:
- page/Performance.idl:
- page/PerformanceObserver.idl:
- page/PerformanceObserverCallback.idl:
- page/RemoteDOMWindow.idl:
- page/ResizeObserver.idl:
- page/ResizeObserverCallback.idl:
- page/UndoManager.idl:
- page/WindowOrWorkerGlobalScope.idl:
- plugins/DOMPluginArray.idl:
- storage/Storage.idl:
- storage/StorageEvent.idl:
- svg/SVGAngle.idl:
- svg/SVGAnimationElement.idl:
- svg/SVGFEDropShadowElement.idl:
- svg/SVGFEGaussianBlurElement.idl:
- svg/SVGFEMorphologyElement.idl:
- svg/SVGLength.idl:
- svg/SVGLengthList.idl:
- svg/SVGMarkerElement.idl:
- svg/SVGNumberList.idl:
- svg/SVGPathSegList.idl:
- svg/SVGPointList.idl:
- svg/SVGSVGElement.idl:
- svg/SVGStringList.idl:
- svg/SVGTextContentElement.idl:
- svg/SVGTransform.idl:
- svg/SVGTransformList.idl:
- testing/InternalSettings.idl:
- testing/Internals.idl:
- testing/MockCDMFactory.idl:
- testing/MockPageOverlay.idl:
- testing/MockPaymentCoordinator.idl:
- testing/ServiceWorkerInternals.idl:
- testing/TypeConversions.idl:
- testing/WebFakeXRDevice.idl:
- testing/WebFakeXRInputController.idl:
- testing/WebXRTest.idl:
- testing/XRSimulateUserActivationFunction.idl:
- workers/DedicatedWorkerGlobalScope.idl:
- workers/Worker.idl:
- workers/WorkerGlobalScope.idl:
- workers/service/ExtendableEvent.idl:
- workers/service/FetchEvent.idl:
- workers/service/ServiceWorker.idl:
- workers/service/ServiceWorkerClient.idl:
- workers/service/ServiceWorkerClients.idl:
- workers/service/ServiceWorkerContainer.idl:
- workers/service/ServiceWorkerGlobalScope.idl:
- workers/service/ServiceWorkerRegistration.idl:
- worklets/PaintWorkletGlobalScope.idl:
- worklets/Worklet.idl:
- xml/XMLHttpRequest.idl:
- xml/XSLTProcessor.idl:
Tools:
- DumpRenderTree/Bindings/CodeGeneratorDumpRenderTree.pm:
(_generateImplementationFile):
(_returnExpression):
- TestRunnerShared/UIScriptContext/Bindings/UIScriptController.idl:
- WebKitTestRunner/InjectedBundle/Bindings/AccessibilityController.idl:
- WebKitTestRunner/InjectedBundle/Bindings/AccessibilityUIElement.idl:
- WebKitTestRunner/InjectedBundle/Bindings/CodeGeneratorTestRunner.pm:
(_generateImplementationFile):
(_returnExpression):
- WebKitTestRunner/InjectedBundle/Bindings/EventSendingController.idl:
- WebKitTestRunner/InjectedBundle/Bindings/GCController.idl:
- WebKitTestRunner/InjectedBundle/Bindings/TestRunner.idl:
- WebKitTestRunner/InjectedBundle/Bindings/TextInputController.idl:
- 4:03 PM Changeset in webkit [266310] by
-
- 1 copy in tags/Safari-609.4.1
Tag Safari-609.4.1.
- 4:02 PM Changeset in webkit [266309] by
-
- 1 copy in tags/Safari-610.1.28.1.5
Tag Safari-610.1.28.1.5.
- 3:58 PM Changeset in webkit [266308] by
-
- 8 edits in branches/safari-610.1.28.1-branch/Source
Revert "Versioning."
This reverts revision 266299.
- 3:58 PM Changeset in webkit [266307] by
-
- 16 edits2 adds in branches/safari-610.1.28.1-branch
Cherry-pick r266292. rdar://problem/67963517
Vertical scrolling gets stuck when a horizontal scroller is under the mouse (google search results)
https://bugs.webkit.org/show_bug.cgi?id=215641
<rdar://problem/67430532>
Reviewed by Tim Horton.
Source/WebCore:
There are two parts to this fix. First, findEnclosingScrollableContainer() needs
to use the same vertical-biasing delta fixup that we use in other places, to bias towards
vertical scrolling.
Second, when we've determined that the main frame should perform the scroll and dispatch
the wheel event to the scrolling thread, we used to hit-test from scratch on the scrolling
thread and and try to send the event to a scroller which we already know should not handle
it. So pass along a target ScrollingNodeID, and start the scrolling thread handling from
that node.
Test: fast/scrolling/mac/horizontal-overflow-trapping-small-deltas.html
- page/FrameView.cpp: (WebCore::FrameView::wheelEvent):
- page/mac/EventHandlerMac.mm: (WebCore::findEnclosingScrollableContainer):
- page/scrolling/ScrollingCoordinator.h: (WebCore::ScrollingCoordinator::handleWheelEvent):
- page/scrolling/ScrollingTree.cpp: (WebCore::ScrollingTree::handleWheelEvent): (WebCore::ScrollingTree::handleWheelEventWithNode):
- page/scrolling/ScrollingTree.h:
- page/scrolling/ThreadedScrollingTree.cpp: (WebCore::ThreadedScrollingTree::handleWheelEventAfterMainThread):
- page/scrolling/ThreadedScrollingTree.h:
- page/scrolling/mac/ScrollingCoordinatorMac.h:
- page/scrolling/mac/ScrollingCoordinatorMac.mm: (WebCore::ScrollingCoordinatorMac::handleWheelEvent):
- page/scrolling/nicosia/ScrollingCoordinatorNicosia.cpp: (WebCore::ScrollingCoordinatorNicosia::handleWheelEvent):
- page/scrolling/nicosia/ScrollingCoordinatorNicosia.h:
LayoutTests:
After this change iframe-latch-small-deltas.html would time-out because WheelEventTestMonitor
would get stuck with a "content scrolling" defer region, due to begin/end wheel
events not getting both sent to the iframe. But the test was actually broken; logging shows
that it reset latching state anyway, and it's not testing the shipping configuration of
async iframe scrolling. So fix the test to use async iframe scrolling, and to avoid
rubber-banding, so that latching is not cleared due to elapsed time.
- fast/scrolling/latching/iframe-latch-small-deltas-expected.txt:
- fast/scrolling/latching/iframe-latch-small-deltas.html:
- fast/scrolling/mac/horizontal-overflow-trapping-small-deltas-expected.txt: Added.
- fast/scrolling/mac/horizontal-overflow-trapping-small-deltas.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@266292 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- 3:58 PM Changeset in webkit [266306] by
-
- 4 edits2 adds in branches/safari-610.1.28.1-branch
Cherry-pick r266262. rdar://problem/67963578
Scrolling on select element doesn't work after scrolling the page
https://bugs.webkit.org/show_bug.cgi?id=215900
Source/WebCore:
<rdar://problem/67766032>
Reviewed by Tim Horton.
The scrolling thread could use a node that was latched some time ago; we need to
check the freshness of the latched node before using it.
Test: fast/scrolling/latching/latching-stuck-to-main-page.html
- page/scrolling/ScrollingTreeLatchingController.cpp: (WebCore::ScrollingTreeLatchingController::receivedWheelEvent): (WebCore::ScrollingTreeLatchingController::latchedNodeForEvent const): (WebCore::ScrollingTreeLatchingController::latchedNodeIsRelevant const):
- page/scrolling/ScrollingTreeLatchingController.h:
LayoutTests:
Reviewed by Tim Horton.
- fast/scrolling/latching/latching-stuck-to-main-page-expected.txt: Added.
- fast/scrolling/latching/latching-stuck-to-main-page.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@266262 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- 3:58 PM Changeset in webkit [266305] by
-
- 7 edits2 adds in branches/safari-610.1.28.1-branch
Cherry-pick r266248. rdar://problem/67963534
Occasional crashes when restoring replaced text under Editor::changeBackToReplacedString
https://bugs.webkit.org/show_bug.cgi?id=215892
<rdar://problem/67731156>
Reviewed by Tim Horton.
Source/WebCore:
While reverting the replaced string in
Editor::changeBackToReplacedString
, it's possible for text replacement
(Editor::replaceSelectionWithText
) to cause the text checking paragraph range to become orphaned (if for no
reason other than the fact that arbitrary script can run in between). If this happens, then our attempts to
expand the checking range to paragraph boundaries underneathTextCheckingParagraph::paragraphRange
will result
in a null dereference, since the boundary points computed underneath theexpandToParagraphBoundary
helper will
benullopt
.
Mitigate this (and any other potentially similar crashes) by making
expandToParagraphBoundary
robust in the
case where expanding to the start and end fails, and just fall back to returning the original text checking
range instead.
Test: editing/mac/input/change-back-to-replaced-string.html
- editing/Editor.h:
- editing/TextCheckingHelper.cpp: (WebCore::expandToParagraphBoundary):
See above for more details.
- testing/Internals.cpp: (WebCore::Internals::changeBackToReplacedString):
- testing/Internals.h:
- testing/Internals.idl:
Add a simple internal testing hook to change the currently selected text back to the given replaced string.
LayoutTests:
Add a layout test to exercise the crash.
- editing/mac/input/change-back-to-replaced-string-expected.txt: Added.
- editing/mac/input/change-back-to-replaced-string.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@266248 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- 3:57 PM Changeset in webkit [266304] by
-
- 5 edits2 adds in branches/safari-610.1.28.1-branch
Cherry-pick r266241. rdar://problem/67963565
REGRESSION (r263506): timing of CSS Animation on https://animate.style is incorrect
https://bugs.webkit.org/show_bug.cgi?id=215807
<rdar://problem/66770136>
Reviewed by Simon Fraser.
Source/WebCore:
Test: webanimations/accelerated-css-animation-with-easing.html
In r263506, we added a way for accelerated animations to adhere to both a timing function set on the
animation, affecting the timing of the entire animation, as well as a timing function set on individual
keyframes, affecting the timing of the animation in a given interval. Alas, this change broke handling
of timing functions with implicit animation-timing-function on keyframes.
That code change assumed that the Animation object ultimately passed to GraphicsLayerCA::setupAnimation()
would return the animation-wide timing function through Animation::timingFunction(). This was correct for
JS-originated animations, but not for CSS Animations, since that value was set based on the computed CSS
property animation-timing-function for the target element. However, that CSS property does not specify
the animation-wide timing function, but rather the default timing function to use for keyframes should
they fail to specify an explicit timing function.
To fix this, first, we ensure that the animation-wide timing function is set on the Animation object,
changing KeyframeEffect::backingAnimationForCompositedRenderer() to always generate an Animation object
based on the effect, divorcing itself from the Animation object created through CSS.
Then, we add a new member to Animation to specify the default timing function for keyframes, allowing
GraphicsLayerCA to use it to determine the timing function when building keyframes.
- animation/KeyframeEffect.cpp: (WebCore::KeyframeEffect::backingAnimationForCompositedRenderer const): Always generate an Animation object based on the effect's properties, also setting the new defaultTimingFunctionForKeyframes in the case of a CSS Animation object.
- platform/animation/Animation.h: (WebCore::Animation::defaultTimingFunctionForKeyframes const): (WebCore::Animation::setDefaultTimingFunctionForKeyframes):
- platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::animationHasStepsTimingFunction): Use the optional defaultTimingFunctionForKeyframes to determine whether a keyframe uses a steps timing function. (WebCore::GraphicsLayerCA::timingFunctionForAnimationValue): Use the optional defaultTimingFunctionForKeyframes to determine the timing function for a keyframe.
LayoutTests:
Add a new test that checks that a CSS Animation with implicit and explicit animation-timing-function
values behave the same when running accelerated.
- webanimations/accelerated-css-animation-with-easing-expected.html: Added.
- webanimations/accelerated-css-animation-with-easing.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@266241 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- 3:57 PM Changeset in webkit [266303] by
-
- 2 edits in branches/safari-610.1.28.1-branch/Source/WebKit
Cherry-pick r266240. rdar://problem/67963545
REGRESSION(r264710): Initializing the AVPlayer Obj-C class at process start up causes a regression in power-use tests
https://bugs.webkit.org/show_bug.cgi?id=215861
Reviewed by Eric Carlson.
Calling +instancesRespondToSelector: will cause the underyling Obj-C class to be initialized, which in the case of
AVPlayer does a bunch of work which shows up on power-use tests. Because the API being checked for will always exist
in macOS 11, and that code is already protected by a HAVE(AVPLAYER_VIDEORANGEOVERRIDE) pragma, just return early
here without running the +instancesRespondToSelector: check.
- WebProcess/cocoa/WebProcessCocoa.mm: (WebKit::WebProcess::updatePageScreenProperties):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@266240 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- 3:57 PM Changeset in webkit [266302] by
-
- 3 edits3 adds in branches/safari-610.1.28.1-branch
Cherry-pick r266239. rdar://problem/67963554
REGRESSION (r258215): Title preview movie isn't displayed at www.thismmalife.com
https://bugs.webkit.org/show_bug.cgi?id=215774
<rdar://problem/67707957>
Reviewed by Eric Carlson.
Source/WebCore:
Test: media/video-as-img-output-pts.html
Use CMSampleBufferGetOutputPresentationTimeStamp() for sample PTS.
- platform/graphics/avfoundation/objc/ImageDecoderAVFObjC.mm: (WebCore::ImageDecoderAVFObjC::storeSampleBuffer):
LayoutTests:
- media/content/video-as-img.mp4: Added.
- media/video-as-img-output-pts-expected.txt: Added.
- media/video-as-img-output-pts.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@266239 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- 3:57 PM Changeset in webkit [266301] by
-
- 2 edits in trunk/Source/WebKit
[macOS] Fix iokit-get-properties sandbox violations
https://bugs.webkit.org/show_bug.cgi?id=215925
Reviewed by Brent Fulgham.
Add iokit-property to WebContent sandbox that should have been a part of r266282.
- WebProcess/com.apple.WebProcess.sb.in:
- 3:53 PM Changeset in webkit [266300] by
-
- 2 edits in trunk/LayoutTests
[ macOS ] Tests expectations changed as test passing but expected to fail
https://bugs.webkit.org/show_bug.cgi?id=215786
Unreviewed test gardening.
- platform/mac/TestExpectations: corrections for BigSur+
- 3:37 PM Changeset in webkit [266299] by
-
- 8 edits in branches/safari-610.1.28.1-branch/Source
Versioning.
WebKit-7610.1.28.1.6
- 3:17 PM Changeset in webkit [266298] by
-
- 2 edits in trunk/LayoutTests
[ BigSur+ ] fast/images/image-controls-basic.html
<rdar://64947666>
Unreviewed test gardening.
- platform/mac/TestExpectations:
- 3:15 PM WebKitGTK/2.30.x edited by
- (diff)
- 3:09 PM Changeset in webkit [266297] by
-
- 4 edits in trunk
[WPE] Enable Notifications support
https://bugs.webkit.org/show_bug.cgi?id=215924
Reviewed by Philippe Normand.
.:
Enable web notifications support for the WPE port. The code is shared with the GTK
port, but as we do not try to find and enable libnotify the default implementation
is a stub: embedders can customize how to display notifications by connecting to
the WebKitWebView:show-notification signal. This way we do not add any additional
build dependencies.
- Source/cmake/OptionsWPE.cmake: Flip ENABLE_NOTIFICATIONS to ON.
LayoutTests:
- platform/wpe/TestExpectations: Update test expectations.
- 2:40 PM Changeset in webkit [266296] by
-
- 3 edits in trunk/LayoutTests
[GTK][WPE] Gardening some failures.
Unreviewed test gardening.
- platform/glib/TestExpectations:
- platform/glib/imported/w3c/web-platform-tests/webaudio/the-audio-api/the-biquadfilternode-interface/biquad-automation-expected.txt:
- 2:37 PM Changeset in webkit [266295] by
-
- 49 edits in trunk
Remove almost all the remaining uses of live ranges
https://bugs.webkit.org/show_bug.cgi?id=215755
Reviewed by Sam Weinig.
Source/WebCore:
- accessibility/AXObjectCache.cpp:
(WebCore::characterOffsetsInOrder): Use documentOrder.
- accessibility/AccessibilityObject.cpp:
(WebCore::AccessibilityObject::misspellingRange const): Ditto.
(WebCore::rangeClosestToRange): Use characterCount. Also removed unnnecessary assertions
and special cases.
(WebCore::AccessibilityObject::rangeOfStringClosestToRangeInDirection const): Use documentOrder.
- accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::documentBasedSelectedTextRange const): Use intersects.
Also use firstRange instead of toNormalizedRange.
- dom/Element.h: Export attachShadow so it can be used in unit tests.
- dom/Node.cpp:
(WebCore::depthInComposedTree): Renamed and changed to use parentInComposedTree
instead of parentOrShadowHostNode to handle cases involving assigned slots correctly.
(WebCore::commonInclusiveAncestorAndChildren): Ditto.
(WebCore::isSiblingSubsequent): Added assertions.
(WebCore::documentOrder): Add logic so shadow roots are ordered before the
other children of their hosts in cases that don't involve assigned slots.
Without this the results would be inconsistent based on the order of the
arguments passed to documentOrder.
- dom/Node.h: Added is_neq, is_lteq, and is_gteq for use with PartialOrdering,
matching the names and behavior from C++20.
- dom/Range.cpp: Removed unused CoordinateSpace.
(WebCore::Range::Range): Updated since RangeBoundaryPoint now uses references.
(WebCore::Range::updateDocument): Renamed from setDocument and changed so that
it maintains the invariants without requiring the caller pass any arguments.
Also updated for RangeBoundaryPoint anme changes.
(WebCore::Range::compareBoundaryPoints): Deleted.
(WebCore::checkForDifferentRootContainer): Deleted.
(WebCore::Range::setStart): Updated for checkNodeOffsetPair name change.
Removed too-complex code for moving between documents. Use documentOrder
instead of checkForDifferentRootContainer to check for start/end that have
incorrect order or no common ancestor. Use updateDocument.
(WebCore::Range::setEnd): Ditto.
(WebCore::Range::isPointInRange): Use isPointInRange that takes a SimpleRange.
Also updated for checkNodeOffsetPair name change.
(WebCore::Range::comparePoint const): Use documentOrder. Ditto.
(WebCore::Range::compareNode const): Use documentOrder.
(WebCore::top): Deleted.
(WebCore::Range::compareBoundaryPoints const): Deleted more overloads.
Kept only the one needed for bindings, and renamed it from
compareBoundaryPointsForBindings. Also fixed the implementatin to be simpler,
use documentOrder, and match the DOM specification more closely (without a
behavior change).
(WebCore::Range::boundaryPointsValid const): Deleted.
(WebCore::Range::intersectsNode const): Use intersects from SimpleRange.
Also don't raise exceptions.
(WebCore::lengthOfContentsInNode): Deleted. This gives the same result as
Node::length, but less efficiently.
(WebCore::Range::processContents): Updated since RangeBoundaryPoint now
uses references.
(WebCore::processContentsBetweenOffsets): Removed obsolete comment.
(WebCore::Range::toString const): Use intersectingNodes and
characterDataOffsetRange.
(WebCore::Range::text const): Deleted.
(WebCore::Range::detach): Moved to the header.
(WebCore::Range::checkNodeOffsetPair const): Rneamed from checkNodeWOffset.
(WebCore::Range::cloneRange): Use m_ownerDocument.
(WebCore::Range::setStartAfter): Tweaked coding style.
(WebCore::Range::setEndBefore): Ditto.
(WebCore::Range::setEndAfter): Ditto.
(WebCore::Range::selectNode): Ditto. Also removed unnecessary code to
call setDocument, since setStart takes care of that.
(WebCore::Range::selectNodeContents): Ditto.
(WebCore::Range::setStartBefore): Ditto.
(WebCore::Range::firstNode const): Deleted.
(WebCore::Range::pastLastNode const): Deleted.
(WebCore::Range::contains const): Deleted.
(WebCore::areRangesEqual): Deleted.
(WebCore::rangesOverlap): Deleted.
(WebCore::boundaryNodeChildrenChanged): Tweaked coding style and updated
since RangeBoundaryPoint uses a reference.
(WebCore::Range::nodeChildrenChanged): Use m_ownerDocument.
(WebCore::boundaryNodeChildrenWillBeRemoved): Removed unneeded code
to handle case where one of the children is the childBefore but the
node is not inside the container -- that case doesn't exist. Use
Node::contains.
(WebCore::Range::nodeChildrenWillBeRemoved): Use m_ownerDocument.
(WebCore::boundaryNodeWillBeRemoved): Simplify logic using
Range::contains.
(WebCore::Range::nodeWillBeRemoved): Use m_ownerDocument.
(WebCore::Range::parentlessNodeMovedToNewDocumentAffectsRange):
Updated since RangeBoundaryPoint uses a reference.
(WebCore::boundaryTextInserted): Ditto.
(WebCore::boundaryTextRemoved): Ditto.
(WebCore::boundaryTextNodesMerged): Ditto.
(WebCore::boundaryTextNodesSplit): Ditto.
(showTree): Removed call to boundaryPointsValid. That's checking something
the class already treats as an invariant, so it's obsolete code left from
long ago when we didn't have that implemented correctly.
- dom/Range.h: Reorganized the header to match the IDL file.
Removed ownerDocument, compareBoundaryPoints overloads other than the one
used from bindings, firstNode, pastLastNode, text, contains, areRangesEqual,
and rangesOverlap. Also updated since RangeBoundaryPoint::container is now
a reference. Moved detach function here to the header as an empty inline.
- dom/Range.idl: Changed start/endOffset to unsigned. Stopped using the
name compareBoundaryPointersForBindings. Updated intersectsNode to not
throw exceptions as called for in the DOM specification. Tweaked comments.
- dom/RangeBoundaryPoint.h: Changed constructor argument to a reference.
Renamed setToBeforeChild, setToAfterChild, setToStartOfNode, and setToEndOfNode
to setToBeforeNode, setToAfterNode, setToBeforeContents, setToAfterContents.
Merged ensureOffsetIsValid into the offset function. Changed m_containerNode
into a Ref<> and renamed it m_container. Removed clear.
- dom/SimpleRange.cpp:
(WebCore::isOffsetBeforeChild): Add logic so shadow roots are ordered after offset
0 but before offset 1 in their hosts in cases that don't involve assigned slots.
This is an arbitrary choice, but is consistent with how we order shadow roots before
other child nodes.
(WebCore::documentOrder): Use parentInComposedTree instead of parentOrShadowHostNode
to handle cases involving assigned slots correctly.
(WebCore::isPointInRange): Added.
(WebCore::documentOrder): Added.
(WebCore::contains): Added.
(WebCore::intersects): Added.
(WebCore::unionRange): Added.
- dom/SimpleRange.h: Updated for above.
- editing/Editing.cpp:
(WebCore::comparePositions): Use documentOrder.
(WebCore::isNodeVisiblyContainedWithin): Use contains.
- editing/Editor.cpp:
(WebCore::isFrameInRange): Use intersects.
(WebCore::Editor::scanSelectionForTelephoneNumbers): Use intersects.
- editing/EditorCommand.cpp:
(WebCore::unionRanges): Moved to SimpleRange.cpp, where it's now
implemented using documentOrder.
(WebCore::executeDeleteToMark): Updated name to unionRange.
(WebCore::executeSelectToMark): Ditto.
- editing/FrameSelection.cpp:
(WebCore::FrameSelection::respondToNodeModification): Use intersects.
- editing/mac/DictionaryLookupLegacy.mm:
(WebCore::selectionContainsPosition): Use isPointInRange.
- page/DOMSelection.cpp:
(WebCore::DOMSelection::extend): Use makeRefPtr and Node::length
to streamline the implementation.
(WebCore::DOMSelection::getRangeAt): Use makeBoundaryPointBeforeNode
and remove unneeded null check that's already covered above.
(WebCore::DOMSelection::addRange): Use intersects and unionRange.
(WebCore::DOMSelection::containsNode const): Use intersects and contains.
(WebCore::DOMSelection::isValidForPosition const): Use boolean logic.
- page/DragController.cpp:
(WebCore::DragController::draggableElement const): Use contains.
(WebCore::DragController::insertDroppedImagePlaceholdersAtCaret): Use intersects.
- page/EventHandler.cpp:
(WebCore::EventHandler::dispatchMouseEvent): Use contains.
- page/Page.cpp:
(WebCore::Page::findTextMatches): Use documentOrder.
- page/mac/ServicesOverlayController.mm:
(WebCore::ServicesOverlayController::findTelephoneNumberHighlightContainingSelectionHighlight):
Use contains.
- testing/Internals.cpp:
(WebCore::Internals::rangeAsText): Use plainText.
(WebCore::join): Added.
(WebCore::Internals::rangeAsTextUsingBackwardsTextIterator): Use a vector
that we reverse and join rather than inserting at the start of a string.
(WebCore::Internals::subrange): Added a call to updateLayout since text iterator
functions depend on the state of the render tree. Realized this was theoretically
needed because of Internals::rangeAsText.
(WebCore::Internals::rangeOfStringNearLocation): Ditto.
Source/WebKit:
- WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMRange.cpp:
(webkit_dom_range_compare_boundary_points): Update for rename of
compareBoundaryPointsForBindings.
(webkit_dom_range_intersects_node): Removed exception handling.
(webkit_dom_range_get_text): Use plainText.
- WebProcess/InjectedBundle/API/mac/WKDOMRange.mm:
(-[WKDOMRange initWithDocument:]): Tightened up into a one-liner.
(-[WKDOMRange text]): Use plainText.
(-[WKDOMRange textRects]): Stopped using Range::ownerDocument.
- WebProcess/InjectedBundle/DOM/InjectedBundleRangeHandle.cpp:
(WebKit::InjectedBundleRangeHandle::document): Stopped using
Range::ownerDocument.
(WebKit::InjectedBundleRangeHandle::boundingRectInWindowCoordinates const):
Ditto.
(WebKit::InjectedBundleRangeHandle::renderedImage): Ditto.
(WebKit::InjectedBundleRangeHandle::text const): Use plainText.
- WebProcess/WebPage/glib/WebPageGLib.cpp:
(WebKit::WebPage::getPlatformEditorState const): Use contains.
- WebProcess/WebPage/ios/WebPageIOS.mm:
(WebKit::WebPage::requestDocumentEditingContext): Use intersects.
Source/WebKitLegacy/mac:
- DOM/DOMRange.mm:
(-[DOMRange text]): Use plainText.
(-[DOMRange compareBoundaryPoints:sourceRange:]): Update for rename of
compareBoundaryPointsForBindings.
(-[DOMRange intersectsNode:]): Removed raiseOnDOMError.
- DOM/DOMUIKitExtensions.mm:
(firstNodeAfter): Moved Range::firstNode here since it's the only remaining caller;
might need to keep legacy behavior depending on how this is used by UIKit.
(-[DOMRange firstNode]): Call firstNodeAfter.
- DOM/WebDOMOperations.mm:
(-[DOMRange markupString]): Stop using Range::ownerDocument.
Tools:
- TestWebKitAPI/Tests/WebCore/DocumentOrder.cpp:
(TestWebKitAPI::TEST): Added test cases for node ordering involving non-user-agent
shadow tree. Added test case that showed the old "use node order if offsets are equal"
rule for boundary point ordering was wrong. Added tests for isPointInRange,
intersects, and contains functions.
- 2:35 PM Changeset in webkit [266294] by
-
- 2 edits in trunk/LayoutTests
[ BigSur+ ] fast/forms/hidpi-textfield-background-bleeding.html
<rdar://66295552>
Unreviewed test gardening.
- platform/mac/TestExpectations:
- 2:27 PM Changeset in webkit [266293] by
-
- 9 edits in trunk
AudioParam.value setter should call setValueAtTime(value, now)
https://bugs.webkit.org/show_bug.cgi?id=215946
Reviewed by Eric Carlson.
LayoutTests/imported/w3c:
Rebaseline WPT tests now that more checks are passing.
- web-platform-tests/webaudio/the-audio-api/the-biquadfilternode-interface/no-dezippering-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-delaynode-interface/no-dezippering-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-gainnode-interface/no-dezippering-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/automation-changes-expected.txt:
Source/WebCore:
AudioParam.value setter should call setValueAtTime(value, now), as per:
No new tests, rebaselined existing tests.
- Modules/webaudio/AudioParam.cpp:
(WebCore::AudioParam::valueForBindings const):
(WebCore::AudioParam::setValueForBindings):
- Modules/webaudio/AudioParam.h:
- Modules/webaudio/AudioParam.idl:
- 2:09 PM Changeset in webkit [266292] by
-
- 16 edits2 adds in trunk
Vertical scrolling gets stuck when a horizontal scroller is under the mouse (google search results)
https://bugs.webkit.org/show_bug.cgi?id=215641
<rdar://problem/67430532>
Reviewed by Tim Horton.
Source/WebCore:
There are two parts to this fix. First, findEnclosingScrollableContainer() needs
to use the same vertical-biasing delta fixup that we use in other places, to bias towards
vertical scrolling.
Second, when we've determined that the main frame should perform the scroll and dispatch
the wheel event to the scrolling thread, we used to hit-test from scratch on the scrolling
thread and and try to send the event to a scroller which we already know should not handle
it. So pass along a target ScrollingNodeID, and start the scrolling thread handling from
that node.
Test: fast/scrolling/mac/horizontal-overflow-trapping-small-deltas.html
- page/FrameView.cpp:
(WebCore::FrameView::wheelEvent):
- page/mac/EventHandlerMac.mm:
(WebCore::findEnclosingScrollableContainer):
- page/scrolling/ScrollingCoordinator.h:
(WebCore::ScrollingCoordinator::handleWheelEvent):
- page/scrolling/ScrollingTree.cpp:
(WebCore::ScrollingTree::handleWheelEvent):
(WebCore::ScrollingTree::handleWheelEventWithNode):
- page/scrolling/ScrollingTree.h:
- page/scrolling/ThreadedScrollingTree.cpp:
(WebCore::ThreadedScrollingTree::handleWheelEventAfterMainThread):
- page/scrolling/ThreadedScrollingTree.h:
- page/scrolling/mac/ScrollingCoordinatorMac.h:
- page/scrolling/mac/ScrollingCoordinatorMac.mm:
(WebCore::ScrollingCoordinatorMac::handleWheelEvent):
- page/scrolling/nicosia/ScrollingCoordinatorNicosia.cpp:
(WebCore::ScrollingCoordinatorNicosia::handleWheelEvent):
- page/scrolling/nicosia/ScrollingCoordinatorNicosia.h:
LayoutTests:
After this change iframe-latch-small-deltas.html would time-out because WheelEventTestMonitor
would get stuck with a "content scrolling" defer region, due to begin/end wheel
events not getting both sent to the iframe. But the test was actually broken; logging shows
that it reset latching state anyway, and it's not testing the shipping configuration of
async iframe scrolling. So fix the test to use async iframe scrolling, and to avoid
rubber-banding, so that latching is not cleared due to elapsed time.
- fast/scrolling/latching/iframe-latch-small-deltas-expected.txt:
- fast/scrolling/latching/iframe-latch-small-deltas.html:
- fast/scrolling/mac/horizontal-overflow-trapping-small-deltas-expected.txt: Added.
- fast/scrolling/mac/horizontal-overflow-trapping-small-deltas.html: Added.
- 1:21 PM Changeset in webkit [266291] by
-
- 6 edits in trunk/Source/WebCore
No need to run full can-use-for (fast inline layout codepath) check on every style change.
https://bugs.webkit.org/show_bug.cgi?id=215937
<rdar://problem/67951360>
Reviewed by Antti Koivisto.
Let's use the StyleDifference to figure out how extensive the can-use-for check should be.
We can certainly skip some relatively expensive content checks when we know that the style change only triggers repaint or positioned-movement-only changes.
- layout/integration/LayoutIntegrationLineLayout.cpp:
(WebCore::LayoutIntegration::LineLayout::canUseForAfterStyleChange):
- layout/integration/LayoutIntegrationLineLayout.h:
- rendering/RenderBlockFlow.cpp:
(WebCore::RenderBlockFlow::styleDidChange):
- rendering/SimpleLineLayout.cpp:
(WebCore::SimpleLineLayout::canUseForAfterStyleChange):
- rendering/SimpleLineLayout.h:
- 1:21 PM Changeset in webkit [266290] by
-
- 5 edits2 adds in trunk
[iOS] Vertical text's logical width calculation is stale from the previous height of the WKWebView
https://bugs.webkit.org/show_bug.cgi?id=215910
Reviewed by Simon Fraser.
Source/WebCore:
When there is vertical text (really: orthogonal flows) in the content, the available width of the vertical
text is the height of its containing block. However, the height of the containing block (indeed: all ancestors)
may be "auto", in which case the CSS spec says the available width of the element should be "the initial
containing block's size."[1]
Previously, we were using the FrameView's visibleHeight as this metric. However, the visibleHeight is calculated
asynchronously, after layout, after a round-trip from the Web Process to the UI Process and back to the Web
Process. Therefore, if content changes the WKWebView's size and the web view immediatey re-lays-out, this
visibleHeight metric is stale from whatever the previous height of the WKWebView was. In addition, the
visibleHeight metric isn't even supposed to be used inside layout; it's instead only supposed to be used for
things like position: fixed elements.
Instead, we should use FrameView::layoutSize(), which is set before layout to the size of the WKWebView. The
name even indicates that it should be used for layout purposes.
[1] https://drafts.csswg.org/css-writing-modes-4/#orthogonal-auto
Test: WebKit.OrthogonalFlowAvailableSize
- rendering/RenderBox.cpp:
(WebCore::RenderBox::perpendicularContainingBlockLogicalHeight const):
- rendering/RenderView.cpp:
(WebCore::RenderView::availableLogicalHeight const):
Tools:
- TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
- TestWebKitAPI/Tests/WebKit/OrthogonalFlowAvailableSize.mm: Added.
(TEST):
- TestWebKitAPI/Tests/WebKit/orthogonal-flow-available-size.html: Added.
- 11:52 AM Changeset in webkit [266289] by
-
- 16 edits1 copy7 adds in trunk/LayoutTests/imported/w3c
Resync web-platform-tests/webaudio from upstream
https://bugs.webkit.org/show_bug.cgi?id=215935
Reviewed by Sam Weinig.
Resync web-platform-tests/webaudio from upstream 37c7777f647d4383943.
- web-platform-tests/webaudio/the-audio-api/the-analysernode-interface/test-analyser-minimum-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-audiocontext-interface/audiocontextoptions-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-audiocontext-interface/constructor-allowed-to-start-expected.txt: Added.
- web-platform-tests/webaudio/the-audio-api/the-biquadfilternode-interface/no-dezippering-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-convolvernode-interface/realtime-conv-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-gainnode-interface/no-dezippering-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-oscillatornode-interface/detune-limiting-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-oscillatornode-interface/detune-overflow-expected.txt: Added.
- web-platform-tests/webaudio/the-audio-api/the-oscillatornode-interface/osc-basic-waveform-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-periodicwave-interface/periodicWave-expected.txt: Added.
- web-platform-tests/webaudio/the-audio-api/the-scriptprocessornode-interface/simple-input-output-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-stereopanner-interface/no-dezippering-expected.txt:
- 11:37 AM Changeset in webkit [266288] by
-
- 4 edits in trunk
Fix text-transform inheritance to ::marker
https://bugs.webkit.org/show_bug.cgi?id=214359
Patch by Tyler Wilcock <Tyler Wilcock> on 2020-08-28
Reviewed by Darin Adler.
Per https://github.com/w3c/csswg-drafts/issues/4206, default ::marker
pseudoelement style totext-transform: none
in the UA stylesheet to
avoid surprising effects from inheritance.
No tests added -- change is tested by existing WPTs.
- css/html.css:
(::marker):
- 11:33 AM Changeset in webkit [266287] by
-
- 2 edits in trunk/Source/WebKit
REGRESSION(r266187): ARGUMENT BAD: WebIconUtilities.mm(138) : RetainPtr<UIImage> WebKit::iconForFile(NSURL *) file, [file isFileURL]
https://bugs.webkit.org/show_bug.cgi?id=215931
Patch by Alex Christensen <achristensen@webkit.org> on 2020-08-28
Reviewed by Wenson Hsieh.
It's possible for JavaScript to make a filename that doesn't convert to a file URL, such as the empty string.
When this happens, we would've returned null for the icon later anyways, but adding a null check prevents an assertion.
Covered by 7 existing tests which would have hit that assertion.
- WebProcess/WebCoreSupport/ios/WebChromeClientIOS.mm:
(WebKit::WebChromeClient::createIconForFiles):
- 11:02 AM Changeset in webkit [266286] by
-
- 2 edits in trunk/Source/WebCore
Remove the system tracing from DisplayList::Replayer::replay()
https://bugs.webkit.org/show_bug.cgi?id=215933
Patch by Said Abou-Hallawa <sabouhallawa@apple.com> on 2020-08-28
Reviewed by Simon Fraser.
Remove this expensive tracing point since it can affect the tracing of
the MotionMark Design test.
- platform/graphics/displaylists/DisplayListReplayer.cpp:
(WebCore::DisplayList::Replayer::replay):
- 11:02 AM Changeset in webkit [266285] by
-
- 3 edits1 add in branches/safari-610-branch
Cherry-pick r266254. rdar://problem/67949890
GetByVal constant folding over a Double OutOfBoundsSaneChain array with no BytecodeUsesAsOther should constant fold to PNaN, not undefined
https://bugs.webkit.org/show_bug.cgi?id=215894
<rdar://problem/67669696>
Reviewed by Michael Saboff and Keith Miller.
JSTests:
- stress/oob-sane-chain-double-constant-fold-to-PNaN.js: Added.
Source/JavaScriptCore:
GetByVals of the form { OutOfBoundsSaneChain, Double } where there are no
BytecodeUsesAsOther return PNaN for holes and OOB accesses, not jsUndefined().
The constant folding for this though was folding to jsUndefined(). I forgot
to update that code to constant fold to PNaN when I wrote the OutOfBoundsSaneChain
implementation.
- dfg/DFGAbstractInterpreterInlines.h: (JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@266254 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- 10:31 AM Changeset in webkit [266284] by
-
- 2 edits in trunk/Source/WebKit
Enable TransformStream by default
https://bugs.webkit.org/show_bug.cgi?id=215891
Reviewed by Darin Adler.
- Shared/WebPreferences.yaml:
- 10:24 AM Changeset in webkit [266283] by
-
- 13 edits in trunk
initializeReadableStream should check for highWaterMark be undefined
https://bugs.webkit.org/show_bug.cgi?id=215890
Reviewed by Alex Christensen.
LayoutTests/imported/w3c:
- web-platform-tests/streams/transform-streams/backpressure.any-expected.txt:
- web-platform-tests/streams/transform-streams/backpressure.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/errors.any-expected.txt:
- web-platform-tests/streams/transform-streams/errors.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/reentrant-strategies.any-expected.txt:
- web-platform-tests/streams/transform-streams/reentrant-strategies.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/strategies.any-expected.txt:
- web-platform-tests/streams/transform-streams/strategies.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/terminate.any-expected.txt:
- web-platform-tests/streams/transform-streams/terminate.any.worker-expected.txt:
Source/WebCore:
Covered by updated tests.
- Modules/streams/ReadableStream.js:
(initializeReadableStream):
- 10:14 AM Changeset in webkit [266282] by
-
- 2 edits in trunk/Source/WebKit
[macOS] Fix iokit-get-properties sandbox violations
https://bugs.webkit.org/show_bug.cgi?id=215925
Reviewed by Darin Adler.
A few iokit properties needs to be added to the WebContent process' sandbox after observing these being in use.
- WebProcess/com.apple.WebProcess.sb.in:
- 10:00 AM Changeset in webkit [266281] by
-
- 2 edits in trunk/Source/WebCore
Remove adopted node from TextManipulationController
https://bugs.webkit.org/show_bug.cgi?id=215914
Reviewed by Wenson Hsieh.
Remove Node from TextManipulationController when it gets adopted to a new document.
- dom/Node.cpp:
(WebCore::Node::moveNodeToNewDocument):
- 9:47 AM Changeset in webkit [266280] by
-
- 5 edits2 adds in trunk
REGRESSION (r263506): scale transform transitions won't overshoot
https://bugs.webkit.org/show_bug.cgi?id=215826
<rdar://problem/67759310>
Reviewed by Simon Fraser.
Source/WebCore:
Test: webanimations/accelerated-css-transition-with-easing-y-axis-above-1.html
In r263506 we made a change where accelerated animations would set their animation-wide
timing functions using PlatformCAAnimation::setTimingFunction() instead of setting it
on individual keyframes. This change was required to support the unique ability of
JS-originated animations to specify both an animation-wide timing function as well as
per-keyframe timing functions.
In the case of CSS Transitions, this meant that instead of setting a per-keyframe timing
function, this change would set the animation-wide timing function since CSS Transitions
should map the transition-timing-function property to the "easing" property of the Animation
object exposed by the Web Animations API, not as the timing function of the single keyframe
interval.
However, this change surfaced a bug in Core Animation on macOS and iOS where a cubic-bezier()
timing function with y values above 1 get clipped when applied to a CAKeyframeAnimation using
the timingFunction property (singular) instead of the timingFunctions property (plural).
To work around this issue, we set Animation::property() on the generated Animation object passed
to GraphicsLayerCA to make it possible to detect when we're dealing with a CSS Transition and
set the timing function on the keyframe interval rather than on the animation itself.
Alas, this does not fix the case where a JS-originated animation will specify an animation-wide
timing function with a cubic-bezier() timing function with y values above 1. There is no known
workaround at this time for this case and this is covered by bug 215918.
- animation/KeyframeEffect.cpp:
(WebCore::KeyframeEffect::backingAnimationForCompositedRenderer const):
- platform/graphics/ca/GraphicsLayerCA.cpp:
(WebCore::GraphicsLayerCA::createTransformAnimationsFromKeyframes):
LayoutTests:
Add a new test that checks that a CSS Transition using a cubic-bezier() timing function with
a y value beyond 1 does not get clipped.
- platform/win/TestExpectations:
- webanimations/accelerated-css-transition-with-easing-y-axis-above-1-expected.html: Added.
- webanimations/accelerated-css-transition-with-easing-y-axis-above-1.html: Added.
- 9:08 AM Changeset in webkit [266279] by
-
- 2 edits in trunk/LayoutTests
[ macOS iOS ] compositing/video/poster.html is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=215929
Unreviewed test gardening.
- platform/ios-wk2/TestExpectations:
- 8:23 AM Changeset in webkit [266278] by
-
- 2 edits in trunk/LayoutTests
[ macOS wk1 ] svg/custom/object-sizing.xhtml is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=215926
Unreviewed test gardening.
- platform/mac-wk1/TestExpectations:
- 8:21 AM Changeset in webkit [266277] by
-
- 5 edits1 delete in trunk/Tools
[webkitcorepy] Move Timeout to webkitcorepy (Part 3)
https://bugs.webkit.org/show_bug.cgi?id=215584
<rdar://problem/67270713>
Reviewed by Darin Adler.
- Scripts/webkitpy/benchmark_runner/webserver_benchmark_runner.py: Use webkitcorepy's Timeout object.
- Scripts/webkitpy/common/timeout_context.py: Removed.
- Scripts/webkitpy/port/simulator_process.py: Use webkitcorepy's Timeout object.
- Scripts/webkitpy/webdriver_plt/liveplt.py: Ditto.
- Scripts/webkitpy/xcode/simulated_device.py: Ditto.
- 6:37 AM Changeset in webkit [266276] by
-
- 10 edits1 add in trunk/Source/WebCore
[LFC][IFC] Move inline alignment code to LineBox
https://bugs.webkit.org/show_bug.cgi?id=215117
Reviewed by Antti Koivisto.
LineBox is responsible for aligning the runs both vertically and horizontally (LineContentAligner -> LineBox).
(Note that LineBox::m_runRectList is transitional and will eventually be converted to inline boxes.)
- Sources.txt:
- WebCore.xcodeproj/project.pbxproj:
- layout/inlineformatting/InlineFormattingContext.cpp:
(WebCore::Layout::InlineFormattingContext::lineLayout):
(WebCore::Layout::InlineFormattingContext::setDisplayBoxesForLine):
(WebCore::Layout::HangingContent::width const): Deleted.
(WebCore::Layout::HangingContent::isConditional const): Deleted.
(WebCore::Layout::HangingContent::setIsConditional): Deleted.
(WebCore::Layout::HangingContent::expand): Deleted.
(): Deleted.
(WebCore::Layout::HangingContent::reset): Deleted.
(WebCore::Layout::LineContentAligner::inlineRectList const): Deleted.
(WebCore::Layout::LineContentAligner::formattingContext const): Deleted.
(WebCore::Layout::LineContentAligner::layoutState const): Deleted.
(WebCore::Layout::LineContentAligner::LineContentAligner): Deleted.
(WebCore::Layout::LineContentAligner::alignHorizontally): Deleted.
(WebCore::Layout::LineContentAligner::alignVertically): Deleted.
(WebCore::Layout::LineContentAligner::adjustBaselineAndLineHeight): Deleted.
(WebCore::Layout::LineContentAligner::collectHangingContent const): Deleted.
- layout/inlineformatting/InlineFormattingContext.h:
- layout/inlineformatting/InlineLineBox.h:
(WebCore::Layout::LineBox::inlineRectList const):
(WebCore::Layout::LineBox::logicalBottom const):
(WebCore::Layout::LineBox::logicalRect const):
(WebCore::Layout::LineBox::scrollableOverflow const):
(WebCore::Layout::LineBox::formattingContext const):
(WebCore::Layout::LineBox::layoutState const):
(WebCore::Layout::LineBox::setLogicalHeight): Deleted.
(WebCore::Layout::LineBox::setLogicalWidth): Deleted.
(WebCore::Layout::LineBox::moveHorizontally): Deleted.
(WebCore::Layout::LineBox::expandHorizontally): Deleted.
(WebCore::Layout::LineBox::shrinkHorizontally): Deleted.
(WebCore::Layout::LineBox::expandVertically): Deleted.
(WebCore::Layout::LineBox::shrinkVertically): Deleted.
(WebCore::Layout::LineBox::isConsideredEmpty const): Deleted.
(WebCore::Layout::LineBox::setIsConsideredEmpty): Deleted.
(WebCore::Layout::LineBox::setIsConsideredNonEmpty): Deleted.
(WebCore::Layout::LineBox::LineBox): Deleted.
(WebCore::Layout::LineBox::resetAlignmentBaseline): Deleted.
- layout/inlineformatting/InlineLineBuilder.h:
(WebCore::Layout::LineBuilder::lineLogicalWidth const):
(WebCore::Layout::LineBuilder::availableWidth const):
- layout/inlineformatting/LineLayoutContext.cpp:
(WebCore::Layout::LineLayoutContext::close):
- layout/inlineformatting/LineLayoutContext.h:
- 1:46 AM Changeset in webkit [266275] by
-
- 21 edits in trunk
Add WebGL(2) context support to OffscreenCanvas
https://bugs.webkit.org/show_bug.cgi?id=209809
Reviewed by Dean Jackson.
LayoutTests/imported/w3c:
Rebaseline OffscreenCanvas tests after enabling WebGL contexts.
- web-platform-tests/html/canvas/offscreen/manual/the-offscreen-canvas/offscreencanvas.getcontext-expected.txt:
- web-platform-tests/html/canvas/offscreen/manual/the-offscreen-canvas/offscreencanvas.getcontext.worker-expected.txt:
Source/WebCore:
Enable WebGL contexts with OffscreenCanvas by making sure the required
settings are marshalled to workers and that any functionality that
doesn't require an HTMLCanvasElement uses CanvasBase instead.
No new tests, rebaselined existing test.
- html/CanvasBase.h:
- html/CustomPaintCanvas.cpp:
(WebCore::CustomPaintCanvas::clearCopiedImage const):
- html/CustomPaintCanvas.h:
- html/HTMLCanvasElement.cpp:
(WebCore::HTMLCanvasElement::clearCopiedImage const):
- html/HTMLCanvasElement.h:
- html/OffscreenCanvas.cpp:
(WebCore::requiresAcceleratedCompositingForWebGL):
(WebCore::shouldEnableWebGL):
(WebCore::OffscreenCanvas::createContextWebGL):
(WebCore::OffscreenCanvas::getContext):
- html/OffscreenCanvas.h:
- html/OffscreenCanvas.idl:
- html/canvas/WebGL2RenderingContext.idl:
- html/canvas/WebGLRenderingContext.idl:
- html/canvas/WebGLRenderingContextBase.cpp:
(WebCore::WebGLRenderingContextBase::WebGLRenderingContextBase):
(WebCore::WebGLRenderingContextBase::markContextChanged):
(WebCore::WebGLRenderingContextBase::paintRenderingResultsToCanvas):
- workers/WorkerGlobalScope.cpp:
(WebCore::WorkerGlobalScope::WorkerGlobalScope):
- workers/WorkerGlobalScope.h:
(WebCore::WorkerGlobalScope::acceleratedCompositingEnabled const):
(WebCore::WorkerGlobalScope::webGLEnabled const):
- workers/WorkerMessagingProxy.cpp:
(WebCore::WorkerMessagingProxy::startWorkerGlobalScope):
- workers/WorkerThread.cpp:
(WebCore::WorkerParameters::isolatedCopy const):
- workers/WorkerThread.h:
- workers/service/context/ServiceWorkerThread.cpp:
(WebCore::ServiceWorkerThread::ServiceWorkerThread):
Aug 27, 2020:
- 9:49 PM Changeset in webkit [266274] by
-
- 13 edits in trunk/LayoutTests
[GLIB] Rebaseline websocket tests after r266230
Unreviewed test gardening.
- platform/glib/TestExpectations:
- platform/glib/http/tests/websocket/tests/hybi/close-code-and-reason-expected.txt:
- platform/glib/http/tests/websocket/tests/hybi/handshake-ok-with-http-version-beyond-1_1-expected.txt:
- platform/glib/http/tests/websocket/tests/hybi/invalid-encode-length-expected.txt:
- platform/glib/http/tests/websocket/tests/hybi/invalid-masked-frames-from-server-expected.txt:
- platform/glib/http/tests/websocket/tests/hybi/long-control-frame-expected.txt:
- platform/glib/http/tests/websocket/tests/hybi/long-invalid-header-expected.txt:
- platform/glib/http/tests/websocket/tests/hybi/network-process-crash-error-expected.txt:
- platform/glib/http/tests/websocket/tests/hybi/reserved-bits-expected.txt:
- platform/glib/http/tests/websocket/tests/hybi/reserved-opcodes-expected.txt:
- platform/glib/http/tests/websocket/tests/hybi/too-long-payload-expected.txt:
- platform/glib/http/tests/websocket/tests/hybi/workers/close-code-and-reason-expected.txt:
- 8:56 PM Changeset in webkit [266273] by
-
- 1 edit4 adds in trunk/LayoutTests
Unreviewed test gardening, add Windows baselines after r266228.
- platform/win/streams/reference-implementation/transform-stream-errors-expected.txt: Added.
- platform/win/streams/reference-implementation/transform-stream-expected.txt: Added.
- 8:24 PM Changeset in webkit [266272] by
-
- 2 edits in trunk/Tools
REGRESSION(r266221) Layout tests step hanging due to slow WaveDiff comparison
https://bugs.webkit.org/show_bug.cgi?id=215886
Reviewed by Darin Adler.
After r266221, a couple of webaudio tests generated WAV files with a
small difference in a large number of samples. This caused the
WaveDiff comparison to take too long.
While it works fine for small diffs, this ends up creating repeatedly
large strings for each concatenation, bringing the comparisons to a
halt.
Instead of relying on string concatenation, use a list to collect then
and only concat when actually retrieving the full string.
- Scripts/webkitpy/common/wavediff.py:
(WaveDiff.init):
(WaveDiff._diffParam):
(WaveDiff._diffSample):
(WaveDiff.diffText):
- 8:07 PM Changeset in webkit [266271] by
-
- 7 edits in trunk/Source/WebCore
[LFC][IFC] LineBuilder:Run should not have vertical geometry
https://bugs.webkit.org/show_bug.cgi?id=215068
Reviewed by Antti Koivisto.
Since LineBuilder only takes care of the horizontal ascpect of a run, LineBuilder::Run should only have logical left and width geometry.
The vertical geometry is computed as part of the content alignment phase in LineContentAligner (along with the final horizontal position
adjusted with the alignment).
This is still a transitional state (see InlineRunRectList m_runRectList) and all the line box related code will get moved to the LineBox class.
This patch also enables us to keep LineBuilder::Run const.
- layout/inlineformatting/InlineFormattingContext.cpp:
(WebCore::Layout::LineContentAligner::inlineRectList const):
(WebCore::Layout::LineContentAligner::LineContentAligner):
(WebCore::Layout::LineContentAligner::alignHorizontally):
(WebCore::Layout::LineContentAligner::alignVertically):
(WebCore::Layout::LineContentAligner::adjustBaselineAndLineHeight):
(WebCore::Layout::InlineFormattingContext::lineLayout):
(WebCore::Layout::InlineFormattingContext::setDisplayBoxesForLine):
(WebCore::Layout::LineContentAligner::justifyRuns): Deleted.
(WebCore::Layout::LineContentAligner::runContentHeight const): Deleted.
- layout/inlineformatting/InlineFormattingContext.h:
- layout/inlineformatting/InlineLineBuilder.cpp:
(WebCore::Layout::LineBuilder::close):
(WebCore::Layout::LineBuilder::visuallyCollapsePreWrapOverflowContent):
(WebCore::Layout::LineBuilder::TrimmableTrailingContent::addFullyTrimmableContent):
(WebCore::Layout::LineBuilder::TrimmableTrailingContent::addPartiallyTrimmableContent):
(WebCore::Layout::LineBuilder::Run::Run):
(WebCore::Layout::LineBuilder::Run::expand):
(WebCore::Layout::LineBuilder::Run::removeTrailingWhitespace):
(WebCore::Layout::LineBuilder::Run::setHorizontalExpansion):
(WebCore::Layout::m_textContent): Deleted.
(WebCore::Layout::LineBuilder::Run::setComputedHorizontalExpansion): Deleted.
- layout/inlineformatting/InlineLineBuilder.h:
(WebCore::Layout::LineBuilder::Run::style const):
(WebCore::Layout::LineBuilder::Run::logicalWidth const):
(WebCore::Layout::LineBuilder::Run::logicalLeft const):
(WebCore::Layout::LineBuilder::Run::expansion const):
(WebCore::Layout::LineBuilder::Run::hasExpansionOpportunity const):
(WebCore::Layout::LineBuilder::Run::expansionOpportunityCount const):
(WebCore::Layout::LineBuilder::Run::hasTrailingWhitespace const):
(WebCore::Layout::LineBuilder::Run::trailingWhitespaceWidth const):
(WebCore::Layout::LineBuilder::Run::moveHorizontally):
(WebCore::Layout::LineBuilder::Run::shrinkHorizontally):
(WebCore::Layout::LineBuilder::Run::hasCollapsedTrailingWhitespace const):
(WebCore::Layout::LineBuilder::runs const):
(WebCore::Layout::LineBuilder::Run::logicalRect const): Deleted.
(WebCore::Layout::LineBuilder::Run::adjustLogicalTop): Deleted.
(WebCore::Layout::LineBuilder::Run::moveVertically): Deleted.
(WebCore::Layout::LineBuilder::Run::setLogicalHeight): Deleted.
(WebCore::Layout::LineBuilder::runs): Deleted.
- 7:56 PM Changeset in webkit [266270] by
-
- 6 edits2 adds in trunk
AudioParam's minValue / maxValue are ignored
https://bugs.webkit.org/show_bug.cgi?id=215905
Reviewed by Darin Adler.
Source/WebCore:
Clamp the AudioParam's values so that they are always in the
[minValue, maxValue] range.
Test: webaudio/audioparam-nominal-range.html
- Modules/webaudio/AudioParam.cpp:
(WebCore::AudioParam::value):
(WebCore::AudioParam::setValue):
(WebCore::AudioParam::smooth):
(WebCore::AudioParam::calculateFinalValues):
(WebCore::AudioParam::calculateTimelineValues):
- Modules/webaudio/AudioParamTimeline.cpp:
(WebCore::AudioParamTimeline::valueForContextTime):
(WebCore::AudioParamTimeline::valuesForTimeRange):
- Modules/webaudio/AudioParamTimeline.h:
- Modules/webaudio/BiquadProcessor.cpp:
(WebCore::BiquadProcessor::BiquadProcessor):
LayoutTests:
Import layout test coverage from Blink.
- webaudio/audioparam-nominal-range-expected.txt: Added.
- webaudio/audioparam-nominal-range.html: Added.
- 7:55 PM Changeset in webkit [266269] by
-
- 12 edits in trunk
Prevent infinite recursion when upgrading custom elements
https://bugs.webkit.org/show_bug.cgi?id=206605
Reviewed by Antti Koivisto.
LayoutTests/imported/w3c:
Rebaselined the test now that one more test case is passing.
- web-platform-tests/custom-elements/upgrading-expected.txt:
Source/WebCore:
This patch updates our implementation of the concept to upgrade an element [1] and related algorithms
to match the latest HTML5 specification. In particular, it incorporates the algorithmic change [2] to
prevent infinite recursion an element is re-inserted into a document tree inside its constructor.
The key code change is in JSCustomElementInterface::upgradeElement where this patch adds an early exit
when custom element is not "undefined" or "uncustomized" and the custom element state is set to "failed"
immediately before invoking the constructor. The rest of code changes deals with this "failed" state
appearing during upgrades and updates various debug assertions.
[1] https://html.spec.whatwg.org/multipage/custom-elements.html#concept-upgrade-an-element
[2] https://github.com/whatwg/html/pull/5126
[3] https://html.spec.whatwg.org/multipage/custom-elements.html#concept-try-upgrade
Test: imported/w3c/web-platform-tests/custom-elements/upgrading.html
- bindings/js/JSCustomElementInterface.cpp:
(WebCore::JSCustomElementInterface::constructElementWithFallback):
(WebCore::JSCustomElementInterface::upgradeElement): Implements the new behavior. Note that we still
need to clear the queue where we used to set the custom element state to "failed" to avoid memory leaks.
- dom/CustomElementReactionQueue.cpp:
(WebCore::CustomElementReactionQueue::hasJustUpgradeReaction const): Added.
(WebCore::CustomElementReactionQueue::enqueueElementUpgrade): Enqueue the element to the element queue
even if it had already been scheduled to upgrade previously. This makes the innermost attempt to upgrade
to succeed instead of the outermost. Also updated debug assertions.
(WebCore::CustomElementReactionQueue::tryToUpgradeElement): Renamed from enqueueElementUpgradeIfDefined
to match the spec's name [3]. Unlike the concept in the spec, this function doesn't get called when
the custom element state of the elemnt is either "undefined" or "uncustomized" to avoid unnecessary work.
- dom/CustomElementReactionQueue.h:
(WebCore::CustomElementReactionQueue::isEmpty const): Added.
- dom/CustomElementRegistry.cpp:
(WebCore::upgradeElementsInShadowIncludingDescendants):
(WebCore::CustomElementRegistry::upgrade):
- dom/Document.cpp:
(WebCore::createFallbackHTMLElement): Call setIsCustomElementUpgradeCandidate on a newly created since
we can no longer update node flags in enqueueToUpgrade
- dom/Element.cpp:
(WebCore::Element::insertedIntoAncestor):
(WebCore::Element::setIsFailedCustomElement): Removed the unused function argument.
(WebCore::Element::setIsFailedCustomElementWithoutClearingReactionQueue): Extracted from
setIsFailedCustomElement.
(WebCore::Element::clearReactionQueueFromFailedCustomElement): Ditto.
(WebCore::Element::enqueueToUpgrade): No longer updates node flags as this would clear "failed" state
from a custom element which is currently being upgraded and cause all sorts of issues.
(WebCore::Element::reactionQueue const): Updated debug assertions.
- dom/Element.h:
LayoutTests:
Removed the crash expectation from a test now that it's passing.
- 7:32 PM Changeset in webkit [266268] by
-
- 15 edits in trunk/Source
Remove the feature flag for capped cookies set in 3rd-party CNAME cloaked HTTP responses and add ITP debug logging
https://bugs.webkit.org/show_bug.cgi?id=215902
<rdar://problem/66699731>
Reviewed by Brent Fulgham.
This change removes the feature flag, effectively turning the feature on.
It also adds log output of cookie names that have been capped when ITP
Debug Mode is enabled.
Source/WebCore:
WebCore::NetworkStorageSession now has a member flag for ITP debug logging.
No new tests. This feature already has tests.
- page/Settings.yaml:
- platform/network/NetworkStorageSession.cpp:
(WebCore::NetworkStorageSession::setResourceLoadStatisticsDebugLoggingEnabled):
(WebCore::NetworkStorageSession::resourceLoadStatisticsDebugLoggingEnabled const):
New function to be able to conditionalize log output in the new feature.
- platform/network/NetworkStorageSession.h:
Source/WebKit:
- NetworkProcess/Classifier/WebResourceLoadStatisticsStore.cpp:
(WebKit::WebResourceLoadStatisticsStore::setResourceLoadStatisticsDebugMode):
Now sets the flag in WebCore::NetworkStorageSession through the new function
WebCore::NetworkStorageSession::setResourceLoadStatisticsDebugLoggingEnabled().
- NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::resetParametersToDefaultValues):
(WebKit::NetworkProcess::setIsRunningResourceLoadStatisticsTest):
- NetworkProcess/NetworkSession.cpp:
(WebKit::NetworkSession::NetworkSession):
(WebKit::NetworkSession::setFirstPartyHostCNAMEDomain):
(WebKit::NetworkSession::firstPartyHostCNAMEDomain):
- NetworkProcess/NetworkSession.h:
(WebKit::NetworkSession::setCNAMECloakingMitigationEnabled): Deleted.
(WebKit::NetworkSession::cnameCloakingMitigationEnabled const): Deleted.
- NetworkProcess/cocoa/NetworkDataTaskCocoa.mm:
(WebKit::NetworkDataTaskCocoa::updateFirstPartyInfoForSession):
(WebKit::NetworkDataTaskCocoa::applyCookiePolicyForThirdPartyCNAMECloaking):
- Shared/ResourceLoadStatisticsParameters.h:
(WebKit::ResourceLoadStatisticsParameters::encode const):
(WebKit::ResourceLoadStatisticsParameters::decode):
- Shared/WebPreferences.yaml:
- UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::ensureNetworkProcess):
- UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm:
(WebKit::WebsiteDataStore::platformSetNetworkParameters):
- UIProcess/WebsiteData/WebsiteDataStore.cpp:
(WebKit::WebsiteDataStore::parameters):
- 7:00 PM Changeset in webkit [266267] by
-
- 2 edits in trunk/Tools
[webkitpy] repository svn version is incorrect when working on a branch
https://bugs.webkit.org/show_bug.cgi?id=215863
Reviewed by Jonathan Bedard.
When working on a SVN branch, the information pointed in the Revision field
doesn't reflect the last change on the branch, but the last change on trunk.
Instead of picking that changeset, pick the "Last Changed Rev" one that should
reflect the last change done in the current checkout (working directory).
- Scripts/webkitpy/common/checkout/scm/svn.py:
(SVN.svn_revision):
- 6:24 PM Changeset in webkit [266266] by
-
- 2 edits in trunk/Source/WebCore
CSS Filter invert() and opacity() Implementation Should Use Linear Transfer Function for Clarity and Simplicity
https://bugs.webkit.org/show_bug.cgi?id=215896
Simplify CSS filters, invert() and opacity() implementation by using "linear" transfer function,
instead of using a more confusing transfer function, "table". For invert(), the transfer function
is type="table" tableValues="[amount] (1 - [amount])", which is equivalent to a linear function
of slope 1 - 2 * [amount] and intercept [amount]; for opacity the transfer function is type="table" tableValues="0 [amount]"
which is equivalent to a linear transfer function of slope [amount] and intercept 0. This change will make the implementation
clearer and simpler.
Patch by Frank Yang <guowei_yang@apple.com> on 2020-08-27
Reviewed by Myles C. Maxfield.
No new tests needed, using existing tests in css/filter-effects
- rendering/CSSFilter.cpp:
(WebCore::CSSFilter::build): Modified the implementation of invert() and opacity() so that they both use
a linear component transfer function instead of a more confusing table transfer function.
- 6:18 PM Changeset in webkit [266265] by
-
- 22 edits in trunk
[iOS] provide a way to get previously inserted alternatives for the selected text
https://bugs.webkit.org/show_bug.cgi?id=215816
<rdar://problem/66646042>
Reviewed by Darin Adler.
Source/WebCore:
- editing/cocoa/AlternativeTextUIController.h:
- editing/cocoa/AlternativeTextUIController.mm:
(WebCore::AlternativeTextUIController::alternativesForContext):
Return the rawNSTextAlternatives *
and let the caller create theVector<String>
if they
need to so that callers that don't can use the actualNSTextAlternatives *
.
- editing/Editor.h:
- editing/Editor.cpp:
(WebCore::Editor::applyDictationAlternative): Added.
(WebCore::Editor::applyDictationAlternativelternative): Deleted.
- page/ContextMenuController.cpp:
(WebCore::ContextMenuController::contextMenuItemSelected):
Drive-by: fix typo.
Source/WebKit:
- UIProcess/ios/WKContentViewInteraction.mm:
(-[WKContentView alternativesForSelectedText]):
- UIProcess/WebPageProxy.h:
- UIProcess/Cocoa/WebPageProxyCocoa.mm:
(WebKit::WebPageProxy::platformDictationAlternatives): Added.
- UIProcess/PageClient.h:
- UIProcess/Cocoa/PageClientImplCocoa.h:
- UIProcess/Cocoa/PageClientImplCocoa.mm:
(WebKit::PageClientImplCocoa::dictationAlternatives):
(WebKit::PageClientImplCocoa::platformDictationAlternatives): Added.
Provide a way to get the rawNSTextAlternatives *
for a givenWebCore::DictationContext
.
- Shared/EditorState.h:
- Shared/EditorState.cpp:
(WebKit::EditorState::PostLayoutData::encode const):
(WebKit::EditorState::PostLayoutData::decode):
- WebProcess/WebPage/ios/WebPageIOS.mm:
(WebKit::WebPage::getPlatformEditorState const):
Include aVector<WebCore::DictationContext>
as part of theEditorState
that contains all
of theWebCore::DictationContext
that exist in the currently selected range (or the range
of the word containing the cursor if nothing is selected).
- Platform/spi/ios/UIKitSPI.h:
Source/WebKitLegacy/mac:
- WebView/WebView.mm:
(-[WebView _dictationAlternatives:]):
Create aVector<String>
from the returnedNSTextAlternatives *
now that the member
WebCore::AlternativeTextUIController::alternativesForContext
returns it.
Tools:
- TestWebKitAPI/Tests/WebKitCocoa/InsertTextAlternatives.mm:
(InsertTextAlternatives.Simple):
- TestWebKitAPI/ios/UIKitSPI.h:
- 5:27 PM Changeset in webkit [266264] by
-
- 14 edits1 add in trunk
proto in object literal should perform SetPrototypeOf? directly
https://bugs.webkit.org/show_bug.cgi?id=215769
Reviewed by Ross Kirsling.
JSTests:
- microbenchmarks/object-literal-underscore-proto-setter.js: Added.
- stress/syntax-checker-duplicate-underscore-proto.js:
Rewrite the test to ensure each eval() call throws a SyntaxError.
Source/JavaScriptCore:
To fix proto usage in object literals if Object.prototype.proto is overridden
or removed, this patch sets the Prototype? directly, aligning JSC with V8 and
SpiderMonkey. We are safe to skip method table lookups and cycle checks, as the
spec [1] calls SetPrototypeOf? on newly created (unreferenced) ordinary objects.
This change removes PropertyNode::PutType because its only purpose was to accomodate
proto in object literals. Since emitPutConstantProperty() handles static public
class fields, which don't needsuper
binding, PropertyNode::isUnderscoreProtoSetter()
is extended to reject class properties.
This patch speeds up creating object literals with proto by 25%.
[1]: https://tc39.es/ecma262/#sec-__proto__-property-names-in-object-initializers (step 7.a)
- bytecompiler/BytecodeGenerator.cpp:
(JSC::BytecodeGenerator::emitDirectPutById):
(JSC::BytecodeGenerator::emitDirectSetPrototypeOf):
- Remove unused
dst
parameter to align with otherput
methods. - Remove
divot*
parameters as it's cumbersome to pass them through, and globalFuncSetPrototypeDirect() never throws anyway.
- bytecompiler/BytecodeGenerator.h:
- bytecompiler/NodesCodegen.cpp:
(JSC::PropertyListNode::emitPutConstantProperty):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_putByIdDirect):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_putByIdDirectPrivate):
(JSC::ClassExprNode::emitBytecode):
- parser/ASTBuilder.h:
(JSC::ASTBuilder::createGetterOrSetterProperty):
(JSC::ASTBuilder::createProperty):
(JSC::ASTBuilder::isUnderscoreProtoSetter const):
- parser/NodeConstructors.h:
(JSC::PropertyNode::PropertyNode):
- parser/Nodes.h:
- parser/Parser.cpp:
(JSC::Parser<LexerType>::parseClass):
(JSC::Parser<LexerType>::parseProperty):
- parser/SyntaxChecker.h:
(JSC::SyntaxChecker::createProperty):
- runtime/JSGlobalObjectFunctions.cpp:
(JSC::globalFuncSetPrototypeDirect):
- Ignore a prototype value of incorrect type as per spec [1], which is unobservable for call sites in ClassExprNode::emitBytecode().
- Assert that JSObject::setPrototypeDirect() doesn't throw.
LayoutTests:
- js/script-tests/object-literal-direct-put.js:
- 4:48 PM Changeset in webkit [266263] by
-
- 3 edits in trunk/Source/WebCore
AudioParam does not need to store its values as double
https://bugs.webkit.org/show_bug.cgi?id=215903
Reviewed by Darin Adler.
AudioParam does not need to store its values as double. float precision is sufficient for
Web Audio values and we end up casting these values to float anyway.
- Modules/webaudio/AudioParam.cpp:
(WebCore::AudioParam::AudioParam):
(WebCore::AudioParam::value):
(WebCore::AudioParam::smoothedValue):
(WebCore::AudioParam::smooth):
(WebCore::AudioParam::calculateFinalValues):
(WebCore::AudioParam::calculateTimelineValues):
- Modules/webaudio/AudioParam.h:
- 4:47 PM Changeset in webkit [266262] by
-
- 4 edits2 adds in trunk
Scrolling on select element doesn't work after scrolling the page
https://bugs.webkit.org/show_bug.cgi?id=215900
Source/WebCore:
<rdar://problem/67766032>
Reviewed by Tim Horton.
The scrolling thread could use a node that was latched some time ago; we need to
check the freshness of the latched node before using it.
Test: fast/scrolling/latching/latching-stuck-to-main-page.html
- page/scrolling/ScrollingTreeLatchingController.cpp:
(WebCore::ScrollingTreeLatchingController::receivedWheelEvent):
(WebCore::ScrollingTreeLatchingController::latchedNodeForEvent const):
(WebCore::ScrollingTreeLatchingController::latchedNodeIsRelevant const):
- page/scrolling/ScrollingTreeLatchingController.h:
LayoutTests:
Reviewed by Tim Horton.
- fast/scrolling/latching/latching-stuck-to-main-page-expected.txt: Added.
- fast/scrolling/latching/latching-stuck-to-main-page.html: Added.
- 4:43 PM Changeset in webkit [266261] by
-
- 11 edits in trunk
Fix AudioParam.linearRampToValueAtTime() formula to match specification
https://bugs.webkit.org/show_bug.cgi?id=215904
Reviewed by Darin Adler.
LayoutTests/imported/w3c:
Rebaseline WPT tests now that more checks are passing or now that they keep passing/failing with
slightly different values.
- web-platform-tests/webaudio/the-audio-api/the-audioparam-interface/audioparam-linearRampToValueAtTime-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-audioparam-interface/k-rate-audiobuffersource-connections-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-audioparam-interface/k-rate-dynamics-compressor-connections-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-audioparam-interface/k-rate-panner-connections-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-audioparam-interface/k-rate-panner-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-audioparam-interface/retrospective-linearRampToValueAtTime-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-biquadfilternode-interface/biquad-automation-expected.txt:
- web-platform-tests/webaudio/the-audio-api/the-pannernode-interface/panner-automation-position-expected.txt:
Source/WebCore:
Fix AudioParam.linearRampToValueAtTime() formula to match specification:
We were using a different formula to compute the value and this was causing us
to fail some web platform tests.
No new tests, rebaselined existing tests.
- Modules/webaudio/AudioParamTimeline.cpp:
(WebCore::AudioParamTimeline::valuesForTimeRangeImpl):
- 3:18 PM Changeset in webkit [266260] by
-
- 2 edits in trunk/Tools
Ensure that email notifications are not sent for obsolete and r- patches
https://bugs.webkit.org/show_bug.cgi?id=215867
Reviewed by Jonathan Bedard.
- BuildSlaveSupport/ews-build/steps.py:
(BugzillaMixin.should_send_email): Method to check if patch is obsolete/r-.
(AnalyzeCompileWebKitResults): Inherit from BugzillaMixin.
(AnalyzeCompileWebKitResults.send_email_for_new_build_failure): Check for patch state before sending email.
(AnalyzeLayoutTestsResults): Ditto.
(AnalyzeLayoutTestsResults.send_email_for_new_test_failures): Ditto.
- 3:15 PM Changeset in webkit [266259] by
-
- 1 copy in tags/Safari-610.2.2.4
Tag Safari-610.2.2.4.
- 2:48 PM Changeset in webkit [266258] by
-
- 8 edits in branches/safari-610.2.2-branch/Source
Versioning.
WebKit-7610.2.2.4
- 2:32 PM Changeset in webkit [266257] by
-
- 3 edits1 add in trunk
[JSC] setLength in Array#push could get very large length
https://bugs.webkit.org/show_bug.cgi?id=215897
<rdar://problem/67859149>
Reviewed by Keith Miller.
JSTests:
- stress/array-push-more-than-max-size.js: Added.
(shouldBe):
(shouldThrow):
Source/JavaScriptCore:
Array#push can get length larger than UINT32_MAX. And in this case, we should throw a RangeError.
Before r266215, it was using putLength which throws an error. But it was replaced with setLength,
and JSC::setLength assumes that it never gets a length greater than UINT32_MAX by asserting. We
should fix it so that Array#push should thrown an error correctly.
- runtime/ArrayPrototype.cpp:
(JSC::setLength):
- 2:31 PM Changeset in webkit [266256] by
-
- 3 edits in trunk/Tools
Send ews email notifications to myself
https://bugs.webkit.org/show_bug.cgi?id=215815
Reviewed by Jonathan Bedard.
- BuildSlaveSupport/ews-build/send_email.py:
(send_email_to_patch_author): Send a copy of email notification to myself.
- BuildSlaveSupport/ews-build/master.cfg: Stop receiveing old style email notifications for build failures.
- 2:28 PM Changeset in webkit [266255] by
-
- 2 edits6 adds in trunk/Tools
[webkitscmpy] Initial commit
https://bugs.webkit.org/show_bug.cgi?id=215864
<rdar://problem/67827063>
Reviewed by Dewei Zhu.
- Scripts/libraries/webkitscmpy: Added.
- Scripts/libraries/webkitscmpy/MANIFEST.in: Added.
- Scripts/libraries/webkitscmpy/README.md: Added.
- Scripts/libraries/webkitscmpy/setup.py: Added.
(readme):
- Scripts/libraries/webkitscmpy/webkitscmpy: Added.
- Scripts/libraries/webkitscmpy/webkitscmpy/init.py: Added.
(_maybe_add_webkitcorepy_path):
- Scripts/webkitpy/test/main.py:
(main): Add webkitscmpy to test-webkitpy.
- 2:16 PM Changeset in webkit [266254] by
-
- 3 edits1 add in trunk
GetByVal constant folding over a Double OutOfBoundsSaneChain array with no BytecodeUsesAsOther should constant fold to PNaN, not undefined
https://bugs.webkit.org/show_bug.cgi?id=215894
<rdar://problem/67669696>
Reviewed by Michael Saboff and Keith Miller.
JSTests:
- stress/oob-sane-chain-double-constant-fold-to-PNaN.js: Added.
Source/JavaScriptCore:
GetByVals of the form { OutOfBoundsSaneChain, Double } where there are no
BytecodeUsesAsOther return PNaN for holes and OOB accesses, not jsUndefined().
The constant folding for this though was folding to jsUndefined(). I forgot
to update that code to constant fold to PNaN when I wrote the OutOfBoundsSaneChain
implementation.
- dfg/DFGAbstractInterpreterInlines.h:
(JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects):
- 2:10 PM Changeset in webkit [266253] by
-
- 9 edits6 adds in trunk
Implement @supports selector().
https://bugs.webkit.org/show_bug.cgi?id=199237
Reviewed by Antti Koivisto.
This feature allows authors to test if the browser supports the tested selector syntax.
The corresponding spec is https://drafts.csswg.org/css-conditional-4/#at-supports-ext.
And unknown -webkit- pseudo elements are not supported according to the spec,
https://drafts.csswg.org/css-conditional-4/#support-definition-ext.
LayoutTests/imported/w3c:
- web-platform-tests/css/cssom/CSS-expected.txt:
Source/WebCore:
Tests: css3/conditional/w3c/at-supports-040.html
css3/conditional/w3c/at-supports-041.html
css3/conditional/w3c/at-supports-042.html
- css/CSSValueKeywords.in:
- css/parser/CSSParserImpl.h:
(WebCore::CSSParserImpl::context const):
- css/parser/CSSSelectorParser.cpp:
(WebCore::CSSSelectorParser::supportsComplexSelector):
(WebCore::CSSSelectorParser::containsUnknownWebKitPseudoElements):
- css/parser/CSSSupportsParser.cpp:
(WebCore::CSSSupportsParser::supportsCondition):
(WebCore::CSSSupportsParser::consumeCondition):
(WebCore::CSSSupportsParser::consumeNegation):
(WebCore::CSSSupportsParser::consumeSupportsFeatureOrGeneralEnclosed):
(WebCore::CSSSupportsParser::consumeSupportsSelectorFunction):
(WebCore::CSSSupportsParser::consumeConditionInParenthesis):
(WebCore::CSSSupportsParser::consumeDeclarationConditionOrGeneralEnclosed): Deleted.
- css/parser/CSSSupportsParser.h:
LayoutTests:
- css3/conditional/w3c/at-supports-040-expected.html: Added.
- css3/conditional/w3c/at-supports-040.html: Added.
- css3/conditional/w3c/at-supports-041-expected.html: Added.
- css3/conditional/w3c/at-supports-041.html: Added.
- css3/conditional/w3c/at-supports-042-expected.html: Added.
- css3/conditional/w3c/at-supports-042.html: Added.
- 2:02 PM Changeset in webkit [266252] by
-
- 4 edits in trunk/Source/WebCore
ScrollLatchingState::previousWheelScrolledElement() is unused
https://bugs.webkit.org/show_bug.cgi?id=215881
Reviewed by Zalan Bujtas.
Scroll latching's previousWheelScrolledElement() was unused. It was never set since
we never end up with a null wheelEventElement. This functionality has been retained
from ancient code with little understanding of its utility.
- page/EventHandler.cpp:
(WebCore::handleWheelEventInAppropriateEnclosingBox):
(WebCore::EventHandler::defaultWheelEventHandler):
- page/scrolling/ScrollLatchingState.cpp:
(WebCore::ScrollLatchingState::clear):
(WebCore::operator<<):
(WebCore::ScrollLatchingState::setPreviousWheelScrolledElement): Deleted.
- page/scrolling/ScrollLatchingState.h:
- 1:11 PM Changeset in webkit [266251] by
-
- 5 edits in trunk/Source/JavaScriptCore
structureOrNull should take VM instead of getting it from the marked block
https://bugs.webkit.org/show_bug.cgi?id=215899
Reviewed by Yusuke Suzuki.
It's slightly faster use an existing VM over recomputing the address. It probably doesn't
happen to matter here for performance but it's good hygiene.
- API/tests/JSWrapperMapTests.mm:
(+[JSWrapperMapTests testStructureIdentity]):
- jit/JITOperations.cpp:
- runtime/JSCJSValue.h:
- runtime/JSCJSValueInlines.h:
(JSC::JSValue::structureOrNull const):
(JSC::JSValue::structureOrUndefined const): Deleted.
- 1:01 PM Changeset in webkit [266250] by
-
- 4 edits in trunk/Source/JavaScriptCore
[JSC] Use auxiliary memory for JSBigInt storage
https://bugs.webkit.org/show_bug.cgi?id=215876
Reviewed by Mark Lam.
This makes JSBigInt non-destructible cell. And it makes allocating JSBigInt from JIT easy.
- runtime/JSBigInt.cpp:
(JSC::JSBigInt::JSBigInt):
(JSC::JSBigInt::visitChildren):
(JSC::JSBigInt::createWithLength):
(JSC::JSBigInt::destroy): Deleted.
- runtime/JSBigInt.h:
- runtime/VM.cpp:
(JSC::VM::VM):
- 12:52 PM Changeset in webkit [266249] by
-
- 4 edits in trunk
[PlayStation] Build fix due to memmem not always being available.
https://bugs.webkit.org/show_bug.cgi?id=215893
Unreviewed build fix
.:
- Source/cmake/OptionsPlayStation.cmake: Check for memmem
Source/WTF:
- wtf/PlatformHave.h: Don't always HAVE_MEMMEM on playstation
- 11:56 AM Changeset in webkit [266248] by
-
- 7 edits2 adds in trunk
Occasional crashes when restoring replaced text under Editor::changeBackToReplacedString
https://bugs.webkit.org/show_bug.cgi?id=215892
<rdar://problem/67731156>
Reviewed by Tim Horton.
Source/WebCore:
While reverting the replaced string in
Editor::changeBackToReplacedString
, it's possible for text replacement
(Editor::replaceSelectionWithText
) to cause the text checking paragraph range to become orphaned (if for no
reason other than the fact that arbitrary script can run in between). If this happens, then our attempts to
expand the checking range to paragraph boundaries underneathTextCheckingParagraph::paragraphRange
will result
in a null dereference, since the boundary points computed underneath theexpandToParagraphBoundary
helper will
benullopt
.
Mitigate this (and any other potentially similar crashes) by making
expandToParagraphBoundary
robust in the
case where expanding to the start and end fails, and just fall back to returning the original text checking
range instead.
Test: editing/mac/input/change-back-to-replaced-string.html
- editing/Editor.h:
- editing/TextCheckingHelper.cpp:
(WebCore::expandToParagraphBoundary):
See above for more details.
- testing/Internals.cpp:
(WebCore::Internals::changeBackToReplacedString):
- testing/Internals.h:
- testing/Internals.idl:
Add a simple internal testing hook to change the currently selected text back to the given replaced string.
LayoutTests:
Add a layout test to exercise the crash.
- editing/mac/input/change-back-to-replaced-string-expected.txt: Added.
- editing/mac/input/change-back-to-replaced-string.html: Added.
- 11:06 AM Changeset in webkit [266247] by
-
- 5 edits4 adds in branches/safari-610.1.28.1-branch
Apply patch. rdar://problem/67877085
- 11:02 AM Changeset in webkit [266246] by
-
- 8 edits in branches/safari-610.1.28.1-branch/Source
Versioning.
WebKit-7610.1.28.1.5
- 10:28 AM Changeset in webkit [266245] by
-
- 3 edits2 adds in trunk
:host() pseudo-selector reported as :host in CSSStyleRule
https://bugs.webkit.org/show_bug.cgi?id=207567
<rdar://problem/59365547>
Reviewed by Simon Fraser.
Source/WebCore:
Test case by jlo@logrocket.com
Test: fast/shadow-dom/css-scoping-host-serialization.html
- css/CSSSelector.cpp:
(WebCore::CSSSelector::selectorText const):
Serialize the function form of :host.
LayoutTests:
- fast/shadow-dom/css-scoping-host-serialization-expected.txt: Added.
- fast/shadow-dom/css-scoping-host-serialization.html: Added.
- 10:17 AM Changeset in webkit [266244] by
-
- 4 edits in trunk
REGRESSION (r264790): IndexedDB may abort transactions of in-memory databases
https://bugs.webkit.org/show_bug.cgi?id=215855
<rdar://problem/67418574>
Reviewed by Youenn Fablet.
Source/WebCore:
We don't abort transactions on in-memory database before process suspension because they don't hold database
file lock. Before r264790, only ephemeral session would use in-memory databases, so we only checked session. Now
persistent session also uses in-memory databases for third-party storage, so we need to add another check.
API test: IndexedDB.SuspendImminentlyForThirdPartyDatabases
- Modules/indexeddb/server/IDBServer.cpp:
(WebCore::IDBServer::IDBServer::stopDatabaseActivitiesOnMainThread):
Tools:
- TestWebKitAPI/Tests/WebKitCocoa/IndexedDBSuspendImminently.mm:
(postResult):
(catch):
- 10:13 AM Changeset in webkit [266243] by
-
- 9 edits in trunk
Fix propagation of errors in TransformStream
https://bugs.webkit.org/show_bug.cgi?id=215888
Reviewed by Alex Christensen.
LayoutTests/imported/w3c:
- web-platform-tests/streams/transform-streams/backpressure.any-expected.txt:
- web-platform-tests/streams/transform-streams/errors.any-expected.txt:
- web-platform-tests/streams/transform-streams/errors.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/strategies.any-expected.txt:
- web-platform-tests/streams/transform-streams/strategies.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/terminate.any-expected.txt:
Source/WebCore:
We were not passing the right values to the reject function of the promise capability.
Covered by rebased tests.
- Modules/streams/TransformStreamInternals.js:
(transformStreamDefaultControllerPerformTransform):
(transformStreamDefaultSinkWriteAlgorithm):
(transformStreamDefaultSinkCloseAlgorithm):
- 10:10 AM Changeset in webkit [266242] by
-
- 9 edits in trunk/Source/JavaScriptCore
OSR availability validation should run for any node with exitOK
https://bugs.webkit.org/show_bug.cgi?id=215672
Reviewed by Saam Barati.
Currently we only validate OSR exit availability if a node would
saymayExit(graph, node) != DoesNotExit
and the node is marked
as exitOK. However, it would be perfectly valid to insert a node
that exits anywhere we have a node marked exitOK. So with this
patch we now validate all places where it would ever be possible
to OSR exit.
Relaxing our criteria revealed a number of bugs however. Which I
will describe below in, IMO, increasing complexity/subtly.
First, we currently don't mark arity fixup during inlining as not
exitOK. However, since our arity code says its code origin is
OpEnter, we assume arity fixup has already happened.
Second, OpGetScope, should not mark its first argument as used
since it's not actually used. This is problematic because we could
have a loop where OpGetScope is the first bytecode, namely when
doing tail recursive inlining. If we were in that position, there
could be a local that was used at a merge point at the loop
backedge that had two MovHint defs from both predecessors. In DFG
IR this would look like:
BB#1:
@1: MovHint(Undefined, loc1)
...
Jump(#2)
BB#2:
... loc1 is live here in bytecode
@2: MovHint(@scopeObject, loc1)
@3: SetLocal(@scopeObject, loc1)
Branch(#3, #4) #4 is the successor of the tail call loop
BB#3:
@4 MovHint(Undefined, loc1)
...
Jump(#2)
When we do CPS conversion the MovHints at @1 and @4 will be seen
as different variables (there's no GetLocal). Then, after, during
SSA conversion we won't insert a phi connecting them, making the
argument to OpGetScope, in this case loc1, unrecoverable there are
conflicting nodes and the value isn't saved on the stack.
There were also issues with MovHintRemoval Phase but rather than
fix them we opted to just remove the phase as it didn't show any
performance impact. I'll describe the issues I found below for
completeness, however.
Third, MovHint removal phase had a bug where it would not mark
sections where a zombied MovHint has yet to be killed as not
exitOK. So in theory another phase could come along and insert an
exiting node there.
Fourth, MovHint removal phase had a second bug where a MovHint
that was not killed in the current block would be zombied, which
is wrong for SSA. It's wrong because the MovHinted value could
still be live for OSR exit in a successor block.
Lastly, this patch adds some new verbose options as well as the ability to
dump a DFG::BasicBlock without dereferencing it.
- bytecode/BytecodeUseDef.cpp:
(JSC::computeUsesForBytecodeIndexImpl):
- dfg/DFGBasicBlock.cpp:
(WTF::printInternal):
- dfg/DFGBasicBlock.h:
- dfg/DFGByteCodeParser.cpp:
(JSC::DFG::ByteCodeParser::inlineCall):
- dfg/DFGCPSRethreadingPhase.cpp:
(JSC::DFG::CPSRethreadingPhase::propagatePhis):
- dfg/DFGEpoch.h:
(JSC::DFG::Epoch::operator bool const):
- dfg/DFGOSRAvailabilityAnalysisPhase.cpp:
(JSC::DFG::OSRAvailabilityAnalysisPhase::run):
- dfg/DFGSSACalculator.cpp:
(JSC::DFG::SSACalculator::dump const):
- 10:02 AM Changeset in webkit [266241] by
-
- 5 edits2 adds in trunk
REGRESSION (r263506): timing of CSS Animation on https://animate.style is incorrect
https://bugs.webkit.org/show_bug.cgi?id=215807
<rdar://problem/66770136>
Reviewed by Simon Fraser.
Source/WebCore:
Test: webanimations/accelerated-css-animation-with-easing.html
In r263506, we added a way for accelerated animations to adhere to both a timing function set on the
animation, affecting the timing of the entire animation, as well as a timing function set on individual
keyframes, affecting the timing of the animation in a given interval. Alas, this change broke handling
of timing functions with implicit animation-timing-function on keyframes.
That code change assumed that the Animation object ultimately passed to GraphicsLayerCA::setupAnimation()
would return the animation-wide timing function through Animation::timingFunction(). This was correct for
JS-originated animations, but not for CSS Animations, since that value was set based on the computed CSS
property animation-timing-function for the target element. However, that CSS property does not specify
the animation-wide timing function, but rather the default timing function to use for keyframes should
they fail to specify an explicit timing function.
To fix this, first, we ensure that the animation-wide timing function is set on the Animation object,
changing KeyframeEffect::backingAnimationForCompositedRenderer() to always generate an Animation object
based on the effect, divorcing itself from the Animation object created through CSS.
Then, we add a new member to Animation to specify the default timing function for keyframes, allowing
GraphicsLayerCA to use it to determine the timing function when building keyframes.
- animation/KeyframeEffect.cpp:
(WebCore::KeyframeEffect::backingAnimationForCompositedRenderer const): Always generate an Animation
object based on the effect's properties, also setting the new defaultTimingFunctionForKeyframes in the
case of a CSS Animation object.
- platform/animation/Animation.h:
(WebCore::Animation::defaultTimingFunctionForKeyframes const):
(WebCore::Animation::setDefaultTimingFunctionForKeyframes):
- platform/graphics/ca/GraphicsLayerCA.cpp:
(WebCore::animationHasStepsTimingFunction): Use the optional defaultTimingFunctionForKeyframes to determine
whether a keyframe uses a steps timing function.
(WebCore::GraphicsLayerCA::timingFunctionForAnimationValue): Use the optional defaultTimingFunctionForKeyframes
to determine the timing function for a keyframe.
LayoutTests:
Add a new test that checks that a CSS Animation with implicit and explicit animation-timing-function
values behave the same when running accelerated.
- webanimations/accelerated-css-animation-with-easing-expected.html: Added.
- webanimations/accelerated-css-animation-with-easing.html: Added.
- 10:01 AM Changeset in webkit [266240] by
-
- 2 edits in trunk/Source/WebKit
REGRESSION(r264710): Initializing the AVPlayer Obj-C class at process start up causes a regression in power-use tests
https://bugs.webkit.org/show_bug.cgi?id=215861
Reviewed by Eric Carlson.
Calling +instancesRespondToSelector: will cause the underyling Obj-C class to be initialized, which in the case of
AVPlayer does a bunch of work which shows up on power-use tests. Because the API being checked for will always exist
in macOS 11, and that code is already protected by a HAVE(AVPLAYER_VIDEORANGEOVERRIDE) pragma, just return early
here without running the +instancesRespondToSelector: check.
- WebProcess/cocoa/WebProcessCocoa.mm:
(WebKit::WebProcess::updatePageScreenProperties):
- 9:55 AM Changeset in webkit [266239] by
-
- 3 edits3 adds in trunk
REGRESSION (r258215): Title preview movie isn't displayed at www.thismmalife.com
https://bugs.webkit.org/show_bug.cgi?id=215774
<rdar://problem/67707957>
Reviewed by Eric Carlson.
Source/WebCore:
Test: media/video-as-img-output-pts.html
Use CMSampleBufferGetOutputPresentationTimeStamp() for sample PTS.
- platform/graphics/avfoundation/objc/ImageDecoderAVFObjC.mm:
(WebCore::ImageDecoderAVFObjC::storeSampleBuffer):
LayoutTests:
- media/content/video-as-img.mp4: Added.
- media/video-as-img-output-pts-expected.txt: Added.
- media/video-as-img-output-pts.html: Added.
- 9:48 AM Changeset in webkit [266238] by
-
- 2 edits in trunk/Source/WebKit
Unreviewed, add missing double quote in r266216.
- WebProcess/com.apple.WebProcess.sb.in:
- 9:46 AM Changeset in webkit [266237] by
-
- 3 edits in trunk/Source/WebKit
syntax error in "DELETE FROM StorageAccessUnderTopFrameDomains"
https://bugs.webkit.org/show_bug.cgi?id=215813
<rdar://problem/67743521>
Reviewed by Darin Adler.
Delete unnecessary code. ResourceLoadStatisticsDatabaseStore::clearExpiredUserInteractions()
is not needed. User interaction and storage access will be cleared for
expired statistics in
ResourceLoadStatisticsDatabaseStore::hasHadUnexpiredRecentUserInteraction(),
which exactly matches behavior in the memory store. If we clear the
expired statistics before setting the local var oldestUserInteraction,
then that var will likely always be equal to zero, and the check for
if (!parameters().isRunningTest && now - oldestUserInteraction < parameters().minimumTimeBetweenDataRecordsRemoval)
will always be false.
The only reason we did not notice this extra code is because the
SQLite query in clearExpiredUserInteractions() was incorrect, causing
an early return. So the function never worked properly. Fixing the
query revealed incorrect behavior.
No new tests, behavior should not change because this code was not
being run due to the SQLite error.
- NetworkProcess/Classifier/ResourceLoadStatisticsDatabaseStore.cpp:
(WebKit::ResourceLoadStatisticsDatabaseStore::registrableDomainsToDeleteOrRestrictWebsiteDataFor):
(WebKit::ResourceLoadStatisticsDatabaseStore::findExpiredUserInteractions const): Deleted.
(WebKit::ResourceLoadStatisticsDatabaseStore::clearExpiredUserInteractions): Deleted.
(WebKit::ResourceLoadStatisticsDatabaseStore::statisticsExpirationTime const): Deleted.
- 9:41 AM Changeset in webkit [266236] by
-
- 5 edits in trunk/Source/JavaScriptCore
JSClassRef should work with JS class syntax.
https://bugs.webkit.org/show_bug.cgi?id=215047
Reviewed by Darin Adler.
This is done by checking if value returned by the
callAsConstructor parameter to JSObjectMakeConstructor returns an
object allocated as the jsClass parameter. When that happens we
replace the prototype of the returned object with the prototype of
the new.target. Ideally we would have passed the derived classes
constructor from the beginning of our support for JS subclassing
but at this point that's probably not compatible with too many
applications.
- API/APICallbackFunction.h:
(JSC::APICallbackFunction::construct):
- API/JSObjectRef.h:
- API/tests/testapi.cpp:
(APIString::APIString):
(TestAPI::markedJSValueArrayAndGC):
(TestAPI::classDefinitionWithJSSubclass):
(testCAPIViaCpp):
- API/tests/testapi.mm:
(testObjectiveCAPI):
- 9:37 AM Changeset in webkit [266235] by
-
- 4 edits in trunk/Source/WebCore
Remove startedGestureAtScrollLimit which is always false.
- 8:54 AM Changeset in webkit [266234] by
-
- 3 edits in trunk/Source/WebCore
Unreviewed, address review comments for r266221 that I missed before landing.
- Modules/webaudio/AudioScheduledSourceNode.cpp:
(WebCore::AudioScheduledSourceNode::updateSchedulingInfo):
- platform/audio/AudioUtilities.cpp:
(WebCore::AudioUtilities::timeToSampleFrame):
- 8:50 AM Changeset in webkit [266233] by
-
- 2 edits in trunk/LayoutTests
[GLIB] Skip some tests causing the suite to hang up
Unreviewed test gardening.
These two tests were making the entire suite to hang after r266221.
- platform/glib/TestExpectations:
- 8:42 AM Changeset in webkit [266232] by
-
- 4 edits in trunk
Step animations invalidate style on every rendering update whether or not they need to
https://bugs.webkit.org/show_bug.cgi?id=215229
<rdar://problem/66636153>
Reviewed by Antoine Quint.
Source/WebCore:
Step timing functions with transforms try and fail to start accelerated which causes them to repeatedly schedule unnecessary rendering updates.
- animation/KeyframeEffect.cpp:
(WebCore::KeyframeEffect::updateAcceleratedActions):
Step timing functions are never accelerated so don't bother trying to start them in accelerated state.
This is similar to treatmeant of unaccelerated properties.
LayoutTests:
- animations/steps-transform-rendering-updates-expected.txt:
- 6:42 AM Changeset in webkit [266231] by
-
- 8 edits in trunk/Source/WebCore
[LFC][IFC] Move LineBox out of LineBuilder
https://bugs.webkit.org/show_bug.cgi?id=215048
Reviewed by Antti Koivisto.
Since none of the LineBox related operations (content alignment, line sizing etc) is done during line building,
we don't really need to construct a LineBox until after we finished placing all the runs on the current line.
It enables us to decouple the LineBuilder and the LineBox.
Let the LineBuilder place the runs and compute horizontal geometry, while LineBox takes care of the final alignment of those runs.
- layout/inlineformatting/InlineFormattingContext.cpp:
(WebCore::Layout::LineContentAligner::adjustBaselineAndLineHeight):
(WebCore::Layout::InlineFormattingContext::lineLayout):
(WebCore::Layout::InlineFormattingContext::computedIntrinsicWidthForConstraint const):
(WebCore::Layout::InlineFormattingContext::constraintsForLine):
(WebCore::Layout::InlineFormattingContext::setDisplayBoxesForLine):
- layout/inlineformatting/InlineFormattingContext.h:
- layout/inlineformatting/InlineFormattingContextQuirks.cpp:
(WebCore::Layout::InlineFormattingContext::Quirks::initialLineHeight const):
- layout/inlineformatting/InlineLineBox.h:
(WebCore::Layout::LineBox::halfLeadingMetrics):
- layout/inlineformatting/InlineLineBuilder.cpp:
(WebCore::Layout::LineBuilder::open):
(WebCore::Layout::LineBuilder::clearContent):
(WebCore::Layout::LineBuilder::close):
(WebCore::Layout::LineBuilder::removeTrailingTrimmableContent):
(WebCore::Layout::LineBuilder::visuallyCollapsePreWrapOverflowContent):
(WebCore::Layout::LineBuilder::moveLogicalLeft):
(WebCore::Layout::LineBuilder::appendWith):
(WebCore::Layout::LineBuilder::appendNonBreakableSpace):
(WebCore::Layout::LineBuilder::appendInlineContainerEnd):
(WebCore::Layout::LineBuilder::appendTextContent):
(WebCore::Layout::LineBuilder::appendNonReplacedInlineBox):
(WebCore::Layout::LineBuilder::isRunVisuallyNonEmpty const):
(WebCore::Layout::LineBuilder::initialize): Deleted.
(WebCore::Layout::LineBuilder::clear): Deleted.
(WebCore::Layout::LineBuilder::isVisuallyNonEmpty const): Deleted.
(WebCore::Layout::LineBuilder::halfLeadingMetrics): Deleted.
- layout/inlineformatting/InlineLineBuilder.h:
(WebCore::Layout::LineBuilder::setHasIntrusiveFloat):
(WebCore::Layout::LineBuilder::hasIntrusiveFloat const):
(WebCore::Layout::LineBuilder::isVisuallyEmpty const):
(WebCore::Layout::LineBuilder::contentLogicalWidth const):
(WebCore::Layout::LineBuilder::contentLogicalRight const):
(WebCore::Layout::LineBuilder::lineBox const): Deleted.
(WebCore::Layout::LineBuilder::lineBox): Deleted.
- layout/inlineformatting/LineLayoutContext.cpp:
(WebCore::Layout::LineLayoutContext::layoutInlineContent):
(WebCore::Layout::LineLayoutContext::commitFloats):
(WebCore::Layout::LineLayoutContext::rebuildLine):
- 3:44 AM Changeset in webkit [266230] by
-
- 89 edits27 deletes in trunk/LayoutTests
Remove console logs from expected.txt files for some WebSocket tests
https://bugs.webkit.org/show_bug.cgi?id=215848
Reviewed by Alex Christensen.
Some WebSocket tests show platform specific console log messages.
By removing them, we can keep one expected.txt file for all platforms.
- http/tests/websocket/connection-refusal-in-frame-resource-load-statistics-expected.txt:
- http/tests/websocket/connection-refusal-in-frame-resource-load-statistics.html:
- http/tests/websocket/tests/hybi/bad-handshake-crash-expected.txt:
- http/tests/websocket/tests/hybi/bad-handshake-crash.html:
- http/tests/websocket/tests/hybi/broken-utf8-expected.txt:
- http/tests/websocket/tests/hybi/broken-utf8.html:
- http/tests/websocket/tests/hybi/close-code-and-reason-expected.txt:
- http/tests/websocket/tests/hybi/close-code-and-reason.html:
- http/tests/websocket/tests/hybi/closed-port-delay-expected.txt:
- http/tests/websocket/tests/hybi/closed-port-delay.html:
- http/tests/websocket/tests/hybi/compressed-control-frame-expected.txt:
- http/tests/websocket/tests/hybi/compressed-control-frame.html:
- http/tests/websocket/tests/hybi/contentextensions/upgrade-expected.txt:
- http/tests/websocket/tests/hybi/contentextensions/upgrade-worker-expected.txt:
- http/tests/websocket/tests/hybi/contentextensions/upgrade-worker.html:
- http/tests/websocket/tests/hybi/contentextensions/upgrade.html:
- http/tests/websocket/tests/hybi/deflate-frame-comp-bit-onoff.html:
- http/tests/websocket/tests/hybi/deflate-frame-invalid-parameter-expected.txt:
- http/tests/websocket/tests/hybi/deflate-frame-invalid-parameter.html:
- http/tests/websocket/tests/hybi/deflate-frame-set-bfinal.html:
- http/tests/websocket/tests/hybi/error-event-ready-state-non-existent-url-with-server-responding-404-expected.txt:
- http/tests/websocket/tests/hybi/error-event-ready-state-non-existent-url-with-server-responding-404.html:
- http/tests/websocket/tests/hybi/fragmented-control-frame-expected.txt:
- http/tests/websocket/tests/hybi/fragmented-control-frame.html:
- http/tests/websocket/tests/hybi/handshake-error-expected.txt:
- http/tests/websocket/tests/hybi/handshake-error.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-extensions-header-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-extensions-header.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-invalid-http-version-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-invalid-http-version.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-maxlength-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-maxlength.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-mismatch-protocol-header-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-mismatch-protocol-header.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-more-accept-header-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-more-accept-header.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-more-extensions-header-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-more-extensions-header.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-more-protocol-header-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-more-protocol-header.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-no-accept-header-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-no-accept-header.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-no-connection-header-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-no-connection-header.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-no-cr-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-no-cr.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-no-upgrade-header-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-no-upgrade-header.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-non-ascii-header-value-sec-websocket-accept-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-non-ascii-header-value-sec-websocket-accept.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-non-ascii-header-value-sec-websocket-extensions-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-non-ascii-header-value-sec-websocket-extensions.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-non-ascii-header-value-sec-websocket-protocol-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-non-ascii-header-value-sec-websocket-protocol.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-non-ascii-status-line-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-non-ascii-status-line.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-null-char-in-status-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-null-char-in-status.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-prepended-null-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-prepended-null.html:
- http/tests/websocket/tests/hybi/handshake-fail-by-wrong-accept-header-expected.txt:
- http/tests/websocket/tests/hybi/handshake-fail-by-wrong-accept-header.html:
- http/tests/websocket/tests/hybi/handshake-ok-with-http-version-beyond-1_1-expected.txt:
- http/tests/websocket/tests/hybi/handshake-ok-with-http-version-beyond-1_1.html:
- http/tests/websocket/tests/hybi/interleaved-fragments-expected.txt:
- http/tests/websocket/tests/hybi/interleaved-fragments.html:
- http/tests/websocket/tests/hybi/invalid-continuation-expected.txt:
- http/tests/websocket/tests/hybi/invalid-continuation.html:
- http/tests/websocket/tests/hybi/invalid-encode-length-expected.txt:
- http/tests/websocket/tests/hybi/invalid-encode-length.html:
- http/tests/websocket/tests/hybi/invalid-masked-frames-from-server-expected.txt:
- http/tests/websocket/tests/hybi/invalid-masked-frames-from-server.html:
- http/tests/websocket/tests/hybi/invalid-subprotocol-characters-expected.txt:
- http/tests/websocket/tests/hybi/invalid-subprotocol-characters.html:
- http/tests/websocket/tests/hybi/long-control-frame-expected.txt:
- http/tests/websocket/tests/hybi/long-control-frame.html:
- http/tests/websocket/tests/hybi/long-invalid-header-expected.txt:
- http/tests/websocket/tests/hybi/long-invalid-header.html:
- http/tests/websocket/tests/hybi/network-process-crash-error-expected.txt:
- http/tests/websocket/tests/hybi/network-process-crash-error.html:
- http/tests/websocket/tests/hybi/reserved-bits-expected.txt:
- http/tests/websocket/tests/hybi/reserved-bits.html:
- http/tests/websocket/tests/hybi/reserved-opcodes-expected.txt:
- http/tests/websocket/tests/hybi/reserved-opcodes.html:
- http/tests/websocket/tests/hybi/too-long-payload-expected.txt:
- http/tests/websocket/tests/hybi/too-long-payload.html:
- http/tests/websocket/tests/hybi/workers/close-code-and-reason-expected.txt:
- http/tests/websocket/tests/hybi/workers/close-code-and-reason.html:
- platform/glib/http/tests/websocket/connection-refusal-in-frame-resource-load-statistics-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/broken-utf8-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/closed-port-delay-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/compressed-control-frame-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/deflate-frame-invalid-parameter-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/error-event-ready-state-non-existent-url-with-server-responding-404-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/fragmented-control-frame-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-error-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-extensions-header-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-invalid-http-version-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-maxlength-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-mismatch-protocol-header-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-more-accept-header-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-more-extensions-header-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-more-protocol-header-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-no-accept-header-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-no-connection-header-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-no-cr-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-no-upgrade-header-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-non-ascii-header-value-sec-websocket-accept-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-non-ascii-header-value-sec-websocket-extensions-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-non-ascii-header-value-sec-websocket-protocol-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-non-ascii-status-line-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-null-char-in-status-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-prepended-null-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/handshake-fail-by-wrong-accept-header-expected.txt: Removed.
- platform/glib/http/tests/websocket/tests/hybi/interleaved-fragments-expected.txt: Removed.
- 3:21 AM Changeset in webkit [266229] by
-
- 2 edits in trunk/Source/WebCore
Animation are invalidating too much
https://bugs.webkit.org/show_bug.cgi?id=215849
Reviewed by Antoine Quint.
- animation/KeyframeEffect.cpp:
(WebCore::invalidateElement):
Changes due to animations can't affect sibling elements so we can use internal invalidation.
- 3:00 AM Changeset in webkit [266228] by
-
- 57 edits5 adds in trunk
Add support for TransformStream
https://bugs.webkit.org/show_bug.cgi?id=215607
Reviewed by Alex Christensen.
LayoutTests/imported/w3c:
- web-platform-tests/streams/idlharness.any-expected.txt:
- web-platform-tests/streams/idlharness.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/backpressure.any-expected.txt:
- web-platform-tests/streams/transform-streams/backpressure.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/errors.any-expected.txt:
- web-platform-tests/streams/transform-streams/errors.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/flush.any-expected.txt:
- web-platform-tests/streams/transform-streams/flush.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/general.any-expected.txt:
- web-platform-tests/streams/transform-streams/general.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/lipfuzz.any-expected.txt:
- web-platform-tests/streams/transform-streams/lipfuzz.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/patched-global.any-expected.txt:
- web-platform-tests/streams/transform-streams/patched-global.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/properties.any-expected.txt:
- web-platform-tests/streams/transform-streams/properties.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/reentrant-strategies.any-expected.txt:
- web-platform-tests/streams/transform-streams/reentrant-strategies.any.js:
- web-platform-tests/streams/transform-streams/reentrant-strategies.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/strategies.any-expected.txt:
- web-platform-tests/streams/transform-streams/strategies.any.worker-expected.txt:
- web-platform-tests/streams/transform-streams/terminate.any-expected.txt:
- web-platform-tests/streams/transform-streams/terminate.any.worker-expected.txt:
Source/WebCore:
Add initial implementation as JS built-ins as per spec.
Covered by rebased tests.
- CMakeLists.txt:
- DerivedSources-input.xcfilelist:
- DerivedSources-output.xcfilelist:
- DerivedSources.make:
- Headers.cmake:
- Modules/streams/ReadableStreamInternals.js:
(readableStreamDefaultControllerShouldCallPull):
(readableStreamDefaultControllerCallPullIfNeeded):
- Modules/streams/TransformStream.idl: Added.
- Modules/streams/TransformStream.js: Added.
(initializeTransformStream):
(getter.readable):
(writable):
- Modules/streams/TransformStreamDefaultController.idl: Added.
- Modules/streams/TransformStreamDefaultController.js: Added.
(initializeTransformStreamDefaultController):
(getter.desiredSize):
(enqueue):
(error):
(terminate):
- Modules/streams/TransformStreamInternals.js: Added.
(isTransformStream):
(isTransformStreamDefaultController):
(initializeTransformStream):
(transformStreamError):
(transformStreamErrorWritableAndUnblockWrite):
(transformStreamSetBackpressure):
(setUpTransformStreamDefaultController):
(setUpTransformStreamDefaultControllerFromTransformer):
(transformStreamDefaultControllerClearAlgorithms):
(transformStreamDefaultControllerEnqueue):
(transformStreamDefaultControllerError):
(transformStreamDefaultControllerPerformTransform):
(transformStreamDefaultSinkWriteAlgorithm):
(transformStreamDefaultSinkAbortAlgorithm):
(transformStreamDefaultSinkCloseAlgorithm):
(transformStreamDefaultSourcePullAlgorithm):
- Modules/streams/WritableStream.js:
(initializeWritableStream):
- Modules/streams/WritableStreamInternals.js:
(createWritableStream):
- Sources.txt:
- WebCore.xcodeproj/project.pbxproj:
- bindings/js/WebCoreBuiltinNames.h:
- page/RuntimeEnabledFeatures.h:
(WebCore::RuntimeEnabledFeatures::setTransformStreamAPIEnabled):
(WebCore::RuntimeEnabledFeatures::transformStreamAPIEnabled const):
Source/WebKit:
Add a runtime flag for TransformStream, off by default.
- Shared/WebPreferences.yaml:
- WebProcess/InjectedBundle/InjectedBundle.cpp:
(WebKit::InjectedBundle::overrideBoolPreferenceForTestRunner):
Source/WebKitLegacy/mac:
- WebView/WebPreferenceKeysPrivate.h:
- WebView/WebPreferences.mm:
(+[WebPreferences initialize]):
(-[WebPreferences transformStreamAPIEnabled]):
(-[WebPreferences setTransformStreamAPIEnabled:]):
- WebView/WebPreferencesPrivate.h:
- WebView/WebView.mm:
(-[WebView _preferencesChanged:]):
Tools:
- DumpRenderTree/mac/DumpRenderTree.mm:
(enableExperimentalFeatures):
- WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:
(WTR::InjectedBundle::beginTesting):
- WebKitTestRunner/InjectedBundle/TestRunner.cpp:
(WTR::TestRunner::setWritableStreamAPIEnabled):
(WTR::TestRunner::setTransformStreamAPIEnabled):
- WebKitTestRunner/InjectedBundle/TestRunner.h:
LayoutTests:
Rebased tests now that TransformStream is defined.
- streams/reference-implementation/pipe-through-expected.txt:
- streams/reference-implementation/pipe-to-expected.txt:
- streams/reference-implementation/transform-stream-errors-expected.txt:
- streams/reference-implementation/transform-stream-expected.txt:
- 2:56 AM Changeset in webkit [266227] by
-
- 12 edits in trunk/Source
[GTK] Include the run loop source name in frame rendering timeline
https://bugs.webkit.org/show_bug.cgi?id=215847
Reviewed by Adrian Perez de Castro.
Source/WebCore:
Include the given name in rendering frame record data.
- inspector/TimelineRecordFactory.cpp:
(WebCore::TimelineRecordFactory::createRenderingFrameData):
- inspector/TimelineRecordFactory.h:
- inspector/agents/InspectorTimelineAgent.cpp:
(WebCore::InspectorTimelineAgent::internalStart):
Source/WebInspectorUI:
Show the frame name if present in timeline panel.
- Localizations/en.lproj/localizedStrings.js:
- UserInterface/Controllers/TimelineManager.js:
(WI.TimelineManager.prototype._processRecord):
- UserInterface/Models/RenderingFrameTimelineRecord.js:
(WI.RenderingFrameTimelineRecord):
(WI.RenderingFrameTimelineRecord.prototype.get name):
- UserInterface/Views/TimelineTabContentView.js:
(WI.TimelineTabContentView.displayNameForRecord):
Source/WTF:
Pass the run loop source name to the observer.
- wtf/RunLoop.h:
- wtf/glib/RunLoopGLib.cpp:
(WTF::RunLoop::RunLoop):
(WTF::RunLoop::notify):
- 1:25 AM Changeset in webkit [266226] by
-
- 3 edits in trunk/Source/WebCore
Drop unnecessary BufferPlaybackMode enum from AudioBufferSourceNode
https://bugs.webkit.org/show_bug.cgi?id=215878
Reviewed by Youenn Fablet.
Drop unnecessary BufferPlaybackMode enum from AudioBufferSourceNode, only
one of the enum values is ever used.
No new tests, no web-facing behavior change.
- Modules/webaudio/AudioBufferSourceNode.cpp:
(WebCore::AudioBufferSourceNode::startLater):
(WebCore::AudioBufferSourceNode::startPlaying):
- Modules/webaudio/AudioBufferSourceNode.h: