dev-qt/qtwebengine: backport :6 favicon display fix

Not worth a revbump rebuild, but is still annoying and
wanted to toss it here at least for future builds.

Signed-off-by: Ionen Wolkens <ionen@gentoo.org>
This commit is contained in:
Ionen Wolkens
2022-08-17 14:32:03 -04:00
parent 598dc36ae3
commit 86ee2059ef
2 changed files with 29 additions and 0 deletions

View File

@@ -0,0 +1,28 @@
Backport dev branch fix for favicons sometime not showing in
some consumers like qutebrowser. Not currently included in
upcoming 6.3.2 so may be needed beyond 6.3.1.
https://bugreports.qt.io/browse/QTBUG-103735
https://github.com/qt/qtwebengine/commit/c5de1b2123
--- a/src/core/favicon_driver_qt.cpp
+++ b/src/core/favicon_driver_qt.cpp
@@ -300,15 +300,15 @@ void FaviconDriverQt::DidStartNavigation(content::NavigationHandle *navigation_h
return;
m_faviconUrls.reset();
- m_completedHandlersCount = 0;
- m_latestFavicon = FaviconStatusQt();
if (!navigation_handle->IsSameDocument()) {
+ m_completedHandlersCount = 0;
+ m_latestFavicon = FaviconStatusQt();
m_documentOnLoadCompleted = false;
m_manifestUrl = GURL();
- }
- m_viewClient->iconChanged(QUrl());
+ m_viewClient->iconChanged(QUrl());
+ }
content::ReloadType reload_type = navigation_handle->GetReloadType();
if (reload_type == content::ReloadType::NONE || IsOffTheRecord())

View File

@@ -88,6 +88,7 @@ DEPEND="${RDEPEND}
PATCHES=(
"${FILESDIR}"/${P}-widevine.patch
"${FILESDIR}"/${P}-favicon.patch
)
python_check_deps() {