[webkit-changes] [WebKit/WebKit] 643f0a: Align const member functions of CheckedRef/Checked...

2023-08-31 Thread Ryosuke Niwa
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 643f0a10c60b19057c260f790290b9d56ecb7de1
  
https://github.com/WebKit/WebKit/commit/643f0a10c60b19057c260f790290b9d56ecb7de1
  Author: Ryosuke Niwa 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WTF/wtf/CheckedPtr.h
M Source/WTF/wtf/CheckedRef.h

  Log Message:
  ---
  Align const member functions of CheckedRef/CheckedPtr with Ref/RefPtr
https://bugs.webkit.org/show_bug.cgi?id=261001

Reviewed by Chris Dumez.

Make const members of CheckedRef/CheckedPtr return mutable T&/T* instead of
const T/&T* to match the corresponding functions in Ref/RefPtr.

* Source/WTF/wtf/CheckedPtr.h:
(WTF::CheckedPtr::operator bool const):
(WTF::CheckedPtr::operator! const): Deleted.
(WTF::CheckedPtr::get const):
(WTF::CheckedPtr::get): Deleted.
(WTF::CheckedPtr::operator* const):
(WTF::CheckedPtr::operator*): Deleted.
(WTF::CheckedPtr::operator-> const):
(WTF::CheckedPtr::operator->): Deleted.
* Source/WTF/wtf/CheckedRef.h:
(WTF::CheckedRef::ptr const):
(WTF::CheckedRef::ptr): Deleted.
(WTF::CheckedRef::get const):
(WTF::CheckedRef::get): Deleted.
(WTF::CheckedRef::operator-> const):
(WTF::CheckedRef::operator->): Deleted.
(WTF::CheckedRef::operator T& const):
(WTF::CheckedRef::operator const T& const): Deleted.
(WTF::CheckedRef::operator bool const):
(WTF::CheckedRef::operator! const): Deleted.
(WTF::CheckedRef::operator T&): Deleted.

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] b332b3: [CMake] rewrite-compile-commands reports a lot of ...

2023-08-31 Thread Fujii Hironori
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b332b310db505e67e0cc8561fcdb47bd3117b79a
  
https://github.com/WebKit/WebKit/commit/b332b310db505e67e0cc8561fcdb47bd3117b79a
  Author: Fujii Hironori 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Tools/Scripts/rewrite-compile-commands

  Log Message:
  ---
  [CMake] rewrite-compile-commands reports a lot of warning messages "Failed to 
find ..."
https://bugs.webkit.org/show_bug.cgi?id=260841

Reviewed by Don Olmstead.

rewrite-compile-commands scripts reported a lot of warning messages of
missing derived sources for the build-webkit first invocation. We
don't have a nice solution yet. And, derived sources don't have to be
included in compile_commands.json because we don't directly edit them.
Let's just ignore the missing derived sources at the moment.

* Tools/Scripts/rewrite-compile-commands:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 6566fb: [Fast text codepath] CharactersTreatedAsSpace is o...

2023-08-31 Thread Myles C. Maxfield
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 6566fb8f9216aae136892c5f32e6e50367de2969
  
https://github.com/WebKit/WebKit/commit/6566fb8f9216aae136892c5f32e6e50367de2969
  Author: Myles C. Maxfield 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M 
LayoutTests/platform/ios/tables/layering/paint-test-layering-1-expected.txt
M 
LayoutTests/platform/ios/tables/layering/paint-test-layering-2-expected.txt
M LayoutTests/platform/ios/tables/mozilla/bugs/bug4427-expected.txt
M 
LayoutTests/platform/ios/tables/mozilla_expected_failures/other/test4-expected.txt
M Source/WebCore/platform/graphics/WidthIterator.cpp

  Log Message:
  ---
  [Fast text codepath] CharactersTreatedAsSpace is overly prescriptive
https://bugs.webkit.org/show_bug.cgi?id=260843
rdar://114607926

Reviewed by Cameron McCormack.

When we lay out web content, we make multiple passes over the text,
chopping it up multiple different ways. Because of this, we have an
invariant that the width of a space can't change depending on the
bounds of how we're chopping up the text. Kerning can affect the
width of a space, so we enforce the invariant by having a postprocess
that we perform after shaping to retroactively go back and adjust
the width of any glyphs associated with space characters, to reset
their widths. This is called "CharactersTreatedAsSpace."

However, CharactersTreatedAsSpace actually does something else, too:
it also adjusts the width of the character directly before the space
to be whatever it was before shaping. However, this is kind of
bogus - it's totally valid for shaping to adjust the width of
whichever character happens to be before the space.

Now that we're trying to make the fast text codepath handle complex
character clusters, we're hitting the case more often where shaping
is adjusting the non-space glyphs to be correct, but then our
postprocess is coming along and messing them up. This patch removes
that logic, just for the glyphs before the space characters. The
logic about trying to reset the space characters' width is still
there.

* Source/WebCore/platform/graphics/WidthIterator.cpp:
(WebCore::OriginalAdvancesForCharacterTreatedAsSpace::OriginalAdvancesForCharacterTreatedAsSpace):
(WebCore::WidthIterator::applyFontTransforms):
(WebCore::WidthIterator::advanceInternal):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 19378f: [css-fonts] Implement `font-variant-emoji` CSS pro...

2023-08-31 Thread Tim Nguyen
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 19378f7d7d7dc8af96607511ee1cf490ae4ae404
  
https://github.com/WebKit/WebKit/commit/19378f7d7d7dc8af96607511ee1cf490ae4ae404
  Author: Tim Nguyen 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M LayoutTests/TestExpectations
M LayoutTests/fast/css/font-property-priority-expected.txt
M LayoutTests/fast/css/font-shorthand-expected.txt
M 
LayoutTests/fast/css/getComputedStyle/computed-style-font-family-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/css/css-fonts/font-shorthand-serialization-prevention-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/css/css-fonts/font-shorthand-subproperties-reset-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/css/css-fonts/inheritance-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/css/css-fonts/parsing/font-variant-emoji-computed-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/css/css-fonts/parsing/font-variant-emoji-valid-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/css/css-typed-om/the-stylepropertymap/properties/font-variant-emoji-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/css/css-typed-om/the-stylepropertymap/properties/font-variant-emoji.html
M 
LayoutTests/imported/w3c/web-platform-tests/css/cssom/font-shorthand-serialization-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/editing/run/removeformat-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/animation-types/accumulation-per-property-001-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/animation-types/addition-per-property-001-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/animation-types/interpolation-per-property-001-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/animation-types/property-list.js
M LayoutTests/platform/glib/TestExpectations
M 
LayoutTests/platform/gtk/imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml-expected.txt
M 
LayoutTests/platform/ios-wk2/imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml-expected.txt
M LayoutTests/platform/ios/TestExpectations
M 
LayoutTests/platform/ipad/imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml-expected.txt
M 
LayoutTests/platform/mac-wk1/imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml-expected.txt
M 
LayoutTests/platform/wpe/imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml-expected.txt
M Source/WebCore/animation/CSSPropertyAnimation.cpp
M Source/WebCore/css/CSSPrimitiveValueMappings.h
M Source/WebCore/css/CSSProperties.json
M Source/WebCore/css/CSSValueKeywords.in
M Source/WebCore/css/ComputedStyleExtractor.cpp
M Source/WebCore/css/ShorthandSerializer.cpp
M Source/WebCore/css/parser/CSSPropertyParser.cpp
M Source/WebCore/platform/graphics/FontCascade.cpp
M Source/WebCore/platform/graphics/FontCascadeCache.h
M Source/WebCore/platform/graphics/FontCascadeDescription.h
M Source/WebCore/platform/graphics/FontDescription.cpp
M Source/WebCore/platform/graphics/FontDescription.h
M Source/WebCore/platform/text/TextFlags.h

  Log Message:
  ---
  [css-fonts] Implement `font-variant-emoji` CSS property
https://bugs.webkit.org/show_bug.cgi?id=246911
rdar://101722745

Reviewed by Myles C. Maxfield.

This builds off existing code that Myles wrote in bug 259139 for platform 
support.

font-variant-emoji: normal | text | emoji | unicode

https://drafts.csswg.org/css-fonts-4/#font-variant-emoji-prop

* LayoutTests/TestExpectations:
* LayoutTests/fast/css/font-property-priority-expected.txt:
* LayoutTests/fast/css/font-shorthand-expected.txt:
* LayoutTests/fast/css/getComputedStyle/computed-style-font-family-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-fonts/font-shorthand-serialization-prevention-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-fonts/font-shorthand-subproperties-reset-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-fonts/inheritance-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-fonts/parsing/font-variant-emoji-computed-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-fonts/parsing/font-variant-emoji-valid-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-typed-om/the-stylepropertymap/properties/font-variant-emoji-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-typed-om/the-stylepropertymap/properties/font-vari

[webkit-changes] [WebKit/WebKit] 2e3f11: [visionOS] Find in HTML note obscures matches when...

2023-08-31 Thread Aditya Keerthi
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 2e3f11b2736f256ef25d919e5a7ebb7294ac9be9
  
https://github.com/WebKit/WebKit/commit/2e3f11b2736f256ef25d919e5a7ebb7294ac9be9
  Author: Aditya Keerthi 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebKit/UIProcess/API/ios/WKWebViewIOS.mm
M Tools/TestWebKitAPI/Tests/WebKitCocoa/FindInPage.mm

  Log Message:
  ---
  [visionOS] Find in HTML note obscures matches when they are at the top
https://bugs.webkit.org/show_bug.cgi?id=260982
rdar://109165939

Reviewed by Wenson Hsieh.

On visionOS, the find bar initially appears in the scroll view's inset area.
As the scroll view is scrolled, the find bar's frame is adjusted to ensure
it is always at the top of the currently visible rect. Both these behaviors are
implemented by UIKit.

When matches are found, scrolling is performed to make them visible. This is
achieved by using `-[WKWebView _scrollToRect:origin:minimumScrollDistance:]`.
However, that method always enforces a minimum content offset of (0, 0). This
is incompatible with the find bar's position, since for matches at the top of
the scroll view, a scroll to (0, 0) is forced, even when match is already 
visible.
The forced scroll to (0, 0) then ends up obscuring the match, as the find bar's
position is adjusted to keep it at the top.

Fix by accounting for content insets when determining the minimum content offset
for targeted scrolling.

* Source/WebKit/UIProcess/API/ios/WKWebViewIOS.mm:
(-[WKWebView _initialContentOffsetForScrollView]):
(constrainContentOffset):
(-[WKWebView _scrollToRect:origin:minimumScrollDistance:]):
* Tools/TestWebKitAPI/Tests/WebKitCocoa/FindInPage.mm:
(TEST):

Use a 500ms delay to ensure that scrolling did not occur. Waiting for two
presentation updates is unfortunately insufficient.

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 86bdc4: [IFC] Add support for out-of-flow static positioni...

2023-08-31 Thread Alan Baradlay
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 86bdc446a589be89d1762044237b660ea79564fb
  
https://github.com/WebKit/WebKit/commit/86bdc446a589be89d1762044237b660ea79564fb
  Author: Alan Baradlay 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M 
LayoutTests/platform/ios/fast/block/positioning/auto/vertical-lr/006-expected.txt
M 
Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.cpp
M 
Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.h
M Source/WebCore/layout/integration/LayoutIntegrationCoverage.cpp
M Source/WebCore/layout/integration/LayoutIntegrationCoverage.h

  Log Message:
  ---
  [IFC] Add support for out-of-flow static positioning in bidi content
https://bugs.webkit.org/show_bug.cgi?id=260832

Reviewed by Antti Koivisto.

* 
Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.cpp:
(WebCore::Layout::InlineDisplayContentBuilder::processBidiContent):
(WebCore::Layout::InlineDisplayContentBuilder::setGeometryForBlockLevelOutOfFlowBoxes):
* 
Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.h:
(WebCore::Layout::InlineDisplayContentBuilder::setGeometryForBlockLevelOutOfFlowBoxes):
* Source/WebCore/layout/integration/LayoutIntegrationCoverage.cpp:
(WebCore::LayoutIntegration::printReason):
(WebCore::LayoutIntegration::canUseForChild):
* Source/WebCore/layout/integration/LayoutIntegrationCoverage.h:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 0f288b: [git-webkit] Dupe CCed bug when racing bug importer

2023-08-31 Thread Jonathan Bedard
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 0f288bbe5ec0dcff5c1cb7fba8a3a36789f99b1d
  
https://github.com/WebKit/WebKit/commit/0f288bbe5ec0dcff5c1cb7fba8a3a36789f99b1d
  Author: Jonathan Bedard 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Tools/Scripts/libraries/webkitbugspy/setup.py
M Tools/Scripts/libraries/webkitbugspy/webkitbugspy/__init__.py
M Tools/Scripts/libraries/webkitbugspy/webkitbugspy/bugzilla.py
M Tools/Scripts/libraries/webkitbugspy/webkitbugspy/github.py
M Tools/Scripts/libraries/webkitbugspy/webkitbugspy/issue.py
M Tools/Scripts/libraries/webkitbugspy/webkitbugspy/mocks/bugzilla.py
M Tools/Scripts/libraries/webkitbugspy/webkitbugspy/mocks/radar.py
M Tools/Scripts/libraries/webkitbugspy/webkitbugspy/radar.py
M 
Tools/Scripts/libraries/webkitbugspy/webkitbugspy/tests/bugzilla_unittest.py
M Tools/Scripts/libraries/webkitbugspy/webkitbugspy/tests/github_unittest.py
M Tools/Scripts/libraries/webkitbugspy/webkitbugspy/tests/radar_unittest.py
M Tools/Scripts/libraries/webkitscmpy/setup.py
M Tools/Scripts/libraries/webkitscmpy/webkitscmpy/__init__.py
M Tools/Scripts/libraries/webkitscmpy/webkitscmpy/program/branch.py

  Log Message:
  ---
  [git-webkit] Dupe CCed bug when racing bug importer
https://bugs.webkit.org/show_bug.cgi?id=260747
rdar://114272849

Reviewed by Aakash Jain.

In some situations, git-webkit ends up racing our bug importer.
In that case, take the radar created by our bug importer and dupe it
to the radar provided by the contributor.

* Tools/Scripts/libraries/webkitbugspy/setup.py: Bump version.
* Tools/Scripts/libraries/webkitbugspy/webkitbugspy/__init__.py: Ditto.
* Tools/Scripts/libraries/webkitbugspy/webkitbugspy/bugzilla.py:
(Tracker.populate): Populate bug dupe status.
(Tracker.set): Allow for bug to be closed as a dupe.
* Tools/Scripts/libraries/webkitbugspy/webkitbugspy/github.py:
(Tracker.set): Link closed bug to a dupe, if applicable.
* Tools/Scripts/libraries/webkitbugspy/webkitbugspy/issue.py:
(Issue.__init__): Add _original value.
(Issue.close): Allow caller to close bug as a duplicate.
(Issue.original): Return the original bug, if closed as a dupe.
* Tools/Scripts/libraries/webkitbugspy/webkitbugspy/mocks/bugzilla.py:
(Bugzilla._issue): Support duping.
* Tools/Scripts/libraries/webkitbugspy/webkitbugspy/mocks/radar.py:
(RadarModel.__init__): Support duping.
(RadarModel.commit_changes): Ditto.
* Tools/Scripts/libraries/webkitbugspy/webkitbugspy/radar.py:
(Tracker.populate): Populate bug dupe status.
(Tracker.set): Allow for bug to be closed as a dupe.
* Tools/Scripts/libraries/webkitbugspy/webkitbugspy/tests/bugzilla_unittest.py:
(TestBugzilla.test_duplicate):
* Tools/Scripts/libraries/webkitbugspy/webkitbugspy/tests/github_unittest.py:
(TestGitHub.test_duplicate):
* Tools/Scripts/libraries/webkitbugspy/webkitbugspy/tests/radar_unittest.py:
(TestRadar.test_duplicate):
* 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 the cced radar and provided radar don't match, close the cced 
radar
as a dupe of the provided radar.

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] eb81fb: [WPE] LayoutTests/fast/block/positioning passing u...

