[webkit-changes] [WebKit/WebKit] 6be143: Generate serializers for NSArray, NSDictionary, an...

2023-11-12 Thread Brady Eidson
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 6be1434f578ebcb90ea96a1c71bca5348812e60a
  
https://github.com/WebKit/WebKit/commit/6be1434f578ebcb90ea96a1c71bca5348812e60a
  Author: Brady Eidson 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M Source/WebCore/platform/mac/SerializedPlatformDataCueMac.h
M Source/WebCore/platform/mac/SerializedPlatformDataCueMac.mm
M Source/WebKit/Platform/IPC/Decoder.h
M Source/WebKit/Shared/Cocoa/ArgumentCodersCocoa.h
M Source/WebKit/Shared/Cocoa/ArgumentCodersCocoa.mm
M Source/WebKit/Shared/Cocoa/CoreIPCArray.h
A Source/WebKit/Shared/Cocoa/CoreIPCArray.mm
M Source/WebKit/Shared/Cocoa/CoreIPCArray.serialization.in
M Source/WebKit/Shared/Cocoa/CoreIPCCFType.h
M Source/WebKit/Shared/Cocoa/CoreIPCColor.h
M Source/WebKit/Shared/Cocoa/CoreIPCDictionary.h
A Source/WebKit/Shared/Cocoa/CoreIPCDictionary.mm
M Source/WebKit/Shared/Cocoa/CoreIPCDictionary.serialization.in
M Source/WebKit/Shared/Cocoa/CoreIPCFont.h
A Source/WebKit/Shared/Cocoa/CoreIPCFont.mm
M Source/WebKit/Shared/Cocoa/CoreIPCFont.serialization.in
M Source/WebKit/Shared/Cocoa/CoreIPCNSCFObject.h
M Source/WebKit/Shared/Cocoa/CoreIPCNSCFObject.mm
M Source/WebKit/Shared/Cocoa/CoreIPCNSCFObject.serialization.in
M Source/WebKit/Shared/Cocoa/CoreIPCSecureCoding.h
M Source/WebKit/Shared/Cocoa/CoreIPCString.h
M Source/WebKit/Shared/Cocoa/CoreIPCURL.h
M Source/WebKit/Shared/cf/CoreIPCNumber.h
M Source/WebKit/WebKit.xcodeproj/project.pbxproj
M Tools/TestWebKitAPI/Tests/IPC/IPCSerialization.mm

  Log Message:
  ---
  Generate serializers for NSArray, NSDictionary, and NSFont
https://bugs.webkit.org/show_bug.cgi?id=264696

Reviewed by Alex Christensen.

Now that we have the base CoreIPCNSCFObject wrapper and .serialization.in files 
for fundamental NS types, we can:
- Turn NSArray into a "Vector of CoreIPCNSCFObjects"
- Turn NSDictionary into a "Vector of key/value pairs where the keys and values 
are CoreIPCNSCFObjects"
- Directly serialize NSFont/UIFont, since we used an NSDictionary to do so 
before

* Source/WebKit/Shared/Cocoa/ArgumentCodersCocoa.h:
* Source/WebKit/Shared/Cocoa/ArgumentCodersCocoa.mm:
(IPC::isSerializableValue):
(IPC::encodeObjectDirectly): Deleted.
(IPC::decodeObjectDirectlyRequiringAllowedClasses): Deleted.
(IPC::encodeObjectDirectly): Deleted.
(IPC::id>>): Deleted.
(IPC::decodeObjectDirectlyRequiringAllowedClasses): Deleted.
(IPC::encodeObjectDirectly): Deleted.
(IPC::decodeObjectDirectlyRequiringAllowedClasses): Deleted.
* Source/WebKit/Shared/Cocoa/CoreIPCArray.h:
(WebKit::CoreIPCArray::CoreIPCArray):
(WebKit::CoreIPCArray::toID): Deleted.
* Source/WebKit/Shared/Cocoa/CoreIPCArray.mm: Copied from 
Source/WebKit/Shared/Cocoa/CoreIPCDictionary.h.
(WebKit::CoreIPCArray::CoreIPCArray):
(WebKit::CoreIPCArray::toID const):
* Source/WebKit/Shared/Cocoa/CoreIPCArray.serialization.in:
* Source/WebKit/Shared/Cocoa/CoreIPCDictionary.h:
(WebKit::CoreIPCDictionary::CoreIPCDictionary):
(WebKit::CoreIPCDictionary::toID): Deleted.
* Source/WebKit/Shared/Cocoa/CoreIPCDictionary.mm: Copied from 
Source/WebKit/Shared/Cocoa/CoreIPCNSCFObject.h.
(WebKit::CoreIPCDictionary::CoreIPCDictionary):
(WebKit::CoreIPCDictionary::toID const):
* Source/WebKit/Shared/Cocoa/CoreIPCDictionary.serialization.in:
* Source/WebKit/Shared/Cocoa/CoreIPCFont.h:
(WebKit::CoreIPCFont::CoreIPCFont):
(WebKit::CoreIPCFont::toID): Deleted.
* Source/WebKit/Shared/Cocoa/CoreIPCFont.mm: Copied from 
Source/WebKit/Shared/Cocoa/CoreIPCArray.h.
(WebKit::CoreIPCFont::toID const):
* Source/WebKit/Shared/Cocoa/CoreIPCFont.serialization.in:
* Source/WebKit/Shared/Cocoa/CoreIPCNSCFObject.h:
* Source/WebKit/Shared/Cocoa/CoreIPCNSCFObject.mm:
(WebKit::CoreIPCNSCFObject::valueIsAllowed):
* Source/WebKit/Shared/Cocoa/CoreIPCNSCFObject.serialization.in:
* Source/WebKit/Shared/Cocoa/CoreIPCSecureCoding.h:
(WebKit::CoreIPCSecureCoding::objectClass):
* Source/WebKit/WebKit.xcodeproj/project.pbxproj:
* Tools/TestWebKitAPI/Tests/IPC/IPCSerialization.mm:
(operator==):
(TEST):

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


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


