Domain Reliability: Simplify configs and reports

components/domain_reliability/google_configs.h

Index: components/domain_reliability/google_configs.h
diff --git a/components/domain_reliability/google_configs.h b/components/domain_reliability/google_configs.h
new file mode 100644
index 0000000000000000000000000000000000000000..51326c2abdf4315612d9cb0f4a17d40628354038
--- /dev/null
+++ b/components/domain_reliability/google_configs.h
@@ -0,0 +1,19 @@
+// Copyright 2015 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_DOMAIN_RELIABILITY_GOOGLE_CONFIGS_H_
+#define COMPONENTS_DOMAIN_RELIABILITY_GOOGLE_CONFIGS_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "components/domain_reliability/config.h"
+#include "components/domain_reliability/domain_reliability_export.h"
+
+namespace domain_reliability {
+
+void DOMAIN_RELIABILITY_EXPORT GetAllGoogleConfigs(
+    std::vector<DomainReliabilityConfig*>* configs_out);

[Randy Smith (Not in Mondays), 2015/10/29 22:36:11]

nit, idea (weaker than a suggestion): Did you consider using a ScopedVector and
using weak_erase at the point of removing them from the vector?  It's all where
you push the ugliness, but that feels better to me.  (Really, ScopedVector
should have a scoped_ptr<> GetAndRemove(iterator) method, but it doesn't, and
I'm certain there are subtle C++ arguments against it doing so.)

[Deprecated (see juliatuttle), 2015/11/02 23:19:30]

I did consider it, but it felt uglier to me. :)

[Randy Smith (Not in Mondays), 2015/11/03 21:48:12]

Acknowledged.

+
+}  // namespace domain_reliability
+
+#endif // COMPONENTS_DOMAIN_RELIABILITY_GOOGLE_CONFIGS_H_