301624 2025-10-29 04:38:34 -0700 Page icon load decisions should be queried all at once 2026-03-31 15:33:50 -0700 1 1 1 Unclassified WebKit WebKit2 WebKit Nightly Build Unspecified Unspecified RESOLVED DUPLICATE 301086 InRadar P2 Normal --- 301088 301086 1 aperez aperez kkinnunen webkit-bug-importer oldest_to_newest 2154955 0 aperez 2025-10-29 04:38:34 -0700 While working on bug #301088, I noticed that in the WebProcess we know all the icons for a page, but then querying whether they are to be loaded is done one at a time: void WebLocalFrameLoaderClient::getLoadDecisionForIcons(const Vector<std::pair<WebCore::LinkIcon&, uint64_t>>& icons) { RefPtr webPage = m_frame->page(); if (!webPage) return; for (auto& icon : icons) webPage->send(Messages::WebPageProxy::GetLoadDecisionForIcon(icon.first, CallbackID::fromInteger(icon.second))); } It would be desirable to pass all icons at once for the following reasons: - Currently the UIProcess cannot not know how many icons will be loaded, and therefore cannot know when the job is done for a given page. Passing all the candidate icons we can know for which set of loads to complete. We would like to do this for the GTK and WPE ports (see bug #301086) - The IconLoadingClient in the UIProcess can make a better decision about which icons to load if it knows about all of them. - Sending a single message should be fine, they won't typically be a huge amount of them, and not very long. 2156553 1 webkit-bug-importer 2025-11-05 03:39:10 -0800 <rdar://problem/164079873> 2195699 2 aperez 2026-03-31 15:33:50 -0700 Implemented this as part of bug #301086 and landed in 310308@main *** This bug has been marked as a duplicate of bug 301086 ***