2023-08-31 Thread Amanda Falke
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: eb81fbb250e1155e83ece2f3488d34c1857446ab
  
https://github.com/WebKit/WebKit/commit/eb81fbb250e1155e83ece2f3488d34c1857446ab
  Author: Amanda Falke 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
A LayoutTests/platform/glib/fast/block/positioning/056-expected.txt
R LayoutTests/platform/gtk/fast/block/positioning/056-expected.txt
R LayoutTests/platform/wpe/fast/block/positioning/056-expected.txt

  Log Message:
  ---
  [WPE] LayoutTests/fast/block/positioning passing unexpectedly

Unreviewed test gardening.

This test was failing as expected in all ports (see webkit.org/b/230593,
and 243008@main where we added this test expectation as failing in all ports
of WebKit due to lack of IFC support regarding gaps between glyphs when
kerning is on, particularly when content spans across multiple inline
boxes like this test). At some point, the WPE expectations were updated,
but the GTK and WPE expectation files for this test should be matching.

Moved GTK expectation up to GLib expectations; now WPE fails - as GTK did
(and does) - as expected.

* LayoutTests/platform/glib/fast/block/positioning/056-expected.txt: Renamed 
from LayoutTests/platform/gtk/fast/block/positioning/056-expected.txt.
* LayoutTests/platform/wpe/fast/block/positioning/056-expected.txt: Removed.

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 0dcc7a: WebGLFramebuffer uses renderbuffer and texture att...

2023-08-31 Thread Kimmo Kinnunen
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 0dcc7a680e6c5e7bf96006e9c3bd17d185861520
  
https://github.com/WebKit/WebKit/commit/0dcc7a680e6c5e7bf96006e9c3bd17d185861520
  Author: Kimmo Kinnunen 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebCore/html/canvas/WebGL2RenderingContext.cpp
M Source/WebCore/html/canvas/WebGLFramebuffer.cpp
M Source/WebCore/html/canvas/WebGLFramebuffer.h
M Source/WebCore/html/canvas/WebGLObject.h
M Source/WebCore/html/canvas/WebGLRenderbuffer.h
M Source/WebCore/html/canvas/WebGLRenderingContext.cpp
M Source/WebCore/html/canvas/WebGLRenderingContextBase.cpp
M Source/WebCore/html/canvas/WebGLTexture.h

  Log Message:
  ---
  WebGLFramebuffer uses renderbuffer and texture attachments via WebGLObject
https://bugs.webkit.org/show_bug.cgi?id=260955
rdar://114750589

Reviewed by Dan Glastonbury.

Framebuffer attachments can be 2d textures, 3d texture layers or
renderbuffers. Use std::variant to encode this instead of
a virtual base class. This removes a lot of unneeded complexity in
the code. This also enables future bugfixes related to non-typesafe
handling of these variants.

When obtaining a pointer to the attached object, use std::variant
instead of WebGLObject*. This way future commits may break the
dependency to a common base class. The objects do not have intrinsic
polymorphic operations, rather their types are always known at the
invocation site, from JS through WebGLRenderingContextBase.

* Source/WebCore/html/canvas/WebGL2RenderingContext.cpp:
(WebCore::WebGL2RenderingContext::framebufferTextureLayer):
(WebCore::WebGL2RenderingContext::getFramebufferAttachmentParameter):
* Source/WebCore/html/canvas/WebGLFramebuffer.cpp:
(WebCore::entryAddMembersToOpaqueRoots):
(WebCore::entryDetachAndClear):
(WebCore::entryAttach):
(WebCore::entryContextSetAttachment):
(WebCore::entryObject):
(WebCore::entryHasObject):
(WebCore::WebGLFramebuffer::setAttachmentForBoundFramebuffer):
(WebCore::WebGLFramebuffer::getAttachmentObject const):
(WebCore::WebGLFramebuffer::removeAttachmentFromBoundFramebuffer):
(WebCore::WebGLFramebuffer::deleteObjectImpl):
(WebCore::WebGLFramebuffer::drawBuffersIfNecessary):
(WebCore::WebGLFramebuffer::addMembersToOpaqueRoots):
(WebCore::WebGLFramebuffer::setAttachmentInternal):
(WebCore::WebGLFramebuffer::TextureAttachment::operator== const):
(WebCore::WebGLFramebuffer::TextureLayerAttachment::operator== const):
(): Deleted.
(WebCore::WebGLFramebuffer::attach): Deleted.
(WebCore::WebGLFramebuffer::getAttachment const): Deleted.
(WebCore::WebGLFramebuffer::removeAttachmentInternal): Deleted.
* Source/WebCore/html/canvas/WebGLFramebuffer.h:
* Source/WebCore/html/canvas/WebGLObject.h:
(WebCore::WebGLObject::isRenderbuffer const): Deleted.
(WebCore::WebGLObject::isTexture const): Deleted.
* Source/WebCore/html/canvas/WebGLRenderbuffer.h:
* Source/WebCore/html/canvas/WebGLRenderingContext.cpp:
(WebCore::WebGLRenderingContext::getFramebufferAttachmentParameter):
* Source/WebCore/html/canvas/WebGLRenderingContextBase.cpp:
(WebCore::WebGLRenderingContextBase::framebufferRenderbuffer):
(WebCore::WebGLRenderingContextBase::framebufferTexture2D):
* Source/WebCore/html/canvas/WebGLTexture.h:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] e803ed: [Gardening]: [ macOS wk2 ] swipe/pushState-cached-...

2023-08-31 Thread Ben
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: e803ed36e11d0a0d37f23143b6b9ba45f55ce9b9
  
https://github.com/WebKit/WebKit/commit/e803ed36e11d0a0d37f23143b6b9ba45f55ce9b9
  Author: Ben Schwartz 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M LayoutTests/platform/mac-wk2/TestExpectations

  Log Message:
  ---
  [Gardening]: [ macOS wk2 ] swipe/pushState-cached-back-swipe.html is a flaky 
timeout / text failure.
rdar://114797952
https://bugs.webkit.org/show_bug.cgi?id=261005

Unreviewed test gardening.

* LayoutTests/platform/mac-wk2/TestExpectations:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 4d0e5a: [PlayStation] Enable variation fonts experimentally

2023-08-31 Thread Don Olmstead
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 4d0e5aebe06eca31e00687fd7474b2db135355ea
  
https://github.com/WebKit/WebKit/commit/4d0e5aebe06eca31e00687fd7474b2db135355ea
  Author: Don Olmstead 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/cmake/OptionsPlayStation.cmake

  Log Message:
  ---
  [PlayStation] Enable variation fonts experimentally
https://bugs.webkit.org/show_bug.cgi?id=260987

Reviewed by Fujii Hironori.

Check the versions of freetype, harfbuzz and fontconfig to see if its
possible to turn on `ENABLE_VARIATION_FONTS`.

* Source/cmake/OptionsPlayStation.cmake:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 075685: [JSC] Reland wyhash in StringImpl for MacOS

2023-08-31 Thread Commit Queue
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 0756854fafd4312b8f0126de111792e39db106ef
  
https://github.com/WebKit/WebKit/commit/0756854fafd4312b8f0126de111792e39db106ef
  Author: Yijia Huang 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/JavaScriptCore/create_hash_table
M Source/JavaScriptCore/runtime/JSCBytecodeCacheVersion.cpp.in
M Source/JavaScriptCore/runtime/JSCell.cpp
M Source/JavaScriptCore/runtime/TemplateObjectDescriptor.h
M Source/JavaScriptCore/tools/JSDollarVM.cpp
M Source/JavaScriptCore/tools/VMInspector.cpp
M Source/JavaScriptCore/yarr/generateYarrUnicodePropertyTables.py
M Source/JavaScriptCore/yarr/hasher.py
M Source/WTF/WTF.xcodeproj/project.pbxproj
M Source/WTF/wtf/CMakeLists.txt
M Source/WTF/wtf/Hasher.h
M Source/WTF/wtf/text/CString.cpp
M Source/WTF/wtf/text/StringHash.h
M Source/WTF/wtf/text/StringHasher.h
A Source/WTF/wtf/text/StringHasherInlines.h
M Source/WTF/wtf/text/StringImpl.h
A Source/WTF/wtf/text/SuperFastHash.h
A Source/WTF/wtf/text/WYHash.h
M Source/WTF/wtf/unicode/UTF8Conversion.cpp
M Source/WebCore/bindings/scripts/CodeGeneratorJS.pm
M Source/WebCore/bindings/scripts/Hasher.pm
M Source/WebCore/bindings/scripts/test/JS/JSTestGlobalObject.cpp
M Source/WebCore/contentextensions/DFAMinimizer.cpp
M Source/WebCore/contentextensions/HashableActionList.h
M Source/WebCore/platform/SharedStringHash.cpp
M Source/WebCore/platform/graphics/WidthCache.h
M Tools/TestWebKitAPI/CMakeLists.txt
M Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj
M Tools/TestWebKitAPI/Tests/WTF/StringHasher.cpp
A Tools/TestWebKitAPI/Tests/WTF/SuperFastHash.cpp
A Tools/TestWebKitAPI/Tests/WTF/WYHash.cpp

  Log Message:
  ---
  [JSC] Reland wyhash in StringImpl for MacOS
https://bugs.webkit.org/show_bug.cgi?id=260957
rdar://114755168

Reviewed by Yusuke Suzuki.

Previously, we reverted the wyhash patch (landed at 266929@main and
reverted at 267004@main) which causes regressions on iOS devices.
This patch only enable wyhash for macOS.

* Source/JavaScriptCore/create_hash_table:
* Source/JavaScriptCore/runtime/JSCBytecodeCacheVersion.cpp.in:
* Source/JavaScriptCore/runtime/JSCell.cpp:
(JSC::reportZappedCellAndCrash):
* Source/JavaScriptCore/runtime/TemplateObjectDescriptor.h:
(JSC::TemplateObjectDescriptor::calculateHash):
* Source/JavaScriptCore/tools/JSDollarVM.cpp:
* Source/JavaScriptCore/tools/VMInspector.cpp:
(JSC::VMInspector::dumpSubspaceHashes):
* Source/JavaScriptCore/yarr/generateYarrUnicodePropertyTables.py:
(PropertyData.createAndDumpHashTable.createAndDumpHashTableHelper):
(PropertyData.createAndDumpHashTable):
* Source/JavaScriptCore/yarr/hasher.py:
(stringHash):
(avalancheBits):
(maskTop8BitsAndAvoidZero):
(superFastHash):
(wyhash):
(wyhash.add64):
(wyhash.multi64):
(wyhash.wymum):
(wyhash.wymix):
(wyhash.convert32BitTo64Bit):
(wyhash.convert16BitTo32Bit):
(wyhash.c2i):
(wyhash.wyr8):
(wyhash.wyr4):
(wyhash.wyr2):
(ceilingToPowerOf2):
(createHashTable):
(createHashTable.createHashTableHelper):
* Source/WTF/WTF.xcodeproj/project.pbxproj:
* Source/WTF/wtf/CMakeLists.txt:
* Source/WTF/wtf/Hasher.h:
* Source/WTF/wtf/text/CString.cpp:
(WTF::CString::hash const):
* Source/WTF/wtf/text/StringHash.h:
(WTF::ASCIICaseInsensitiveHash::hash):
* Source/WTF/wtf/text/StringHasher.h:
(WTF::StringHasher::finalize):
(WTF::StringHasher::finalizeAndMaskTop8Bits):
(WTF::StringHasher::addCharactersAssumingAligned): Deleted.
(WTF::StringHasher::addCharacter): Deleted.
(WTF::StringHasher::addCharacters): Deleted.
(WTF::StringHasher::hashWithTop8BitsMasked const): Deleted.
(WTF::StringHasher::hash const): Deleted.
(WTF::StringHasher::computeHashAndMaskTop8Bits): Deleted.
(WTF::StringHasher::computeHash): Deleted.
(WTF::StringHasher::computeLiteralHash): Deleted.
(WTF::StringHasher::computeLiteralHashAndMaskTop8Bits): Deleted.
(WTF::StringHasher::calculateWithRemainingLastCharacter): Deleted.
(WTF::StringHasher::calculateWithTwoCharacters): Deleted.
(WTF::StringHasher::computeHashImpl): Deleted.
(WTF::StringHasher::processPendingCharacter const): Deleted.
* Source/WTF/wtf/text/StringHasherInlines.h: Added.
(WTF::StringHasher::computeHashAndMaskTop8Bits):
(WTF::StringHasher::computeLiteralHashAndMaskTop8Bits):
(WTF::StringHasher::addCharacter):
(WTF::StringHasher::hashWithTop8BitsMasked):
* Source/WTF/wtf/text/StringImpl.h:
* Source/WTF/wtf/text/SuperFastHash.h: Copied from 
Source/WTF/wtf/text/StringHasher.h.
(WTF::SuperFastHash::addCharactersAssumingAligned):
(WTF::SuperFastHash::addCharacter):
(WTF::SuperFastHash::addCharacters):
(WTF::SuperFastHash::hashWithTop8BitsMasked const):
(WTF::SuperFastHash::hash const):
(WTF::SuperFastHash::computeHashAndMaskTop8Bits):
(WTF::SuperFastHash::computeHash):
(WTF::SuperFastHash::computeLiteralHash):
(WTF::SuperFastHash::computeLiteralHashAndMaskTop8Bits):
(WTF::SuperFastHash::addChara

[webkit-changes] [WebKit/WebKit] 41f7a3: [CSS] Always serialize implicit nesting selector/rule

2023-08-31 Thread Matthieu Dubet
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 41f7a301e412e75355904d8aa98393810cd91e3a
  
https://github.com/WebKit/WebKit/commit/41f7a301e412e75355904d8aa98393810cd91e3a
  Author: Matthieu Dubet 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M 
LayoutTests/imported/w3c/web-platform-tests/css/css-nesting/parsing-expected.txt
M LayoutTests/imported/w3c/web-platform-tests/css/css-nesting/parsing.html
M 
LayoutTests/imported/w3c/web-platform-tests/css/css-nesting/serialize-group-rules-with-decls-expected.txt
M 
LayoutTests/inspector/css/getMatchedStylesForNodeNestingStyleGrouping-expected.txt
M Source/WebCore/css/CSSGroupingRule.cpp
M Source/WebCore/css/SelectorChecker.cpp
M Source/WebCore/css/parser/CSSParserSelector.cpp
M Source/WebCore/css/parser/CSSParserSelector.h
M Source/WebCore/css/parser/CSSSelectorParser.cpp

  Log Message:
  ---
  [CSS] Always serialize implicit nesting selector/rule
https://bugs.webkit.org/show_bug.cgi?id=260707
rdar://112900363

Reviewed by Patrick Angle and Antti Koivisto.

https://drafts.csswg.org/css-nesting/#cssom

"When serializing a relative selector in a nested style rule,
the selector must be absolutized, with the implied nesting selector inserted.
When serializing a nested group rule, it must serialize solely with child 
rules."

This patch moves the code which build the selector with the implicit & added
from rule set building time to parsing time.

* 
LayoutTests/imported/w3c/web-platform-tests/css/css-nesting/parsing-expected.txt:
* LayoutTests/imported/w3c/web-platform-tests/css/css-nesting/parsing.html:
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-nesting/serialize-group-rules-with-decls-expected.txt:
* 
LayoutTests/inspector/css/getMatchedStylesForNodeNestingStyleGrouping-expected.txt:
* Source/WebCore/css/CSSGroupingRule.cpp:
(WebCore::CSSGroupingRule::cssTextForDeclsAndRules const):
* Source/WebCore/css/SelectorChecker.cpp:
(WebCore::SelectorChecker::checkOne const):
* Source/WebCore/css/parser/CSSParserSelector.cpp:
(WebCore::CSSParserSelector::hasExplicitNestingParent const):
* Source/WebCore/css/parser/CSSParserSelector.h:
(WebCore::CSSParserSelector::selector const):
* Source/WebCore/css/parser/CSSSelectorParser.cpp:
(WebCore::CSSSelectorParser::consumeNestedComplexSelector):
(WebCore::CSSSelectorParser::resolveNestingParent):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 3b90f5: [Gardening]: [ macOS ] media/video-orientation-can...

2023-08-31 Thread Ben
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 3b90f500d404d08cca15168eaa57fdf54dbebdbe
  
https://github.com/WebKit/WebKit/commit/3b90f500d404d08cca15168eaa57fdf54dbebdbe
  Author: Ben Schwartz 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M LayoutTests/platform/mac/TestExpectations

  Log Message:
  ---
  [Gardening]: [ macOS ] media/video-orientation-canvas.html is a flaky text 
failure.
rdar://114793358
https://bugs.webkit.org/show_bug.cgi?id=261000

Unreviewed test gardening.

* LayoutTests/platform/mac/TestExpectations:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 73f6c6: Do not allow container units in DOMMatrix() and DO...

2023-08-31 Thread Tim Nguyen
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 73f6c6d69991fded4847d7d8fdf9153859f3189d
  
https://github.com/WebKit/WebKit/commit/73f6c6d69991fded4847d7d8fdf9153859f3189d
  Author: Tim Nguyen 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M 
LayoutTests/imported/w3c/web-platform-tests/css/geometry/DOMMatrix-001-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/css/geometry/DOMMatrix-001.html
M Source/WebCore/css/CSSPrimitiveValueMappings.h

  Log Message:
  ---
  Do not allow container units in DOMMatrix() and DOMMatrixReadOnly()
https://bugs.webkit.org/show_bug.cgi?id=260988
rdar://114759860

Reviewed by Simon Fraser.

Prevent a nullptr deref by forbidding container unit usage in DOMMatrix().

* 
LayoutTests/imported/w3c/web-platform-tests/css/geometry/DOMMatrix-001-expected.txt:
* LayoutTests/imported/w3c/web-platform-tests/css/geometry/DOMMatrix-001.html:
* Source/WebCore/css/CSSPrimitiveValueMappings.h:
(WebCore::CSSPrimitiveValue::convertingToLengthHasRequiredConversionData const):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] dc3286: Deploy more smart pointers in Source/WebKit/UIProc...