[webkit-changes] [WebKit/WebKit] 53c6f5: [WebGPU] Queue::writeTexture is not implemented co...

2023-11-12 Thread mwyrzykowski
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 53c6f57a22b2c293c5645b3b51d94c6eae606a38
  
https://github.com/WebKit/WebKit/commit/53c6f57a22b2c293c5645b3b51d94c6eae606a38
  Author: Mike Wyrzykowski 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M Source/WebGPU/WebGPU/Queue.mm

  Log Message:
  ---
  [WebGPU] Queue::writeTexture is not implemented correctly for compressed 
texture formats
https://bugs.webkit.org/show_bug.cgi?id=263546
rdar://117368535

Reviewed by Dan Glastonbury.

webgpu:api,operation,command_buffer,* passes locally after
fixing this bug but the test times out run via WKTR so no
expectations are added.

* Source/WebGPU/WebGPU/Queue.mm:
(WebGPU::Queue::writeTexture):

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


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


[webkit-changes] [WebKit/WebKit] 41848d: Stop using Vector::unsafeAppendWithoutCapacityChec...

2023-11-12 Thread Chris Dumez
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 41848d931422aa7edaeead93a1c57d44609fe022
  
https://github.com/WebKit/WebKit/commit/41848d931422aa7edaeead93a1c57d44609fe022
  Author: Chris Dumez 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M Source/JavaScriptCore/wasm/WasmBBQJIT.cpp

  Log Message:
  ---
  Stop using Vector::unsafeAppendWithoutCapacityCheck() in WasmBBQJIT.cpp
https://bugs.webkit.org/show_bug.cgi?id=264646

Reviewed by Darin Adler.

* Source/JavaScriptCore/wasm/WasmBBQJIT.cpp:
(JSC::Wasm::BBQJIT::addSwitch):

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


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


[webkit-changes] [WebKit/WebKit] 6b87ac: [MediaStream] Add support for ImageCapture.takePhoto

2023-11-12 Thread Eric Carlson
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 6b87ac816d8acb2ee9016d88e7a5d639b19e5818
  
https://github.com/WebKit/WebKit/commit/6b87ac816d8acb2ee9016d88e7a5d639b19e5818
  Author: Eric Carlson 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
A LayoutTests/fast/mediastream/image-capture-take-photo-expected.txt
A LayoutTests/fast/mediastream/image-capture-take-photo.html
M Source/WebCore/Modules/mediastream/ImageCapture.cpp
M Source/WebCore/Modules/mediastream/ImageCapture.h
M Source/WebCore/Modules/mediastream/ImageCapture.idl
M Source/WebCore/Modules/mediastream/MediaStreamTrack.cpp
M Source/WebCore/Modules/mediastream/MediaStreamTrack.h
M Source/WebCore/PAL/pal/cocoa/AVFoundationSoftLink.h
M Source/WebCore/PAL/pal/cocoa/AVFoundationSoftLink.mm
M Source/WebCore/dom/TaskSource.h
M Source/WebCore/platform/mediastream/MediaStreamTrackPrivate.cpp
M Source/WebCore/platform/mediastream/MediaStreamTrackPrivate.h
M Source/WebCore/platform/mediastream/PhotoSettings.h
M Source/WebCore/platform/mediastream/RealtimeMediaSource.cpp
M Source/WebCore/platform/mediastream/RealtimeMediaSource.h
M Source/WebCore/platform/mediastream/RealtimeVideoCaptureSource.cpp
M Source/WebCore/platform/mediastream/RealtimeVideoCaptureSource.h
M 
Source/WebCore/platform/mediastream/gstreamer/MockRealtimeVideoSourceGStreamer.cpp
M Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.h
M Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.mm
M Source/WebCore/platform/mediastream/mac/MockRealtimeVideoSourceMac.mm
M Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp
M Source/WebCore/platform/mock/MockRealtimeVideoSource.h
M Source/WebKit/GPUProcess/webrtc/RemoteSampleBufferDisplayLayer.h
M Source/WebKit/UIProcess/Cocoa/UserMediaCaptureManagerProxy.cpp
M Source/WebKit/UIProcess/Cocoa/UserMediaCaptureManagerProxy.h
M Source/WebKit/UIProcess/Cocoa/UserMediaCaptureManagerProxy.messages.in
M Source/WebKit/WebProcess/cocoa/RemoteRealtimeMediaSource.cpp
M Source/WebKit/WebProcess/cocoa/RemoteRealtimeMediaSource.h
M Source/WebKit/WebProcess/cocoa/RemoteRealtimeMediaSourceProxy.cpp
M Source/WebKit/WebProcess/cocoa/RemoteRealtimeMediaSourceProxy.h

  Log Message:
  ---
  [MediaStream] Add support for ImageCapture.takePhoto
