[webkit-changes] [WebKit/WebKit] e6d5e1: [WebGPU] Index buffer can read outside the range o...

2024-05-22 Thread mwyrzykowski
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: e6d5e1a6604e5aa2b7786e3f40859274b6353c90
  
https://github.com/WebKit/WebKit/commit/e6d5e1a6604e5aa2b7786e3f40859274b6353c90
  Author: Mike Wyrzykowski 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebGPU/WebGPU/BindableResource.h
M Source/WebGPU/WebGPU/Buffer.h
M Source/WebGPU/WebGPU/Buffer.mm
M Source/WebGPU/WebGPU/CommandEncoder.h
M Source/WebGPU/WebGPU/CommandEncoder.mm
M Source/WebGPU/WebGPU/ComputePassEncoder.mm
M Source/WebGPU/WebGPU/Device.h
M Source/WebGPU/WebGPU/Device.mm
M Source/WebGPU/WebGPU/Pipeline.mm
M Source/WebGPU/WebGPU/RenderBundleEncoder.h
M Source/WebGPU/WebGPU/RenderBundleEncoder.mm
M Source/WebGPU/WebGPU/RenderPassEncoder.h
M Source/WebGPU/WebGPU/RenderPassEncoder.mm

  Log Message:
  ---
  [WebGPU] Index buffer can read outside the range of a vertex buffer
https://bugs.webkit.org/show_bug.cgi?id=273828


Reviewed by Tadeu Zagallo.

Prevent out of bounds accesses into vertex buffers via drawIndexed,
drawIndirect, and drawIndexedIndirect by running a non-rasterizing
vertex shader immedietly before the draw call.

ICB path rewrites the ICB render commands to avoid out of bounds
accesses.

* Source/WebGPU/WebGPU/BindableResource.h:
* Source/WebGPU/WebGPU/Buffer.h:
* Source/WebGPU/WebGPU/Buffer.mm:
(WebGPU::Buffer::Buffer):
(WebGPU::Buffer::maxIndex const):
(WebGPU::Buffer::indirectBuffer const):
(WebGPU::Buffer::indirectIndexedBuffer const):
(WebGPU::Buffer::indirectBufferRequiresRecomputation const):
(WebGPU::Buffer::indirectBufferRecomputed):
(WebGPU::Buffer::indirectBufferInvalidated):
(WebGPU::Buffer::recomputeMaxIndexValues const): Deleted.
* Source/WebGPU/WebGPU/CommandEncoder.h:
* Source/WebGPU/WebGPU/CommandEncoder.mm:
(WebGPU::CommandEncoder::copyBufferToBuffer):
(WebGPU::CommandEncoder::copyTextureToBuffer):
(WebGPU::CommandEncoder::clearBuffer):
* Source/WebGPU/WebGPU/ComputePassEncoder.mm:
(WebGPU::addResourceToActiveResources):
(WebGPU::ComputePassEncoder::runPredispatchIndirectCallValidation):
(WebGPU::setCommandEncoder):
* Source/WebGPU/WebGPU/Device.h:
* Source/WebGPU/WebGPU/Device.mm:
(WebGPU::GPUFrameCapture::captureFrame):
(WebGPU::Device::dispatchCallBuffer):
(WebGPU::Device::dispatchCallPipelineState):
(WebGPU::Device::copyIndexIndirectArgsPipeline):
(wgpuDeviceReference): Deleted.
(wgpuDeviceRelease): Deleted.
(wgpuDeviceCreateBindGroup): Deleted.
(wgpuDeviceCreateBindGroupLayout): Deleted.
(wgpuDeviceCreateBuffer): Deleted.
(wgpuDeviceCreateCommandEncoder): Deleted.
(wgpuDeviceCreateComputePipeline): Deleted.
(wgpuDeviceCreateComputePipelineAsync): Deleted.
(wgpuDeviceCreateComputePipelineAsyncWithBlock): Deleted.
(wgpuDeviceCreatePipelineLayout): Deleted.
(wgpuDeviceCreateQuerySet): Deleted.
(wgpuDeviceCreateRenderBundleEncoder): Deleted.
(wgpuDeviceCreateRenderPipeline): Deleted.
(wgpuDeviceCreateRenderPipelineAsync): Deleted.
(wgpuDeviceCreateRenderPipelineAsyncWithBlock): Deleted.
(wgpuDeviceCreateSampler): Deleted.
(wgpuDeviceImportExternalTexture): Deleted.
(wgpuDeviceCreateShaderModule): Deleted.
(wgpuDeviceCreateSwapChain): Deleted.
(wgpuDeviceCreateTexture): Deleted.
(wgpuDeviceDestroy): Deleted.
(wgpuDeviceEnumerateFeatures): Deleted.
(wgpuDeviceGetLimits): Deleted.
(wgpuDeviceGetQueue): Deleted.
(wgpuDeviceHasFeature): Deleted.
(wgpuDevicePopErrorScope): Deleted.
(wgpuDevicePopErrorScopeWithBlock): Deleted.
(wgpuDevicePushErrorScope): Deleted.
(wgpuDeviceSetDeviceLostCallback): Deleted.
(wgpuDeviceSetDeviceLostCallbackWithBlock): Deleted.
(wgpuDeviceSetUncapturedErrorCallback): Deleted.
(wgpuDeviceSetUncapturedErrorCallbackWithBlock): Deleted.
(wgpuDeviceSetLabel): Deleted.
* Source/WebGPU/WebGPU/Pipeline.mm:
(WebGPU::validateBindGroup):
* Source/WebGPU/WebGPU/RenderBundleEncoder.h:
* Source/WebGPU/WebGPU/RenderBundleEncoder.mm:
(-[RenderBundleICBWithResources 
initWithICB:containerBuffer:pipelineState:depthStencilState:cullMode:frontFace:depthClipMode:depthBias:depthBiasSlopeScale:depthBiasClamp:fragmentDynamicOffsetsBuffer:pipeline:]):
(-[RenderBundleICBWithResources minVertexCountForDrawCommand]):
(WebGPU::makeRenderBundleICBWithResources):
(WebGPU::RenderBundleEncoder::addResource):
(WebGPU::RenderBundleEncoder::computeMininumVertexCount const):
(WebGPU::RenderBundleEncoder::storeVertexBufferCountsForValidation):
(WebGPU::RenderBundleEncoder::drawIndexed):
(WebGPU::RenderBundleEncoder::drawIndexedIndirect):
(WebGPU::RenderBundleEncoder::drawIndirect):
(WebGPU::RenderBundleEncoder::endCurrentICB):
(WebGPU::RenderBundleEncoder::setIndexBuffer):
(WebGPU::RenderBundleEncoder::setVertexBuffer):
(-[RenderBundleICBWithResources 
initWithICB:pipelineState:depthStencilState:cullMode:frontFace:depthClipMode:depthBias:depthBiasSlopeScale:depthBiasClamp:fragmentDynamicOffsetsBuffer:pipeline:]):
 Deleted.
* Source/WebGPU/WebGPU/RenderPassEncoder.h:
* Source/WebGPU/WebGPU/RenderPassEncoder.mm:

[webkit-changes] [WebKit/WebKit] 2b5ed0: [JSC] ArrayBuffer/SharedArrayBuffer constructor sh...

2024-05-22 Thread SUZUKI Sosuke
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 2b5ed009bee397beb4099f6ea0e40072d2aaec0f
  
https://github.com/WebKit/WebKit/commit/2b5ed009bee397beb4099f6ea0e40072d2aaec0f
  Author: Sosuke Suzuki 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M JSTests/test262/expectations.yaml
M Source/JavaScriptCore/runtime/JSArrayBufferConstructor.cpp

  Log Message:
  ---
  [JSC] ArrayBuffer/SharedArrayBuffer constructor should check length before 
creating an instance
https://bugs.webkit.org/show_bug.cgi?id=272809

Reviewed by Darin Adler.

According to the specs[1][2], the constructors for ArrayBuffer and 
SharedArrayBuffer should check
if `byteLength > maxByteLength` and throw a `RangeError` before creating an 
instance.

This patch changes to perform these checks before creating an instance.

`toTypedArrayIndex`, there is an observable side effect of throwing a 
`RangeError` when `length`
exceeds `MAX_ARRAY_BUFFER_SIZE`. So, this patch use `toNumber` instead when 
checking
`byteLength > maxByteLength`.

[1]: https://tc39.es/ecma262/#sec-allocatearraybuffer
[2]: https://tc39.es/ecma262/#sec-allocatesharedarraybuffer

* JSTests/test262/expectations.yaml:
* Source/JavaScriptCore/runtime/JSArrayBufferConstructor.cpp:
(JSC::JSGenericArrayBufferConstructor::constructImpl):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] d7bdc4: [JSC] Use `var` instead of `let` in built-in `Iter...

2024-05-22 Thread SUZUKI Sosuke
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: d7bdc4ae05980d538508f43938c1e1ee21b844bd
  
https://github.com/WebKit/WebKit/commit/d7bdc4ae05980d538508f43938c1e1ee21b844bd
  Author: Sosuke Suzuki 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/JavaScriptCore/builtins/IteratorHelpers.js

  Log Message:
  ---
  [JSC] Use `var` instead of `let` in built-in `IteratorHelpers.js`
https://bugs.webkit.org/show_bug.cgi?id=274453

Reviewed by Alexey Shvayka.

In the built-in JS files, `var` was adopted instead of `let` / `const` by 
219251@main[1]. However,
in the functions in `IteratorHelpers.js` added by 250474@main[2], `let` is used.

This patch changes to use `var` instead of `let`.

[1]: https://commits.webkit.org/219251@main
[2]: https://commits.webkit.org/250474@main

* Source/JavaScriptCore/builtins/IteratorHelpers.js:
(set linkTimeConstant.builtinMapIterable):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] fe5796: Fixed Playstation build break after 279166@main

2024-05-22 Thread jigenzhou
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: fe5796bc5acc530a367de9ddeee7c993d3bf2edd
  
https://github.com/WebKit/WebKit/commit/fe5796bc5acc530a367de9ddeee7c993d3bf2edd
  Author: Jigen Zhou 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/platform/graphics/nicosia/NicosiaImageBufferPipe.cpp

  Log Message:
  ---
  Fixed Playstation build break after 279166@main
https://bugs.webkit.org/show_bug.cgi?id=274566

Reviewed by Ross Kirsling.

Only allow using 'fence' if PLATFORM(GTK) || PLATFORM(WPE)

* Source/WebCore/platform/graphics/nicosia/NicosiaImageBufferPipe.cpp:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 9b7b8d: [Remote Inspection] Add the ability to target elem...

2024-05-22 Thread Wenson Hsieh
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 9b7b8ded8fa6f6aa85d72ec5f68bee3252c00a81
  
https://github.com/WebKit/WebKit/commit/9b7b8ded8fa6f6aa85d72ec5f68bee3252c00a81
  Author: Wenson Hsieh 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/page/ElementTargetingController.cpp
M Source/WebCore/page/ElementTargetingController.h
M Source/WebCore/page/ElementTargetingTypes.h
M Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in
M Source/WebKit/UIProcess/API/APITargetedElementInfo.h
M Source/WebKit/UIProcess/API/APITargetedElementRequest.cpp
M Source/WebKit/UIProcess/API/APITargetedElementRequest.h
M Source/WebKit/UIProcess/API/Cocoa/_WKTargetedElementInfo.h
M Source/WebKit/UIProcess/API/Cocoa/_WKTargetedElementInfo.mm
M Source/WebKit/UIProcess/API/Cocoa/_WKTargetedElementRequest.h
M Source/WebKit/UIProcess/API/Cocoa/_WKTargetedElementRequest.mm
M Tools/TestWebKitAPI/Tests/WebKitCocoa/ElementTargetingTests.mm

  Log Message:
  ---
  [Remote Inspection] Add the ability to target elements based on selector 
information
https://bugs.webkit.org/show_bug.cgi?id=274532

Reviewed by Aditya Keerthi.

Add the ability to target elements based on compound selectors (i.e. selectors 
including all shadow
hosts). Additionally, expose a new property on targeted element info to 
indicate whether or not the
element is inside of an adjustment subtree.

See below for more details.

Test: ElementTargeting.RequestElementsFromSelectors

* Source/WebCore/page/ElementTargetingController.cpp:
(WebCore::targetedElementInfo):
(WebCore::ElementTargetingController::findTargets):
(WebCore::ElementTargetingController::findNodes):

Add `WebCore::TargetedElementSelectors` as a new type of targeted element 
request data, alongside
a point in root view coordinates and a search string. Add a new `findNodes()` 
implementation for
this targeted element selectors list. Build on top of the 
`findElementFromSelectors` method (see
below for more details).

(WebCore::ElementTargetingController::extractTargets):

Drive-by adjustment: only target elements if they're non-empty. This can 
otherwise happen in the
case where the list of candidates was derived from ascending the DOM when 
performing a text search.

(WebCore::ElementTargetingController::adjustVisibilityInRepeatedlyTargetedRegions):
(WebCore::resolveSelectorToQuery):

Pull this out into a separate static helper function, so we can use it from 
both methods below.

(WebCore::ElementTargetingController::applyVisibilityAdjustmentFromSelectors):

Pull logic out of this method, and into `findElementFromSelectors`. Also, 
remove the document
argument (which was only ever the main document), and instead just retrieve the 
main document from
the `m_page` (or exit early if it's null).

(WebCore::ElementTargetingController::findElementFromSelectors):

Factor out logic for mapping a list of compound selectors (including all shadow 
hosts) into a
separate private method, so that it can be reused in multiple places in this 
class.

(WebCore::ElementTargetingController::mainDocument const):

Avoid some code duplication by adding a private helper method to return the 
main document, through
`m_page`.

(WebCore::ElementTargetingController::selectorBasedVisibilityAdjustmentTimerFired):
* Source/WebCore/page/ElementTargetingController.h:
* Source/WebCore/page/ElementTargetingTypes.h:
* Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in:

Add the new IPC types.

* Source/WebKit/UIProcess/API/APITargetedElementInfo.h:
* Source/WebKit/UIProcess/API/APITargetedElementRequest.cpp:
(API::TargetedElementRequest::setSelectors):
(API::TargetedElementRequest::searchText const): Deleted.

Remove an unused getter for `-searchText`, along with support in the API object.

* Source/WebKit/UIProcess/API/APITargetedElementRequest.h:
* Source/WebKit/UIProcess/API/Cocoa/_WKTargetedElementInfo.h:
* Source/WebKit/UIProcess/API/Cocoa/_WKTargetedElementInfo.mm:
(-[_WKTargetedElementInfo isInVisibilityAdjustmentSubtree]):
* Source/WebKit/UIProcess/API/Cocoa/_WKTargetedElementRequest.h:
* Source/WebKit/UIProcess/API/Cocoa/_WKTargetedElementRequest.mm:
(-[_WKTargetedElementRequest initWithSelectors:]):

Add a new initializer that takes a compound selector representing the element 
to target.

(-[_WKTargetedElementRequest searchText]): Deleted.
(-[_WKTargetedElementRequest point]): Deleted.
* Tools/TestWebKitAPI/Tests/WebKitCocoa/ElementTargetingTests.mm:
(-[WKWebView targetedElementInfoWithSelectors:]):
(TestWebKitAPI::TEST(ElementTargeting, RequestElementsFromSelectors)):

Add a new API test to exercise the change.

(TestWebKitAPI::TEST(ElementTargeting, AdjustVisibilityFromPseudoSelectors)):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications

[webkit-changes] [WebKit/WebKit] 0752e9: [WebGPU] Invalidate all command buffers on submit, ...

2024-05-22 Thread mwyrzykowski
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 0752e9050165de7947d5aeb3c613755099311b25
  
https://github.com/WebKit/WebKit/commit/0752e9050165de7947d5aeb3c613755099311b25
  Author: Mike Wyrzykowski 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M LayoutTests/platform/mac-wk2/TestExpectations
M Source/WebGPU/WebGPU/Queue.mm

  Log Message:
  ---
  [WebGPU] Invalidate all command buffers on submit, regardless of it the 
submit call succeeds
https://bugs.webkit.org/show_bug.cgi?id=274244>


Reviewed by Dan Glastonbury.

Invalidate all command buffers passed to GPUQueue.submit, regardless
of whether or not they get committed.

Fixes the test being added in https://github.com/gpuweb/cts/pull/3716

Reland with regression fixed and test enabled.

* LayoutTests/platform/mac-wk2/TestExpectations:
* Source/WebGPU/WebGPU/Queue.mm:
(WebGPU::invalidateCommandBuffers):
(WebGPU::Queue::submit):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 72e754: Build fix for TextIndicatorStyle.

2024-05-22 Thread megangardner
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 72e7542a458b37df50599619295d9703876ebcda
  
https://github.com/WebKit/WebKit/commit/72e7542a458b37df50599619295d9703876ebcda
  Author: Megan Gardner 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebKit/UIProcess/mac/WKTextIndicatorStyleManager.h

  Log Message:
  ---
  Build fix for TextIndicatorStyle.
https://bugs.webkit.org/show_bug.cgi?id=274569
rdar://128584738

Unreviewed build fix.

Switched with TextIndicatorStyle type to use without
switching headers.

* Source/WebKit/UIProcess/mac/WKTextIndicatorStyleManager.h:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] ab81e7: [Skia] Enable ImageBitmap acceleration

2024-05-22 Thread Carlos Garcia Campos
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ab81e7f867ac35e5208bea45000caea21b7c5e9f
  
https://github.com/WebKit/WebKit/commit/ab81e7f867ac35e5208bea45000caea21b7c5e9f
  Author: Carlos Garcia Campos 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M LayoutTests/platform/glib/TestExpectations
M Source/WebCore/html/ImageBitmap.cpp

  Log Message:
  ---
  [Skia] Enable ImageBitmap acceleration
https://bugs.webkit.org/show_bug.cgi?id=274518

Reviewed by Miguel Gomez.

* LayoutTests/platform/glib/TestExpectations:
* Source/WebCore/html/ImageBitmap.cpp:
(WebCore::bufferRenderingMode):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 683c07: Unreviewed, reverting 278501@main (e84b70e7fa81)

2024-05-22 Thread martadarbinyan
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 683c07d76d889210ea9adc6415e2ee61d58a04d4
  
https://github.com/WebKit/WebKit/commit/683c07d76d889210ea9adc6415e2ee61d58a04d4
  Author: Marta Darbinyan 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/Document-write-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/Document-write.html
M 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/block-string-assignment-to-Document-write-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/block-string-assignment-to-Document-write.html
M Source/WebCore/dom/Document+HTML.idl
M Source/WebCore/dom/Document.cpp
M Source/WebCore/dom/Document.h

  Log Message:
  ---
  Unreviewed, reverting 278501@main (e84b70e7fa81)
https://bugs.webkit.org/show_bug.cgi?id=274567
rdar://128491467

REGRESSION (278501@main) [ Sonoma ASan Release ] - Multiple crashing tests with 
AddressSanitizer error

Reverted change:

Update implementation of TT enforcement for document.write(ln)
https://bugs.webkit.org/show_bug.cgi?id=273819
278501@main (e84b70e7fa81)

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] d7994b: [visionOS] build-webkit tries to build ImageDiff f...

2024-05-22 Thread Elliott Williams
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: d7994bbd6d7f0c1e2e2683c9ad6e02e75937bcc6
  
https://github.com/WebKit/WebKit/commit/d7994bbd6d7f0c1e2e2683c9ad6e02e75937bcc6
  Author: Elliott Williams 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Tools/Scripts/webkitdirs.pm

  Log Message:
  ---
  [visionOS] build-webkit tries to build ImageDiff for visionOS, not macOS
rdar://128579426
https://bugs.webkit.org/show_bug.cgi?id=274564

Unreviewed build fix.

Add --visionos to the list of arguments that get stripped from
build-webkit when calling build-imagediff.

Before:
> perl -ITools/Scripts -Mwebkitdirs -e 'print 
extractNonMacOSHostConfiguration(\@ARGV)' -- --visionos-device
--visionos-device

After:
> perl -ITools/Scripts -Mwebkitdirs -e 'print 
extractNonMacOSHostConfiguration(\@ARGV)' -- --visionos-device

* Tools/Scripts/webkitdirs.pm:
(extractNonMacOSHostConfiguration):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] ac3c97: [Gardening] Mark expected failing Navigation test

2024-05-22 Thread Patrick
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ac3c97671310494a611244c8ad75e76c4a1c7352
  
https://github.com/WebKit/WebKit/commit/ac3c97671310494a611244c8ad75e76c4a1c7352
  Author: Patrick Griffis 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M LayoutTests/TestExpectations

  Log Message:
  ---
  [Gardening] Mark expected failing Navigation test

Unreviewed test gardening.

* LayoutTests/TestExpectations:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 140660: [WebGPU] RenderBundleEncoder::setVertexBuffer may ...

2024-05-22 Thread mwyrzykowski
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 140660a4096bd07ed39c47760aa8c75eb4e9cf7c
  
https://github.com/WebKit/WebKit/commit/140660a4096bd07ed39c47760aa8c75eb4e9cf7c
  Author: Mike Wyrzykowski 
  Date:   2024-05-22 (Wed, 22 May 2024)

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

  Log Message:
  ---
  [WebGPU] RenderBundleEncoder::setVertexBuffer may result in a UAF
https://bugs.webkit.org/show_bug.cgi?id=274525


Reviewed by Dan Glastonbury.

Every other call site to recordCommand captures by Ref, this
was the only outlier passing a raw C++ pointer to the lambda.

* Source/WebGPU/WebGPU/RenderBundleEncoder.mm:
(WebGPU::RenderBundleEncoder::setVertexBuffer):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] a0dd96: [view-transitions] Spinner animation on https://ww...

2024-05-22 Thread mattwoodrow
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: a0dd96a17f0a14f6ceab095230852d772289c052
  
https://github.com/WebKit/WebKit/commit/a0dd96a17f0a14f6ceab095230852d772289c052
  Author: Matt Woodrow 
  Date:   2024-05-22 (Wed, 22 May 2024)

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

  Log Message:
  ---
  [view-transitions] Spinner animation on 
https://www.kvin.me/posts/button-interaction jumps
https://bugs.webkit.org/show_bug.cgi?id=274400


Reviewed by Tim Nguyen.

When an element is captured in a view transition, its 'transform' gets applied 
to the
::view-transition-group pseudo element, and we stop it being applied to the 
layer
for the element itself.
This makes sure we don't accidentally put it back again with an async animation.

* Source/WebCore/rendering/RenderLayerBacking.cpp:
(WebCore::RenderLayerBacking::startAnimation):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] ece697: Close XPC connection to OpenDirectory before revok...

2024-05-22 Thread Per Arne Vollan
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ece697a8bb31fb41166345f9b24a60496eee9f45
  
https://github.com/WebKit/WebKit/commit/ece697a8bb31fb41166345f9b24a60496eee9f45
  Author: Per Arne Vollan 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebKit/Shared/mac/AuxiliaryProcessMac.mm

  Log Message:
  ---
  Close XPC connection to OpenDirectory before revoking sandbox extension
https://bugs.webkit.org/show_bug.cgi?id=274557
rdar://128451478

Reviewed by Chris Dumez.

Close XPC connection to OpenDirectory before revoking sandbox extension. 
Otherwise,
another thread trying to write on a temporarily open connection could cause a 
crash,
because the sandbox extension has been revoked.

* Source/WebKit/Shared/mac/AuxiliaryProcessMac.mm:
(WebKit::AuxiliaryProcess::openDirectoryCacheInvalidated):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 563c79: Resolve non-replaced URLs in style sheets when sav...

2024-05-22 Thread Sihui
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 563c795b80f1c0de51382050f74087012e70b4db
  
https://github.com/WebKit/WebKit/commit/563c795b80f1c0de51382050f74087012e70b4db
  Author: Sihui Liu 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/css/CSSFontFaceSrcValue.cpp
M Source/WebCore/css/CSSFontFaceSrcValue.h
M Source/WebCore/css/CSSImageValue.cpp
M Source/WebCore/css/CSSImageValue.h
M Tools/TestWebKitAPI/Tests/WebKitCocoa/CreateWebArchive.mm

  Log Message:
  ---
  Resolve non-replaced URLs in style sheets when saving web page resources
https://bugs.webkit.org/show_bug.cgi?id=270960
rdar://120498768

Reviewed by Ryosuke Niwa.

Some CSS rules allow developers to specify multiple resources so that browser 
could pick the most suitable one to use
(e.g. @font-face). In our current implementation, when saving web page 
resources, we only save the resource that is
loaded. That means, URLs of other candidate resources will not be replaced with 
local file path in saved web page. It is
common that style sheets use relative path for subresource URL, and the result 
is the relative paths in saved style
sheets now point to non-existent files (as the candidate resources are not 
saved). To fix that problem, we now resolve
the non-replaced URLs in saved style sheets. With this change, when browser 
loads the saved web page and decides to load
candidate resource, it could send the request to correct server.

API test: WebArchive.SaveResourcesStyle

* Source/WebCore/css/CSSFontFaceSrcValue.cpp:
(WebCore::CSSFontFaceSrcResourceValue::customSetReplacementURLForSubresources):
(WebCore::CSSFontFaceSrcResourceValue::customClearReplacementURLForSubresources):
(WebCore::CSSFontFaceSrcResourceValue::customCSSText const):
* Source/WebCore/css/CSSFontFaceSrcValue.h:
* Source/WebCore/css/CSSImageValue.cpp:
(WebCore::CSSImageValue::customSetReplacementURLForSubresources):
(WebCore::CSSImageValue::customClearReplacementURLForSubresources):
(WebCore::CSSImageValue::customCSSText const):
* Source/WebCore/css/CSSImageValue.h:
* Tools/TestWebKitAPI/Tests/WebKitCocoa/CreateWebArchive.mm:
(TestWebKitAPI::(WebArchive, SaveResourcesStyleWithUnloadedResources)):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 298b67: REGRESSION(279097@main): [Win GTK WPE] ASSERTION F...

2024-05-22 Thread Chris Dumez
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 298b67cd8c41d4b73bd73016e0548f3440871751
  
https://github.com/WebKit/WebKit/commit/298b67cd8c41d4b73bd73016e0548f3440871751
  Author: Chris Dumez 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/platform/image-decoders/ico/ICOImageDecoder.cpp

  Log Message:
  ---
  REGRESSION(279097@main): [Win GTK WPE] ASSERTION FAILED: 
isASCII(characters[i]) in WTF::StringLiterals::operator""_span under 
WebCore::ICOImageDecoder::imageTypeAtIndex
https://bugs.webkit.org/show_bug.cgi?id=274549

Reviewed by Fujii Hironori.

Don't use ""_span since it requires all characters to be ASCII.

* Source/WebCore/platform/image-decoders/ico/ICOImageDecoder.cpp:
(WebCore::ICOImageDecoder::imageTypeAtIndex):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 3216dc: Fixed Playstation build break after 279154@main

2024-05-22 Thread jigenzhou
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 3216dc5420087aadbf282e20672d8b74493f990b
  
https://github.com/WebKit/WebKit/commit/3216dc5420087aadbf282e20672d8b74493f990b
  Author: Jigen Zhou 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/platform/graphics/nicosia/NicosiaImageBufferPipe.cpp

  Log Message:
  ---
  Fixed Playstation build break after 279154@main
https://bugs.webkit.org/show_bug.cgi?id=274559

Reviewed by Don Olmstead.

Fix linking error on undefined symbol `WebCore::GLFence::~GLFence()

