Timeline



May 9, 2022:

11:53 PM Changeset in webkit [294003] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Check if node is connected after calling mergeWithNeighboringLists
https://bugs.webkit.org/show_bug.cgi?id=240234

Patch by Rob Buis <rbuis@igalia.com> on 2022-05-09
Reviewed by Wenson Hsieh.

Check if node is connected after calling mergeWithNeighboringLists since
DOM mutations can change the DOM tree.

  • editing/InsertListCommand.cpp:

(WebCore::InsertListCommand::doApplyForSingleParagraph):

11:52 PM Changeset in webkit [294002] by Oriol Brufau
  • 19 edits
    2 adds in trunk

[cssom] Don't index perspective/transform-origin-* in computed styles
https://bugs.webkit.org/show_bug.cgi?id=239670

Reviewed by Darin Adler.

LayoutTests/imported/w3c:

Update some test expectations.

Add serialize-all-longhands.html, it still fails due to
https://webkit.org/b/239965 and https://webkit.org/b/239989.

  • web-platform-tests/css/css-cascade/all-prop-initial-xml-expected.txt:
  • web-platform-tests/css/css-cascade/all-prop-revert-layer-expected.txt:
  • web-platform-tests/css/cssom/getComputedStyle-detached-subtree-expected.txt:
  • web-platform-tests/css/cssom/serialize-all-longhands-expected.txt: Added.
  • web-platform-tests/css/cssom/serialize-all-longhands.html: Added.

Source/WebCore:

'perspective-origin' and 'transform-origin' are shorthands in WebKit.
The longhands are 'perspective-origin-x/y' and 'transform-origin-x/y/z'.
But these aren't standard properties, so they serialize to empty string
in computed styles. However, they were still indexed.

This patch stops indexing these longhands, so they won't appear when
iterating a computed style. Also, the shorthands will now be indexed.

Test: fast/css/getComputedStyle/computed-style.html

fast/css/getComputedStyle/computed-style-without-renderer.html
imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml.html
imported/w3c/web-platform-tests/css/css-cascade/all-prop-revert-layer.html
imported/w3c/web-platform-tests/css/cssom/getComputedStyle-detached-subtree.html
imported/w3c/web-platform-tests/css/cssom/serialize-all-longhands.html
svg/css/getComputedStyle-basic.xhtml

  • css/CSSProperties.json:

Add "computable" flag.

  • css/makeprop.pl:

(skippedFromComputedStyle):
(addProperty):
Let "computable" flag override default logic for indexing or not in
computed styles.

Tools:

Add checker for new "computable" flag.

  • Scripts/webkitpy/style/checkers/jsonchecker.py:

(JSONCSSPropertiesChecker.check_codegen_properties):

LayoutTests:

Update test expectations.

  • fast/css/getComputedStyle/computed-style-expected.txt:
  • fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
  • platform/gtk/imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml-expected.txt:
  • platform/gtk/imported/w3c/web-platform-tests/css/cssom/getComputedStyle-detached-subtree-expected.txt:
  • platform/ios/imported/w3c/web-platform-tests/css/cssom/getComputedStyle-detached-subtree-expected.txt:
  • platform/mac-wk1/imported/w3c/web-platform-tests/css/css-cascade/all-prop-revert-layer-expected.txt:
  • platform/wpe/imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml-expected.txt:
  • platform/wpe/imported/w3c/web-platform-tests/css/cssom/getComputedStyle-detached-subtree-expected.txt:
  • svg/css/getComputedStyle-basic-expected.txt:
11:48 PM Changeset in webkit [294001] by Nikos Mouchtaris
  • 13 edits in trunk

Support ray() shape in offset-path
https://bugs.webkit.org/show_bug.cgi?id=233344

Reviewed by Simon Fraser.

Source/WebCore:

Add support for rendering ray path. Involves getting containing block and offset
of the element from the containing block. Using the size of the containing block
and offset, calculate the length of the ray and create a Line path using the length
and angle. Calculating closest/farthest-side and closest/farthest-corner are simple
but calculating side requires some explanation. First, we get the two possible sides of
the containing block the ray could be intersecting with, based on the angle. To calculate
which side is being intersected, if tan(theta) * top/bottom is outside of the containing
block, this means that the ray is intersecting with the other side. Finally, we calculate
the acute angle based on which side we intersected with. Using the length and angle, we
calculate the length of the hypotenuse, which corresponds with the length of the start point
of the ray to its intersection with a side of the containing block. Test 7 is still failing
due to getting incorrect width of containing block (doesn't happen when animating the path
for some reason). Will implement contain in another patch.

  • WebCore.xcodeproj/project.pbxproj:
  • rendering/PathOperation.cpp:

(WebCore::toPositiveAngle):
(WebCore::RayPathOperation::getLengthForPath const):
(WebCore::RayPathOperation::pathForReferenceRect const):

  • rendering/PathOperation.h:
  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::setReferenceBoxForPathOperations):
(WebCore::RenderLayer::updateTransform):

  • rendering/RenderLayer.h:
  • rendering/style/RenderStyle.cpp:

(WebCore::getPathFromPathOperation):

LayoutTests:

10:20 PM Changeset in webkit [294000] by Ross Kirsling
  • 2 edits in trunk/Source/JavaScriptCore

Unreviewed, address Darin's feedback on r250361.

  • runtime/JSCJSValueInlines.h:

(JSC::JSValue::toIntegerWithoutRounding const):
Normalize away -0 by unconditionally adding positive 0 (instead of making a separate zero check).

10:08 PM Changeset in webkit [293999] by rniwa@webkit.org
  • 2 edits in trunk

Introduction.md: Explain active DOM objects
https://bugs.webkit.org/show_bug.cgi?id=240212

Reviewed by Chris Dumez.

Added an elementary description of active DOM objects to Introduction.md.

  • Introduction.md:
9:41 PM Changeset in webkit [293998] by Alan Bujtas
  • 2 edits in trunk/Source/WebCore

Do not use the cached renderer's parent in handleFragmentedFlowStateChange lambda
https://bugs.webkit.org/show_bug.cgi?id=240266

Reviewed by Simon Fraser.

  • rendering/updating/RenderTreeBuilder.cpp:

(WebCore::RenderTreeBuilder::normalizeTreeAfterStyleChange):

8:49 PM Changeset in webkit [293997] by Ross Kirsling
  • 9 edits in trunk

Temporal round and total methods should accept string param
https://bugs.webkit.org/show_bug.cgi?id=240249

Reviewed by Yusuke Suzuki.

This patch implements https://github.com/tc39/proposal-temporal/pull/1875,
which allows certain required string options to be passed directly instead of as part of an options object.

Namely:

  • {Duration, Instant, PlainTime}::round now accept smallestUnit as a string param
  • Duration::total now accepts unit as a string param
  • stress/temporal-duration.js:
  • stress/temporal-instant.js:
  • stress/temporal-plaintime.js:

Add test cases.

  • test262/expectations.yaml:

Mark 24 test cases passing.
(This number should be 26, but two still fail as the harness expects PlainDateTime and ZonedDateTime to exist.)

  • runtime/TemporalDuration.cpp:

(JSC::TemporalDuration::round const):
(JSC::TemporalDuration::total const):

  • runtime/TemporalInstant.cpp:
  • runtime/TemporalPlainTime.cpp:

(JSC::TemporalPlainTime::round const):

Canonical link: https://commits.webkit.org/250433@main

7:52 PM Changeset in webkit [293996] by Ross Kirsling
  • 6 edits in trunk

Temporal and Date must reject expanded year -000000
https://bugs.webkit.org/show_bug.cgi?id=240263

Reviewed by Yusuke Suzuki.

As of the following two PRs, -000000 is officially disallowed as a representation of the year zero in ISO date strings.
https://github.com/tc39/ecma262/pull/2550
https://github.com/tc39/proposal-temporal/pull/1992

This patch implements the change for Temporal and Date alike.

  • test262/expectations.yaml:

Mark 24 test cases as passing.

  • runtime/ISO8601.cpp:

(JSC::ISO8601::parseDate):

  • wtf/DateMath.cpp:

(WTF::parseES5DatePortion):

Canonical link: https://commits.webkit.org/250432@main

5:16 PM Changeset in webkit [293995] by Alan Coon
  • 7 edits in branches/safari-613-branch

Cherry-pick r292888. rdar://problem/80059355

[iOS] [WK2] Managed pasteboard should function for all managed domains
https://bugs.webkit.org/show_bug.cgi?id=239319
rdar://80059355

Reviewed by Kate Cheney.

Source/WebCore/PAL:

Add an SPI method on MCProfileConnection.

  • pal/spi/ios/ManagedConfigurationSPI.h:

Source/WebKit:

Unless a WebKit client has specified a data owner for the web view that is not _UIDataOwnerUndefined, fall back
to _UIDataOwnerEnterprise when the current domain of the WKWebView is managed (that is, `-[MCProfileConnection
isURLManaged:]` returns YES for the web view's current URL). This allows managed pasteboard to work for all
WebKit clients, if the current URL is managed.

Test: UIPasteboardTests.PerformAsDataOwnerWithManagedURL

  • Platform/spi/ios/UIKitSPI.h:

Drive-by fix: move the staged declarations of -_dataOwnerForCopy and -_dataOwnerForPaste out of the IPI
section, and into the non-internal SDK section.

  • UIProcess/ios/WKContentViewInteraction.mm: (-[WKContentView _dataOwnerForPasteboard:]):

Tools:

Add a new API test to verify that we fall back to consulting -[MCProfileConnection isURLManaged:] when
determining the data owner for copy and paste, unless a data owner is already explicitly set on a view in the
responder chain (specifically, the WKWebView).

  • TestWebKitAPI/Tests/ios/UIPasteboardTests.mm: (+[TestUIPasteboard _performAsDataOwner:block:]): (-[TestMCProfileConnection isURLManaged:]): (TestWebKitAPI::TEST):

Canonical link: https://commits.webkit.org/249658@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@292888 268f45cc-cd09-0410-ab3c-d52691b4dbfc

5:06 PM Changeset in webkit [293994] by Wenson Hsieh
  • 13 edits in trunk/Source

[iOS] Adjust some viewport behaviors when multitasking mode is enabled
https://bugs.webkit.org/show_bug.cgi?id=240151
rdar://87157773

Reviewed by Tim Horton.

Add a new ViewportConfiguration flag to prefer horizontal scrolling over shrinking to fit when the view layout
size falls under the current "default desktop webpage" of 980pt. See WebKit changes for more details.

  • page/ViewportConfiguration.cpp:

(WebCore::ViewportConfiguration::initialScaleFromSize const):
(WebCore::ViewportConfiguration::webpageParameters):
(WebCore::ViewportConfiguration::imageDocumentParameters):

Pull the magic value representing the "assumed width of most desktop webpages" (980) out into a named constant,
defaultDesktopViewportWidth, so that we can consult it when computing the initial scale.

(WebCore::ViewportConfiguration::description const):

  • page/ViewportConfiguration.h:

(WebCore::ViewportConfiguration::setPrefersHorizontalScrollingBelowDesktopViewportWidths):
[iOS] Adjust some viewport behaviors when multitasking mode is enabled
https://bugs.webkit.org/show_bug.cgi?id=240151
rdar://87157773

Reviewed by Tim Horton.

Make some adjustments to viewport behaviors when multitasking mode is enabled. See the comments below for more
details. There are no changes in behavior when multitasking mode is disabled; tests for the new behaviors in
multitasking mode will be added in a subsequent patch.

  • Shared/WebPageCreationParameters.cpp:

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

  • Shared/WebPageCreationParameters.h:

Add plumbing to inform the web process when "multitasking mode" state changes; we use this bit in WebPage to
determine whether or not we should use multitasking mode viewport behaviors (see below).

  • UIProcess/API/ios/WKWebViewIOS.mm:

(-[WKWebView _registerForNotifications]):
(-[WKWebView didMoveToWindow]):
(-[WKWebView _multitaskingModeDidChange:]):

Send IPC to WebPage in these two places, to keep "multitasking mode" state in sync with the native view.

(-[WKWebView _beginAnimatedResizeWithUpdates:]):

Make a minor adjustment here to ignore oldWebViewWidthInContentViewCoordinates when computing a target scale
to zoom to when performing animated resize, in multitasking mode. This is required to prevent us from zooming
in excessively when the width of the view increases, since we'd otherwise attempt to keep the same content in
the page visible by zooming in (for instance, if an image covers most of the visual viewport at a lower view
width, this min() logic would cause us to zoom in, such that the image would still cover most of the viewport
at a larger width). This behavior is undesirable in multitasking mode.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::creationParameters):
(WebKit::WebPageProxy::setIsInMultitaskingMode):

  • UIProcess/WebPageProxy.h:
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::didCommitLoad):
(WebKit::WebPage::setIsInMultitaskingMode):

Add plumbing to set the m_isInMultitaskingMode flag on WebPage, and update the viewport configuration flag to
prefer horizontal scrolling below 980pt.

  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/WebPage.messages.in:
  • WebProcess/WebPage/ios/WebPageIOS.mm:

(WebKit::scaleAfterViewportWidthChange):

Refactor a bit of code here (without changing any behaviors), to make it a bit clearer:

  • Rename userHasChangedPageScaleFactor to scaleToFitContent to better describe how this flag affects the

adjusted target scale during dynamic resize.

  • Make the log messages specific to both branches, and also log the adjusted viewport scale instead of the

(currently unused) given scale in the non-scaleToFitContent codepath.

(WebKit::WebPage::dynamicViewportSizeUpdate):

Make another "multitasking mode viewport behavior" adjustment here by maintaining the initial scale (only if the
viewport was already at initial scale) when performing dynamic viewport size updates. By default, we currently
adjust the scale such that the same content is still visible at the new viewport size; however, when allowing
horizontal scrolling, this causes us to zoom in excessively when making the window width larger. Instead, when
multitasking mode is enabled, we should try to preserve initial scale when changing window size, such that only
the horizontal scroll amount changes.

(WebKit::WebPage::usesMultitaskingModeViewportBehaviors const):

Add a helper method to encapsulate whether or not multitasking mode viewport behaviors should be used; this
should be true only when both desktop-class viewport behaviors are active, *and* multitasking mode is also
active.

Canonical link: https://commits.webkit.org/250431@main

4:50 PM Changeset in webkit [293993] by timothy_horton@apple.com
  • 42 edits in trunk/Source/WebCore

Const-ify Node::willRespondTo*Events()
https://bugs.webkit.org/show_bug.cgi?id=240246

Reviewed by Wenson Hsieh.

Constify these four methods, because there's no
reason for them not to be, and because it makes
them usable in a const context in a future patch.

  • dom/EventNames.h:

(WebCore::EventNames::isTouchRelatedEventType const):

  • dom/EventTarget.cpp:

(WebCore::EventTarget::eventTypes const):

  • dom/EventTarget.h:
  • dom/Node.cpp:

(WebCore::Node::willRespondToMouseMoveEvents const):
(WebCore::Node::willRespondToTouchEvents const):
(WebCore::Node::willRespondToMouseClickEvents const):
(WebCore::Node::willRespondToMouseWheelEvents const):

  • dom/Node.h:
  • html/HTMLAnchorElement.cpp:

(WebCore::HTMLAnchorElement::willRespondToMouseClickEvents const):

  • html/HTMLAnchorElement.h:
  • html/HTMLButtonElement.cpp:

(WebCore::HTMLButtonElement::willRespondToMouseClickEvents const):

  • html/HTMLButtonElement.h:
  • html/HTMLElement.cpp:

(WebCore::HTMLElement::willRespondToMouseMoveEvents const):
(WebCore::HTMLElement::willRespondToMouseWheelEvents const):
(WebCore::HTMLElement::willRespondToMouseClickEvents const):

  • html/HTMLElement.h:
  • html/HTMLImageElement.cpp:

(WebCore::HTMLImageElement::willRespondToMouseClickEvents const):

  • html/HTMLImageElement.h:
  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::willRespondToMouseClickEvents const):

  • html/HTMLInputElement.h:
  • html/HTMLLabelElement.cpp:

(WebCore::HTMLLabelElement::willRespondToMouseClickEvents const):

  • html/HTMLLabelElement.h:
  • html/HTMLPlugInElement.cpp:

(WebCore::HTMLPlugInElement::willRespondToMouseClickEvents const):

  • html/HTMLPlugInElement.h:
  • html/HTMLSelectElement.cpp:

(WebCore::HTMLSelectElement::willRespondToMouseClickEvents const):

  • html/HTMLSelectElement.h:
  • html/HTMLSummaryElement.cpp:

(WebCore::HTMLSummaryElement::willRespondToMouseClickEvents const):

  • html/HTMLSummaryElement.h:
  • html/HTMLTextAreaElement.cpp:

(WebCore::HTMLTextAreaElement::willRespondToMouseClickEvents const):

  • html/HTMLTextAreaElement.h:
  • html/TextFieldInputType.cpp:

(WebCore::TextFieldInputType::shouldSpinButtonRespondToMouseEvents const):
(WebCore::TextFieldInputType::shouldSpinButtonRespondToWheelEvents const):

  • html/TextFieldInputType.h:
  • html/shadow/SliderThumbElement.cpp:

(WebCore::SliderThumbElement::willRespondToMouseMoveEvents const):
(WebCore::SliderThumbElement::willRespondToMouseClickEvents const):

  • html/shadow/SliderThumbElement.h:
  • html/shadow/SpinButtonElement.cpp:

(WebCore::SpinButtonElement::willRespondToMouseMoveEvents const):
(WebCore::SpinButtonElement::willRespondToMouseClickEvents const):
(WebCore::SpinButtonElement::shouldRespondToMouseEvents const):

  • html/shadow/SpinButtonElement.h:
  • html/shadow/TextControlInnerElements.cpp:

(WebCore::SearchFieldResultsButtonElement::willRespondToMouseClickEvents const):
(WebCore::SearchFieldCancelButtonElement::willRespondToMouseClickEvents const):

  • html/shadow/TextControlInnerElements.h:
  • mathml/MathMLElement.cpp:

(WebCore::MathMLElement::willRespondToMouseClickEvents const):

  • mathml/MathMLElement.h:
  • mathml/MathMLSelectElement.cpp:

(WebCore::MathMLSelectElement::willRespondToMouseClickEvents const):

  • mathml/MathMLSelectElement.h:
  • svg/SVGAElement.cpp:

(WebCore::SVGAElement::willRespondToMouseClickEvents const):

  • svg/SVGAElement.h:
4:03 PM Changeset in webkit [293992] by Jonathan Bedard
  • 4 edits in trunk/Websites/bugs.webkit.org

[bugs.webkit.org] Use GitHub as baseline when diffing
https://bugs.webkit.org/show_bug.cgi?id=240073
<rdar://problem/92741303>

Reviewed by Ryan Haddad and Dewei Zhu.

  • Websites/bugs.webkit.org/code-review.js: Use GitHub instead of

Subversion for tip-of-tree baseline, blame and log links.

  • Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb: Ditto.

Use GitHub instead of Subversion for tip-of-tree baseline.

Canonical link: https://commits.webkit.org/250429@main

4:02 PM Changeset in webkit [293991] by timothy_horton@apple.com
  • 2 edits in trunk/Source/ThirdParty/libwebrtc

Unreviewed build fix; disable -Wimplicit-fallthrough in libwebrtc

  • Configurations/Base.xcconfig:
2:17 PM Changeset in webkit [293990] by pvollan@apple.com
  • 4 edits in trunk/Source

[macOS] HTTP traffic is not filtered in the parental controls filter
https://bugs.webkit.org/show_bug.cgi?id=240180
<rdar://problem/92875540>

Reviewed by Alexey Proskuryakov.

Source/WebCore:

Traditionally, we have not filtered HTTP in the parental controls filter on macOS, since other parts of the system
has taken care of this. This has changed in the most recent macOS version, and WebKit should also filter HTTP in
addition to HTTPS.

  • platform/cocoa/ParentalControlsContentFilter.mm:

(WebCore::canHandleResponse):

Source/WTF:

Add HAVE macro which tells us which OS versions are filtering HTTP traffic on behalf of WebKit.

  • wtf/PlatformHave.h:
1:56 PM Changeset in webkit [293989] by Simon Fraser
  • 5 edits in trunk/Source/WebCore

Cache the viewport size inside SVGLengthContext
https://bugs.webkit.org/show_bug.cgi?id=240157

Reviewed by Alan Bujtas.

Each call to SVGLengthContext::determineViewport() did an ancestor element walk
looking for a viewport element, and some call sites hit this two or more times
(e.g. for width, then height). So cache m_viewportSize in the class, modernizing
the code to use optionals.

  • rendering/RenderElement.cpp:

(WebCore::RenderElement::referenceBoxRect const):

  • rendering/svg/LegacyRenderSVGRoot.cpp:

(WebCore::LegacyRenderSVGRoot::computeFloatVisibleRectInContainer const):

  • rendering/svg/SVGRenderSupport.cpp:

(WebCore::clipPathReferenceBox):

  • svg/SVGLengthContext.cpp:

(WebCore::SVGLengthContext::valueForLength):
(WebCore::SVGLengthContext::convertValueToUserUnits const):
(WebCore::SVGLengthContext::convertValueFromUserUnitsToPercentage const):
(WebCore::SVGLengthContext::convertValueFromPercentageToUserUnits const):
(WebCore::SVGLengthContext::viewportSize const):
(WebCore::SVGLengthContext::computeViewportSize const):
(WebCore::SVGLengthContext::determineViewport const): Deleted.

  • svg/SVGLengthContext.h:
1:43 PM Changeset in webkit [293988] by Ben Nham
  • 2 edits in trunk/Source/WebKit

Allow log streaming from webpushd
https://bugs.webkit.org/show_bug.cgi?id=240238

Reviewed by Per Arne Vollan.

log stream outputs nothing from webpushd because we blocked the diagnosticd service in its
sandbox. We should allow that and also block awdd since it's unlikely that webpushd needs
access to the wireless diagnostics daemon.

  • webpushd/mac/com.apple.WebKit.webpushd.sb.in:

Canonical link: https://commits.webkit.org/250425@main

1:37 PM Changeset in webkit [293987] by ntim@apple.com
  • 16 edits
    2 adds in trunk

Implement CSS :modal pseudo class
https://bugs.webkit.org/show_bug.cgi?id=240109

Reviewed by Simon Fraser.

LayoutTests/imported/w3c:

Add and extend tests for :modal pseudo-class.

  • web-platform-tests/css/selectors/invalidation/modal-pseudo-class-in-has-expected.txt: Added.
  • web-platform-tests/css/selectors/invalidation/modal-pseudo-class-in-has.html: Added.
  • web-platform-tests/html/semantics/interactive-elements/the-dialog-element/dialog-show-modal.html:

Source/WebCore:

Test: imported/w3c/web-platform-tests/css/selectors/invalidation/modal-pseudo-class-in-has.html

Renames :-internal-modal-dialog to :modal and adds :has() invalidation support.

  • css/CSSSelector.cpp:

(WebCore::CSSSelector::selectorText const):

  • css/CSSSelector.h:
  • css/SelectorChecker.cpp:

(WebCore::SelectorChecker::checkOne const):

  • css/SelectorCheckerTestFunctions.h:

(WebCore::matchesModalPseudoClass):
(WebCore::matchesModalDialogPseudoClass): Deleted.

  • css/SelectorPseudoClassAndCompatibilityElementMap.in:
  • css/dialog.css:

(dialog:modal):
(dialog:-internal-modal-dialog): Deleted.

  • css/parser/CSSSelectorParser.cpp:

(WebCore::CSSSelectorParser::consumePseudo):

  • cssjit/SelectorCompiler.cpp:

(WebCore::SelectorCompiler::JSC_DEFINE_JIT_OPERATION):
(WebCore::SelectorCompiler::addPseudoClassType):

  • html/HTMLDialogElement.cpp:

(WebCore::HTMLDialogElement::showModal):
(WebCore::HTMLDialogElement::close):
(WebCore::HTMLDialogElement::removedFromAncestor):
(WebCore::HTMLDialogElement::setIsModal):

  • html/HTMLDialogElement.h:

LayoutTests:

Removes :-internal-modal-dialog from internal pseudo classes.

  • fast/css/pseudo-class-internal-expected.txt:
  • fast/css/pseudo-class-internal.html:
1:04 PM Changeset in webkit [293986] by Jonathan Bedard
  • 4 edits in trunk/Tools

[git-webkit] Verify forks after creation
https://bugs.webkit.org/show_bug.cgi?id=240085
<rdar://problem/92978491>

Reviewed by Stephanie Lewis.

  • Tools/Scripts/libraries/webkitscmpy/setup.py: Bump version.
  • Tools/Scripts/libraries/webkitscmpy/webkitscmpy/init.py: Ditto.
  • Tools/Scripts/libraries/webkitscmpy/webkitscmpy/program/setup.py:

(Setup.github): Check parent of fork, wait until fork is created before continuing.

Canonical link: https://commits.webkit.org/250423@main

12:40 PM Changeset in webkit [293985] by ysuzuki@apple.com
  • 2 edits in trunk/Source/WebKit

Unreviewed, build fix for Internal iOS build
https://bugs.webkit.org/show_bug.cgi?id=240206

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::updateTouchEventTracking):

12:10 PM Changeset in webkit [293984] by commit-queue@webkit.org
  • 3 edits
    1 add in trunk/Source/WebKit

WebKit has a broken module in Mac Catalyst
https://bugs.webkit.org/show_bug.cgi?id=240175
rdar://92703419

Patch by Ian Anderson <iana@apple.com> on 2022-05-09
Reviewed by Tim Horton.

Add a module map for Mac Catalyst that excludes WebKit/WebKitLegacy.h,
matching the exclusion from WebKit/WebKit.h.

  • Configurations/WebKit.xcconfig:
  • Modules/MacCatalyst.modulemap: Added.
  • WebKit.xcodeproj/project.pbxproj:
11:48 AM Changeset in webkit [293983] by Russell Epstein
  • 1 copy in branches/safari-7614.1.13-branch

New branch.

11:27 AM Changeset in webkit [293982] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

[GLIB] Update some test expectations for known failures
https://bugs.webkit.org/show_bug.cgi?id=240240

