Index: trunk/src/chrome/browser/password_manager/password_manager_delegate_impl.cc |
=================================================================== |
--- trunk/src/chrome/browser/password_manager/password_manager_delegate_impl.cc (revision 238401) |
+++ trunk/src/chrome/browser/password_manager/password_manager_delegate_impl.cc (working copy) |
@@ -9,7 +9,6 @@ |
#include "base/strings/utf_string_conversions.h" |
#include "base/timer/elapsed_timer.h" |
#include "chrome/browser/infobars/confirm_infobar_delegate.h" |
-#include "chrome/browser/infobars/infobar.h" |
#include "chrome/browser/infobars/infobar_service.h" |
#include "chrome/browser/password_manager/password_form_manager.h" |
#include "chrome/browser/password_manager/password_manager.h" |
@@ -43,11 +42,11 @@ |
class SavePasswordInfoBarDelegate : public ConfirmInfoBarDelegate { |
public: |
// If we won't be showing the one-click signin infobar, creates a save |
- // password infobar and delegate and adds the infobar to the InfoBarService |
- // for |web_contents|. |uma_histogram_suffix| is empty, or one of the |
- // "group_X" suffixes used in the histogram names for infobar usage reporting; |
- // if empty, the usage is not reported, otherwise the suffix is used to choose |
- // the right histogram. |
+ // password infobar delegate and adds it to the InfoBarService for |
+ // |web_contents|. |uma_histogram_suffix| is empty, or one of the "group_X" |
+ // suffixes used in the histogram names for infobar usage reporting; if empty, |
+ // the usage is not reported, otherwise the suffix is used to choose the right |
+ // histogram. |
static void Create(content::WebContents* web_contents, |
PasswordFormManager* form_to_save, |
const std::string& uma_histogram_suffix); |
@@ -61,7 +60,8 @@ |
NUM_RESPONSE_TYPES, |
}; |
- SavePasswordInfoBarDelegate(PasswordFormManager* form_to_save, |
+ SavePasswordInfoBarDelegate(InfoBarService* infobar_service, |
+ PasswordFormManager* form_to_save, |
const std::string& uma_histogram_suffix); |
virtual ~SavePasswordInfoBarDelegate(); |
@@ -113,16 +113,18 @@ |
return; |
#endif |
- InfoBarService::FromWebContents(web_contents)->AddInfoBar( |
- ConfirmInfoBarDelegate::CreateInfoBar(scoped_ptr<ConfirmInfoBarDelegate>( |
- new SavePasswordInfoBarDelegate(form_to_save, |
- uma_histogram_suffix)))); |
+ InfoBarService* infobar_service = |
+ InfoBarService::FromWebContents(web_contents); |
+ infobar_service->AddInfoBar( |
+ scoped_ptr<InfoBarDelegate>(new SavePasswordInfoBarDelegate( |
+ infobar_service, form_to_save, uma_histogram_suffix))); |
} |
SavePasswordInfoBarDelegate::SavePasswordInfoBarDelegate( |
+ InfoBarService* infobar_service, |
PasswordFormManager* form_to_save, |
const std::string& uma_histogram_suffix) |
- : ConfirmInfoBarDelegate(), |
+ : ConfirmInfoBarDelegate(infobar_service), |
form_to_save_(form_to_save), |
infobar_response_(NO_RESPONSE), |
uma_histogram_suffix_(uma_histogram_suffix) { |