Skip to content

Conversation

@sammygill
Copy link
Contributor

@sammygill sammygill commented Apr 9, 2024

67629e3

REGRESSION (276433@main): Snapshots are partially dark-shaded.
https://bugs.webkit.org/show_bug.cgi?id=272410
rdar://125791769

Reviewed by Simon Fraser and Abrar Rahman Protyasha.

These dark regions are due to the fact that we are missing the white
background when painting the previews. This patch adds them back in the
async case since the preview is painted without the async renderer.

* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm:
(WebKit::UnifiedPDFPlugin::paintPDFContent):

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

0177888

Misc iOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 wincairo
✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug 🧪 wpe-wk2
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ⏳ 🧪 api-mac ✅ 🧪 api-wpe
✅ 🧪 ios-wk2-wpt ⏳ 🧪 mac-wk1 ✅ 🛠 wpe-skia
✅ 🧪 api-ios ⏳ 🧪 mac-wk2 ✅ 🛠 gtk
✅ 🛠 tv ✅ 🧪 mac-AS-debug-wk2 ✅ 🧪 gtk-wk2
✅ 🛠 tv-sim ✅ 🧪 api-gtk
✅ 🛠 🧪 merge 🛠 watch
✅ 🛠 watch-sim

@sammygill sammygill requested a review from cdumez as a code owner April 9, 2024 18:54
@sammygill sammygill self-assigned this Apr 9, 2024
@sammygill sammygill added the PDF For bugs in WebKit's built-in PDF support. label Apr 9, 2024
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't need this. context.fillRect() is not changing state.

@sammygill sammygill force-pushed the eng/REGRESSION-276433main-snapshots-are-partially-dark-shaded branch from 28adbab to 0177888 Compare April 9, 2024 19:27
@sammygill sammygill added the merge-queue Applied to send a pull request to merge-queue label Apr 9, 2024
https://bugs.webkit.org/show_bug.cgi?id=272410
rdar://125791769

Reviewed by Simon Fraser and Abrar Rahman Protyasha.

These dark regions are due to the fact that we are missing the white
background when painting the previews. This patch adds them back in the
async case since the preview is painted without the async renderer.

* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm:
(WebKit::UnifiedPDFPlugin::paintPDFContent):

Canonical link: https://commits.webkit.org/277273@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/REGRESSION-276433main-snapshots-are-partially-dark-shaded branch from 0177888 to 67629e3 Compare April 9, 2024 22:47
@webkit-commit-queue
Copy link
Collaborator

Committed 277273@main (67629e3): https://commits.webkit.org/277273@main

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

@webkit-commit-queue webkit-commit-queue merged commit 67629e3 into WebKit:main Apr 9, 2024
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PDF For bugs in WebKit's built-in PDF support.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants