1 files changed, 21 insertions, 4 deletions
diff --git a/src/core/web_contents_delegate_qt.cpp b/src/core/web_contents_delegate_qt.cpp
index 96c7c4c41..f51a6dff3 100644
--- a/src/core/web_contents_delegate_qt.cpp
+++ b/src/core/web_contents_delegate_qt.cpp
@@ -79,6 +79,16 @@
 
 namespace QtWebEngineCore {
 
+static gfx::Rect rootViewToScreenRect(content::WebContents *web_contents, const gfx::Rect &anchor_in_root_view)
+{
+    RenderWidgetHostViewQt *rwhv = static_cast<RenderWidgetHostViewQt *>(web_contents->GetRenderWidgetHostView());
+    if (!rwhv)
+        return gfx::Rect();
+    content::ScreenInfo screenInfo;
+    rwhv->GetScreenInfo(&screenInfo);
+    return gfx::ScaleToEnclosingRect(anchor_in_root_view, 1 / screenInfo.device_scale_factor);
+}
+
 // Maps the LogSeverity defines in base/logging.h to the web engines message levels.
 static WebContentsAdapterClient::JavaScriptConsoleMessageLevel mapToJavascriptConsoleMessageLevel(int32_t messageLevel) {
     if (messageLevel < 1)
@@ -335,6 +345,9 @@ void WebContentsDelegateQt::DidUpdateFaviconURL(const std::vector<content::Favic
         faviconCandidates.append(toFaviconInfo(candidate));
     }
 
+    // Favicon URL can be changed from JavaScript too. Thus we need to reset
+    // the current candidate icon list to not handle previous icon as a candidate.
+    m_faviconManager->resetCandidates();
     m_faviconManager->update(faviconCandidates);
 }
 
@@ -515,8 +528,10 @@ void WebContentsDelegateQt::launchExternalURL(const QUrl &url, ui::PageTransitio
 
 void WebContentsDelegateQt::ShowValidationMessage(content::WebContents *web_contents, const gfx::Rect &anchor_in_root_view, const base::string16 &main_text, const base::string16 &sub_text)
 {
-    Q_UNUSED(web_contents);
-    m_viewClient->showValidationMessage(toQt(anchor_in_root_view), toQt(main_text), toQt(sub_text));
+    gfx::Rect anchor = rootViewToScreenRect(web_contents, anchor_in_root_view);
+    if (anchor.IsEmpty())
+        return;
+    m_viewClient->showValidationMessage(toQt(anchor), toQt(main_text), toQt(sub_text));
 }
 
 void WebContentsDelegateQt::HideValidationMessage(content::WebContents *web_contents)
@@ -527,8 +542,10 @@ void WebContentsDelegateQt::HideValidationMessage(content::WebContents *web_cont
 
 void WebContentsDelegateQt::MoveValidationMessage(content::WebContents *web_contents, const gfx::Rect &anchor_in_root_view)
 {
-    Q_UNUSED(web_contents);
-    m_viewClient->moveValidationMessage(toQt(anchor_in_root_view));
+    gfx::Rect anchor = rootViewToScreenRect(web_contents, anchor_in_root_view);
+    if (anchor.IsEmpty())
+        return;
+    m_viewClient->moveValidationMessage(toQt(anchor));
 }
 
 void WebContentsDelegateQt::BeforeUnloadFired(content::WebContents *tab, bool proceed, bool *proceed_to_fire_unload)