* Source/WebCore/platform/graphics/nicosia/NicosiaImageBufferPipe.cpp:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] cc61eb: Pass additional data for correct TextIndicator sty...

2024-05-22 Thread megangardner
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: cc61ebcd1d9dffbc763dfb9d72050db3ea27b594
  
https://github.com/WebKit/WebKit/commit/cc61ebcd1d9dffbc763dfb9d72050db3ea27b594
  Author: Megan Gardner 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebKit/Scripts/webkit/messages.py
M Source/WebKit/Shared/UnifiedTextReplacement.serialization.in
M Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm
M Source/WebKit/UIProcess/API/Cocoa/WKWebViewInternal.h
M Source/WebKit/UIProcess/Cocoa/PageClientImplCocoa.h
M Source/WebKit/UIProcess/Cocoa/PageClientImplCocoa.mm
M Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm
M Source/WebKit/UIProcess/PageClient.h
M Source/WebKit/UIProcess/TextIndicatorStyle.h
M Source/WebKit/UIProcess/WKSTextStyleManager.h
M Source/WebKit/UIProcess/WebPageProxy.cpp
M Source/WebKit/UIProcess/WebPageProxy.h
M Source/WebKit/UIProcess/WebPageProxy.messages.in
M Source/WebKit/UIProcess/mac/WKTextIndicatorStyleManager.h
M Source/WebKit/UIProcess/mac/WebViewImpl.h
M Source/WebKit/UIProcess/mac/WebViewImpl.mm
M Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.mm
M Source/WebKit/WebProcess/WebPage/Cocoa/UnifiedTextReplacementController.mm
M Source/WebKit/WebProcess/WebPage/Cocoa/WebPageCocoa.mm
M Source/WebKit/WebProcess/WebPage/UnifiedTextReplacementController.h
M Source/WebKit/WebProcess/WebPage/WebPage.cpp
M Source/WebKit/WebProcess/WebPage/WebPage.h

  Log Message:
  ---
  Pass additional data for correct TextIndicator styling.
https://bugs.webkit.org/show_bug.cgi?id=274505
rdar://128475613

Reviewed by Aditya Keerthi.

* Source/WebKit/Scripts/webkit/messages.py:
(headers_for_type):
* Source/WebKit/Shared/UnifiedTextReplacement.serialization.in:
* Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm:
(-[WKWebView _addTextIndicatorStyleForID:withData:]):
(-[WKWebView _enableTextIndicatorStylingAfterElementWithID:]):
(-[WKWebView _enableTextIndicatorStylingForElementWithID:]):
(-[WKWebView _addTextIndicatorStyleForID:withStyleType:]): Deleted.
* Source/WebKit/UIProcess/API/Cocoa/WKWebViewInternal.h:
* Source/WebKit/UIProcess/Cocoa/PageClientImplCocoa.h:
* Source/WebKit/UIProcess/Cocoa/PageClientImplCocoa.mm:
(WebKit::PageClientImplCocoa::addTextIndicatorStyleForID):
* Source/WebKit/UIProcess/Cocoa/WKTextIndicatorStyleType.h:
* Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm:
(WebKit::WebPageProxy::addTextIndicatorStyleForID):
* Source/WebKit/UIProcess/PageClient.h:
* Source/WebKit/UIProcess/TextIndicatorStyle.h:
* Source/WebKit/UIProcess/WKSTextStyleManager.h:
* Source/WebKit/UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::setTextIndicatorFromFrame):
* Source/WebKit/UIProcess/WebPageProxy.h:
* Source/WebKit/UIProcess/WebPageProxy.messages.in:
* Source/WebKit/UIProcess/mac/WKTextIndicatorStyleManager.h:
* Source/WebKit/UIProcess/mac/WebViewImpl.h:
* Source/WebKit/UIProcess/mac/WebViewImpl.mm:
(WebKit::WebViewImpl::addTextIndicatorStyleForID):
* Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.mm:
* Source/WebKit/WebProcess/WebPage/Cocoa/UnifiedTextReplacementController.mm:
(WebKit::UnifiedTextReplacementController::removeTransparentMarkersForUUID):
(WebKit::UnifiedTextReplacementController::removeTransparentMarkersForSession):
(WebKit::UnifiedTextReplacementController::textReplacementSessionDidReceiveTextWithReplacementRange):
(WebKit::UnifiedTextReplacementController::contextRangeForSessionOrRangeWithUUID
 const):
* Source/WebKit/WebProcess/WebPage/Cocoa/WebPageCocoa.mm:
(WebKit::WebPage::updateTextIndicatorStyleVisibilityForID):
* Source/WebKit/WebProcess/WebPage/UnifiedTextReplacementController.h:
* Source/WebKit/WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::addTextIndicatorStyleForID):
* Source/WebKit/WebProcess/WebPage/WebPage.h:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 365ec3: REGRESSION (268367@main): [ iOS ] imported/w3c/web...

2024-05-22 Thread Jay Saintfleur
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 365ec381a9cbaf85d6dc334f8ed1b92c30a3d467
  
https://github.com/WebKit/WebKit/commit/365ec381a9cbaf85d6dc334f8ed1b92c30a3d467
  Author: Jay Stfleur 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M LayoutTests/platform/ios/TestExpectations
M 
LayoutTests/platform/ios/imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml-expected.txt

  Log Message:
  ---
  REGRESSION (268367@main): [ iOS ] 
imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml.html is a 
consistent failure
https://bugs.webkit.org/show_bug.cgi?id=274245
rdar://128180039

Unreviewed test gardening.

Adding rebaseline and removing test expectations.

* LayoutTests/platform/ios/TestExpectations:
* 
LayoutTests/platform/ios/imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml-expected.txt:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 056665: Remove mac-monterey folders from LayoutTests/platf...

2024-05-22 Thread Ben
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 05666581bf4ccf74ec8700a949363341524bc6c3
  
https://github.com/WebKit/WebKit/commit/05666581bf4ccf74ec8700a949363341524bc6c3
  Author: Ben Schwartz 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
R 
LayoutTests/platform/mac-monterey-wk2/fast/scrolling/mac/scrollbars/scrollbars-controller-type-expected.txt
R LayoutTests/platform/mac-monterey/TestExpectations
R 
LayoutTests/platform/mac-monterey/accessibility/math-multiscript-attributes-expected.txt
R 
LayoutTests/platform/mac-monterey/css1/box_properties/acid_test-expected.txt
R 
LayoutTests/platform/mac-monterey/css2.1/t09-c5526c-display-00-e-expected.txt
R 
LayoutTests/platform/mac-monterey/editing/caret/color-span-inside-editable-background-expected.html
R 
LayoutTests/platform/mac-monterey/editing/caret/color-span-inside-editable-expected.html
R LayoutTests/platform/mac-monterey/fast/block/basic/011-expected.txt
R LayoutTests/platform/mac-monterey/fast/block/basic/minheight-expected.txt
R 
LayoutTests/platform/mac-monterey/fast/block/margin-collapse/103-expected.txt
R LayoutTests/platform/mac-monterey/fast/css/002-expected.txt
R 
LayoutTests/platform/mac-monterey/fast/events/invalid/missing-end-tag-expected.txt
R 
LayoutTests/platform/mac-monterey/fast/events/mouse-cursor-multiframecur-expected.txt
R LayoutTests/platform/mac-monterey/fast/invalid/008-expected.txt
R 
LayoutTests/platform/mac-monterey/fast/invalid/missing-end-tag-expected.png
R 
LayoutTests/platform/mac-monterey/fast/invalid/missing-end-tag-expected.txt
R LayoutTests/platform/mac-monterey/fast/overflow/003-expected.txt
R 
LayoutTests/platform/mac-monterey/fast/text/basic/generic-family-reset-expected.txt
R 
LayoutTests/platform/mac-monterey/fast/text/canvas-color-fonts/stroke-color-COLR-expected.html
R 
LayoutTests/platform/mac-monterey/fast/text/canvas-color-fonts/stroke-color-shadow-COLR-expected.html
R 
LayoutTests/platform/mac-monterey/fast/text/canvas-color-fonts/stroke-color-shadow-ctm-COLR-expected.html
R 
LayoutTests/platform/mac-monterey/fast/text/canvas-color-fonts/stroke-gradient-COLR-expected.html
R LayoutTests/platform/mac-monterey/fast/text/resources/Ahem-COLR.ttf
R 
LayoutTests/platform/mac-monterey/http/tests/resourceLoadStatistics/website-data-removal-for-site-navigated-to-with-filtered-link-decoration-after-long-deletion-expected.txt
R 
LayoutTests/platform/mac-monterey/http/tests/resourceLoadStatistics/website-data-removal-for-site-navigated-to-with-unfiltered-link-decoration-after-short-deletion-expected.txt
R 
LayoutTests/platform/mac-monterey/http/tests/resourceLoadStatistics/website-data-removal-for-site-navigated-to-without-link-decoration-expected.txt
R 
LayoutTests/platform/mac-monterey/http/tests/resourceLoadStatistics/website-data-removal-for-site-without-user-interaction-expected.txt
R 
LayoutTests/platform/mac-monterey/http/tests/security/xss-DENIED-xsl-external-entity-expected.txt
R 
LayoutTests/platform/mac-monterey/http/tests/security/xss-DENIED-xsl-external-entity-redirect-expected.txt
R 
LayoutTests/platform/mac-monterey/http/tests/security/xss-DENIED-xsl-external-entity-xslt-docloader-expected.txt
R 
LayoutTests/platform/mac-monterey/imported/w3c/web-platform-tests/cookie-store/change_eventhandler_for_document_cookie.https.window-expected.txt
R 
LayoutTests/platform/mac-monterey/imported/w3c/web-platform-tests/cookie-store/change_eventhandler_for_http_cookie_and_set_cookie_headers.https.window-expected.txt
R 
LayoutTests/platform/mac-monterey/imported/w3c/web-platform-tests/cookie-store/encoding.https.any-expected.txt
R 
LayoutTests/platform/mac-monterey/imported/w3c/web-platform-tests/cookies/value/value-ctl-expected.txt
R 
LayoutTests/platform/mac-monterey/imported/w3c/web-platform-tests/html/canvas/element/manual/wide-gamut-canvas/canvas-createImageBitmap-e_srgb-expected.txt
R 
LayoutTests/platform/mac-monterey/imported/w3c/web-platform-tests/html/dom/documents/resource-metadata-management/document-cookie-expected.txt
R 
LayoutTests/platform/mac-monterey/imported/w3c/web-platform-tests/mathml/presentation-markup/mrow/mrow-fallback-expected.txt
R 
LayoutTests/platform/mac-monterey/imported/w3c/web-platform-tests/mathml/relations/css-styling/ignored-properties-001-expected.txt
R 
LayoutTests/platform/mac-monterey/imported/w3c/web-platform-tests/mathml/relations/css-styling/not-participating-to-parent-layout-expected.txt
R 
LayoutTests/platform/mac-monterey/imported/w3c/web-platform-tests/mathml/relations/css-styling/padding-border-margin/border-002-expected.txt
R 
LayoutTests/platform/mac-monterey/imported/w3c/web-platform-tests/mathml/relations/css-styling/padding-border-margin/padding-002-expected.txt
R 

