Enable cloud policy by default.

chrome/browser/policy/browser_policy_connector.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/browser_policy_connector.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/file_path.h"
@@ skipping 203 matching lines @@
 }
 
 DeviceMode BrowserPolicyConnector::GetDeviceMode() {
   return install_attributes_.get() ? install_attributes_->GetMode()
                                    : DEVICE_MODE_NOT_SET;
 }
 #endif
 
 void BrowserPolicyConnector::ScheduleServiceInitialization(
     int64 delay_milliseconds) {
-  device_management_service_->ScheduleInitialization(delay_milliseconds);
+  // Skip device initialization if the BrowserPolicyConnector was never
+  // initialized (unit tests).
+  if (device_management_service_.get())
+    device_management_service_->ScheduleInitialization(delay_milliseconds);
 }
 
 #if defined(OS_CHROMEOS)
 void BrowserPolicyConnector::InitializeUserPolicy(
     const std::string& user_name,
     bool is_public_account,
     bool wait_for_policy_fetch) {
   // If the user is managed then importing certificates from ONC policy is
   // allowed, otherwise it's not. Update this flag once the user has signed in,
   // and before user policy is loaded.
@@ skipping 127 matching lines @@
   status = U_ZERO_ERROR;
   UBool match = matcher.matches(status);
   DCHECK(U_SUCCESS(status));
   return !!match;  // !! == convert from UBool to bool
 }
 
 }  // namespace
 
 // static
 bool BrowserPolicyConnector::IsNonEnterpriseUser(const std::string& username) {
-  if (username.empty()) {
-    // This means incognito user in case of ChromiumOS and
-    // no logged-in user in case of Chromium (SigninService).
+  if (username.empty() || username.find('@') == std::string::npos) {
+    // An empty username means incognito user in case of ChromiumOS and
+    // no logged-in user in case of Chromium (SigninService). Many tests use
+    // nonsense email addresses (e.g. 'test') so treat those as non-enterprise
+    // users.
     return true;
   }
 
   // Exclude many of the larger public email providers as we know these users
   // are not from hosted enterprise domains.
   static const wchar_t* kNonManagedDomainPatterns[] = {
     L"aol\\.com",
     L"googlemail\\.com",
     L"gmail\\.com",
     L"hotmail(\\.co|\\.com|)\\.[^.]+", // hotmail.com, hotmail.it, hotmail.co.uk
@@ skipping 135 matching lines @@
     return new AsyncPolicyProvider(loader.Pass());
   } else {
     return NULL;
   }
 #else
   return NULL;
 #endif
 }
 
 }  // namespace policy