https://bugs.webkit.org/show_bug.cgi?id=262467
rdar://116322637

Reviewed by Youenn Fablet.

Implement ImageCapture.takePhoto in AVVideoCaptureSource and 
MockRealtimeVideoSource.
We reconfigure the capture source when asked for a photo larger than the 
current preset,
so it always returns an image at least as large as the requested size, but it 
doesn't
yet resize the image if the requested size doesn't exactly match the preset.

Taking a photo is an asynchronous operation that can take an arbitrary amount 
of time, so
we don't do it on the main thread. Because we may need to reconfigure the 
capture device
to take a photo with the requested settings, we need to prevent the capture 
device from
being reconfigured by `applyConstraints` while a photo is being generated. To 
do this we
serialize calls to `takePhoto` and `applyConstraints` so they don't interfere 
with one
another.

* LayoutTests/fast/mediastream/image-capture-take-photo-expected.txt: Added.
* LayoutTests/fast/mediastream/image-capture-take-photo.html: Added.
* Source/WebCore/Modules/mediastream/ImageCapture.cpp:
(WebCore::ImageCapture::takePhoto):
* Source/WebCore/Modules/mediastream/ImageCapture.h:
* Source/WebCore/Modules/mediastream/ImageCapture.idl:
* Source/WebCore/Modules/mediastream/MediaStreamTrack.cpp:
(WebCore::MediaStreamTrack::stopTrack):
(WebCore::MediaStreamTrack::queueAndProcessSerialAction):
(WebCore::MediaStreamTrack::processNextSerialAction):
(WebCore::MediaStreamTrack::takePhoto):
(WebCore::MediaStreamTrack::applyConstraints):
* Source/WebCore/Modules/mediastream/MediaStreamTrack.h:
* Source/WebCore/PAL/pal/cocoa/AVFoundationSoftLink.h:
* Source/WebCore/PAL/pal/cocoa/AVFoundationSoftLink.mm:
* Source/WebCore/platform/mediastream/MediaStreamTrackPrivate.cpp:
(WebCore::MediaStreamTrackPrivate::takePhoto):
* Source/WebCore/platform/mediastream/MediaStreamTrackPrivate.h:
* Source/WebCore/platform/mediastream/PhotoSettings.h:
(WebCore::operator==):
* Source/WebCore/platform/mediastream/RealtimeMediaSource.cpp:
(WebCore::RealtimeMediaSource::takePhoto):
* Source/WebCore/platform/mediastream/RealtimeMediaSource.h:
* Source/WebCore/platform/mediastream/RealtimeVideoCaptureSource.cpp:
(WebCore::RealtimeVideoCaptureSource::bestSupportedSizeFrameRateAndZoomConsideringObservers):
(WebCore::RealtimeVideoCaptureSource::setSizeFrameRateAndZoom):
(WebCore::RealtimeVideoCaptureSource::takePhotoInternal):
(WebCore::RealtimeVideoCaptureSource::takePhoto):
* Source/WebCore/platform/mediastream/RealtimeVideoCaptureSource.h:
* Source/WebCore/platform/mediastream/mac/AVVide

[webkit-changes] [WebKit/WebKit] a39795: [WebGPU] Expand metal capture to capture a number ...

2023-11-12 Thread mwyrzykowski
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: a3979565d4436eb9d167edb50629feb7cfd223e4
  
https://github.com/WebKit/WebKit/commit/a3979565d4436eb9d167edb50629feb7cfd223e4
  Author: Mike Wyrzykowski 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M Source/WebGPU/WebGPU/Device.h
M Source/WebGPU/WebGPU/Device.mm
M Source/WebGPU/WebGPU/Queue.mm

  Log Message:
  ---
  [WebGPU] Expand metal capture to capture a number of calls to submit
https://bugs.webkit.org/show_bug.cgi?id=264601
rdar://118237625

Reviewed by Dan Glastonbury.

Some websites may call GPUQueue.submit several times per frame so
to capture the entire list of submissions we need to allow the capture
to continue across multiple calls to GPUQueue.submit.

Allow specifying the state of CaptureFrame like so:
notifyutil -s com.apple.WebKit.WebGPU.CaptureFrame 10

to capture 10 submit calls.

* Source/WebGPU/WebGPU/Device.h:
* Source/WebGPU/WebGPU/Device.mm:
(WebGPU::GPUFrameCapture::registerForFrameCapture):
(WebGPU::GPUFrameCapture::shouldStopCaptureAfterSubmit):
(WebGPU::Device::shouldStopCaptureAfterSubmit):
* Source/WebGPU/WebGPU/Queue.mm:
(WebGPU::Queue::submit):

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


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


[webkit-changes] [WebKit/WebKit] c8e81a: [Cocoa] Fix position of AutoFill buttons in vertic...

2023-11-12 Thread Aditya Keerthi
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c8e81acd7ac4111a9cb94c4c037b8bf9d7c82c47
  
https://github.com/WebKit/WebKit/commit/c8e81acd7ac4111a9cb94c4c037b8bf9d7c82c47
  Author: Aditya Keerthi 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M Source/WebCore/css/html.css
M Source/WebCore/html/HTMLInputElement.cpp

  Log Message:
  ---
  [Cocoa] Fix position of AutoFill buttons in vertical writing mode
https://bugs.webkit.org/show_bug.cgi?id=264691
rdar://118265730

Reviewed by Tim Nguyen.

Use logical properties to ensure correct positioning in vertical writing mode.

* Source/WebCore/css/html.css:
(input::-webkit-strong-password-auto-fill-button):
(input::-webkit-credentials-auto-fill-button):
(input::-webkit-contacts-auto-fill-button):
(input::-webkit-credit-card-auto-fill-button):
(input::-webkit-loading-auto-fill-button):
* Source/WebCore/html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::createInnerTextStyle):

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


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


[webkit-changes] [WebKit/WebKit] c9f3a6: Stop using Vector::unsafeAppendWithoutCapacityChec...

2023-11-12 Thread Chris Dumez
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c9f3a660aa34d9f8b65b2aada77a27574a9a642e
  
https://github.com/WebKit/WebKit/commit/c9f3a660aa34d9f8b65b2aada77a27574a9a642e
  Author: Chris Dumez 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M Source/JavaScriptCore/wasm/WasmFunctionParser.h
M Source/WTF/wtf/Vector.h

  Log Message:
  ---
  Stop using Vector::unsafeAppendWithoutCapacityCheck() in WasmFunctionParser's 
parse()
https://bugs.webkit.org/show_bug.cgi?id=264652

Reviewed by Darin Adler.

* Source/JavaScriptCore/wasm/WasmFunctionParser.h:
(JSC::Wasm::FunctionParser::parse):
* Source/WTF/wtf/Vector.h:
(WTF::Malloc>::appendUsingFunctor):

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


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


[webkit-changes] [WebKit/WebKit] b1c357: Use dynamicDowncast more in the DOM

2023-11-12 Thread Chris Dumez
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b1c357dd6e0f3eede4100c8152dc5be930a2b55d
  
https://github.com/WebKit/WebKit/commit/b1c357dd6e0f3eede4100c8152dc5be930a2b55d
  Author: Chris Dumez 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M Source/WebCore/dom/ActiveDOMObject.cpp
M Source/WebCore/dom/ComposedTreeIterator.cpp
M Source/WebCore/dom/ContainerNode.cpp
M Source/WebCore/dom/ContainerNodeAlgorithms.cpp
M Source/WebCore/dom/CustomElementRegistry.cpp
M Source/WebCore/dom/DataTransfer.cpp
M Source/WebCore/dom/Document.cpp
M Source/WebCore/dom/DocumentTouch.cpp
M Source/WebCore/dom/Element.cpp

  Log Message:
  ---
  Use dynamicDowncast more in the DOM
https://bugs.webkit.org/show_bug.cgi?id=264695

Reviewed by Ryosuke Niwa.

Use dynamicDowncast more in the DOM instead of is() + downcast(). It is
less error-prone and often results in more concise code. I am also hoping to
have downcast<>() do a type check on release builds in the future. It is
currently too expensive to do so but it may become affordable if we use
dynamicDowncast() instead when possible.