[webkit-changes] [WebKit/WebKit] b378f0: Unreviewed, reverting 279155@main.

2024-05-22 Thread Commit Queue
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b378f0009318ff3b6c5a18d3e8e50457999e2a40
  
https://github.com/WebKit/WebKit/commit/b378f0009318ff3b6c5a18d3e8e50457999e2a40
  Author: Commit Queue 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebKit/GPUProcess/mac/GPUProcessMac.mm
M Source/WebKit/Shared/AuxiliaryProcess.h
M Source/WebKit/Shared/mac/AuxiliaryProcessMac.mm
M Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm

  Log Message:
  ---
  Unreviewed, reverting 279155@main.
https://bugs.webkit.org/show_bug.cgi?id=274555

Incorrect revert, this will be resolved differently

Reverted changeset:

"Unreviewed, reverting 277204@main."
https://bugs.webkit.org/show_bug.cgi?id=274553
https://commits.webkit.org/279155@main

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 354d46: RenderLayer : Unbalanced begin/endTransparencyLaye...

2024-05-22 Thread darryl-apple
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 354d46a613b7dd92b44cd2d770f52127b33e546c
  
https://github.com/WebKit/WebKit/commit/354d46a613b7dd92b44cd2d770f52127b33e546c
  Author: Nisha Jain 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
A 
LayoutTests/fast/rendering/render-layer-with-providedbacking-crash-expected.txt
A LayoutTests/fast/rendering/render-layer-with-providedbacking-crash.html
M Source/WebCore/rendering/RenderLayer.cpp

  Log Message:
  ---
  RenderLayer : Unbalanced begin/endTransparencyLayers with nested masked 
elements and backing sharing causes a crash.
https://bugs.webkit.org/show_bug.cgi?id=269865
rdar://11314.

Reviewed by Simon Fraser.

In order to balance the begin/endTransparencyLayers call for a 
TransparencyLayer with nested masked elements and backing sharing, added 
'provided backing/paint to' into account while doing 'ancestor' traversal.

* 
LayoutTests/fast/rendering/render-layer-with-providedbacking-crash-expected.txt:
 Added.
* LayoutTests/fast/rendering/render-layer-with-providedbacking-crash.html: 
Added.
* Source/WebCore/rendering/RenderLayer.cpp:
(WebCore::RenderLayer::transparentPaintingAncestor):

Originally-landed-as: 272448.600@safari-7618-branch (fc090f6ee88d). 
rdar://128545892
Canonical link: https://commits.webkit.org/279158@main


  Commit: 6398a57fc5a53f9fed588bfd32a13a8f55285ca0
  
https://github.com/WebKit/WebKit/commit/6398a57fc5a53f9fed588bfd32a13a8f55285ca0
  Author: Scott Marcy 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M LayoutTests/TestExpectations
A LayoutTests/ipc/removeMediaUsageManagerSession-test-expected.txt
A LayoutTests/ipc/removeMediaUsageManagerSession-test.html

  Log Message:
  ---
  Adds a simplified test case to verify an over-release in ScreenTime is fixed.
https://bugs.webkit.org/show_bug.cgi?id=270461
rdar://115279815

Reviewed by David Kilzer.

This test is currently disabled in TestExpectations due to rdar://120486467 
(Provide a way to notify WebKitTestRunner when we hit a purposeful 
MESSAGE_CHECK (267714)).

* LayoutTests/TestExpectations:
* LayoutTests/ipc/removeMediaUsageManagerSession-test-expected.txt: Added.
* LayoutTests/ipc/removeMediaUsageManagerSession-test.html: Added.

Originally-landed-as: 272448.683@safari-7618-branch (6e9c0ea8de57). 
rdar://128546108
Canonical link: https://commits.webkit.org/279159@main


  Commit: d8547c86a7407a7a4e6e39c82773391be270d94f
  
https://github.com/WebKit/WebKit/commit/d8547c86a7407a7a4e6e39c82773391be270d94f
  Author: Rob Buis 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
A 
LayoutTests/security/schedule-request-object-then-disconnect-crash-expected.txt
A LayoutTests/security/schedule-request-object-then-disconnect-crash.html

  Log Message:
  ---
  Add test case for object loading crash
https://bugs.webkit.org/show_bug.cgi?id=268771
rdar://121959099

Reviewed by Ryosuke Niwa.

This test case applies to this bug as well as bug 264626, meaning it will crash
without the fix from bug 264626. Note that it makes more sense to land this test
than the one from bug 264626 as that one is flaky and this one is small and
reproduces every time.

* 
LayoutTests/security/schedule-request-object-then-disconnect-crash-expected.txt:
 Added.
* LayoutTests/security/schedule-request-object-then-disconnect-crash.html: 
Added.

Originally-landed-as: 274097.11@webkit-2024.2-embargoed (22a024aa0b40). 
rdar://128546360
Canonical link: https://commits.webkit.org/279160@main


  Commit: 345a36c5552dab4470ceb58265f42188094d0b0d
  
https://github.com/WebKit/WebKit/commit/345a36c5552dab4470ceb58265f42188094d0b0d
  Author: Darryl Parkinson 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/workers/service/server/SWServerWorker.cpp
M Source/WebCore/workers/service/server/SWServerWorker.h
M 
Source/WebKit/NetworkProcess/ServiceWorker/WebSWServerToContextConnection.cpp
M 
Source/WebKit/NetworkProcess/ServiceWorker/WebSWServerToContextConnection.h

  Log Message:
  ---
  Ensure that the SWServerWorker is in the correct state before finishing 
installation
rdar://121429889

Reviewed by Youenn Fablet.

Ensure that the SWServerWorker is in the expected state and bail if this method 
is triggered on a SWServerWorker in a different state. This method is callable 
over CoreIPC passing a ServiceWorkerIdentifier. Passing the ID of a service 
worker in any other state will reach the RELEASE_ASSERT

* Source/WebCore/workers/service/server/SWServerWorker.cpp:
(WebCore::SWServerWorker::didFinishInstall):

Originally-landed-as: 272448.701@safari-7618-branch (be630dbb12c9). 
rdar://128546484
Canonical link: https://commits.webkit.org/279161@main


Compare: https://github.com/WebKit/WebKit/compare/b287b6cc9662...345a36c5552d

To unsubscribe from these emails, change your notification settings at 

[webkit-changes] [WebKit/WebKit] 4d2933: [JSC] Harden CustomGetterSetter by adding MethodTa...

2024-05-22 Thread lericaa
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 4d29332a09b0f3b17eff2d44f464dd8dff77cad0
  
https://github.com/WebKit/WebKit/commit/4d29332a09b0f3b17eff2d44f464dd8dff77cad0
  Author: Alexey Shvayka 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/JavaScriptCore/runtime/CustomGetterSetter.h

  Log Message:
  ---
  [JSC] Harden CustomGetterSetter by adding MethodTable overrides that always 
crash
https://bugs.webkit.org/show_bug.cgi?id=268897


Reviewed by Mark Lam.

Just like GetterSetter, CustomGetterSetter is never purposely exposed to 
userland code.
However, to make exploitation of accidentally exposed CustomGetterSetter 
objects difficult, this
patch implements MethodTable overrides that abort the program when reached, 
similar to GetterSetter.

* Source/JavaScriptCore/runtime/CustomGetterSetter.h:
(JSC::CustomGetterSetter::getOwnPropertySlot):
(JSC::CustomGetterSetter::put):
(JSC::CustomGetterSetter::putByIndex):
(JSC::CustomGetterSetter::setPrototype):
(JSC::CustomGetterSetter::defineOwnProperty):
(JSC::CustomGetterSetter::deleteProperty):

Originally-landed-as: 272448.523@safari-7618-branch (66d8614c41ca). 
rdar://128498125
Canonical link: https://commits.webkit.org/279156@main


  Commit: b287b6cc9662e88415c7958132ab001431b35f9f
  
https://github.com/WebKit/WebKit/commit/b287b6cc9662e88415c7958132ab001431b35f9f
  Author: Erica Li 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
A 
LayoutTests/ipc/create-media-source-with-invalid-constraints-crash-expected.txt
A LayoutTests/ipc/create-media-source-with-invalid-constraints-crash.html
M Source/WebCore/platform/mediastream/MediaConstraints.cpp
M Source/WebCore/platform/mediastream/MediaConstraints.h
M Source/WebKit/UIProcess/Cocoa/UserMediaCaptureManagerProxy.cpp

  Log Message:
  ---
  WTFCrashWithSecurityImplication in 
WebCore::RealtimeMediaSource::fitnessDistance
https://bugs.webkit.org/show_bug.cgi?id=268800
rdar://122105977

Reviewed by Youenn Fablet.

This is short-term suggested fix to add isValid check to 
MediaTrackConstraintSetMap to ensure each incomming contraint from IPC call has 
the right MediaConstraintType.

* 
LayoutTests/ipc/create-media-source-with-invalid-constraints-crash-expected.txt:
 Added.
* LayoutTests/ipc/create-media-source-with-invalid-constraints-crash.html: 
Added.
* Source/WebCore/platform/mediastream/MediaConstraints.cpp:
(WebCore::MediaTrackConstraintSetMap::isValid const):
* Source/WebCore/platform/mediastream/MediaConstraints.h:
* Source/WebKit/UIProcess/Cocoa/UserMediaCaptureManagerProxy.cpp:
(WebKit::UserMediaCaptureManagerProxy::createMediaSourceForCaptureDeviceWithConstraints):
(WebKit::UserMediaCaptureManagerProxy::applyConstraints):

Originally-landed-as: 272448.542@safari-7618-branch (01389d47b6ec). 
rdar://128498600
Canonical link: https://commits.webkit.org/279157@main


Compare: https://github.com/WebKit/WebKit/compare/223c3b4280f0...b287b6cc9662

To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 223c3b: Unreviewed, reverting 277204@main.

2024-05-22 Thread Commit Queue
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 223c3b4280f02d3f524c66dfa7c9878f5f5b1c10
  
https://github.com/WebKit/WebKit/commit/223c3b4280f02d3f524c66dfa7c9878f5f5b1c10
  Author: Commit Queue 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebKit/GPUProcess/mac/GPUProcessMac.mm
M Source/WebKit/Shared/AuxiliaryProcess.h
M Source/WebKit/Shared/mac/AuxiliaryProcessMac.mm
M Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm

  Log Message:
  ---
  Unreviewed, reverting 277204@main.
https://bugs.webkit.org/show_bug.cgi?id=274553

Introduced crash

Reverted changeset:

"Avoid IPC to OpenDirectory XPC service on main thread"
https://bugs.webkit.org/show_bug.cgi?id=272192
https://commits.webkit.org/277204@main

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit]

2024-05-22 Thread Jonathan Bedard
  Branch: refs/tags/WebKit-7618.2.12.10.10
  Home:   https://github.com/WebKit/WebKit

To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit]

2024-05-22 Thread Jonathan Bedard
  Branch: refs/tags/WebKit-7618.2.12.11.7
  Home:   https://github.com/WebKit/WebKit

To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] c1950e: Cherry-pick 2eca0fcc0b79. rdar://128223572

2024-05-22 Thread Jonathan Bedard
  Branch: refs/heads/safari-7618.2.12.10-branch
  Home:   https://github.com/WebKit/WebKit
  Commit: c1950e218f1bcbfadae41d0a9b414243cff8e86f
  
https://github.com/WebKit/WebKit/commit/c1950e218f1bcbfadae41d0a9b414243cff8e86f
  Author: Mohsin Qureshi 
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
M Source/WTF/wtf/text/WTFString.h
M Source/WebKit/Shared/Cocoa/CoreIPCContacts.mm
M Tools/TestWebKitAPI/Tests/IPC/IPCSerialization.mm

  Log Message:
  ---
  Cherry-pick 2eca0fcc0b79. rdar://128223572

Apply patch. rdar://127453631

Canonical link: https://commits.webkit.org/272448.990@safari-7618-branch


  Commit: 41839bce22645f9876aaa4b023a89c6e5d760d18
  
https://github.com/WebKit/WebKit/commit/41839bce22645f9876aaa4b023a89c6e5d760d18
  Author: Mohsin Qureshi 
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
M Configurations/Version.xcconfig

  Log Message:
  ---
  Versioning.

WebKit-7618.2.12.10.10


Compare: https://github.com/WebKit/WebKit/compare/5b0e147c0955...41839bce2264

To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] f3c82e: Cherry-pick 2eca0fcc0b79. rdar://128223572

2024-05-22 Thread Jonathan Bedard
  Branch: refs/heads/safari-7618.2.12.11-branch
  Home:   https://github.com/WebKit/WebKit
  Commit: f3c82e4863a7def40f462638f2449f4279801d17
  
https://github.com/WebKit/WebKit/commit/f3c82e4863a7def40f462638f2449f4279801d17
  Author: Mohsin Qureshi 
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
M Source/WTF/wtf/text/WTFString.h
M Source/WebKit/Shared/Cocoa/CoreIPCContacts.mm
M Tools/TestWebKitAPI/Tests/IPC/IPCSerialization.mm

  Log Message:
  ---
  Cherry-pick 2eca0fcc0b79. rdar://128223572

Apply patch. rdar://127453631

Canonical link: https://commits.webkit.org/272448.990@safari-7618-branch


  Commit: 94aa5569ec700548c53cfb064e4ca8c23c63ca57
  
https://github.com/WebKit/WebKit/commit/94aa5569ec700548c53cfb064e4ca8c23c63ca57
  Author: Mohsin Qureshi 
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
M Configurations/Version.xcconfig

  Log Message:
  ---
  Versioning.

WebKit-7618.2.12.11.7


Compare: https://github.com/WebKit/WebKit/compare/d5d9247df661...94aa5569ec70

To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 1fc4f3: Fixed Playstation build break after 279130@main

2024-05-22 Thread jigenzhou
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 1fc4f38b76059284f949c8978d6d39782bd8a938
  
https://github.com/WebKit/WebKit/commit/1fc4f38b76059284f949c8978d6d39782bd8a938
  Author: Jigen Zhou 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/platform/graphics/nicosia/NicosiaImageBufferPipe.cpp

  Log Message:
  ---
  Fixed Playstation  build break after 279130@main
https://bugs.webkit.org/show_bug.cgi?id=274547

Reviewed by Don Olmstead.

TextureMapperPlatformLayerBuffer::setFence is only available if PLATFORM(GTK) 
|| PLATFORM(WPE)

* Source/WebCore/platform/graphics/nicosia/NicosiaImageBufferPipe.cpp:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit]

2024-05-22 Thread Jonathan Bedard
  Branch: refs/tags/WebKit-7618.2.12.13.2
  Home:   https://github.com/WebKit/WebKit

To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit]

2024-05-22 Thread Jonathan Bedard
  Branch: refs/tags/WebKit-7618.2.12.13.1
  Home:   https://github.com/WebKit/WebKit

To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] ea2439: Versioning.

2024-05-22 Thread Keith Miller
  Branch: refs/heads/safari-7618.2.12.13-branch
  Home:   https://github.com/WebKit/WebKit
  Commit: ea2439209d9fe9154b5f04fb3bba08379bfc8d72
  
https://github.com/WebKit/WebKit/commit/ea2439209d9fe9154b5f04fb3bba08379bfc8d72
  Author: Mohsin Qureshi 
  Date:   2024-04-15 (Mon, 15 Apr 2024)

  Changed paths:
M Configurations/Version.xcconfig

  Log Message:
  ---
  Versioning.

WebKit-7618.2.12.13.1

Canonical link: https://commits.webkit.org/272448.932@safari-7618.2.12.13-branch


  Commit: b76830ca75df8b00dbdb36c592ad84f7ed110866
  
https://github.com/WebKit/WebKit/commit/b76830ca75df8b00dbdb36c592ad84f7ed110866
  Author: Chris Dumez 
  Date:   2024-04-16 (Tue, 16 Apr 2024)

  Changed paths:
M Source/WebKit/Shared/AuxiliaryProcess.h
M Source/WebKit/Shared/AuxiliaryProcess.messages.in
M Source/WebKit/Shared/Cocoa/AuxiliaryProcessCocoa.mm
M Source/WebKit/UIProcess/AuxiliaryProcessProxy.cpp
M Source/WebKit/UIProcess/AuxiliaryProcessProxy.h
M Source/WebKit/UIProcess/Cocoa/AuxiliaryProcessProxyCocoa.mm

  Log Message:
  ---
  Cherry-pick d6540a38e780. rdar://126492909

Regression(277427@main) Crash under 
AuxiliaryProcessProxy::notifyPreferencesChanged()
https://bugs.webkit.org/show_bug.cgi?id=272695
rdar://126492909

Reviewed by Per Arne Vollan.

We were using a HashMap to store preferences whose key was a 
std::pair.
The first String was the domain and the second the preference name. 
However, for global
preferences, the domain is null, causing a crash when hashing the key.

To address an issue, we now store global preferences in a separate HashMap.

* Source/WebKit/Shared/AuxiliaryProcess.h:
* Source/WebKit/Shared/AuxiliaryProcess.messages.in:
* Source/WebKit/Shared/Cocoa/AuxiliaryProcessCocoa.mm:
(WebKit::AuxiliaryProcess::preferencesDidUpdate):
* Source/WebKit/UIProcess/AuxiliaryProcessProxy.cpp:
(WebKit::AuxiliaryProcessProxy::didChangeThrottleState):
* Source/WebKit/UIProcess/AuxiliaryProcessProxy.h:
* Source/WebKit/UIProcess/Cocoa/AuxiliaryProcessProxyCocoa.mm:
(WebKit::AuxiliaryProcessProxy::notifyPreferencesChanged):

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


  Commit: c4b640fbedbee2518b1fb4b1847657a91df811ed
  
https://github.com/WebKit/WebKit/commit/c4b640fbedbee2518b1fb4b1847657a91df811ed
  Author: Dan Robson 
  Date:   2024-04-17 (Wed, 17 Apr 2024)

  Changed paths:
M Configurations/Version.xcconfig

  Log Message:
  ---
  Versioning.

WebKit-7618.2.12.13.2

Canonical link: https://commits.webkit.org/272448.934@safari-7618.2.12.13-branch


  Commit: d7e2f94c57ea9901695253d7882747b6f62b6ab8
  
https://github.com/WebKit/WebKit/commit/d7e2f94c57ea9901695253d7882747b6f62b6ab8
  Author: Per Arne Vollan 
  Date:   2024-04-17 (Wed, 17 Apr 2024)

  Changed paths:
M Source/WebKit/Platform/cocoa/XPCUtilities.h
M Source/WebKit/Platform/cocoa/XPCUtilities.mm
M Source/WebKit/Shared/Authentication/cocoa/AuthenticationManagerCocoa.mm
M Source/WebKit/Shared/Cocoa/XPCEndpoint.mm
M 
Source/WebKit/Shared/EntryPointUtilities/Cocoa/XPCService/XPCServiceMain.mm

  Log Message:
  ---
  Cherry-pick 3c2c899f692d. rdar://126479653

WebKit process termination with xpc_connection_kill does not always work
https://bugs.webkit.org/show_bug.cgi?id=272669
rdar://126479653

Reviewed by Chris Dumez.

WebKit process termination with xpc_connection_kill does not always work. 
We are currently seeing flaky
termination behavior on macOS, where the child processes are not always 
terminated successfully.
Additionally, on iOS, the XPC connection has become anonymous due to 
migration to extensions for WebKit
processes, and xpc_connection_kill does not support anonymous connections. 
This patch addresses this
issue by creating and sending a XPC message to the child process to request 
termination. This has a
high chance of success, since we know that the XPC connection termination 
watchdog is holding a
background assertion on the process, so it is not suspended. Additionally, 
the XPC message is being
handled on the XPC event handler thread, which is handling very few 
messages, so it is very unlikely
that it is blocked and cannot handle the message. This gives the process a 
chance to exit cleanly and
send a reply back. If the UI process does not receive the expected reply, 
it will try calling
xpc_connection_kill.

* Source/WebKit/Platform/cocoa/XPCUtilities.h:
* Source/WebKit/Platform/cocoa/XPCUtilities.mm:
(WebKit::terminateWithReason):
(WebKit::handleXPCExitMessage):
* Source/WebKit/Shared/Authentication/cocoa/AuthenticationManagerCocoa.mm:
(WebKit::AuthenticationManager::initializeConnection):
* Source/WebKit/Shared/Cocoa/XPCEndpoint.mm:
(WebKit::XPCEndpoint::XPCEndpoint):
* 

[webkit-changes] [WebKit/WebKit] d61c82: ImageFailure sometimes happen with fast/images/dec...

2024-05-22 Thread Said Abou-Hallawa
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: d61c828315d694a910352391aa51a1493b94aebb
  
https://github.com/WebKit/WebKit/commit/d61c828315d694a910352391aa51a1493b94aebb
  Author: Said Abou-Hallawa 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M LayoutTests/fast/images/decode-render-animated-image.html

  Log Message:
  ---
  ImageFailure sometimes happen with 
fast/images/decode-render-animated-image.html
https://bugs.webkit.org/show_bug.cgi?id=271408
rdar://125573776

Unreviewed test gardening

Change the test to draw the first frame but call decode() for the following
frames. This will make decode() behave the same way for all frames. It will
call startAnimation() and wait for imageFrameAtIndexAvailable() to be called.

* LayoutTests/fast/images/decode-render-animated-image.html:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 1085de: [Site Isolation] Begin to fix the back forward lis...

2024-05-22 Thread Charlie Wolfe
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 1085dea9ba4113642ffc393e3f9ac06a778ff26c
  
https://github.com/WebKit/WebKit/commit/1085dea9ba4113642ffc393e3f9ac06a778ff26c
  Author: Charlie Wolfe 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/history/BackForwardClient.h
M Source/WebCore/history/BackForwardController.cpp
M Source/WebCore/history/BackForwardController.h
M Source/WebCore/loader/EmptyClients.cpp
M Source/WebCore/loader/HistoryController.cpp
M Source/WebCore/page/Page.cpp
M Source/WebCore/page/Page.h
M Source/WebKit/Shared/WebBackForwardListItem.h
M Source/WebKit/UIProcess/ProvisionalPageProxy.cpp
M Source/WebKit/UIProcess/ProvisionalPageProxy.h
M Source/WebKit/UIProcess/WebBackForwardList.cpp
M Source/WebKit/UIProcess/WebFrameProxy.h
M Source/WebKit/UIProcess/WebPageProxy.cpp
M Source/WebKit/UIProcess/WebPageProxy.h
M Source/WebKit/UIProcess/WebPageProxy.messages.in
M Source/WebKit/WebProcess/WebPage/WebBackForwardListProxy.cpp
M Source/WebKit/WebProcess/WebPage/WebBackForwardListProxy.h
M Source/WebKit/WebProcess/WebPage/WebPage.cpp
M Source/WebKitLegacy/mac/History/BackForwardList.h
M Source/WebKitLegacy/mac/History/BackForwardList.mm
M Source/WebKitLegacy/mac/History/WebBackForwardList.mm
M Source/WebKitLegacy/mac/WebView/WebView.mm
M Tools/TestWebKitAPI/Tests/WebKitCocoa/SiteIsolation.mm

  Log Message:
  ---
  [Site Isolation] Begin to fix the back forward list for same-origin iframe 
navigations
https://bugs.webkit.org/show_bug.cgi?id=274419
rdar://128420789

Reviewed by Alex Christensen.

Some back forward list state for child frames is currently stored only in the 
web process. This needs to
change with site isolation because web processes can only have history items 
corresponding to frames they
are hosting or have embedded. With this patch, whenever a root child frame is 
embedded, we add a separate
item to the back forward list in the UI process. This way, when navigating back 
or forward, iframe
processes can request history items containing state about same-origin frames 
that have been embedded by
another process without receiving information about the entire frame tree. We 
also need to keep track of
which back forward items correspond to initial root child frame loads. This is 
necessary to determine
which item to navigate an isolated iframe to, while also ignoring these items 
when the main frame is
moved back or forward.

Further work is required to make the back forward list behave correctly after 
cross-origin iframe
navigations. More details below.

* Source/WebCore/history/BackForwardClient.h:
* Source/WebCore/history/BackForwardController.cpp:
(WebCore::BackForwardController::addItem):
* Source/WebCore/history/BackForwardController.h:
* Source/WebCore/loader/EmptyClients.cpp:
* Source/WebCore/loader/HistoryController.cpp:
(WebCore::HistoryController::goToItem):
(WebCore::HistoryController::updateBackForwardListClippedAtTarget):
(WebCore::HistoryController::pushState):
* Source/WebCore/page/Page.cpp:
(WebCore::Page::goToItem):
* Source/WebCore/page/Page.h:

Update `addItem` to include a FrameIdentifier parameter so the UI process knows 
which frame is being
targeted. Also remove several unneeded local frame downcasts.

* Source/WebKit/Shared/WebBackForwardListItem.h:
(WebKit::WebBackForwardListItem::setIsRootChildFrameItem):
(WebKit::WebBackForwardListItem::isRootChildFrameItem const):

Add `m_isRootChildFrameItem` to indicate when a back forward item was created 
by an initial root child
frame load.

* Source/WebKit/UIProcess/ProvisionalPageProxy.cpp:
(WebKit::ProvisionalPageProxy::backForwardAddItem):
* Source/WebKit/UIProcess/ProvisionalPageProxy.h:

* Source/WebKit/UIProcess/WebBackForwardList.cpp:
(WebKit::itemSkippingBackForwardItemsAddedByJSWithoutUserGesture):

Skip root child frame items to avoid targeting initial child frame loads when 
trying to move the main
frame back/forward. This logic probably still isn’t correct when going 
back/forward between
cross-origin iframes.

* Source/WebKit/UIProcess/WebFrameProxy.h:
(WebKit::WebFrameProxy::setHasPendingBackForwardItem):
(WebKit::WebFrameProxy::takeHasPendingBackforwardItem):

Add `m_hasPendingBackForwardItem` to indicate when a root frame is waiting for 
its initial back forward
item to be created.

* Source/WebKit/UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::goToBackForwardItem):

Change this function to send back forward list items corresponding to isolated 
iframes to the process
that navigated. `m_lastProcessIdentifier` on `WebBackForwardListItem` is 
already used for process
selection for PSON. It will initially be the identifier of the web process that 
constructed the object,
so we can use it for iframe isolation too.

(WebKit::WebPageProxy::didChangeBackForwardList):

Do not call `didChangeBackForwardList` delegates or 

[webkit-changes] [WebKit/WebKit] ca45ce: Unreviewed, stale PR gets uploaded

2024-05-22 Thread Yusuke Suzuki
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ca45ce0afc7301452f0d2f38cc6d33bafaed1b0e
  
https://github.com/WebKit/WebKit/commit/ca45ce0afc7301452f0d2f38cc6d33bafaed1b0e
  Author: Yusuke Suzuki 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/JavaScriptCore/runtime/JSGlobalObject.cpp

  Log Message:
  ---
  Unreviewed, stale PR gets uploaded
https://bugs.webkit.org/show_bug.cgi?id=274548
rdar://128562540

* Source/JavaScriptCore/runtime/JSGlobalObject.cpp:
(JSC::JSGlobalObject::init):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 708db4: textarea's width calculations aren't fully writing...

2024-05-22 Thread Luke Warlow
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 708db469479bb6c79a55f65532177b6964dd49b0
  
https://github.com/WebKit/WebKit/commit/708db469479bb6c79a55f65532177b6964dd49b0
  Author: Luke Warlow 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M LayoutTests/TestExpectations
M 
LayoutTests/imported/w3c/web-platform-tests/html/rendering/widgets/textarea-scrollbar-sizing-002-expected.txt
M Source/WebCore/rendering/RenderTextControlMultiLine.cpp

  Log Message:
  ---
  textarea's width calculations aren't fully writing mode aware
https://bugs.webkit.org/show_bug.cgi?id=274332

Reviewed by Aditya Keerthi.

Updates the width cacluations to take into account writing mode when decided 
whether to add scrollbar thickness.

* LayoutTests/TestExpectations:
* 
LayoutTests/imported/w3c/web-platform-tests/html/rendering/widgets/textarea-scrollbar-sizing-002-expected.txt:
* Source/WebCore/rendering/RenderTextControlMultiLine.cpp:
(WebCore::RenderTextControlMultiLine::preferredContentLogicalWidth const):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] c471e6: [Navigation] Implement cross-document navigate events

2024-05-22 Thread Patrick
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c471e606082fe814511934ed3dacaf7f76cdb31e
  
https://github.com/WebKit/WebKit/commit/c471e606082fe814511934ed3dacaf7f76cdb31e
  Author: Patrick Griffis 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/click-crossdocument-sameorigin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/location-crossdocument-sameorigin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/open-crossdocument-sameorigin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/submit-crossdocument-crossorigin-sameorigindomain.sub-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/submit-crossdocument-sameorigin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/intercept-cross-document-same-origin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/intercept-cross-origin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/intercept-detach-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/intercept-detach-multiple-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-anchor-cross-origin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-anchor-download-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-anchor-download-userInitiated-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-anchor-same-origin-cross-document-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-anchor-with-target-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-destination-getState-reload-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-form-get-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-history-go-0-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-meta-refresh-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-to-srcdoc-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigation-methods/return-value/reload-rejection-order-pagehide-unserializablestate-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/ordering-and-transition/transition-finished-mark-as-handled-expected.txt
M 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/click-crossdocument-crossorigin-sameorigindomain.sub-expected.txt
M 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/location-crossdocument-crossorigin-sameorigindomain.sub-expected.txt
M 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/open-crossdocument-crossorigin-sameorigindomain.sub-expected.txt
A 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/navigation-api/navigation-history-entry/current-basic-expected.txt
M Source/WTF/wtf/URL.cpp
M Source/WTF/wtf/URL.h
M Source/WebCore/loader/FrameLoader.cpp
M Source/WebCore/loader/FrameLoader.h

  Log Message:
  ---
  [Navigation] Implement cross-document navigate events
https://bugs.webkit.org/show_bug.cgi?id=274426

Reviewed by Alex Christensen.

This emits the NavigateEvent for cross-document navigations when the navigation 
is same-origin-domain.

* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/click-crossdocument-sameorigin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/location-crossdocument-sameorigin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/open-crossdocument-sameorigin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/submit-crossdocument-crossorigin-sameorigindomain.sub-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/submit-crossdocument-sameorigin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/intercept-cross-document-same-origin-expected.txt:
* 

[webkit-changes] [WebKit/WebKit] 9c2a2b: [Unified Text Replacement] When receiving text wit...

2024-05-22 Thread Richard Robinson
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 9c2a2b42f8fab8d7d5a776f5e2342772fa2ef6c7
  
https://github.com/WebKit/WebKit/commit/9c2a2b42f8fab8d7d5a776f5e2342772fa2ef6c7
  Author: Richard Robinson 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebKit/WebProcess/WebPage/Cocoa/UnifiedTextReplacementController.mm
M Source/WebKit/WebProcess/WebPage/Cocoa/WebPageCocoa.mm

  Log Message:
  ---
  [Unified Text Replacement] When receiving text with a replacement range, a 
trailing newline can sometimes go missing
https://bugs.webkit.org/show_bug.cgi?id=274504
rdar://128084320

Reviewed by Ryosuke Niwa.

When creating the context, omit any trailing newlines in the first place, since 
they will not be
actually used, and newlines behave semi-erratically anyways.

* Source/WebKit/WebProcess/WebPage/Cocoa/UnifiedTextReplacementController.mm:
(WebKit::UnifiedTextReplacementController::textReplacementSessionDidReceiveTextWithReplacementRange):
* Source/WebKit/WebProcess/WebPage/Cocoa/WebPageCocoa.mm:
(WebKit::WebPage::autocorrectionContextRange):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] ab5b67: Change my status in contributors.json

2024-05-22 Thread Yijia Huang
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ab5b6787c95f685280c1d539952f556cd7b4cdc0
  
https://github.com/WebKit/WebKit/commit/ab5b6787c95f685280c1d539952f556cd7b4cdc0
  Author: Yijia Huang 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M metadata/contributors.json

  Log Message:
  ---
  Change my status in contributors.json

Unreviewed.

* metadata/contributors.json:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] fb4278: [WebXR] Service video frame callbacks in WebXRSess...

2024-05-22 Thread Ada Chan
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: fb427896ee568c0bc173d1611575da7d8b26be49
  
https://github.com/WebKit/WebKit/commit/fb427896ee568c0bc173d1611575da7d8b26be49
  Author: Ada Chan 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/Modules/webxr/WebXRSession.cpp
M Source/WebCore/Modules/webxr/WebXRSession.h
M Source/WebCore/Modules/webxr/WebXRSystem.cpp
M Source/WebCore/Modules/webxr/WebXRSystem.h
M Source/WebCore/page/Page.cpp
M Source/WebCore/page/Page.h
M Source/WebKit/WebProcess/WebPage/WebPage.cpp

  Log Message:
  ---
  [WebXR] Service video frame callbacks in WebXRSession if page backgrounds 
during immersive WebXR session
https://bugs.webkit.org/show_bug.cgi?id=273901
rdar://124236441

Reviewed by Simon Fraser and Jer Noble.

Instead of not freezing the layer tree if the page gets backgrounded
during an immersive WebXR session so video frame callbacks can continue
to be serviced, just service the video frame callbacks in the XR session.

* Source/WebCore/Modules/webxr/WebXRSession.cpp:
(WebCore::WebXRSession::minimalUpdateRendering):
We'll special case servicing the video frame callbacks as the minimum render
updates we'll do in XR session's rAF if the page is backgrounded.
(WebCore::WebXRSession::onFrame):
* Source/WebCore/Modules/webxr/WebXRSession.h:
* Source/WebCore/Modules/webxr/WebXRSystem.cpp:
(WebCore::WebXRSystem::activeImmersiveSession const):
(WebCore::WebXRSystem::hasActiveImmersiveSession const): Deleted.
* Source/WebCore/Modules/webxr/WebXRSystem.h:
* Source/WebCore/page/Page.cpp:
(WebCore::Page::applicationDidEnterBackground):
(WebCore::Page::applicationWillEnterForeground):
(WebCore::Page::hasActiveImmersiveSession const):
(WebCore::Page::activeImmersiveXRSession const):
(WebCore::Page::shouldBlockLayerTreeFreezingForVideo): Deleted.
* Source/WebCore/page/Page.h:
* Source/WebKit/WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::updateDrawingAreaLayerTreeFreezeState):
Remove the logic to prevent the freezing of the layer tree if the
page is backgrounded due to an active immersive session. It's not good
for performance to continue render updates for the page when it's
backgrounded and not visible.

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 538cc4: [build.webkit.org] Do not try to download build pr...

2024-05-22 Thread Aakash J
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 538cc4d94b8a51496c3f982522eb0e50cb6aa732
  
https://github.com/WebKit/WebKit/commit/538cc4d94b8a51496c3f982522eb0e50cb6aa732
  Author: Aakash Jain 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Tools/CISupport/build-webkit-org/steps.py

  Log Message:
  ---
  [build.webkit.org] Do not try to download build product from master
https://bugs.webkit.org/show_bug.cgi?id=274538

Reviewed by Ryan Haddad.

We stopped uploading archives to build master in 276...@main.so, while 
downloading
archives, when download from s3 fails, there is no point in trying to fallback 
to
download from build master.

We should make it a hard failure when download from s3 fails.
This is similar to 276539@main for ews.

* Tools/CISupport/build-webkit-org/steps.py:
(DownloadBuiltProduct):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] e058a9: Web Inspector: CSS source editor can adopt fuzzy s...

2024-05-22 Thread Qianlang Chen
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: e058a9c99c8c35dac22dc6095343a4895a6fb168
  
https://github.com/WebKit/WebKit/commit/e058a9c99c8c35dac22dc6095343a4895a6fb168
  Author: Qianlang Chen 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js
M Source/WebInspectorUI/UserInterface/Base/Setting.js
M 
Source/WebInspectorUI/UserInterface/Controllers/CodeMirrorCompletionController.js
M Source/WebInspectorUI/UserInterface/Main.html
M Source/WebInspectorUI/UserInterface/Models/CSSCompletions.js
A Source/WebInspectorUI/UserInterface/Models/Completions.js
M Source/WebInspectorUI/UserInterface/Test.html
M Source/WebInspectorUI/UserInterface/Views/SettingsTabContentView.js

  Log Message:
  ---
  Web Inspector: CSS source editor can adopt fuzzy search for code completion
rdar://125030691
https://bugs.webkit.org/show_bug.cgi?id=271260

Reviewed by Devin Rousso.

Use CSSCompletions.executeQuery instead of CSSCompletions.startsWith
when language mode is CSS, since executeQuery uses fuzzy search.

Also add an setting item in the Settings tab's Experimental sub-tab to
optionally turn this feature off.

* 
Source/WebInspectorUI/UserInterface/Controllers/CodeMirrorCompletionController.js:
(WI.CodeMirrorCompletionController.prototype._generateCSSCompletions):
   - Optionally apply fuzzy search by using `executeQuery`.

(WI.CodeMirrorCompletionController.prototype.updateCompletions):
   - The argument `completions` could now also be an array of objects
 instead of always an array of strings (due to executeQuery
 returning an array of objects). Those objects also contain info on
 which characters match in the fuzzy search, so the suggestion box
 can bold the matching text. `this._completions` however does not
 need this extra info, so we extract only the texts from
 `completions` for that.

