HttpServerProperties - Don't persist if SetServerNetworkStats is called

HttpServerProperties - Don't persist if SetServerNetworkStats is called
with the same origin (HostPortPair) and value (ServerNetworkStats).

This change reduces the number of times SetServerNetworkStats tries
to persist HttpServerProperties to disk (currently 17% of persist calls
are due to SetServerNetworkStats).

BUG=451256
R=rch@chromium.org

Committed: https://crrev.com/cce34d58dbec10d9d499e6b54e4a15d9fa66fcba
Cr-Commit-Position: refs/heads/master@{#333173}

[ramant (doing other things), 2015-05-28 22:32:42]

Patchset #2 (id:20001) has been deleted

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

https://codereview.chromium.org/1158823005/diff/40001/net/http/http_server_pr...
File net/http/http_server_properties_impl.cc (right):

https://codereview.chromium.org/1158823005/diff/40001/net/http/http_server_pr...
net/http/http_server_properties_impl.cc:497: return !old_stats || *old_stats !=
stats;
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]

https://codereview.chromium.org/1158823005/diff/40001/net/http/http_server_pr...
File net/http/http_server_properties_impl.cc (right):

https://codereview.chromium.org/1158823005/diff/40001/net/http/http_server_pr...
net/http/http_server_properties_impl.cc:497: return !old_stats || *old_stats !=
stats;
On 2015/05/28 22:33:38, ramant wrote:
> Hi Ryan,
>   Made a small change to update the Memory cache always (to keep the MRU up to
> date).

Please ignore this comment.

[Ryan Hamilton, 2015-06-03 23:43:25]

lgtm

[ramant (doing other things), 2015-06-04 01:02:51]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2015-06-04 01:04:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1158823005/60001

[commit-bot: I haz the power, 2015-06-04 02:54:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-04 02:54:34]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[ramant (doing other things), 2015-06-05 18:40:31]

Patchset #4 (id:80001) has been deleted

[ramant (doing other things), 2015-06-05 18:47:17]

Patchset #4 (id:100001) has been deleted

[ramant (doing other things), 2015-06-05 18:47:40]

The CQ bit was checked by rtenneti@chromium.org

[ramant (doing other things), 2015-06-05 18:47:40]

The patchset sent to the CQ was uploaded after l-g-t-m from rch@chromium.org
Link to the patchset: https://codereview.chromium.org/1158823005/#ps120001
(title: "Fix ASAN build")

[commit-bot: I haz the power, 2015-06-05 18:48:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1158823005/120001

[commit-bot: I haz the power, 2015-06-05 20:10:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-05 20:10:43]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[ramant (doing other things), 2015-06-05 22:40:49]

Hi Ryan,
  After you had LGTM'ed the change, made a change to fix ASAN error and I did a
rebase to merge recent changes.

  Wondering if GetServerNetworkStats should return the object instead of a
pointer to avoid ownership issues. If so, will submit another cleanup CL that
returns the object (instead of a pointer).

PTAL.

thanks
raman

[ramant (doing other things), 2015-06-05 23:08:36]

The CQ bit was checked by rtenneti@chromium.org

[ramant (doing other things), 2015-06-05 23:08:36]

The patchset sent to the CQ was uploaded after l-g-t-m from rch@chromium.org
Link to the patchset: https://codereview.chromium.org/1158823005/#ps140001
(title: "Rebase and fix valgrind error")

[commit-bot: I haz the power, 2015-06-05 23:10:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1158823005/140001

[commit-bot: I haz the power, 2015-06-05 23:36:45]

Committed patchset #5 (id:140001)

[commit-bot: I haz the power, 2015-06-05 23:37:38]

Patchset 5 (id:??) landed as
https://crrev.com/cce34d58dbec10d9d499e6b54e4a15d9fa66fcba
Cr-Commit-Position: refs/heads/master@{#333173}