Re-land r248110 with ASAN error fixed.

chrome/browser/ui/autofill/password_generation_popup_controller_impl.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 "chrome/browser/ui/autofill/password_generation_popup_controller_impl.h"
 
 #include <math.h>
 
 #include "base/strings/utf_string_conversion_utils.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 87 matching lines @@
   switch (event.windowsKeyCode) {
     case ui::VKEY_UP:
     case ui::VKEY_DOWN:
       PasswordSelected(true);
       return true;
     case ui::VKEY_ESCAPE:
       Hide();
       return true;
     case ui::VKEY_RETURN:
     case ui::VKEY_TAB:
-      // We supress tab if the password is selected because we will
+      // We suppress tab if the password is selected because we will
       // automatically advance focus anyway.
       return PossiblyAcceptPassword();
     default:
       return false;
   }
 }
 
 bool PasswordGenerationPopupControllerImpl::PossiblyAcceptPassword() {
-  if (password_selected_)
-    PasswordAccepted();
+  if (password_selected_) {
+    PasswordAccepted();  // This will delete |this|

[Evan Stade, 2014/01/31 22:39:57]

nit: final punctuation.

[Garrett Casto, 2014/01/31 22:58:00]

Done.

+    return true;
+  }
 
-  return password_selected_;
+  return false;
 }
 
 void PasswordGenerationPopupControllerImpl::PasswordSelected(bool selected) {
   password_selected_ = selected;
   view_->UpdateBoundsAndRedrawPopup();
 }
 
 void PasswordGenerationPopupControllerImpl::PasswordAccepted() {
   web_contents()->GetRenderViewHost()->Send(
       new AutofillMsg_GeneratedPasswordAccepted(
@@ skipping 69 matching lines @@
     view_ = PasswordGenerationPopupView::Create(this);
     view_->Show();
   }
 
   controller_common_.RegisterKeyPressCallback();
 
   if (observer_)
     observer_->OnPopupShown();
 }
 
+void PasswordGenerationPopupControllerImpl::HideAndDestroy() {
+  Hide();
+}
+
 void PasswordGenerationPopupControllerImpl::Hide() {
   controller_common_.RemoveKeyPressCallback();
 
   if (view_)
     view_->Hide();
 
   if (observer_)
     observer_->OnPopupHidden();
 
   delete this;
@@ skipping 72 matching lines @@
 
 base::string16 PasswordGenerationPopupControllerImpl::HelpText() {
   return l10n_util::GetStringUTF16(IDS_PASSWORD_GENERATION_PROMPT);
 }
 
 base::string16 PasswordGenerationPopupControllerImpl::LearnMoreLink() {
   return l10n_util::GetStringUTF16(IDS_PASSWORD_GENERATION_LEARN_MORE_LINK);
 }
 
 }  // namespace autofill