(WI.CodeMirrorCompletionController.prototype._applyCompletionHint):
(WI.CodeMirrorCompletionController.prototype._applyCompletionHint.update):
   - Fuzzy search can now match completions not necessarily starting
 with the same text as currentText. Only show the gray watermark
 text (hint marker in code) when it's actually a prefix match.

*   Source/WebInspectorUI/UserInterface/Models/Completions.js: Added.
* Source/WebInspectorUI/UserInterface/Models/CSSCompletions.js:
(WI.CSSCompletions.getCompletionText):
   - As the getCompletionText helper function doesn't have to work for
 only CSS suggestions, it better belongs to a class with a more
 generic name like Completion instead of CSSCompletion.
   - Leave a note to eventually migrate all other usages of the old
 CSSCompletion.getCompletionText to Completion.getCompletionText.

* Source/WebInspectorUI/UserInterface/Main.html:
* Source/WebInspectorUI/UserInterface/Test.html:
   - Include the new Completions.js source file.

* Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js:
* Source/WebInspectorUI/UserInterface/Base/Setting.js:
* Source/WebInspectorUI/UserInterface/Views/SettingsTabContentView.js:
   - Add an experimental setting item for this feature change.

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] ac7aec: Videos in in-window mode should have their WebKitP...

2024-05-22 Thread Dana Estra
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ac7aecf284923b3ad201b085a68b66a1d614b285
  
https://github.com/WebKit/WebKit/commit/ac7aecf284923b3ad201b085a68b66a1d614b285
  Author: Dana Estra 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/Modules/mediacontrols/MediaControlsHost.cpp
M Source/WebCore/Modules/mediacontrols/MediaControlsHost.h
M Source/WebCore/Modules/mediacontrols/MediaControlsHost.idl
M Source/WebCore/Modules/modern-media-controls/media/media-controller.js
M Source/WebCore/html/HTMLVideoElement.cpp
M Source/WebCore/html/HTMLVideoElement.h
M Source/WebCore/html/HTMLVideoElement.idl

  Log Message:
  ---
  Videos in in-window mode should have their WebKitPresentationMode property 
set to inline
https://bugs.webkit.org/show_bug.cgi?id=274477
rdar://128482499

Reviewed by Jer Noble.

This fix adds a method HTMLVideoElement::webkitPresentationModeForBindings(). 
The
default media controls now consult MediaControlsHost::inWindowFullscreen() to
determine that a video is in in-window mode.

* Source/WebCore/Modules/mediacontrols/MediaControlsHost.cpp:
(WebCore::MediaControlsHost::inWindowFullscreen const):
* Source/WebCore/Modules/mediacontrols/MediaControlsHost.h:
* Source/WebCore/Modules/mediacontrols/MediaControlsHost.idl:
* Source/WebCore/Modules/modern-media-controls/media/media-controller.js:
(MediaController.prototype.get isFullscreen):
(MediaController.prototype._updateControlsIfNeeded):
* Source/WebCore/html/HTMLVideoElement.cpp:
(WebCore::HTMLVideoElement::webkitPresentationModeForBindings const):
* Source/WebCore/html/HTMLVideoElement.h:
* Source/WebCore/html/HTMLVideoElement.idl:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] c29a71: Add myself (Darryl Pogue) to contributors.json

2024-05-22 Thread Darryl Pogue
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c29a713c5f577bbbae708cb993ccdadb5045d83f
  
https://github.com/WebKit/WebKit/commit/c29a713c5f577bbbae708cb993ccdadb5045d83f
  Author: Darryl Pogue 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M metadata/contributors.json

  Log Message:
  ---
  Add myself (Darryl Pogue) to contributors.json
https://bugs.webkit.org/show_bug.cgi?id=274541

Reviewed by Jonathan Bedard.

* metadata/contributors.json:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 8cb609: [WGSL] Fix failing wgslc tests

2024-05-22 Thread Tadeu Zagallo
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 8cb609477c197cfc25fd66fab6f51b7017c7e8dd
  
https://github.com/WebKit/WebKit/commit/8cb609477c197cfc25fd66fab6f51b7017c7e8dd
  Author: Tadeu Zagallo 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebGPU/WGSL/AST/ASTForward.h
M Source/WebGPU/WGSL/AST/ASTParameter.h
M Source/WebGPU/WGSL/AST/ASTVariable.h
M Source/WebGPU/WGSL/GlobalVariableRewriter.cpp
M Source/WebGPU/WGSL/Metal/MetalFunctionWriter.cpp
M Source/WebGPU/WGSL/WGSLShaderModule.h
M Source/WebGPU/WGSL/tests/valid/packing.wgsl

  Log Message:
  ---
  [WGSL] Fix failing wgslc tests
https://bugs.webkit.org/show_bug.cgi?id=274479
rdar://128485179

Reviewed by Mike Wyrzykowski.

There were 2 issues after 278991@main:
- we emitted definitions for `__unpack(PackedVec3)` even when we didn't emit
  the PackedVec3 struct
- in a previous patch we tried skipping some explicit pack/unpack calls for
  vec3/packed_vec3, but that doesn't work for PackedVec3, so we can no longer
  skip these calls.

* Source/WebGPU/WGSL/AST/ASTForward.h:
* Source/WebGPU/WGSL/AST/ASTParameter.h:
* Source/WebGPU/WGSL/AST/ASTVariable.h:
* Source/WebGPU/WGSL/GlobalVariableRewriter.cpp:
(WGSL::RewriteGlobalVariables::visitCallee):
(WGSL::RewriteGlobalVariables::visit):
(WGSL::RewriteGlobalVariables::pack):
(WGSL::RewriteGlobalVariables::packStructResource):
(WGSL::RewriteGlobalVariables::packArrayResource):
(WGSL::RewriteGlobalVariables::insertMaterializations):
* Source/WebGPU/WGSL/Metal/MetalFunctionWriter.cpp:
(WGSL::Metal::FunctionDefinitionWriter::emitNecessaryHelpers):
(WGSL::Metal::FunctionDefinitionWriter::visit):
(WGSL::Metal::FunctionDefinitionWriter::shouldPackType const):
(WGSL::Metal::FunctionDefinitionWriter::emitPackedVector):
(WGSL::Metal::FunctionDefinitionWriter::serializeVariable):
* Source/WebGPU/WGSL/WGSLShaderModule.h:
(WGSL::ShaderModule::usesPackVector const):
(WGSL::ShaderModule::setUsesPackVector):
(WGSL::ShaderModule::clearUsesPackVector):
(WGSL::ShaderModule::usesUnpackVector const):
(WGSL::ShaderModule::setUsesUnpackVector):
(WGSL::ShaderModule::clearUsesUnpackVector):
* Source/WebGPU/WGSL/tests/valid/packing.wgsl:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 625de0: Disable thread-safety-reference-return warnings in...

2024-05-22 Thread youennf
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 625de0728603e766a988d7a6ea9f712806162a3f
  
https://github.com/WebKit/WebKit/commit/625de0728603e766a988d7a6ea9f712806162a3f
  Author: Youenn Fablet 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/ThirdParty/libwebrtc/Configurations/Base.xcconfig

  Log Message:
  ---
  Disable thread-safety-reference-return warnings in libwebrtc
rdar://126802873
https://bugs.webkit.org/show_bug.cgi?id=274508

Reviewed by David Kilzer.

We disable thread-safety-reference-return as libwebrtc is not yet ready for 
this.
A few places would need additional annotations, and some other places are 
checking that the method is called in the right thread,
not that a mutex is held.

* Source/ThirdParty/libwebrtc/Configurations/Base.xcconfig:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] f05e0d: Unreviewed, reverting 279084@main (69a3317d5271)

2024-05-22 Thread mwyrzykowski
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: f05e0d98cff6255208eaa6d018199b13814a8389
  
https://github.com/WebKit/WebKit/commit/f05e0d98cff6255208eaa6d018199b13814a8389
  Author: Mike Wyrzykowski 
  Date:   2024-05-22 (Wed, 22 May 2024)

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

  Log Message:
  ---
  Unreviewed, reverting 279084@main (69a3317d5271)
https://bugs.webkit.org/show_bug.cgi?id=274537


279084@main caused a regression in duplicate command buffer submission

Reverted change:

[WebGPU] Invalidate all command buffers on submit, regardless of it the 
submit call succeeds
https://bugs.webkit.org/show_bug.cgi?id=274244
rdar://128177642
279084@main (69a3317d5271)

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 513f6e: Update base64EncodeToString() / base64URLEncodeToS...

2024-05-22 Thread Chris Dumez
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 513f6e7095e6a8730bfd06084c8b3e68a974e856
  
https://github.com/WebKit/WebKit/commit/513f6e7095e6a8730bfd06084c8b3e68a974e856
  Author: Chris Dumez 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WTF/wtf/text/Base64.h
M Source/WebCore/Modules/identity/DigitalCredential.cpp
M Source/WebCore/Modules/webauthn/PublicKeyCredential.cpp
M Source/WebCore/Modules/webauthn/WebAuthenticationUtils.cpp
M Source/WebCore/Modules/websockets/WebSocketHandshake.cpp
M Source/WebCore/crypto/cocoa/CryptoKeyECMac.cpp
M Source/WebCore/crypto/gcrypt/CryptoKeyECGCrypt.cpp
M Source/WebCore/inspector/DOMPatchSupport.cpp
M Source/WebCore/inspector/agents/InspectorNetworkAgent.cpp
M Source/WebCore/inspector/agents/InspectorPageAgent.cpp
M Source/WebCore/loader/cocoa/PrivateClickMeasurementCocoa.mm
M Source/WebCore/page/Base64Utilities.cpp
M Source/WebCore/platform/encryptedmedia/clearkey/CDMClearKey.cpp
M Source/WebCore/platform/graphics/win/FontCustomPlatformDataWin.cpp
M Source/WebCore/platform/network/soup/SoupNetworkSession.cpp
M Source/WebCore/rendering/adwaita/RenderThemeAdwaita.cpp
M Source/WebCore/storage/StorageUtilities.cpp
M Source/WebCore/workers/service/server/SWScriptStorage.cpp
M 
Source/WebKit/NetworkProcess/PrivateClickMeasurement/PrivateClickMeasurementManager.cpp
M Source/WebKit/NetworkProcess/storage/NetworkStorageManager.cpp
M Source/WebKit/UIProcess/Automation/cairo/WebAutomationSessionCairo.cpp
M Source/WebKit/UIProcess/Automation/gtk/WebAutomationSessionGtk.cpp
M Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalAuthenticator.mm
M Source/WebKit/UIProcess/WebAuthentication/Mock/MockLocalConnection.mm
M Tools/TestWebKitAPI/Tests/WebCore/cocoa/PrivateClickMeasurementCocoa.mm
M Tools/TestWebKitAPI/Tests/WebKitCocoa/EventAttribution.mm
M Tools/TestWebKitAPI/cocoa/HTTPServer.mm

  Log Message:
  ---
  Update base64EncodeToString() / base64URLEncodeToString() overloads
https://bugs.webkit.org/show_bug.cgi?id=274503

Reviewed by Sihui Liu and Darin Adler.

Update base64EncodeToString() / base64URLEncodeToString() overloads to force
the call sites to provide either a `std::span` or a
`std::span`.

This is a step towards our broader std::span adoption and using more consistent
typing to represent bytes.

* Source/WTF/wtf/text/Base64.h:
* Source/WebCore/Modules/identity/DigitalCredential.cpp:
(WebCore::DigitalCredential::DigitalCredential):
* Source/WebCore/Modules/webauthn/PublicKeyCredential.cpp:
(WebCore::PublicKeyCredential::PublicKeyCredential):
* Source/WebCore/Modules/webauthn/WebAuthenticationUtils.cpp:
(WebCore::buildClientDataJson):
* Source/WebCore/Modules/websockets/WebSocketHandshake.cpp:
(WebCore::WebSocketHandshake::getExpectedWebSocketAccept):
* Source/WebCore/crypto/cocoa/CryptoKeyECMac.cpp:
(WebCore::CryptoKeyEC::platformAddFieldElements const):
* Source/WebCore/crypto/gcrypt/CryptoKeyECGCrypt.cpp:
(WebCore::CryptoKeyEC::platformAddFieldElements const):
* Source/WebCore/inspector/DOMPatchSupport.cpp:
(WebCore::DOMPatchSupport::createDigest):
* Source/WebCore/inspector/agents/InspectorNetworkAgent.cpp:
(WebCore::InspectorNetworkAgent::getSerializedCertificate):
* Source/WebCore/inspector/agents/InspectorPageAgent.cpp:
(WebCore::InspectorPageAgent::archive):
* Source/WebCore/loader/cocoa/PrivateClickMeasurementCocoa.mm:
(WebCore::PrivateClickMeasurement::calculateAndUpdateUnlinkableToken):
(WebCore::PrivateClickMeasurement::calculateAndUpdateSecretToken):
* Source/WebCore/page/Base64Utilities.cpp:
(WebCore::Base64Utilities::btoa):
* Source/WebCore/platform/encryptedmedia/clearkey/CDMClearKey.cpp:
(WebCore::extractKeyidsFromCencInitData):
(WebCore::CDMInstanceSessionClearKey::removeSessionData):
* Source/WebCore/platform/graphics/win/FontCustomPlatformDataWin.cpp:
(WebCore::createUniqueFontName):
* Source/WebCore/platform/network/soup/SoupNetworkSession.cpp:
(WebCore::HostTLSCertificateSet::computeCertificateHash):
* Source/WebCore/rendering/adwaita/RenderThemeAdwaita.cpp:
(WebCore::RenderThemeAdwaita::mediaControlsBase64StringForIconNameAndType):
* Source/WebCore/storage/StorageUtilities.cpp:
(WebCore::StorageUtilities::encodeSecurityOriginForFileName):
* Source/WebCore/workers/service/server/SWScriptStorage.cpp:
(WebCore::SWScriptStorage::sha2Hash const):
* 
Source/WebKit/NetworkProcess/PrivateClickMeasurement/PrivateClickMeasurementManager.cpp:
(WebKit::PrivateClickMeasurementManager::fireConversionRequest):
* Source/WebKit/NetworkProcess/storage/NetworkStorageManager.cpp:
(WebKit::encode):
* Source/WebKit/UIProcess/Automation/cairo/WebAutomationSessionCairo.cpp:
(WebKit::base64EncodedPNGData):
* Source/WebKit/UIProcess/Automation/gtk/WebAutomationSessionGtk.cpp:
(WebKit::base64EncodedPNGData):
* Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalAuthenticator.mm:

[webkit-changes] [WebKit/WebKit] 2a8d5d: Fix sysctl read sandbox violation in the GPU process

2024-05-22 Thread Per Arne Vollan
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 2a8d5d0ba6a64bf7e655deb0152cae1e9c3c8c6a
  
https://github.com/WebKit/WebKit/commit/2a8d5d0ba6a64bf7e655deb0152cae1e9c3c8c6a
  Author: Per Arne Vollan 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.GPU.sb.in

  Log Message:
  ---
  Fix sysctl read sandbox violation in the GPU process
https://bugs.webkit.org/show_bug.cgi?id=274528
rdar://122952474

Reviewed by Chris Dumez.

Fix sysctl read sandbox violation in the GPU process on iOS by allowing reading 
a sysctl name
which has been confirmed to be in use.

* Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.GPU.sb.in:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 57affb: [JSC] Handler IC Getter / Setter / Proxy accessors...

2024-05-22 Thread Yusuke Suzuki
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 57affb62969db66ef07975567f4876d62d86da55
  
https://github.com/WebKit/WebKit/commit/57affb62969db66ef07975567f4876d62d86da55
  Author: Yusuke Suzuki 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M LayoutTests/platform/mac/TestExpectations
M Source/JavaScriptCore/bytecode/InlineCacheCompiler.cpp
M Source/JavaScriptCore/dfg/DFGJITCode.cpp
M Source/JavaScriptCore/llint/LLIntData.cpp
M Source/JavaScriptCore/runtime/JSGlobalObject.cpp
M Source/JavaScriptCore/runtime/JSGlobalObject.h
M Source/JavaScriptCore/runtime/JSGlobalObjectInlines.h
M Source/JavaScriptCore/runtime/StructureInlines.h

  Log Message:
  ---
  [JSC] Handler IC Getter / Setter / Proxy accessors should not use 
m_globalObject
https://bugs.webkit.org/show_bug.cgi?id=274485
rdar://128494111

Reviewed by Mark Lam.

Since Handler IC can be shared across JSGlobalObjects, the generated code 
should not rely on that.
This patch fixes so that all access are done through JSGlobalObject offered at 
runtime.
We made all necessary fields of JSGlobalObject from JIT.

* Source/JavaScriptCore/bytecode/InlineCacheCompiler.cpp:
(JSC::InlineCacheCompiler::generateWithGuard):
(JSC::InlineCacheCompiler::generateImpl):
(JSC::InlineCacheCompiler::emitProxyObjectAccess):
* Source/JavaScriptCore/runtime/JSGlobalObject.cpp:
(JSC::JSGlobalObject::init):
(JSC::JSGlobalObject::visitChildrenImpl):
* Source/JavaScriptCore/runtime/JSGlobalObject.h:
(JSC::JSGlobalObject::symbolPrototype const):
(JSC::JSGlobalObject::dateStructure const):
(JSC::JSGlobalObject::symbolObjectStructure const):
(JSC::JSGlobalObject::offsetOfPerformProxyObjectHasFunction):
(JSC::JSGlobalObject::offsetOfPerformProxyObjectGetFunction):
(JSC::JSGlobalObject::offsetOfPerformProxyObjectGetByValFunction):
(JSC::JSGlobalObject::offsetOfPerformProxyObjectSetStrictFunction):
(JSC::JSGlobalObject::offsetOfPerformProxyObjectSetSloppyFunction):
(JSC::JSGlobalObject::offsetOfNullSetterStrictFunction):
(JSC::JSGlobalObject::offsetOfStringPrototype):
(JSC::JSGlobalObject::offsetOfBigIntPrototype):
(JSC::JSGlobalObject::offsetOfSymbolPrototype):
* Source/JavaScriptCore/runtime/JSGlobalObjectInlines.h:
(JSC::JSGlobalObject::performProxyObjectHasFunction const):
(JSC::JSGlobalObject::performProxyObjectGetFunction const):
(JSC::JSGlobalObject::performProxyObjectGetFunctionConcurrently const):
(JSC::JSGlobalObject::performProxyObjectGetByValFunction const):
(JSC::JSGlobalObject::performProxyObjectGetByValFunctionConcurrently const):
(JSC::JSGlobalObject::performProxyObjectSetSloppyFunction const):
(JSC::JSGlobalObject::performProxyObjectSetSloppyFunctionConcurrently const):
(JSC::JSGlobalObject::performProxyObjectSetStrictFunction const):
(JSC::JSGlobalObject::performProxyObjectSetStrictFunctionConcurrently const):
* Source/JavaScriptCore/runtime/StructureInlines.h:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 19697c: [UnifiedPDF] Add the concept of "rows" to PDFDocum...

2024-05-22 Thread Simon Fraser
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 19697c5f3ef870f6c4ab84937a06156ee11d0a53
  
https://github.com/WebKit/WebKit/commit/19697c5f3ef870f6c4ab84937a06156ee11d0a53
  Author: Simon Fraser 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/PDFDocumentLayout.h
M Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/PDFDocumentLayout.mm
M Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.h
M Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm

  Log Message:
  ---
  [UnifiedPDF] Add the concept of "rows" to PDFDocumentLayout
https://bugs.webkit.org/show_bug.cgi?id=274496
rdar://128504528

Reviewed by Abrar Rahman Protyasha.

Discrete (non-scrolling) mode makes heavy use of the "row" concept, where
a row is a set of one or two pages viewed together. So introduce PDFLayoutRow,
and pass an optional row argument to nearestPageIndexForDocumentPoint().

Other namespace-related cleanup and utility additions for rows.

* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/PDFDocumentLayout.h:
(WebKit::PDFDocumentLayout::isSinglePageDisplayMode):
(WebKit::PDFDocumentLayout::isTwoUpDisplayMode):
(WebKit::PDFDocumentLayout::isScrollingDisplayMode):
(WebKit::PDFDocumentLayout::isDiscreteDisplayMode):
(WebKit::PDFDocumentLayout::isSinglePageDisplayMode const):
(WebKit::PDFDocumentLayout::isTwoUpDisplayMode const):
(WebKit::PDFDocumentLayout::isScrollingDisplayMode const):
(WebKit::PDFDocumentLayout::isDiscreteDisplayMode const):
(WebKit::PDFLayoutRow::numPages const):
(WebKit::PDFLayoutRow::containsPage const):
* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/PDFDocumentLayout.mm:
(WebKit::PDFDocumentLayout::isRightPageIndex const):
(WebKit::PDFDocumentLayout::isLastPageIndex const):
(WebKit::PDFDocumentLayout::lastPageIndex const):
(WebKit::PDFDocumentLayout::isFirstPageOfRow const):
(WebKit::PDFDocumentLayout::indexForPage const):
(WebKit::PDFDocumentLayout::nearestPageIndexForDocumentPoint const):
(WebKit::PDFDocumentLayout::pageIndexAndPagePointForDocumentYOffset const):
(WebKit::PDFDocumentLayout::updateLayout):
(WebKit::PDFDocumentLayout::rowCount const):
(WebKit::PDFDocumentLayout::rows const):
(WebKit::PDFDocumentLayout::rowForPageIndex const):
(WebKit::PDFDocumentLayout::rowIndexForPageIndex const):
(WebKit::PDFDocumentLayout::layoutBoundsForRow const):
* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.h:
* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm:
(WebKit::UnifiedPDFPlugin::visibleRow const):
(WebKit::UnifiedPDFPlugin::shouldDisplayPage):
(WebKit::UnifiedPDFPlugin::indexForCurrentPageInView const):
(WebKit::UnifiedPDFPlugin::handleMouseEvent):
(WebKit::UnifiedPDFPlugin::createContextMenu const):
(WebKit::UnifiedPDFPlugin::continueAutoscroll):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 76a6ff: nonce hiding in SVG is buggy

2024-05-22 Thread Wenson Hsieh
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 76a6ff4a88da11885fb1c5b8af8b41bf0a7844b5
  
https://github.com/WebKit/WebKit/commit/76a6ff4a88da11885fb1c5b8af8b41bf0a7844b5
  Author: Ryosuke Niwa 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
A 
LayoutTests/http/tests/security/contentSecurityPolicy/nonce-hiding-on-svg-script-expected.txt
A 
LayoutTests/http/tests/security/contentSecurityPolicy/nonce-hiding-on-svg-script.py
M Source/WebCore/dom/ScriptElement.cpp
M Source/WebCore/svg/SVGElement.cpp

  Log Message:
  ---
  nonce hiding in SVG is buggy
https://bugs.webkit.org/show_bug.cgi?id=268598
rdar://122151552

Reviewed by Chris Dumez.

The bug was caused by SVGElement::insertedIntoAncestor hiding nonce after it 
had an early exit for returning
InsertedIntoAncestorResult::NeedsPostInsertionCallback. Fixed the bug by hiding 
it before this early exit.

* 
LayoutTests/http/tests/security/contentSecurityPolicy/nonce-hiding-on-svg-script-expected.txt:
 Added.
* 
LayoutTests/http/tests/security/contentSecurityPolicy/nonce-hiding-on-svg-script.py:
 Added.
* Source/WebCore/dom/ScriptElement.cpp:
(WebCore::ScriptElement::didFinishInsertingNode):
* Source/WebCore/svg/SVGElement.cpp:
(WebCore::SVGElement::insertedIntoAncestor):

Originally-landed-as: 272448.567@safari-7618-branch (d915a3b6357c). 
rdar://128504044
Canonical link: https://commits.webkit.org/279131@main


  Commit: 9e5283739c9809f145488a20f8a52f09fd3a85f4
  
https://github.com/WebKit/WebKit/commit/9e5283739c9809f145488a20f8a52f09fd3a85f4
  Author: Chris Dumez 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/workers/WorkerOrWorkletThread.cpp

  Log Message:
  ---
  Flaky crash under WorkerDedicatedRunLoop::runCleanupTasks() during fuzzing
https://bugs.webkit.org/show_bug.cgi?id=269731
rdar://121961101

Reviewed by Brent Fulgham.

I haven't been able to reproduce but based on the ASAN report, it looks
like the WorkerOrWorkletGlobalScope is getting destroyed in the middle
of WorkerDedicatedRunLoop::runCleanupTasks(), causing a use-after free
of the context.

To make sure this can't happen, apply our smart pointer adoption rules
and make sure the WorkerOrWorkletGlobalScope is being protected on the
stack at the call site of WorkerDedicatedRunLoop::run() before passing
it in argument.

* Source/WebCore/workers/WorkerOrWorkletThread.cpp:
(WebCore::WorkerOrWorkletThread::runEventLoop):

Originally-landed-as: 272448.578@safari-7618-branch (459d377c63c2). 
rdar://128504577
Canonical link: https://commits.webkit.org/279132@main


  Commit: 5944980db644fb2ea879bf3e2d6e5efd47156cdc
  
https://github.com/WebKit/WebKit/commit/5944980db644fb2ea879bf3e2d6e5efd47156cdc
  Author: Wenson Hsieh 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/loader/EmptyClients.cpp
M Source/WebCore/page/EditorClient.h
M Source/WebCore/page/LocalFrame.cpp
M Source/WebKit/UIProcess/WebPageProxy.cpp
M Source/WebKit/UIProcess/WebPageProxy.h
M Source/WebKit/UIProcess/WebPageProxy.messages.in
M Source/WebKit/WebProcess/WebCoreSupport/WebEditorClient.cpp
M Source/WebKit/WebProcess/WebCoreSupport/WebEditorClient.h
M Source/WebKit/WebProcess/WebPage/WebPage.cpp
M Source/WebKit/WebProcess/WebPage/WebPage.h
M Source/WebKitLegacy/mac/WebCoreSupport/WebEditorClient.h

  Log Message:
  ---
  Compromised web process can grant pasteboard access by spamming 
WebPage::RequestDOMPasteAccess
https://bugs.webkit.org/show_bug.cgi?id=269769
rdar://97343267

Reviewed by Ryosuke Niwa and Richard Robinson.

It's currently possible for a compromised web process to send arbitrary 
`originIdentifiers` through
`requestDOMPasteAccess` to the UI process during programmatic paste. Since the 
UI process
automatically grants programmatic pasteboard access in the case where the 
incoming origin ID matches
the origin ID of the current pasteboard content, it's possible to send a large 
number of origins
through this IPC endpoint, with the end goal of discovering both (1) which 
website origin the user
last copied from, and (2) the contents of the pasteboard in the case where the 
pasteboard copied
from web content in WebKit.

To mitigate this attack vector, we add a `FrameIdentifier` in the IPC endpoint, 
which is used in the
UI process to verify that:

1.  The incoming frame ID corresponds to a frame underneath the destination 
page.
2.  The pasteboard origin matches the origin of the frame, unless the 
pasteboard origin is an opaque
(null) origin.

Additionally, we throttle pasteboard access requests to a maximum of 10 
different domains every 5
seconds, to mitigate another variation on this attack vector where the 
compromised web process loads
a large number of cross-origin frames and requests pasteboard access on behalf 
of those origins, in
an attempt to get around the limitations above.

With this change, a compromised 

[webkit-changes] [WebKit/WebKit] 2c9a11: [Skia][Nicosia] Add support for accelerated offscr...

2024-05-22 Thread Carlos Garcia Campos
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 2c9a11ae2283ca6a64bea080f8a7187af59c8dfd
  
https://github.com/WebKit/WebKit/commit/2c9a11ae2283ca6a64bea080f8a7187af59c8dfd
  Author: Carlos Garcia Campos 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/html/CanvasBase.cpp
M Source/WebCore/platform/graphics/nicosia/NicosiaImageBufferPipe.cpp
M Source/WebCore/platform/graphics/nicosia/NicosiaImageBufferPipe.h

  Log Message:
  ---
  [Skia][Nicosia] Add support for accelerated offscreen canvas
https://bugs.webkit.org/show_bug.cgi?id=274461

Reviewed by Miguel Gomez.

* Source/WebCore/html/CanvasBase.cpp:
(WebCore::CanvasBase::shouldAccelerate const):
* Source/WebCore/platform/graphics/nicosia/NicosiaImageBufferPipe.cpp:
(Nicosia::NicosiaImageBufferPipeSource::handle):
* Source/WebCore/platform/graphics/nicosia/NicosiaImageBufferPipe.h:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 567bd6: [WPE][Qt] Add support for building QtWPE against Qt 6

2024-05-22 Thread Nikolas Zimmermann
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 567bd6a58583b3944fc027e0d493d117aefb5337
  
https://github.com/WebKit/WebKit/commit/567bd6a58583b3944fc027e0d493d117aefb5337
  Author: Nikolas Zimmermann 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebKit/PlatformWPE.cmake
R Source/WebKit/UIProcess/API/wpe/qt/CMakeLists.txt
R Source/WebKit/UIProcess/API/wpe/qt/WPEQmlExtensionPlugin.cpp
R Source/WebKit/UIProcess/API/wpe/qt/WPEQmlExtensionPlugin.h
R Source/WebKit/UIProcess/API/wpe/qt/WPEQtView.cpp
R Source/WebKit/UIProcess/API/wpe/qt/WPEQtView.h
R Source/WebKit/UIProcess/API/wpe/qt/WPEQtViewBackend.cpp
R Source/WebKit/UIProcess/API/wpe/qt/WPEQtViewBackend.h
R Source/WebKit/UIProcess/API/wpe/qt/WPEQtViewLoadRequest.cpp
R Source/WebKit/UIProcess/API/wpe/qt/WPEQtViewLoadRequest.h
R Source/WebKit/UIProcess/API/wpe/qt/WPEQtViewLoadRequestPrivate.h
R Source/WebKit/UIProcess/API/wpe/qt/qmldir
A Source/WebKit/UIProcess/API/wpe/qt5/CMakeLists.txt
A Source/WebKit/UIProcess/API/wpe/qt5/WPEQmlExtensionPlugin.cpp
A Source/WebKit/UIProcess/API/wpe/qt5/WPEQmlExtensionPlugin.h
A Source/WebKit/UIProcess/API/wpe/qt5/WPEQtView.cpp
A Source/WebKit/UIProcess/API/wpe/qt5/WPEQtView.h
A Source/WebKit/UIProcess/API/wpe/qt5/WPEQtViewBackend.cpp
A Source/WebKit/UIProcess/API/wpe/qt5/WPEQtViewBackend.h
A Source/WebKit/UIProcess/API/wpe/qt5/WPEQtViewLoadRequest.cpp
A Source/WebKit/UIProcess/API/wpe/qt5/WPEQtViewLoadRequest.h
A Source/WebKit/UIProcess/API/wpe/qt5/WPEQtViewLoadRequestPrivate.h
A Source/WebKit/UIProcess/API/wpe/qt5/qmldir
A Source/WebKit/UIProcess/API/wpe/qt6/WPEDisplayQtQuick.cpp
A Source/WebKit/UIProcess/API/wpe/qt6/WPEDisplayQtQuick.h
A Source/WebKit/UIProcess/API/wpe/qt6/WPEQmlExtensionPlugin.cpp
A Source/WebKit/UIProcess/API/wpe/qt6/WPEQmlExtensionPlugin.h
A Source/WebKit/UIProcess/API/wpe/qt6/WPEQtView.cpp
A Source/WebKit/UIProcess/API/wpe/qt6/WPEQtView.h
A Source/WebKit/UIProcess/API/wpe/qt6/WPEQtViewLoadRequest.cpp
A Source/WebKit/UIProcess/API/wpe/qt6/WPEQtViewLoadRequest.h
A Source/WebKit/UIProcess/API/wpe/qt6/WPEQtViewLoadRequestPrivate.h
A Source/WebKit/UIProcess/API/wpe/qt6/WPEViewQtQuick.cpp
A Source/WebKit/UIProcess/API/wpe/qt6/WPEViewQtQuick.h
A Source/WebKit/UIProcess/API/wpe/qt6/qmldir
M Source/cmake/OptionsWPE.cmake
M Tools/MiniBrowser/wpe/CMakeLists.txt
R Tools/MiniBrowser/wpe/qt/CMakeLists.txt
R Tools/MiniBrowser/wpe/qt/main.cpp
R Tools/MiniBrowser/wpe/qt/main.qml
R Tools/MiniBrowser/wpe/qt/qml.qrc
A Tools/MiniBrowser/wpe/qt5/CMakeLists.txt
A Tools/MiniBrowser/wpe/qt5/main.cpp
A Tools/MiniBrowser/wpe/qt5/main.qml
A Tools/MiniBrowser/wpe/qt5/qml.qrc
A Tools/MiniBrowser/wpe/qt6/CMakeLists.txt
A Tools/MiniBrowser/wpe/qt6/main.cpp
A Tools/MiniBrowser/wpe/qt6/main.qml
A Tools/MiniBrowser/wpe/qt6/qml.qrc
M Tools/Scripts/run-qt-wpe-minibrowser
M Tools/Scripts/webkitdirs.pm
M Tools/Scripts/webkitpy/style/checker.py
M Tools/TestWebKitAPI/Tests/WPEQt/WPEQtTest.cpp
M Tools/TestWebKitAPI/Tests/WPEQt/WPEQtTest.h
M Tools/TestWebKitAPI/glib/CMakeLists.txt

  Log Message:
  ---
  [WPE][Qt] Add support for building QtWPE against Qt 6
https://bugs.webkit.org/show_bug.cgi?id=270161

Reviewed by Carlos Garcia Campos.

- Keep the existing Qt 5 based WPEQt implementation and rename:
  - Source/WebKit/UIProcess/API/wpe/qt -> Source/WebKit/UIProcess/API/wpe/qt5
  - Tools/MiniBrowser/wpe/qt -> Tools/MiniBrowser/wpe/qt5

- Add a new WPEQt implementation for Qt 6, based on the new WPE platform
  API, that was recently developed, not relying on WPEBackend(-fdo) anymore.

  Touch / wheel / hover / mouse / keyboard events are supported, only
  keyboard+mouse events were tested so far.

- Polish UI: Add navigation buttons (back|forward|reload) / URL bar for
  Qt 6 MiniBrowser

- Make the WebKitWebView ref-counted in WPEQtView, by wrapping in
  GRefPtr (needs some moc related plumbing to allow for config.h
  inclusion, which is a pre-requisite to get wtf/ headers includable).

Tested using:
build-webkit --wpe --debug --cmakeargs "-DUSE_QT6=ON|OFF"
run-qt-wpe-minibrowser --debug https://www.igalia.com

Covered by existing tests, no change in default behavior.

* Source/WebKit/PlatformWPE.cmake:
* Source/WebKit/UIProcess/API/wpe/qt5/CMakeLists.txt: Renamed from 
Source/WebKit/UIProcess/API/wpe/qt/CMakeLists.txt.
* Source/WebKit/UIProcess/API/wpe/qt5/WPEQmlExtensionPlugin.cpp: Copied from 
Source/WebKit/UIProcess/API/wpe/qt/WPEQmlExtensionPlugin.cpp.
* Source/WebKit/UIProcess/API/wpe/qt5/WPEQmlExtensionPlugin.h: Copied from 
Source/WebKit/UIProcess/API/wpe/qt/WPEQmlExtensionPlugin.h.
* Source/WebKit/UIProcess/API/wpe/qt5/WPEQtView.cpp: Copied from 
Source/WebKit/UIProcess/API/wpe/qt/WPEQtView.cpp.
* Source/WebKit/UIProcess/API/wpe/qt5/WPEQtView.h: Copied 

[webkit-changes] [WebKit/WebKit] 403141: ASSERTION FAILED: isPublicSuffixCF(publicSuffix) i...

2024-05-22 Thread Sihui
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 4031410eeeb157432f0a6e0ca143e54d02665096
  
https://github.com/WebKit/WebKit/commit/4031410eeeb157432f0a6e0ca143e54d02665096
  Author: Sihui Liu 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/Headers.cmake
M Source/WebCore/WebCore.xcodeproj/project.pbxproj
A Source/WebCore/platform/PublicSuffix.h
M Source/WebCore/platform/PublicSuffixStore.cpp
M Source/WebCore/platform/PublicSuffixStore.h
M Source/WebCore/platform/cocoa/PublicSuffixStoreCocoa.mm
M Source/WebKit/Shared/GoToBackForwardItemParameters.h
M Source/WebKit/Shared/GoToBackForwardItemParameters.serialization.in
M Source/WebKit/Shared/LoadParameters.h
M Source/WebKit/Shared/LoadParameters.serialization.in
M Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in
M Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp

  Log Message:
  ---
  ASSERTION FAILED: isPublicSuffixCF(publicSuffix) in 
PublicSuffixStore::addPublicSuffix
https://bugs.webkit.org/show_bug.cgi?id=274418
rdar://128255984

Reviewed by Chris Dumez.

The assertion was introduced in 276834@main to help debug rdar://125417343, 
where we suspected the cause is unexpected
entry being added to public suffix cache, like 276635@main. However, because UI 
process and web process have different
sandbox access, isPublicSuffixCF (_CFHostIsDomainTopLevel) might return 
different values on the same string. That means,
in web process, the assertion would be hit even though the string is a valid 
public suffix according to UI process's
check.

As the original goal is to ensure UI process only sends valid public suffix (or 
empty string) to web process, the patch
does the hardening by introducing a new class PublicSuffix. UI process needs to 
explicity contruct it from URL with
PublicSuffixStore and send PublicSuffix object to web process, which is less 
error-prone than allowing UI process sends
public suffix in string. With this change, we no longer need the assertion.

* Source/WebCore/Headers.cmake:
* Source/WebCore/WebCore.xcodeproj/project.pbxproj:
* Source/WebCore/platform/PublicSuffix.h: Added.
(WebCore::PublicSuffix::fromRawString):
(WebCore::PublicSuffix::isValid const):
(WebCore::PublicSuffix::string const):
(WebCore::PublicSuffix::isolatedCopy const):
(WebCore::PublicSuffix::PublicSuffix):
(WebCore::PublicSuffix::operator== const):
(WebCore::PublicSuffix::isHashTableDeletedValue const):
(WebCore::PublicSuffix::hash const):
(WebCore::PublicSuffix::PublicSuffixHash::hash):
(WebCore::PublicSuffix::PublicSuffixHash::equal):
* Source/WebCore/platform/PublicSuffixStore.cpp:
(WebCore::PublicSuffixStore::publicSuffix const):
* Source/WebCore/platform/PublicSuffixStore.h:
(): Deleted.
* Source/WebCore/platform/cocoa/PublicSuffixStoreCocoa.mm:
(WebCore::PublicSuffixStore::platformIsPublicSuffix const):
(WebCore::PublicSuffixStore::enablePublicSuffixCache):
(WebCore::PublicSuffixStore::addPublicSuffix):
* Source/WebKit/Shared/GoToBackForwardItemParameters.h:
* Source/WebKit/Shared/GoToBackForwardItemParameters.serialization.in:
* Source/WebKit/Shared/LoadParameters.h:
* Source/WebKit/Shared/LoadParameters.serialization.in:
* Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in:
* Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:
(TestWebKitAPI::TEST_F):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 1d5ea2: [UnifiedPDF] Clean up selection repaint code

2024-05-22 Thread Simon Fraser
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 1d5ea2d61eff45027240a20a5561f6a393c7ad13
  
https://github.com/WebKit/WebKit/commit/1d5ea2d61eff45027240a20a5561f6a393c7ad13
  Author: Simon Fraser 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.h
M Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm

  Log Message:
  ---
  [UnifiedPDF] Clean up selection repaint code
https://bugs.webkit.org/show_bug.cgi?id=274499
rdar://128505796

Reviewed by Abrar Rahman Protyasha.

Selection repaint currently works by computing a vector of rectangles for
the selection (and possibly the old selection too), and then repainting them.
This loses information about which page the rect belongs to, and for future
discrete mode changes we'll need this.

So repaint selections by enumerating their page coverage. If we have to
repaint the old selection, just pass it directly to 
`repaintOnSelectionChange()`.
Rename that method since it was called for selection repaint in general,
not just on active state changes.

* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.h:
* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm:
(WebKit::UnifiedPDFPlugin::windowActivityDidChange):
(WebKit::UnifiedPDFPlugin::repaintOnSelectionChange):
(WebKit::UnifiedPDFPlugin::setCurrentSelection):
(WebKit::UnifiedPDFPlugin::setNeedsRepaintInDocumentRects): Deleted.
(WebKit::UnifiedPDFPlugin::boundsForSelection const): Deleted.
(WebKit::UnifiedPDFPlugin::repaintOnSelectionActiveStateChangeIfNeeded): 
Deleted.

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 3a6e63: Fix linker error in JSC testing configuration

2024-05-22 Thread Keith Miller
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 3a6e636fb3bed4bda1a737658411a2af43c86e00
  
https://github.com/WebKit/WebKit/commit/3a6e636fb3bed4bda1a737658411a2af43c86e00
  Author: Keith Miller 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/JavaScriptCore/bytecode/UnlinkedMetadataTable.cpp
M Source/JavaScriptCore/bytecode/UnlinkedMetadataTableInlines.h

  Log Message:
  ---
  Fix linker error in JSC testing configuration
https://bugs.webkit.org/show_bug.cgi?id=274523
rdar://128542072

Reviewed by Mark Lam.

It seems like someone holding a Ref/RefPtr to UnlinkedMetadataTable didn't
include the inlines header. Rather than figure out how to find the missing
include I just moved the destructor to the .cpp file. I don't think the
destructor is hot enough that inlining is needed.

* Source/JavaScriptCore/bytecode/UnlinkedMetadataTable.cpp:
(JSC::UnlinkedMetadataTable::~UnlinkedMetadataTable):
* Source/JavaScriptCore/bytecode/UnlinkedMetadataTableInlines.h:
(JSC::UnlinkedMetadataTable::~UnlinkedMetadataTable): Deleted.

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 89de00: Introduce a TypedArrayView create that solely take...

2024-05-22 Thread Anne van Kesteren
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 89de003bd68db1f46ecfc9000f341ae919c326ac
  
https://github.com/WebKit/WebKit/commit/89de003bd68db1f46ecfc9000f341ae919c326ac
  Author: Anne van Kesteren 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/JavaScriptCore/runtime/GenericTypedArrayView.h
M Source/JavaScriptCore/runtime/GenericTypedArrayViewInlines.h
M Source/WebCore/Modules/compression/CompressionStreamEncoder.cpp
M Source/WebCore/Modules/compression/CompressionStreamEncoder.h
M Source/WebCore/Modules/compression/DecompressionStreamDecoder.cpp
M Source/WebCore/Modules/compression/DecompressionStreamDecoder.h
M Source/WebCore/Modules/fetch/FetchBodyConsumer.cpp
M Source/WebCore/html/HTMLMediaElement.cpp
M Source/WebCore/platform/graphics/ByteArrayPixelBuffer.cpp
M 
Source/WebCore/platform/graphics/avfoundation/cf/CDMSessionAVFoundationCF.cpp
M 
Source/WebCore/platform/graphics/avfoundation/objc/CDMSessionAVFoundationObjC.mm

  Log Message:
  ---
  Introduce a TypedArrayView create that solely takes an ArrayBuffer
https://bugs.webkit.org/show_bug.cgi?id=274309

Reviewed by Youenn Fablet, Keith Miller and Darin Adler.

This simplifies the setup for callers as they no longer have to obtain
the byteLength or provide the 0 offset.

Also use Ref instead of RefPtr where possible in the compression
module.

* Source/JavaScriptCore/runtime/GenericTypedArrayView.h:
* Source/JavaScriptCore/runtime/GenericTypedArrayViewInlines.h:
(JSC::GenericTypedArrayView::create):
* Source/WebCore/Modules/compression/CompressionStreamEncoder.cpp:
(WebCore::CompressionStreamEncoder::encode):
(WebCore::CompressionStreamEncoder::flush):
(WebCore::CompressionStreamEncoder::compress):
* Source/WebCore/Modules/compression/CompressionStreamEncoder.h:
* Source/WebCore/Modules/compression/DecompressionStreamDecoder.cpp:
(WebCore::DecompressionStreamDecoder::decode):
(WebCore::DecompressionStreamDecoder::flush):
(WebCore::DecompressionStreamDecoder::decompress):
(WebCore::DecompressionStreamDecoder::decompressZlib):
(WebCore::DecompressionStreamDecoder::decompressAppleCompressionFramework):
* Source/WebCore/Modules/compression/DecompressionStreamDecoder.h:
* Source/WebCore/Modules/fetch/FetchBodyConsumer.cpp:
(WebCore::FetchBodyConsumer::resolve):
* Source/WebCore/html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::mediaPlayerKeyNeeded):
* Source/WebCore/platform/graphics/ByteArrayPixelBuffer.cpp:
(WebCore::ByteArrayPixelBuffer::tryCreate):
* Source/WebCore/platform/graphics/avfoundation/cf/CDMSessionAVFoundationCF.cpp:
(WebCore::CDMSessionAVFoundationCF::generateKeyRequest):
* 
Source/WebCore/platform/graphics/avfoundation/objc/CDMSessionAVFoundationObjC.mm:
(WebCore::CDMSessionAVFoundationObjC::generateKeyRequest):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 24bc94: iOS: Fix gamepad detection when becoming firstResp...

2024-05-22 Thread Darryl Pogue
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 24bc94ef702df472e0b0e75d7448090b147c1c84
  
https://github.com/WebKit/WebKit/commit/24bc94ef702df472e0b0e75d7448090b147c1c84
  Author: Darryl Pogue 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm

  Log Message:
  ---
  iOS: Fix gamepad detection when becoming firstResponder
https://bugs.webkit.org/show_bug.cgi?id=269292

Reviewed by Brady Eidson.

Gamepads were only detected at initial load of the page if the WKWebView
were marked as the firstResponder, rather than detecting when the
WKWebView becomes or resigns the firstResponder.

This aligns the behaviour with WKWebView on macOS.

* Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:
(-[WKContentView becomeFirstResponderForWebView]):
(-[WKContentView resignFirstResponderForWebView]):
Enable/disable checking for gamepads when the WKWebView becomes or
resigns firstResponder status.

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 00d92c: Address Darin's review feedback on 279097@main

2024-05-22 Thread Chris Dumez
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 00d92c3139d072e389479b411faf047eb0b6ed00
  
https://github.com/WebKit/WebKit/commit/00d92c3139d072e389479b411faf047eb0b6ed00
  Author: Chris Dumez 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/platform/cocoa/SharedMemoryCocoa.cpp
M Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp
M 
Source/WebCore/platform/graphics/gstreamer/mse/SourceBufferPrivateGStreamer.cpp

  Log Message:
  ---
  Address Darin's review feedback on 279097@main
https://bugs.webkit.org/show_bug.cgi?id=274502

Unreviewed, address Darin's review feedback on 279097@main, which I somewhere 
failed
to upload before merging the PR.

* Source/WebCore/platform/cocoa/SharedMemoryCocoa.cpp:
(WebCore::SharedMemory::wrapMap):
* Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
(WebCore::MediaPlayerPrivateGStreamer::initializationDataEncountered):
* 
Source/WebCore/platform/graphics/gstreamer/mse/SourceBufferPrivateGStreamer.cpp:
(WebCore::SourceBufferPrivateGStreamer::appendInternal):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] f987e5: Adopt more smart pointers in HTMLCanvasElement

2024-05-22 Thread Rob Buis
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: f987e56ae559b455cb008420ff25583cf0081905
  
https://github.com/WebKit/WebKit/commit/f987e56ae559b455cb008420ff25583cf0081905
  Author: Rob Buis 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/html/HTMLCanvasElement.cpp

  Log Message:
  ---
  Adopt more smart pointers in HTMLCanvasElement
https://bugs.webkit.org/show_bug.cgi?id=274463

Reviewed by Ryosuke Niwa.

Adopt more smart pointers in HTMLCanvasElement based on the
[alpha.webkit.UncountedLocalVarsChecker] warning.

* Source/WebCore/html/HTMLCanvasElement.cpp:
(WebCore::HTMLCanvasElement::getContext):
(WebCore::HTMLCanvasElement::createContextWebGL):
(WebCore::HTMLCanvasElement::paint):
(WebCore::HTMLCanvasElement::didMoveToNewDocument):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 0b2576: Synchronize FileAPI WPT

2024-05-22 Thread Anne van Kesteren
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 0b25769a33bbdf56e9f725697a4e577ac8033478
  
https://github.com/WebKit/WebKit/commit/0b25769a33bbdf56e9f725697a4e577ac8033478
  Author: Anne van Kesteren 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M LayoutTests/imported/w3c/resources/resource-files.json
A 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/FileReader/Progress_event_bubbles_cancelable-expected.txt
A 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/FileReader/Progress_event_bubbles_cancelable.html
M LayoutTests/imported/w3c/web-platform-tests/FileAPI/META.yml
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/blob/Blob-constructor-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/blob/Blob-methods-from-detached-frame-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/blob/Blob-methods-from-detached-frame.html
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/blob/Blob-slice-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/blob/Blob-slice-overflow-expected.txt
A 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/blob/Blob-stream-byob-crash.html
A 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/blob/Blob-stream-sync-xhr-crash.html
M 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/blob/Blob-stream.any-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/blob/Blob-stream.any.js
M 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/blob/Blob-stream.any.worker-expected.txt
M LayoutTests/imported/w3c/web-platform-tests/FileAPI/blob/w3c-import.log
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/file/File-constructor-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/file/send-file-formdata-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/file/send-file-formdata-utf-8-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/fileReader-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/reading-data-section/Determining-Encoding-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/reading-data-section/FileReader-event-handler-attributes-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/reading-data-section/FileReader-multiple-reads-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/reading-data-section/filereader_abort-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/reading-data-section/filereader_error-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/reading-data-section/filereader_readAsArrayBuffer-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/reading-data-section/filereader_readAsBinaryString-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/reading-data-section/filereader_readAsDataURL-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/reading-data-section/filereader_readAsText-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/reading-data-section/filereader_readystate-expected.txt
R 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/reading-data-section/filereader_result-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/url/sandboxed-iframe-expected.txt
M LayoutTests/imported/w3c/web-platform-tests/FileAPI/url/url-format.any.js
M 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/url/url-with-fetch.any.js
A 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/url/url_xmlhttprequest_img-ref.html
M LayoutTests/imported/w3c/web-platform-tests/FileAPI/url/w3c-import.log
M LayoutTests/tests-options.json

  Log Message:
  ---
  Synchronize FileAPI WPT
https://bugs.webkit.org/show_bug.cgi?id=274457

Reviewed by Tim Nguyen.

Quite a few caveats unfortunately:

- Ignore a test change with Float16Array for now as it would reduce coverage.
- BlobURL/cross-partition.tentative.https.html is not imported for now.
- blob/Blob-methods-from-detached-frame.html is removed as we exported it as
  Blob-methods-from-detached-frame.html and imported that one at a later point.
  So in effect this deduplicates it.
- Many -expected.txt files are removed because --clean-dest-dir was not used
  during prior imports.
- A change to FileAPI/url/multi-global-origin-serialization.sub.html is ignored
  for now.

Latest upstream commit for this directory:
https://github.com/web-platform-tests/wpt/commit/5aa50dd4151b5bc1d04d5505366c6e27df30af5b

* LayoutTests/imported/w3c/resources/resource-files.json:
* 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/FileReader/Progress_event_bubbles_cancelable-expected.txt:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/FileAPI/FileReader/Progress_event_bubbles_cancelable.html:
 Added.
* 

[webkit-changes] [WebKit/WebKit] 19f5b8: [TextureMapper] BitmapTexture::create should not c...

2024-05-22 Thread Carlos Garcia Campos
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 19f5b8907ca43c6f30a5417d02a532cbafe5a037
  
https://github.com/WebKit/WebKit/commit/19f5b8907ca43c6f30a5417d02a532cbafe5a037
  Author: Carlos Garcia Campos 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/platform/graphics/texmap/BitmapTexture.cpp

  Log Message:
  ---
  [TextureMapper] BitmapTexture::create should not change the bound texture
https://bugs.webkit.org/show_bug.cgi?id=274510

Reviewed by Miguel Gomez.

When creating a new texture we should keep the previously bound texture.

* Source/WebCore/platform/graphics/texmap/BitmapTexture.cpp:
(WebCore::BitmapTexture::BitmapTexture):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 685354: [Skia] Use a different GL context on every thread

2024-05-22 Thread Carlos Garcia Campos
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 685354bc969a297feb8030122b0ab696294debd7
  
https://github.com/WebKit/WebKit/commit/685354bc969a297feb8030122b0ab696294debd7
  Author: Carlos Garcia Campos 
  Date:   2024-05-22 (Wed, 22 May 2024)

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

  Log Message:
  ---
  [Skia] Use a different GL context on every thread
https://bugs.webkit.org/show_bug.cgi?id=274454

Reviewed by Miguel Gomez.

This will allow to have accelerated ImageBitmap and offscreen canvas.

* Source/WebCore/platform/graphics/PlatformDisplay.cpp:
(WebCore::PlatformDisplay::clearSharingGLContext):
* Source/WebCore/platform/graphics/PlatformDisplay.h:
(WebCore::PlatformDisplay::skiaGrContext): Deleted.
* Source/WebCore/platform/graphics/skia/PlatformDisplaySkia.cpp:
(WebCore::skiaGLInterface):
(WebCore::SkiaGLContext::create):
(WebCore::SkiaGLContext::invalidate):
(WebCore::SkiaGLContext::skiaGLContext const):
(WebCore::SkiaGLContext::skiaGrContext const):
(WebCore::SkiaGLContext::SkiaGLContext):
(WebCore::PlatformDisplay::skiaGLContext):
(WebCore::PlatformDisplay::skiaGrContext):
(WebCore::PlatformDisplay::invalidateSkiaGLContexts):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] be4cdc: Implement further enforcement of Trusted Types for...

2024-05-22 Thread Luke Warlow
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: be4cdc9c602803f8505921b2c66f86aa0f392497
  
https://github.com/WebKit/WebKit/commit/be4cdc9c602803f8505921b2c66f86aa0f392497
  Author: Luke Warlow 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/block-string-assignment-to-attribute-via-attribute-node-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/modify-attributes-in-callback-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/modify-attributes-in-callback.html
M Source/WebCore/dom/Attr.cpp
M Source/WebCore/dom/Attr.h
M Source/WebCore/dom/CharacterData.cpp
M Source/WebCore/dom/CharacterData.h
M Source/WebCore/dom/Element.cpp
M Source/WebCore/dom/Element.h
M Source/WebCore/dom/Node.cpp
M Source/WebCore/dom/Node.h

  Log Message:
  ---
  Implement further enforcement of Trusted Types for Attributes
https://bugs.webkit.org/show_bug.cgi?id=274267

Reviewed by Ryosuke Niwa.

This patch adds trusted types enforcement to Attr textContent, value and 
nodeValue.

It also improves error handling for mutating attributes within default policy.

* 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/block-string-assignment-to-attribute-via-attribute-node-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/modify-attributes-in-callback-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/modify-attributes-in-callback.html:
* Source/WebCore/dom/Attr.cpp:
(WebCore::Attr::setValue):
(WebCore::Attr::setNodeValue):
* Source/WebCore/dom/Attr.h:
* Source/WebCore/dom/CharacterData.cpp:
(WebCore::CharacterData::setNodeValue):
* Source/WebCore/dom/CharacterData.h:
* Source/WebCore/dom/Element.cpp:
(WebCore::Element::validateAttributeIndex const):
(WebCore::Element::setAttribute):
* Source/WebCore/dom/Element.h:
* Source/WebCore/dom/Node.cpp:
(WebCore::Node::setNodeValue):
(WebCore::Node::setTextContent):
* Source/WebCore/dom/Node.h:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] f33204: Implement trusted types enforcement for script ele...

2024-05-22 Thread Luke Warlow
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: f33204d52243ae13948a2dee417d69e25442c2d8
  
https://github.com/WebKit/WebKit/commit/f33204d52243ae13948a2dee417d69e25442c2d8
  Author: Luke Warlow 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M LayoutTests/TestExpectations
M 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/block-Node-multiple-arguments-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/block-text-node-insertion-into-script-element-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/trusted-types-report-only-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/trusted-types-svg-script-expected.txt
M Source/WebCore/dom/ScriptElement.cpp
M Source/WebCore/dom/ScriptElement.h
M Source/WebCore/html/HTMLScriptElement.cpp
M Source/WebCore/html/HTMLScriptElement.h
M Source/WebCore/svg/SVGScriptElement.cpp
M Source/WebCore/svg/SVGScriptElement.h

  Log Message:
  ---
  Implement trusted types enforcement for script elements
https://bugs.webkit.org/show_bug.cgi?id=269365

Reviewed by Darin Adler and Ryosuke Niwa.

Add script text string to script element. This is used to ensure that script 
elements can't be manipulated in untrusted ways.

