Allow HostCache to handle network changes itself

Allow HostCache to handle network changes itself

HostCache can already expire all of its entries when the network changes
and return them only when entries from other networks are allowed. This
change makes HostResolverImpl notify HostCache on network change instead
of clearing it, and modifies the StaleHostResolver tests to simulate a
network change using NetworkChangeNotifier rather than directly
notifying HostCache so that the relevant codepath is tested. The
addition of net_java in deps is necessary to use NetworkChangeNotifier
on Android.

This change finishes implementation of the "allow_other_networks" option
in Cronet's "StaleDNS" experiment.

BUG=691078
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

Review-Url: https://codereview.chromium.org/2692043002
Cr-Commit-Position: refs/heads/master@{#450096}
Committed: https://chromium.googlesource.com/chromium/src/+/5bb3b9d0c571ca00d511f36411a4dd06397b181f

[mgersh, 2017-02-13 17:56:34]

Description was changed from

==========
Allow HostCache to handle network changes itself

HostCache can already expire all of its entries when the network changes
and return them only when entries from other networks are allowed. This
change makes HostResolverImpl notify HostCache on network change instead
of clearing it, and modifies the StaleHostResolver tests to simulate a
network change using NetworkChangeNotifier rather than directly
notifying HostCache so that the relevant codepath is tested. The
addition of net_java in deps is necessary to use NetworkChangeNotifier
on Android.

BUG=691078
==========

to

==========
Allow HostCache to handle network changes itself

HostCache can already expire all of its entries when the network changes
and return them only when entries from other networks are allowed. This
change makes HostResolverImpl notify HostCache on network change instead
of clearing it, and modifies the StaleHostResolver tests to simulate a
network change using NetworkChangeNotifier rather than directly
notifying HostCache so that the relevant codepath is tested. The
addition of net_java in deps is necessary to use NetworkChangeNotifier
on Android.

BUG=691078
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

[mgersh, 2017-02-13 18:01:15]

Description was changed from

==========
Allow HostCache to handle network changes itself

HostCache can already expire all of its entries when the network changes
and return them only when entries from other networks are allowed. This
change makes HostResolverImpl notify HostCache on network change instead
of clearing it, and modifies the StaleHostResolver tests to simulate a
network change using NetworkChangeNotifier rather than directly
notifying HostCache so that the relevant codepath is tested. The
addition of net_java in deps is necessary to use NetworkChangeNotifier
on Android.

BUG=691078
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

to

==========
Allow HostCache to handle network changes itself

HostCache can already expire all of its entries when the network changes
and return them only when entries from other networks are allowed. This
change makes HostResolverImpl notify HostCache on network change instead
of clearing it, and modifies the StaleHostResolver tests to simulate a
network change using NetworkChangeNotifier rather than directly
notifying HostCache so that the relevant codepath is tested. The
addition of net_java in deps is necessary to use NetworkChangeNotifier
on Android.

This change finishes implementation of the "allow_other_networks" option
in Cronet's "StaleDNS" experiment.

BUG=691078
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

[mgersh, 2017-02-13 18:01:35]

mgersh@chromium.org changed reviewers:
+ juliatuttle@chromium.org

[mgersh, 2017-02-13 18:01:36]

PTAL

[Julia Tuttle, 2017-02-13 19:48:33]

On 2017/02/13 18:01:36, mgersh wrote:
> PTAL

lgtm

[mgersh, 2017-02-13 19:51:31]

The CQ bit was checked by mgersh@chromium.org

[commit-bot: I haz the power, 2017-02-13 19:52:23]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-13 21:27:14]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487015491653290, "parent_rev":
"1e17147805b80e1953b51b0ec29cf26798c09b13", "commit_rev":
"5bb3b9d0c571ca00d511f36411a4dd06397b181f"}

[commit-bot: I haz the power, 2017-02-13 21:27:52]

Description was changed from

==========
Allow HostCache to handle network changes itself

HostCache can already expire all of its entries when the network changes
and return them only when entries from other networks are allowed. This
change makes HostResolverImpl notify HostCache on network change instead
of clearing it, and modifies the StaleHostResolver tests to simulate a
network change using NetworkChangeNotifier rather than directly
notifying HostCache so that the relevant codepath is tested. The
addition of net_java in deps is necessary to use NetworkChangeNotifier
on Android.

This change finishes implementation of the "allow_other_networks" option
in Cronet's "StaleDNS" experiment.

BUG=691078
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

to

==========
Allow HostCache to handle network changes itself

HostCache can already expire all of its entries when the network changes
and return them only when entries from other networks are allowed. This
change makes HostResolverImpl notify HostCache on network change instead
of clearing it, and modifies the StaleHostResolver tests to simulate a
network change using NetworkChangeNotifier rather than directly
notifying HostCache so that the relevant codepath is tested. The
addition of net_java in deps is necessary to use NetworkChangeNotifier
on Android.

This change finishes implementation of the "allow_other_networks" option
in Cronet's "StaleDNS" experiment.

BUG=691078
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

Review-Url: https://codereview.chromium.org/2692043002
Cr-Commit-Position: refs/heads/master@{#450096}
Committed:
https://chromium.googlesource.com/chromium/src/+/5bb3b9d0c571ca00d511f36411a4...
==========

[commit-bot: I haz the power, 2017-02-13 21:27:53]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5bb3b9d0c571ca00d511f36411a4...