Index: chrome/browser/password_manager/chrome_password_manager_client.cc |
diff --git a/chrome/browser/password_manager/chrome_password_manager_client.cc b/chrome/browser/password_manager/chrome_password_manager_client.cc |
index 0ad9df43ffb94fe6c980db669d15e1f2aa783695..874f5c59f4a2f54690d3b194f3da2ca4e520088a 100644 |
--- a/chrome/browser/password_manager/chrome_password_manager_client.cc |
+++ b/chrome/browser/password_manager/chrome_password_manager_client.cc |
@@ -131,9 +131,10 @@ bool ChromePasswordManagerClient::IsPasswordManagerEnabledForCurrentPage() |
return entry->GetURL().host() != chrome::kChromeUIChromeSigninHost; |
} |
-bool ChromePasswordManagerClient::ShouldAskUserToSubmitURL() { |
- return password_manager::urls_collection_experiment::ShouldShowBubble( |
- GetPrefs()); |
+bool ChromePasswordManagerClient::ShouldAskUserToSubmitURL(const GURL& url) { |
+ return url.is_valid() && !url.is_empty() && url.has_host() && |
+ password_manager::urls_collection_experiment::ShouldShowBubble( |
+ GetPrefs()); |
} |
bool ChromePasswordManagerClient::ShouldFilterAutofillResult( |
@@ -167,9 +168,10 @@ bool ChromePasswordManagerClient::IsSyncAccountCredential( |
} |
void ChromePasswordManagerClient::AskUserAndMaybeReportURL( |
- const std::string& url) const { |
- // TODO(melandory) Show bubble which asks user if he wants to report the URL |
- // and report URL if needed. |
+ const GURL& url) const { |
+ ManagePasswordsUIController* manage_passwords_ui_controller = |
+ ManagePasswordsUIController::FromWebContents(web_contents()); |
+ manage_passwords_ui_controller->OnAskToReportURL(url); |
} |
void ChromePasswordManagerClient::AutofillResultsComputed() { |