2023-08-31 Thread Ryosuke Niwa
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: dc328646ce2fb251e968f41ac2c54f5dac601358
  
https://github.com/WebKit/WebKit/commit/dc328646ce2fb251e968f41ac2c54f5dac601358
  Author: Ryosuke Niwa 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebKit/UIProcess/Notifications/WebNotificationManagerProxy.cpp
M Source/WebKit/UIProcess/Notifications/WebNotificationProvider.cpp
M Source/WebKit/UIProcess/PDF/WKPDFHUDView.mm

  Log Message:
  ---
  Deploy more smart pointers in Source/WebKit/UIProcess/Notifications and 
Source/WebKit/UIProcess/PDF
https://bugs.webkit.org/show_bug.cgi?id=260952

Reviewed by Chris Dumez.

Deployed more smart pointers as found by clang static analyzer.

* Source/WebKit/UIProcess/Notifications/WebNotificationManagerProxy.cpp:
(WebKit::WebNotificationManagerProxy::sharedServiceWorkerManager):
(WebKit::WebNotificationManagerProxy::providerDidCloseNotifications):
* Source/WebKit/UIProcess/Notifications/WebNotificationProvider.cpp:
(WebKit::WebNotificationProvider::notificationPermissions):
* Source/WebKit/UIProcess/PDF/WKPDFHUDView.mm:
(-[WKPDFHUDView hitTest:]):
(-[WKPDFHUDView _performActionForControl:]):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 1b8732: [JSC] Use MonotonicTime for SamplingProfiler JSON ...

2023-08-31 Thread Yusuke Suzuki
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 1b8732286c537bc76aac6dd482134c7029aefbf8
  
https://github.com/WebKit/WebKit/commit/1b8732286c537bc76aac6dd482134c7029aefbf8
  Author: Yusuke Suzuki 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/JavaScriptCore/inspector/agents/InspectorScriptProfilerAgent.cpp
M Source/JavaScriptCore/runtime/SamplingProfiler.cpp
M Source/JavaScriptCore/runtime/SamplingProfiler.h
M Source/WTF/wtf/Stopwatch.h

  Log Message:
  ---
  [JSC] Use MonotonicTime for SamplingProfiler JSON dump timestamp
https://bugs.webkit.org/show_bug.cgi?id=260998
rdar://114791556

Reviewed by Justin Michaud.

Replace timestamp of SamplingProfiler JSON dump from Stopwatch's elapsedTime
to MonotonicTime to easily sync it with the system wide samples.

* Source/JavaScriptCore/inspector/agents/InspectorScriptProfilerAgent.cpp:
(Inspector::buildSamples):
(Inspector::InspectorScriptProfilerAgent::trackingComplete):
* Source/JavaScriptCore/runtime/SamplingProfiler.cpp:
(JSC::SamplingProfiler::timerLoop):
(JSC::SamplingProfiler::takeSample):
(JSC::SamplingProfiler::processUnverifiedStackTraces):
(JSC::SamplingProfiler::noticeVMEntry):
(JSC::SamplingProfiler::stackTracesAsJSON):
* Source/JavaScriptCore/runtime/SamplingProfiler.h:
* Source/WTF/wtf/Stopwatch.h:
(WTF::Stopwatch::elapsedTime const):
(WTF::Stopwatch::elapsedTimeAndTimestamp const):
(WTF::Stopwatch::create): Deleted.
(WTF::Stopwatch::isActive const): Deleted.
(WTF::Stopwatch::Stopwatch): Deleted.

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] b3a4a4: [JSC] LLInt slow path of op_iterator_next doesn't ...

2023-08-31 Thread Commit Queue
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b3a4a487b56ade502613ebf7e0136f35ba68fa84
  
https://github.com/WebKit/WebKit/commit/b3a4a487b56ade502613ebf7e0136f35ba68fa84
  Author: Alexey Shvayka 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
A JSTests/stress/regress-260980.js
M Source/JavaScriptCore/llint/LLIntSlowPaths.cpp

  Log Message:
  ---
  [JSC] LLInt slow path of op_iterator_next doesn't always check m_done value
https://bugs.webkit.org/show_bug.cgi?id=260980


Reviewed by Mark Lam.

Before this change, if `iteratorResult.done` wasn't of boolean type, its value 
wasn't checked.
This patch adds ToBoolean coercion and the missing check, aligning JSC with V8 
and SpiderMonkey.

* Source/JavaScriptCore/llint/LLIntSlowPaths.cpp:
(JSC::LLInt::LLINT_SLOW_PATH_DECL):
* JSTests/stress/regress-260980.js: Added.

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 8ec729: Sync css/css-contain from WPT

2023-08-31 Thread Rob Buis
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 8ec72902380925a43ae13e914d08fe8386d43b8d
  
https://github.com/WebKit/WebKit/commit/8ec72902380925a43ae13e914d08fe8386d43b8d
  Author: Rob Buis 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-085.html
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-086-expected.txt
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-086.html
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-088-expected.txt
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-088.html
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-089-expected.txt
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-089.html
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-090-expected.txt
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-090.html
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-svg-path-expected.txt
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-svg-path.html
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-svg-rect-expected.txt
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-svg-rect.html
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-svg-text-expected.txt
A 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-svg-text.html
M 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/w3c-import.log
M 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/counter-scoping-001.html

  Log Message:
  ---
  Sync css/css-contain from WPT
https://bugs.webkit.org/show_bug.cgi?id=260992

Reviewed by Tim Nguyen.

>From WPT revision 3d46d558a0.

* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-085.html:
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-086-expected.txt:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-086.html:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-088-expected.txt:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-088.html:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-089-expected.txt:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-089.html:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-090-expected.txt:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-090.html:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-svg-path-expected.txt:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-svg-path.html:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-svg-rect-expected.txt:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-svg-rect.html:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-svg-text-expected.txt:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/content-visibility-svg-text.html:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/content-visibility/w3c-import.log:
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-contain/counter-scoping-001.html:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] c377f2: [motion-path] Update "" implementation ...

2023-08-31 Thread Nikos Mouchtaris
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c377f2da36eb80d46bc851a8090f020bab79752f
  
https://github.com/WebKit/WebKit/commit/c377f2da36eb80d46bc851a8090f020bab79752f
  Author: Nikolaos Mouchtaris 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M LayoutTests/TestExpectations
M Source/WebCore/rendering/MotionPath.cpp
M Source/WebCore/rendering/MotionPath.h
M Source/WebCore/rendering/PathOperation.h
M Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in

  Log Message:
  ---
  [motion-path] Update "" implementation for offset-path
https://bugs.webkit.org/show_bug.cgi?id=260944
rdar://110938788

Reviewed by Tim Nguyen.

Update coord-box to use the containing block's bounding rect if it
 exists. We also need to take into account the containing block's
border radius.

* LayoutTests/TestExpectations:
* Source/WebCore/rendering/MotionPath.cpp:
(WebCore::offsetFromContainer):
(WebCore::MotionPath::motionPathDataForRenderer):
* Source/WebCore/rendering/PathOperation.cpp:
(WebCore::BoxPathOperation::getPath const):
* Source/WebCore/rendering/PathOperation.h:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] e2aa0c: Use memcpy in SVGPathByteStreamSource::readType

2023-08-31 Thread Chris Dumez
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: e2aa0c20fc47b29231c6e5413347d827c36d4186
  
https://github.com/WebKit/WebKit/commit/e2aa0c20fc47b29231c6e5413347d827c36d4186
  Author: Chris Dumez 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebCore/svg/SVGPathByteStream.h
M Source/WebCore/svg/SVGPathByteStreamBuilder.cpp
M Source/WebCore/svg/SVGPathByteStreamBuilder.h
M Source/WebCore/svg/SVGPathByteStreamSource.h

  Log Message:
  ---
  Use memcpy in SVGPathByteStreamSource::readType
https://bugs.webkit.org/show_bug.cgi?id=250676
rdar://104552389

Reviewed by Ryosuke Niwa.

Use memcpy in SVGPathByteStreamSource::readType and simplify the
SVGPathByteStreamBuilder::writeType() logic.

* Source/WebCore/svg/SVGPathByteStream.h:
(WebCore::SVGPathByteStream::append):
(): Deleted.
* Source/WebCore/svg/SVGPathByteStreamBuilder.cpp:
(WebCore::SVGPathByteStreamBuilder::moveTo):
(WebCore::SVGPathByteStreamBuilder::lineTo):
(WebCore::SVGPathByteStreamBuilder::lineToHorizontal):
(WebCore::SVGPathByteStreamBuilder::lineToVertical):
(WebCore::SVGPathByteStreamBuilder::curveToCubic):
(WebCore::SVGPathByteStreamBuilder::curveToCubicSmooth):
(WebCore::SVGPathByteStreamBuilder::curveToQuadratic):
(WebCore::SVGPathByteStreamBuilder::curveToQuadraticSmooth):
(WebCore::SVGPathByteStreamBuilder::arcTo):
(WebCore::SVGPathByteStreamBuilder::closePath):
* Source/WebCore/svg/SVGPathByteStreamBuilder.h:
* Source/WebCore/svg/SVGPathByteStreamSource.h:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 589a9d: Add support for Web Extension windows APIs.

2023-08-31 Thread Timothy Hatcher
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 589a9da68d4ea19dffeee69c65375b872cca7c1b
  
https://github.com/WebKit/WebKit/commit/589a9da68d4ea19dffeee69c65375b872cca7c1b
  Author: Timothy Hatcher 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebKit/DerivedSources-input.xcfilelist
M Source/WebKit/DerivedSources.make
M Source/WebKit/Scripts/webkit/messages.py
M Source/WebKit/Scripts/webkit/tests/MessageArgumentDescriptions.cpp
M 
Source/WebKit/Shared/Extensions/WebExtensionEventListenerType.serialization.in
A Source/WebKit/Shared/Extensions/WebExtensionTab.serialization.in
M Source/WebKit/Shared/Extensions/WebExtensionTabIdentifier.h
A Source/WebKit/Shared/Extensions/WebExtensionTabParameters.h
A Source/WebKit/Shared/Extensions/WebExtensionWindow.serialization.in
M Source/WebKit/Shared/Extensions/WebExtensionWindowIdentifier.h
A Source/WebKit/Shared/Extensions/WebExtensionWindowParameters.h
M Source/WebKit/Shared/Extensions/_WKWebExtensionUtilities.mm
M 
Source/WebKit/UIProcess/API/Cocoa/_WKWebExtensionTabCreationOptionsInternal.h
M Source/WebKit/UIProcess/API/Cocoa/_WKWebExtensionWindow.h
M Source/WebKit/UIProcess/API/Cocoa/_WKWebExtensionWindowCreationOptions.h
M 
Source/WebKit/UIProcess/API/Cocoa/_WKWebExtensionWindowCreationOptionsInternal.h
A 
Source/WebKit/UIProcess/Extensions/Cocoa/API/WebExtensionContextAPIWindowsCocoa.mm
M Source/WebKit/UIProcess/Extensions/Cocoa/WebExtensionContextCocoa.mm
M Source/WebKit/UIProcess/Extensions/Cocoa/WebExtensionTabCocoa.mm
M Source/WebKit/UIProcess/Extensions/Cocoa/WebExtensionWindowCocoa.mm
M Source/WebKit/UIProcess/Extensions/WebExtensionContext.h
M Source/WebKit/UIProcess/Extensions/WebExtensionContext.messages.in
M Source/WebKit/UIProcess/Extensions/WebExtensionTab.h
M Source/WebKit/UIProcess/Extensions/WebExtensionWindow.h
M Source/WebKit/WebKit.xcodeproj/project.pbxproj
M Source/WebKit/WebProcess/Extensions/API/Cocoa/WebExtensionAPITabsCocoa.mm
M 
Source/WebKit/WebProcess/Extensions/API/Cocoa/WebExtensionAPIWindowsCocoa.mm
M 
Source/WebKit/WebProcess/Extensions/API/Cocoa/WebExtensionAPIWindowsEventCocoa.mm
M Source/WebKit/WebProcess/Extensions/API/WebExtensionAPITabs.h
M Source/WebKit/WebProcess/Extensions/API/WebExtensionAPIWindows.h
M Source/WebKit/WebProcess/Extensions/API/WebExtensionAPIWindowsEvent.h
M Source/WebKit/WebProcess/Extensions/Interfaces/WebExtensionAPIWindows.idl
M Tools/TestWebKitAPI/SourcesCocoa.txt
M Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj
A Tools/TestWebKitAPI/Tests/WebKitCocoa/WKWebExtensionAPIWindows.mm
M Tools/TestWebKitAPI/Tests/WebKitCocoa/WKWebExtensionTab.mm
M Tools/TestWebKitAPI/Tests/WebKitCocoa/WKWebExtensionWindow.mm
M Tools/TestWebKitAPI/cocoa/TestWebExtensionsDelegate.h
M Tools/TestWebKitAPI/cocoa/TestWebExtensionsDelegate.mm
M Tools/TestWebKitAPI/cocoa/WebExtensionUtilities.h
M Tools/TestWebKitAPI/cocoa/WebExtensionUtilities.mm

  Log Message:
  ---
  Add support for Web Extension windows APIs.