Unreviewed test garderning.

Patch by Arcady Goldmints-Orlov <Arcady Goldmints-Orlov> on 2022-05-09

  • platform/glib/TestExpectations:
10:44 AM Changeset in webkit [293981] by gnavamarino@apple.com
  • 2 edits in trunk/Source/WebCore

ASSERT in WebCore::RenderTreeUpdater::updateRenderTree
https://bugs.webkit.org/show_bug.cgi?id=240237

Reviewed by Antti Koivisto.

There are instances where calling findRenderingRoots() in RenderTreeUpdater::commit will
returns two rendering roots, with one of them being an ancestor of the other.

Calling updateRenderTree on the ancestor rendering root could end up removing
the renderer of the descendant rendering root.

This patch merges findRenderingRoots() to RenderTreeUpdater::commit() which, and now we will
ignore the nested root if there is no longer a renderer when it is being processed.

  • rendering/updating/RenderTreeUpdater.cpp:
10:32 AM Changeset in webkit [293980] by Wenson Hsieh
  • 5 edits
    2 adds in trunk

[iOS] Double tapping on YouTube video causes playback to pause instead of seek
https://bugs.webkit.org/show_bug.cgi?id=240222
rdar://92637636

Reviewed by Aditya Keerthi and Kate Cheney.

Add a new layout test. See WebKit/ChangeLog for more details.

  • fast/events/touch/ios/touch-events-when-double-tapping-after-selecting-text-expected.txt: Added.
  • fast/events/touch/ios/touch-events-when-double-tapping-after-selecting-text.html: Added.

[iOS] Double tapping on YouTube video causes playback to pause instead of seek
https://bugs.webkit.org/show_bug.cgi?id=240222
rdar://92637636

Reviewed by Aditya Keerthi and Kate Cheney.

Add a new compile-time flag to guard the presence of -[UITextInteractionAssistant contextMenuInteraction]. See
WebKit/ChangeLog for more details.

  • wtf/PlatformHave.h:

[iOS] Double tapping on YouTube video causes playback to pause instead of seek
https://bugs.webkit.org/show_bug.cgi?id=240222
rdar://92637636

Reviewed by Aditya Keerthi and Kate Cheney.

On some recent versions of iOS, UITextInteractionAssistant installs its own context menu interaction by
default. This makes UIKit introduce a _UIRelationshipGestureRecognizer to the gesture recognizer graph when
using WKWebView when the text interaction assistant has been activated in order to implement the asynchronous
context menu configuration API.

Currently, we add a direct failure requirement from the immediately resettable deferring gesture to this new
UIKit relationship gesture. This is problematic, since the context menu interaction will automatically add
failure requirements to other long press gestures on the web view, such as the image analysis gestures that
trigger Live Text analysis when long pressing on images with text. As a result, the set of immediately
resettable gestures is now connected to the set of delayed resettable gestures after the first tap when
performing a double tap gesture. Since the UIWebTouchEventsGestureRecognizer is a part of this immediate reset
subgraph, this means that the touch event gesture recognizer doesn't get reset until about 320 ms after a tap
when the text interaction assistant has been activated, due to this new context menu interaction.

On YouTube.com, this manifests in a double-tap on the main video player being broken since a double tap gesture
only dispatches touchstart and touchend on the first tap, and YouTube's script only seeks forwards or
backwards if it observes two touchstart events that are close to each other within 350 ms.

To fix this, we simply teach the deferring gesture subgraph partitioning logic in
-deferringGestureRecognizer:shouldDeferOtherGestureRecognizer: to funnel the relationship gesture (i.e.
-gestureRecognizerForFailureRelationships) of the text interaction assistant's context menu interaction into
the delayed reset subgraph instead of the immediate reset subgraph.

Test: fast/events/touch/ios/touch-events-when-double-tapping-after-selecting-text.html

  • UIProcess/ios/WKContentViewInteraction.mm:

(-[WKContentView deferringGestureRecognizer:shouldDeferOtherGestureRecognizer:]):

  • UIProcess/ios/WKDeferringGestureRecognizer.mm:

(-[WKDeferringGestureRecognizer shouldDeferGestureRecognizer:]):

Canonical link: https://commits.webkit.org/250418@main

9:52 AM Changeset in webkit [293979] by pvollan@apple.com
  • 3 edits in trunk/Source/WebKit

Create reports for long process launch times
https://bugs.webkit.org/show_bug.cgi?id=240127

Unreviewed, fix typo.

  • UIProcess/AuxiliaryProcessProxy.cpp:

(WebKit::AuxiliaryProcessProxy::connect):
(WebKit::AuxiliaryProcessProxy::didFinishLaunching):

  • UIProcess/AuxiliaryProcessProxy.h:
9:41 AM Changeset in webkit [293978] by Tyler Wilcock
  • 2 edits in trunk/Source/WebCore

AXLogger::streamAXCoreObject is missing a null check for AccessibilityObject dynamicDowncast
https://bugs.webkit.org/show_bug.cgi?id=240228

Reviewed by Andres Gonzalez.

When checking if an object has display:contents, we do:

if (auto* axObject = dynamicDowncast<AccessibilityObject>(&object); axObject->hasDisplayContents())

Which does not have a nullcheck for when the dynamicDowncast fails (i.e. because
the object is not an AccessibilityObject).

  • accessibility/AXLogger.cpp:

(WebCore::streamAXCoreObject):

9:22 AM Changeset in webkit [293977] by Kate Cheney
  • 25 edits
    2 adds in trunk

Image controls menu button is not appearing for multi-page PDFs
https://bugs.webkit.org/show_bug.cgi?id=240120
rdar://86425721

Reviewed by Megan Gardner.

Source/WebCore:

Test: fast/attachment/attachment-image-controls-basic.html

Refactor image controls button code so it can also be used for PDF
attachments.

  • dom/mac/ImageControlsMac.cpp:

(WebCore::ImageControlsMac::handleEvent):
(WebCore::ImageControlsMac::isImageMenuEnabled):
(WebCore::ImageControlsMac::updateImageControls):
(WebCore::ImageControlsMac::tryCreateImageControls):
(WebCore::ImageControlsMac::destroyImageControls):
(WebCore::ImageControlsMac::hasImageControls):

  • dom/mac/ImageControlsMac.h:

Handle the PDF case when a click on the image controls button happens.
Move all image controls code to ImageControlsMac where it can be
shared by image and attachment elements.

  • html/HTMLAttachmentElement.cpp:

(WebCore::HTMLAttachmentElement::parseAttribute):
(WebCore::HTMLAttachmentElement::childShouldCreateRenderer const):

  • html/HTMLAttachmentElement.h:
  • html/HTMLImageElement.cpp:

(WebCore::HTMLImageElement::parseAttribute):
(WebCore::HTMLImageElement::didAttachRenderers):
(WebCore::HTMLImageElement::setAttachmentElement):
(WebCore::HTMLImageElement::updateImageControls): Deleted.
(WebCore::HTMLImageElement::tryCreateImageControls): Deleted.
(WebCore::HTMLImageElement::destroyImageControls): Deleted.
(WebCore::HTMLImageElement::hasImageControls const): Deleted.

  • html/HTMLImageElement.h:

(WebCore::HTMLImageElement::isImageMenuEnabled const):
(WebCore::HTMLImageElement::setImageMenuEnabled):
(WebCore::HTMLImageElement::imageMenuEnabled const): Deleted.

  • html/shadow/mac/imageControlsMac.css:

The image controls button was originally offset by 20px from the top
and the right. This causes the image controls button to appear in the
middle of a PDF attachment and be difficult to see. This change makes
it flush against the top right corner for both images and PDFs.

(button#image-controls-button):

  • page/ChromeClient.h:

(WebCore::ChromeClient::handlePDFServiceClick):

  • rendering/RenderAttachment.cpp:

(WebCore::RenderAttachment::RenderAttachment):
(WebCore::RenderAttachment::layout):
(WebCore::RenderAttachment::layoutShadowContent):

  • rendering/RenderAttachment.h:

Add canHaveGeneratedChildren() and canHaveChildren() so that
we render the image controls button in the shadow tree.

  • rendering/RenderImage.cpp:

(WebCore::RenderImage::RenderImage):

  • testing/Internals.cpp:

(WebCore::Internals::hasImageControls const):

Source/WebKit:

Refactor image controls button code so it can also be used for PDF
attachments.

  • Shared/ContextMenuContextData.cpp:

(WebKit::ContextMenuContextData::controlledDataIsEditable const):
We need to make sure PDF attachments are marked as editable so the
context menu gets properly populated.

  • Shared/ContextMenuContextData.h:

(WebKit::ContextMenuContextData::ContextMenuContextData):

  • UIProcess/API/Cocoa/APIAttachmentCocoa.mm:

(API::Attachment::enclosingImageNSData const):
Remove the image check so we return the PDF NSData in the attachment
case.

  • UIProcess/mac/WebContextMenuProxyMac.mm:

(WebKit::WebContextMenuProxyMac::setupServicesMenu):

  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::handlePDFServiceClick):

  • WebProcess/WebCoreSupport/WebChromeClient.h:
  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/mac/WebPageMac.mm:

(WebKit::WebPage::handlePDFServiceClick):
Plumbing for handling a click on the image controls button for PDFs.

LayoutTests:

  • TestExpectations:
  • fast/attachment/attachment-image-controls-basic.html: Added.
  • platform/mac-wk2/TestExpectations:
  • platform/mac/fast/attachment/attachment-image-controls-basic-expected.txt: Added.
8:48 AM Changeset in webkit [293976] by Ziran Sun
  • 4 edits in trunk

Make input placeholder line-height declaration !important
https://bugs.webkit.org/show_bug.cgi?id=240225

Reviewed by Tim Nguyen.

Source/WebCore:

This is the same the changes made in chromium at
https://chromium-review.googlesource.com/c/chromium/src/+/3594234
and Firefox change at
https://phabricator.services.mozilla.com/D116907

  • css/html.css:

(input::placeholder):

LayoutTests:

Unskip the test that passes.

8:17 AM Changeset in webkit [293975] by graouts@webkit.org
  • 3 edits
    2 adds in trunk

REGRESSION (r291817): NativeImage passed to RemoteResourceCacheProxy::recordNativeImageUse may be null
https://bugs.webkit.org/show_bug.cgi?id=239649
rdar://92018859

Reviewed by Dean Jackson.

Test: system-preview/svg-image.html

In the case of an <img> pointing to an SVG resource, the nativeImage() is null. We need to handle this case
properly (filed bug 239673), but for now we can at least not crash.

  • platform/graphics/displaylists/DisplayListRecorder.cpp:

(WebCore::DisplayList::Recorder::drawSystemImage):

Canonical link: https://commits.webkit.org/250413@main

7:57 AM Changeset in webkit [293974] by keith_miller@apple.com
  • 2 edits in trunk/JSTests

Unreviewed test gardening.

  • test262/expectations.yaml:
7:46 AM Changeset in webkit [293973] by keith_miller@apple.com
  • 2 edits in trunk/Tools

Test262 runner should show progress
https://bugs.webkit.org/show_bug.cgi?id=240181

Reviewed by Yusuke Suzuki.

The progress will appear as something like [42/245669].

  • Scripts/test262/Runner.pm:

(processCLI):
(main):

Canonical link: https://commits.webkit.org/250411@main

7:18 AM WebKitGTK/2.36.x edited by Adrian Perez de Castro
(diff)
7:05 AM Changeset in webkit [293972] by youenn@apple.com
  • 3 edits in trunk/Source/WebCore

CoreAudioCaptureSource::settingsDidChange should not reconfigure the audio unit if CoreAudioCaptureSource is not started
https://bugs.webkit.org/show_bug.cgi?id=240059

Reviewed by Eric Carlson.

We should only ask to reconfigure when source settings change if the source is actually started.
Otherwise, we can wait for the source to start to actually set the unit values and reconfigure if needed.
To make sure to correctly expose settings, we reset them in CoreAudioCaptureSource::settingsDidChange and in
CoreAudioCaptureSource::initializeToStartProducingData.
We also only use the audio unit sample rate if it is rendering audio, otherwise we can change the sample rate at will.

Manually tested.

  • platform/mediastream/mac/CoreAudioCaptureSource.cpp:
3:54 AM Changeset in webkit [293971] by magomez@igalia.com
  • 2 edits in trunk/Source/WebCore

[Nicosia] Canvas animations don't work with threaded rendering
https://bugs.webkit.org/show_bug.cgi?id=227760
<rdar://problem/80588913>

Reviewed by Adrian Perez de Castro.

Implement Nicosia::CairoOperationRecorder::draImageBuffer(), which is required in order to paint
canvas contents into a GraphicsContext.

  • platform/graphics/nicosia/cairo/NicosiaCairoOperationRecorder.cpp:

(Nicosia::CairoOperationRecorder::drawImageBuffer):

3:47 AM Changeset in webkit [293970] by Manuel Rego Casasnovas
  • 4 edits in trunk

[WinCairo][WK1] accessibility/aria-combobox-control-owns-elements.html is crashing after 250325@main
https://bugs.webkit.org/show_bug.cgi?id=240218
<rdar://problem/92929718>

Reviewed by Joanmarie Diggs.

Source/WebCore:

Fix a wrong ASSERT for AriaReflectionForElementReferencesEnabled.
If AriaReflectionForElementReferencesEnabled is disabled, and you set
one of the attributes that have reflection under that flag,
we'll hit an ASSERT in Element::attributeChanged().

The fact that we set such attribute doesn't mean that the flag
has to be enabled.

Removed the ASSERT and added an if to check if the flag is enabled,
otherwise we don't need to do anything with the map.

  • dom/Element.cpp:

(WebCore::Element::attributeChanged):

LayoutTests:

  • platform/wincairo-wk1/TestExpectations: Mark test as timeout as it

was before r293958.

1:41 AM Changeset in webkit [293969] by Diego Pino Garcia
  • 8 edits in trunk/Source

Unreviewed, non-unified build fixes after r293562

Source/WebCore:

  • workers/shared/context/SharedWorkerThread.cpp:
  • workers/shared/context/SharedWorkerThreadProxy.cpp:

Source/WebKit:

  • NetworkProcess/SharedWorker/WebSharedWorkerServerConnection.h:
  • WebProcess/Storage/WebSharedWorkerContextManagerConnection.cpp:

(WebKit::WebSharedWorkerContextManagerConnection::launchSharedWorker):

Source/WTF:

  • wtf/text/StringCommon.h:

May 8, 2022:

11:28 PM Changeset in webkit [293968] by Said Abou-Hallawa
  • 4 edits in trunk/Source

[macOS] REGRESSION (r293825): Find highlight snapshots are incorrectly scaled
https://bugs.webkit.org/show_bug.cgi?id=240203
rdar://92892014

Reviewed by Tim Horton.

Source/WebCore:

takeSnapshots() depends on the snapshot ImageBuffer::resolutionScale() to
set the size of the TextIndicatorData image. r293825 scaled the size of
the ImageBuffer before creation and moved the scaling to the GraphicsContext.
So we have correct scaled pixels but the resolutionScale is 1. So we get
enlarged incorrect image.

The fix is to revert r293825 and fix the iOS snapshot without having to
change snapshotFrameRectWithClip().

  • page/FrameSnapshotting.cpp:

(WebCore::snapshotFrameRectWithClip):

Source/WebKit:

In getShareableBitmapForImageBufferWithQualifiedIdentifier(), we used to
pass the backendSize as the srcRect and the backendSize as the destRect
to GraphicsContext::drawImageBuffer(). The backendSize is the logicalSize
scaled by the resolutionScale. But in ImageBufferCGBackend::draw() we
scale the srcRect by the resolutionScale one more time. This double-
scaled srcRect draws a srcRect whose size = backendSize * resolutionScale
to a destRect whose size = backendSize. And this results in shrinking the
desired snapshot image by 1 / resolutionScale.

  • GPUProcess/graphics/RemoteRenderingBackend.cpp:

(WebKit::RemoteRenderingBackend::getShareableBitmapForImageBufferWithQualifiedIdentifier):

11:14 PM Changeset in webkit [293967] by ysuzuki@apple.com
  • 18 edits in trunk/Source

Introduce EventTrackingRegions::Event enum
https://bugs.webkit.org/show_bug.cgi?id=240206

Reviewed by Mark Lam.

We noticed that EventNames 260~ AtomStrings are allocated in scrolling thread only because we are using
eventNames() for EventTrackingRegions. But since use of it is limited, we can just use enum instead.

  1. We can make EventTrackingRegions more efficient by using enum instead of String.
  2. We can save memory by avoiding EventNames string allocations & AtomStringTable registration.
  3. We can make this parameter more strictly typed compared to accepting any kind of Strings
  • Source/WebKit/Shared/RemoteLayerTree/RemoteScrollingCoordinatorTransaction.cpp:

(WebKit::dump):

  • Source/WebKit/Shared/WebCoreArgumentCoders.cpp:

(IPC::ArgumentCoder<EventTrackingRegions>::decode):

  • Source/WebKit/UIProcess/RemoteLayerTree/RemoteScrollingCoordinatorProxy.cpp:

(WebKit::RemoteScrollingCoordinatorProxy::eventTrackingTypeForPoint const):

  • Source/WebKit/UIProcess/RemoteLayerTree/RemoteScrollingCoordinatorProxy.h:
  • Source/WebKit/UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::updateTouchEventTracking):

  • Source/WebCore/page/DebugPageOverlays.cpp:

(WebCore::NonFastScrollableRegionOverlay::drawRect):

  • Source/WebCore/page/Page.cpp:

(WebCore::Page::touchEventRectsForEventForTesting):

  • Source/WebCore/page/Page.h:
  • Source/WebCore/page/scrolling/ScrollingCoordinator.cpp:

(WebCore::ScrollingCoordinator::absoluteEventTrackingRegionsForFrame const):

  • Source/WebCore/page/scrolling/ScrollingStateFrameScrollingNode.cpp:

(WebCore::ScrollingStateFrameScrollingNode::dumpProperties const):

  • Source/WebCore/page/scrolling/ScrollingTree.cpp:

(WebCore::ScrollingTree::computeWheelProcessingSteps):
(WebCore::ScrollingTree::eventTrackingTypeForPoint):

  • Source/WebCore/page/scrolling/ScrollingTree.h:
  • Source/WebCore/platform/EventTrackingRegions.cpp:

(WebCore::EventTrackingRegions::eventName):
(WebCore::EventTrackingRegions::trackingTypeForPoint):
(WebCore::EventTrackingRegions::uniteSynchronousRegion):

  • Source/WebCore/platform/EventTrackingRegions.h:
  • Source/WebCore/testing/Internals.cpp:

(WebCore::Internals::touchEventRectsForEvent):

  • Source/WebCore/testing/Internals.h:

Canonical link: https://commits.webkit.org/250405@main

9:05 PM Changeset in webkit [293966] by Russell Epstein
  • 2 edits in branches/safari-7614.1.12-branch/Source/WebKit

Cherry-pick r293945. rdar://problem/92859012

Fix another internal build
https://bugs.webkit.org/show_bug.cgi?id=240201
<rdar://92859012>

Patch by Alex Christensen <achristensen@webkit.org> on 2022-05-06
Reviewed by Alexey Proskuryakov.

Bug 240184 introduced a classic "but I have spaces in my path" bug.

  • Configurations/adattributiond.xcconfig:

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

9:05 PM Changeset in webkit [293965] by Russell Epstein
  • 4 edits
    1 add in branches/safari-7614.1.12-branch/Source/WebGPU

Cherry-pick r293937. rdar://problem/92524485

[WebGPU] Fix build on downlevel OSes in Apple's build system
https://bugs.webkit.org/show_bug.cgi?id=240159
<rdar://problem/92524485>

Reviewed by Alexey Proskuryakov.

Some variables from WebKitTargetConditionals.xcconfig and
PlatformSupport.xcconfig are used, so we have to make sure
we include them. Also refactor the INSTALL_PATH variable a
bit.

  • Configurations/Base.xcconfig:
  • Configurations/WebGPU.xcconfig:
  • Configurations/WebKitTargetConditionals.xcconfig: Added.
  • WebGPU.xcodeproj/project.pbxproj:

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

9:04 PM Changeset in webkit [293964] by Russell Epstein
  • 2 edits in branches/safari-7614.1.12-branch/Source/WebKit

Cherry-pick r293936. rdar://problem/92859012

Fix adattributiond build in internal build on older macOS
https://bugs.webkit.org/show_bug.cgi?id=240184
<rdar://92859012>

Patch by Alex Christensen <achristensen@webkit.org> on 2022-05-06
Reviewed by Alexey Proskuryakov.

We need to search for WK_OVERRIDE_FRAMEWORKS_DIR to find the right frameworks

  • Configurations/adattributiond.xcconfig:

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

8:55 PM Changeset in webkit [293963] by Russell Epstein
  • 1 copy in tags/WebKit-7614.1.11.8

Tag WebKit-7614.1.11.8.

8:54 PM Changeset in webkit [293962] by Russell Epstein
  • 2 edits in branches/safari-614.1.11-branch/Source/WebKit

Cherry-pick r293945. rdar://problem/92859012

Fix another internal build
https://bugs.webkit.org/show_bug.cgi?id=240201
<rdar://92859012>

Patch by Alex Christensen <achristensen@webkit.org> on 2022-05-06
Reviewed by Alexey Proskuryakov.

Bug 240184 introduced a classic "but I have spaces in my path" bug.

  • Configurations/adattributiond.xcconfig:

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

8:54 PM Changeset in webkit [293961] by Russell Epstein
  • 4 edits
    1 add in branches/safari-614.1.11-branch/Source/WebGPU

Cherry-pick r293937. rdar://problem/92524485

[WebGPU] Fix build on downlevel OSes in Apple's build system
https://bugs.webkit.org/show_bug.cgi?id=240159
<rdar://problem/92524485>

Reviewed by Alexey Proskuryakov.

Some variables from WebKitTargetConditionals.xcconfig and
PlatformSupport.xcconfig are used, so we have to make sure
we include them. Also refactor the INSTALL_PATH variable a
bit.

  • Configurations/Base.xcconfig:
  • Configurations/WebGPU.xcconfig:
  • Configurations/WebKitTargetConditionals.xcconfig: Added.
  • WebGPU.xcodeproj/project.pbxproj:

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

8:54 PM Changeset in webkit [293960] by Russell Epstein
  • 2 edits in branches/safari-614.1.11-branch/Source/WebKit

Cherry-pick r293936. rdar://problem/92859012

Fix adattributiond build in internal build on older macOS
https://bugs.webkit.org/show_bug.cgi?id=240184
<rdar://92859012>

Patch by Alex Christensen <achristensen@webkit.org> on 2022-05-06
Reviewed by Alexey Proskuryakov.

We need to search for WK_OVERRIDE_FRAMEWORKS_DIR to find the right frameworks

  • Configurations/adattributiond.xcconfig:

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

8:45 PM Changeset in webkit [293959] by Russell Epstein
  • 9 edits in branches/safari-614.1.11-branch/Source

Versioning.

WebKit-7614.1.11.8

5:19 PM Changeset in webkit [293958] by hironori.fujii@sony.com
  • 2 edits in trunk/LayoutTests

[WinCairo][WK1] accessibility/aria-combobox-control-owns-elements.html is crashing after 250325@main
https://bugs.webkit.org/show_bug.cgi?id=240218
<rdar://problem/92929718>

Unreviewed test gardening.

  • platform/wincairo-wk1/TestExpectations: Skip it.

Canonical link: https://commits.webkit.org/250404@main

10:08 AM Changeset in webkit [293957] by rniwa@webkit.org
  • 2 edits in trunk

Introduction.md: Fix typos found by mcatanzaro
https://bugs.webkit.org/show_bug.cgi?id=240211

Reviewed by Tim Nguyen.

Fixed typos.

  • Introduction.md:
12:12 AM Changeset in webkit [293956] by Antti Koivisto
  • 15 edits in trunk

Avoid resolving style for elements that only inherit changes from parent
https://bugs.webkit.org/show_bug.cgi?id=240132
<rdar://problem/92867346>

Reviewed by Sam Weinig.

Source/WebCore:

When an inherited property changes we need to update all descendant styles too. Currently this
is done by simply running the full style resolution for them including selector matching and
style building steps.

In the common case nothing changes in descendants expect the inherited properties. This patch implements
a fast-path mechanism that allows us to simply copy the relevant properties from the parent, skipping
all the other work. This is possible if we can prove that the value of these properties is not changed
by any rules targeting the child and that there are no other properties affected by this value change.

The mechanism is only implemented for the 'color' property for now. It can be expanded for other properties later.
It would be especially useful for custom properties.

  • css/CSSProperties.json:
  • css/makeprop.pl:

Add "fast-path-inherited" codegen property that disables the fast-path if the property is explicitly mutated.

(generateInitialValueSetter):
(generateValueSetter):

  • rendering/style/RenderStyle.cpp:

(WebCore::RenderStyle::RenderStyle):
(WebCore::RenderStyle::fastPathInheritFrom):

Inherit fast path properties.

(WebCore::RenderStyle::fastPathInheritedEqual const):
(WebCore::RenderStyle::nonFastPathInheritedEqual const):

Test for fast-path property changes.

  • rendering/style/RenderStyle.h:

(WebCore::RenderStyle::disallowsFastPathInheritance const):
(WebCore::RenderStyle::setDisallowsFastPathInheritance):

Track if can use the mechanism.

(WebCore::RenderStyle::NonInheritedFlags::operator== const):

  • rendering/style/StyleInheritedData.cpp:

(WebCore::StyleInheritedData::operator== const):
(WebCore::StyleInheritedData::fastPathInheritedEqual const):
(WebCore::StyleInheritedData::nonFastPathInheritedEqual const):
(WebCore::StyleInheritedData::fastPathInheritFrom):

  • rendering/style/StyleInheritedData.h:
  • style/StyleBuilderCustom.h:

(WebCore::Style::BuilderCustom::applyValueFill):
(WebCore::Style::BuilderCustom::applyValueStroke):

  • style/StyleBuilderState.cpp:

(WebCore::Style::BuilderState::colorFromPrimitiveValueWithResolvedCurrentColor const):

Color value 'currentcolor' is mostly resolved during use time but in the few places we still resolve it during style building
we need to disable the fast-path, since it makes other properties depend on 'color' property.

  • style/StyleChange.cpp:

(WebCore::Style::determineChange):

New style change type for fast-path inheritance.

  • style/StyleChange.h:
  • style/StyleTreeResolver.cpp:

(WebCore::Style::TreeResolver::styleForStyleable):

Resolve by just copying properties from the parent.

(WebCore::Style::TreeResolver::computeDescendantsToResolve):
(WebCore::Style::TreeResolver::resolveElement):
(WebCore::Style::TreeResolver::determineResolutionType):

Use fast-path if the only reason we are resolving this element is that some supported inherited property changed.

(WebCore::Style::TreeResolver::resolveComposedTree):
(WebCore::Style::TreeResolver::shouldResolveElement): Deleted.

  • style/StyleTreeResolver.h:

Tools:

  • Scripts/webkitpy/style/checkers/jsonchecker.py:

(JSONCSSPropertiesChecker.check_codegen_properties):

May 7, 2022:

10:50 PM Changeset in webkit [293955] by rniwa@webkit.org
  • 7 edits in trunk/Source/WebCore

Add helper functions to queue task on Node with GCReachableRef
https://bugs.webkit.org/show_bug.cgi?id=240204

Reviewed by Chris Dumez.

Added Node::queueTaskKeepingThisNodeAlive, which queues a task to the event loop while keeping "this" Node
and its JS wrapper alive, and Node::queueTaskToDispatchEvent, which queues a task to the event loop to
dispatch an event on "this" Node while keeping it and its JS wrapper alive, and deployed them in elements.

  • Modules/model-element/HTMLModelElement.cpp:

(WebCore::HTMLModelElement::setSourceURL): Need to disambiguate which queueTaskToDispatchEvent to use
between Node and ActiveDOMObject.
(WebCore::HTMLModelElement::notifyFinished): Ditto.

  • dom/Node.cpp:

(WebCore::Node::queueTaskKeepingThisNodeAlive): Added.
(WebCore::Node::queueTaskToDispatchEvent): Added.

  • dom/Node.h:
  • html/HTMLDetailsElement.cpp:

(WebCore::HTMLDetailsElement::parseAttribute): Use newly added functions.

  • html/HTMLDialogElement.cpp:

(WebCore::HTMLDialogElement::close): Ditto.
(WebCore::HTMLDialogElement::queueCancelTask): Ditto.

  • html/HTMLTextFormControlElement.cpp:

(WebCore::HTMLTextFormControlElement::scheduleSelectEvent): Ditto.

9:19 PM Changeset in webkit [293954] by Chris Dumez
  • 2 edits in trunk/Source/WebKit

Do WebKitAdditions header replacement only when a specific environment variable is set
https://bugs.webkit.org/show_bug.cgi?id=240210
<rdar://92885915>

Reviewed by Tim Horton.

  • Source/WebKit/Configurations/WebKit.xcconfig:
  • Source/WebKit/mac/replace-webkit-additions-includes.py:

(check_should_do_replacement):
(main):
(is_supported_os): Deleted.

Canonical link: https://commits.webkit.org/250400@main

8:47 PM Changeset in webkit [293953] by Simon Fraser
  • 2 edits in trunk/Source/WebCore

Use an OptionSet iterator to skip directly to GraphicsContextState::Change bits to handle
https://bugs.webkit.org/show_bug.cgi?id=239954
<rdar://problem/92593424>

Patch by Cameron McCormack <Cameron McCormack> on 2022-05-07
Reviewed by Simon Fraser.

It would be rare for display list SetState items to have many Change
bits set on them. Instead of checking for all 15 bits in mergeChanges,
we can use OptionSet's iterator to skip directly to each changed bit.

Using ctz to turn the Change bit into a bit position helps the
compiler generate compact code to jump to each case in the switch
statement.

On an iPad I tested with, this is a ~2% win on the MotionMark Design
subtest, 5.5% on Leaves, and 1.4% overall.

  • platform/graphics/GraphicsContextState.cpp:

(WebCore::toIndex):
(WebCore::GraphicsContextState::mergeChanges):

7:41 PM Changeset in webkit [293952] by mark.lam@apple.com
  • 2 edits in trunk/Source/bmalloc

Force PAS_ASSERT to generate different crash sites for each assertion.
https://bugs.webkit.org/show_bug.cgi?id=240209

Reviewed by Yusuke Suzuki.

Clang currently optimizes all crash sites into one in each function. Hence, if we
get a crash address at the 1 crash site, we don't know which failed assertion got
us there. This patch uses an asm statement to force Clang to emit a different
crash site for each assertion.

Benchmarks show that performance is neutral on both Jetstream2 and Speedometer2.

Size-wise, there is some increase. The following is the "size" output on
JavaScriptCore on M1:

TEXT. DATA OBJC others dec hex

old 19628032 180224 0 18792448 38600704 24d0000
new 19644416 180224 0 19251200 39075840 2544000

diff 16384 0 0 458752 475136

The increase in the "others" categories are mostly in the String Table, Symbol
Table, and Function Start Addresses. These take up disk space but should not
impact RAM usage unless they are accessed by a a debugger.

  • libpas/src/libpas/pas_utils.h:

(pas_assertion_failed):

4:34 PM Changeset in webkit [293951] by Cameron McCormack
  • 13 edits
    2 adds in trunk

Don't propagate GraphicsContextState change bits into TextPainter's glyph display list recorder
https://bugs.webkit.org/show_bug.cgi?id=239952
<rdar://problem/92635604>

Source/WebCore:

Reviewed by Said Abou-Hallawa and Antti Koivisto.

In FontCascade::displayListForTextRun, we create a
DisplayList::Recorder, then call drawGlyphBuffer. We initialize the
DisplayList::Recorder with the GraphicsContextState of the
GraphicsContext we're drawing to. Just before this, we will have set the
current fill color on that GraphicsContext.

When GPUP DOM rendering is disabled, GraphicsContextCG responds to
setFillColor etc. by updating GraphicsContextState, including setting
the Change flag, then immediately updating the CGContext, and clearing
the Change flag.

But when GPUP DOM rendering is enabled, the GraphicsContext is a
DisplayList::Recorder for the layer we're painting in to. Because
DisplayList::Recorder applies its state changes lazily, it can be in the
situation where its GraphicsContextState has had the fill brush changed,
and the Change flag is still set. So DisplayList::Recorder starts off
with a GraphicsContextState with unapplied changes in it. We end up in
DisplayList::Recorder::drawGlyphsAndCacheFont, which calls
appendStateChangeItemIfNecessary, which sees that the Change bit is set,
and generates a SetInlineFillColor display list item, which is
recorded and then replayed the next time the same text is painted.
This recorded fill color then may be wrong for the next TextPainter
that wants to reuse the cached glyph display list.

Display list recorders should never be initialized with a
GraphicsContextState that has change flags set on it. We can assert
this, then make FontCascade explicitly clear those flags on the state
object it passes in to the DisplayList::Recorder.

Test: fast/text/glyph-display-list-color.html

  • platform/graphics/FontCascade.cpp:

(WebCore::FontCascade::displayListForTextRun const):

  • platform/graphics/GraphicsContextState.cpp:

(WebCore::GraphicsContextState::cloneForRecording const):

  • platform/graphics/GraphicsContextState.h:
  • platform/graphics/displaylists/DisplayListRecorder.cpp:

(WebCore::DisplayList::Recorder::Recorder):

Add setForceUseGlyphDisplayListForTesting and
cachedGlyphDisplayListsForTextNode functions on Internal for the
test to use:

  • rendering/GlyphDisplayListCache.h:

(WebCore::GlyphDisplayListCache::getIfExists):

  • rendering/TextPainter.cpp:

(WebCore::TextPainter::shouldUseGlyphDisplayList):
(WebCore::TextPainter::setForceUseGlyphDisplayListForTesting):
(WebCore::TextPainter::cachedGlyphDisplayListsForTextNodeAsText):

  • rendering/TextPainter.h:

(WebCore::TextPainter::glyphDisplayListIfExists):

  • testing/Internals.cpp:

(WebCore::Internals::setForceUseGlyphDisplayListForTesting):
(WebCore::Internals::cachedGlyphDisplayListsForTextNode):

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

LayoutTests:

Reviewed by Antti Koivisto.

  • fast/text/glyph-display-list-color-expected.txt: Added.
  • fast/text/glyph-display-list-color.html: Added.
11:15 AM Changeset in webkit [293950] by rniwa@webkit.org
  • 2 edits in trunk

Explain now node reference counting works in Introduction.md
https://bugs.webkit.org/show_bug.cgi?id=240202

Unreviewed. Fix an obvious typo.

  • Introduction.md:
10:58 AM Changeset in webkit [293949] by rniwa@webkit.org
  • 2 edits in trunk

Explain now node reference counting works in Introduction.md
https://bugs.webkit.org/show_bug.cgi?id=240202

Reviewed by Chris Dumez.

Added explanation on how Node reference counting works.

  • Introduction.md:
10:34 AM Changeset in webkit [293948] by Chris Dumez
  • 8 edits in trunk/Source

Modernize / Optimize HTMLInputElement a bit
https://bugs.webkit.org/show_bug.cgi?id=240194

Reviewed by Darin Adler.

  • Source/WebCore/html/ColorInputType.h:
  • Source/WebCore/html/FileInputType.cpp:

(WebCore::FileInputType::firstElementPathForInputValue const):
(WebCore::FileInputType::files): Deleted.
(WebCore::FileInputType::canSetValue): Deleted.
(WebCore::FileInputType::getTypeSpecificValue): Deleted.

  • Source/WebCore/html/FileInputType.h:
  • Source/WebCore/html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::HTMLInputElement):
(WebCore::HTMLInputElement::endEditing):
(WebCore::HTMLInputElement::updateType):
(WebCore::HTMLInputElement::accessKeyAction):
(WebCore::HTMLInputElement::initializeInputType):
(WebCore::HTMLInputElement::parseAttribute):
(WebCore::HTMLInputElement::finishParsingChildren):
(WebCore::HTMLInputElement::appendFormData):
(WebCore::HTMLInputElement::reset):
(WebCore::HTMLInputElement::setChecked):
(WebCore::HTMLInputElement::setIndeterminate):
(WebCore::HTMLInputElement::value const):
(WebCore::HTMLInputElement::valueWithDefault const):
(WebCore::HTMLInputElement::setValue):
(WebCore::HTMLInputElement::defaultEventHandler):
(WebCore::HTMLInputElement::acceptMIMETypes const):
(WebCore::HTMLInputElement::acceptFileExtensions const):
(WebCore::HTMLInputElement::setShowAutoFillButton):
(WebCore::HTMLInputElement::files):
(WebCore::HTMLInputElement::setFiles):
(WebCore::HTMLInputElement::onSearch):
(WebCore::HTMLInputElement::prepareForDocumentSuspension):
(WebCore::HTMLInputElement::valueAsColor const):
(WebCore::HTMLInputElement::selectColor):
(WebCore::HTMLInputElement::suggestedColors const):
(WebCore::HTMLInputElement::dataList const):
(WebCore::HTMLInputElement::updateValueIfNeeded):
(WebCore::HTMLInputElement::isInRequiredRadioButtonGroup):
(WebCore::HTMLInputElement::radioButtonGroup const):
(WebCore::HTMLInputElement::checkedRadioButtonForGroup const):
(WebCore::HTMLInputElement::size const): Deleted.
(WebCore::HTMLInputElement::setValueForUser): Deleted.
(WebCore::HTMLInputElement::acceptMIMETypes): Deleted.
(WebCore::HTMLInputElement::acceptFileExtensions): Deleted.
(WebCore::HTMLInputElement::canReceiveDroppedFiles const): Deleted.

  • Source/WebCore/html/HTMLInputElement.h:

(WebCore::HTMLInputElement::size const):
(WebCore::HTMLInputElement::setValueForUser):
(WebCore::HTMLInputElement::hasDirtyValue const):
(WebCore::HTMLInputElement::hasAutoFillStrongPasswordButton const):
(WebCore::HTMLInputElement::canReceiveDroppedFiles const):

  • Source/WebCore/html/InputType.cpp:

(WebCore::InputType::files): Deleted.
(WebCore::InputType::setFiles): Deleted.
(WebCore::InputType::getTypeSpecificValue): Deleted.
(WebCore::InputType::canSetValue): Deleted.
(WebCore::InputType::valueAsColor const): Deleted.
(WebCore::InputType::selectColor): Deleted.
(WebCore::InputType::suggestedColors const): Deleted.

  • Source/WebCore/html/InputType.h:

(WebCore::InputType::supportsValidation const):
(WebCore::InputType::InputType):
(WebCore::InputType::canHaveTypeSpecificValue const): Deleted.

Canonical link: https://commits.webkit.org/250394@main

May 6, 2022:

10:27 PM Changeset in webkit [293947] by Megan Gardner
  • 11 edits in trunk

Fix flakey test by using the old API on old systems.
https://bugs.webkit.org/show_bug.cgi?id=240195

Reviewed by Tim Horton.

Source/WebKit:

Turn fast/images/text-recognition/ios/show-data-detector-context-menu.html back on.

  • UIProcess/ios/WKActionSheetAssistant.mm:

(-[WKActionSheetAssistant contextMenuInteraction:configuration:highlightPreviewForItemWithIdentifier:contextMenuInteraction:previewForHighlightingMenuWithConfiguration:]):
(-[WKActionSheetAssistant contextMenuInteraction:configuration:highlightPreviewForItemWithIdentifier:]): Deleted.

  • UIProcess/ios/WKContentViewInteraction.mm:

(-[WKContentView contextMenuInteraction:configuration:highlightPreviewForItemWithIdentifier:contextMenuInteraction:previewForHighlightingMenuWithConfiguration:]):
(-[WKContentView contextMenuInteraction:configuration:dismissalPreviewForItemWithIdentifier:contextMenuInteraction:previewForDismissingMenuWithConfiguration:]):
(-[WKContentView contextMenuInteraction:configuration:highlightPreviewForItemWithIdentifier:]): Deleted.
(-[WKContentView contextMenuInteraction:configuration:dismissalPreviewForItemWithIdentifier:]): Deleted.

  • UIProcess/ios/WebDataListSuggestionsDropdownIOS.mm:

(-[WKDataListSuggestionsDropdown contextMenuInteraction:configuration:highlightPreviewForItemWithIdentifier:contextMenuInteraction:previewForHighlightingMenuWithConfiguration:]):
(-[WKDataListSuggestionsDropdown contextMenuInteraction:configuration:highlightPreviewForItemWithIdentifier:]): Deleted.

  • UIProcess/ios/forms/WKDateTimeInputControl.mm:

(-[WKDateTimePicker contextMenuInteraction:configuration:highlightPreviewForItemWithIdentifier:contextMenuInteraction:previewForHighlightingMenuWithConfiguration:]):
(-[WKDateTimePicker contextMenuInteraction:configuration:highlightPreviewForItemWithIdentifier:]): Deleted.

  • UIProcess/ios/forms/WKFileUploadPanel.mm:

(-[WKFileUploadPanel contextMenuInteraction:configuration:highlightPreviewForItemWithIdentifier:contextMenuInteraction:previewForHighlightingMenuWithConfiguration:]):
(-[WKFileUploadPanel contextMenuInteraction:configuration:highlightPreviewForItemWithIdentifier:]): Deleted.

  • UIProcess/ios/forms/WKFormSelectPicker.mm:

(-[WKSelectPicker contextMenuInteraction:configuration:highlightPreviewForItemWithIdentifier:contextMenuInteraction:previewForHighlightingMenuWithConfiguration:]):
(-[WKSelectPicker contextMenuInteraction:configuration:highlightPreviewForItemWithIdentifier:]): Deleted.

Source/WTF:

  • wtf/PlatformHave.h:

LayoutTests:

  • platform/ios/TestExpectations:
9:16 PM Changeset in webkit [293946] by Wenson Hsieh
  • 2 edits in trunk/Source/WebCore

Block-style image overlay containers should have lighter shadows
https://bugs.webkit.org/show_bug.cgi?id=240198
rdar://92890194

Reviewed by Tim Horton.

Make some minor adjustments to the style of block-style image overlays. The short and long box-shadows
underneath these boxes should be much lighter, and the font-weight should be normal (400) instead of bold.

  • html/shadow/imageOverlay.css:

(div.image-overlay-block):

Canonical link: https://commits.webkit.org/250392@main

9:02 PM Changeset in webkit [293945] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit

Fix another internal build
https://bugs.webkit.org/show_bug.cgi?id=240201
<rdar://92859012>

Patch by Alex Christensen <achristensen@webkit.org> on 2022-05-06
Reviewed by Alexey Proskuryakov.

Bug 240184 introduced a classic "but I have spaces in my path" bug.

  • Configurations/adattributiond.xcconfig:
7:17 PM Changeset in webkit [293944] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[WebXR] Update color format used for the multisample render buffer
https://bugs.webkit.org/show_bug.cgi?id=240179

Patch by Ada Chan <adachan@apple.com> on 2022-05-06
Reviewed by Dean Jackson.

  • Modules/webxr/WebXROpaqueFramebuffer.cpp:

(WebCore::WebXROpaqueFramebuffer::setupFramebuffer):

7:07 PM Changeset in webkit [293943] by Simon Fraser
  • 6 edits in trunk/Source/WebCore

Optimize shouldApplyContainment methods
https://bugs.webkit.org/show_bug.cgi?id=240164

Reviewed by Alan Bujtas.

Based on a patch by Rob Buis.

Optimize shouldApplyContainment methods by allowing OptionSet as a parameter.
Also restrict the principal box checks to a single location.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::computeIntrinsicLogicalWidths const):

  • rendering/RenderBlockFlow.cpp:

(WebCore::RenderBlockFlow::computeIntrinsicLogicalWidths const):

  • rendering/RenderElement.h:

(WebCore::RenderElement::canContainFixedPositionObjects const):
(WebCore::RenderElement::canContainAbsolutelyPositionedObjects const):
(WebCore::RenderElement::shouldApplyLayoutOrPaintContainment const):
(WebCore::RenderElement::shouldApplySizeOrStyleContainment const):
(WebCore::RenderElement::shouldApplyLayoutContainment const):
(WebCore::RenderElement::shouldApplyPaintContainment const):
(WebCore::RenderElement::shouldApplySizeContainment const):
(WebCore::RenderElement::shouldApplyInlineSizeContainment const):
(WebCore::RenderElement::shouldApplyStyleContainment const):
(WebCore::RenderElement::shouldApplyAnyContainment const):

  • rendering/RenderFlexibleBox.cpp:

(WebCore::RenderFlexibleBox::computeIntrinsicLogicalWidths const):

  • rendering/RenderGrid.cpp:

(WebCore::RenderGrid::layoutBlock):
(WebCore::RenderGrid::computeEmptyTracksForAutoRepeat const):

6:52 PM Changeset in webkit [293942] by Ross Kirsling
  • 5 edits in trunk

Temporal.Duration#toString should never ignore fractionalSecondDigits
https://bugs.webkit.org/show_bug.cgi?id=240193

Reviewed by Yusuke Suzuki.

This patch implements the spec correction of https://github.com/tc39/proposal-temporal/pull/1956:
new Temporal.Duration.from(1).toString({ fractionalSecondDigits: 2 }) should be P1Y0.00S, not just P1Y.

  • stress/temporal-duration.js: Add a test case.
  • test262/expectations.yaml: Mark two test cases as passing.
  • runtime/TemporalDuration.cpp:

(JSC::TemporalDuration::toString):

Canonical link: https://commits.webkit.org/250388@main

6:41 PM Changeset in webkit [293941] by commit-queue@webkit.org
  • 21 edits
    6 adds in trunk/Source

Unreviewed, reverting r293824.
https://bugs.webkit.org/show_bug.cgi?id=240197

Speedometer2/Angular2-TypeScript-TodoMVC 8% regression

Reverted changeset:

"SWOriginStore is no longer needed"
https://bugs.webkit.org/show_bug.cgi?id=240003
https://commits.webkit.org/r293824

6:38 PM Changeset in webkit [293940] by Patrick Griffis
  • 3 edits
    2 adds in trunk

CSP: Fix script-src-elem policies in workers
https://bugs.webkit.org/show_bug.cgi?id=239840

Reviewed by Kate Cheney.

Source/WebCore:

Test: http/tests/security/contentSecurityPolicy/script-src-strict-dynamic-and-script-src-elem.html

  • page/csp/ContentSecurityPolicyDirectiveList.cpp:

(WebCore::ContentSecurityPolicyDirectiveList::violatedDirectiveForScript const):

LayoutTests:

CSP: Fix script-src-elem policies in workers

  • http/tests/security/contentSecurityPolicy/script-src-strict-dynamic-and-script-src-elem-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/script-src-strict-dynamic-and-script-src-elem.html: Added.
6:32 PM Changeset in webkit [293939] by ntim@apple.com
  • 2 edits in trunk/Source/WebCore

Use dynamicDowncast in getPathFromPathOperation()
https://bugs.webkit.org/show_bug.cgi?id=240190

Reviewed by Chris Dumez.

Small cleanup.

  • style/StyleBuilderConverter.h:

(WebCore::Style::BuilderConverter::convertPathOperation):

6:15 PM Changeset in webkit [293938] by Wenson Hsieh
  • 12 edits in trunk

Don't show "Markup Image" item for images that have already been marked up
https://bugs.webkit.org/show_bug.cgi?id=240133
rdar://92064534

Reviewed by Aditya Keerthi.

Source/WebKit:

Add some logic to avoid showing the "Markup Image" menu item on both macOS (in the context menu) and iOS (in the
callout bar or editing context menu) for image elements that were created and inserted in the process of
invoking "Markup Image". See below for more details.

Tests: ImageAnalysisTests.AllowImageAnalysisMarkupOnce

  • UIProcess/Cocoa/WebPageProxyCocoa.mm:
  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::shouldAllowImageMarkup):

Add an async reply IPC message that the UI process uses to determine whether or not we should enable image
markup for an element corresponding to the given ElementContext.

  • UIProcess/WebPageProxy.h:

Also move the declaration of replaceImageWithMarkupResults into the ENABLE(IMAGE_ANALYSIS_ENHANCEMENTS)
section.

  • UIProcess/ios/WKContentViewInteraction.mm:

(-[WKContentView doAfterComputingImageAnalysisResultsForMarkup:]):

  • UIProcess/mac/WebContextMenuProxyMac.mm:

(WebKit::WebContextMenuProxyMac::appendMarkupItemToControlledImageMenuIfNeeded):

Consult shouldAllowImageMarkup() before triggering image analysis when determining whether or not we should
include the "Markup Image" item.

  • WebProcess/WebPage/Cocoa/WebPageCocoa.mm:

(WebKit::WebPage::replaceImageWithMarkupResults):

Keep track of image elements that were created to replace existing images when performing image markup. To
achieve this, we scan the selection range before the caret after replacing the image to find the newly inserted
image and add it to m_elementsToExcludeFromMarkup.

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::didCommitLoad):

Clear out the m_imageElementsToExcludeFromMarkup vector.

(WebKit::WebPage::shouldAllowImageMarkup const):

  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/WebPage.messages.in:

Tools:

Add an API test to verify that the option to Markup Image goes away after invoking Markup Image on an image
element.

  • TestWebKitAPI/Tests/WebKitCocoa/ImageAnalysisTests.mm:

(TestWebKitAPI::runMarkupTest):
(TestWebKitAPI::TEST):

Canonical link: https://commits.webkit.org/250384@main

5:48 PM Changeset in webkit [293937] by mmaxfield@apple.com
  • 4 edits
    1 add in trunk/Source/WebGPU

[WebGPU] Fix build on downlevel OSes in Apple's build system
https://bugs.webkit.org/show_bug.cgi?id=240159
<rdar://problem/92524485>

Reviewed by Alexey Proskuryakov.

Some variables from WebKitTargetConditionals.xcconfig and
PlatformSupport.xcconfig are used, so we have to make sure
we include them. Also refactor the INSTALL_PATH variable a
bit.

  • Configurations/Base.xcconfig:
  • Configurations/WebGPU.xcconfig:
  • Configurations/WebKitTargetConditionals.xcconfig: Added.
  • WebGPU.xcodeproj/project.pbxproj:
5:41 PM Changeset in webkit [293936] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit

Fix adattributiond build in internal build on older macOS
https://bugs.webkit.org/show_bug.cgi?id=240184
<rdar://92859012>

Patch by Alex Christensen <achristensen@webkit.org> on 2022-05-06
Reviewed by Alexey Proskuryakov.

We need to search for WK_OVERRIDE_FRAMEWORKS_DIR to find the right frameworks

  • Configurations/adattributiond.xcconfig:
5:15 PM Changeset in webkit [293935] by Ross Kirsling
  • 4 edits in trunk

ISO8601::Duration should guard against -0
https://bugs.webkit.org/show_bug.cgi?id=240185

Reviewed by Yusuke Suzuki.

Currently, when we parse a negative ISO duration string or negate a positive Duration object,
we end up storing -0 for the zero fields. This patch ensures that we always store +0 instead.

  • test262/expectations.yaml:

Mark 32 test cases as passing.

  • runtime/ISO8601.h:

Add guards for zero cases.

Canonical link: https://commits.webkit.org/250381@main

5:10 PM Changeset in webkit [293934] by Stephanie Lewis
  • 1 edit in trunk/Tools/Scripts/webkitpy/benchmark_runner/browser_driver/osx_chrome_driver.py

Enable field trial configs when testing Chrome Betas
https://bugs.webkit.org/show_bug.cgi?id=240187

Reviewed by Saam Barati.

  • Tools/Scripts/webkitpy/benchmark_runner/browser_driver/osx_chrome_driver.py:

(OSXChromeDriverBase):
(OSXChromeDriverBase.launch_args_with_url):
(OSXChromeDriverBase.launch_url):
(OSXChromeCanaryDriver.launch_args_with_url):
(OSXChromeBetaDriver.launch_args_with_url):
(OSXChromeDevDriver._set_chrome_binary_location):
(OSXChromeDevDriver):
(OSXChromeDevDriver.launch_args_with_url):

Canonical link: https://commits.webkit.org/250380@main

5:08 PM Changeset in webkit [293933] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

Use correct document as root for lazy image observer
https://bugs.webkit.org/show_bug.cgi?id=240083

Patch by Rob Buis <rbuis@igalia.com> on 2022-05-06
Reviewed by Simon Fraser.

LayoutTests/imported/w3c:

Add a test for scrolling a lazy loaded image into view in an iframe.

  • web-platform-tests/html/semantics/embedded-content/the-img-element/scrolling-below-viewport-image-lazy-loading-in-iframe-expected.txt: Added.
  • web-platform-tests/html/semantics/embedded-content/the-img-element/scrolling-below-viewport-image-lazy-loading-in-iframe.html: Added.

Source/WebCore:

Use correct document as root for lazy image observer, meaning
elements in the main frame will use the main frame document and
elements in iframes will use the iframe document as root.

Test: imported/w3c/web-platform-tests/html/semantics/embedded-content/the-img-element/scrolling-below-viewport-image-lazy-loading-in-iframe.html

  • html/LazyLoadImageObserver.cpp:

(WebCore::LazyLoadImageObserver::intersectionObserver):

5:04 PM Changeset in webkit [293932] by Alan Coon
  • 1 copy in tags/WebKit-7614.1.12.1

Tag WebKit-7614.1.12.1.

5:01 PM Changeset in webkit [293931] by pvollan@apple.com
  • 2 edits in trunk/Source/WebCore

[macOS] Content filter blocking shield is not displayed
https://bugs.webkit.org/show_bug.cgi?id=240178
<rdar://91586177>

Reviewed by Geoffrey Garen.

This is a regression from moving content filtering from the WebContent process to the Network process in r291630.
After r291630, the function to determine if a blocked request can be unblocked, is called in the Network process,
but it is not handling the case where the function to unblock the request is not set. This is causing the load
for blocked URLs to never finish, since the decision handler provided to the unblock request function is never
called. This patch addresses this by calling the decisionhandler with the blocked flag set in this case.

  • platform/cocoa/ContentFilterUnblockHandlerCocoa.mm:

(WebCore::ContentFilterUnblockHandler::requestUnblockAsync const):

4:17 PM Changeset in webkit [293930] by Jonathan Bedard
  • 4 edits in trunk/Tools

[git-webkit] Forward environment to git commit
https://bugs.webkit.org/show_bug.cgi?id=240191
<rdar://problem/92885927>

Reviewed by Stephanie Lewis.

  • Scripts/libraries/webkitscmpy/setup.py: Bump version.
  • Scripts/libraries/webkitscmpy/webkitscmpy/init.py: Ditto.
  • Scripts/libraries/webkitscmpy/webkitscmpy/program/pull_request.py:

(PullRequest.create_commit): Pass caller's environment.

Canonical link: https://commits.webkit.org/250377@main

4:10 PM Changeset in webkit [293929] by pvollan@apple.com
  • 2 edits in trunk/Source/WebCore

[macOS] HTTP traffic is not filtered in the parental controls filter
https://bugs.webkit.org/show_bug.cgi?id=240180
<rdar://problem/92875540>

Reviewed by Geoffrey Garen.

Traditionally, we have not filtered HTTP in the parental controls filter on macOS, since other parts of the system
has taken care of this. This appears to have changed now, and WebKit should also filter HTTP in addition to HTTPS.

  • platform/cocoa/ParentalControlsContentFilter.mm:

(WebCore::canHandleResponse):

3:33 PM Changeset in webkit [293928] by Brent Fulgham
  • 5 edits in trunk/Source

Remove unused ApplePayRemoteUIEnabled preference
https://bugs.webkit.org/show_bug.cgi?id=240177

Reviewed by Devin Rousso.

Remove the internal WKPreference for ApplePayRemoteUIEnabled now that
we always use the high-security remote ApplePay UI.

Source/WebKit:

  • WebProcess/ApplePay/WebPaymentCoordinator.cpp:

(WebKit::WebPaymentCoordinator::networkProcessConnectionClosed):
(WebKit::WebPaymentCoordinator::messageSenderConnection const):
(WebKit::WebPaymentCoordinator::remoteUIEnabled const): Deleted.

  • WebProcess/ApplePay/WebPaymentCoordinator.h:

Source/WTF:

  • Scripts/Preferences/WebPreferencesInternal.yaml:
3:20 PM Changeset in webkit [293927] by Karl Rackler
  • 2 edits in trunk/LayoutTests

[Gardening] REGRESSION (r293117): [ iOS ] fast/innerHTML/001.html is a flaky image failure
https://bugs.webkit.org/show_bug.cgi?id=240170

Unreviewed test gardening.

Canonical link: https://commits.webkit.org/250374@main

3:06 PM Changeset in webkit [293926] by Manuel Rego Casasnovas
  • 3 edits
    2 adds in trunk

[selectors] Double script focus after mouse click shouldn't match :focus-visible
https://bugs.webkit.org/show_bug.cgi?id=239472
<rdar://problem/92301472>

Reviewed by Antti Koivisto.

LayoutTests/imported/w3c:

  • web-platform-tests/css/selectors/focus-visible-script-focus-020-expected.txt: Added.
  • web-platform-tests/css/selectors/focus-visible-script-focus-020.html: Added.

Source/WebCore:

When you do the second script focus, we were setting
m_latestFocusTrigger to FocusTrigger::Bindings, that makes us lose the
information about the previous element that was focused via mouse
click and start matching :focus-visible on that case.

Test: imported/w3c/web-platform-tests/css/selectors/focus-visible-script-focus-020.html

  • dom/Document.cpp:

(WebCore::Document::setFocusedElement): We avoid setting
m_latestFocusTrigger if we come from script focus. That way we know if
the previous focused element was focused via mouse click after several
script focus.

2:54 PM Changeset in webkit [293925] by ysuzuki@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

[JSC] Add more information about MarkedBlock assertion
https://bugs.webkit.org/show_bug.cgi?id=240176

Reviewed by Mark Lam and Saam Barati.

Collect more information about assertion via CRASH_WITH_INFO.

  • Source/JavaScriptCore/heap/MarkedBlockInlines.h:

(JSC::MarkedBlock::Handle::specializedSweep):

Canonical link: https://commits.webkit.org/250372@main

2:51 PM Changeset in webkit [293924] by Karl Rackler
  • 2 edits in trunk/LayoutTests

[Gardening][ iOS ] fast/innerHTML/001.html is a flaky image failure
https://bugs.webkit.org/show_bug.cgi?id=240170

Reviewed by Jonathan Bedard.

Canonical link: https://commits.webkit.org/250371@main

2:10 PM Changeset in webkit [293923] by Brent Fulgham
  • 7 edits in trunk/Source

Remove abandoned WKPreference for SelectionPaintingWithoutSelectionGaps
https://bugs.webkit.org/show_bug.cgi?id=240129

Reviewed by Alan Bujtas.

Remove the abandoned WKPreference 'SelectionPaintingWithoutSelectionGaps', which was added to support
the EFL port. As the EFL port is no longer buildable, and this code is never used outside of EFL, we
should remove this dead code.

Source/WebCore:

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::shouldPaintSelectionGaps const):

Source/WebKit:

  • UIProcess/API/C/WKPreferences.cpp:

(WKPreferencesSetSelectionPaintingWithoutSelectionGapsEnabled): Deleted.
(WKPreferencesGetSelectionPaintingWithoutSelectionGapsEnabled): Deleted.

  • UIProcess/API/C/WKPreferencesRefPrivate.h:

Source/WTF:

  • Scripts/Preferences/WebPreferences.yaml:
2:03 PM Changeset in webkit [293922] by gnavamarino@apple.com
  • 2 edits in trunk/Source/WebCore

null ptr deref in WebCore::Frame::setPrinting
https://bugs.webkit.org/show_bug.cgi?id=240173

Reviewed by Wenson Hsieh.

Check m_doc in Frame::setPrinting before access, as it is a RefPtr that can become null.

  • page/Frame.cpp:

(WebCore::Frame::setPrinting):

1:26 PM Changeset in webkit [293921] by Ben Nham
  • 26 edits
    2 adds in trunk

Add support for Notification objects with custom data
https://bugs.webkit.org/show_bug.cgi?id=240153

Reviewed by Chris Dumez.

This adds support for the data attribute in Notification. This holds an arbitrary structured
cloneable object passed through the constructor. The accessor is marked SameObject, which we
implement via CachedAttribute. The wire form of the data is serialized and deserialized in
NotificationData so that persistent notifications properly support this property.

  • Modules/notifications/Notification.cpp:

(WebCore::createSerializedNotificationData):
(WebCore::Notification::create):
(WebCore::Notification::createForServiceWorker):
(WebCore::Notification::Notification):
(WebCore::Notification::dataForBindings):

  • Modules/notifications/Notification.h:
  • Modules/notifications/Notification.idl:
  • Modules/notifications/NotificationData.cpp:

(WebCore::NotificationData::isolatedCopy const):
(WebCore::NotificationData::isolatedCopy):

  • Modules/notifications/NotificationData.h:

(WebCore::NotificationData::encode const):
(WebCore::NotificationData::decode):

  • Modules/notifications/NotificationDataCocoa.mm:

(WebCore::NotificationData::fromDictionary):
(WebCore::NotificationData::dictionaryRepresentation const):

  • Modules/notifications/NotificationOptions.idl:
  • workers/service/ServiceWorkerRegistration.cpp:

(WebCore::ServiceWorkerRegistration::showNotification):

LayoutTests/imported/w3c:

  • web-platform-tests/notifications/idlharness.https.any-expected.txt:
  • web-platform-tests/notifications/idlharness.https.any.serviceworker-expected.txt:
  • web-platform-tests/wasm/serialization/module/serialization-via-notifications-api.any-expected.txt:

LayoutTests:

  • http/tests/notifications/notification-expected.txt:
  • http/tests/notifications/notification.html:
  • http/tests/workers/service/getnotifications-expected.txt:
  • http/tests/workers/service/getnotifications-stop-expected.txt:
  • http/tests/workers/service/getnotifications-stop.html:
  • http/tests/workers/service/getnotifications.html:
  • http/tests/workers/service/openwindow-from-notification-click.html:
  • http/tests/workers/service/resources/shownotification-openwindow-worker.js:

(async tryShow):
(async getNotes):

  • http/tests/workers/service/resources/shownotification-worker.js:

(async event):
(async tryShow):
(async tryShowInvalidData.try.data):
(async tryShowInvalidData):
(async getNotes):

  • http/tests/workers/service/shownotification-allowed-document-expected.txt:
  • http/tests/workers/service/shownotification-allowed-document.html:
  • http/tests/workers/service/shownotification-allowed.html:
  • http/tests/workers/service/shownotification-invalid-data-expected.txt: Added.
  • http/tests/workers/service/shownotification-invalid-data.html:

Canonical link: https://commits.webkit.org/250368@main

12:55 PM Changeset in webkit [293920] by Ben Nham
  • 3 edits in trunk/Source/WebKit

Improve Mac webpushd sandbox
https://bugs.webkit.org/show_bug.cgi?id=240110

Reviewed by Per Arne Vollan.

This tightens the webpushd sandbox to only allow essential operations.

The first ~280 lines or so are similar to the other WebKit sandboxes, although I removed
many of the lines in this prelude as most of the services listed in the other profiles
have no use in webpushd.

The last ~50 lines of the profile (starting with the comment about SQLite) allows the
operations necessary for functioning of the daemon:

  • Communicating with the push service daemon
  • Waking up processes in response to a push
  • Persisting subscriptions to the PushDatabase

I've also switched webpushd to read CF preferences directly like the other daemons, since our
sandbox profiles expect this.

  • webpushd/WebPushDaemonMain.mm:
  • webpushd/mac/com.apple.WebKit.webpushd.sb.in:

Canonical link: https://commits.webkit.org/250367@main

12:41 PM Changeset in webkit [293919] by Brent Fulgham
  • 19 edits in trunk/Source

Remove internal WKPreference for non-iTunesAVOutput now that all platform support it
https://bugs.webkit.org/show_bug.cgi?id=240141

Reviewed by Eric Carlson.

WebKit on macOS has used a different AV Output target than IOS_FAMILY targets because those platforms lacked
the AVOutput option. Now that all platforms share this target, we can consolidate on the correct endpoint and
remove this preference code.

Source/WebCore:

  • Modules/airplay/WebMediaSessionManager.cpp:

(WebCore::WebMediaSessionManager::showPlaybackTargetPicker):

  • Modules/airplay/WebMediaSessionManagerClient.h:

(WebCore::WebMediaSessionManagerClient::alwaysOnLoggingAllowed const):
(WebCore::WebMediaSessionManagerClient::useiTunesAVOutputContext const): Deleted.

  • platform/graphics/MediaPlaybackTargetPicker.h:
  • platform/graphics/avfoundation/objc/AVPlaybackTargetPicker.h:
  • platform/graphics/avfoundation/objc/AVRoutePickerViewTargetPicker.h:
  • platform/graphics/avfoundation/objc/AVRoutePickerViewTargetPicker.mm:

(WebCore::AVRoutePickerViewTargetPicker::outputContextInternal):
(WebCore::AVRoutePickerViewTargetPicker::showPlaybackTargetPicker):

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

(WebCore::MediaPlaybackTargetPickerMac::showPlaybackTargetPicker):

Source/WebKit:

  • UIProcess/WebPageProxy.h:
  • UIProcess/mac/WebPageProxyMac.mm:

(WebKit::WebPageProxy::useiTunesAVOutputContext const): Deleted.

Source/WTF:

  • Scripts/Preferences/WebPreferencesInternal.yaml:
12:26 PM Changeset in webkit [293918] by Chris Dumez
  • 6 edits in trunk/Source/WebCore

[IDL] Drop redundant [AtomString] on attributes marked as [Reflect] already
https://bugs.webkit.org/show_bug.cgi?id=240152

Reviewed by Darin Adler.

Drop redundant [AtomString] on attributes marked as [Reflect] already. All
attributes of string types and marked as [Reflect] now use AtomString
automatically, without having to specify it in the IDL.

  • Source/WebCore/accessibility/AccessibilityRole.idl:
  • Source/WebCore/accessibility/AriaAttributes.idl:
  • Source/WebCore/dom/Element.idl:
  • Source/WebCore/html/HTMLAnchorElement.idl:
  • Source/WebCore/html/HTMLHtmlElement.idl:

Canonical link: https://commits.webkit.org/250365@main

12:13 PM Changeset in webkit [293917] by Adrian Perez de Castro
  • 2 edits in releases/WebKitGTK/webkit-2.36/Source/WebCore

Merge r290718 - Fix deprecations for ERB.new in GenerateSettings.rb
https://bugs.webkit.org/show_bug.cgi?id=237237

Reviewed by Don Olmstead.

Ruby 3.1.0 reported the following warning:

GenerateSettings.rb:283: warning: Passing safe_level with the 2nd argument of ERB.new is deprecated. Do not use it, and specify other arguments as keyword arguments.
GenerateSettings.rb:283: warning: Passing trim_mode with the 3rd argument of ERB.new is deprecated. Use keyword argument like ERB.new(str, trim_mode: ...) instead.

r290104 and r290331 fixed the same problem for
GeneratePreferences.rb, but forgot GenerateSettings.rb.

  • Scripts/GenerateSettings.rb: Use the keyword argument for ERB.new for Ruby 2.6+.
12:07 PM Changeset in webkit [293916] by gnavamarino@apple.com
  • 2 edits in trunk/Source/WebCore

ASSERT in WebCore::StyleProperties::getGridTemplateValue
https://bugs.webkit.org/show_bug.cgi?id=240172

Reviewed by Dean Jackson.

Update StyleProperties::getGridTemplateValue with a more thorough check
as it is currently only checking the grid:none/grid:initial special keyword.

  • css/StyleProperties.cpp:

(WebCore::StyleProperties::getGridTemplateValue const):

12:06 PM Changeset in webkit [293915] by Adrian Perez de Castro
  • 3 edits in releases/WebKitGTK/webkit-2.36/Source/WebCore

[Nicosia] Images in webkit.org/blog/ don't show up with threaded rendering
https://bugs.webkit.org/show_bug.cgi?id=238259

Patch by Miguel Gomez <magomez@igalia.com> on 2022-05-06
Reviewed by Carlos Garcia Campos.

Implement Nicosia::CairoOperationRecorder::drawFilteredImageBuffer(), which is required
to render images that have CSS filters applied.

  • platform/graphics/nicosia/cairo/NicosiaCairoOperationRecorder.cpp:

(Nicosia::CairoOperationRecorder::drawFilteredImageBuffer):

  • platform/graphics/nicosia/cairo/NicosiaCairoOperationRecorder.h:
12:06 PM WebKitGTK/2.36.x edited by Adrian Perez de Castro
(diff)
12:06 PM Changeset in webkit [293914] by Adrian Perez de Castro
  • 4 edits in releases/WebKitGTK/webkit-2.36/Source/WebCore

Merge r293826 - REGRESSION(249114@main) [GTK] Crashes on shutdown if the display is not set
https://bugs.webkit.org/show_bug.cgi?id=239767

Patch by Carlos Garcia Campos <cgarcia@igalia.com> on 2022-05-05
Reviewed by Michael Catanzaro.

Handle the case of PlatformDisplay created with a nullptr GdkDisplay.

  • platform/graphics/PlatformDisplay.cpp:

(WebCore::PlatformDisplay::PlatformDisplay):

  • platform/graphics/wayland/PlatformDisplayWayland.cpp:

(WebCore::PlatformDisplayWayland::PlatformDisplayWayland):
(WebCore::PlatformDisplayWayland::~PlatformDisplayWayland):

  • platform/graphics/x11/PlatformDisplayX11.cpp:

(WebCore::PlatformDisplayX11::PlatformDisplayX11):
(WebCore::PlatformDisplayX11::~PlatformDisplayX11):

12:00 PM Changeset in webkit [293913] by Adrian Perez de Castro
  • 2 edits in releases/WebKitGTK/webkit-2.36/Source/WTF

Merge r290331 - Ruby ERB.new compatibility fix
https://bugs.webkit.org/show_bug.cgi?id=237035

Reviewed by Fujii Hironori.

ERB.new has changed its calling convention in newer ruby versions.
This was exposed by https://commits.webkit.org/247450@main, which
tried to silence the warning emitted by newer ruby versions.
Unfortunately, this resulted in failures with older ruby versions
(e.g.
https://build.webkit.org/#/builders/46/builds/11387/steps/8/logs/stdio).

Use the compatibility hack suggested by RuboCop to get this working
across ruby versions (without any warnings).

  • Scripts/GeneratePreferences.rb:
11:59 AM Changeset in webkit [293912] by Adrian Perez de Castro
  • 2 edits in releases/WebKitGTK/webkit-2.36/Source/WebKit

Merge r292265 - GTK doesn't compile with ENABLE_ACCESSIBILITY=0
https://bugs.webkit.org/show_bug.cgi?id=238669

Reviewed by Michael Catanzaro.

Fix GTK compilation with ENABLE_ACCESSIBILITY=0

No new tests.

  • UIProcess/API/gtk/WebKitWebViewBase.cpp:

(webkitWebViewBaseDispose):
(webkit_web_view_base_class_init):

11:53 AM Changeset in webkit [293911] by Adrian Perez de Castro
  • 2 edits in releases/WebKitGTK/webkit-2.36/Source/WebCore

Merge r293441 - [GTK] Initialize m_eglDisplay in PlatformDisplay::PlatformDisplay(GdkDisplay*)
https://bugs.webkit.org/show_bug.cgi?id=239760

Patch by Xi Ruoyao <xry111@mengyan1223.wang> on 2022-04-26
Reviewed by Michael Catanzaro.

No new tests because there is no behavior change.

  • platform/graphics/PlatformDisplay.cpp:

(WebCore::PlatformDisplay::PlatformDisplay):
initialize m_eglDisplay to EGL_NO_DISPLAY.

11:37 AM Changeset in webkit [293910] by Justin Michaud
  • 3 edits in trunk/Tools

Update PGO profile collection scripts to include MotionMark
https://bugs.webkit.org/show_bug.cgi?id=240171

Reviewed by Mark Lam.

  • Scripts/build-and-collect-pgo-profiles:
  • Scripts/pgo-profile:
11:09 AM WebKitGTK/2.36.x edited by Adrian Perez de Castro
(diff)
10:59 AM Changeset in webkit [293909] by Alan Coon
  • 2 edits in branches/safari-7614.1.12-branch/Source/WebKit

Cherry-pick r293818. rdar://problem/92751323

REGRESSION (r293716): macCatalyst WebKit build fails; overlapping content at /System/Library/FeatureFlags/Domain/WebKit.plist
https://bugs.webkit.org/show_bug.cgi?id=240099
<rdar://92751323>

Reviewed by Chris Dumez.

  • WebKit.xcodeproj/project.pbxproj: Avoid copying the feature flags plist for macCatalyst, since it installs into the same place as macOS.

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

10:59 AM Changeset in webkit [293908] by Alan Coon
  • 2 edits
    2 adds in branches/safari-7614.1.12-branch/Source/WebKit

Cherry-pick r293716. rdar://problem/92644672

REGRESSION (r292351): Processes no longer get the right jetsam priority on iOS
https://bugs.webkit.org/show_bug.cgi?id=239992
<rdar://problem/92644672>

Reviewed by Chris Dumez.

  • FeatureFlags/WebKit.plist: Added.
  • WebKit.xcodeproj/project.pbxproj: Temporarily reinstate RB_full_manage_WK_jetsam, since it is being read by a different project.

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

10:59 AM Changeset in webkit [293907] by J Pascoe
  • 2 edits in trunk/Source/WebKit

[WebAuthn] Get rid of ASCAgentProxy instance after success/error/cancel
https://bugs.webkit.org/show_bug.cgi?id=240143
rdar://problem/92825715

Reviewed by Brent Fulgham.

For internal reasons, the ASCAgentProxy instance cannot be reused, so we
should clear it after a request or cancellation.

  • UIProcess/WebAuthentication/Cocoa/WebAuthenticatorCoordinatorProxy.mm:
10:56 AM Changeset in webkit [293906] by Adrian Perez de Castro
  • 1 copy in releases/WPE WebKit/webkit-2.34.7/webkit-2.34

WPE WebKit 2.34.7

10:55 AM Changeset in webkit [293905] by Adrian Perez de Castro
  • 4 edits in releases/WebKitGTK/webkit-2.34

Unreviewed. Update OptionsWPE.cmake and NEWS for the 2.34.7 release

.:

  • Source/cmake/OptionsWPE.cmake: Bump version numbers.

Source/WebKit:

  • wpe/NEWS: Add release notes for 2.34.7
10:39 AM Changeset in webkit [293904] by Ross Kirsling
  • 5 edits in trunk/Source/JavaScriptCore

Temporal.Duration should handle -0 properly
https://bugs.webkit.org/show_bug.cgi?id=240145

Reviewed by Yusuke Suzuki.

r250284 broke a new test262 test which verifies that -0 is normalized to 0 in cases like
new Temporal.Duration(-0) and Temporal.Duration.from({years: -0 }).
This patch properly aligns these paths with the spec.

  • runtime/JSCJSValue.h:
  • runtime/JSCJSValueInlines.h:

(JSC::JSValue::toIntegerWithoutRounding const): Added.
(JSC::JSValue::toIntegerOrInfinity const):

  • runtime/TemporalDuration.cpp:

(JSC::TemporalDuration::fromDurationLike):

  • runtime/TemporalDurationConstructor.cpp:

(JSC::JSC_DEFINE_HOST_FUNCTION):

Canonical link: https://commits.webkit.org/250361@main

10:32 AM Changeset in webkit [293903] by Brent Fulgham
  • 18 edits in trunk

Remove the viewportFitEnabled WKPreference now that it is always on
https://bugs.webkit.org/show_bug.cgi?id=240147

Reviewed by Tim Horton.

Remove the WKPreference 'ViewportFitEnabled' now that it is always enabled on all platforms, and
we do not wish to turn it off in testing or in other debugging circumstances.

Source/WebCore:

  • dom/ViewportArguments.cpp:

(WebCore::setViewportFeature):

  • dom/ViewportArguments.h:

Source/WebKit:

  • UIProcess/API/ios/WKWebViewIOS.mm:

(viewportArgumentsFromDictionary):
(-[WKWebView _overrideViewportWithArguments:]):

Source/WebKitLegacy/mac:

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

(-[WebPreferences viewportFitEnabled]): Deleted.
(-[WebPreferences setViewportFitEnabled:]): Deleted.

  • WebView/WebPreferencesPrivate.h:

Source/WTF:

  • Scripts/Preferences/WebPreferences.yaml:

LayoutTests:

  • fast/css/variables/env/ios/safe-area-inset-env-set-expected.html:
  • fast/css/variables/env/ios/safe-area-inset-env-set.html:
  • fast/events/ios/rotation/safe-area-insets-during-safari-type-rotation.html:
  • fast/viewport/ios/viewport-fit-auto.html:
  • fast/viewport/ios/viewport-fit-contain.html:
  • fast/viewport/ios/viewport-fit-cover.html:
10:25 AM Changeset in webkit [293902] by Alan Coon
  • 9 edits in branches/safari-7614.1.12-branch/Source

Versioning.

WebKit-7614.1.12.1

10:23 AM Changeset in webkit [293901] by Kate Cheney
  • 2 edits in trunk

Unreviewed, add github info to contributors.json.

  • metadata/contributors.json:
10:10 AM Changeset in webkit [293900] by Simon Fraser
  • 30 edits in trunk/Source/WebCore

Make the various shouldApplyContainment() functions member functions on RenderElement
https://bugs.webkit.org/show_bug.cgi?id=240156

Reviewed by Alan Bujtas.

These were free functions, but make more sense as member functions on RenderElement
since most callers passed *this. Just one call site needs to downcast<> to
RenderElement.

No behavior change.

  • rendering/GridTrackSizingAlgorithm.cpp:
  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::computeIntrinsicLogicalWidths const):
(WebCore::RenderBlock::computeBlockPreferredLogicalWidths const):
(WebCore::RenderBlock::firstLineBaseline const):
(WebCore::RenderBlock::inlineBlockBaseline const):

  • rendering/RenderBlockFlow.cpp:

(WebCore::RenderBlockFlow::computeIntrinsicLogicalWidths const):
(WebCore::RenderBlockFlow::adjustBlockChildForPagination):
(WebCore::RenderBlockFlow::adjustSizeContainmentChildForPagination):
(WebCore::RenderBlockFlow::firstLineBaseline const):
(WebCore::RenderBlockFlow::inlineBlockBaseline const):
(WebCore::RenderBlockFlow::computeInlinePreferredLogicalWidths const):

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::styleDidChange):
(WebCore::RenderBox::updateLogicalHeight):
(WebCore::RenderBox::isUnsplittableForPagination const):
(WebCore::RenderBox::layoutOverflowRectForPropagation const):

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::updateFromStyle):

  • rendering/RenderButton.cpp:

(WebCore::RenderButton::baselinePosition const):

  • rendering/RenderCounter.cpp:

(WebCore::makeCounterNode):
(WebCore::RenderCounter::rendererSubtreeAttached):

  • rendering/RenderDeprecatedFlexibleBox.cpp:

(WebCore::RenderDeprecatedFlexibleBox::computeIntrinsicLogicalWidths const):

  • rendering/RenderElement.h:

(WebCore::RenderElement::canContainFixedPositionObjects const):
(WebCore::RenderElement::canContainAbsolutelyPositionedObjects const):
(WebCore::RenderElement::shouldApplyLayoutContainment const):
(WebCore::RenderElement::shouldApplySizeContainment const):
(WebCore::RenderElement::shouldApplyInlineSizeContainment const):
(WebCore::RenderElement::shouldApplyStyleContainment const):
(WebCore::RenderElement::shouldApplyPaintContainment const):
(WebCore::RenderElement::shouldApplyAnyContainment const):

  • rendering/RenderFileUploadControl.cpp:

(WebCore::RenderFileUploadControl::computeIntrinsicLogicalWidths const):

  • rendering/RenderFlexibleBox.cpp:

(WebCore::RenderFlexibleBox::computeIntrinsicLogicalWidths const):
(WebCore::RenderFlexibleBox::firstLineBaseline const):

  • rendering/RenderGrid.cpp:

(WebCore::RenderGrid::layoutBlock):
(WebCore::RenderGrid::computeEmptyTracksForAutoRepeat const):
(WebCore::RenderGrid::firstLineBaseline const):

  • rendering/RenderImage.cpp:

(WebCore::RenderImage::computeIntrinsicRatioInformation const):

  • rendering/RenderInline.h:
  • rendering/RenderLayer.cpp:

(WebCore::canCreateStackingContext):
(WebCore::RenderLayer::shouldBeCSSStackingContext const):
(WebCore::RenderLayer::setAncestorChainHasSelfPaintingLayerDescendant):
(WebCore::RenderLayer::setAncestorChainHasVisibleDescendant):
(WebCore::RenderLayer::calculateClipRects const):

  • rendering/RenderListBox.cpp:

(WebCore::RenderListBox::computeIntrinsicLogicalWidths const):
(WebCore::RenderListBox::baselinePosition const):

  • rendering/RenderMenuList.cpp:

(RenderMenuList::computeIntrinsicLogicalWidths const):

  • rendering/RenderObject.cpp:

(WebCore::objectIsRelayoutBoundary):
(WebCore::shouldApplyLayoutContainment): Deleted.
(WebCore::shouldApplySizeContainment): Deleted.
(WebCore::shouldApplyInlineSizeContainment): Deleted.
(WebCore::shouldApplyStyleContainment): Deleted.
(WebCore::shouldApplyPaintContainment): Deleted.
(WebCore::shouldApplyAnyContainment): Deleted.

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

(WebCore::RenderReplaced::computeAspectRatioInformationForRenderBox const):
(WebCore::RenderReplaced::computeIntrinsicRatioInformation const):

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

(WebCore::RenderSlider::computeIntrinsicLogicalWidths const):

  • rendering/RenderTable.cpp:

(WebCore::RenderTable::firstLineBaseline const):

  • rendering/RenderTextControl.cpp:

(WebCore::RenderTextControl::computeIntrinsicLogicalWidths const):

  • rendering/RenderVideo.cpp:

(WebCore::RenderVideo::calculateIntrinsicSize):

  • rendering/RenderView.cpp:

(WebCore::RenderView::rendererForRootBackground const):

  • rendering/svg/LegacyRenderSVGRoot.cpp:

(WebCore::LegacyRenderSVGRoot::computeIntrinsicRatioInformation const):

  • rendering/svg/RenderSVGRoot.cpp:

(WebCore::RenderSVGRoot::computeIntrinsicRatioInformation const):

  • style/ContainerQueryEvaluator.cpp:

(WebCore::Style::ContainerQueryEvaluator::evaluateSizeFeature const):

9:49 AM Changeset in webkit [293899] by jer.noble@apple.com
  • 7 edits
    4 adds in trunk

[Cocoa] Seeking into a xHE-AAC track backed by a SourceBuffer can stall playback
https://bugs.webkit.org/show_bug.cgi?id=239750
<rdar://91922569>

Reviewed by Eric Carlson.

Source/WebCore:

Test: media/media-source/media-mp4-xhe-aac.html

xHE-AAC audio has the concept of an "independant sample", which behaves similarly to an
i-frame in a video codec. This sample is a prerequisite for decoding all samples that
come after it.

When such a track is parsed, those samples are marked as "sync", meaning they, as well as all
subsequent samples before the playhead, will be enqueued as "non-displaying". However, the
AVSampleBufferAudioRenderer does not use the DoNotDisplay sample attachment to control this
behavior. Instead, it relies on the TrimDurationAtStart (which has the effect of setting the
duration of the sample to zero, rendering it inaudible). Without this attachment, the renderer
will get "stuck" waiting for playback to clear out it's queue of samples.

Adopt this TrimDurationAtStart attachment if a given CMSampleBuffer is of the Audio type.

  • platform/graphics/avfoundation/objc/MediaSampleAVFObjC.mm:

(WebCore::MediaSampleAVFObjC::createNonDisplayingCopy const):

Source/WebCore/PAL:

Fix a spelling error in the CoreMediaSoftLink macros.

  • pal/cf/CoreMediaSoftLink.cpp:
  • pal/cf/CoreMediaSoftLink.h:

LayoutTests:

  • media/media-source/content/test-xhe-aac-manifest.json: Added.
  • media/media-source/content/test-xhe-aac.m4a: Added.
  • media/media-source/media-mp4-xhe-aac-expected.txt: Added.
  • media/media-source/media-mp4-xhe-aac.html: Added.
9:43 AM Changeset in webkit [293898] by Karl Rackler
  • 2 edits in trunk/LayoutTests

[ iOS ] fast/innerHTML/001.html is a flaky image failure
https://bugs.webkit.org/show_bug.cgi?id=240170

Unreviewed test gardening.

Canonical link: https://commits.webkit.org/250356@main

9:40 AM Changeset in webkit [293897] by Patrick Griffis
  • 3 edits
    2 adds in trunk

CSP: Fix incorrect blocked-uri for inline scripts and strict-dynamic policies
https://bugs.webkit.org/show_bug.cgi?id=240136

Reviewed by Kate Cheney.

Source/WebCore:

Test: http/tests/security/contentSecurityPolicy/script-src-strict-dynamic-inline-report.py

  • page/csp/ContentSecurityPolicy.cpp:

(WebCore::ContentSecurityPolicy::allowNonParserInsertedScripts const):

LayoutTests:

  • http/tests/security/contentSecurityPolicy/script-src-strict-dynamic-inline-report-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/script-src-strict-dynamic-inline-report.py: Added.
9:29 AM Changeset in webkit [293896] by jer.noble@apple.com
  • 4 edits
    3 adds in trunk

Video playback fails when using postMessage() during a User Gesture
https://bugs.webkit.org/show_bug.cgi?id=239781
<rdar://91281385>

Reviewed by Eric Carlson.

Source/WebCore:

Test: workers/worker-user-gesture.html

Certain frameworks handle user gestures by sending messages through postMessage() to
to a non-shared worker, where the message is processed and a response is sent back
to the page. In the case of APIs which require a user gesture, this gesture is lost
during the response event, even if it was present in the originating postMessage().
Other web APIs, such as Fetch, have been modified to forward this user gesture through
to the resulting event handlers. This change will expand that forwarding behavior to
include postMessage() to a non-shared Worker context.

When a message is sent to a WorkerContext, the current UserGestureToken is saved inside
a thread-safe RefCounted utility class WorkerUserGestureForwarder. This forwarder is
saved into an ivar of WorkerMessagingProxy, where it can be read during the handling
of a WorkerContext-originated postMessage(). The ivar is cleared immediately after sending
the initial postMessage(). The effect is that all calls to workerScope.postMessage()
made during the handling of a worker.postMessage() event will inherit the user gesture
of that initial call.

  • workers/WorkerMessagingProxy.cpp:

(WebCore::WorkerUserGestureForwarder::create):
(WebCore::WorkerUserGestureForwarder::~WorkerUserGestureForwarder):
(WebCore::WorkerUserGestureForwarder::userGestureToForward const):
(WebCore::WorkerUserGestureForwarder::WorkerUserGestureForwarder):
(WebCore::WorkerMessagingProxy::postMessageToWorkerObject):
(WebCore::WorkerMessagingProxy::postMessageToWorkerGlobalScope):

  • workers/WorkerMessagingProxy.h:

LayoutTests:

  • workers/worker-user-gesture-expected.txt: Added.
  • workers/worker-user-gesture.html: Added.
  • workers/worker-user-gesture.js: Added.
9:26 AM Changeset in webkit [293895] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Add lambda for fragmented flow state change handling
https://bugs.webkit.org/show_bug.cgi?id=240165

Patch by Rob Buis <rbuis@igalia.com> on 2022-05-06
Reviewed by Alan Bujtas.

Add lambda for fragmented flow state change handling
and call it in the case where the parent is affected.

  • rendering/updating/RenderTreeBuilder.cpp:

(WebCore::RenderTreeBuilder::normalizeTreeAfterStyleChange):

9:09 AM Changeset in webkit [293894] by jer.noble@apple.com
  • 4 edits in trunk/Source

[iOS] Infinite recursion at -[WKFullScreenWindowController _exitFullscreenImmediately]
https://bugs.webkit.org/show_bug.cgi?id=239744
<rdar://74201964>

Reviewed by Eric Carlson.

Source/WebCore:

Even after fullscreen is correctly torn down during an immediate exit, fullscreen can never again
be entered for the lifetime of the page. This is due to the ivar m_pendingExitFullscreen being set
to true with no opportunity for that state to be cleared. This occurs because the callbacks for
exiting fullscreen are performed before the lambda form FullscreenManager::exitFullscreen() is
called.

Set m_pendingExitFullscreen inside exitFullscreen(), rather than the lambda, and clear it explicitly
in the cases where fullscreen exiting is not actually necessary.

The mock behavior of Fullscreen inside DumpRenderTree and WebKitTestRunner are not able to exercise
this same path. Track improvements to this mock in bug #239747.

  • dom/FullscreenManager.cpp:

(WebCore::FullscreenManager::requestFullscreenForElement):
(WebCore::FullscreenManager::exitFullscreen):
(WebCore::FullscreenManager::setAnimatingFullscreen):
(WebCore::FullscreenManager::setFullscreenControlsHidden):

Source/WebKit:

When exiting fullscreen immediately, we trigger the correct state transition by calling
-[self exitFullScreen]. However, when no fullscreen manager exists (as may happen during
navigation), -exitFullscreen may itself call -_exitFullscreenImmediately, which causes an
infinite recursion and stack overflow.

Unroll the implementation by performing all the same pieces as would have been done by
-exitFullScreen, but inline. Refactor out common behavior into -_reinsertWebViewUnderPlaceholder,
and call that both from -_exitFullscreenImmediately and _completedExitFullScreen.

Unfortunately, no API tests are possible here because TestWebKitAPI is not an actual iOS UI
application.

  • UIProcess/ios/fullscreen/WKFullScreenWindowControllerIOS.mm:

(-[WKFullScreenWindowController exitFullScreen]):
(-[WKFullScreenWindowController _reinsertWebViewUnderPlaceholder]):
(-[WKFullScreenWindowController _completedExitFullScreen]):
(-[WKFullScreenWindowController _exitFullscreenImmediately]):

8:58 AM Changeset in webkit [293893] by pvollan@apple.com
  • 3 edits in trunk/Source/WebKit

Create reports for long process launch times
https://bugs.webkit.org/show_bug.cgi?id=240127

Reviewed by Yusuke Suzuki.

We have reports that it can take a long time to launch WebKit processes in some cases. This is the time
it takes from the XPC message is sent until the XPC reply is received. Add reporting when this happens
in order to help diagnose the issue.

  • UIProcess/AuxiliaryProcessProxy.cpp:

(WebKit::AuxiliaryProcessProxy::connect):
(WebKit::AuxiliaryProcessProxy::didFinishLaunching):

  • UIProcess/AuxiliaryProcessProxy.h:
8:23 AM Changeset in webkit [293892] by pvollan@apple.com
  • 4 edits in trunk/Source/WebKit

[WP] Wait for Launch Services database after Network process connection has been established
https://bugs.webkit.org/show_bug.cgi?id=240125
<rdar://92107043>

Reviewed by Geoffrey Garen.

Since the Launch Services database is provided to the WebContent process by the Network process, it makes sense
to wait for the database when we're certain that the Network process is running. This should fix main thread hangs
in the cases where we before started waiting for the database before the Network process had launched. To support
this move, we also need to delay the initialization of accessibility in NSApplication, since that depends on having
the database available. This is now being done in WebPage::platformInitializeAccessibility, which is a natural
place for this initialization to take place.

  • WebProcess/WebPage/mac/WebPageMac.mm:

(WebKit::WebPage::platformInitializeAccessibility):

  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::ensureNetworkProcessConnection):

  • WebProcess/cocoa/WebProcessCocoa.mm:

(WebKit::WebProcess::platformInitializeWebProcess):

8:21 AM Changeset in webkit [293891] by Jonathan Bedard
  • 3 edits in trunk/Tools

[ews-build.webkit.org] Do not block EWS queues for merging-blocked label
https://bugs.webkit.org/show_bug.cgi?id=240169
<rdar://problem/92860451>

Reviewed by Aakash Jain.

  • Tools/CISupport/ews-build/steps.py:

(ValidateChange.validate_github): merging-blocked label should only block merge-queue.

  • Tools/CISupport/ews-build/steps_unittest.py:

Canonical link: https://commits.webkit.org/250349@main

8:13 AM Changeset in webkit [293890] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit

GPU Process crash under IPC::Connection::open() ASSERTION FAILED: m_sendPort
https://bugs.webkit.org/show_bug.cgi?id=240146

Patch by Kimmo Kinnunen <kkinnunen@apple.com> on 2022-05-06
Reviewed by Chris Dumez.

If WP creates connection to GPUP but then exits, the
sent mach send right for communication comes out as MACH_PORT_DEAD_NAME in GPUP.
This will prevent proper initialization of IPC::Connection. This will
later assert in IPC::Connection::open().

Fix by checking identifierIsValid(). This will succeed only if the
listening port was alive at the time the send port was received.

Speculative fix.
Regression from r293829.

  • GPUProcess/GPUProcess.cpp:

(WebKit::GPUProcess::createGPUConnectionToWebProcess):

8:02 AM Changeset in webkit [293889] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

[Flatpak SDK] Include port / build type in shell prompt
https://bugs.webkit.org/show_bug.cgi?id=240166

Patch by Philippe Normand <pnormand@igalia.com> on 2022-05-06
Reviewed by Adrian Perez de Castro.

This might help avoiding confusions like running debug tools in a release shell, or wpe
things in a gtk shell.

  • flatpak/flatpakutils.py:

(WebkitFlatpak.run_in_sandbox):

Canonical link: https://commits.webkit.org/250347@main

7:50 AM Changeset in webkit [293888] by Karl Rackler
  • 2 edits in trunk/LayoutTests

[ iOS Release ] fast/css-custom-paint/animate-repaint.html is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=240167

Unreviewed test gardening.

Canonical link: https://commits.webkit.org/250346@main

7:45 AM Changeset in webkit [293887] by Adrian Perez de Castro
  • 2 edits
    2 adds in trunk/Tools/buildstream

Update mold to version 1.2.1 https://bugs.webkit.org/show_bug.cgi?id=240162

Reviewed by Philippe Normand.

Update Mold to version 1.2.1, which now properly supports --gdb-index and Clang's
ThinLTO (on top of regular LTO support). The copy of libtbb bundled with the Mold
sources does no longer build with the rest of tooling from the SDK, so this patch
adds Buildstream elements for it and its hwloc dependency; both these two packages
are built with the minimal set of options needed to build Mold.

  • elements/sdk/hwloc.bst: Added.
  • elements/sdk/mold.bst: Updated to version 1.2.1
  • elements/sdk/tbb.bst: Added.

Canonical link: https://commits.webkit.org/250345@main

5:55 AM WebKitFlatpakSDK/DebugWithRR edited by Manuel Rego Casasnovas
(diff)
2:52 AM Changeset in webkit [293886] by youenn@apple.com
  • 19 edits
    1 copy in trunk/Source

Allow MediaPlayerPrivateMediaStreamAVFObjC to avoid pixel conformers if IOSurfaces are not allowed
https://bugs.webkit.org/show_bug.cgi?id=240113
rdar://92629845

Reviewed by Eric Carlson.

Source/WebCore:

Add a callback in MediaPlayerPrivateMediaStreamAVFObjC to allow use a custom way to create a NativeImage from a VideoFrame.
Small refactoring in PixelBufferConformerCV to share more code with WebKit.
Make sure that SharedVideoFrameReader/Writer support kCVPixelFormatType_420YpCbCr10BiPlanarFullRange.
Covered by existing tests.

  • WebCore.xcodeproj/project.pbxproj:
  • platform/cocoa/SharedVideoFrameInfo.mm:
  • platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaStreamAVFObjC.h:
  • platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaStreamAVFObjC.mm:
  • platform/graphics/cv/CVUtilities.h:
  • platform/graphics/cv/PixelBufferConformerCV.cpp:
  • platform/graphics/cv/PixelBufferConformerCV.h:

Source/WebKit:

Implement VideoFrame conversion to BGRA by sending IPC to GPUProcess, use pixel conformer in GPUProcess and send back the results to WebProcess.
This goes through RemoteVideoFrameObjectHeapProxy/RemoteVideoFrameObjectHeapProxyProcessor which exchanges with RemoteVideoFrameObjectHeap.
Make use of this conversion code path for MediaPlayerPrivateMediaStreamAVFObjC by setting MediaPlayerPrivateMediaStreamAVFObjC nativeImageCreator callback.

  • GPUProcess/media/RemoteVideoFrameObjectHeap.cpp:
  • GPUProcess/media/RemoteVideoFrameObjectHeap.h:
  • GPUProcess/media/RemoteVideoFrameObjectHeap.messages.in:
  • GPUProcess/media/RemoteVideoFrameObjectHeap.mm: Added.
  • SourcesCocoa.txt:
  • WebKit.xcodeproj/project.pbxproj:
  • WebProcess/GPU/media/RemoteMediaPlayerManager.cpp:
  • WebProcess/GPU/webrtc/RemoteVideoFrameObjectHeapProxy.h:
  • WebProcess/GPU/webrtc/RemoteVideoFrameObjectHeapProxyProcessor.cpp:
  • WebProcess/GPU/webrtc/RemoteVideoFrameObjectHeapProxyProcessor.h:
  • WebProcess/GPU/webrtc/RemoteVideoFrameObjectHeapProxyProcessor.messages.in:
2:30 AM WebKitGTK/2.36.x edited by Adrian Perez de Castro
(diff)
1:44 AM Changeset in webkit [293885] by youenn@apple.com
  • 10 edits in trunk/Source

Remove usage of PixelBufferConformer in RealtimeOutgoingVideoSourceCocoa
https://bugs.webkit.org/show_bug.cgi?id=240118

Reviewed by Eric Carlson.

Source/ThirdParty/libwebrtc:

Add a routine to convert RGB CVPixelBuffer to YUV webrtc VideoFrameBuffer.

  • Configurations/libwebrtc.iOS.exp:
  • Configurations/libwebrtc.iOSsim.exp:
  • Configurations/libwebrtc.mac.exp:
  • Source/webrtc/sdk/WebKit/WebKitUtilities.h:
  • Source/webrtc/sdk/WebKit/WebKitUtilities.mm:

(webrtc::convertBGRAToYUV):

Source/WebCore:

Remove RealtimeOutgoingVideoSourceCocoa::convertToYUV, instead use libyuv based converter routine.
Covered by existing canvas -> peer connection tests.

  • platform/mediastream/mac/RealtimeOutgoingVideoSourceCocoa.cpp:
  • platform/mediastream/mac/RealtimeOutgoingVideoSourceCocoa.h:
  • platform/mediastream/mac/RealtimeOutgoingVideoSourceCocoa.mm:
1:18 AM Changeset in webkit [293884] by ysuzuki@apple.com
  • 2 edits in trunk/Source/WebCore

Make readArrayBufferViewImpl defensive
https://bugs.webkit.org/show_bug.cgi?id=240154
rdar://92113248

Reviewed by Mark Lam.

Check deserialized value's type before starting using it as JSArrayBuffer*.

  • Source/WebCore/bindings/js/SerializedScriptValue.cpp:

(WebCore::CloneDeserializer::readArrayBufferViewImpl):

Canonical link: https://commits.webkit.org/250342@main

12:43 AM Changeset in webkit [293883] by magomez@igalia.com
  • 3 edits in trunk/Source/WebCore

[Nicosia] Images in webkit.org/blog/ don't show up with threaded rendering
https://bugs.webkit.org/show_bug.cgi?id=238259

Reviewed by Carlos Garcia Campos.

Implement Nicosia::CairoOperationRecorder::drawFilteredImageBuffer(), which is required
to render images that have CSS filters applied.

  • platform/graphics/nicosia/cairo/NicosiaCairoOperationRecorder.cpp:

(Nicosia::CairoOperationRecorder::drawFilteredImageBuffer):

  • platform/graphics/nicosia/cairo/NicosiaCairoOperationRecorder.h:
12:26 AM Changeset in webkit [293882] by Fujii Hironori
  • 7 edits in trunk/Source/JavaScriptCore

builtins-generator-tests are failing after 250242@main
https://bugs.webkit.org/show_bug.cgi?id=239792
<rdar://problem/92532725>

Unreviewed resetting the results.
Tools/Scripts/run-builtins-generator-tests --reset-results

  • Scripts/tests/builtins/expected/WebCore-AnotherGuardedInternalBuiltin-Separate.js-result:
  • Scripts/tests/builtins/expected/WebCore-ArbitraryConditionalGuard-Separate.js-result:
  • Scripts/tests/builtins/expected/WebCore-GuardedBuiltin-Separate.js-result:
  • Scripts/tests/builtins/expected/WebCore-GuardedInternalBuiltin-Separate.js-result:
  • Scripts/tests/builtins/expected/WebCore-UnguardedBuiltin-Separate.js-result:
  • Scripts/tests/builtins/expected/WebCore-xmlCasingTest-Separate.js-result:

May 5, 2022:

9:44 PM Changeset in webkit [293881] by commit-queue@webkit.org
  • 62 edits in trunk/Source

[EME] CDM and friends should be using SharedBuffer rather FragmentedSharedBuffer
https://bugs.webkit.org/show_bug.cgi?id=240111
rdar://92780561

Patch by Jean-Yves Avenard <jean-yves.avenard@apple.com> on 2022-05-05
Reviewed by Eric Carlson.

Source/WebCore:

By using SharedBuffer directly we can simplify the code.
The change while extensive is rather trivial as the SharedBuffer used
were already contiguous.

No change in observable behaviour

  • Modules/encryptedmedia/CDM.cpp:

(WebCore::CDM::sanitizeInitData):
(WebCore::CDM::supportsInitData):
(WebCore::CDM::sanitizeResponse):

  • Modules/encryptedmedia/CDM.h:
  • Modules/encryptedmedia/CDMClient.h:
  • Modules/encryptedmedia/InitDataRegistry.cpp:

(WebCore::extractKeyIDsKeyids):
(WebCore::sanitizeKeyids):
(WebCore::InitDataRegistry::extractPsshBoxesFromCenc):
(WebCore::InitDataRegistry::extractKeyIDsCenc):
(WebCore::InitDataRegistry::sanitizeCenc):
(WebCore::sanitizeWebM):
(WebCore::extractKeyIDsWebM):
(WebCore::InitDataRegistry::sanitizeInitData):
(WebCore::InitDataRegistry::extractKeyIDs):

  • Modules/encryptedmedia/InitDataRegistry.h:
  • Modules/encryptedmedia/MediaKeySession.cpp:

(WebCore::MediaKeySession::generateRequest):
(WebCore::MediaKeySession::update):
(WebCore::MediaKeySession::remove):
(WebCore::MediaKeySession::enqueueMessage):
(WebCore::MediaKeySession::sendMessage):

  • Modules/encryptedmedia/MediaKeySession.h:
  • Modules/encryptedmedia/MediaKeyStatusMap.cpp:

(WebCore::keyIdsMatch):

  • Modules/encryptedmedia/MediaKeyStatusMap.h:
  • Modules/encryptedmedia/MediaKeys.cpp:

(WebCore::MediaKeys::unrequestedInitializationDataReceived):

  • Modules/encryptedmedia/MediaKeys.h:
  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::cdmClientUnrequestedInitializationDataReceived):

  • html/HTMLMediaElement.h:
  • platform/encryptedmedia/CDMInstance.h:
  • platform/encryptedmedia/CDMInstanceSession.h:
  • platform/encryptedmedia/CDMPrivate.h:
  • platform/encryptedmedia/CDMProxy.h:

(WebCore::KeyHandle::idAsSharedBuffer const):

  • platform/encryptedmedia/CDMUtilities.cpp:

(WebCore::CDMUtilities::parseJSONObject):

  • platform/encryptedmedia/CDMUtilities.h:
  • platform/encryptedmedia/clearkey/CDMClearKey.cpp:

(WebCore::extractKeyidsLocationFromCencInitData):
(WebCore::isCencInitData):
(WebCore::extractKeyidsFromCencInitData):
(WebCore::extractKeyIdFromWebMInitData):
(WebCore::CDMPrivateClearKey::supportsInitData const):
(WebCore::CDMPrivateClearKey::sanitizeResponse const):
(WebCore::CDMInstanceClearKey::setServerCertificate):
(WebCore::CDMInstanceSessionClearKey::requestLicense):
(WebCore::CDMInstanceSessionClearKey::updateLicense):
(WebCore::CDMInstanceSessionClearKey::removeSessionData):

  • platform/encryptedmedia/clearkey/CDMClearKey.h:
  • platform/graphics/avfoundation/CDMFairPlayStreaming.cpp:

(WebCore::extractSinfData):
(WebCore::extractSchemeAndKeyIdFromSinf):
(WebCore::CDMPrivateFairPlayStreaming::extractKeyIDsSinf):
(WebCore::CDMPrivateFairPlayStreaming::sanitizeSinf):
(WebCore::CDMPrivateFairPlayStreaming::sanitizeSkd):
(WebCore::CDMPrivateFairPlayStreaming::extractKeyIDsSkd):
(WebCore::CDMPrivateFairPlayStreaming::supportsInitData const):
(WebCore::CDMPrivateFairPlayStreaming::sanitizeResponse const):

  • platform/graphics/avfoundation/CDMFairPlayStreaming.h:
  • platform/graphics/avfoundation/objc/CDMInstanceFairPlayStreamingAVFObjC.h:
  • platform/graphics/avfoundation/objc/CDMInstanceFairPlayStreamingAVFObjC.mm:

(WebCore::initializationDataForRequest):
(WebCore::CDMInstanceFairPlayStreamingAVFObjC::takeUnexpectedKeyRequestForInitializationData):
(WebCore::parseJSONValue):
(WebCore::CDMInstanceFairPlayStreamingAVFObjC::setServerCertificate):
(WebCore::CDMInstanceFairPlayStreamingAVFObjC::sessionForKeyIDs const):
(WebCore::CDMInstanceSessionFairPlayStreamingAVFObjC::requestLicense):
(WebCore::isEqual):
(WebCore::CDMInstanceSessionFairPlayStreamingAVFObjC::updateLicense):
(WebCore::CDMInstanceSessionFairPlayStreamingAVFObjC::closeSession):
(WebCore::CDMInstanceSessionFairPlayStreamingAVFObjC::removeSessionData):
(WebCore::CDMInstanceSessionFairPlayStreamingAVFObjC::didProvideRequests):

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

(WebCore::MediaPlayerPrivateAVFoundationObjC::attemptToDecryptWithInstance):

  • platform/graphics/avfoundation/objc/MediaSampleAVFObjC.h:
  • platform/graphics/avfoundation/objc/SourceBufferPrivateAVFObjC.h:
  • platform/graphics/avfoundation/objc/SourceBufferPrivateAVFObjC.mm:

(WebCore::copyKeyIDs):

  • platform/graphics/gstreamer/eme/CDMThunder.cpp:

(WebCore::CDMPrivateThunder::supportsInitData const):
(WebCore::CDMPrivateThunder::sanitizeResponse const):
(WebCore::CDMInstanceThunder::setServerCertificate):
(WebCore::ParsedResponseMessage::ParsedResponseMessage):
(WebCore::ParsedResponseMessage::payload const):
(WebCore::ParsedResponseMessage::payload):
(WebCore::CDMInstanceSessionThunder::challengeGeneratedCallback):
(WebCore::CDMInstanceSessionThunder::errorCallback):
(WebCore::CDMInstanceSessionThunder::requestLicense):
(WebCore::CDMInstanceSessionThunder::updateLicense):
(WebCore::CDMInstanceSessionThunder::loadSession):
(WebCore::CDMInstanceSessionThunder::removeSessionData):

  • platform/graphics/gstreamer/eme/CDMThunder.h:
  • testing/MockCDMFactory.cpp:

(WebCore::MockCDMFactory::hasSessionWithID):
(WebCore::MockCDMFactory::removeSessionWithID):
(WebCore::MockCDMFactory::addKeysToSessionWithID):
(WebCore::MockCDMFactory::removeKeysFromSessionWithID):
(WebCore::MockCDMFactory::keysForSessionWithID const):
(WebCore::MockCDM::supportsInitData const):
(WebCore::MockCDM::sanitizeResponse const):
(WebCore::MockCDMInstance::setServerCertificate):
(WebCore::MockCDMInstanceSession::requestLicense):
(WebCore::MockCDMInstanceSession::updateLicense):
(WebCore::MockCDMInstanceSession::removeSessionData):

  • testing/MockCDMFactory.h:

Source/WebKit:

Remove the use of IPC::SharedBufferCopy objects to send data over IPC;
it adds no value here as SharedBuffer are fully serialisable.

  • GPUProcess/media/RemoteCDMInstanceProxy.cpp:

(WebKit::RemoteCDMInstanceProxy::unrequestedInitializationDataReceived):
(WebKit::RemoteCDMInstanceProxy::setServerCertificate):

  • GPUProcess/media/RemoteCDMInstanceProxy.h:
  • GPUProcess/media/RemoteCDMInstanceProxy.messages.in:
  • GPUProcess/media/RemoteCDMInstanceSessionProxy.cpp:

(WebKit::RemoteCDMInstanceSessionProxy::requestLicense):
(WebKit::RemoteCDMInstanceSessionProxy::updateLicense):
(WebKit::RemoteCDMInstanceSessionProxy::removeSessionData):
(WebKit::RemoteCDMInstanceSessionProxy::sendMessage):

  • GPUProcess/media/RemoteCDMInstanceSessionProxy.h:
  • GPUProcess/media/RemoteCDMInstanceSessionProxy.messages.in:
  • GPUProcess/media/RemoteCDMProxy.cpp:

(WebKit::RemoteCDMProxy::supportsInitData):
(WebKit::RemoteCDMProxy::sanitizeResponse):

  • GPUProcess/media/RemoteCDMProxy.h:
  • GPUProcess/media/RemoteLegacyCDMProxy.h:
  • GPUProcess/media/RemoteLegacyCDMSessionProxy.cpp:

(WebKit::convertToUint8Array):
(WebKit::convertToOptionalSharedBuffer):
(WebKit::RemoteLegacyCDMSessionProxy::generateKeyRequest):
(WebKit::RemoteLegacyCDMSessionProxy::update):

  • GPUProcess/media/RemoteLegacyCDMSessionProxy.h:
  • GPUProcess/media/RemoteLegacyCDMSessionProxy.messages.in:
  • Shared/WebCoreArgumentCoders.cpp:

(IPC::ArgumentCoder<WebCore::CDMInstanceSession::Message>::encode):
(IPC::ArgumentCoder<WebCore::CDMInstanceSession::Message>::decode):

  • WebProcess/GPU/media/RemoteCDM.cpp:

(WebKit::RemoteCDM::supportsInitData const):
(WebKit::RemoteCDM::sanitizeResponse const):

  • WebProcess/GPU/media/RemoteCDM.h:
  • WebProcess/GPU/media/RemoteCDMInstance.cpp:

(WebKit::RemoteCDMInstance::unrequestedInitializationDataReceived):
(WebKit::RemoteCDMInstance::setServerCertificate):

  • WebProcess/GPU/media/RemoteCDMInstance.h:
  • WebProcess/GPU/media/RemoteCDMInstance.messages.in:
  • WebProcess/GPU/media/RemoteCDMInstanceSession.cpp:

(WebKit::RemoteCDMInstanceSession::requestLicense):
(WebKit::RemoteCDMInstanceSession::updateLicense):
(WebKit::RemoteCDMInstanceSession::removeSessionData):
(WebKit::RemoteCDMInstanceSession::sendMessage):

  • WebProcess/GPU/media/RemoteCDMInstanceSession.h:
  • WebProcess/GPU/media/RemoteCDMInstanceSession.messages.in:
  • WebProcess/GPU/media/RemoteLegacyCDMSession.cpp:

(WebKit::convertToArrayBuffer):
(WebKit::convertToUint8Array):
(WebKit::convertToSharedBuffer):
(WebKit::RemoteLegacyCDMSession::generateKeyRequest):
(WebKit::RemoteLegacyCDMSession::update):
(WebKit::RemoteLegacyCDMSession::cachedKeyForKeyID const):
(WebKit::RemoteLegacyCDMSession::sendMessage):

  • WebProcess/GPU/media/RemoteLegacyCDMSession.h:
  • WebProcess/GPU/media/RemoteLegacyCDMSession.messages.in:

Canonical link: https://commits.webkit.org/250339@main

9:10 PM Changeset in webkit [293880] by Simon Fraser
  • 3 edits in trunk/Source/WebCore

Reduce the number of calls to canContainFixedPositionObjects()
https://bugs.webkit.org/show_bug.cgi?id=240150

Reviewed by Alan Bujtas.

canContainFixedPositionObjects() isn't super cheap, so only call it when we need to remove
the IsFixed bit.

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::mapLocalToContainer const):
(WebCore::RenderBox::mapAbsoluteToLocalPoint const):

  • rendering/svg/RenderSVGRoot.cpp:

(WebCore::RenderSVGRoot::mapLocalToContainer const):

7:34 PM Changeset in webkit [293879] by Chris Dumez
  • 28 edits in trunk/Source

Identifier::string() should return an AtomString
https://bugs.webkit.org/show_bug.cgi?id=240122

Reviewed by Yusuke Suzuki.

Identifier::string() should return an AtomString instead of a String, since
it holds an AtomString internally.

Also add some overloads to jsString() to resolve ambiguity for some callers.

  • Source/JavaScriptCore/API/JSContext.mm:

(-[JSContext dependencyIdentifiersForModuleJSScript:]):

  • Source/JavaScriptCore/bytecompiler/NodesCodegen.cpp:

(JSC::processClauseList):

  • Source/JavaScriptCore/dfg/DFGOperations.cpp:

(JSC::DFG::JSC_DEFINE_JIT_OPERATION):

  • Source/JavaScriptCore/inspector/JSInjectedScriptHost.cpp:

(Inspector::JSInjectedScriptHost::functionDetails):

  • Source/JavaScriptCore/jsc.cpp:

(JSC_DEFINE_HOST_FUNCTION):

  • Source/JavaScriptCore/runtime/Error.cpp:

(JSC::addErrorInfo):

  • Source/JavaScriptCore/runtime/ErrorInstance.cpp:

(JSC::ErrorInstance::finishCreation):

  • Source/JavaScriptCore/runtime/ErrorPrototype.cpp:

(JSC::JSC_DEFINE_HOST_FUNCTION):

  • Source/JavaScriptCore/runtime/FunctionExecutable.cpp:

(JSC::FunctionExecutable::toStringSlow):

  • Source/JavaScriptCore/runtime/Identifier.h:

(JSC::Identifier::string const):
(JSC::Identifier::atomString const): Deleted.

  • Source/JavaScriptCore/runtime/IdentifierInlines.h:

(JSC::identifierToJSValue):
(JSC::identifierToSafePublicJSValue):

  • Source/JavaScriptCore/runtime/IntlDateTimeFormat.cpp:

(JSC::IntlDateTimeFormat::format const):
(JSC::IntlDateTimeFormat::formatRange):

  • Source/JavaScriptCore/runtime/IntlDisplayNames.cpp:

(JSC::IntlDisplayNames::of const):

  • Source/JavaScriptCore/runtime/IntlListFormat.cpp:

(JSC::IntlListFormat::format const):

  • Source/JavaScriptCore/runtime/IntlRelativeTimeFormat.cpp:

(JSC::IntlRelativeTimeFormat::format const):

  • Source/JavaScriptCore/runtime/JSFunction.cpp:

(JSC::JSFunction::reifyName):

  • Source/JavaScriptCore/runtime/JSModuleLoader.cpp:

(JSC::JSModuleLoader::requestImportModule):

  • Source/JavaScriptCore/runtime/JSString.h:

(JSC::jsString):

  • Source/JavaScriptCore/runtime/JSStringInlines.h:

(JSC::repeatCharacter):

  • Source/JavaScriptCore/runtime/StringPrototype.cpp:

(JSC::jsSpliceSubstrings):
(JSC::jsSpliceSubstringsWithSeparators):
(JSC::toLocaleCase):
(JSC::normalize):

  • Source/JavaScriptCore/runtime/SymbolPrototype.cpp:

(JSC::JSC_DEFINE_CUSTOM_GETTER):

  • Source/WTF/wtf/PrintStream.cpp:

(WTF::printInternal):

  • Source/WTF/wtf/PrintStream.h:
  • Source/WebCore/animation/KeyframeEffect.cpp:

(WebCore::processKeyframeLikeObject):

  • Source/WebCore/inspector/WebInjectedScriptHost.cpp:

(WebCore::WebInjectedScriptHost::getInternalProperties):

Canonical link: https://commits.webkit.org/250337@main

7:23 PM Changeset in webkit [293878] by jer.noble@apple.com
  • 5 edits
    2 adds in trunk

Removing x-webkit-wirelessvideoplaybackdisabled attribute from video does not enable AirPlay.
https://bugs.webkit.org/show_bug.cgi?id=239031

Reviewed by Jer Noble.

  • Source/WebKit/GPUProcess/media/RemoteMediaPlayerProxy.cpp:

(WebKit::RemoteMediaPlayerProxy::setWirelessVideoPlaybackDisabled):

  • Source/WebKit/WebProcess/GPU/media/MediaPlayerPrivateRemote.cpp:

(WebKit::MediaPlayerPrivateRemote::setWirelessVideoPlaybackDisabled):
Test: media/airplay-wirelessvideoplaybackdisabled.html

  • Source/WebCore/html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::attributeChanged):

  • Source/WebCore/html/HTMLMediaElement.h:
  • Source/WebCore/html/HTMLVideoElement.cpp:

(WebCore::HTMLVideoElement::parseAttribute):

  • LayoutTests/media/airplay-wirelessvideoplaybackdisabled-expected.txt: Added.
  • LayoutTests/media/airplay-wirelessvideoplaybackdisabled.html: Added.

Canonical link: https://commits.webkit.org/250336@main

6:28 PM Changeset in webkit [293877] by eric.carlson@apple.com
  • 2 edits in trunk/Source/WebCore/platform/graphics/avfoundation/objc

[Cocoa] QueuedVideoOutput constructor missing nil check
https://bugs.webkit.org/show_bug.cgi?id=240149

Reviewed by Eric Carlson.

Covered by existing tests.

  • Source/WebCore/platform/graphics/avfoundation/objc/QueuedVideoOutput.h:
  • Source/WebCore/platform/graphics/avfoundation/objc/QueuedVideoOutput.mm:

(WebCore::QueuedVideoOutput::create):
(WebCore::QueuedVideoOutput::QueuedVideoOutput):
(WebCore::QueuedVideoOutput::valid):

Canonical link: https://commits.webkit.org/250335@main

5:59 PM Changeset in webkit [293876] by Karl Rackler
  • 2 edits in trunk/LayoutTests

[ iOS ] fast/images/exif-orientation-background-image-repeat.html is a consistent image failure
https://bugs.webkit.org/show_bug.cgi?id=240148

Unreviewed test gardening.

Canonical link: https://commits.webkit.org/250334@main

5:49 PM Changeset in webkit [293875] by Jonathan Bedard
  • 7 edits in trunk/Tools

[git-webkit] Automatically file bug
https://bugs.webkit.org/show_bug.cgi?id=240139
<rdar://problem/92818331>

Reviewed by Dewei Zhu.

  • Tools/Scripts/libraries/webkitscmpy/setup.py: Bump version.
  • Tools/Scripts/libraries/webkitscmpy/webkitscmpy/init.py: Ditto.
  • Tools/Scripts/libraries/webkitscmpy/webkitscmpy/program/branch.py:

(Branch.main): If a project has defined bug trackers and the user provides a bug title instead of a bug
url or branch name (determined by spaces in the provided string), walk the user through bug creation.

  • Tools/Scripts/libraries/webkitscmpy/webkitscmpy/test/branch_unittest.py:

(TestBranch): Add mock issues, rebase prompts.
(TestBranch.test_create_bug): Added.

  • Tools/Scripts/libraries/webkitscmpy/webkitscmpy/test/pull_request_unittest.py:
  • Tools/Scripts/libraries/webkitscmpy/webkitscmpy/test/revert_unittest.py:

(TestRevert.test_github): Add mock issues, rebase prompts.
(TestRevert.test_modified): Ditto.

Canonical link: https://commits.webkit.org/250333@main

5:24 PM Changeset in webkit [293874] by Alan Coon
  • 1 copy in tags/WebKit-7613.3.1

Tag WebKit-7613.3.1.

5:12 PM Changeset in webkit [293873] by pvollan@apple.com
  • 2 edits in trunk

Add GitHub user name
https://bugs.webkit.org/show_bug.cgi?id=240144

Unreviewed, add github user name to contributors.json.

  • metadata/contributors.json:
5:03 PM Changeset in webkit [293872] by Alan Coon
  • 1 copy in tags/WebKit-7614.1.11.7

Tag WebKit-7614.1.11.7.

5:01 PM Changeset in webkit [293871] by Alan Coon
  • 9 edits in branches/safari-614.1.11-branch/Source

Versioning.

WebKit-7614.1.11.7

4:50 PM Changeset in webkit [293870] by Nikos Mouchtaris
  • 3 edits in trunk/Source/WebCore

Add check if referenced element is SVG for clip-path
https://bugs.webkit.org/show_bug.cgi?id=240096

Reviewed by Simon Fraser.

Add check when creating new ReferencePathOperation that it is referencing
an svg element.

  • style/StyleBuilderConverter.h:

(WebCore::Style::BuilderConverter::convertPathOperation):

4:43 PM Changeset in webkit [293869] by keith_miller@apple.com
  • 2 edits in trunk/JSTests

Rebaseline icu tests to public sdk's icu
https://bugs.webkit.org/show_bug.cgi?id=240142

Reviewed by Yusuke Suzuki.

  • test262/expectations.yaml:
4:28 PM Changeset in webkit [293868] by Alan Coon
  • 9 edits in branches/safari-613-branch/Source

Versioning.

WebKit-7613.3.1

4:15 PM Changeset in webkit [293867] by Cameron McCormack
  • 5 edits in trunk/Source/WebCore

Avoid using WebCore::Colors created with out-of-line components in DrawGlyphsRecorder
https://bugs.webkit.org/show_bug.cgi?id=235604
rdar://88345680

Reviewed by Simon Fraser.

Source/WebCore:

In DrawGlyphsRecorder::recordDrawGlyphs we interrogate the CGContext's
state to find out the current fill and stroke color. This needed to
record color font glyphs correctly. When we're not using a color font,
the CGColors for the fill and stroke will be the same as we set them
in prepareInternalContext.

Those CGColors we create in prepareInternalContext from the
WebCore::Colors (created by calling cachedCGColor) don't record the
exact form the original colors had. CGColors only expose float
components, so Color::createAndPreserveColorSpace conservatively
creates a Color with out-of-line components to store those float
values, even if the original Color the CGColor was created from had
uint8_t components.

Color::operator== treats colors of different forms as being different,
which means we generate additional, unnecessary SetState display list
items changing say the fill color from Color(255, 255, 255) (inline) to
Color(1.0, 1.0, 1.0) (out of line).

We can avoid this by checking whether the context state's fill and
stroke CGColors are the same object as the ones we set in
prepareInternalContext, and if so, set the current fill/stroke brush
based on m_originalState.fillBrush/strokeBrush. In the common case of
a non-color glyph, this brush change will be detected as the same as
the current value, and the SetState item generation will be avoided.

This improves the MotionMark design subtest by 2.4%, and the overall
score by 0.7%, when GPUP DOM rendering is enabled.

  • platform/graphics/DrawGlyphsRecorder.h:
  • platform/graphics/coretext/DrawGlyphsRecorderCoreText.cpp:

(WebCore::DrawGlyphsRecorder::recordInitialColors):
(WebCore::DrawGlyphsRecorder::prepareInternalContext):
(WebCore::DrawGlyphsRecorder::updateFillColor):
(WebCore::DrawGlyphsRecorder::updateStrokeColor):

Source/WebCore/PAL:

  • pal/spi/cg/CoreGraphicsSPI.h: Add CGContextGetStrokeColorAsColor.
4:11 PM Changeset in webkit [293866] by Alan Coon
  • 18 edits in branches/safari-613-branch

Revert r293850. rdar://problem/88904160

This reverts a patch application at r293850.

4:11 PM Changeset in webkit [293865] by Alan Coon
  • 5 edits in branches/safari-613-branch/Source/WebKit

Revert r293851. rdar://problem/88904160

This reverts a patch application at r293851.

4:10 PM Changeset in webkit [293864] by Alan Coon
  • 4 edits in branches/safari-613-branch/Source/WebKit

Revert r293852. rdar://problem/88904160

This reverts a patch application at r293852.

3:42 PM Changeset in webkit [293863] by Brent Fulgham
  • 8 edits in trunk/Source

Remove abandoned WKPreference for AlwaysUseAcceleratedOverflowScroll
https://bugs.webkit.org/show_bug.cgi?id=240130

Reviewed by Simon Fraser.

The preference 'AlwaysUseAcceleratedOverflowScroll' and related code are no longer used. We should remove this dead code.

Source/WebCore:

  • rendering/RenderLayerScrollableArea.cpp:

(WebCore::RenderLayerScrollableArea::canUseCompositedScrolling const):

Source/WebKitLegacy/mac:

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

(-[WebPreferences _setAlwaysUseAcceleratedOverflowScroll:]): Deleted.
(-[WebPreferences _alwaysUseAcceleratedOverflowScroll]): Deleted.

  • WebView/WebPreferencesPrivate.h:

Source/WTF:

  • Scripts/Preferences/WebPreferences.yaml:
3:27 PM Changeset in webkit [293862] by Megan Gardner
  • 2 edits in trunk

Add github info for myself to contributors.json
https://bugs.webkit.org/show_bug.cgi?id=240140

Unreviewed metadata addition.

  • metadata/contributors.json:
3:14 PM Changeset in webkit [293861] by achristensen@apple.com
  • 3 edits in trunk/Source/WebKit

Unreviewed, reverting r293697.

Mergred to branch, not needed on trunk

Reverted changeset:

"Revert all use of
_setPrivacyProxyFailClosedForUnreachableNonMainHosts"
https://bugs.webkit.org/show_bug.cgi?id=239977
https://commits.webkit.org/r293697

2:40 PM Changeset in webkit [293860] by Manuel Rego Casasnovas
  • 19 edits in trunk

ARIA reflection for Element attributes
https://bugs.webkit.org/show_bug.cgi?id=239852

Reviewed by Chris Dumez.

LayoutTests/imported/w3c:

Small fix on the test and update test results with the new PASS.
Add new test case to cover elements moved to a different document.

  • web-platform-tests/dom/nodes/aria-element-reflection.tentative-expected.txt:
  • web-platform-tests/dom/nodes/aria-element-reflection.tentative.html:

Source/WebCore:

Implement ARIA reflection for attributes that refer to a single Element:
aria-activedescendant & aria-errormessage.
For the properties names this patch uses "Element" suffix:
ariaActiveDescendantElement & ariaErrorMessageElement;
this matches Chromium implementation and AOM explainer, but not AOM spec:
https://github.com/w3c/aria/issues/1732

This implementation is done following the proposed spec text defined at:
https://github.com/whatwg/html/pull/3917

  • accessibility/AriaAttributes.idl: Add ariaActiveDescendantElement &

ariaErrorMessageElement.

  • bindings/scripts/CodeGenerator.pm:

(GetterExpression): Add function for Element attributes.
(SetterExpression): Ditto.

  • bindings/scripts/test/JS/JSTestObj.cpp: Add tests for element

attribute reflection.
(WebCore::JSTestObjDOMConstructor::construct):
(WebCore::jsTestObj_reflectedElementAttrGetter):
(WebCore::JSC_DEFINE_CUSTOM_GETTER):
(WebCore::setJSTestObj_reflectedElementAttrSetter):
(WebCore::JSC_DEFINE_CUSTOM_SETTER):

  • bindings/scripts/test/TestObj.idl: Add reflectedElementAttr

attribute.

  • dom/Element.cpp:

(WebCore::isElementReflectionAttribute): Utility function to identify
Element reflection attributes.
(WebCore::Element::attributeChanged): If an element reflection
attribute has changed we need to synchronize the map entry by removing
it as it'll be properly updated in setElementAttribute() when needed.
(WebCore::Element::explicitlySetAttrElementsMap): Kind of getter
for the ExplicitlySetAttrElementsMap but that creates the map if it
doesn't exist yet.
(WebCore::Element::explicitlySetAttrElementsMapIfExists const):
Getter for the map.
(WebCore::Element::getElementAttribute const): Implement getter for
element attribute.
(WebCore::Element::setElementAttribute): Implement setter for
element attribute.

  • dom/Element.h: Add new method headers and defines the

ExplicitlySetAttrElementsMap, which so far only stores one Element in
the Vector, but uses a Vector in preparation for supporting
FrozenArray<Element> reflection in the future.

  • dom/ElementRareData.cpp: Update size of SameSizeAsElementRareData to

include the new pointer.

  • dom/ElementRareData.h: Add m_explicitlySetAttrElementsMap attribute.

(WebCore::ElementRareData::explicitlySetAttrElementsMap): Getter.
(WebCore::ElementRareData::useTypes const):Include
ExplicitlySetAttrElementsMap.

  • dom/Node.cpp:

(WebCore::stringForRareDataUseType): Add ExplicitlySetAttrElementsMap.

  • dom/NodeRareData.h: Ditto.

Source/WTF:

Add new runtime flag AriaReflectionForElementReferencesEnabled.

  • Scripts/Preferences/WebPreferencesExperimental.yaml:

LayoutTests:

Update test to include the new reflected attributes.

  • accessibility/ARIA-reflection-expected.txt:
  • accessibility/ARIA-reflection.html:
2:10 PM Changeset in webkit [293859] by Karl Rackler
  • 2 edits in trunk/LayoutTests

