Clean-up RemoveSuggestion and related codes

components/password_manager/core/browser/password_autofill_manager.cc

 // Copyright 2013 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/core/browser/password_autofill_manager.h"
 
 #include <vector>
 
 #include "base/i18n/case_conversion.h"
 #include "base/logging.h"
@@ skipping 240 matching lines @@
 bool PasswordAutofillManager::GetDeletionConfirmationText(
     const base::string16& value,
     int identifier,
     base::string16* title,
     base::string16* body) {
   return false;
 }
 
 bool PasswordAutofillManager::RemoveSuggestion(const base::string16& value,
                                                int identifier) {
-  // http://crbug.com/329038
-  NOTIMPLEMENTED();
   return false;

[vabr (Chromium), 2015/11/06 10:57:40]

Please add a comment referencing http://crbug.com/329038#c15 and stating that
removing password suggestions is not allowed. For example:

// Password suggestions cannot be deleted this way. See
http://crbug.com/329038#c15.

 }
 
 void PasswordAutofillManager::ClearPreviewedForm() {
   password_manager_driver_->ClearPreviewedForm();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // PasswordAutofillManager, private:
 
 bool PasswordAutofillManager::GetPasswordAndRealmForUsername(
@@ skipping 44 matching lines @@
     autofill::PasswordFormFillData* found_password) {
   LoginToPasswordInfoMap::iterator iter = login_to_password_info_.find(key);
   if (iter == login_to_password_info_.end())
     return false;
 
   *found_password = iter->second;
   return true;
 }
 
 }  //  namespace password_manager