Disable supervised user creation when force sign in is enabled.

chrome/browser/supervised_user/supervised_user_creation_policy_handler.h

+// Copyright 2016 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.
+
+#ifndef CHROME_BROWSER_SUPERVISED_USER_SUPERVISED_USER_CREATION_POLICY_HANDLER_H_
+#define CHROME_BROWSER_SUPERVISED_USER_SUPERVISED_USER_CREATION_POLICY_HANDLER_H_
+
+#include "base/macros.h"
+#include "components/policy/core/browser/configuration_policy_handler.h"
+
+class PrefValueMap;
+
+namespace policy {

[Marc Treib, 2016/12/06 09:28:36]

This probably shouldn't be in the policy namespace if it's in the
supervised_user folder.

[zmin, 2016/12/06 17:04:55]

I have checked other policy handlers that are not located in policy/ folder.
About half of them are using the policy namespace while the rest of them are
either in its own namespace or no namespace at all. So, even there isn't very
strong consistency, most of handlers are using policy namespace. In the other
hand, I think the folder and the namespace is able to show that this class is
associated with "supervised user" and "policy" at the same time.

[Marc Treib, 2016/12/06 17:13:39]

Hm alright, fair enough. I still think this kind of "scattered namespace" is
very uncommon outside of policy, but oh well.

+
+class PolicyMap;
+
+class SupervisedUserCreationPolicyHandler : public TypeCheckingPolicyHandler {
+ public:
+  SupervisedUserCreationPolicyHandler();
+  ~SupervisedUserCreationPolicyHandler() override;
+
+  void ApplyPolicySettings(const PolicyMap& policies,
+                           PrefValueMap* prefs) override;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(SupervisedUserCreationPolicyHandler);
+};
+
+}  // namespace policy
+
+#endif  // CHROME_BROWSER_SUPERVISED_USER_SUPERVISED_USER_CREATION_POLICY_HANDLER_H_