[ iOS ][ Monterey Release wk2 ] imported/w3c/web-platform-tests/content-security-policy/inheritance/blob-url-in-main-window-self-navigate-inherits.sub.html is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=239568

Unreviewed test gardening.

Canonical link: https://commits.webkit.org/250324@main

12:44 PM Changeset in webkit [293858] by ysuzuki@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Unreviewed, partial revert of r293813 because of proposal's issue.
https://bugs.webkit.org/show_bug.cgi?id=240102

  • runtime/IntlNumberFormat.cpp:

(JSC::IntlNumberFormat::initializeNumberFormat):

11:58 AM Changeset in webkit [293857] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Remove unnecessary calls to CachedResource::updateBuffer and CachedResource::updateData
https://bugs.webkit.org/show_bug.cgi?id=240126

Patch by Alex Christensen <achristensen@webkit.org> on 2022-05-05
Reviewed by Chris Dumez.

It adds a function call that does nothing but a redundant assert.

  • loader/cache/CachedImage.cpp:

(WebCore::CachedImage::updateBufferInternal):
(WebCore::CachedImage::updateBuffer):
(WebCore::CachedImage::updateData):

11:54 AM Changeset in webkit [293856] by Alan Coon
  • 1 copy in tags/WebKit-7613.2.7.1.9

Tag WebKit-7613.2.7.1.9.

11:50 AM Changeset in webkit [293855] by Alan Coon
  • 9 edits in branches/safari-613.2.7.1-branch/Source

Versioning.

WebKit-7613.2.7.1.9

11:42 AM Changeset in webkit [293854] by Robert Jenner
  • 1 edit
    5 deletes in trunk/LayoutTests

Remove and cleanup platform expectations folders
https://bugs.webkit.org/show_bug.cgi?id=240131

Unreviewed test gardening.

  • platform/mac-catalina-wk1/TestExpectations: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/anchor-element-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/attrib-string-colors-with-color-filter-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/attribute-string-for-copy-with-color-filter-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-1-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-2-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-3-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-4-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-5-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-with-style-1-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-with-style-2-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/attributed-string-for-typing-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/attributed-string-for-typing-with-color-filter-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/basic-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/comment-cdata-section-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/font-size-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/font-style-variant-effect-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/font-weight-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/letter-spacing-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/text-decorations-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/mac/attributed-string/vertical-align-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/selection/select-across-readonly-input-4-expected.txt: Removed.
  • platform/mac-catalina-wk1/editing/selection/select-across-readonly-input-5-expected.txt: Removed.
  • platform/mac-catalina-wk1/http/tests/cookies/js-get-and-set-http-only-cookie-expected.txt: Removed.
  • platform/mac-catalina-wk1/imported/w3c/web-platform-tests/mathml/presentation-markup/operators/mo-stretch-properties-dynamic-001-expected.txt: Removed.
  • platform/mac-catalina-wk2/fast/events/contextmenu-lookup-action-for-image-expected.txt: Removed.
  • platform/mac-catalina-wk2/http/tests/websocket/tests/hybi/send-object-tostring-check-expected.txt: Removed.
  • platform/mac-catalina-wk2/http/tests/workers/service/serviceworker-websocket.https-expected.txt: Removed.
  • platform/mac-catalina-wk2/imported/w3c/web-platform-tests/media-source/mediasource-invalid-codec-expected.txt: Removed.
  • platform/mac-catalina-wk2/imported/w3c/web-platform-tests/service-workers/service-worker/websocket-in-service-worker.https-expected.txt: Removed.
  • platform/mac-catalina-wk2/imported/w3c/web-platform-tests/service-workers/service-worker/websocket.https-expected.txt: Removed.
  • platform/mac-catalina-wk2/imported/w3c/web-platform-tests/websockets/basic-auth.any-expected.txt: Removed.
  • platform/mac-catalina-wk2/imported/w3c/web-platform-tests/websockets/basic-auth.any.worker-expected.txt: Removed.
  • platform/mac-catalina-wk2/imported/w3c/web-platform-tests/websockets/remove-own-iframe-during-onerror.window-expected.txt: Removed.
  • platform/mac-catalina/TestExpectations: Removed.
  • platform/mac-catalina/editing/input/reveal-caret-of-multiline-input-expected.txt: Removed.
  • platform/mac-catalina/editing/pasteboard/pasting-tabs-expected.txt: Removed.
  • platform/mac-catalina/editing/selection/3690703-2-expected.txt: Removed.
  • platform/mac-catalina/editing/selection/3690703-expected.txt: Removed.
  • platform/mac-catalina/editing/selection/3690719-expected.txt: Removed.
  • platform/mac-catalina/editing/selection/select-from-textfield-outwards-expected.txt: Removed.
  • platform/mac-catalina/editing/selection/vertical-rl-rtl-extend-line-backward-br-expected.txt: Removed.
  • platform/mac-catalina/editing/selection/vertical-rl-rtl-extend-line-backward-br-mixed-expected.txt: Removed.
  • platform/mac-catalina/editing/selection/vertical-rl-rtl-extend-line-backward-p-expected.txt: Removed.
  • platform/mac-catalina/editing/selection/vertical-rl-rtl-extend-line-backward-p-mixed-expected.txt: Removed.
  • platform/mac-catalina/editing/selection/vertical-rl-rtl-extend-line-forward-br-expected.txt: Removed.
  • platform/mac-catalina/editing/selection/vertical-rl-rtl-extend-line-forward-br-mixed-expected.txt: Removed.
  • platform/mac-catalina/editing/selection/vertical-rl-rtl-extend-line-forward-p-expected.txt: Removed.
  • platform/mac-catalina/editing/selection/vertical-rl-rtl-extend-line-forward-p-mixed-expected.txt: Removed.
  • platform/mac-catalina/fast/block/basic/001-expected.txt: Removed.
  • platform/mac-catalina/fast/block/float/float-avoidance-expected.txt: Removed.
  • platform/mac-catalina/fast/css/apple-system-control-colors-expected.txt: Removed.
  • platform/mac-catalina/fast/css/continuationCrash-expected.txt: Removed.
  • platform/mac-catalina/fast/css/rtl-ordering-expected.txt: Removed.
  • platform/mac-catalina/fast/css/text-overflow-input-expected.txt: Removed.
  • platform/mac-catalina/fast/dom/HTMLInputElement/input-slider-update-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/auto-fill-button/hide-auto-fill-strong-password-viewable-treatment-when-form-is-reset-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/basic-inputs-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/basic-textareas-quirks-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/box-shadow-override-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/button-positioned-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/button-sizes-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/date/date-input-rendering-basic-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/form-element-geometry-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/hidpi-textfield-background-bleeding-expected.html: Removed.
  • platform/mac-catalina/fast/forms/input-appearance-height-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/input-appearance-preventDefault-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/input-appearance-spinbutton-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/input-appearance-spinbutton-up-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/input-button-sizes-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/input-disabled-color-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/input-placeholder-visibility-1-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/input-placeholder-visibility-3-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/input-readonly-dimmed-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/input-table-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/input-text-word-wrap-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/input-value-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/listbox-bidi-align-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/listbox-width-change-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/number/number-appearance-spinbutton-disabled-readonly-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/option-text-clip-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/plaintext-mode-2-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/range/input-appearance-range-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/range/slider-padding-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/range/slider-thumb-shared-style-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/range/thumbslider-no-parent-slider-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/search-rtl-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/search/search-size-with-decorations-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/search/search-zoom-computed-style-height-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/select-change-listbox-to-popup-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/select-change-popup-to-listbox-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/select-selected-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/select-visual-hebrew-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/select/optgroup-rendering-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/textAreaLineHeight-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/textarea-placeholder-visibility-1-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/textarea-placeholder-visibility-2-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/textfield-outline-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/time/time-input-rendering-basic-expected.txt: Removed.
  • platform/mac-catalina/fast/forms/visual-hebrew-text-field-expected.txt: Removed.
  • platform/mac-catalina/fast/parser/document-write-option-expected.txt: Removed.
  • platform/mac-catalina/fast/parser/entity-comment-in-textarea-expected.txt: Removed.
  • platform/mac-catalina/fast/parser/open-comment-in-textarea-expected.txt: Removed.
  • platform/mac-catalina/fast/repaint/block-inputrange-repaint-expected.txt: Removed.
  • platform/mac-catalina/fast/repaint/slider-thumb-drag-release-expected.txt: Removed.
  • platform/mac-catalina/fast/sandbox/mac/sandbox-mach-lookup-expected.txt: Removed.
  • platform/mac-catalina/fast/text/backslash-to-yen-sign-euc-expected.txt: Removed.
  • platform/mac-catalina/fast/text/basic/014-expected.txt: Removed.
  • platform/mac-catalina/fast/text/capitalize-boundaries-expected.txt: Removed.
  • platform/mac-catalina/fast/text/drawBidiText-expected.txt: Removed.
  • platform/mac-catalina/fast/text/hyphenate-avoid-orphaned-word-expected.txt: Removed.
  • platform/mac-catalina/fast/text/hyphenate-character-expected.txt: Removed.
  • platform/mac-catalina/fast/text/hyphenate-first-word-expected.txt: Removed.
  • platform/mac-catalina/fast/text/hyphenate-limit-before-after-expected.txt: Removed.
  • platform/mac-catalina/fast/text/hyphenate-limit-lines-expected.txt: Removed.
  • platform/mac-catalina/fast/text/hyphenate-locale-expected.txt: Removed.
  • platform/mac-catalina/fast/text/hyphens-expected.txt: Removed.
  • platform/mac-catalina/fast/text/indic-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/bidi-fallback-font-weight-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/bidi-linebreak-001-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/bidi-linebreak-002-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/bidi-linebreak-003-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/danda-space-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/pop-up-button-text-alignment-and-direction-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/system-language/navigator-language/navigator-language-en-US-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/system-language/navigator-language/navigator-language-en-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/system-language/navigator-language/navigator-language-es-ES-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/system-language/navigator-language/navigator-language-es-MX-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/system-language/navigator-language/navigator-language-es-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/system-language/navigator-language/navigator-language-fr-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/system-language/navigator-language/navigator-language-hi-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/system-language/navigator-language/navigator-language-pt-BR-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/system-language/navigator-language/navigator-language-ru-expected.txt: Removed.
  • platform/mac-catalina/fast/text/international/system-language/system-font-punctuation-expected.txt: Removed.
  • platform/mac-catalina/fast/text/justify-ideograph-leading-expansion-expected.txt: Removed.
  • platform/mac-catalina/fast/text/midword-break-after-breakable-char-expected.txt: Removed.
  • platform/mac-catalina/fast/text/vertical-rl-rtl-linebreak-expected.txt: Removed.
  • platform/mac-catalina/fast/text/vertical-rl-rtl-linebreak-mixed-expected.txt: Removed.
  • platform/mac-catalina/http/tests/navigation/javascriptlink-frames-expected.txt: Removed.
  • platform/mac-catalina/http/tests/xmlhttprequest/methods-async-expected.txt: Removed.
  • platform/mac-catalina/http/tests/xmlhttprequest/methods-expected.txt: Removed.
  • platform/mac-catalina/http/tests/xmlhttprequest/workers/methods-async-expected.txt: Removed.
  • platform/mac-catalina/http/tests/xmlhttprequest/workers/methods-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/fetch/api/redirect/redirect-method.any-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/fetch/api/redirect/redirect-method.any.worker-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/fetch/redirect-navigate/preserve-fragment-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/html/semantics/embedded-content/media-elements/mime-types/canPlayType-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/html/semantics/embedded-content/the-video-element/resize-during-playback-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/mathml/presentation-markup/operators/mo-minsize-maxsize-001-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/mathml/presentation-markup/operators/mo-stretch-properties-dynamic-001-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/mathml/presentation-markup/operators/operator-dictionary-combining-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/mathml/presentation-markup/operators/operator-dictionary-stretchy-001-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/mathml/presentation-markup/operators/operator-dictionary-stretchy-002-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/mathml/presentation-markup/operators/operator-dictionary-symmetric-001-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/mathml/presentation-markup/operators/operator-dictionary-symmetric-005-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/mathml/presentation-markup/operators/operator-dictionary-symmetric-006-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/mathml/relations/css-styling/ignored-properties-001-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/mathml/relations/css-styling/padding-border-margin/border-002-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/mathml/relations/css-styling/padding-border-margin/padding-002-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/mathml/relations/html5-tree/dynamic-childlist-001-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/media-source/mediasource-addsourcebuffer-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/media-source/mediasource-invalid-codec-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/websockets/interfaces/WebSocket/bufferedAmount/bufferedAmount-getting-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/xhr/send-entity-body-empty-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/xhr/send-entity-body-get-head-async-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/xhr/send-entity-body-get-head-expected.txt: Removed.
  • platform/mac-catalina/imported/w3c/web-platform-tests/xhr/send-entity-body-none-expected.txt: Removed.
  • platform/mac-catalina/inspector/css/get-system-fonts-expected.txt: Removed.
  • platform/mac-catalina/media/media-can-play-webm-expected.txt: Removed.
  • platform/mac-catalina/platform/mac/fast/loader/file-url-mimetypes-3-expected.txt: Removed.
  • platform/mac-catalina/platform/mac/fast/text/bidi-fallback-font-weight-expected.txt: Removed.
  • platform/mac-catalina/platform/mac/fast/text/international/Geeza-Pro-vertical-metrics-adjustment-expected.txt: Removed.
  • platform/mac-catalina/platform/mac/fast/text/international/bidi-fallback-font-weight-expected.txt: Removed.
  • platform/mac-catalina/svg/W3C-I18N/tspan-direction-rtl-expected.txt: Removed.
  • platform/mac-catalina/svg/custom/svg-fonts-without-missing-glyph-expected.txt: Removed.
  • platform/mac-catalina/svg/text/bidi-tspans-expected.txt: Removed.
  • platform/mac-catalina/tables/mozilla/bugs/bug18359-expected.txt: Removed.
  • platform/mac-catalina/tables/mozilla/bugs/bug2479-3-expected.txt: Removed.
  • platform/mac-catalina/tables/mozilla/bugs/bug26178-expected.txt: Removed.
  • platform/mac-catalina/tables/mozilla/bugs/bug30692-expected.txt: Removed.
  • platform/mac-catalina/tables/mozilla/bugs/bug33855-expected.txt: Removed.
  • platform/mac-catalina/tables/mozilla/bugs/bug60749-expected.txt: Removed.
  • platform/mac-catalina/tables/mozilla/bugs/bug7342-expected.txt: Removed.
  • platform/mac-catalina/tables/mozilla/other/wa_table_thtd_rowspan-expected.txt: Removed.
  • platform/mac-catalina/tables/mozilla/other/wa_table_tr_align-expected.txt: Removed.
  • platform/mac-catalina/transforms/2d/zoom-menulist-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/anchor-element-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/attrib-string-colors-with-color-filter-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/attribute-string-for-copy-with-color-filter-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-1-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-2-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-3-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-4-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-5-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-with-style-1-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/attributed-string-across-shadow-boundaries-with-style-2-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/attributed-string-for-typing-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/attributed-string-for-typing-with-color-filter-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/basic-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/comment-cdata-section-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/font-size-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/font-style-variant-effect-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/font-weight-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/letter-spacing-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/text-decorations-expected.txt: Removed.
  • platform/mac-mojave-wk1/editing/mac/attributed-string/vertical-align-expected.txt: Removed.
  • platform/mac-mojave-wk1/imported/w3c/web-platform-tests/fetch/api/redirect/redirect-method.any-expected.txt: Removed.
  • platform/mac-mojave-wk1/imported/w3c/web-platform-tests/fetch/api/redirect/redirect-method.any.worker-expected.txt: Removed.
  • platform/mac-mojave/css1/basic/inheritance-expected.txt: Removed.
  • platform/mac-mojave/css2.1/t0602-c13-inh-underlin-00-e-expected.txt: Removed.
  • platform/mac-mojave/css2.1/t0805-c5522-brdr-02-e-expected.txt: Removed.
  • platform/mac-mojave/css2.1/t1202-counter-09-b-expected.txt: Removed.
  • platform/mac-mojave/css2.1/t1202-counters-09-b-expected.txt: Removed.
  • platform/mac-mojave/editing/input/reveal-caret-of-multiline-input-expected.txt: Removed.
  • platform/mac-mojave/editing/mac/selection/context-menu-select-editability-expected.txt: Removed.
  • platform/mac-mojave/editing/selection/3690703-2-expected.txt: Removed.
  • platform/mac-mojave/editing/selection/3690703-expected.txt: Removed.
  • platform/mac-mojave/editing/selection/3690719-expected.txt: Removed.
  • platform/mac-mojave/fast/block/basic/001-expected.txt: Removed.
  • platform/mac-mojave/fast/block/float/float-avoidance-expected.txt: Removed.
  • platform/mac-mojave/fast/css-generated-content/014-expected.png: Removed.
  • platform/mac-mojave/fast/css-generated-content/014-expected.txt: Removed.
  • platform/mac-mojave/fast/css/apple-system-control-colors-expected.txt: Removed.
  • platform/mac-mojave/fast/css/continuationCrash-expected.txt: Removed.
  • platform/mac-mojave/fast/css/css3-nth-child-expected.txt: Removed.
  • platform/mac-mojave/fast/css/line-height-font-order-expected.png: Removed.
  • platform/mac-mojave/fast/css/line-height-font-order-expected.txt: Removed.
  • platform/mac-mojave/fast/css/rtl-ordering-expected.txt: Removed.
  • platform/mac-mojave/fast/css/text-overflow-input-expected.txt: Removed.
  • platform/mac-mojave/fast/dom/34176-expected.txt: Removed.
  • platform/mac-mojave/fast/dom/clone-node-dynamic-style-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/basic-inputs-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/button-positioned-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/button-sizes-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/date/date-input-rendering-basic-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/input-appearance-spinbutton-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/input-button-sizes-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/input-disabled-color-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/input-readonly-dimmed-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/input-text-word-wrap-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/listbox-bidi-align-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/listbox-width-change-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/option-text-clip-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/plaintext-mode-2-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/search-rtl-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/search/search-size-with-decorations-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/select-change-listbox-to-popup-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/select-change-popup-to-listbox-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/select-selected-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/select/optgroup-rendering-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/textfield-outline-expected.txt: Removed.
  • platform/mac-mojave/fast/forms/time/time-input-rendering-basic-expected.txt: Removed.
  • platform/mac-mojave/fast/images/image-controls-basic-expected.png: Removed.
  • platform/mac-mojave/fast/indic-expected.txt: Removed.
  • platform/mac-mojave/fast/invalid/003-expected.txt: Removed.
  • platform/mac-mojave/fast/invalid/004-expected.txt: Removed.
  • platform/mac-mojave/fast/invalid/nestedh3s-expected.txt: Removed.
  • platform/mac-mojave/fast/overflow/007-expected.png: Removed.
  • platform/mac-mojave/fast/overflow/007-expected.txt: Removed.
  • platform/mac-mojave/fast/parser/document-write-option-expected.txt: Removed.
  • platform/mac-mojave/fast/sandbox/mac/sandbox-mach-lookup-expected.txt: Removed.
  • platform/mac-mojave/fast/selectors/018-expected.txt: Removed.
  • platform/mac-mojave/fast/table/frame-and-rules-expected.txt: Removed.
  • platform/mac-mojave/fast/text/atsui-multiple-renderers-expected.txt: Removed.
  • platform/mac-mojave/fast/text/bidi-embedding-pop-and-push-same-expected.txt: Removed.
  • platform/mac-mojave/fast/text/font-weights-expected.txt: Removed.
  • platform/mac-mojave/fast/text/font-weights-zh-expected.txt: Removed.
  • platform/mac-mojave/fast/text/hyphenate-avoid-orphaned-word-expected.txt: Removed.
  • platform/mac-mojave/fast/text/hyphenate-character-expected.png: Removed.
  • platform/mac-mojave/fast/text/hyphenate-character-expected.txt: Removed.
  • platform/mac-mojave/fast/text/hyphens-expected.png: Removed.
  • platform/mac-mojave/fast/text/hyphens-expected.txt: Removed.
  • platform/mac-mojave/fast/text/indic-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/bidi-fallback-font-weight-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/bidi-mirror-he-ar-expected.png: Removed.
  • platform/mac-mojave/fast/text/international/bidi-mirror-he-ar-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/danda-space-expected.png: Removed.
  • platform/mac-mojave/fast/text/international/danda-space-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/pop-up-button-text-alignment-and-direction-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/navigator-language/navigator-language-en-US-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/navigator-language/navigator-language-en-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/navigator-language/navigator-language-es-419-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/navigator-language/navigator-language-es-ES-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/navigator-language/navigator-language-es-MX-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/navigator-language/navigator-language-es-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/navigator-language/navigator-language-fr-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/navigator-language/navigator-language-hi-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/navigator-language/navigator-language-ja-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/navigator-language/navigator-language-pt-BR-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/navigator-language/navigator-language-ru-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/navigator-language/navigator-language-zh-HK-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/navigator-language/navigator-language-zh-Hant-HK-expected.txt: Removed.
  • platform/mac-mojave/fast/text/international/system-language/system-font-punctuation-expected.txt: Removed.
  • platform/mac-mojave/http/tests/cookies/same-site/fetch-after-navigating-iframe-in-cross-origin-page-expected.txt: Removed.
  • platform/mac-mojave/http/tests/cookies/same-site/fetch-after-top-level-cross-origin-redirect-expected.txt: Removed.
  • platform/mac-mojave/http/tests/cookies/same-site/fetch-after-top-level-navigation-from-cross-origin-page-expected.txt: Removed.
  • platform/mac-mojave/http/tests/cookies/same-site/fetch-after-top-level-navigation-initiated-from-iframe-in-cross-origin-page-expected.txt: Removed.
  • platform/mac-mojave/http/tests/cookies/same-site/fetch-in-cross-origin-service-worker-expected.txt: Removed.
  • platform/mac-mojave/http/tests/cookies/same-site/popup-cross-site-expected.txt: Removed.
  • platform/mac-mojave/http/tests/cookies/same-site/popup-cross-site-post-expected.txt: Removed.
  • platform/mac-mojave/http/tests/cookies/same-site/popup-same-site-via-cross-site-redirect-expected.txt: Removed.
  • platform/mac-mojave/http/tests/inspector/network/resource-sizes-network-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/cors/access-control-expose-headers-parsing.window-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/cors/credentials-flag-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/cors/origin-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/css/css-fonts/generic-family-keywords-001-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/css/css-pseudo/text-selection-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/fetch/api/basic/header-value-combining.any-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/fetch/api/basic/header-value-combining.any.worker-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/fetch/api/redirect/redirect-method.any-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/fetch/api/redirect/redirect-method.any.worker-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/fetch/content-type/script.window-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/fetch/nosniff/parsing-nosniff.window-actual.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/fetch/nosniff/parsing-nosniff.window-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/mathml/relations/css-styling/ignored-properties-001-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/mathml/relations/css-styling/padding-border-margin/padding-002-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/xhr/getallresponseheaders-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/xhr/getresponseheader.any-expected.txt: Removed.
  • platform/mac-mojave/imported/w3c/web-platform-tests/xhr/getresponseheader.any.worker-expected.txt: Removed.
  • platform/mac-mojave/platform/mac/fast/text/international/bidi-fallback-font-weight-expected.txt: Removed.
  • platform/mac-mojave/svg/W3C-SVG-1.1/animate-elem-46-t-expected.txt: Removed.
  • platform/mac-mojave/svg/W3C-SVG-1.1/struct-use-01-t-expected.txt: Removed.
  • platform/mac-mojave/svg/batik/text/textStyles-expected.txt: Removed.
  • platform/mac-mojave/svg/custom/glyph-selection-arabic-forms-expected.txt: Removed.
  • platform/mac-mojave/tables/mozilla/bugs/bug10296-1-expected.png: Removed.
  • platform/mac-mojave/tables/mozilla/bugs/bug10296-1-expected.txt: Removed.
  • platform/mac-mojave/tables/mozilla/bugs/bug139524-2-expected.png: Removed.
  • platform/mac-mojave/tables/mozilla/bugs/bug139524-2-expected.txt: Removed.
  • platform/mac-mojave/tables/mozilla/bugs/bug2479-3-expected.txt: Removed.
  • platform/mac-mojave/tables/mozilla/bugs/bug30692-expected.txt: Removed.
  • platform/mac-mojave/tables/mozilla/bugs/bug33855-expected.txt: Removed.
  • platform/mac-mojave/tables/mozilla/other/wa_table_thtd_rowspan-expected.txt: Removed.
  • platform/mac-mojave/tables/mozilla/other/wa_table_tr_align-expected.txt: Removed.
  • platform/mac-mojave/tables/mozilla_expected_failures/bugs/bug2479-5-expected.png: Removed.
  • platform/mac-mojave/tables/mozilla_expected_failures/bugs/bug2479-5-expected.txt: Removed.

Canonical link: https://commits.webkit.org/250321@main

11:37 AM Changeset in webkit [293853] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit

Allow hw.cpu64bit_capable sysctl-read in network process sandbox on iOS
https://bugs.webkit.org/show_bug.cgi?id=240124
<rdar://92797350>

Patch by Alex Christensen <achristensen@webkit.org> on 2022-05-05
Reviewed by Per Arne Vollan.

It's used on watchOS.

  • Resources/SandboxProfiles/ios/com.apple.WebKit.Networking.sb.in:
11:36 AM Changeset in webkit [293852] by Alan Coon
  • 4 edits in branches/safari-613-branch/Source/WebKit

Apply patch. rdar://problem/88904160

11:35 AM Changeset in webkit [293851] by Alan Coon
  • 5 edits in branches/safari-613-branch/Source/WebKit

Apply patch. rdar://problem/88904160

11:35 AM Changeset in webkit [293850] by Alan Coon
  • 18 edits in branches/safari-613-branch

Apply patch. rdar://problem/88904160

11:31 AM Changeset in webkit [293849] by Jonathan Bedard
  • 2 edits in trunk/Websites/bugs.webkit.org

[bugs.webkit.org] Replace svn.webkit.org in committers autocomplete
https://bugs.webkit.org/show_bug.cgi?id=240090
<rdar://problem/92758558>

