Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Filters] Add an internal feature key for enabling GraphicsContext filters #6906

Conversation

shallawa
Copy link
Contributor

@shallawa shallawa commented Nov 29, 2022

ba14273

[Filters] Add an internal feature key for enabling GraphicsContext filters
https://bugs.webkit.org/show_bug.cgi?id=248375
rdar://102689264

Reviewed by Sam Weinig.

This key will be used for enabling GraphicsContext filters giving that all the
FilterEffects in the filter chain support CoreGraphics filters.

* Source/WTF/Scripts/Preferences/WebPreferencesInternal.yaml:
* Source/WebCore/Sources.txt:
* Source/WebCore/WebCore.xcodeproj/project.pbxproj:
* Source/WebCore/page/Page.cpp:
(WebCore::Page::preferredFilterRenderingModes const):
* Source/WebCore/platform/graphics/GraphicsContext.h:
(WebCore::GraphicsContext::style const):
(WebCore::GraphicsContext::setStyle):
* Source/WebCore/platform/graphics/GraphicsContextState.cpp:
(WebCore::GraphicsContextState::mergeLastChanges):
* Source/WebCore/platform/graphics/filters/FilterImageTargetSwitcher.cpp:
(WebCore::FilterImageTargetSwitcher::beginClipAndDrawSourceImage):
(WebCore::FilterImageTargetSwitcher::endClipAndDrawSourceImage):
(WebCore::FilterImageTargetSwitcher::endDrawSourceImage):
(WebCore::FilterImageTargetSwitcher::didDrawSourceImage): Deleted.
* Source/WebCore/platform/graphics/filters/FilterImageTargetSwitcher.h:
* Source/WebCore/platform/graphics/filters/FilterStyleTargetSwitcher.cpp: Copied from Source/WebCore/platform/graphics/filters/FilterTargetSwitcher.cpp.
(WebCore::FilterStyleTargetSwitcher::FilterStyleTargetSwitcher):
(WebCore::FilterStyleTargetSwitcher::beginDrawSourceImage):
(WebCore::FilterStyleTargetSwitcher::endDrawSourceImage):
* Source/WebCore/platform/graphics/filters/FilterStyleTargetSwitcher.h: Copied from Source/WebCore/platform/graphics/filters/FilterImageTargetSwitcher.h.
* Source/WebCore/platform/graphics/filters/FilterTargetSwitcher.cpp:
(WebCore::FilterTargetSwitcher::create):
(WebCore::FilterTargetSwitcher::willDrawSourceImage): Deleted.
(WebCore::FilterTargetSwitcher::didDrawSourceImage): Deleted.
* Source/WebCore/platform/graphics/filters/FilterTargetSwitcher.h:
(WebCore::FilterTargetSwitcher::needsRedrawSourceImage const):
(WebCore::FilterTargetSwitcher::beginClipAndDrawSourceImage):
(WebCore::FilterTargetSwitcher::endClipAndDrawSourceImage):
(WebCore::FilterTargetSwitcher::beginDrawSourceImage):
(WebCore::FilterTargetSwitcher::endDrawSourceImage):
(WebCore::FilterTargetSwitcher::needsRedrawSourceImage): Deleted.
* Source/WebCore/rendering/RenderLayerFilters.cpp:
(WebCore::RenderLayerFilters::beginFilterEffect):
(WebCore::RenderLayerFilters::applyFilterEffect):

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

e4407b3

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios βœ… πŸ›  mac βœ… πŸ›  wpe   πŸ›  πŸ§ͺ win
βœ… πŸ§ͺ bindings βœ… πŸ›  ios-sim βœ… πŸ›  mac-debug   πŸ›  gtk   πŸ›  wincairo
βœ… πŸ§ͺ webkitperl   πŸ§ͺ ios-wk2 βœ… πŸ›  mac-AS-debug ❌ πŸ§ͺ gtk-wk2
  πŸ§ͺ api-ios   πŸ§ͺ api-mac   πŸ§ͺ api-gtk
βœ… πŸ›  πŸ§ͺ jsc βœ… πŸ›  tv   πŸ§ͺ mac-wk1 βœ… πŸ›  jsc-armv7
  πŸ›  tv-sim   πŸ§ͺ mac-wk2 βœ… πŸ§ͺ jsc-armv7-tests
βœ… πŸ›  πŸ§ͺ merge βœ… πŸ›  watch   πŸ§ͺ mac-AS-debug-wk2 βœ… πŸ›  jsc-mips
βœ… πŸ›  watch-sim βœ… πŸ§ͺ mac-wk2-stress βœ… πŸ§ͺ jsc-mips-tests