https://bugs.webkit.org/show_bug.cgi?id=260155
rdar://problem/114736173

Reviewed by Brian Weinstein.

Fully plumbs the windows API from JavaScript up to the Objective-C API.

* Source/WebKit/DerivedSources-input.xcfilelist: Addeed new files.
* Source/WebKit/DerivedSources.make: Added new files.
* Source/WebKit/Scripts/webkit/messages.py: Added new extension types.
(serialized_identifiers):
(types_that_cannot_be_forward_declared):
* Source/WebKit/Scripts/webkit/tests/MessageArgumentDescriptions.cpp: Updated 
with make command mentioned in messages.py comment.
(IPC::serializedIdentifiers):
* 
Source/WebKit/Shared/Extensions/WebExtensionEventListenerType.serialization.in:
* Source/WebKit/Shared/Extensions/WebExtensionTab.serialization.in: Added.
* Source/WebKit/Shared/Extensions/WebExtensionTabIdentifier.h:
(WebKit::isNone): Added.
(WebKit::isValid): Added.
(WebKit::toWebExtensionTabIdentifier): Added.
(WebKit::toWebAPI): Added.
* Source/WebKit/Shared/Extensions/WebExtensionTabParameters.h: Copied from 
Source/WebKit/UIProcess/API/Cocoa/_WKWebExtensionWindowCreationOptionsInternal.h.
* Source/WebKit/Shared/Extensions/WebExtensionWindow.serialization.in: Added.
* Source/WebKit/Shared/Extensions/WebExtensionWindowIdentifier.h:
(WebKit::isNone): Added.
(WebKit::isCurrent): Added.
(WebKit::isValid): Added.
(WebKit::toWebExtensionWindowIdentifier): Added.
(WebKit::toWebAPI): Added.
* Source/WebKit/Shared/Extensions/WebExtensionWindowParameters.h: Copied from 
Source/WebKit/Shared/Extensions/WebExtensionTabIdentifier.h.
* Source/WebKit/Shared/Extensions/_WKWebExtensionUtilities.mm:
(classToClassString): Use dot-syntax.
(+[_WKWebExtensionUtilities 
validateContentsOfDictionary:requiredKeys:optionalKeys:keyToExpectedValueType:outExceptionString:]):
Added support for NSSet input, where any class or 

[webkit-changes] [WebKit/WebKit] f6b1fa: Use generator to set CMAKE_EXPORT_COMPILE_COMMANDS

2023-08-31 Thread Don Olmstead
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: f6b1fa7647c7e2b2f4eefbb45c79dbd27d5479ab
  
https://github.com/WebKit/WebKit/commit/f6b1fa7647c7e2b2f4eefbb45c79dbd27d5479ab
  Author: Don Olmstead 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/cmake/WebKitCommon.cmake
M Tools/Scripts/webkitdirs.pm

  Log Message:
  ---
  Use generator to set CMAKE_EXPORT_COMPILE_COMMANDS
https://bugs.webkit.org/show_bug.cgi?id=260666

Reviewed by Michael Catanzaro.

In 266947@main a target `RewriteCompileCommands` was added for unified
builds where `CMAKE_EXPORT_COMPILE_COMMANDS` is `ON`. However this
is only valid for Makefile and Ninja generators. On Visual Studio it
is ignored so the target will always fail to build which was found out
because its always set to `ON` within `build-webkit`.

Move the logic for turning it `ON` into the CMake code and base it on
the `CMAKE_GENERATOR`.

* Source/cmake/WebKitCommon.cmake
* Tools/Scripts/webkitdirs.pm

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] ff53b2: [JSC] Refactor and restrict IC operations are only...

2023-08-31 Thread Yusuke Suzuki
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ff53b2cbf74de112c510f533ab39366ca1366606
  
https://github.com/WebKit/WebKit/commit/ff53b2cbf74de112c510f533ab39366ca1366606
  Author: Yusuke Suzuki 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/JavaScriptCore/bytecode/Repatch.cpp
M Source/JavaScriptCore/bytecode/StructureStubInfo.h
M Source/JavaScriptCore/dfg/DFGOperations.cpp
M Source/JavaScriptCore/dfg/DFGOperations.h
M Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp
M Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp
M Source/JavaScriptCore/dfg/DFGSpeculativeJIT64.cpp
M Source/JavaScriptCore/ftl/FTLLowerDFGToB3.cpp
M Source/JavaScriptCore/jit/JITInlineCacheGenerator.cpp
M Source/JavaScriptCore/jit/JITInlineCacheGenerator.h
M Source/JavaScriptCore/jit/JITOperations.cpp
M Source/JavaScriptCore/jit/JITOperations.h

  Log Message:
  ---
  [JSC] Refactor and restrict IC operations are only called from IC
https://bugs.webkit.org/show_bug.cgi?id=260937
rdar://114736570

Reviewed by Keith Miller and Justin Michaud.

In this patch, we refactor operations used by IC and others. The goal of this 
patch is that we ensure that
IC operations are called only from IC. We would like to make these operations 
assuming more, like, it always
has StructureStubInfo so that it does not need to set CallSiteIndex in the 
caller side in the future.
This is really nice since we can remove get_by_id's Slow path in Baseline JIT 
code for example later, and
making slow path of get_by_id one shared one, this will be set up later for 
handler IC. So, handler IC can
just always call this slow path handler which calls operations, and operations 
are aware that they are called
from IC, and CallSiteIndex etc. are appropriately set up from passed 
StructureStubInfo*. So we can make this
entrance similar to the fast path IC, which removes a lot of weird stub code 
necessary for handler IC.

We also resolve many naming inconsistencies in these operations. We have solid 
rule now about Optimize, Generic,
Megamorphic, MegamorphicGeneric, and GaveUp suffixes.

* Source/JavaScriptCore/bytecode/Repatch.cpp:
(JSC::appropriateGetByOptimizeFunction):
(JSC::appropriateGetByGaveUpFunction):
(JSC::tryCacheGetBy):
(JSC::repatchGetBy):
(JSC::repatchGetBySlowPathCall):
(JSC::repatchArrayGetByVal):
(JSC::appropriatePutByGaveUpFunction):
(JSC::repatchPutBySlowPathCall):
(JSC::appropriatePutByOptimizeFunction):
(JSC::tryCachePutBy):
(JSC::repatchPutBy):
(JSC::repatchArrayPutByVal):
(JSC::repatchDeleteBy):
(JSC::appropriateInByOptimizeFunction):
(JSC::appropriateInByGaveUpFunction):
(JSC::repatchInBy):
(JSC::repatchHasPrivateBrand):
(JSC::repatchCheckPrivateBrand):
(JSC::repatchSetPrivateBrand):
(JSC::repatchInstanceOf):
(JSC::resetGetBy):
(JSC::resetInBy):
(JSC::appropriateOptimizingGetByFunction): Deleted.
(JSC::appropriateGetByFunction): Deleted.
(JSC::appropriateGenericPutByFunction): Deleted.
(JSC::appropriateOptimizingPutByFunction): Deleted.
(JSC::appropriateOptimizingInByFunction): Deleted.
(JSC::appropriateGenericInByFunction): Deleted.
* Source/JavaScriptCore/bytecode/StructureStubInfo.h:
(JSC::appropriateGetByIdOptimizeFunction):
(JSC::appropriateGetByIdGenericFunction):
(JSC::appropriatePutByIdOptimizeFunction):
(JSC::appropriateOptimizingGetByIdFunction): Deleted.
(JSC::appropriateGenericGetByIdFunction): Deleted.
* Source/JavaScriptCore/dfg/DFGOperations.cpp:
(JSC::DFG::putByVal): Deleted.
(JSC::DFG::putByValInternal): Deleted.
* Source/JavaScriptCore/dfg/DFGOperations.h:
* Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compileDeleteById):
(JSC::DFG::SpeculativeJIT::compileDeleteByVal):
(JSC::DFG::SpeculativeJIT::compilePutByVal):
(JSC::DFG::SpeculativeJIT::compilePutByValForIntTypedArray):
* Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::cachedGetById):
(JSC::DFG::SpeculativeJIT::compileGetByVal):
(JSC::DFG::SpeculativeJIT::compileGetByValWithThis):
* Source/JavaScriptCore/dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::cachedGetById):
(JSC::DFG::SpeculativeJIT::compileGetByVal):
(JSC::DFG::SpeculativeJIT::compileGetByIdMegamorphic):
(JSC::DFG::SpeculativeJIT::compileGetByIdWithThisMegamorphic):
(JSC::DFG::SpeculativeJIT::compileGetByValMegamorphic):
(JSC::DFG::SpeculativeJIT::compileGetByValWithThisMegamorphic):
(JSC::DFG::SpeculativeJIT::compilePutByIdMegamorphic):
(JSC::DFG::SpeculativeJIT::compilePutByValMegamorphic):
* Source/JavaScriptCore/ftl/FTLLowerDFGToB3.cpp:
(JSC::FTL::DFG::LowerDFGToB3::compileGetById):
(JSC::FTL::DFG::LowerDFGToB3::compileGetByIdMegamorphic):
(JSC::FTL::DFG::LowerDFGToB3::compileGetByValMegamorphic):
(JSC::FTL::DFG::LowerDFGToB3::compileGetByIdWithThisMegamorphic):
(JSC::FTL::DFG::LowerDFGToB3::compileGetByValWithThisMegamorphic):
(JSC::FTL::DFG::LowerDFGToB3::compileGetPrivateName):
(JSC::FTL::DFG::LowerDFGToB3::compilePutByValMega

[webkit-changes] [WebKit/WebKit] 53a54c: [CMake] Turn off pkg-config on PlayStation

2023-08-31 Thread Don Olmstead
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 53a54cd39755b3ea25db0a1064f102f920f16236
  
https://github.com/WebKit/WebKit/commit/53a54cd39755b3ea25db0a1064f102f920f16236
  Author: Don Olmstead 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/cmake/OptionsPlayStation.cmake

  Log Message:
  ---
  [CMake] Turn off pkg-config on PlayStation
https://bugs.webkit.org/show_bug.cgi?id=260983

Reviewed by Michael Catanzaro.

The CMake find modules use pkg-config to get hints for a library's
location. With a Windows host it can pick up .pc files in the perl
install accidentally. PlayStation isn't using .pc files so just turn it
completely off.

Prevent the find module from running and override `pkg_check_modules`
so it doesn't return anything.

* Source/cmake/OptionsPlayStation.cmake:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] c379c0: Cleanup code for shorthand parsing and font-varian...

2023-08-31 Thread Tim Nguyen
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c379c056f604bb93b731e512c76d7011dca13096
  
https://github.com/WebKit/WebKit/commit/c379c056f604bb93b731e512c76d7011dca13096
  Author: Tim Nguyen 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebCore/css/ComputedStyleExtractor.cpp
M Source/WebCore/css/parser/CSSPropertyParser.cpp

  Log Message:
  ---
  Cleanup code for shorthand parsing and font-variant computed style
https://bugs.webkit.org/show_bug.cgi?id=260985
rdar://114780529

Reviewed by Myles C. Maxfield.

* Source/WebCore/css/ComputedStyleExtractor.cpp:
(WebCore::ComputedStyleExtractor::valueForPropertyInStyle const):
(WebCore::fontVariantPositionPropertyValue): Deleted.
(WebCore::fontVariantCapsPropertyValue): Deleted.

The toCSSValueID methods in CSSPrimitiveValueMappings.h allow us to remove this 
redundant code.

* Source/WebCore/css/parser/CSSPropertyParser.cpp:
(WebCore::CSSPropertyParser::consumeFontVariantShorthand):
(WebCore::CSSPropertyParser::consumeListStyleShorthand):

This is consistent with style used for the columns property.

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] fab917: Reduce the amount of SVGElement::updateRelativeLen...

2023-08-31 Thread Chris Dumez
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: fab91723ea6b142820bc5c7186672e122391f272
  
https://github.com/WebKit/WebKit/commit/fab91723ea6b142820bc5c7186672e122391f272
  Author: Chris Dumez 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebCore/svg/SVGElement.cpp

  Log Message:
  ---
  Reduce the amount of SVGElement::updateRelativeLengthsInformation() calls
https://bugs.webkit.org/show_bug.cgi?id=260971

Reviewed by Ryosuke Niwa.

Reduce the amount of SVGElement::updateRelativeLengthsInformation() calls.

In SVGElement::removedFromAncestor(), if `removalType.disconnectedFromDocument`
was true, we would call `updateRelativeLengthsInformation(false, *this)` to 
remove
the element from m_elementsWithRelativeLengths and from our ancestor's map.

However, when SVGElement::removedFromAncestor() gets called, we know that
the element either doesn't have a parent or that it's parent is not connected
(I added an assertion to this effect). As a result, recursively calling
`updateRelativeLengthsInformation(false, element)` on the ancestors had no
effect, since the function would early return at the `!isConnected()` check.

As a result, it is sufficient to clear m_elementsWithRelativeLengths in
SVGElement::removedFromAncestor(), without calling
`updateRelativeLengthsInformation(false, *this)` or worrying about the
ancestors.

* Source/WebCore/svg/SVGElement.cpp:
(WebCore::SVGElement::removedFromAncestor):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 3b214f: Migrate macOS Sonoma test expectations to OpenSour...

2023-08-31 Thread Commit Queue
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 3b214fee9054bb69614243f6c6a2ea66a26f354f
  
https://github.com/WebKit/WebKit/commit/3b214fee9054bb69614243f6c6a2ea66a26f354f
  Author: Dawn Flores 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M LayoutTests/platform/mac-monterey-wk1/TestExpectations
A 
LayoutTests/platform/mac-monterey-wk1/editing/inserting/caret-position-expected.txt
A 
LayoutTests/platform/mac-monterey-wk1/editing/mac/input/caret-primary-bidi-expected.txt
A 
LayoutTests/platform/mac-monterey-wk1/editing/mac/input/firstrectforcharacterrange-plain-expected.txt
A 
LayoutTests/platform/mac-monterey-wk1/editing/mac/input/firstrectforcharacterrange-styled-expected.txt
A 
LayoutTests/platform/mac-monterey-wk1/editing/selection/5825350-2-expected.txt
A 
LayoutTests/platform/mac-monterey-wk1/editing/selection/mixed-editability-10-expected.txt
A 
LayoutTests/platform/mac-monterey-wk1/media/media-can-play-mpeg4-video-expected.txt
A 
LayoutTests/platform/mac-monterey-wk1/media/video-source-type-params-expected.txt
M LayoutTests/platform/mac-monterey-wk2/TestExpectations
A 
LayoutTests/platform/mac-monterey-wk2/composting/backing/inline-block-no-backing-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/composting/backing/whitespace-nodes-no-backing-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/composting/fixed-image-loading-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/composting/tilling/huge-layer-img-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/composting/visibility/visibility-image-layers-dynamic-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/editing/caret/caret-at-the-edge-of-contenteditable-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/imported/w3c/web-platform-tests/fetch/content-length/api-and-duplicate-headers.any-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/imported/w3c/web-platform-tests/fetch/content-length/api-and-duplicate-headers.any.worker-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/imported/w3c/web-platform-tests/fetch/range/non-matching-range-response-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/imported/w3c/web-platform-tests/mimesniff/media/media-sniff.window-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/media/media-can-play-mpeg4-video-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/media/video-source-type-params-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/scrollingcoordinator/mac/fixed-backgrounds/fixed-background-in-nested-non-cb-overflow-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/scrollingcoordinator/mac/fixed-backgrounds/fixed-background-in-nested-overflow-expected.txt
A 
LayoutTests/platform/mac-monterey-wk2/scrollingcoordinator/mac/fixed-backgrounds/fixed-background-in-nested-overflow2-expected.txt
M LayoutTests/platform/mac-monterey/TestExpectations
A 
LayoutTests/platform/mac-monterey/css1/box_properties/acid_test-expected.txt
A 
LayoutTests/platform/mac-monterey/css2.1/t09-c5526c-display-00-e-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/caret/insert-paragraph-does-not-paint-stale-carets-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/input/caret-at-the-edge-of-contenteditable-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/input/caret-at-the-edge-of-input-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/input/editable-container-with-word-wrap-normal-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/inserting/before-after-input-element-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/inserting/multiple-lines-selected-expected.txt
A LayoutTests/platform/mac-monterey/editing/pasteboard/4806874-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/pasteboard/input-field-1-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/pasteboard/interchange-newline-2-expected.txt
A LayoutTests/platform/mac-monterey/editing/selection/3690703-2-expected.txt
A LayoutTests/platform/mac-monterey/editing/selection/3690703-expected.txt
A LayoutTests/platform/mac-monterey/editing/selection/3690719-expected.txt
A LayoutTests/platform/mac-monterey/editing/selection/4895428-3-expected.txt
A LayoutTests/platform/mac-monterey/editing/selection/4975120-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/selection/block-cursor-overtype-mode-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/selection/caret-in-empty-inline-1-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/selection/caret-in-empty-inline-2-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/selection/drag-select-1-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/selection/internal-caret-rect-expected.txt
A 
LayoutTests/platform/mac-monterey/editing/selection/select-from-textfield-outwards-expect

[webkit-changes] [WebKit/WebKit] 7d6cd5: [GStreamer] Video not resized with playbin3 on i.M...

2023-08-31 Thread Commit Queue
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 7d6cd5e4b26870a15004fe75d6d163bdd3539a0d
  
https://github.com/WebKit/WebKit/commit/7d6cd5e4b26870a15004fe75d6d163bdd3539a0d
  Author: Marek Vasut 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp

  Log Message:
  ---
  [GStreamer] Video not resized with playbin3 on i.MX8M Plus and COG
https://bugs.webkit.org/show_bug.cgi?id=260482

Reviewed by Philippe Normand.

When using WEBKIT_GST_USE_PLAYBIN3=1 and cog to open h264 video (e.g.
Big Buck Bunny h264 1920x1080 mp4), then only single message is delivered
to MediaPlayerPrivateGStreamer::handleStreamCollectionMessage() . The
current workaround check would bail out since the message source is
decodebin3, while the m_source is filesrc . This prevents player->updateTracks
from being called, and thus m_hasVideo from being set, and thus
hasVideo() in MediaPlayerPrivateGStreamer::naturalSize() returns false
and MediaPlayerPrivateGStreamer::naturalSize returns FloatSize()
size, which is 0x0. The resulting video element in the browser has
minimum height set and is not correctly resized.

Limit the MediaPlayerPrivateGStreamer::handleStreamCollectionMessage()
workaround only to non-filesrc, since filesrc is not generating the
stream-collection events.

* Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
(WebCore::MediaPlayerPrivateGStreamer::handleStreamCollectionMessage):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] f3e388: [WPE] IFC : fast/text/whitespace/nowrap-clear-floa...

2023-08-31 Thread Amanda Falke
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: f3e388e446c42ff7578a5725c2a58705481fe546
  
https://github.com/WebKit/WebKit/commit/f3e388e446c42ff7578a5725c2a58705481fe546
  Author: Amanda Falke 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
A 
LayoutTests/platform/glib/fast/text/whitespace/nowrap-clear-float-expected.png
A 
LayoutTests/platform/glib/fast/text/whitespace/nowrap-clear-float-expected.txt
R 
LayoutTests/platform/gtk/fast/text/whitespace/nowrap-clear-float-expected.png
R 
LayoutTests/platform/gtk/fast/text/whitespace/nowrap-clear-float-expected.txt
R 
LayoutTests/platform/wpe/fast/text/whitespace/nowrap-clear-float-expected.txt

  Log Message:
  ---
  [WPE] IFC : fast/text/whitespace/nowrap-clear-float.html passes unexpectedly

Unreviewed test gardening.

In 254439@main when IFC intrusive floats were introduced, webkit.org/b/245080
was filed, marking failing expectations for all ports for this test. Since
then, the expectations were updated for WPE only in large meta-gardening patches
very recently, but this test in WPE should continue to fail, as in GTK
and in all other ports, until webkit.org/b/245080 is addressed wrt
line wrapping and intrusive floats.

Removed WPE expectations, and moved GTK expectations and png up to GLib
dir; now WPE fails (as GTK, and all other ports do), as expected.

* 
LayoutTests/platform/glib/fast/text/whitespace/nowrap-clear-float-expected.png: 
Renamed from 
LayoutTests/platform/gtk/fast/text/whitespace/nowrap-clear-float-expected.png.
* 
LayoutTests/platform/glib/fast/text/whitespace/nowrap-clear-float-expected.txt: 
Renamed from 
LayoutTests/platform/gtk/fast/text/whitespace/nowrap-clear-float-expected.txt.
* 
LayoutTests/platform/wpe/fast/text/whitespace/nowrap-clear-float-expected.txt: 
Removed.

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] b680f8: Add general quirk for canvas2d fingerprinting

2023-08-31 Thread Matthew Finkel
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b680f873f60959c6077d9a9297ab933a35d7bbf4
  
https://github.com/WebKit/WebKit/commit/b680f873f60959c6077d9a9297ab933a35d7bbf4
  Author: Matthew Finkel 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml
M Source/WebCore/html/HTMLCanvasElement.cpp
M Source/WebCore/page/Quirks.cpp
M Source/WebCore/page/Quirks.h
M Source/WebCore/platform/Logging.h
M Tools/TestWebKitAPI/Tests/WebKit/AdvancedPrivacyProtections.mm

  Log Message:
  ---
  Add general quirk for canvas2d fingerprinting
https://bugs.webkit.org/show_bug.cgi?id=260842
rdar://113039941

Reviewed by Wenson Hsieh.

In 266400@main I added a quirk for canvas fingerprinting that targeted specific
pages on two specific domains. As a follow up to that, this patch expands the
application of that quirk broadly by comparing attributes of the script instead
of looking at the specific document URL. This step is important because we know
protections against canvas fingerprinting cause breakage on some sites, and
there is likely more greater impact than we're aware at this time.

Similar to the current behavior, this patch relies on matching the last drawn
text string. In addition, it compares the dimensions of the canvas and the size
of the script. These characteristics seem sufficiently descriptive that we can
safely use them for identifying the target script.

This patch also introduces the CanvasFingerprintingQuirkEnabled preference for
controlling whether the quirk is available. It also adapts the existing API
test.

* Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml:
* Source/WebCore/html/HTMLCanvasElement.cpp:
(WebCore::HTMLCanvasElement::toDataURL):
* Source/WebCore/page/Quirks.cpp:
(WebCore::Quirks::advancedPrivacyProtectionSubstituteDataURLForScriptWithFeatures
 const):
(WebCore::Quirks::shouldEnableCanvas2DAdvancedPrivacyProtectionQuirk const): 
Deleted.
(WebCore::Quirks::advancedPrivacyProtectionSubstituteDataURLForText const): 
Deleted.
* Source/WebCore/page/Quirks.h:
* Source/WebCore/platform/Logging.h:
* Tools/TestWebKitAPI/Tests/WebKit/AdvancedPrivacyProtections.mm:
(TestWebKitAPI::TEST):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 32a30d: Let the compiler generate more comparison operator...

2023-08-31 Thread Chris Dumez
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 32a30dfe6ffb0b6380bca678439c115b60da5be9
  
https://github.com/WebKit/WebKit/commit/32a30dfe6ffb0b6380bca678439c115b60da5be9
  Author: Chris Dumez 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebCore/Modules/highlight/AppHighlightRangeData.h
M Source/WebCore/Modules/indexeddb/IDBDatabaseIdentifier.h
M Source/WebCore/Modules/indexeddb/shared/IDBResourceIdentifier.h
M Source/WebCore/Modules/mediasession/MediaPositionState.h
M Source/WebCore/Modules/permissions/PermissionDescriptor.h
M Source/WebCore/Modules/webaudio/BaseAudioContext.h
M Source/WebCore/PAL/pal/SessionID.h
M Source/WebCore/contentextensions/ContentExtensionActions.cpp
M Source/WebCore/contentextensions/ContentExtensionActions.h
M Source/WebCore/contentextensions/ContentExtensionRule.h
M Source/WebCore/contentextensions/NFAToDFA.cpp
M Source/WebCore/contentextensions/Term.h
M Source/WebCore/css/CSSCounterStyleDescriptors.h
M Source/WebCore/css/CSSCounterStyleRegistry.cpp
M Source/WebCore/css/CSSCustomPropertyValue.h
M Source/WebCore/css/CSSFontFace.h
M Source/WebCore/css/CSSGradientValue.cpp
M Source/WebCore/css/CSSGradientValue.h

  Log Message:
  ---
  Let the compiler generate more comparison operators in WebCore
https://bugs.webkit.org/show_bug.cgi?id=260943

Reviewed by Timothy Hatcher, Ryosuke Niwa and Darin Adler.

Let the compiler generate more comparison operators in WebCore now that we
support C++20.

* Source/WebCore/Modules/highlight/AppHighlightRangeData.h:
(WebCore::AppHighlightRangeData::NodePathComponent::operator== const): Deleted.
* Source/WebCore/Modules/indexeddb/IDBDatabaseIdentifier.h:
(WebCore::IDBDatabaseIdentifier::operator== const): Deleted.
* Source/WebCore/Modules/indexeddb/shared/IDBResourceIdentifier.h:
(WebCore::IDBResourceIdentifier::operator== const): Deleted.
* Source/WebCore/Modules/mediasession/MediaPositionState.h:
(WebCore::MediaPositionState::operator== const): Deleted.
* Source/WebCore/Modules/permissions/PermissionDescriptor.h:
(WebCore::PermissionDescriptor::operator== const): Deleted.
* Source/WebCore/Modules/webaudio/BaseAudioContext.h:
* Source/WebCore/PAL/pal/SessionID.h:
(PAL::SessionID::operator== const): Deleted.
* Source/WebCore/contentextensions/ContentExtensionActions.cpp:
(WebCore::ContentExtensions::ModifyHeadersAction::operator== const): Deleted.
(WebCore::ContentExtensions::ModifyHeadersAction::ModifyHeaderInfo::operator== 
const): Deleted.
(WebCore::ContentExtensions::RedirectAction::operator== const): Deleted.
(WebCore::ContentExtensions::RedirectAction::URLTransformAction::operator== 
const): Deleted.
(WebCore::ContentExtensions::RedirectAction::URLTransformAction::QueryTransform::operator==
 const): Deleted.
(WebCore::ContentExtensions::RedirectAction::URLTransformAction::QueryTransform::QueryKeyValue::operator==
 const): Deleted.
* Source/WebCore/contentextensions/ContentExtensionActions.h:
(WebCore::ContentExtensions::ActionWithoutMetadata::operator== const): Deleted.
(WebCore::ContentExtensions::ActionWithStringMetadata::operator== const): 
Deleted.
* Source/WebCore/contentextensions/ContentExtensionRule.h:
(WebCore::ContentExtensions::Trigger::operator== const): Deleted.
(WebCore::ContentExtensions::Action::operator== const): Deleted.
(WebCore::ContentExtensions::ContentExtensionRule::operator== const): Deleted.
* Source/WebCore/contentextensions/NFAToDFA.cpp:
* Source/WebCore/contentextensions/Term.h:
(WebCore::ContentExtensions::Term::CharacterSet::operator== const): Deleted.
(WebCore::ContentExtensions::Term::Group::operator== const): Deleted.
* Source/WebCore/css/CSSCounterStyleDescriptors.h:
(WebCore::CSSCounterStyleDescriptors::operator== const):
(WebCore::CSSCounterStyleDescriptors::Symbol::operator== const): Deleted.
(WebCore::CSSCounterStyleDescriptors::Pad::operator== const): Deleted.
(WebCore::CSSCounterStyleDescriptors::NegativeSymbols::operator== const): 
Deleted.
* Source/WebCore/css/CSSCounterStyleRegistry.cpp:
(WebCore::CSSCounterStyleRegistry::operator== const):
* Source/WebCore/css/CSSCustomPropertyValue.h:
* Source/WebCore/css/CSSFontFace.h:
* Source/WebCore/css/CSSGradientValue.cpp:
(WebCore::CSSLinearGradientValue::Angle::operator==):
(WebCore::CSSPrefixedLinearGradientValue::operator==):
(WebCore::CSSRadialGradientValue::operator==):
(WebCore::CSSPrefixedRadialGradientValue::operator==):
(WebCore::CSSConicGradientValue::operator==):
* Source/WebCore/css/CSSGradientValue.h:
(WebCore::CSSGradientColorInterpolationMethod::legacyMethod):
(WebCore::operator==):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 7a9106: [32 bit] DFG graph generation: intrinsic getters a...

2023-08-31 Thread jjgriego
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 7a9106c7a56f0cd97901a535361ba5418900e058
  
https://github.com/WebKit/WebKit/commit/7a9106c7a56f0cd97901a535361ba5418900e058
  Author: Joseph Griego 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
A 
JSTests/stress/typed-array-intrinsic-getter-with-conflicting-value-profile.js
M Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp

  Log Message:
  ---
  [32 bit] DFG graph generation: intrinsic getters are fallible
https://bugs.webkit.org/show_bug.cgi?id=260908

Reviewed by Yusuke Suzuki.

On 32-bit, unlike 64-bit, some of the DFG intrinsic getters (really, the
TypedArray ones) are _fallible_: if the SpeculatedType doesn't match our
expecations (a non-strict subset of SpecInt32Only), we refuse to generate code. 
[1]

However, DFG::ByteCodeParser::handleGetById doesn't appear to handle this case
gracefully--if `handleIntrinsicGetter` fails, we attempt to generate a call to
the getter, but in the case of TypedArray intrinsics, we won't have the
necessary CallLinkStatus and while attempting to do so, we crash.

To fix this, I've added a bit of code that handles the failure from
handleIntrinsicGetter and emits an ordinary `GetById` node instead of trying to
inline anything for this op.

I've added a test that demonstrates the current behavior (a segfault) on armv7
and passes with tihs patch.

[1] For what it's worth, maybe this shouldn't be the case: it does seem like we
should still be able to generate code in these cases anyhow, but it's simpler to
just cope with the failure.

* 
JSTests/stress/typed-array-intrinsic-getter-with-conflicting-value-profile.js: 
Added.
(foo):
(i.null.foo.Object.create):
(i.42.foo):
* Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp:
(JSC::DFG::ByteCodeParser::handleGetById):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 65b402: Remove PerformanceResourceTimingSensitivePropertie...

2023-08-31 Thread Anne van Kesteren
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 65b4029558faf9fbcbe7add5ea24465b1a448bc6
  
https://github.com/WebKit/WebKit/commit/65b4029558faf9fbcbe7add5ea24465b1a448bc6
  Author: Anne van Kesteren 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml
M Source/WebCore/page/PerformanceResourceTiming.idl

  Log Message:
  ---
  Remove PerformanceResourceTimingSensitivePropertiesEnabled preference
https://bugs.webkit.org/show_bug.cgi?id=260963
rdar://114763113

Reviewed by Alex Christensen.

Simplify the code by removing a preference that no longer has a purpose.

* Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml:
* Source/WebCore/page/PerformanceResourceTiming.idl:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] af90c2: [prepare-ChangeLog] Handle versioned Perl/Python s...

2023-08-31 Thread Sam Sneddon
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: af90c2a471d7f1ccacf90d1877db64452a124680
  
https://github.com/WebKit/WebKit/commit/af90c2a471d7f1ccacf90d1877db64452a124680
  Author: Sam Sneddon 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Tools/Scripts/prepare-ChangeLog

  Log Message:
  ---
  [prepare-ChangeLog] Handle versioned Perl/Python shebangs
https://bugs.webkit.org/show_bug.cgi?id=260918

Reviewed by Alexey Proskuryakov.

* Tools/Scripts/prepare-ChangeLog:
(get_function_line_ranges):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 762ed2: [Gardening]: REGRESSION(267119@main): [ macOS wk2 ...

2023-08-31 Thread Ben
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 762ed20888ea28e0d7fe28e39e1a751472d6d429
  
https://github.com/WebKit/WebKit/commit/762ed20888ea28e0d7fe28e39e1a751472d6d429
  Author: Ben Schwartz 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M LayoutTests/platform/mac-wk2/TestExpectations

  Log Message:
  ---
  [Gardening]: REGRESSION(267119@main): [ macOS wk2 release ] 
fast/repaint/placeholder-after-caps-lock-hidden.html is a constant timeout.
rdar://114772162
https://bugs.webkit.org/show_bug.cgi?id=260974

Unreviewed test gardening.

* LayoutTests/platform/mac-wk2/TestExpectations:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 1f2410: [JSC] Make JSON.parse even faster with identifier-...

2023-08-31 Thread Yusuke Suzuki
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 1f24101561da7ba92edc5b8efab210af2393c7a6
  
https://github.com/WebKit/WebKit/commit/1f24101561da7ba92edc5b8efab210af2393c7a6
  Author: Yusuke Suzuki 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
A JSTests/microbenchmarks/todomvc-javascript-es5-json-parse.js
A JSTests/microbenchmarks/todomvc-javascript-es6-webpack-json-parse.js
A JSTests/stress/ensure-json-put-does-not-see-may-be-prototype.js
M Source/JavaScriptCore/runtime/JSObject.cpp
M Source/JavaScriptCore/runtime/JSObject.h
M Source/JavaScriptCore/runtime/JSObjectInlines.h
M Source/JavaScriptCore/runtime/LiteralParser.cpp
M Source/JavaScriptCore/runtime/LiteralParser.h
M Source/JavaScriptCore/runtime/StructureInlines.h

  Log Message:
  ---
  [JSC] Make JSON.parse even faster with identifier-hint and putDirect fast path
https://bugs.webkit.org/show_bug.cgi?id=260921
rdar://114715367

Reviewed by Keith Miller.

This patch accelerates JSON.parse even more by introducing two optimizations.

1. We add IndeitiferHint to JSON.parse's lexing fast path. This slightly 
changes how we scan the string,
   given the information that this will be used as an identifier for objects. 
We swap the ordering of
   comparison for UChar JSON.parse in isSafeStringCharacterForIdentifier, since 
identifier is, in almost all cases, ASCII!
2. We found that `putDirect` function call in JSON.parse fast path is costly, 
and if we can inline the fast cases of this,
   we can get performance boost. Given that this is JSON.parse's fast path for 
JSObject creation, we know that
a. The object is definitely JSFinalObject.
b. The object rarely has duplicate properties.
c. Many same-shaped objects would be created from JSON. Thus very likely, 
there is already an existing Structure.
   Based on this observation, we add super fast path in JSON.parse which does 
existing Structure transition quickly.
3. Added JSON.parse microbenchmarks extracted from Speedometer3.0.

 ToT 
Patched

todomvc-javascript-es5-json-parse  58.1822+-0.2031 ^ 
52.2885+-0.3449^ definitely 1.1127x faster
vanilla-es2015-babel-webpack-todomvc-json-parse
   57.4813+-0.5289 ^ 
54.7597+-0.1805^ definitely 1.0497x faster
todomvc-javascript-es6-webpack-json-parse
   89.5463+-0.5104 ^ 
83.3758+-0.5923^ definitely 1.0740x faster
vanilla-es2015-todomvc-json-parse  57.2404+-0.4405 ^ 
54.6702+-0.1448^ definitely 1.0470x faster
vanilla-todomvc-json-parse 42.4967+-0.2446 ^ 
40.6605+-0.0723^ definitely 1.0452x faster

* JSTests/microbenchmarks/todomvc-javascript-es5-json-parse.js: Added.
(test):
* JSTests/microbenchmarks/todomvc-javascript-es6-webpack-json-parse.js: Added.
(test):
* Source/JavaScriptCore/runtime/JSObject.cpp:
(JSC::JSObject::putDirectForJSONSlow):
* Source/JavaScriptCore/runtime/JSObject.h:
* Source/JavaScriptCore/runtime/JSObjectInlines.h:
(JSC::JSObject::putDirectInternal):
* Source/JavaScriptCore/runtime/LiteralParser.cpp:
(JSC::isJSONWhiteSpace):
(JSC::LiteralParser::Lexer::lex):
(JSC::LiteralParser::Lexer::next):
(JSC::LiteralParser::Lexer::nextMaybeIdentifier):
(JSC::isSafeStringCharacterForIdentifier):
(JSC::LiteralParser::Lexer::lexString):
(JSC::LiteralParser::parseRecursively):
* Source/JavaScriptCore/runtime/LiteralParser.h:
* Source/JavaScriptCore/runtime/StructureInlines.h:
(JSC::Structure::addPropertyTransitionToExistingStructureImpl):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 2a902c: [IFC][BiDi] Fix fast/scrolling/rtl-scrollbars-elem...

2023-08-31 Thread Alan Baradlay
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 2a902cee37d868ca599244e5343052dfd25bb300
  
https://github.com/WebKit/WebKit/commit/2a902cee37d868ca599244e5343052dfd25bb300
  Author: Alan Baradlay 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebCore/layout/integration/inline/LayoutIntegrationLineLayout.cpp

  Log Message:
  ---
  [IFC][BiDi] Fix fast/scrolling/rtl-scrollbars-elementFromPoint-static.html
https://bugs.webkit.org/show_bug.cgi?id=260964

Reviewed by Antti Koivisto.

Vertical scrollbar on the left (user preference) imposes an offset on LTR 
content.

e.g.
 _
| |This left to right content has
| |scrollbar on the left.
| |
|_||

* Source/WebCore/layout/integration/inline/LayoutIntegrationLineLayout.cpp:
(WebCore::LayoutIntegration::LineLayout::updateInlineContentConstraints):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 416515: [Cocoa] VideoFullscreenInterfaceContext::m_manager...

2023-08-31 Thread aestes
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 4165157020d8e4b60062b5d144279c06b8c6c0ab
  
https://github.com/WebKit/WebKit/commit/4165157020d8e4b60062b5d144279c06b8c6c0ab
  Author: Andy Estes 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebKit/WebProcess/cocoa/VideoFullscreenManager.h
M Source/WebKit/WebProcess/cocoa/VideoFullscreenManager.mm

  Log Message:
  ---
  [Cocoa] VideoFullscreenInterfaceContext::m_manager should be a WeakPtr
https://bugs.webkit.org/show_bug.cgi?id=260873
rdar://114654278

Reviewed by Jer Noble.

Changed VideoFullscreenInterfaceContext::m_manager from a 
VideoFullscreenManager* to a
WeakPtr.

* Source/WebKit/WebProcess/cocoa/VideoFullscreenManager.h:
(WebKit::VideoFullscreenInterfaceContext::invalidate): Deleted.
* Source/WebKit/WebProcess/cocoa/VideoFullscreenManager.mm:
(WebKit::VideoFullscreenInterfaceContext::VideoFullscreenInterfaceContext):
(WebKit::VideoFullscreenManager::~VideoFullscreenManager):
(WebKit::VideoFullscreenManager::removeContext):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] dd6d24: Unreviewed, fix cloop build

2023-08-31 Thread Michael Catanzaro
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: dd6d24eb821c56b879594e5599f5a56246db
  
https://github.com/WebKit/WebKit/commit/dd6d24eb821c56b879594e5599f5a56246db
  Author: Michael Catanzaro 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/JavaScriptCore/b3/air/testair.cpp

  Log Message:
  ---
  Unreviewed, fix cloop build
https://bugs.webkit.org/show_bug.cgi?id=260966

* Source/JavaScriptCore/b3/air/testair.cpp:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 855a2c: [GTK][WPE] Do not use epoxy PlatformDisplay::creat...

2023-08-31 Thread Carlos Garcia Campos
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 855a2ca13f6a85b0aacb0576fbbc9a926e646b2e
  
https://github.com/WebKit/WebKit/commit/855a2ca13f6a85b0aacb0576fbbc9a926e646b2e
  Author: Carlos Garcia Campos 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebCore/platform/graphics/PlatformDisplay.cpp

  Log Message:
  ---
  [GTK][WPE] Do not use epoxy PlatformDisplay::create|destroyEGLImage
https://bugs.webkit.org/show_bug.cgi?id=260968

Reviewed by Michael Catanzaro.

libepoxy needs a current context to work, but creating EGL images
doesn't need any context, it's display API.

* Source/WebCore/platform/graphics/PlatformDisplay.cpp:
(WebCore::PlatformDisplay::createEGLImage const):
(WebCore::PlatformDisplay::destroyEGLImage const):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 2d4251: Move test that are directly under fast/ to subdire...

2023-08-31 Thread Antti Koivisto
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 2d425142d355653ac95b46d152e46db2c2c12a2d
  
https://github.com/WebKit/WebKit/commit/2d425142d355653ac95b46d152e46db2c2c12a2d
  Author: Antti Koivisto 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
R LayoutTests/fast/check-layout-error-no-attributes-expected.txt
R LayoutTests/fast/check-layout-error-no-attributes.html
A LayoutTests/fast/css/check-layout-error-no-attributes-expected.txt
A LayoutTests/fast/css/check-layout-error-no-attributes.html
A LayoutTests/fast/css/has-sibling-without-parent-crash-expected.html
A LayoutTests/fast/css/has-sibling-without-parent-crash.html
R LayoutTests/fast/has-sibling-without-parent-crash-expected.html
R LayoutTests/fast/has-sibling-without-parent-crash.html

  Log Message:
  ---
  Move test that are directly under fast/ to subdirectories
https://bugs.webkit.org/show_bug.cgi?id=260948
rdar://114746451

Unreviewed test moving.

* LayoutTests/fast/css/check-layout-error-no-attributes-expected.txt: Renamed 
from LayoutTests/fast/check-layout-error-no-attributes-expected.txt.
* LayoutTests/fast/css/check-layout-error-no-attributes.html: Renamed from 
LayoutTests/fast/check-layout-error-no-attributes.html.
* LayoutTests/fast/css/has-sibling-without-parent-crash-expected.html: Renamed 
from LayoutTests/fast/has-sibling-without-parent-crash-expected.html.
* LayoutTests/fast/css/has-sibling-without-parent-crash.html: Renamed from 
LayoutTests/fast/has-sibling-without-parent-crash.html.

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 778f6c: [GTK] Remove webkitWebViewBaseMakeGLContextCurrent

2023-08-31 Thread Carlos Garcia Campos
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 778f6cd65a7ad987dfbb6eab18927339f6536b9c
  
https://github.com/WebKit/WebKit/commit/778f6cd65a7ad987dfbb6eab18927339f6536b9c
  Author: Carlos Garcia Campos 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebKit/UIProcess/API/gtk/WebKitWebViewBase.cpp
M Source/WebKit/UIProcess/API/gtk/WebKitWebViewBasePrivate.h
M Source/WebKit/UIProcess/WebPageProxy.h
M Source/WebKit/UIProcess/gtk/AcceleratedBackingStore.h
M Source/WebKit/UIProcess/gtk/AcceleratedBackingStoreDMABuf.cpp
M Source/WebKit/UIProcess/gtk/AcceleratedBackingStoreDMABuf.h
M Source/WebKit/UIProcess/gtk/AcceleratedBackingStoreWayland.h
M Source/WebKit/UIProcess/gtk/WebPageProxyGtk.cpp

  Log Message:
  ---
  [GTK] Remove webkitWebViewBaseMakeGLContextCurrent
https://bugs.webkit.org/show_bug.cgi?id=260965

Reviewed by Michael Catanzaro.

This is a leftover from the nested wayland compositor and it's now unused.

* Source/WebKit/UIProcess/API/gtk/WebKitWebViewBase.cpp:
(webkitWebViewBaseMakeGLContextCurrent): Deleted.
* Source/WebKit/UIProcess/API/gtk/WebKitWebViewBasePrivate.h:
* Source/WebKit/UIProcess/WebPageProxy.h:
* Source/WebKit/UIProcess/gtk/AcceleratedBackingStore.h:
(WebKit::AcceleratedBackingStore::unrealize):
(WebKit::AcceleratedBackingStore::makeContextCurrent): Deleted.
* Source/WebKit/UIProcess/gtk/AcceleratedBackingStoreDMABuf.cpp:
(WebKit::AcceleratedBackingStoreDMABuf::makeContextCurrent): Deleted.
* Source/WebKit/UIProcess/gtk/AcceleratedBackingStoreDMABuf.h:
* Source/WebKit/UIProcess/gtk/AcceleratedBackingStoreWayland.h:
* Source/WebKit/UIProcess/gtk/WebPageProxyGtk.cpp:
(WebKit::WebPageProxy::makeGLContextCurrent): Deleted.

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 630bb6: Add some libwebrtc logging in case of packet loss ...

2023-08-31 Thread youennf
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 630bb64e52ef650fa4d6c60b9ea3547db20d31e5
  
https://github.com/WebKit/WebKit/commit/630bb64e52ef650fa4d6c60b9ea3547db20d31e5
  Author: Youenn Fablet 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M 
Source/ThirdParty/libwebrtc/Source/webrtc/modules/congestion_controller/goog_cc/send_side_bandwidth_estimation.cc

  Log Message:
  ---
  Add some libwebrtc logging in case of packet loss triggering bandwidth 
estimation reduction
https://bugs.webkit.org/show_bug.cgi?id=260950
rdar://problem/114747794

Reviewed by Jean-Yves Avenard.

In case of ex excessive packet loss, bandwidth gets reduced, we now log this at 
info level.

* 
Source/ThirdParty/libwebrtc/Source/webrtc/modules/congestion_controller/goog_cc/send_side_bandwidth_estimation.cc:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 9ea25d: Further migrate some scripts in WPT workers/module...

2023-08-31 Thread youennf
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 9ea25d9b008d180395413d471014af97d79c2589
  
https://github.com/WebKit/WebKit/commit/9ea25d9b008d180395413d471014af97d79c2589
  Author: Youenn Fablet 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import-blob-url.any-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import-blob-url.any.worker-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import.any-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import.any.worker-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/resources/dynamic-import-remote-origin-credentials-checker-worker.sub.js
M 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/resources/dynamic-import-remote-origin-script-worker.sub.js
M 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/resources/static-import-remote-origin-credentials-checker-worker.sub.js
M 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/resources/static-import-remote-origin-script-worker.sub.js
M 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/shared-worker-import.window-expected.txt
R 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import-blob-url.any-expected.txt
R 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import-blob-url.any.worker-expected.txt
R 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import.any-expected.txt
R 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import.any.worker-expected.txt
R 
LayoutTests/platform/gtk/imported/w3c/web-platform-tests/workers/modules/shared-worker-import-blob-url.window-expected.txt
R 
LayoutTests/platform/gtk/imported/w3c/web-platform-tests/workers/modules/shared-worker-import.window-expected.txt
R 
LayoutTests/platform/wpe/imported/w3c/web-platform-tests/workers/modules/shared-worker-import-blob-url.window-expected.txt
R 
LayoutTests/platform/wpe/imported/w3c/web-platform-tests/workers/modules/shared-worker-import.window-expected.txt

  Log Message:
  ---
  Further migrate some scripts in WPT workers/module from domains[www1] to 