Reviewed by Aakash Jain.

  • Websites/bugs.webkit.org/committers-autocomplete.js: Replace

svn.webkit.org with raw.githubusercontent.com.

Canonical link: https://commits.webkit.org/250319@main

11:28 AM Changeset in webkit [293848] by Alan Coon
  • 6 edits in branches/safari-613-branch/Source/WebKit

Revert r293791. rdar://problem/88904160

This reverts the application of a patch landed at r293791.

11:26 AM Changeset in webkit [293847] by ysuzuki@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

[JSC] Clean up StructureID related data
https://bugs.webkit.org/show_bug.cgi?id=240114

Reviewed by Mark Lam.

This patch moves structureHeapAddressSize to StructureID. And define it only when we use it.
We also use decontaminate() in ADDRESS32 tryDecode. Strictly speaking, it is not necessary
for now since 32bit environment does not have concurrent GC & concurrent JIT compiler, but
it can have that.

  • Source/JavaScriptCore/runtime/JSCConfig.h:
  • Source/JavaScriptCore/runtime/StructureID.h:

(JSC::StructureID::tryDecode const):

Canonical link: https://commits.webkit.org/250318@main

11:00 AM Changeset in webkit [293846] by Said Abou-Hallawa
  • 10 edits
    1 add in trunk/Source

[GPU Process] NativeImage should share the ShareableBitmap data when it's sent to GPU Process
https://bugs.webkit.org/show_bug.cgi?id=239874
rdar://92031359

Reviewed by Simon Fraser.

Source/WebCore:

  • Headers.cmake:
  • WebCore.xcodeproj/project.pbxproj:
  • platform/graphics/BackingStoreCopy.h: Added.
  • platform/graphics/ImageBufferBackend.h:

(): Deleted.

Source/WebKit:

Instead of allocating a new memory buffer in GPUProcess and copying the
ShareableBitmap data to it, we can create a PlatformImage that directly
references this data. This will make all the NativeImages data be attributed
to WebProcess.

  • GPUProcess/graphics/RemoteRenderingBackend.cpp:

(WebKit::RemoteRenderingBackend::cacheNativeImageWithQualifiedIdentifier):

  • Shared/ShareableBitmap.h:

(WebKit::ShareableBitmap::createPlatformImage):

  • Shared/cg/ShareableBitmapCG.cpp:

(WebKit::ShareableBitmap::makeCGImage):
(WebKit::ShareableBitmap::createPlatformImage):
(WebKit::ShareableBitmap::createCGImage const):
(WebKit::ShareableBitmap::releaseDataProviderData): Deleted.

  • WebProcess/GPU/graphics/ImageBufferShareableBitmapBackend.cpp:

(WebKit::ImageBufferShareableBitmapBackend::copyNativeImage const):

  • WebProcess/GPU/graphics/RemoteImageBufferProxy.h:
10:27 AM Changeset in webkit [293845] by Robert Jenner
  • 2 edits in trunk/LayoutTests

[ Catalina EWS ] webgl/2.0.0/* tests are flaky crashing ASSERTION FAILED: !needsLayout()
https://bugs.webkit.org/show_bug.cgi?id=229580

Unreviewed test gardening. Skip WebGL tests on mac-wk1.

  • platform/mac-wk1/TestExpectations:

Canonical link: https://commits.webkit.org/250316@main

10:13 AM Changeset in webkit [293844] by Russell Epstein
  • 1 copy in tags/WebKit-7614.1.12

Tag WebKit-7614.1.12.

9:52 AM Changeset in webkit [293843] by Russell Epstein
  • 2 edits in branches/safari-7614.1.12-branch/Source/WebCore

Cherry-pick r293825. rdar://problem/92635752

[GPU Process] [iOS] REGRESSION(r293570): Snapshot rendering is not scaled with the device scale factor
https://bugs.webkit.org/show_bug.cgi?id=240100
rdar://92635752

Reviewed by Simon Fraser.

The scaling factor is not set in the GraphicsContext of the
ImageBufferShareableBitmapBackend.

To fix this bug is to make snapshotFrameRectWithClip() handle the scaling
outside the ImageBuffer creation. This is similar to what we do in
GraphicsContext::createAlignedImageBuffer() where we scale the size and
create the ImageBuffer with scaleFactor = 1.

  • page/FrameSnapshotting.cpp: (WebCore::snapshotFrameRectWithClip):

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

9:40 AM Changeset in webkit [293842] by sihui_liu@apple.com
  • 5 edits
    1 add in trunk

SuspendableWorkQueue::suspend should invoke callback immediately when queue is suspended
https://bugs.webkit.org/show_bug.cgi?id=240070

Reviewed by Chris Dumez.

Source/WTF:

With current implementation, if suspend() is called when queue is suspended, the completionHandler is not
invoked unitl the queue is resumed and suspended again. This might cause confusion for callers. To fix it,
now SuspendableWorkQueue will invoke callback immediately when queue is already suspended.

API test: WTF_SuspendableWorkQueue.SuspendTwice

  • wtf/SuspendableWorkQueue.cpp:

(WTF::SuspendableWorkQueue::suspend):
(WTF::SuspendableWorkQueue::resume):
(WTF::SuspendableWorkQueue::dispatchSync):
(WTF::SuspendableWorkQueue::suspendIfNeeded):

  • wtf/SuspendableWorkQueue.h:

Tools:

  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
  • TestWebKitAPI/Tests/WTF/SuspendableWorkQueue.cpp: Added.

(TestWebKitAPI::TEST):

9:34 AM Changeset in webkit [293841] by Chris Dumez
  • 7 edits
    1 add in trunk

BroadcastChannel instance in data URL dedicated worker can communicate with creator context
https://bugs.webkit.org/show_bug.cgi?id=240016

Reviewed by Youenn Fablet.

For workers, we used to dispatch to the loader document and then get the origin from that loader document.
This normally works because the worker usually inherits the origin from its loader document. However, when
the worker is loaded from a data URL, its origin is opaque:

To address the issue, we now get the security origin directly from the BroadcastChannel's
ScriptExecutionContext, before dispatching to the main thread.

This led to one issue though because I now have to call isolatedCopy() on the SecurityOrigin to pass it to
the main thread. In particular, 2 BroadcastChannels from the same opaque origin should be able to talk to
each other. This used to work because the SecurityOrigin objects would have the same pointers and
SecurityOrigin::isSameOriginAs() would check for pointer equality. However, now that I call isolatedCopy(),
we get new & different pointers so this wouldn't work anymore. To address this, I replaced the m_isUnique
boolean data member on SecurityOrigin with a global uniqueOriginIdentifier and we check this identifier
to see if 2 opaque/unique SecurityOrigin objects are for the same opaque/unique origin. This means the
check will still work if the SecurityOrigin objects are isolated-copied or even IPC'd.

  • LayoutTests/imported/w3c/web-platform-tests/webmessaging/broadcastchannel/opaque-origin-expected.txt:
  • LayoutTests/imported/w3c/web-platform-tests/webmessaging/broadcastchannel/opaque-origin.html:

Resync to get test coverage from upstream WPT.

  • Source/WebCore/dom/BroadcastChannel.cpp:

(WebCore::BroadcastChannel::MainThreadBridge::registerChannel):

  • Source/WebCore/page/SecurityOrigin.cpp:

(WebCore::SecurityOrigin::isSameOriginAs const):
(WebCore::SecurityOrigin::equal const):

  • Source/WebCore/page/SecurityOrigin.h:

(WebCore::SecurityOrigin::isUnique const):
(WebCore::SecurityOrigin::encode const):
(WebCore::SecurityOrigin::decode):

Canonical link: https://commits.webkit.org/250314@main

9:29 AM Changeset in webkit [293840] by Karl Rackler
  • 3 edits in trunk/LayoutTests

[ iOS ][ macOS ] imported/w3c/web-platform-tests/webrtc/protocol/rtp-clockrate.html is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=240123

Unreviewed test gardening.

Canonical link: https://commits.webkit.org/250313@main

9:13 AM Changeset in webkit [293839] by jer.noble@apple.com
  • 12 edits in trunk/Source

[WK2][Cocoa] Remove use of CVPixelBufferConformer from WebContent process in MediaPlayerPrivateRemote::nativeImageForCurrentTime()
https://bugs.webkit.org/show_bug.cgi?id=240108
<rdar://problem/92773611>

Reviewed by Simon Fraser.

Source/WebCore:

No longer pass a CVPixelBufferRef up through onNewVideoFrameMetadata().

  • platform/graphics/MediaPlayer.cpp:

(WebCore::MediaPlayer::onNewVideoFrameMetadata):

  • platform/graphics/MediaPlayer.h:

(WebCore::MediaPlayerClient::mediaPlayerOnNewVideoFrameMetadata):

  • platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:

(WebCore::MediaPlayerPrivateAVFoundationObjC::checkNewVideoFrameMetadata):

  • platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm:

(WebCore::MediaPlayerPrivateMediaSourceAVFObjC::checkNewVideoFrameMetadata):

Source/WebKit:

CVPixelBufferConformer makes use of hardware acceleration to perform colorspace conversions,
and these calls fail when IOKit access is blocked from the WebContent sandbox. The use of
CVPixelBufferConformer (via PixelBufferConformerCV) in nativeImageForCurrentTime() is a
convenince when a web page is using rVFC() to avoid a call into the GPU process when that
callback results in a video->canvas paint. However, this convenience is not strictly necessary,
and by reverting this optimization, the conformance of the resulting pixel buffer is
performed upon request in the GPU process instead.

  • GPUProcess/media/RemoteMediaPlayerProxy.h:
  • GPUProcess/media/cocoa/RemoteMediaPlayerProxyCocoa.mm:

(WebKit::RemoteMediaPlayerProxy::mediaPlayerOnNewVideoFrameMetadata):

  • WebProcess/GPU/media/MediaPlayerPrivateRemote.cpp:

(WebKit::MediaPlayerPrivateRemote::stopVideoFrameMetadataGathering):

  • WebProcess/GPU/media/MediaPlayerPrivateRemote.h:
  • WebProcess/GPU/media/MediaPlayerPrivateRemote.messages.in:
  • WebProcess/GPU/media/cocoa/MediaPlayerPrivateRemoteCocoa.mm:

(WebKit::MediaPlayerPrivateRemote::pushVideoFrameMetadata):
(WebKit::MediaPlayerPrivateRemote::nativeImageForCurrentTime):

9:02 AM Changeset in webkit [293838] by commit-queue@webkit.org
  • 7 edits
    2 deletes in trunk/Tools/buildstream

[Flatpak SDK] Bump to GStreamer 1.20.2
https://bugs.webkit.org/show_bug.cgi?id=240121

Patch by Philippe Normand <pnormand@igalia.com> on 2022-05-05
Reviewed by Adrian Perez de Castro.

  • elements/sdk/gst-libav.bst:
  • elements/sdk/gst-plugins-bad.bst:
  • elements/sdk/gst-plugins-base.bst:
  • elements/sdk/gst-plugins-good.bst:
  • elements/sdk/gst-plugins-ugly.bst:
  • elements/sdk/gstreamer.bst:
  • patches/gstreamer-0001-typefind-Skip-parsing-of-data-URIs.patch: Removed.
  • patches/gstreamer-0002-uri-Build-doubly-linked-list-by-prepending-items.patch: Removed.

Canonical link: https://commits.webkit.org/250311@main

8:55 AM Changeset in webkit [293837] by Simon Fraser
  • 3 edits in trunk/Source/WebKit

Optimize IPC Decoding for primitive types
https://bugs.webkit.org/show_bug.cgi?id=240106

Reviewed by Cameron McCormack.

Calls to memmove() showed up on profiles under Decoder::decodeFixedLengthData() for
primitive types. Allow the compiler to optimize these away by inlining Decoder::decodeFixedLengthData()
and related functions.

  • Platform/IPC/Decoder.cpp:

(IPC::roundUpToAlignment): Deleted.
(IPC::alignedBufferIsLargeEnoughToContain): Deleted.
(IPC::Decoder::alignBufferPosition): Deleted.
(IPC::Decoder::bufferIsLargeEnoughToContain const): Deleted.
(IPC::Decoder::decodeFixedLengthData): Deleted.

  • Platform/IPC/Decoder.h:

(IPC::roundUpToAlignment):
(IPC::alignedBufferIsLargeEnoughToContain):
(IPC::Decoder::alignBufferPosition):
(IPC::Decoder::bufferIsLargeEnoughToContain const):
(IPC::Decoder::decodeFixedLengthData):

8:48 AM Changeset in webkit [293836] by Jonathan Bedard
  • 1 edit in trunk/Tools/Scripts/webkitpy/common/config/watchlist

Add myself (Youssef Soliman) to watchlist for Media
https://bugs.webkit.org/show_bug.cgi?id=240101

Reviewed by Eric Carlson.

  • Tools/Scripts/webkitpy/common/config/watchlist:

Canonical link: https://commits.webkit.org/250309@main

8:36 AM Changeset in webkit [293835] by Kate Cheney
  • 5 edits
    1 add in trunk

Right click using trackpad crashes Safari
https://bugs.webkit.org/show_bug.cgi?id=240030
rdar://91786269

Reviewed by Tim Horton.

Source/WebCore:

Trackpad on iOS should mimic mac selection behavior and block context
menu clicks that are not made on a text node.

  • page/EventHandler.cpp:

(WebCore::EventHandler::selectClosestContextualWordOrLinkFromHitTestResult):
(WebCore::EventHandler::sendContextMenuEvent):

Tools:

  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
  • TestWebKitAPI/Tests/WebKit/emptyTable.html: Added.
  • TestWebKitAPI/Tests/WebKitCocoa/iOSMouseSupport.mm:

(TEST):

8:18 AM Changeset in webkit [293834] by Karl Rackler
  • 2 edits in trunk/LayoutTests

[Gardening]REGRESSION (r293506): [ iOS ][ macOS ] imported/w3c/web-platform-tests/service-workers/service-worker/registration-updateviacache.https.html is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=240074

Unreviewed test gardening.

Canonical link: https://commits.webkit.org/250307@main

8:15 AM Changeset in webkit [293833] by youenn@apple.com
  • 4 edits in trunk/Source

Add SharedVideoFrameReader/SharedVideoFrameWriter logging for error cases
https://bugs.webkit.org/show_bug.cgi?id=236066
<rdar://problem/88424960>

Reviewed by Eric Carlson.

Source/WebCore:

No change of behavior.

  • platform/cocoa/SharedVideoFrameInfo.mm:

(WebCore::SharedVideoFrameInfo::writePixelBuffer):

Source/WebKit:

  • WebProcess/GPU/webrtc/SharedVideoFrame.cpp:

(WebKit::SharedVideoFrameWriter::prepareWriting):
(WebKit::SharedVideoFrameReader::readBufferFromSharedMemory):
(WebKit::SharedVideoFrameReader::readBuffer):

8:04 AM Changeset in webkit [293832] by Jonathan Bedard
  • 3 edits in trunk/Tools

[webkitbugspy] Include mocks in package
https://bugs.webkit.org/show_bug.cgi?id=240098
<rdar://problem/92763124>

Reviewed by Aakash Jain.

  • Scripts/libraries/webkitbugspy/setup.py:
  • Scripts/libraries/webkitbugspy/webkitbugspy/init.py:

Canonical link: https://commits.webkit.org/250305@main

8:01 AM Changeset in webkit [293831] by Jonathan Bedard
  • 5 edits in trunk/Tools

[git-webkit] Fetch alternate remotes when adding remote
https://bugs.webkit.org/show_bug.cgi?id=240066
<rdar://problem/92733391>

Reviewed by Ryan Haddad.

  • Tools/Scripts/libraries/webkitscmpy/setup.py: Bump version.
  • Tools/Scripts/libraries/webkitscmpy/webkitscmpy/init.py: Ditto,
  • Tools/Scripts/libraries/webkitscmpy/webkitscmpy/mocks/local/git.py: Mock git fetch

for non-fork remotes.

  • Tools/Scripts/libraries/webkitscmpy/webkitscmpy/program/setup.py:

(Setup._add_remote): Optionally fetch remote after adding it.
(Setup.git):

Canonical link: https://commits.webkit.org/250304@main

7:45 AM Changeset in webkit [293830] by Ziran Sun
  • 7 edits
    1 copy
    1 add in trunk

<input type=color> should have box-sizing: border-box in UA stylesheet
https://bugs.webkit.org/show_bug.cgi?id=197878

Reviewed by Tim Nguyen.

LayoutTests/imported/w3c:

  • web-platform-tests/html/rendering/non-replaced-elements/form-controls/resets-expected.txt:

Source/WebCore:

As discussed at [1], We should have box-sizing: border-box for <input type=color>, the same
as buttons. WebKit currently has content-box. This patch is to change it to box-box.

Since <input type=color> is disabled with WebKitLegacy [2], this fix doesn't apply to it. Hence,
for mac-wk1 platform, the test expectation for html/rendering/non-replaced-elements/form-controls/resets.html
stays as it is.

[1] https://github.com/whatwg/html/issues/4281
[2] https://webkit-search.igalia.com/webkit/source/Source/WTF/Scripts/Preferences/WebPreferencesInternal.yaml#425-437

  • rendering/RenderTheme.cpp:

(WebCore::RenderTheme::colorInputStyleSheet const):

  • rendering/RenderThemeIOS.mm:

(WebCore::RenderThemeIOS::colorInputStyleSheet const):

LayoutTests:

  • platform/glib/imported/w3c/web-platform-tests/html/rendering/non-replaced-elements/form-controls/resets-expected.txt:
  • platform/mac-wk1/imported/w3c/web-platform-tests/html/rendering/non-replaced-elements/form-controls/resets-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/html/rendering/non-replaced-elements/form-controls/resets-expected.txt.
6:42 AM Changeset in webkit [293829] by commit-queue@webkit.org
  • 22 edits
    1 delete in trunk/Source/WebKit

Connecting to GPU process may hang if UI process sends sync message simultaneously
https://bugs.webkit.org/show_bug.cgi?id=239905

Patch by Kimmo Kinnunen <kkinnunen@apple.com> on 2022-05-05
Reviewed by Chris Dumez.

Previously, establishing GPU process connection would be a sync message:

  1. WP -> UI sync GetGPUProcessConnection
  2. UI -> GPUP async CreateGPUConnectionToWebProcess
  3. GPUP -> UI async reply CreateGPUConnectionToWebProcess
  4. UI -> WP sync reply GetGPUProcessConnection

If UI would send a message to WP after step 3 and wait for reply, this
would never come as WP would be waiting for the sync reply for GetGPUProcessConnection.

This would happen for example with requestAutocorrectionContextWithCompletionHandler,
which would send HandleAutocorrectionContextRequest and wait for
Messages::WebPageProxy::HandleAutocorrectionContext messages.

Mitigate this by creating the GPU connection in WP and sending
that to GPUP through UI in an async message. This way WP does not block on the GPUP
connection initialization. Send the GPUP connection initialization result, audit token and
vp9 HW decoder presence, as the first user message through the new connection.

  • GPUProcess/GPUConnectionToWebProcess.cpp:

(WebKit::GPUConnectionToWebProcess::create):
(WebKit::GPUConnectionToWebProcess::GPUConnectionToWebProcess):

  • GPUProcess/GPUConnectionToWebProcess.h:
  • GPUProcess/GPUProcess.cpp:

(WebKit::asConnectionIdentifier):
(WebKit::GPUProcess::createGPUConnectionToWebProcess):

  • GPUProcess/GPUProcess.h:
  • GPUProcess/GPUProcess.messages.in:
  • Shared/GPUProcessConnectionInitializationParameters.h:

(WebKit::GPUProcessConnectionInitializationParameters::decode):

  • Shared/GPUProcessConnectionParameters.h:

(WebKit::GPUProcessConnectionParameters::encode const):
(WebKit::GPUProcessConnectionParameters::decode):

  • UIProcess/GPU/GPUProcessProxy.cpp:

(WebKit::GPUProcessProxy::createGPUProcessConnection):

  • UIProcess/GPU/GPUProcessProxy.h:
  • UIProcess/WebProcessPool.cpp:

(WebKit::WebProcessPool::createGPUProcessConnection):

  • UIProcess/WebProcessPool.h:
  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::createGPUProcessConnection):

  • UIProcess/WebProcessProxy.h:
  • UIProcess/WebProcessProxy.messages.in:
  • WebProcess/GPU/GPUProcessConnection.cpp:

(WebKit::getGPUProcessConnectionParameters):
(WebKit::GPUProcessConnection::create):
(WebKit::GPUProcessConnection::GPUProcessConnection):
(WebKit::GPUProcessConnection::auditToken):
(WebKit::GPUProcessConnection::invalidate):
(WebKit::GPUProcessConnection::didInitialize):
(WebKit::GPUProcessConnection::waitForDidInitialize):
(WebKit::GPUProcessConnection::hasVP9HardwareDecoder):

  • WebProcess/GPU/GPUProcessConnection.h:
  • WebProcess/GPU/GPUProcessConnection.messages.in:
  • WebProcess/GPU/GPUProcessConnectionInfo.h:

(WebKit::GPUProcessConnectionInfo::encode const):
(WebKit::GPUProcessConnectionInfo::decode):

  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::ensureGPUProcessConnection):

  • WebProcess/WebProcess.h:
  • WebProcess/cocoa/WebProcessCocoa.mm:
5:12 AM Changeset in webkit [293828] by Diego Pino Garcia
  • 2 edits in trunk/Source/JavaScriptCore

[GCC] REGRESSION(r293605): error: cannot convert ‘<brace-enclosed initializer list>’ to ‘unsigned char:3’ in initialization
https://bugs.webkit.org/show_bug.cgi?id=239897

Reviewed by Yusuke Suzuki.

  • bytecode/MethodOfGettingAValueProfile.h: Move initialization of 'm_kind' to class constructor.
2:48 AM Changeset in webkit [293827] by youenn@apple.com
  • 2 edits in trunk/Source/WebCore

replaceTrack with different constraints stops sending packets
https://bugs.webkit.org/show_bug.cgi?id=239978
<rdar://problem/92624773>

Reviewed by Eric Carlson.

We should always reconfigure the microphone processor even if we are not using it,
as VPIO expects that input and output formats should match.

Manually tested with https://bugs.webkit.org/show_bug.cgi?id=239978 and https://jsfiddle.net/72qsLw9a/.
A follow-up should allow to put more common code between CoreAudioSharedUnit and MockAudioSharedUnit
so that we can write a regression test.

  • platform/mediastream/mac/CoreAudioCaptureSource.cpp:
1:50 AM WebKitGTK/2.36.x edited by Adrian Perez de Castro
(diff)
1:33 AM Changeset in webkit [293826] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

REGRESSION(249114@main) [GTK] Crashes on shutdown if the display is not set
https://bugs.webkit.org/show_bug.cgi?id=239767

Patch by Carlos Garcia Campos <cgarcia@igalia.com> on 2022-05-05
Reviewed by Michael Catanzaro.

Handle the case of PlatformDisplay created with a nullptr GdkDisplay.

  • platform/graphics/PlatformDisplay.cpp:

(WebCore::PlatformDisplay::PlatformDisplay):

  • platform/graphics/wayland/PlatformDisplayWayland.cpp:

(WebCore::PlatformDisplayWayland::PlatformDisplayWayland):
(WebCore::PlatformDisplayWayland::~PlatformDisplayWayland):

  • platform/graphics/x11/PlatformDisplayX11.cpp:

(WebCore::PlatformDisplayX11::PlatformDisplayX11):
(WebCore::PlatformDisplayX11::~PlatformDisplayX11):

1:19 AM Changeset in webkit [293825] by Said Abou-Hallawa
  • 2 edits in trunk/Source/WebCore

[GPU Process] [iOS] REGRESSION(r293570): Snapshot rendering is not scaled with the device scale factor
https://bugs.webkit.org/show_bug.cgi?id=240100
rdar://92635752

Reviewed by Simon Fraser.

The scaling factor is not set in the GraphicsContext of the
ImageBufferShareableBitmapBackend.

To fix this bug is to make snapshotFrameRectWithClip() handle the scaling
outside the ImageBuffer creation. This is similar to what we do in
GraphicsContext::createAlignedImageBuffer() where we scale the size and
create the ImageBuffer with scaleFactor = 1.

  • page/FrameSnapshotting.cpp:

(WebCore::snapshotFrameRectWithClip):

12:38 AM Changeset in webkit [293824] by youenn@apple.com
  • 21 edits
    6 deletes in trunk/Source

SWOriginStore is no longer needed
https://bugs.webkit.org/show_bug.cgi?id=240003

Reviewed by Chris Dumez.

Source/WebCore:

Covered by existing tests.

  • Headers.cmake:
  • Sources.txt:
  • WebCore.xcodeproj/project.pbxproj:
  • loader/DocumentLoader.cpp:
  • workers/service/SWClientConnection.h:
  • workers/service/WorkerSWClientConnection.cpp:
  • workers/service/WorkerSWClientConnection.h:
  • workers/service/server/SWOriginStore.cpp: Removed.
  • workers/service/server/SWOriginStore.h: Removed.
  • workers/service/server/SWServer.cpp:
  • workers/service/server/SWServer.h:

Source/WebKit:

No observable change, we remove the SWOrigin optimization as its main remaining use
is to optimize the case of a page being loaded by service worker instead of app cache which is a tiny edge case.

  • NetworkProcess/NetworkSession.cpp:
  • NetworkProcess/NetworkSession.h:
  • NetworkProcess/ServiceWorker/WebSWOriginStore.cpp: Removed.
  • NetworkProcess/ServiceWorker/WebSWOriginStore.h: Removed.
  • NetworkProcess/ServiceWorker/WebSWServerConnection.cpp:
  • NetworkProcess/ServiceWorker/WebSWServerConnection.h:
  • WebKit.xcodeproj/project.pbxproj:
  • WebProcess/Storage/WebSWClientConnection.cpp:
  • WebProcess/Storage/WebSWClientConnection.h:
  • WebProcess/Storage/WebSWClientConnection.messages.in:
  • WebProcess/Storage/WebSWOriginTable.cpp: Removed.
  • WebProcess/Storage/WebSWOriginTable.h: Removed.
Note: See TracTimeline for information about the timeline view.