Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 7c3fea2e73edc7d7c5373dda8f22bf2c44b1293e
https://github.com/WebKit/WebKit/commit/7c3fea2e73edc7d7c5373dda8f22bf2c44b1293e
Author: Tim Horton <[email protected]>
Date: 2025-10-13 (Mon, 13 Oct 2025)
Changed paths:
A LayoutTests/pdf/load-pdf-twice-in-sequence-expected.txt
A LayoutTests/pdf/load-pdf-twice-in-sequence.html
Log Message:
-----------
Add a test for 301346@main, ensuring that loading cached PDFs succeeds
https://bugs.webkit.org/show_bug.cgi?id=300662
rdar://162560992
Reviewed by Sam Weinig and Abrar Rahman Protyasha.
Upon further investigation, it is trivial to make a 100% repro test
case for the bug fixed in 301346@main, where PDFPlugin would fail to
load PDFs that were already cached, so that we don't have to depend
on a rare race condition to cover that fix.
It turns out that simply waiting for PDFPlugin to itself force-cache
the loaded PDF via PDFPluginBase::addArchiveResource(), and then trying
to load the *same* PDF in another plugin, will take the archive path
(and thus the `deliverResponseAndData` path, and thus exercise the fix
from 301346@main), 100% of the time.
* LayoutTests/pdf/load-pdf-twice-in-sequence-expected.txt: Added.
* LayoutTests/pdf/load-pdf-twice-in-sequence.html: Added.
Canonical link: https://commits.webkit.org/301453@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