hosts[alt][]
https://bugs.webkit.org/show_bug.cgi?id=260901
rdar://114696293

Reviewed by Tim Nguyen.

This allows properly running the tests in Cocoa test infrastructure as they 
would pass from wpt.live.
We remove some GTK/WPE test expectations since they are now the same as the 
global ones.

* 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import-blob-url.any-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import-blob-url.any.worker-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import.any-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import.any.worker-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/resources/dynamic-import-remote-origin-credentials-checker-worker.sub.js:
* 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/resources/dynamic-import-remote-origin-script-worker.sub.js:
* 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/resources/static-import-remote-origin-credentials-checker-worker.sub.js:
* 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/resources/static-import-remote-origin-script-worker.sub.js:
* 
LayoutTests/imported/w3c/web-platform-tests/workers/modules/shared-worker-import.window-expected.txt:
* 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import-blob-url.any-expected.txt:
 Removed.
* 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import-blob-url.any.worker-expected.txt:
 Removed.
* 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import.any-expected.txt:
 Removed.
* 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/workers/modules/dedicated-worker-import.any.worker-expected.txt:
 Removed.
* 
LayoutTests/platform/gtk/imported/w3c/web-platform-tests/workers/modules/shared-worker-import-blob-url.window-expected.txt:
 Removed.
* 
LayoutTests/platform/gtk/imported/w3c/web-platform-tests/workers/modules/shared-worker-import.window-expected.txt:
 Removed.
* 
LayoutTests/platform/wpe/imported/w3c/web-platform-tests/workers/modules/shared-worker-import-blob-url.window-expected.txt:
 Removed.
* 
LayoutTests/platform/wpe/imported/w3c/web-platform-tests/workers/modules/shared-worker-import.window-expected.txt:
 Removed.

Canonical link: https:/

[webkit-changes] [WebKit/WebKit] 94fe30: Correct dynamic handling of elements

2023-08-31 Thread Anne van Kesteren
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 94fe30e306b57acc3a62e6bd75351fee3a393972
  
https://github.com/WebKit/WebKit/commit/94fe30e306b57acc3a62e6bd75351fee3a393972
  Author: Anne van Kesteren 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M LayoutTests/fetch/fetch-url-serialization-expected.txt
M LayoutTests/fetch/fetch-urls.json
A 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/document-metadata/the-base-element/base-data-expected.txt
A 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/document-metadata/the-base-element/base-data.html
A 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/document-metadata/the-base-element/base-javascript-expected.txt
A 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/document-metadata/the-base-element/base-javascript.html
M 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/document-metadata/the-base-element/w3c-import.log
M 
LayoutTests/imported/w3c/web-platform-tests/url/a-element-origin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/url/a-element-origin-xhtml-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/url/a-element-xhtml_exclude=(file_javascript_mailto)-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/url/a-element_exclude=(file_javascript_mailto)-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/url/resources/a-element-origin.js
M LayoutTests/imported/w3c/web-platform-tests/url/resources/a-element.js
M Source/WebCore/dom/Document.cpp
M Source/WebCore/html/HTMLBaseElement.cpp

  Log Message:
  ---
  Correct dynamic handling of  elements
https://bugs.webkit.org/show_bug.cgi?id=260959
rdar://114756660

Reviewed by Chris Dumez.

Document::processBaseElement did not call Document::updateBaseURL when
there was something wrong with the new base URL. However, that meant
that a newly inserted "blocked"  would not impact the document
base URL and instead whatever was the prior document base URL would
continue to be used.

This goes against the HTML standard which requires the first 
element to be used at all times and if that is "blocked" the fallback
base URL would have to be used (typically the document's URL).

Tests are upstreamed via
https://github.com/web-platform-tests/wpt/pull/41731. Chromium matches
our failure, but given that it's an edge case I don't foresee any
issues.

While we're in the general area, stop invoking
Document::processBaseElement for  elements not connected to a
document as that results in a no-op tree traversal.

* LayoutTests/fetch/fetch-url-serialization-expected.txt:
* LayoutTests/fetch/fetch-urls.json:
* 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/document-metadata/the-base-element/base-data-expected.txt:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/document-metadata/the-base-element/base-data.html:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/document-metadata/the-base-element/base-javascript-expected.txt:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/document-metadata/the-base-element/base-javascript.html:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/document-metadata/the-base-element/w3c-import.log:
* LayoutTests/imported/w3c/web-platform-tests/url/a-element-origin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/url/a-element-origin-xhtml-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/url/a-element-xhtml_exclude=(file_javascript_mailto)-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/url/a-element_exclude=(file_javascript_mailto)-expected.txt:
* LayoutTests/imported/w3c/web-platform-tests/url/resources/a-element-origin.js:
(runURLTests):
* LayoutTests/imported/w3c/web-platform-tests/url/resources/a-element.js:
* Source/WebCore/dom/Document.cpp:
(WebCore::Document::processBaseElement):
* Source/WebCore/html/HTMLBaseElement.cpp:
(WebCore::HTMLBaseElement::attributeChanged):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] f8ea32: Use an enum class for SVGPathSegType

2023-08-31 Thread Chris Dumez
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: f8ea32b01fdae3f7c9373634db5ea6c6764b7575
  
https://github.com/WebKit/WebKit/commit/f8ea32b01fdae3f7c9373634db5ea6c6764b7575
  Author: Chris Dumez 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebCore/bindings/js/JSSVGPathSegCustom.cpp
M Source/WebCore/svg/SVGPathBlender.cpp
M Source/WebCore/svg/SVGPathByteStream.h
M Source/WebCore/svg/SVGPathByteStreamBuilder.cpp
M Source/WebCore/svg/SVGPathByteStreamBuilder.h
M Source/WebCore/svg/SVGPathByteStreamSource.cpp
M Source/WebCore/svg/SVGPathByteStreamSource.h
M Source/WebCore/svg/SVGPathParser.cpp
M Source/WebCore/svg/SVGPathParser.h
M Source/WebCore/svg/SVGPathSeg.h
M Source/WebCore/svg/SVGPathSeg.idl
M Source/WebCore/svg/SVGPathSegImpl.h
M Source/WebCore/svg/SVGPathSegListSource.cpp
M Source/WebCore/svg/SVGPathStringViewSource.cpp

  Log Message:
  ---
  Use an enum class for SVGPathSegType
https://bugs.webkit.org/show_bug.cgi?id=260930

Reviewed by Ryosuke Niwa.

Use an enum class for SVGPathSegType instead of an old enum.
Use uint8_t as underlying type and update SVGPathByteStreamBuilder
& SVGPathByteStreamSource to encode/decode it as a uint8_t instead
of an `unsigned short`.

* Source/WebCore/svg/SVGPathBlender.cpp:
(WebCore::coordinateModeOfCommand):
(WebCore::isSegmentEqual):
(WebCore::SVGPathBlender::canBlendPaths):
(WebCore::SVGPathBlender::blendAnimatedPath):
* Source/WebCore/svg/SVGPathByteStreamBuilder.cpp:
(WebCore::SVGPathByteStreamBuilder::moveTo):
(WebCore::SVGPathByteStreamBuilder::lineTo):
(WebCore::SVGPathByteStreamBuilder::lineToHorizontal):
(WebCore::SVGPathByteStreamBuilder::lineToVertical):
(WebCore::SVGPathByteStreamBuilder::curveToCubic):
(WebCore::SVGPathByteStreamBuilder::curveToCubicSmooth):
(WebCore::SVGPathByteStreamBuilder::curveToQuadratic):
(WebCore::SVGPathByteStreamBuilder::curveToQuadraticSmooth):
(WebCore::SVGPathByteStreamBuilder::arcTo):
(WebCore::SVGPathByteStreamBuilder::closePath):
* Source/WebCore/svg/SVGPathByteStreamBuilder.h:
* Source/WebCore/svg/SVGPathParser.cpp:
(WebCore::SVGPathParser::parseCurveToCubicSmoothSegment):
(WebCore::SVGPathParser::parseCurveToQuadraticSmoothSegment):
(WebCore::SVGPathParser::parsePathData):
* Source/WebCore/svg/SVGPathParser.h:
* Source/WebCore/svg/SVGPathSeg.h:
* Source/WebCore/svg/SVGPathSegListSource.cpp:
(WebCore::SVGPathSegListSource::parseMoveToSegment):
(WebCore::SVGPathSegListSource::parseLineToSegment):
(WebCore::SVGPathSegListSource::parseLineToHorizontalSegment):
(WebCore::SVGPathSegListSource::parseLineToVerticalSegment):
(WebCore::SVGPathSegListSource::parseCurveToCubicSegment):
(WebCore::SVGPathSegListSource::parseCurveToCubicSmoothSegment):
(WebCore::SVGPathSegListSource::parseCurveToQuadraticSegment):
(WebCore::SVGPathSegListSource::parseCurveToQuadraticSmoothSegment):
(WebCore::SVGPathSegListSource::parseArcToSegment):
* Source/WebCore/svg/SVGPathStringViewSource.cpp:
(WebCore::nextCommandHelper):
(WebCore::SVGPathStringViewSource::parseSVGSegmentType):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 93f874: Fix removed-in-Python-3.12 deprecations

2023-08-31 Thread Sam Sneddon
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 93f8745ec911a80187109e33a70172e08fdd6583
  
https://github.com/WebKit/WebKit/commit/93f8745ec911a80187109e33a70172e08fdd6583
  Author: Sam Sneddon 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebKit/Scripts/webkit/parser_unittest.py
M Tools/Scripts/webkitpy/benchmark_runner/benchmark_results_unittest.py
M Tools/Scripts/webkitpy/benchmark_runner/http_server_driver/__init__.py
M Tools/Scripts/webkitpy/benchmark_runner/utils.py
M Tools/Scripts/webkitpy/common/checkout/scm/scm_unittest.py
M Tools/Scripts/webkitpy/common/prettypatch_unittest.py
M Tools/Scripts/webkitpy/common/watchlist/watchlistparser_unittest.py
R Tools/Scripts/webkitpy/common/webkitunittest.py
M 
Tools/Scripts/webkitpy/layout_tests/servers/web_platform_test_server_unittest.py
M Tools/Scripts/webkitpy/port/xvfbdriver_unittest.py
M Tools/Scripts/webkitpy/tool/commands/applywatchlistlocal_unittest.py
M Tools/Scripts/webkitpy/tool/commands/commandtest.py
M Tools/Scripts/webkitpy/w3c/test_converter_unittest.py

  Log Message:
  ---
  Fix removed-in-Python-3.12 deprecations
https://bugs.webkit.org/show_bug.cgi?id=260878

Reviewed by Michael Catanzaro.

This largely just deals with the long-deprecated unittest deprecations:

 * assertRegexpMatches -> assertRegex
 * assertNotRegexpMatches -> assertNotRegex
 * assertRaisesRegexp -> assertRaisesRegex
 * assertEquals -> assertEqual

Additionally, this gets rid of usage of distutils (gone in Python
3.12) and imp (also gone in Python 3.12).

* Source/WebKit/Scripts/webkit/parser_unittest.py:
(UnsupportedPrecompilerDirectiveTest.assertRaisesRegex):
(UnsupportedPrecompilerDirectiveTest.test_error_at_else):
(UnsupportedPrecompilerDirectiveTest.test_error_at_elif):
* Tools/Scripts/webkitpy/benchmark_runner/benchmark_results_unittest.py:
(BenchmarkResultsTest.assertRaisesRegex):
(BenchmarkResultsTest.test_init):
* Tools/Scripts/webkitpy/benchmark_runner/http_server_driver/__init__.py:
* Tools/Scripts/webkitpy/benchmark_runner/utils.py:
(load_source):
(load_subclasses):
* Tools/Scripts/webkitpy/common/checkout/scm/scm_unittest.py:
(SCMTest.assertRegex):
(SCMTest):
(SCMTest.assertNotRegex):
(SCMTest._shared_test_diff_for_revision):
(test_commit_logs):
(GitTest.test_create_patch):
(GitTest.test_create_patch_with_git_index):
(GitTest.test_rename_files):
(GitSVNTest.test_delete_branch):
(GitSVNTest.test_remote_merge_base):
(GitSVNTest.test_commit_with_message_working_copy_only):
(GitSVNTest.test_commit_with_message):
(GitSVNTest.test_commit_with_message_git_commit):
(GitSVNTest.test_commit_with_message_git_commit_range):
(GitSVNTest.test_commit_with_message_only_local_commit):
(GitSVNTest.test_commit_with_message_multiple_local_commits_and_working_copy):
(GitSVNTest.test_commit_with_message_multiple_local_commits_always_squash):
(GitSVNTest.test_commit_with_message_multiple_local_commits):
(GitSVNTest.test_commit_with_message_not_synced):
(GitSVNTest.test_create_patch_local_plus_working_copy):
(GitSVNTest.test_create_patch):
(GitSVNTest.test_create_patch_after_merge):
(GitSVNTest.test_create_patch_with_changed_files):
(GitSVNTest.test_create_patch_git_commit):
(GitSVNTest.test_create_patch_git_commit_range):
(GitSVNTest.test_create_patch_working_copy_only):
(GitSVNTest.test_create_patch_multiple_local_commits):
(GitSVNTest.test_create_patch_not_synced):
(GitSVNTest.test_create_binary_patch):
* Tools/Scripts/webkitpy/common/prettypatch_unittest.py:
(PrettyPatchTest.check_ruby):
* Tools/Scripts/webkitpy/common/watchlist/watchlistparser_unittest.py:
(WatchListParserTest):
(WatchListParserTest.setUp):
* Tools/Scripts/webkitpy/common/webkitunittest.py: Removed.
* 
Tools/Scripts/webkitpy/layout_tests/servers/web_platform_test_server_unittest.py:
* Tools/Scripts/webkitpy/port/xvfbdriver_unittest.py:
(XvfbDriverTest.assertRaisesRegex):
(XvfbDriverTest.test_xvfb_not_replying):
* Tools/Scripts/webkitpy/tool/commands/applywatchlistlocal_unittest.py:
(ApplyWatchListLocalTest.assertRaisesRegex):
(test_args_parsing_with_two_bugs):
* Tools/Scripts/webkitpy/tool/commands/commandtest.py:
(CommandsTest):
* Tools/Scripts/webkitpy/w3c/test_converter_unittest.py:
(verify_test_harness_paths):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 1a2b1a: [IFC] Do not try to measure text content across In...

2023-08-31 Thread Alan Baradlay
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 1a2b1a5111fec39251c05deff1f97ae9d5849354
  
https://github.com/WebKit/WebKit/commit/1a2b1a5111fec39251c05deff1f97ae9d5849354
  Author: Alan Baradlay 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
A LayoutTests/fast/text/complex-text-with-bidi-boundary-crash-expected.txt
A LayoutTests/fast/text/complex-text-with-bidi-boundary-crash.html
M Source/WebCore/layout/formattingContexts/inline/InlineContentBreaker.cpp
M Source/WebCore/layout/formattingContexts/inline/text/TextUtil.cpp

  Log Message:
  ---
  [IFC] Do not try to measure text content across InlineItems
https://bugs.webkit.org/show_bug.cgi?id=260935


Reviewed by Antti Koivisto.

This happens when ubidi_getLogicalRun makes us split inside a cluster (bug?) 
producing multiple InlineItems.

(simplifiedMinimumInstrinsicWidthBreak extra check is added to mirror what we 
already do at the slow codepath)

