Fix policy invalidator lifecycle bugs for Android.

chrome/browser/policy/cloud/user_cloud_policy_manager.cc

 // Copyright (c) 2012 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/user_cloud_policy_manager.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "chrome/browser/policy/cloud/cloud_policy_constants.h"
 #include "chrome/browser/policy/cloud/cloud_policy_service.h"
@@ skipping 17 matching lines @@
   UserCloudPolicyManagerFactory::GetInstance()->Register(profile_, this);
 }
 
 UserCloudPolicyManager::~UserCloudPolicyManager() {
   UserCloudPolicyManagerFactory::GetInstance()->Unregister(profile_, this);
 }
 
 void UserCloudPolicyManager::Connect(
     PrefService* local_state, scoped_ptr<CloudPolicyClient> client) {
   core()->Connect(client.Pass());
-  StartRefreshScheduler();
+  core()->StartRefreshScheduler();
   core()->TrackRefreshDelayPref(local_state, prefs::kUserPolicyRefreshRate);
+  StartInvalidator();

[Mattias Nissler (ping if slow), 2013/08/29 16:02:30]

So the signal the invalidator apparently needs to listen to would be "core
connected/disconnected" AFAICS, correct?

[Steve Condie, 2013/08/30 00:16:35]

Yes, these signals are now published by the CloudPolicyCore.

 }
 
 // static
 scoped_ptr<CloudPolicyClient>
 UserCloudPolicyManager::CreateCloudPolicyClient(
     DeviceManagementService* device_management_service) {
   return make_scoped_ptr(
       new CloudPolicyClient(std::string(), std::string(),
                             USER_AFFILIATION_NONE,
                             NULL, device_management_service)).Pass();
 }
 
 void UserCloudPolicyManager::DisconnectAndRemovePolicy() {
+  StopInvalidator();
   core()->Disconnect();
   store_->Clear();
 }
 
 bool UserCloudPolicyManager::IsClientRegistered() const {
   return client() && client()->is_registered();
 }
 
 }  // namespace policy