Domain Reliability: More security review.

components/domain_reliability/monitor.h

Index: components/domain_reliability/monitor.h
diff --git a/components/domain_reliability/monitor.h b/components/domain_reliability/monitor.h
index 01ef84393f1e8a791d0b2a7b474f608f5e415b7d..5f6908c4c5bf8d7273435fa069b193eeccf7613f 100644
--- a/components/domain_reliability/monitor.h
+++ b/components/domain_reliability/monitor.h
@@ -38,9 +38,8 @@ class DOMAIN_RELIABILITY_EXPORT DomainReliabilityMonitor {
   // I/O thread.

[Ryan Sleevi, 2014/04/26 02:37:39]

NB: Pronouns in comments considered harmful :)

https://groups.google.com/a/chromium.org/d/topic/chromium-dev/NH-S6KCkr2M/dis...

Rather than discuss this in the constructor, document it in the class design
itself

[Deprecated (see juliatuttle), 2014/04/28 22:01:13]

Done.

   explicit DomainReliabilityMonitor(
       net::URLRequestContext* url_request_context);
-  DomainReliabilityMonitor(
-      net::URLRequestContext* url_request_context,
-      scoped_ptr<MockableTime> time);
+  DomainReliabilityMonitor(net::URLRequestContext* url_request_context,
+                           scoped_ptr<MockableTime> time);
   ~DomainReliabilityMonitor();
 
   // Adds the "baked-in" configuration(s) for Google sites.

[Ryan Sleevi, 2014/04/26 02:37:39]

// Add in configurations that have been configured at compile-time.

Yes, as the reviewer who stamped this, I'm a horrible person :)

[Deprecated (see juliatuttle), 2014/04/28 22:01:13]

Done.