Blimp Settings framework on the c++ side

blimp/client/core/settings/settings.cc

+// 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.
+
+#include "blimp/client/core/settings/settings.h"
+
+#include "base/command_line.h"
+#include "blimp/client/core/settings/settings_observer.h"
+#include "blimp/client/core/settings/settings_prefs.h"
+#include "blimp/client/core/switches/blimp_client_switches.h"
+#include "components/prefs/pref_registry_simple.h"
+#include "components/prefs/pref_service.h"
+
+namespace blimp {
+namespace client {
+
+Settings::Settings(PrefService* local_state)
+    : local_state_(local_state),
+      blimp_enabled_(false),
+      record_whole_document_(false),
+      show_network_stats_(false) {
+  InitializeFromCommandLineAndPref();
+}
+
+Settings::~Settings() = default;
+
+void Settings::AddObserver(SettingsObserver* observer) {
+  observers_.AddObserver(observer);
+}
+
+void Settings::RemoveObserver(SettingsObserver* observer) {
+  observers_.RemoveObserver(observer);
+}
+
+void Settings::SetShowNetworkStats(bool enable) {
+  if (show_network_stats_ == enable)
+    return;
+
+  show_network_stats_ = enable;
+  FOR_EACH_OBSERVER(SettingsObserver, observers_,
+                    OnShowNetworkStatsChanged(show_network_stats_));
+}
+
+void Settings::SetEnableBlimpMode(bool enable) {
+  if (blimp_enabled_ == enable)
+    return;
+
+  blimp_enabled_ = enable;
+  local_state_->SetBoolean(prefs::kBlimpEnabled, enable);
+  FOR_EACH_OBSERVER(SettingsObserver, observers_, OnBlimpModeEnabled(enable));
+  FOR_EACH_OBSERVER(SettingsObserver, observers_, OnRestartRequired());
+}
+
+void Settings::SetRecordWholeDocument(bool enable) {
+  if (record_whole_document_ == enable)
+    return;
+
+  record_whole_document_ = enable;
+  local_state_->SetBoolean(prefs::kRecordWholeDocument, enable);
+  FOR_EACH_OBSERVER(SettingsObserver, observers_,
+                    OnRecordWholeDocumentChanged(enable));
+  FOR_EACH_OBSERVER(SettingsObserver, observers_, OnRestartRequired());
+}
+
+void Settings::InitializeFromCommandLineAndPref() {
+  const base::CommandLine* cmd_line = base::CommandLine::ForCurrentProcess();
+
+  // Get blimp_enabled_ from cmd line and persistent storage.

[David Trainor- moved to gerrit, 2016/10/05 23:46:20]

s/and/or

[Menglin, 2016/10/06 22:36:38]

Done.

+  blimp_enabled_ = cmd_line->HasSwitch(switches::kEnableBlimp) ||
+      local_state_->GetBoolean(prefs::kBlimpEnabled);
+
+  // Get record_whole_document_ from cmd line and persistent storage, and save

[David Trainor- moved to gerrit, 2016/10/05 23:46:20]

s/and/or

[Menglin, 2016/10/06 22:36:38]

Done.

+  // the result to persistent storage.
+  record_whole_document_ =
+      cmd_line->HasSwitch(switches::kDownloadWholeDocument) ||
+      local_state_->GetBoolean(prefs::kRecordWholeDocument);
+  local_state_->SetBoolean(prefs::kRecordWholeDocument, record_whole_document_);

[David Trainor- moved to gerrit, 2016/10/05 23:46:20]

I'm not sure we want to do this right?  We don't with blimp_enabled_ either.

[Menglin, 2016/10/06 22:36:38]

Done.

+}
+
+// static
+void Settings::RegisterPrefs(PrefRegistrySimple* registry) {
+  registry->RegisterBooleanPref(prefs::kBlimpEnabled, false);
+  registry->RegisterBooleanPref(prefs::kRecordWholeDocument, false);
+}
+
+}  // namespace client
+}  // namespace blimp