UMA metrics for cloud policies.

chrome/browser/policy/cloud_policy_controller.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_controller.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/rand_util.h"
 #include "base/string_util.h"
 #include "chrome/browser/policy/cloud_policy_cache_base.h"
 #include "chrome/browser/policy/cloud_policy_subsystem.h"
 #include "chrome/browser/policy/device_management_backend.h"
 #include "chrome/browser/policy/device_management_service.h"
+#include "chrome/browser/policy/enterprise_metrics.h"
 #include "chrome/browser/policy/proto/device_management_constants.h"
 
 // Domain names that are known not to be managed.
 // We don't register the device when such a user logs in.
 static const char* kNonManagedDomains[] = {
   "@googlemail.com",
   "@gmail.com"
 };
 
 // Checks the domain part of the given username against the list of known
@@ skipping 76 matching lines @@
   if (response.response_size() > 0) {
     if (response.response_size() > 1) {
       LOG(WARNING) << "More than one policy in the response of the device "
                    << "management server, discarding.";
     }
     if (response.response(0).error_code() !=
         DeviceManagementBackend::kErrorServicePolicyNotFound) {
       cache_->SetPolicy(response.response(0));
       SetState(STATE_POLICY_VALID);
     } else {
+      em::LogPolicyOperation(em::kPolicyFetchBadResponse);

[kmixter1, 2011/06/28 01:06:02]

Have we considered/investigated a paging/escalation system for these if we see
spikes in errors?

[Joao da Silva, 2011/06/30 12:57:00]

AFAIK that hasn't been discussed yet. It would be great to have it for
BadResponses and for spikes in other events too. Let's discuss it in the next
meeting.

[kmixter1, 2011/06/30 23:28:59]

Once this hits stable, we should present these in one of the meetings and start
considering how they could be turned into inputs for alert systems.

       SetState(STATE_POLICY_UNAVAILABLE);
     }
+  } else {
+    em::LogPolicyOperation(em::kPolicyFetchBadResponse);
   }
 }
 
 void CloudPolicyController::OnError(DeviceManagementBackend::ErrorCode code) {
   switch (code) {
     case DeviceManagementBackend::kErrorServiceDeviceNotFound:
     case DeviceManagementBackend::kErrorServiceManagementTokenInvalid: {
       LOG(WARNING) << "The device token was either invalid or unknown to the "
                    << "device manager, re-registering device.";
       // Will retry fetching a token but gracefully backing off.
@@ skipping 226 matching lines @@
 }
 
 int64 CloudPolicyController::GetRefreshDelay() {
   int64 deviation = (kPolicyRefreshDeviationFactorPercent *
                      policy_refresh_rate_ms_) / 100;
   deviation = std::min(deviation, kPolicyRefreshDeviationMaxInMilliseconds);
   return policy_refresh_rate_ms_ - base::RandGenerator(deviation + 1);
 }
 
 }  // namespace policy