* LayoutTests/fast/text/complex-text-with-bidi-boundary-crash-expected.txt: 
Added.
* LayoutTests/fast/text/complex-text-with-bidi-boundary-crash.html: Added.
* Source/WebCore/layout/formattingContexts/inline/InlineContentBreaker.cpp:
(WebCore::Layout::InlineContentBreaker::simplifiedMinimumInstrinsicWidthBreak 
const):
* Source/WebCore/layout/formattingContexts/inline/text/TextUtil.cpp:
(WebCore::Layout::TextUtil::firstUserPerceivedCharacterLength):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 734bc0: [IFC][out-of-flow bidi] Fix fast/block/positioning...

2023-08-31 Thread Alan Baradlay
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 734bc0ccb57e1b4e5cc80d8474769f85c8f0ec5d
  
https://github.com/WebKit/WebKit/commit/734bc0ccb57e1b4e5cc80d8474769f85c8f0ec5d
  Author: Alan Baradlay 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebCore/rendering/RenderInline.cpp

  Log Message:
  ---
  [IFC][out-of-flow bidi] Fix 
fast/block/positioning/absolute-in-inline-rtl(3).html
https://bugs.webkit.org/show_bug.cgi?id=260940

Reviewed by Antti Koivisto.

Add missing RTL support in RenderInline::innerPaddingBoxWidth.

* Source/WebCore/rendering/RenderInline.cpp:
(WebCore::RenderInline::innerPaddingBoxWidth const):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] c3b8ad: Addressing review feedback from 267490@main

2023-08-31 Thread Myles C. Maxfield
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c3b8ad19792f0ebd4418dd1dab3b76ff4b752ea1
  
https://github.com/WebKit/WebKit/commit/c3b8ad19792f0ebd4418dd1dab3b76ff4b752ea1
  Author: Myles C. Maxfield 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WTF/wtf/text/TextStream.cpp
M Source/WTF/wtf/text/TextStream.h
M Tools/TestWebKitAPI/Tests/WTF/cocoa/TextStreamCocoa.cpp

  Log Message:
  ---
  Addressing review feedback from 267490@main
https://bugs.webkit.org/show_bug.cgi?id=260956
rdar://114751994

Reviewed by Simon Fraser.

Addressing review feedback from
https://github.com/WebKit/WebKit/pull/17269#pullrequestreview-1603946683

* Source/WTF/wtf/text/TextStream.cpp:
(WTF::TextStream::operator<<):
* Source/WTF/wtf/text/TextStream.h:
* Tools/TestWebKitAPI/Tests/WTF/cocoa/TextStreamCocoa.cpp:
(TEST):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 3ebb0e: [Cocoa] Slightly improve text shaping logging

2023-08-31 Thread Myles C. Maxfield
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 3ebb0e8d48f4262ebf64f5f3bcc0a080c2d83c11
  
https://github.com/WebKit/WebKit/commit/3ebb0e8d48f4262ebf64f5f3bcc0a080c2d83c11
  Author: Myles C. Maxfield 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WebCore/platform/graphics/coretext/FontCoreText.cpp

  Log Message:
  ---
  [Cocoa] Slightly improve text shaping logging
https://bugs.webkit.org/show_bug.cgi?id=260942
rdar://114742142

Reviewed by Cameron McCormack.

This additional logging helped me fix a bug I was investigating.

No test because there is no behavior change.

* Source/WebCore/platform/graphics/coretext/FontCoreText.cpp:
(WebCore::Font::applyTransforms const):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] aed2f1: Don’t open “details” when interacting with interac...

2023-08-31 Thread sideshowbarker
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: aed2f1dba4a42f57914f579efc8f76b7c9fe8f3e
  
https://github.com/WebKit/WebKit/commit/aed2f1dba4a42f57914f579efc8f76b7c9fe8f3e
  Author: Michael[tm] Smith 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M LayoutTests/TestExpectations
M 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/interactive-elements/the-summary-element/anchor-with-inline-element-expected.txt
A 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/interactive-elements/the-summary-element/interactive-content-expected.txt
A 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/interactive-elements/the-summary-element/interactive-content.html
M Source/WebCore/html/HTMLSummaryElement.cpp

  Log Message:
  ---
  Don’t open “details” when interacting with interactive “summary” descendants
https://bugs.webkit.org/show_bug.cgi?id=218758

Reviewed by Tim Nguyen.

This change ensures that interacting with (clicking/pressing/etc.) any
interactive descendants of a “summary” element will trigger the
activation behavior of those descendants themselves — rather than
instead triggering the “summary” element’s “details” parent to open.

Otherwise, without this change, the activation behavior of interactive
content within a “summary” element will get triggered only if that
interactive content is a direct child of the “summary” element.

* LayoutTests/TestExpectations:
* 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/interactive-elements/the-summary-element/anchor-with-inline-element-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/interactive-elements/the-summary-element/interactive-content-expected.txt:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/html/semantics/interactive-elements/the-summary-element/interactive-content.html:
 Added.
* Source/WebCore/html/HTMLSummaryElement.cpp:
(WebCore::isInSummaryInteractiveContent):
(WebCore::HTMLSummaryElement::defaultEventHandler):
(WebCore::isClickableControl): Deleted.

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] a65bf7: Allow hex() to be used with LOG_WITH_STREAM()

2023-08-31 Thread Myles C. Maxfield
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: a65bf781e6e6cb1dbc09b0686b44c940f276823d
  
https://github.com/WebKit/WebKit/commit/a65bf781e6e6cb1dbc09b0686b44c940f276823d
  Author: Myles C. Maxfield 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WTF/wtf/text/TextStream.cpp
M Source/WTF/wtf/text/TextStream.h
M Tools/TestWebKitAPI/Tests/WTF/cocoa/TextStreamCocoa.cpp

  Log Message:
  ---
  Allow hex() to be used with LOG_WITH_STREAM()
https://bugs.webkit.org/show_bug.cgi?id=260945
rdar://114742843

Reviewed by Simon Fraser.

TextStream just needs to be able to use the return type of hex().

* Source/WTF/wtf/text/TextStream.cpp:
(WTF::TextStream::operator<<):
* Source/WTF/wtf/text/TextStream.h:
* Tools/TestWebKitAPI/Tests/WTF/cocoa/TextStreamCocoa.cpp:
(TEST):

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 40a802: Remove ReferrerPolicyAttributeEnabled preference

2023-08-31 Thread Anne van Kesteren
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 40a802ea75efb7a7fdbc874411b02696fd571aeb
  
https://github.com/WebKit/WebKit/commit/40a802ea75efb7a7fdbc874411b02696fd571aeb
  Author: Anne van Kesteren 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml
M Source/WebCore/html/HTMLAnchorElement.cpp
M Source/WebCore/html/HTMLAnchorElement.idl
M Source/WebCore/html/HTMLAreaElement.idl
M Source/WebCore/html/HTMLIFrameElement.cpp
M Source/WebCore/html/HTMLIFrameElement.idl
M Source/WebCore/html/HTMLImageElement.cpp
M Source/WebCore/html/HTMLImageElement.idl
M Source/WebCore/html/HTMLLinkElement.cpp
M Source/WebCore/html/HTMLLinkElement.idl
M Source/WebCore/html/HTMLScriptElement.cpp
M Source/WebCore/html/HTMLScriptElement.idl
M Source/WebKit/UIProcess/API/C/WKPreferences.cpp
M Source/WebKit/UIProcess/API/C/WKPreferencesRefPrivate.h
M Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h
M Source/WebKitLegacy/mac/WebView/WebPreferences.mm
M Source/WebKitLegacy/mac/WebView/WebPreferencesPrivate.h

  Log Message:
  ---
  Remove ReferrerPolicyAttributeEnabled preference
https://bugs.webkit.org/show_bug.cgi?id=260905
rdar://114702566

Reviewed by Youenn Fablet.

This enables the referrerpolicy attribute in WK1 as having additional
branching isn't beneficial for this feature. And it removes the
corresponding preference at the same time.

WKPreferencesSetReferrerPolicyAttributeEnabled and
WKPreferencesGetReferrerPolicyAttributeEnabled are deprecated, but not
removed to retain binary compatibility.

* Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml:
* Source/WebCore/html/HTMLAnchorElement.cpp:
(WebCore::HTMLAnchorElement::referrerPolicy const):
* Source/WebCore/html/HTMLAnchorElement.idl:
* Source/WebCore/html/HTMLAreaElement.idl:
* Source/WebCore/html/HTMLIFrameElement.cpp:
(WebCore::HTMLIFrameElement::referrerPolicy const):
* Source/WebCore/html/HTMLIFrameElement.idl:
* Source/WebCore/html/HTMLImageElement.cpp:
(WebCore::HTMLImageElement::attributeChanged):
(WebCore::HTMLImageElement::referrerPolicy const):
* Source/WebCore/html/HTMLImageElement.idl:
* Source/WebCore/html/HTMLLinkElement.cpp:
(WebCore::HTMLLinkElement::referrerPolicy const):
* Source/WebCore/html/HTMLLinkElement.idl:
* Source/WebCore/html/HTMLScriptElement.cpp:
(WebCore::HTMLScriptElement::referrerPolicy const):
* Source/WebCore/html/HTMLScriptElement.idl:
* Source/WebKit/UIProcess/API/C/WKPreferences.cpp:
(WKPreferencesSetReferrerPolicyAttributeEnabled):
(WKPreferencesGetReferrerPolicyAttributeEnabled):
* Source/WebKit/UIProcess/API/C/WKPreferencesRefPrivate.h:
* Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h:
* Source/WebKitLegacy/mac/WebView/WebPreferences.mm:
(-[WebPreferences referrerPolicyAttributeEnabled]): Deleted.
(-[WebPreferences setReferrerPolicyAttributeEnabled:]): Deleted.
* Source/WebKitLegacy/mac/WebView/WebPreferencesPrivate.h:

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


___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] b5959d: Web Inspector: support OffscreenCanvas for Canvas ...

2023-08-31 Thread Devin Rousso
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b5959d78adc403bde81fdad57864ec9fcf73fa21
  
https://github.com/WebKit/WebKit/commit/b5959d78adc403bde81fdad57864ec9fcf73fa21
  Author: Devin Rousso 
  Date:   2023-08-31 (Thu, 31 Aug 2023)

  Changed paths:
M LayoutTests/inspector/canvas/create-context-2d-expected.txt
M LayoutTests/inspector/canvas/create-context-2d.html
M LayoutTests/inspector/canvas/create-context-bitmaprenderer-expected.txt
M LayoutTests/inspector/canvas/create-context-bitmaprenderer.html
M LayoutTests/inspector/canvas/create-context-webgl-expected.txt
M LayoutTests/inspector/canvas/create-context-webgl.html
M LayoutTests/inspector/canvas/create-context-webgl2-expected.txt
M LayoutTests/inspector/canvas/create-context-webgl2.html
M LayoutTests/inspector/canvas/resources/create-context-utilities.js
A LayoutTests/inspector/canvas/resources/worker.js
M Source/JavaScriptCore/inspector/protocol/Canvas.json
M Source/JavaScriptCore/inspector/protocol/Recording.json
M Source/WebCore/Sources.txt
M Source/WebCore/WebCore.xcodeproj/project.pbxproj
M Source/WebCore/html/CanvasBase.cpp
M Source/WebCore/html/CanvasBase.h
M Source/WebCore/inspector/InspectorCanvas.cpp
M Source/WebCore/inspector/InspectorController.cpp
M Source/WebCore/inspector/InspectorInstrumentation.cpp
M Source/WebCore/inspector/InspectorInstrumentation.h
M Source/WebCore/inspector/InstrumentingAgents.h
M Source/WebCore/inspector/WorkerInspectorController.cpp
M Source/WebCore/inspector/agents/InspectorCanvasAgent.cpp
M Source/WebCore/inspector/agents/InspectorCanvasAgent.h
A Source/WebCore/inspector/agents/page/PageCanvasAgent.cpp
A Source/WebCore/inspector/agents/page/PageCanvasAgent.h
A Source/WebCore/inspector/agents/worker/WorkerCanvasAgent.cpp
A Source/WebCore/inspector/agents/worker/WorkerCanvasAgent.h
M Source/WebCore/workers/WorkerConsoleClient.cpp
M Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js
M Source/WebInspectorUI/UserInterface/Controllers/CanvasManager.js
M Source/WebInspectorUI/UserInterface/Models/Canvas.js
M Source/WebInspectorUI/UserInterface/Models/Geometry.js
M Source/WebInspectorUI/UserInterface/Models/Recording.js
M Source/WebInspectorUI/UserInterface/Models/ShaderProgram.js
M Source/WebInspectorUI/UserInterface/Protocol/CanvasObserver.js
M Source/WebInspectorUI/UserInterface/Protocol/RemoteObject.js
M Source/WebInspectorUI/UserInterface/Views/CanvasContentView.js

  Log Message:
  ---
  Web Inspector: support OffscreenCanvas for Canvas related operations
https://bugs.webkit.org/show_bug.cgi?id=180833


Reviewed by Patrick Angle.

* Source/JavaScriptCore/inspector/protocol/Canvas.json:
* Source/JavaScriptCore/inspector/protocol/Recording.json:
Expose the entire domain (minus a few commands/events) to the `"worker"` target.

* Source/WebCore/inspector/InspectorController.cpp:
(WebCore::InspectorController::createLazyAgents):
* Source/WebCore/inspector/WorkerInspectorController.cpp:
(WebCore::WorkerInspectorController::createLazyAgents):
* Source/WebCore/inspector/agents/InspectorCanvasAgent.h:
* Source/WebCore/inspector/agents/InspectorCanvasAgent.cpp:
(WebCore::InspectorCanvasAgent::InspectorCanvasAgent):
(WebCore::InspectorCanvasAgent::enable):
(WebCore::InspectorCanvasAgent::disable):
(WebCore::InspectorCanvasAgent::enabled const): Added.
(WebCore::InspectorCanvasAgent::internalEnable): Added.
(WebCore::InspectorCanvasAgent::internalDisable): Added.
(WebCore::InspectorCanvasAgent::didChangeCanvasSize): Added.
(WebCore::InspectorCanvasAgent::didChangeCanvasMemory):
(WebCore::InspectorCanvasAgent::unbindCanvas):
(WebCore::InspectorCanvasAgent::requestNode): Deleted.
(WebCore::InspectorCanvasAgent::requestClientNodes): Deleted.
(WebCore::InspectorCanvasAgent::frameNavigated): Deleted.
(WebCore::InspectorCanvasAgent::didChangeCSSCanvasClientNodes): Deleted.
* Source/WebCore/inspector/agents/page/PageCanvasAgent.h: Added.
* Source/WebCore/inspector/agents/page/PageCanvasAgent.cpp: Added.
(WebCore::PageCanvasAgent::PageCanvasAgent):
(WebCore::PageCanvasAgent::enabled const):
(WebCore::PageCanvasAgent::internalEnable):
(WebCore::PageCanvasAgent::internalDisable):
(WebCore::PageCanvasAgent::requestNode):
(WebCore::PageCanvasAgent::requestClientNodes):
(WebCore::PageCanvasAgent::frameNavigated):
(WebCore::PageCanvasAgent::didChangeCSSCanvasClientNodes):
(WebCore::PageCanvasAgent::matchesCurrentContext const):
* Source/WebCore/inspector/agents/worker/WorkerCanvasAgent.h: Added.
* Source/WebCore/inspector/agents/worker/WorkerCanvasAgent.cpp: Added.
(WebCore::WorkerCanvasAgent::WorkerCanvasAgent):
(WebCore::WorkerCanvasAgent::requestNode):
(WebCore::WorkerCanvasAgent::requestClientNodes):
(WebCore::WorkerCanvasAgent::matchesCurrentContext const):
Split `InspectorCanvasAgent` into `PageCanvasAgent` and `WorkerCan