Commit c6555e23 authored by Matt Menke's avatar Matt Menke Committed by Commit Bot

Update PartitionConnectionsByNetworkIsolationKey field trial config.

Add splitting the DNS cache experiment, prior to updating the
server-side field trial to include that feature.

Also fix NetInternalsTest test fixture to be compatible with splitting
the DNS cache by NetworkIsolationKey.

Bug: 963480
Change-Id: Iba98d8e27a9692ab2fe2bcbbe73a15bfd3415bd6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2150685Reviewed-by: default avatarEric Orth <ericorth@chromium.org>
Reviewed-by: default avatarSteven Holte <holte@chromium.org>
Commit-Queue: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#760247}
parent d51ce450
...@@ -147,6 +147,11 @@ class NetInternalsTest::MessageHandler : public content::WebUIMessageHandler { ...@@ -147,6 +147,11 @@ class NetInternalsTest::MessageHandler : public content::WebUIMessageHandler {
NetInternalsTest* net_internals_test_; NetInternalsTest* net_internals_test_;
// Single NetworkIsolationKey used for all DNS lookups, so repeated lookups
// use the same cache key.
net::NetworkIsolationKey network_isolation_key_{
net::NetworkIsolationKey::CreateTransient()};
base::WeakPtrFactory<MessageHandler> weak_factory_{this}; base::WeakPtrFactory<MessageHandler> weak_factory_{this};
DISALLOW_COPY_AND_ASSIGN(MessageHandler); DISALLOW_COPY_AND_ASSIGN(MessageHandler);
...@@ -230,8 +235,7 @@ void NetInternalsTest::MessageHandler::DnsLookup( ...@@ -230,8 +235,7 @@ void NetInternalsTest::MessageHandler::DnsLookup(
weak_factory_.GetWeakPtr())); weak_factory_.GetWeakPtr()));
content::BrowserContext::GetDefaultStoragePartition(browser()->profile()) content::BrowserContext::GetDefaultStoragePartition(browser()->profile())
->GetNetworkContext() ->GetNetworkContext()
->ResolveHost(net::HostPortPair(hostname, 80), ->ResolveHost(net::HostPortPair(hostname, 80), network_isolation_key_,
net::NetworkIsolationKey::CreateTransient(),
std::move(resolve_host_parameters), std::move(client)); std::move(resolve_host_parameters), std::move(client));
} }
......
...@@ -3986,11 +3986,12 @@ ...@@ -3986,11 +3986,12 @@
], ],
"experiments": [ "experiments": [
{ {
"name": "PartitionConnectionsAndSSLSessionsAndQuic", "name": "Enabled4",
"enable_features": [ "enable_features": [
"PartitionConnectionsByNetworkIsolationKey", "PartitionConnectionsByNetworkIsolationKey",
"PartitionHttpServerPropertiesByNetworkIsolationKey", "PartitionHttpServerPropertiesByNetworkIsolationKey",
"PartitionSSLSessionsByNetworkIsolationKey" "PartitionSSLSessionsByNetworkIsolationKey",
"SplitHostCacheByNetworkIsolationKey"
] ]
} }
] ]
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment