Consolidate data storage and notifications in the cloud policy subsystem

chrome/browser/policy/cloud_policy_identity_strategy.cc

 // Copyright (c) 2011 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 "chrome/browser/policy/cloud_policy_identity_strategy.h"
+#include "chrome/common/guid.h"
 
 namespace policy {
 
-CloudPolicyIdentityStrategy::CloudPolicyIdentityStrategy() {}
-
-CloudPolicyIdentityStrategy::~CloudPolicyIdentityStrategy() {}
-
-void CloudPolicyIdentityStrategy::AddObserver(Observer* obs) {
-  observer_list_.AddObserver(obs);
+CloudPolicyIdentityStrategy::CloudPolicyIdentityStrategy(CloudPolicyData* data)
+    : data_(data) {
+  data->AddObserver(this);
 }
 
-void CloudPolicyIdentityStrategy::RemoveObserver(Observer* obs) {
-  observer_list_.RemoveObserver(obs);
+CloudPolicyIdentityStrategy::~CloudPolicyIdentityStrategy() {
+  data()->RemoveObserver(this);
 }
 
-void CloudPolicyIdentityStrategy::NotifyDeviceTokenChanged() {
-  FOR_EACH_OBSERVER(Observer, observer_list_, OnDeviceTokenChanged());
+CloudPolicyData* CloudPolicyIdentityStrategy::data() {
+  return data_;
 }
 
-void CloudPolicyIdentityStrategy::NotifyAuthChanged() {
-  FOR_EACH_OBSERVER(Observer, observer_list_, OnCredentialsChanged());
+void CloudPolicyIdentityStrategy::OnDeviceTokenChanged() {
+}
+
+void CloudPolicyIdentityStrategy::OnCredentialsChanged() {
+  if (data()->token_cache_loaded() &&
+      !data()->user_name().empty() &&
+      !data()->gaia_token().empty() &&
+      data()->device_id().empty()) {
+    data()->set_device_id(guid::GenerateGUID());
+  }
 }
 
 }  // namespace policy