[Credential Management] Smart lock save Credentials bubble should not always pop up.

components/password_manager/content/browser/credential_manager_dispatcher.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/password_manager/content/browser/credential_manager_dispatcher.h"
 
 #include "base/bind.h"
 #include "base/memory/scoped_vector.h"
 #include "base/strings/string16.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 202 matching lines @@
 
 void CredentialManagerDispatcher::OnNotifySignedIn(
     int request_id,
     const password_manager::CredentialInfo& credential) {
   DCHECK(credential.type != CredentialType::CREDENTIAL_TYPE_EMPTY);
   DCHECK(request_id);
   web_contents()->GetRenderViewHost()->Send(
       new CredentialManagerMsg_AcknowledgeSignedIn(
           web_contents()->GetRenderViewHost()->GetRoutingID(), request_id));
 
-  if (!IsSavingEnabledForCurrentPage())
+  if (!client_->IsSavingEnabledForCurrentPage())
     return;
 
   scoped_ptr<autofill::PasswordForm> form(CreatePasswordFormFromCredentialInfo(
       credential, web_contents()->GetLastCommittedURL().GetOrigin()));
   form->skip_zero_click = !IsZeroClickAllowed();
 
   // TODO(mkwst): This is a stub; we should be checking the PasswordStore to
   // determine whether or not the credential exists, and calling UpdateLogin
   // accordingly.
   form_manager_.reset(new CredentialManagerPasswordFormManager(
       client_, GetDriver(), *form, this));
 }
 
 void CredentialManagerDispatcher::OnProvisionalSaveComplete() {
   DCHECK(form_manager_);
-  client_->PromptUserToSavePassword(
-      form_manager_.Pass(), CredentialSourceType::CREDENTIAL_SOURCE_API);
+  if (client_->IsSavingEnabledForCurrentPage() &&
+      !form_manager_->IsBlacklisted()) {
+    client_->PromptUserToSavePassword(
+        form_manager_.Pass(), CredentialSourceType::CREDENTIAL_SOURCE_API);
+  }
 }
 
 void CredentialManagerDispatcher::OnNotifySignedOut(int request_id) {
   DCHECK(request_id);
 
   PasswordStore* store = GetPasswordStore();
   if (store) {
     if (!pending_sign_out_) {
       pending_sign_out_.reset(new PendingSignedOutTask(
           this, web_contents()->GetLastCommittedURL().GetOrigin()));
@@ skipping 43 matching lines @@
 
   // This will result in a callback to
   // PendingRequestTask::OnGetPasswordStoreResults().
   store->GetAutofillableLogins(pending_request_.get());
 }
 
 PasswordStore* CredentialManagerDispatcher::GetPasswordStore() {
   return client_ ? client_->GetPasswordStore() : nullptr;
 }
 
-bool CredentialManagerDispatcher::IsSavingEnabledForCurrentPage() const {
-  // TODO(vasilii): add more, see http://crbug.com/450583.
-  return !client_->IsOffTheRecord();
-}
-
 bool CredentialManagerDispatcher::IsZeroClickAllowed() const {
   return *auto_signin_enabled_ && !client_->IsOffTheRecord();
 }
 
 base::WeakPtr<PasswordManagerDriver> CredentialManagerDispatcher::GetDriver() {
   ContentPasswordManagerDriverFactory* driver_factory =
       ContentPasswordManagerDriverFactory::FromWebContents(web_contents());
   DCHECK(driver_factory);
   PasswordManagerDriver* driver =
       driver_factory->GetDriverForFrame(web_contents()->GetMainFrame());
@@ skipping 22 matching lines @@
           pending_request_->id(), info));
   pending_request_.reset();
 }
 
 void CredentialManagerDispatcher::DoneSigningOut() {
   DCHECK(pending_sign_out_);
   pending_sign_out_.reset();
 }
 
 }  // namespace password_manager