Add managed device signal

chrome/browser/chromeos/extensions/info_private_api.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/chromeos/extensions/info_private_api.h"
 
 #include <stddef.h>
 
 #include <utility>
 
@@ skipping 117 matching lines @@
 const char kPlayStoreStatusNotAvailable[] = "not available";
 
 // Key which corresponds to the "available" value of the PlayStoreStatus enum in
 // JS.
 const char kPlayStoreStatusAvailable[] = "available";
 
 // Key which corresponds to the "enabled" value of the PlayStoreStatus enum in
 // JS.
 const char kPlayStoreStatusEnabled[] = "enabled";
 
+// Key which corresponds to the managedDeviceStatus property in JS.
+const char kPropertyManagedDeviceStatus[] = "managedDeviceStatus";
+
+// Property indicating an unmanaged device,  which corresponds to the "not

[tbarzic, 2017/02/22 06:55:27]

how about:
"Value to which managedDeviceStatus property is set for unmanaged devices."

[Wenzhao (Colin) Zang, 2017/02/22 20:30:56]

Done.

+// mananged" value of the ManagedDeviceStatus enum in JS.
+const char kManagedDeviceStatusNotManaged[] = "not managed";
+
+// Property indicating a managed device, which corresponds to the "managed"
+// value of the ManagedDeviceStatus enum in JS.
+const char kManagedDeviceStatusManaged[] = "managed";
+
 const struct {
   const char* api_name;
   const char* preference_name;
 } kPreferencesMap[] = {
     {kPropertyLargeCursorEnabled, prefs::kAccessibilityLargeCursorEnabled},
     {kPropertyStickyKeysEnabled, prefs::kAccessibilityStickyKeysEnabled},
     {kPropertySpokenFeedbackEnabled,
      prefs::kAccessibilitySpokenFeedbackEnabled},
     {kPropertyHighContrastEnabled, prefs::kAccessibilityHighContrastEnabled},
     {kPropertyScreenMagnifierEnabled,
@@ skipping 113 matching lines @@
   }
 
   if (property_name == kPropertyPlayStoreStatus) {
     if (arc::IsArcAllowedForProfile(Profile::FromBrowserContext(context_)))
       return new base::StringValue(kPlayStoreStatusEnabled);
     if (arc::IsArcAvailable())
       return new base::StringValue(kPlayStoreStatusAvailable);
     return new base::StringValue(kPlayStoreStatusNotAvailable);
   }
 
+  if (property_name == kPropertyManagedDeviceStatus) {
+    policy::BrowserPolicyConnectorChromeOS* connector =
+        g_browser_process->platform_part()->browser_policy_connector_chromeos();
+    if (connector->IsEnterpriseManaged()) {

[tbarzic, 2017/02/22 06:55:27]

nit: no {} here

[Wenzhao (Colin) Zang, 2017/02/22 20:30:56]

Done. Out of curiosity, is there a reason why in general we don't want {} here? 

+      return new base::StringValue(kManagedDeviceStatusManaged);
+    }
+    return new base::StringValue(kManagedDeviceStatusNotManaged);
+  }
+
   if (property_name == kPropertyClientId) {
     return new base::StringValue(GetClientId());
   }
 
   if (property_name == kPropertyTimezone) {
     return chromeos::CrosSettings::Get()->GetPref(
             chromeos::kSystemTimezone)->DeepCopy();
   }
 
   if (property_name == kPropertySupportedTimezones) {
@@ skipping 37 matching lines @@
           param_value);
     } else {
       return RespondNow(Error(kPropertyNotFound, param_name));
     }
   }
 
   return RespondNow(NoArguments());
 }
 
 }  // namespace extensions