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

Unified Diff: blimp/client/core/settings/settings_observer.h

Issue 2349073002: Blimp Settings framework on the c++ side (Closed)
Patch Set: Remove OnRecordWholeDocumentChanged from SettingsObserver and SettingsFeature Created 4 years, 2 months 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: blimp/client/core/settings/settings_observer.h
diff --git a/blimp/client/core/settings/settings_observer.h b/blimp/client/core/settings/settings_observer.h
new file mode 100644
index 0000000000000000000000000000000000000000..11f97a444f2272a38e590c8d14c44ab9ab40f684
--- /dev/null
+++ b/blimp/client/core/settings/settings_observer.h
@@ -0,0 +1,38 @@
+// Copyright 2016 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.
+
+#ifndef BLIMP_CLIENT_CORE_SETTINGS_SETTINGS_OBSERVER_H_
+#define BLIMP_CLIENT_CORE_SETTINGS_SETTINGS_OBSERVER_H_
+
+#include "base/observer_list.h"
+
+namespace blimp {
+namespace client {
+
+class SettingsObserver {
David Trainor- moved to gerrit 2016/10/07 07:04:59 Leave OnRecordWholeDocumentChanged here IMO.
Menglin 2016/10/08 00:20:55 Done.
+ public:
+ virtual ~SettingsObserver() = default;
+
+ // Invoked when enabling or disabling Blimp mode in the Settings.
+ virtual void OnBlimpModeEnabled(bool enable) {}
+
+ // Invoked when enabling or disabling to show the network stats in the
+ // Settings.
+ virtual void OnShowNetworkStatsChanged(bool enable) {}
+
+ // Invoked when a settings changed that requires the application to restart
+ // before it can take effect.
+ virtual void OnRestartRequired() {}
+
+ protected:
+ SettingsObserver() = default;
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(SettingsObserver);
+};
+
+} // namespace client
+} // namespace blimp
+
+#endif // BLIMP_CLIENT_CORE_SETTINGS_SETTINGS_OBSERVER_H_

Powered by Google App Engine
This is Rietveld 408576698