DNS: Retain stale entries in HostCache and return when requested

net/dns/dns_util.h

 // Copyright (c) 2012 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 NET_DNS_DNS_UTIL_H_
 #define NET_DNS_DNS_UTIL_H_
 
 #include <string>
 
 #include "base/strings/string_piece.h"
 #include "base/time/time.h"
 #include "net/base/net_export.h"
 #include "net/base/network_change_notifier.h"
 
 namespace net {
 
+class AddressList;
+
 // DNSDomainFromDot - convert a domain string to DNS format. From DJB's
 // public domain DNS library.
 //
 //   dotted: a string in dotted form: "www.google.com"
 //   out: a result in DNS form: "\x03www\x06google\x03com\x00"
 NET_EXPORT_PRIVATE bool DNSDomainFromDot(const base::StringPiece& dotted,
                                          std::string* out);
 
 // DNSDomainToString converts a domain in DNS format to a dotted string.
 // Excludes the dot at the end.
 NET_EXPORT_PRIVATE std::string DNSDomainToString(
     const base::StringPiece& domain);
 
 // Returns true if it can determine that only loopback addresses are configured.
 // i.e. if only 127.0.0.1 and ::1 are routable.
 // Also returns false if it cannot determine this.
 NET_EXPORT_PRIVATE bool HaveOnlyLoopbackAddresses();
 
 #if !defined(OS_NACL)
 NET_EXPORT_PRIVATE
 base::TimeDelta GetTimeDeltaForConnectionTypeFromFieldTrialOrDefault(
     const char* field_trial_name,
     base::TimeDelta default_delta,
     NetworkChangeNotifier::ConnectionType connection_type);
 #endif  // !defined(OS_NACL)
 
+// Used in histograms; please only insert new entries before MAX.

[Ryan Sleevi, 2016/04/26 02:18:22]

Comment nit: While politeness is a virtue, it leaves ambiguity that it 'might be
ok' to ignore that please.

// Used in histograms. New values should be added at the end, before MAX

[Ryan Sleevi, 2016/04/26 02:18:22]

Comment nit: This doesn't explain the type

// The result of comparing two AddressLists for the set of addresses
// they contain and whether any differences exist.

(Or something?)

[Ryan Sleevi, 2016/04/26 02:18:22]

If they're used in histograms, you should add explicit values. Inserting a value
in between two entries means they would be renumbered and throw off the
histograms.

[Julia Tuttle, 2016/04/27 14:47:04]

Done (all three; the new rietveld UI won't let me reply to individual comments,
just the "thread" attached to the line).

+enum AddressListDeltaType {
+  // a and b contain the same addresses in the same order.

[Ryan Sleevi, 2016/04/26 02:18:22]

nit: "a" and "b" are not defined here (they're an implementation of
FindAddressListDeltaType). Perhaps "Both AddressLists contain the same addresses
in the same order." (and similarly throughout)

[Julia Tuttle, 2016/04/27 14:47:04]

Done.

+  DELTA_IDENTICAL,
+  // a and b contain the same addresses in a different order.
+  DELTA_REORDERED,
+  // a and b have at least one address in common, but not all of them.
+  DELTA_OVERLAP,
+  // a and b have no addresses in common.
+  DELTA_DISJOINT,
+  MAX_DELTA_TYPE

[Ryan Sleevi, 2016/04/26 02:18:22]

naming: Given that you adopted ENUM_PREFIX naming, shouldn't this be
DELTA_MAX_VALUE ?

[Julia Tuttle, 2016/04/27 14:47:04]

I usually make an exception for the max value constant. Is that bad?

+};
+
+AddressListDeltaType FindAddressListDeltaType(const AddressList& a,

[Ryan Sleevi, 2016/04/26 02:18:22]

Document? :)

[Julia Tuttle, 2016/04/27 14:47:04]

Done (the new rietveld UI has no button for this :( ).

+                                              const AddressList& b);
+
 }  // namespace net
 
 #endif  // NET_DNS_DNS_UTIL_H_