[CORS-RFC1918] Add UseCounter integration tests.

Second try, this time minus the flaky test? I am hoping that with the parent CL fixing crbug.com/1134601, UseCounter behavior will be a bit more deterministic. In any case, I broke out the flaky test to its own follow-up CL so that it can be more easily reverted in isolation. This mostly reverts commit fb82736e. Bug: chromium:1134601, chromium:1124358, chromium:1135654 Change-Id: I8ded2e0651f9efee2489b5e45de4ebd5b2df6169 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2490012 Auto-Submit: Titouan Rigoudy <titouan@chromium.org> Reviewed-by: David Roger <droger@chromium.org> Reviewed-by: Arthur Sonzogni <arthursonzogni@chromium.org> Commit-Queue: Titouan Rigoudy <titouan@chromium.org> Cr-Commit-Position: refs/heads/master@{#821201}

[CORS-RFC1918] Add UseCounter integration tests.
Second try, this time minus the flaky test? I am hoping that with the parent CL fixing crbug.com/1134601, UseCounter behavior will be a bit more deterministic. In any case, I broke out the flaky test to its own follow-up CL so that it can be more easily reverted in isolation. This mostly reverts commit fb82736e. Bug: chromium:1134601, chromium:1124358, chromium:1135654 Change-Id: I8ded2e0651f9efee2489b5e45de4ebd5b2df6169 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2490012 Auto-Submit: Titouan Rigoudy <titouan@chromium.org> Reviewed-by: David Roger <droger@chromium.org> Reviewed-by: Arthur Sonzogni <arthursonzogni@chromium.org> Commit-Queue: Titouan Rigoudy <titouan@chromium.org> Cr-Commit-Position: refs/heads/master@{#821201}
927abbb9 · Titouan Rigoudy · Commit Bot · c756c75c · 927abbb9 · 927abbb9
Commit 927abbb9 authored Oct 27, 2020 by Titouan Rigoudy Committed by Commit Bot Oct 27, 2020
5 changed files
--- a/chrome/browser/OWNERS
+++ b/chrome/browser/OWNERS
@@ -89,5 +89,9 @@ per-file chrome_back_forward_cache_browsertest.cc=file://content/OWNERS
 per-file chrome_cross_origin_opener_policy_browsertest.cc=file://content/OWNERS
 per-file chrome_cross_origin_opener_policy_browsertest.cc=arthursonzogni@chromium.org

+# CORS-RFC1918
+per-file chrome_private_network_request_browsertest.cc=file://content/OWNERS
+per-file chrome_private_network_request_browsertest.cc=titouan@chromium.org
+
 # COMPONENT: UI>Browser
 # TEAM: chromium-reviews@chromium.org
--- a/chrome/browser/chrome_private_network_request_browsertest.cc
+++ b/chrome/browser/chrome_private_network_request_browsertest.cc
--- a/chrome/test/BUILD.gn
+++ b/chrome/test/BUILD.gn
@@ -979,6 +979,7 @@ if (!is_android) {
      "../browser/chrome_navigation_browsertest.cc",
      "../browser/chrome_origin_trials_browsertest.cc",
      "../browser/chrome_plugin_browsertest.cc",
+      "../browser/chrome_private_network_request_browsertest.cc",
      "../browser/chrome_security_exploit_browsertest.cc",
      "../browser/chrome_service_worker_browsertest.cc",
      "../browser/chrome_worker_browsertest.cc",

--- a/chrome/test/data/treat-as-public-address.html
+++ b/chrome/test/data/treat-as-public-address.html
+<html>
+  <head>
+    <!-- Disable extra network request for /favicon.ico -->
+    <link rel="icon" href="data:,">
+  </head>
+  <body></body>
+</html>
--- a/chrome/test/data/treat-as-public-address.html.mock-http-headers
+++ b/chrome/test/data/treat-as-public-address.html.mock-http-headers
+HTTP/1.1 200 OK
+Content-Security-Policy: treat-as-public-address