HttpServerProperties - Don't persist if SetServerNetworkStats is called

net/http/http_server_properties_impl.cc

 // 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.
 
 #include "net/http/http_server_properties_impl.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 471 matching lines @@
 
 void HttpServerPropertiesImpl::SetSupportsQuic(bool used_quic,
                                                const IPAddressNumber& address) {
   if (!used_quic) {
     last_quic_address_.clear();
   } else {
     last_quic_address_ = address;
   }
 }
 
-void HttpServerPropertiesImpl::SetServerNetworkStats(
+bool HttpServerPropertiesImpl::SetServerNetworkStats(
     const HostPortPair& host_port_pair,
     ServerNetworkStats stats) {
+  const ServerNetworkStats* old_stats = GetServerNetworkStats(host_port_pair);
   server_network_stats_map_.Put(host_port_pair, stats);
+  return !old_stats || *old_stats != stats;

[ramant (doing other things), 2015/05/28 22:33:38]

Hi Ryan,
  Made a small change to update the Memory cache always (to keep the MRU up to
date).

[ramant (doing other things), 2015/06/03 21:37:00]

Please ignore this comment.

 }
 
 const ServerNetworkStats* HttpServerPropertiesImpl::GetServerNetworkStats(
     const HostPortPair& host_port_pair) {
   ServerNetworkStatsMap::iterator it =
       server_network_stats_map_.Get(host_port_pair);
   if (it == server_network_stats_map_.end()) {
     return NULL;
   }
   return &it->second;
@@ skipping 92 matching lines @@
   base::TimeDelta delay = when > now ? when - now : base::TimeDelta();
   base::MessageLoop::current()->PostDelayedTask(
       FROM_HERE,
       base::Bind(
           &HttpServerPropertiesImpl::ExpireBrokenAlternateProtocolMappings,
           weak_ptr_factory_.GetWeakPtr()),
       delay);
 }
 
 }  // namespace net