* Source/WebCore/dom/ActiveDOMObject.cpp:
(WebCore::suitableScriptExecutionContext):
* Source/WebCore/dom/ComposedTreeIterator.cpp:
(WebCore::ComposedTreeIterator::ComposedTreeIterator):
(WebCore::ComposedTreeIterator::initializeContextStack):
(WebCore::ComposedTreeIterator::traverseNextInShadowTree):
* Source/WebCore/dom/ContainerNode.cpp:
(WebCore::makeChildChangeForInsertion):
(WebCore::ContainerNode::cloneChildNodes):
* Source/WebCore/dom/ContainerNodeAlgorithms.cpp:
(WebCore::notifyNodeInsertedIntoDocument):
(WebCore::notifyNodeInsertedIntoTree):
(WebCore::notifyNodeRemovedFromDocument):
(WebCore::notifyNodeRemovedFromTree):
(WebCore::assertConnectedSubrameCountIsConsistent):
(WebCore::collectFrameOwners):
(WebCore::disconnectSubframes):
* Source/WebCore/dom/CustomElementRegistry.cpp:
(WebCore::CustomElementRegistry::upgrade):
* Source/WebCore/dom/DataTransfer.cpp:
(WebCore::DataTransfer::setDragImage):
* Source/WebCore/dom/Document.cpp:
(WebCore::Document::doctype const):
(WebCore::Document::childrenChanged):
(WebCore::Document::adoptNode):
(WebCore::selectNewTitleElement):
(WebCore::Document::updateLayoutIfDimensionsOutOfDate):
(WebCore::Document::willBeRemovedFromFrame):
(WebCore::Document::setFocusedElement):
(WebCore::Document::focusNavigationStartingNode const):
(WebCore::Document::getCSSCanvasContext):
(WebCore::Document::serviceRequestVideoFrameCallbacks):
(WebCore::Document::absoluteEventRegionForNode):
(WebCore::eventTargetElementForDocument):
(WebCore::Document::updateHoverActiveState):
(WebCore::Document::dir const):
(WebCore::Document::setDir):
(WebCore::Document::keyframesRuleDidChange):
(WebCore::Document::addTopLayerElement):
(WebCore::Document::activeModalDialog const):
(WebCore::Document::handlePopoverLightDismiss):
(WebCore::Document::canvasChanged):
(WebCore::Document::canvasDestroyed):
* Source/WebCore/dom/DocumentTouch.cpp:
(WebCore::DocumentTouch::createTouch):
(WebCore::DocumentTouch::createTouchList): Deleted.
* Source/WebCore/dom/Element.cpp:
(WebCore::Element::cloneShadowTreeIfPossible):
(WebCore::Element::synchronizeAllAttributes const):
(WebCore::Element::synchronizeAttribute const):
(WebCore::listBoxElementScrollIntoView):
(WebCore::Element::absoluteEventBounds):
(WebCore::listBoxElementBoundingBox):
(WebCore::Element::removeAttribute):
(WebCore::Element::setOuterHTML):
(WebCore::Element::setInnerHTML):
(WebCore::forEachRenderLayer):
(WebCore::Element::resolveComputedStyle):
(WebCore::Element::childShouldCreateRenderer const):
(WebCore::Element::fastAttributeLookupAllowed const):
(WebCore::Element::updateName):
(WebCore::Element::updateId):
(WebCore::Element::cloneAttributesFromElement):
(WebCore::contextElementForInsertion):

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


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


[webkit-changes] [WebKit/WebKit] 5de63d: [WinCairo] Unreviewed test gardening

2023-11-12 Thread Fujii Hironori
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 5de63dbccbc45a3b261bc6ac4a5be1da5e33663f
  
https://github.com/WebKit/WebKit/commit/5de63dbccbc45a3b261bc6ac4a5be1da5e33663f
  Author: Fujii Hironori 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M LayoutTests/platform/wincairo/TestExpectations

  Log Message:
  ---
  [WinCairo] Unreviewed test gardening

* LayoutTests/platform/wincairo/TestExpectations:

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


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


[webkit-changes] [WebKit/WebKit] f739f8: Deploy more smart pointers in ScriptExecutionConte...

2023-11-12 Thread Ryosuke Niwa
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: f739f845a11b1ba2b438d9e9b31e346f7cdc2f46
  
https://github.com/WebKit/WebKit/commit/f739f845a11b1ba2b438d9e9b31e346f7cdc2f46
  Author: Ryosuke Niwa 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M Source/WebCore/dom/RejectedPromiseTracker.cpp
M Source/WebCore/dom/ScriptExecutionContext.cpp
M Source/WebCore/dom/ScriptExecutionContext.h
M Source/WebCore/dom/ShadowRoot.cpp

  Log Message:
  ---
  Deploy more smart pointers in ScriptExecutionContext.cpp and 
RejectedPromiseTracker.cpp
https://bugs.webkit.org/show_bug.cgi?id=264621

Reviewed by Chris Dumez.

Deploy smart pointers as warned by the clang static analyzer.

* Source/WebCore/dom/RejectedPromiseTracker.cpp:
(WebCore::RejectedPromiseTracker::reportUnhandledRejections):
(WebCore::RejectedPromiseTracker::reportRejectionHandled):
* Source/WebCore/dom/ScriptExecutionContext.cpp:
(WebCore::ScriptExecutionContext::reportUnhandledPromiseRejection):
(WebCore::ScriptExecutionContext::dispatchErrorEvent):
(WebCore::ScriptExecutionContext::protectedVM): Added.
(WebCore::ScriptExecutionContext::ensureRejectedPromiseTrackerSlow):
* Source/WebCore/dom/ScriptExecutionContext.h:
* Source/WebCore/dom/ShadowRoot.cpp:
(WebCore::ShadowRoot::~ShadowRoot):

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


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


[webkit-changes] [WebKit/WebKit] a1ad99: [Site Isolation][macOS] Propagate drag events to i...

2023-11-12 Thread Charlie Wolfe
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: a1ad995cedf051ec5c280355b0a28696cdbba5d3
  
https://github.com/WebKit/WebKit/commit/a1ad995cedf051ec5c280355b0a28696cdbba5d3
  Author: Charlie Wolfe 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M Source/WebCore/Headers.cmake
M Source/WebCore/WebCore.xcodeproj/project.pbxproj
M Source/WebCore/loader/EmptyClients.cpp
M Source/WebCore/page/DragClient.h
M Source/WebCore/page/DragController.cpp
M Source/WebCore/page/DragController.h
M Source/WebCore/page/EventHandler.cpp
M Source/WebCore/page/EventHandler.h
M Source/WebCore/page/HandleMouseEventResult.h
R Source/WebCore/page/RemoteMouseEventData.h
A Source/WebCore/page/RemoteUserInputEventData.h
M Source/WebCore/platform/DragData.h
M Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in
M Source/WebKit/UIProcess/Cocoa/WebPasteboardProxyCocoa.mm
M Source/WebKit/UIProcess/RemotePageProxy.cpp
M Source/WebKit/UIProcess/WebPageProxy.cpp
M Source/WebKit/UIProcess/WebPageProxy.h
M Source/WebKit/UIProcess/WebProcessProxy.cpp
M Source/WebKit/UIProcess/WebProcessProxy.h
M Source/WebKit/WebProcess/WebCoreSupport/WebDragClient.cpp
M Source/WebKit/WebProcess/WebCoreSupport/WebDragClient.h
M Source/WebKit/WebProcess/WebCoreSupport/gtk/WebDragClientGtk.cpp
M Source/WebKit/WebProcess/WebCoreSupport/mac/WebDragClientMac.mm
M Source/WebKit/WebProcess/WebPage/WebFrame.cpp
M Source/WebKit/WebProcess/WebPage/WebPage.cpp
M Source/WebKit/WebProcess/WebPage/WebPage.h
M Source/WebKit/WebProcess/WebPage/WebPage.messages.in
M Source/WebKitLegacy/mac/WebCoreSupport/WebDragClient.h
M Source/WebKitLegacy/mac/WebCoreSupport/WebDragClient.mm
M Source/WebKitLegacy/mac/WebView/WebFrame.mm
M Source/WebKitLegacy/mac/WebView/WebView.mm
M Tools/TestWebKitAPI/Tests/WebKitCocoa/SiteIsolation.mm

  Log Message:
  ---
  [Site Isolation][macOS] Propagate drag events to isolated frames
https://bugs.webkit.org/show_bug.cgi?id=264502
rdar://118184993

Reviewed by Wenson Hsieh.

This follows the same pattern as mouse events, where IPC is sent between the UI 
process and web
processes until the target frame is reached.

* Source/WebCore/Headers.cmake:
* Source/WebCore/WebCore.xcodeproj/project.pbxproj:
* Source/WebCore/loader/EmptyClients.cpp:
* Source/WebCore/page/DragClient.h:

* Source/WebCore/page/DragController.cpp:
(WebCore::DragController::dragExited):
(WebCore::DragController::dragEnteredOrUpdated):
(WebCore::DragController::tryDocumentDrag):
(WebCore::DragController::tryDHTMLDrag):
(WebCore::DragController::doSystemDrag):
(WebCore::DragController::dragEntered): Deleted.
(WebCore::DragController::dragUpdated): Deleted.
* Source/WebCore/page/DragController.h:

There were several places in `DragController` where drag functions would always 
be called on the main
frame’s event handler. This won’t work when the main frame is being hosted in 
another process. I
changed these functions to pass the frame which the drag is happening in, and 
instead call functions
on the event handler of that frame.

There were also places where we were down casting frames to local when we 
didn’t need to. In
`doSystemDrag()` and `tryDHTMLDrag()` all functions called on the main frame 
and main frame view are
accessible even if the frame is out-of-process.

`dragEntered()` and `dragUpdated()` didn’t seem like a useful abstraction, so I 
removed them.

