Bug 255688
| Summary: | Remove the abandoned 'LoadSiteIconsIgnoringImageLoadingPreference' setting | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Brent Fulgham <bfulgham> |
| Component: | WebKit2 | Assignee: | Adrian Perez <aperez> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | aperez, kkinnunen, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Brent Fulgham
The LoadSiteIconsIgnoringImageLoadingPreference is not used any longer, and we should remove it.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Brent Fulgham
<rdar://107159912>
Brent Fulgham
Pull request: https://github.com/WebKit/WebKit/pull/12944
EWS
Committed 263202@main (4967181732a9): <https://commits.webkit.org/263202@main>
Reviewed commits have been landed. Closing PR #12944 and removing active labels.
Adrian Perez
Reopening to land a quick follow-up to 263202@main to write an
appropriate version in the “Deprecated:” tags.
Brent, thanks for making the change in the API for the WPE/GTK ports :)
Brent Fulgham
(In reply to Adrian Perez from comment #4)
> Reopening to land a quick follow-up to 263202@main to write an
> appropriate version in the “Deprecated:” tags.
>
> Brent, thanks for making the change in the API for the WPE/GTK ports :)
Sorry I couldn't give a good 'Deprecated' message; thank you for completing the task :-)
Adrian Perez
Pull request: https://github.com/WebKit/WebKit/pull/13113
Adrian Perez
(In reply to Brent Fulgham from comment #5)
> (In reply to Adrian Perez from comment #4)
> > Reopening to land a quick follow-up to 263202@main to write an
> > appropriate version in the “Deprecated:” tags.
> >
> > Brent, thanks for making the change in the API for the WPE/GTK ports :)
>
> Sorry I couldn't give a good 'Deprecated' message; thank you for completing
> the task :-)
No problem at all! =)
EWS
Committed 263342@main (c1e563b2e2ec): <https://commits.webkit.org/263342@main>
Reviewed commits have been landed. Closing PR #13113 and removing active labels.