Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1422)

Unified Diff: chrome/browser/content_settings/tab_specific_content_settings.cc

Issue 87853004: Refactoring Manage Passwords Bubble Code to exclude TabSpecificContentSettings (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@movingclasses
Patch Set: Android fix Created 7 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/content_settings/tab_specific_content_settings.cc
diff --git a/chrome/browser/content_settings/tab_specific_content_settings.cc b/chrome/browser/content_settings/tab_specific_content_settings.cc
index 69f90b0843527cbc6004ae1911190b0d7d91f909..0875ded5c1524f6f9618df0d1cfdbd7c9d02015d 100644
--- a/chrome/browser/content_settings/tab_specific_content_settings.cc
+++ b/chrome/browser/content_settings/tab_specific_content_settings.cc
@@ -34,7 +34,6 @@
#include "net/cookies/canonical_cookie.h"
#include "webkit/common/fileapi/file_system_types.h"
-using autofill::PasswordFormMap;
using content::BrowserThread;
using content::NavigationController;
using content::NavigationEntry;
@@ -43,21 +42,6 @@ using content::WebContents;
DEFINE_WEB_CONTENTS_USER_DATA_KEY(TabSpecificContentSettings);
-TabSpecificContentSettings::PasswordObserver::PasswordObserver(
- TabSpecificContentSettings* tab_specific_content_settings)
- : tab_specific_content_settings_(tab_specific_content_settings) {
- tab_specific_content_settings_->SetPasswordObserver(this);
-}
-
-TabSpecificContentSettings::PasswordObserver::~PasswordObserver() {
- if (tab_specific_content_settings_)
- tab_specific_content_settings_->SetPasswordObserver(NULL);
-}
-
-void TabSpecificContentSettings::PasswordObserver::ContentSettingsDestroyed() {
- tab_specific_content_settings_ = NULL;
-}
-
TabSpecificContentSettings::SiteDataObserver::SiteDataObserver(
TabSpecificContentSettings* tab_specific_content_settings)
: tab_specific_content_settings_(tab_specific_content_settings) {
@@ -75,7 +59,6 @@ void TabSpecificContentSettings::SiteDataObserver::ContentSettingsDestroyed() {
TabSpecificContentSettings::TabSpecificContentSettings(WebContents* tab)
: content::WebContentsObserver(tab),
- password_observer_(NULL),
profile_(Profile::FromBrowserContext(tab->GetBrowserContext())),
allowed_local_shared_objects_(profile_),
blocked_local_shared_objects_(profile_),
@@ -84,11 +67,7 @@ TabSpecificContentSettings::TabSpecificContentSettings(WebContents* tab)
pending_protocol_handler_(ProtocolHandler::EmptyProtocolHandler()),
previous_protocol_handler_(ProtocolHandler::EmptyProtocolHandler()),
pending_protocol_handler_setting_(CONTENT_SETTING_DEFAULT),
- load_plugins_link_enabled_(true),
- manage_passwords_icon_to_be_shown_(false),
- password_to_be_saved_(false),
- manage_passwords_bubble_needs_showing_(false),
- password_submitted_(false) {
+ load_plugins_link_enabled_(true) {
ClearBlockedContentSettingsExceptForCookies();
ClearCookieSpecificContentSettings();
@@ -100,8 +79,6 @@ TabSpecificContentSettings::TabSpecificContentSettings(WebContents* tab)
TabSpecificContentSettings::~TabSpecificContentSettings() {
FOR_EACH_OBSERVER(
SiteDataObserver, observer_list_, ContentSettingsDestroyed());
- if (password_observer_)
- password_observer_->ContentSettingsDestroyed();
}
TabSpecificContentSettings* TabSpecificContentSettings::Get(
@@ -494,27 +471,6 @@ void TabSpecificContentSettings::OnProtectedMediaIdentifierPermissionSet(
}
#endif
-void TabSpecificContentSettings::OnPasswordSubmitted(
- PasswordFormManager* form_manager) {
- form_manager_.reset(form_manager);
- password_form_map_ = form_manager_->best_matches();
- manage_passwords_icon_to_be_shown_ = true;
- password_to_be_saved_ = true;
- manage_passwords_bubble_needs_showing_ = true;
- password_submitted_ = true;
- NotifyPasswordObserver();
-}
-
-void TabSpecificContentSettings::OnPasswordAutofilled(
- const PasswordFormMap& password_form_map) {
- password_form_map_ = password_form_map;
- manage_passwords_icon_to_be_shown_ = true;
- password_to_be_saved_ = false;
- manage_passwords_bubble_needs_showing_ = false;
- password_submitted_ = false;
- NotifyPasswordObserver();
-}
-
TabSpecificContentSettings::MicrophoneCameraState
TabSpecificContentSettings::GetMicrophoneCameraState() const {
if (IsContentAllowed(CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC) &&
@@ -698,12 +654,6 @@ void TabSpecificContentSettings::DidNavigateMainFrame(
ClearBlockedContentSettingsExceptForCookies();
GeolocationDidNavigate(details);
MIDIDidNavigate(details);
- // Reset password states for next page.
- manage_passwords_icon_to_be_shown_ = false;
- password_to_be_saved_ = false;
- manage_passwords_bubble_needs_showing_ = false;
- NotifySiteDataObservers();
- NotifyPasswordObserver();
}
}
@@ -739,11 +689,6 @@ void TabSpecificContentSettings::AppCacheAccessed(const GURL& manifest_url,
}
}
-void TabSpecificContentSettings::SavePassword() {
- DCHECK(form_manager_.get());
- form_manager_->Save();
-}
-
void TabSpecificContentSettings::Observe(
int type,
const content::NotificationSource& source,
@@ -769,16 +714,6 @@ void TabSpecificContentSettings::Observe(
}
}
-void TabSpecificContentSettings::SetPasswordObserver(
- PasswordObserver* observer) {
- password_observer_ = observer;
-}
-
-void TabSpecificContentSettings::NotifyPasswordObserver() {
- if (password_observer_)
- password_observer_->OnPasswordAction();
-}
-
void TabSpecificContentSettings::AddSiteDataObserver(
SiteDataObserver* observer) {
observer_list_.AddObserver(observer);
« no previous file with comments | « chrome/browser/content_settings/tab_specific_content_settings.h ('k') | chrome/browser/password_manager/password_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698