* Source/WebCore/page/EventHandler.cpp:
(WebCore::EventHandler::mouseEventDataForRemoteFrame):
(WebCore::EventHandler::dragSourceEndedAt):
* Source/WebCore/page/EventHandler.h:

I changed `dragSourceEndedAt()` to return a `FrameIdentifier` if the hit test 
leads to a remote frame.

* Source/WebCore/page/HandleMouseEventResult.h:
(WebCore::HandleMouseEventResult::HandleMouseEventResult):
(WebCore::HandleMouseEventResult::remoteInputEventData):
(WebCore::HandleMouseEventResult::remoteMouseEventData): Deleted.
* Source/WebCore/page/RemoteUserInputEventData.h: Renamed from 
Source/WebCore/page/RemoteMouseEventData.h.

I renamed `RemoteMouseEventData` to `RemoteUserInputEventData` since we can use 
the struct for things other
than mouse events. I used it here for drag events, and can likely be used for 
touch events too.

* Source/WebCore/platform/DragData.h:
(WebCore::DragData::setClientPosition):

I added `setClientPosition` to update the client positions coordinates on 
`DragData` when passing a drag
event to a remote frame.

* Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in:
* Source/WebKit/UIProcess/Cocoa/WebPasteboardProxyCocoa.mm:
(WebKit::WebPasteboardProxy::determineDataOwner const):

When dragging in a isolated frame, each of the 
`MESSAGE_CHECK_COMPLETION(dataOwner, completionHandler())`
would fail. This is because `determineDataOwner()` is only checking for a 
`WebPageProxy` associated with
the `WebProcessProxy`, which may n

[webkit-changes] [WebKit/WebKit] d53d6d: Move enums under WebCore/rendering/ to use the ser...

2023-11-12 Thread Mikhail R. Gadelha
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: d53d6d23b888ad7f49bd058a9d8e233983e40f07
  
https://github.com/WebKit/WebKit/commit/d53d6d23b888ad7f49bd058a9d8e233983e40f07
  Author: Mikhail R. Gadelha 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M Source/WebCore/page/LocalFrameView.cpp
M Source/WebCore/page/Page.cpp
M Source/WebCore/rendering/Pagination.h
M Source/WebCore/rendering/RenderBlockFlow.cpp
M Source/WebCore/rendering/RenderObject.h
M Source/WebCore/rendering/RenderView.cpp
M Source/WebCore/rendering/style/RenderStyle.cpp
M Source/WebCore/rendering/style/RenderStyle.h
M Source/WebCore/rendering/svg/legacy/LegacyRenderSVGForeignObject.cpp
M Source/WebCore/rendering/svg/legacy/LegacyRenderSVGResource.h
M Source/WebCore/rendering/svg/legacy/LegacyRenderSVGResourceClipper.h
M Source/WebCore/rendering/svg/legacy/LegacyRenderSVGResourceMasker.h
M Source/WebCore/style/StyleResolveForDocument.cpp
M Source/WebCore/testing/Internals.cpp
M Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in
M Source/WebKit/UIProcess/API/C/WKPage.cpp
M Source/WebKit/UIProcess/API/Cocoa/WKBrowsingContextController.mm
M Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm
M Source/WebKit/UIProcess/WebPageProxy.h
M Source/WebKit/WebProcess/WebPage/WebPage.cpp
M Source/WebKit/WebProcess/WebPage/WebPage.h
M Source/WebKit/WebProcess/WebPage/WebPage.messages.in
M Source/WebKitLegacy/mac/WebView/WebView.mm

  Log Message:
  ---
  Move enums under WebCore/rendering/ to use the serializer
https://bugs.webkit.org/show_bug.cgi?id=264623

Reviewed by Chris Dumez.

This patch covers the enums in Pagination.h (WebCore::PaginationMode) and
in RenderObject.h (WebCore::RepaintRectCalculation).

The patch mostly just adds the necessary scopes given that the enums now
are enum classes. The only exception is the EnumeratedArray in the
rendering/svg/legacy/ headers, which needed an extra argument, the last
value in the enum; without it EnumeratedArray tries to call
EnumTraits::values::max> and fails

