Replace the WAS_INSTALLED_BY_CUSTODIAN creation flag with a pref

extensions/common/extension.h

Index: extensions/common/extension.h
diff --git a/extensions/common/extension.h b/extensions/common/extension.h
index 278a8c488bb30f4de4ea0ca92b198125151cb939..a302c764e9b4462ecce4460c069c4f895424a4a2 100644
--- a/extensions/common/extension.h
+++ b/extensions/common/extension.h
@@ -154,7 +154,8 @@ class Extension : public base::RefCountedThreadSafe<Extension> {
 
     // |WAS_INSTALLED_BY_CUSTODIAN| means this extension was installed by the
     // custodian of a supervised user.

[Marc Treib, 2016/06/17 08:56:54]

I'd remove this comment now - if someone cares, they should look at the pref
instead. (This one is bound to get out of date)

[mamir, 2016/06/17 16:12:55]

Done.

-    WAS_INSTALLED_BY_CUSTODIAN = 1 << 11,
+    // DEPRECATED:: WAS_INSTALLED_BY_CUSTODIAN is now stored as a pref instead.
+    // WAS_INSTALLED_BY_CUSTODIAN= 1 << 11,
 
     // |MAY_BE_UNTRUSTED| indicates that this extension came from a potentially
     // unsafe source (e.g., sideloaded from a local CRX file via the Windows
@@ -331,9 +332,6 @@ class Extension : public base::RefCountedThreadSafe<Extension> {
   bool was_installed_by_oem() const {
     return (creation_flags_ & WAS_INSTALLED_BY_OEM) != 0;
   }
-  bool was_installed_by_custodian() const {
-    return (creation_flags_ & WAS_INSTALLED_BY_CUSTODIAN) != 0;
-  }
 
   // Type-related queries.
   bool is_app() const;