Add source column to chrome://policy showing the origins of policies.

components/policy/core/common/configuration_policy_provider.h

 // Copyright 2013 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 COMPONENTS_POLICY_CORE_COMMON_CONFIGURATION_POLICY_PROVIDER_H_
 #define COMPONENTS_POLICY_CORE_COMMON_CONFIGURATION_POLICY_PROVIDER_H_
 
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 56 matching lines @@
   virtual void RefreshPolicies() = 0;
 
   // Observers must detach themselves before the provider is deleted.
   virtual void AddObserver(Observer* observer);
   virtual void RemoveObserver(Observer* observer);
 
   // SchemaRegistry::Observer:
   void OnSchemaRegistryUpdated(bool has_new_schemas) override;
   void OnSchemaRegistryReady() override;
 
+  // Returns where the provided policies originate from.

[Thiemo Nagel, 2015/09/04 20:15:54]

In my opinion, a 1-line getter isn't worth a comment.

Don't document the obvious. ;)

[fhorschig, 2015/09/07 14:09:34]

Done.

+  PolicySource source() const { return source_; }
+
  protected:
   // Subclasses must invoke this to update the policies currently served by
   // this provider. UpdatePolicy() takes ownership of |policies|.
   // The observers are notified after the policies are updated.
   void UpdatePolicy(scoped_ptr<PolicyBundle> bundle);
 
   SchemaRegistry* schema_registry() const;
 
   const scoped_refptr<SchemaMap>& schema_map() const;
 
+  // This provider provides a policies only from one source at a time. And this

[Thiemo Nagel, 2015/09/04 20:15:54]

"a policies" --> "policies"

". And this source is stored in this value" --> ""

[fhorschig, 2015/09/07 14:09:34]

Done.

+  // source is stored in this value (defaulting to POLICY_SOURCE_UNKNOWN).
+  PolicySource source_;
+
  private:
+  // Used to tag all policies provided by this provider as coming from a

[Thiemo Nagel, 2015/09/04 20:15:54]

Not from "a specific source" but from "|source_|".

[fhorschig, 2015/09/07 14:09:34]

Done.

+  // specific source.
+  void SetPolicySources();
+
   // The policies currently configured at this provider.
   PolicyBundle policy_bundle_;
 
   // Used to validate proper Init() and Shutdown() nesting. This flag is set by
   // Init() and cleared by Shutdown() and needs to be false in the destructor.
   bool initialized_;
 
   SchemaRegistry* schema_registry_;
 
   base::ObserverList<Observer, true> observer_list_;
 
   DISALLOW_COPY_AND_ASSIGN(ConfigurationPolicyProvider);
 };
 
 }  // namespace policy
 
 #endif  // COMPONENTS_POLICY_CORE_COMMON_CONFIGURATION_POLICY_PROVIDER_H_