* Source/WebCore/page/LocalFrameView.cpp:
(WebCore::paginationModeForRenderStyle):
* Source/WebCore/page/Page.cpp:
(WebCore::Page::pageCount const):
(WebCore::Page::pageCountAssumingLayoutIsUpToDate const):
* Source/WebCore/rendering/Pagination.h:
* Source/WebCore/rendering/RenderBlockFlow.cpp:
(WebCore::RenderBlockFlow::checkForPaginationLogicalHeightChange):
* Source/WebCore/rendering/RenderObject.h:
* Source/WebCore/rendering/RenderView.cpp:
(WebCore::RenderView::styleDidChange):
(WebCore::RenderView::requiresColumns const):
(WebCore::RenderView::paint):
(WebCore::RenderView::pageNumberForBlockProgressionOffset const):
(WebCore::RenderView::pageCount const):
* Source/WebCore/rendering/style/RenderStyle.cpp:
(WebCore::RenderStyle::setColumnStylesFromPaginationMode):
* Source/WebCore/rendering/style/RenderStyle.h:
* Source/WebCore/rendering/svg/legacy/LegacyRenderSVGForeignObject.cpp:
* Source/WebCore/rendering/svg/legacy/LegacyRenderSVGResource.h:
* Source/WebCore/rendering/svg/legacy/LegacyRenderSVGResourceClipper.h:
* Source/WebCore/rendering/svg/legacy/LegacyRenderSVGResourceMasker.h:
* Source/WebCore/style/StyleResolveForDocument.cpp:
(WebCore::Style::resolveForDocument):
* Source/WebCore/testing/Internals.cpp:
(WebCore::Internals::setPagination):
* Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in:
* Source/WebKit/UIProcess/API/C/WKPage.cpp:
(WKPageSetPaginationMode):
(WKPageGetPaginationMode):
* Source/WebKit/UIProcess/WebPageProxy.h:
* Source/WebKit/WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::setPaginationMode):
* Source/WebKit/WebProcess/WebPage/WebPage.h:
* Source/WebKit/WebProcess/WebPage/WebPage.messages.in:

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


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


[webkit-changes] [WebKit/WebKit] 723ded: [WPE][GTK][Debug] Crashes in imported/w3c/web-plat...

2023-11-12 Thread Philippe Normand
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 723ded32d95eedc80be4ba1b3bc66c5e245da795
  
https://github.com/WebKit/WebKit/commit/723ded32d95eedc80be4ba1b3bc66c5e245da795
  Author: Philippe Normand 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M Source/WebCore/Modules/reporting/ReportingScope.cpp

  Log Message:
  ---
  [WPE][GTK][Debug] Crashes in imported/w3c/web-platform-tests/reporting
https://bugs.webkit.org/show_bug.cgi?id=264709

Reviewed by Michael Catanzaro.

The C++ standard doesn't define an order of evaluation for function arguments, 
so without forcing it
when creating the report we might trigger a use-after-move. This was specially 
noticeable on GCC
builds, not so much with clang.

* Source/WebCore/Modules/reporting/ReportingScope.cpp:
(WebCore::ReportingScope::generateTestReport):

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


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


[webkit-changes] [WebKit/WebKit] 0c4764: [GStreamer][MSE][Debug] media/media-source/media-s...

2023-11-12 Thread Philippe Normand
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 0c47646d8429be663031e151df659552e75d73ae
  
https://github.com/WebKit/WebKit/commit/0c47646d8429be663031e151df659552e75d73ae
  Author: Philippe Normand 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M LayoutTests/platform/glib/TestExpectations

  Log Message:
  ---
  [GStreamer][MSE][Debug] media/media-source/media-source-error-crash.html is 
flaky crash
https://bugs.webkit.org/show_bug.cgi?id=264708

Unreviewed, garden the test.

* LayoutTests/platform/glib/TestExpectations:

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


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


[webkit-changes] [WebKit/WebKit] 6288cd: [WPE][GTK] Layout tests gardening

2023-11-12 Thread Philippe Normand
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 6288cd3be4b3b00f6d8ef08e9047eedb7cd9096f
  
https://github.com/WebKit/WebKit/commit/6288cd3be4b3b00f6d8ef08e9047eedb7cd9096f
  Author: Philippe Normand 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M LayoutTests/platform/glib/TestExpectations
A 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/css/cssom-view/range-bounding-client-rect-with-display-contents-expected.txt
M LayoutTests/platform/gtk/TestExpectations

  Log Message:
  ---
  [WPE][GTK] Layout tests gardening
https://bugs.webkit.org/show_bug.cgi?id=264701

Unreviewed, update of test expectations for WPE/GTK ports.

* LayoutTests/platform/glib/TestExpectations:
* 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/css/cssom-view/range-bounding-client-rect-with-display-contents-expected.txt:
 Added.

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


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


[webkit-changes] [WebKit/WebKit] 6e975c: [WPE][Debug] Garden more scrolling-related assert ...

2023-11-12 Thread Claudio Saavedra
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 6e975c79b0d4766d80b0421a6dedeae64ca84b63
  
https://github.com/WebKit/WebKit/commit/6e975c79b0d4766d80b0421a6dedeae64ca84b63
  Author: Claudio Saavedra 
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
M LayoutTests/platform/wpe/TestExpectations

  Log Message:
  ---
  [WPE][Debug] Garden more scrolling-related assert crashes
https://bugs.webkit.org/show_bug.cgi?id=264697

Unreviewed gardening.

* LayoutTests/platform/wpe/TestExpectations:

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


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