@shallawa shallawa self-assigned this Nov 29, 2022
@shallawa shallawa added the Layout and Rendering For bugs with layout and rendering of Web pages. label Nov 29, 2022
@shallawa shallawa added the merge-queue Applied to send a pull request to merge-queue label Nov 29, 2022
…lters

https://bugs.webkit.org/show_bug.cgi?id=248375
rdar://102689264

Reviewed by Sam Weinig.

This key will be used for enabling GraphicsContext filters giving that all the
FilterEffects in the filter chain support CoreGraphics filters.

* Source/WTF/Scripts/Preferences/WebPreferencesInternal.yaml:
* Source/WebCore/Sources.txt:
* Source/WebCore/WebCore.xcodeproj/project.pbxproj:
* Source/WebCore/page/Page.cpp:
(WebCore::Page::preferredFilterRenderingModes const):
* Source/WebCore/platform/graphics/GraphicsContext.h:
(WebCore::GraphicsContext::style const):
(WebCore::GraphicsContext::setStyle):
* Source/WebCore/platform/graphics/GraphicsContextState.cpp:
(WebCore::GraphicsContextState::mergeLastChanges):
* Source/WebCore/platform/graphics/filters/FilterImageTargetSwitcher.cpp:
(WebCore::FilterImageTargetSwitcher::beginClipAndDrawSourceImage):
(WebCore::FilterImageTargetSwitcher::endClipAndDrawSourceImage):
(WebCore::FilterImageTargetSwitcher::endDrawSourceImage):
(WebCore::FilterImageTargetSwitcher::didDrawSourceImage): Deleted.
* Source/WebCore/platform/graphics/filters/FilterImageTargetSwitcher.h:
* Source/WebCore/platform/graphics/filters/FilterStyleTargetSwitcher.cpp: Copied from Source/WebCore/platform/graphics/filters/FilterTargetSwitcher.cpp.
(WebCore::FilterStyleTargetSwitcher::FilterStyleTargetSwitcher):
(WebCore::FilterStyleTargetSwitcher::beginDrawSourceImage):
(WebCore::FilterStyleTargetSwitcher::endDrawSourceImage):
* Source/WebCore/platform/graphics/filters/FilterStyleTargetSwitcher.h: Copied from Source/WebCore/platform/graphics/filters/FilterImageTargetSwitcher.h.
* Source/WebCore/platform/graphics/filters/FilterTargetSwitcher.cpp:
(WebCore::FilterTargetSwitcher::create):
(WebCore::FilterTargetSwitcher::willDrawSourceImage): Deleted.
(WebCore::FilterTargetSwitcher::didDrawSourceImage): Deleted.
* Source/WebCore/platform/graphics/filters/FilterTargetSwitcher.h:
(WebCore::FilterTargetSwitcher::needsRedrawSourceImage const):
(WebCore::FilterTargetSwitcher::beginClipAndDrawSourceImage):
(WebCore::FilterTargetSwitcher::endClipAndDrawSourceImage):
(WebCore::FilterTargetSwitcher::beginDrawSourceImage):
(WebCore::FilterTargetSwitcher::endDrawSourceImage):
(WebCore::FilterTargetSwitcher::needsRedrawSourceImage): Deleted.
* Source/WebCore/rendering/RenderLayerFilters.cpp:
(WebCore::RenderLayerFilters::beginFilterEffect):
(WebCore::RenderLayerFilters::applyFilterEffect):

Canonical link: https://commits.webkit.org/257104@main
@webkit-early-warning-system webkit-early-warning-system force-pushed the eng/Filters-Add-an-internal-feature-key-for-enabling-GraphicsContext-filters branch from e4407b3 to ba14273 Compare November 29, 2022 06:20
@webkit-commit-queue
Copy link
Collaborator

Committed 257104@main (ba14273): https://commits.webkit.org/257104@main

Reviewed commits have been landed. Closing PR #6906 and removing active labels.

@webkit-early-warning-system webkit-early-warning-system merged commit ba14273 into WebKit:main Nov 29, 2022
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Nov 29, 2022
@shallawa shallawa deleted the eng/Filters-Add-an-internal-feature-key-for-enabling-GraphicsContext-filters branch May 9, 2023 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Layout and Rendering For bugs with layout and rendering of Web pages.
Projects
None yet
3 participants