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

Unified Diff: chrome/browser/protector/base_setting_change.cc

Issue 8612002: Protector bubble cancels itself if user changes default search engine manually. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Review fixes. Created 9 years, 1 month 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/protector/base_setting_change.cc
diff --git a/chrome/browser/protector/base_setting_change.cc b/chrome/browser/protector/base_setting_change.cc
index 255185af3ac425848aea102b043606a8d6d2809a..80fe257afc46b565a33833cac391446c198d1efd 100644
--- a/chrome/browser/protector/base_setting_change.cc
+++ b/chrome/browser/protector/base_setting_change.cc
@@ -4,22 +4,30 @@
#include "chrome/browser/protector/base_setting_change.h"
+#include "base/logging.h"
+
namespace protector {
-BaseSettingChange::BaseSettingChange() {
+BaseSettingChange::BaseSettingChange()
+ : protector_(NULL) {
}
BaseSettingChange::~BaseSettingChange() {
}
bool BaseSettingChange::Init(Protector* protector) {
+ DCHECK(protector);
+ protector_ = protector;
return true;
}
-void BaseSettingChange::Apply(Protector* protector) {
+void BaseSettingChange::Apply() {
+}
+
+void BaseSettingChange::Discard() {
}
-void BaseSettingChange::Discard(Protector* protector) {
+void BaseSettingChange::OnBeforeRemoved() {
}
} // namespace protector
« no previous file with comments | « chrome/browser/protector/base_setting_change.h ('k') | chrome/browser/protector/default_search_provider_change.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698