[webkit-changes] [WebKit/WebKit] 38294b: Add canvas noise injection test

Matthew Finkel noreply at github.com
Thu Oct 19 11:48:34 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 38294bd0b5d959b1c43be0caee0460d5cd3721a5
      https://github.com/WebKit/WebKit/commit/38294bd0b5d959b1c43be0caee0460d5cd3721a5
  Author: Matthew Finkel <sysrqb at apple.com>
  Date:   2023-10-19 (Thu, 19 Oct 2023)

  Changed paths:
    M LayoutTests/TestExpectations
    A LayoutTests/fast/canvas/canvas-noise-injection-expected.txt
    A LayoutTests/fast/canvas/canvas-noise-injection.html
    M Source/WebCore/html/CanvasBase.h
    M Source/WebCore/html/CanvasNoiseInjection.h
    M Source/WebCore/testing/Internals.cpp
    M Source/WebCore/testing/Internals.h
    M Source/WebCore/testing/Internals.idl

  Log Message:
  -----------
  Add canvas noise injection test
https://bugs.webkit.org/show_bug.cgi?id=263094
rdar://116886810

Reviewed by Simon Fraser.

Noise injection should only be applied when the feature is enabled (i.e., when
a salt value is set), and it should not be applied when the canvas only
contains bitmap data (e.g., drawImage, putImageData).

This test fail the latter expectation, and that will be fixed in a follow-up
patch.

This patch exposes some internal state via Internals.

* LayoutTests/TestExpectations:
* LayoutTests/fast/canvas/canvas-noise-injection-expected.txt: Added.
* LayoutTests/fast/canvas/canvas-noise-injection.html: Added.
* Source/WebCore/html/CanvasBase.h:
(WebCore::CanvasBase::setNoiseInjectionSalt):
(WebCore::CanvasBase::havePendingCanvasNoiseInjection const):
* Source/WebCore/html/CanvasNoiseInjection.h:
(WebCore::CanvasNoiseInjection::haveDirtyRects const):
* Source/WebCore/testing/Internals.cpp:
(WebCore::Internals::setCanvasNoiseInjectionSalt):
(WebCore::Internals::doesCanvasHavePendingCanvasNoiseInjection const):
* Source/WebCore/testing/Internals.h:
* Source/WebCore/testing/Internals.idl:

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




More information about the webkit-changes mailing list