| Summary: | [ App Privacy Report ] Restoring a session after clearing the cache results in app initiated loads in Safari | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Kate Cheney <katherine_cheney> | ||||||
| Component: | New Bugs | Assignee: | Kate Cheney <katherine_cheney> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | bfulgham, webkit-bug-importer | ||||||
| Priority: | P2 | Keywords: | InRadar | ||||||
| Version: | WebKit Nightly Build | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| Attachments: |
|
||||||||
|
Description
Kate Cheney
2021-10-25 19:36:05 PDT
Created attachment 442454 [details]
Patch
Comment on attachment 442454 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=442454&action=review R=me > Source/WebKit/WebProcess/WebPage/WebPage.cpp:1898 > + if (documentLoader) If document!oader isn’t used elsewhere, I’d just declare it in the if statement: If (auto* documentLoadwr = corePage()…. Created attachment 442500 [details]
Patch for landing
(In reply to Brent Fulgham from comment #2) > Comment on attachment 442454 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=442454&action=review > > R=me > > > Source/WebKit/WebProcess/WebPage/WebPage.cpp:1898 > > + if (documentLoader) > > If document!oader isn’t used elsewhere, I’d just declare it in the if > statement: > > If (auto* documentLoadwr = corePage()…. Thanks! Fixed in PFL Committed r284875 (243552@main): <https://commits.webkit.org/243552@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 442500 [details]. |