See 
https://w3c.github.io/trusted-types/dist/spec/#htmlscriptelement-script-text 
and https://w3c.github.io/trusted-types/dist/spec/#slot-value-verification

* LayoutTests/TestExpectations:
* 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/block-Node-multiple-arguments-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/block-text-node-insertion-into-script-element-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/trusted-types-report-only-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/trusted-types/trusted-types-svg-script-expected.txt:
* Source/WebCore/dom/ScriptElement.cpp:
(WebCore::ScriptElement::finishParsingChildren):
(WebCore::ScriptElement::prepareScript):
(WebCore::ScriptElement::setTrustedScriptText):
* Source/WebCore/dom/ScriptElement.h:
* Source/WebCore/html/HTMLScriptElement.cpp:
(WebCore::HTMLScriptElement::finishParsingChildren):
(WebCore::HTMLScriptElement::setTextContent):
(WebCore::HTMLScriptElement::setInnerText):
* Source/WebCore/html/HTMLScriptElement.h:
* Source/WebCore/svg/SVGScriptElement.cpp:
(WebCore::SVGScriptElement::finishParsingChildren):
* Source/WebCore/svg/SVGScriptElement.h:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 3414c9: [GTK] `css3/flexbox/image-percent-max-height.html`...

2024-05-22 Thread Vitaly Dyachkov
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 3414c9ad54a72dffbe8ef62a626c62c7174597d6
  
https://github.com/WebKit/WebKit/commit/3414c9ad54a72dffbe8ef62a626c62c7174597d6
  Author: Vitaly Dyachkov 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M LayoutTests/css3/flexbox/image-percent-max-height-expected.html
M LayoutTests/css3/flexbox/image-percent-max-height.html
M LayoutTests/platform/wpe/TestExpectations

  Log Message:
  ---
  [GTK] `css3/flexbox/image-percent-max-height.html` is a constant failure
https://bugs.webkit.org/show_bug.cgi?id=271762

Reviewed by Said Abou-Hallawa.

Use `image-rendering: crisp-edges;` to prevent scaling artifacts.

* LayoutTests/css3/flexbox/image-percent-max-height-expected.html:
* LayoutTests/css3/flexbox/image-percent-max-height.html:
* LayoutTests/platform/mac/TestExpectations:
* LayoutTests/platform/wpe/TestExpectations:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] c2e64b: Implement `field-sizing: content` minimum width fo...

2024-05-22 Thread Luke Warlow
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c2e64ba96fbed90353cfdea4b0c20a855a6c40bb
  
https://github.com/WebKit/WebKit/commit/c2e64ba96fbed90353cfdea4b0c20a855a6c40bb
  Author: Luke Warlow 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
A LayoutTests/fast/forms/text/text-field-sizing-caret-expected.txt
A LayoutTests/fast/forms/text/text-field-sizing-caret.html
A LayoutTests/fast/forms/textarea/textarea-field-sizing-caret-expected.txt
A LayoutTests/fast/forms/textarea/textarea-field-sizing-caret.html
M Source/WebCore/html/HTMLTextFormControlElement.cpp

  Log Message:
  ---
  Implement `field-sizing: content` minimum width for text caret
https://bugs.webkit.org/show_bug.cgi?id=269128

Reviewed by Darin Adler.

Updates text control computed style to include a minimum sizing so the caret 
shows.

* LayoutTests/fast/forms/text/text-field-sizing-caret-expected.txt: Added.
* LayoutTests/fast/forms/text/text-field-sizing-caret.html: Added.
* LayoutTests/fast/forms/textarea/textarea-field-sizing-caret-expected.txt: 
Added.
* LayoutTests/fast/forms/textarea/textarea-field-sizing-caret.html: Added.
* Source/WebCore/html/HTMLTextFormControlElement.cpp:
(WebCore::HTMLTextFormControlElement::adjustInnerTextStyle const):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 45503f: Unreviewed, reverting 279093@main.

2024-05-22 Thread Commit Queue
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 45503f8710637c591ea738906c2a8303dc9663ed
  
https://github.com/WebKit/WebKit/commit/45503f8710637c591ea738906c2a8303dc9663ed
  Author: Commit Queue 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WebCore/animation/KeyframeEffect.cpp

  Log Message:
  ---
  Unreviewed, reverting 279093@main.
https://bugs.webkit.org/show_bug.cgi?id=274511

Caused compositing consistency assertion in debug tests

Reverted changeset:

"[view-transitions] Async animation of snapshot position via transform looks 
jumpy with sync animation of size"
https://bugs.webkit.org/show_bug.cgi?id=274437
https://commits.webkit.org/279093@main

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 05456a: [macOS Sonoma+] Enable microphone capture in GPUPr...

2024-05-22 Thread youennf
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 05456ae93aa145c7f52bb3876fa296469d9424d7
  
https://github.com/WebKit/WebKit/commit/05456ae93aa145c7f52bb3876fa296469d9424d7
  Author: Youenn Fablet 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M Source/WTF/wtf/PlatformHave.h
M Source/WebKit/Shared/WebPreferencesDefaultValues.cpp

  Log Message:
  ---
  [macOS Sonoma+] Enable microphone capture in GPUProcess for all applications
https://bugs.webkit.org/show_bug.cgi?id=274464
rdar://128463834

Reviewed by Jean-Yves Avenard.

Given rdar://problem/29448368 is fixed in Sonoma, we can enable GPU process 
microphone capture.
This allows to reuse VPIO for audio rendering like Safari does.

* Source/WTF/wtf/PlatformHave.h:
* Source/WebKit/Shared/WebPreferencesDefaultValues.cpp:
(WebKit::defaultCaptureAudioInGPUProcessEnabled):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 4ec5cd: [visionOS] In Firefox, YouTube videos immediately ...

2024-05-22 Thread Jean-Yves Avenard
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 4ec5cdeedc10b597d41c3c5ecd264c62e2a7e007
  
https://github.com/WebKit/WebKit/commit/4ec5cdeedc10b597d41c3c5ecd264c62e2a7e007
  Author: Jean-Yves Avenard 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M 
Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm
M 
Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.h
M 
Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm
M Source/WebCore/platform/ios/VideoPresentationInterfaceIOS.mm
M Source/WebKit/UIProcess/WebPageProxy.cpp

  Log Message:
  ---
  [visionOS] In Firefox, YouTube videos immediately exit full screen
https://bugs.webkit.org/show_bug.cgi?id=274458
rdar://12214

Reviewed by Jer Noble.

The primary cause for fullscreen to immediately exit was a typo in the use of 
the PLATFORM
macro which caused requestHideAndExitFullscreen() to be immediately called once 
we had entered
fullscreen.
Then we had two issues that were revealed once the primary issue was fixed. 
When running in
iOS compatibility mode, once we entered fullscreen we would either remove the 
player from the
video layer (CRABS) or teardown the video layer (MSE).
To avoid this, we check if a FigVideoTargetRef is set, which indicate that the 
application
isn't running in iOS compatibility mode and a LMK isn't in use.

Manually tested MSE and CRABS, for both app in both native (Safari) and 
compatibility mode (Firefox)

* 
Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.h:
* 
Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
(WebCore::MediaPlayerPrivateMediaSourceAVFObjC::acceleratedVideoMode const):
(WebCore::MediaPlayerPrivateAVFoundationObjC::isInFullscreenOrPictureInPictureChanged):
* 
Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm:
(WebCore::MediaPlayerPrivateMediaSourceAVFObjC::isInFullscreenOrPictureInPictureChanged):
* Source/WebCore/platform/ios/VideoPresentationInterfaceIOS.mm:
(WebCore::VideoPresentationInterfaceIOS::doSetup):
* Source/WebKit/UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::viewDidLeaveWindow): Fix typo and replace with 
PLATFORM(VISION). This caused to immediately
exit fullscreen as soon as we entered fullscreen.

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] a270d6: Most of WPT webrtc/RTCSctpTransport-maxMessageSize...

2024-05-22 Thread youennf
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: a270d67c53417c216ccfa1ed3bbc30a20e7d4512
  
https://github.com/WebKit/WebKit/commit/a270d67c53417c216ccfa1ed3bbc30a20e7d4512
  Author: Youenn Fablet 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M 
LayoutTests/imported/w3c/web-platform-tests/webrtc/RTCSctpTransport-maxMessageSize-expected.txt
M Source/WebCore/Modules/mediastream/PeerConnectionBackend.cpp
M Source/WebCore/Modules/mediastream/PeerConnectionBackend.h
M Source/WebCore/Modules/mediastream/RTCPeerConnection.cpp
M Source/WebCore/Modules/mediastream/RTCPeerConnection.h
M Source/WebCore/Modules/mediastream/RTCSctpTransport.cpp
M Source/WebCore/Modules/mediastream/RTCSctpTransport.h
M Source/WebCore/Modules/mediastream/gstreamer/GStreamerMediaEndpoint.cpp
M Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp

  Log Message:
  ---
  Most of WPT webrtc/RTCSctpTransport-maxMessageSize.html tests are failing
https://bugs.webkit.org/show_bug.cgi?id=274442
rdar://128444396

Reviewed by Philippe Normand.

We implement https://w3c.github.io/webrtc-pc/#sctp-transport-update-mms.
This is called when successfully applying a SDP description as per 
specification.
In this implementation, the assumption is that canSendSize is 0.
The specific value of 65536 is handled by libwebrtc.

* 
LayoutTests/imported/w3c/web-platform-tests/webrtc/RTCSctpTransport-maxMessageSize-expected.txt:
* Source/WebCore/Modules/mediastream/PeerConnectionBackend.cpp:
(WebCore::PeerConnectionBackend::setLocalDescriptionSucceeded):
(WebCore::PeerConnectionBackend::setRemoteDescriptionSucceeded):
* Source/WebCore/Modules/mediastream/PeerConnectionBackend.h:
* Source/WebCore/Modules/mediastream/RTCPeerConnection.cpp:
(WebCore::RTCPeerConnection::updateSctpBackend):
* Source/WebCore/Modules/mediastream/RTCPeerConnection.h:
* Source/WebCore/Modules/mediastream/RTCSctpTransport.cpp:
(WebCore::RTCSctpTransport::onStateChanged):
(WebCore::RTCSctpTransport::updateMaxMessageSize):
* Source/WebCore/Modules/mediastream/RTCSctpTransport.h:
* Source/WebCore/Modules/mediastream/gstreamer/GStreamerMediaEndpoint.cpp:
(WebCore::GStreamerMediaEndpoint::doSetLocalDescription):
(WebCore::GStreamerMediaEndpoint::doSetRemoteDescription):
* Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp:
(WebCore::SctpTransportState::maxMessageSize const):
(WebCore::LibWebRTCMediaEndpoint::setLocalSessionDescriptionSucceeded):
(WebCore::LibWebRTCMediaEndpoint::setRemoteSessionDescriptionSucceeded):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 6ad608: Unreviewed, reverting 278971@main (7da094e1da19)

2024-05-22 Thread sideshowbarker
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 6ad608ca9d2936751a08a54639756243071542aa
  
https://github.com/WebKit/WebKit/commit/6ad608ca9d2936751a08a54639756243071542aa
  Author: Michael[tm] Smith 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M LayoutTests/editing/execCommand/break-out-of-empty-list-item.html
M LayoutTests/editing/inserting/typing-space-to-trigger-smart-link.html
M LayoutTests/editing/pasteboard/paste-text-events-expected.txt
M LayoutTests/editing/pasteboard/paste-text-events.html
M LayoutTests/editing/style/highlight-insert-paragraph.html
M LayoutTests/fast/events/input-events-fired-when-typing-expected.txt
M LayoutTests/fast/events/input-events-fired-when-typing.html
M LayoutTests/fast/events/ios/submit-form-target-blank-using-return-key.html
M LayoutTests/fast/events/onchange-passwordfield.html
M LayoutTests/fast/events/onchange-searchfield.html
M LayoutTests/fast/events/onchange-textfield.html
M LayoutTests/fast/forms/onchange-change-type.html
M 
LayoutTests/http/tests/navigation/keyboard-events-during-provisional-navigation-expected.txt
M 
LayoutTests/http/tests/navigation/keyboard-events-during-provisional-subframe-navigation-expected.txt
M 
LayoutTests/platform/mac-wk2/http/tests/navigation/keyboard-events-during-provisional-navigation-expected.txt
M LayoutTests/platform/wpe/TestExpectations
M Source/WebCore/dom/Node.cpp
M Source/WebCore/dom/TextEvent.cpp
M Source/WebCore/dom/TextEvent.h
M Source/WebCore/editing/AlternativeTextController.cpp
M Source/WebCore/editing/Editor.cpp
M Source/WebCore/editing/Editor.h
M Source/WebCore/editing/EditorCommand.cpp
M Source/WebCore/page/EventHandler.cpp
M Source/WebCore/page/EventHandler.h

  Log Message:
  ---
  Unreviewed, reverting 278971@main (7da094e1da19)
https://bugs.webkit.org/show_bug.cgi?id=274500
rdar://128463632

REGRESSION (278971@main): [iOS] Form submission does not occur when pressing 
the return key

Reverted change:

The beforeinput event should fire before textInput
https://bugs.webkit.org/show_bug.cgi?id=268988
278971@main (7da094e1da19)

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [WebKit/WebKit] 072b03: [Navigation] Correct cancelable determination of N...

2024-05-22 Thread Rob Buis
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 072b031414e564e51079e9a3448b5843d0f4492d
  
https://github.com/WebKit/WebKit/commit/072b031414e564e51079e9a3448b5843d0f4492d
  Author: Rob Buis 
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/click-samedocument-crossorigin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/click-samedocument-sameorigin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/location-samedocument-crossorigin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/location-samedocument-sameorigin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/open-samedocument-crossorigin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/open-samedocument-crossorigin-sameorigindomain.sub-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/open-samedocument-sameorigin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/submit-samedocument-crossorigin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/submit-samedocument-crossorigin-sameorigindomain.sub-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/submit-samedocument-sameorigin-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-iframe-location-expected.txt
M 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-window-open-expected.txt
M 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/click-samedocument-crossorigin-sameorigindomain.sub-expected.txt
M 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/location-samedocument-crossorigin-sameorigindomain.sub-expected.txt
M Source/WebCore/page/Navigation.cpp

  Log Message:
  ---
  [Navigation] Correct cancelable determination of NavigateEvent
https://bugs.webkit.org/show_bug.cgi?id=274471

Reviewed by Alex Christensen and Darin Adler.

When determining the cancelable flag for NavigateEvent we did not take into 
account the navigation type [1].

By fixing that we fix navigate-anchor-download.html, 
navigate-anchor-download-userInitiated.html and can mark two
further tests as passing.

[1] 
https://html.spec.whatwg.org/multipage/nav-history-apis.html#inner-navigate-event-firing-algorithm
 (Step 11)

* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/click-samedocument-crossorigin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/click-samedocument-sameorigin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/location-samedocument-crossorigin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/location-samedocument-sameorigin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/open-samedocument-crossorigin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/open-samedocument-crossorigin-sameorigindomain.sub-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/open-samedocument-sameorigin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/submit-samedocument-crossorigin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/submit-samedocument-crossorigin-sameorigindomain.sub-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/submit-samedocument-sameorigin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-iframe-location-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/navigation-api/navigate-event/navigate-window-open-expected.txt:
* 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/click-samedocument-crossorigin-sameorigindomain.sub-expected.txt:
* 
LayoutTests/platform/glib/imported/w3c/web-platform-tests/navigation-api/navigate-event/cross-window/location-samedocument-crossorigin-sameorigindomain.sub-expected.txt:
* Source/WebCore/page/Navigation.cpp:
(WebCore::Navigation::innerDispatchNavigateEvent):

Canonical link: 

[webkit-changes] [WebKit/WebKit] adcd97: Further remove MSVC code

2024-05-22 Thread Yusuke Suzuki
  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: adcd97e5a4ae359e0ea83a4cc9ae2348b95d607f
  
https://github.com/WebKit/WebKit/commit/adcd97e5a4ae359e0ea83a4cc9ae2348b95d607f
  Author: Yusuke Suzuki 
  Date:   2024-05-21 (Tue, 21 May 2024)

  Changed paths:
M Source/JavaScriptCore/API/tests/testapi.c
M Source/JavaScriptCore/API/tests/testapi.cpp
M Source/JavaScriptCore/assembler/MacroAssemblerX86Common.cpp
M Source/JavaScriptCore/assembler/X86Assembler.h
M Source/JavaScriptCore/builtins/BuiltinNames.cpp
M Source/JavaScriptCore/config.h
M Source/JavaScriptCore/jit/PCToCodeOriginMap.cpp
M Source/JavaScriptCore/jit/RegisterAtOffset.h
M Source/JavaScriptCore/jsc.cpp
M Source/JavaScriptCore/offlineasm/generate_offset_extractor.rb
M Source/JavaScriptCore/parser/SourceProviderCacheItem.h
M Source/JavaScriptCore/runtime/JSCPtrTag.h
M Source/JavaScriptCore/runtime/VM.h
M Source/JavaScriptCore/testRegExp.cpp
M Source/WTF/wtf/Assertions.cpp
M Source/WTF/wtf/Assertions.h
M Source/WTF/wtf/MathExtras.h
M Source/WTF/wtf/PtrTag.h
M Source/WTF/wtf/SegmentedVector.h
M Source/WTF/wtf/posix/ThreadingPOSIX.cpp

  Log Message:
  ---
  Further remove MSVC code
https://bugs.webkit.org/show_bug.cgi?id=274450
rdar://128448731

Reviewed by Ross Kirsling.

Further remove MSVC hacks. And we use `OS(WINDOWS)` for code which needs to be 
used on Windows (even without MSVC).

* Source/JavaScriptCore/API/tests/testapi.c:
* Source/JavaScriptCore/API/tests/testapi.cpp:
* Source/JavaScriptCore/assembler/MacroAssemblerX86Common.cpp:
(JSC::MacroAssembler::probe):
(JSC::MacroAssemblerX86Common::getCPUIDEx):
* Source/JavaScriptCore/assembler/X86Assembler.h:
* Source/JavaScriptCore/builtins/BuiltinNames.cpp:
* Source/JavaScriptCore/config.h:
* Source/JavaScriptCore/jit/PCToCodeOriginMap.cpp:
* Source/JavaScriptCore/jit/RegisterAtOffset.cpp:
* Source/JavaScriptCore/jsc.cpp:
(JSC_DEFINE_HOST_FUNCTION):
(jscmain):
* Source/JavaScriptCore/offlineasm/generate_offset_extractor.rb:
* Source/JavaScriptCore/parser/SourceProviderCacheItem.h:
* Source/JavaScriptCore/runtime/JSCPtrTag.h:
* Source/JavaScriptCore/runtime/VM.h:
* Source/JavaScriptCore/testRegExp.cpp:
* Source/WTF/wtf/Assertions.cpp:
(WTF::createWithFormatAndArguments):
* Source/WTF/wtf/Assertions.h:
* Source/WTF/wtf/MathExtras.h:
* Source/WTF/wtf/PtrTag.h:
* Source/WTF/wtf/SegmentedVector.h:
* Source/WTF/wtf/posix/ThreadingPOSIX.cpp:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes