Domain Reliability: Mark beacons from previous networks.

components/domain_reliability/beacon.cc

Index: components/domain_reliability/beacon.cc
diff --git a/components/domain_reliability/beacon.cc b/components/domain_reliability/beacon.cc
index 452c7027ca03cb0bca9f8e5693b99fee83936c43..b3f3058bca8367a0e6479c9a8fe4148250e35b01 100644
--- a/components/domain_reliability/beacon.cc
+++ b/components/domain_reliability/beacon.cc
@@ -15,7 +15,9 @@ using base::DictionaryValue;
 DomainReliabilityBeacon::DomainReliabilityBeacon() {}
 DomainReliabilityBeacon::~DomainReliabilityBeacon() {}
 
-Value* DomainReliabilityBeacon::ToValue(base::TimeTicks upload_time) const {
+Value* DomainReliabilityBeacon::ToValue(
+    base::TimeTicks upload_time,
+    base::TimeTicks last_network_change_time) const {
   DictionaryValue* beacon_value = new DictionaryValue();
   if (!url.empty())
     beacon_value->SetString("url", url);
@@ -38,6 +40,8 @@ Value* DomainReliabilityBeacon::ToValue(base::TimeTicks upload_time) const {
                            elapsed.InMilliseconds());
   beacon_value->SetInteger("request_age_ms",
                            (upload_time - start_time).InMilliseconds());
+  if (last_network_change_time > start_time + elapsed)
+    beacon_value->SetBoolean("network_changed", true);

[davidben, 2014/10/30 21:17:59]

If the beacon came up because the network changed while we were fetching the
resource and then that aborted the request, would it end up on the correct side?
Do we want to compare with start_time instead?

[Deprecated (see juliatuttle), 2014/10/31 22:01:02]

Done.

   return beacon_value;
 }