OOR-CORS: Enabled by default

Now the OOR-CORS m79 rollout is started. To follow the best practices
for incremental feature rollouts, this patch enables the OOR-CORS
by default on TOT.

NOAUTOREVERT=true

Bug: 905971, 1001450, 1035763, 1039165
Change-Id: I2ca742eab6a14891968892f7f1ef40adbe8bad67
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1988127
Auto-Submit: Takashi Toyoshima <toyoshim@chromium.org>
Commit-Queue: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#728462}

services/network/public/cpp/features.cc

@@ -31,12 +31,13 @@ const base::Feature kNetworkService {
       base::FEATURE_ENABLED_BY_DEFAULT
 };
 
-// Out of Blink CORS will be launched at m79. The flag will be enabled by
-// default around m81 after the feature rolled out over the finch successfully
-// at m79. Both mode will be maintained at least until m81, or around m83+ for
-// enterprise supports.
+// Out of Blink CORS for browsers is launched at m79 (http://crbug.com/1001450),
+// and one for WebView will be at m81 (http://crbug.com/1035763).
+// The legacy CORS will be also maintained at least until m81 for enterprise
+// users. See https://sites.google.com/a/chromium.org/dev/Home/loading/oor-cors
+// for FYI Builders information.
 const base::Feature kOutOfBlinkCors{"OutOfBlinkCors",
-                                    base::FEATURE_DISABLED_BY_DEFAULT};
+                                    base::FEATURE_ENABLED_BY_DEFAULT};
 
 const base::Feature kReporting{"Reporting", base::FEATURE_ENABLED_BY_DEFAULT};
 

third_party/blink/web_tests/NeverFixTests

@@ -2085,29 +2085,6 @@ external/wpt/uievents/order-of-events/mouse-events/wheel-basic-manual.html [ Ski
 external/wpt/uievents/order-of-events/mouse-events/wheel-scrolling-manual.html [ Skip ]
 external/wpt/html/webappapis/user-prompts/print-manual.html [ Skip ]
 
-# Tests that do not pass with the Blink CORS, but pass with OOR-CORS.
-# We maintain the Blink CORS for a while, and keep it work without regressions.
-# But will not change the existing behavior for Blink CORS when expectations are
-# changed for the spec update, other features side-effects, and so on.
-# TODO(cbur.com/870173): Double check following failed tests later.
-crbug.com/870173 external/wpt/resource-timing/cors-preflight.any.html [ Skip ]
-crbug.com/870173 external/wpt/resource-timing/cors-preflight.any.worker.html [ Skip ]
-crbug.com/870173 http/tests/devtools/console-xhr-logging.js [ Skip ]
-crbug.com/870173 http/tests/devtools/network/extra-info-redirects.js [ Skip ]
-crbug.com/870173 http/tests/eventsource/eventsource-cors-non-http.html [ Skip ]
-crbug.com/870173 http/tests/eventsource/workers/eventsource-cors-non-http.html [ Skip ]
-crbug.com/870173 http/tests/htmlimports/import-script-block-crossorigin-dynamic.html [ Skip ]
-crbug.com/870173 http/tests/inspector-protocol/network-fetch-content-with-error-status-code.js [ Skip ]
-crbug.com/870173 http/tests/preload/warning/request-mode.html [ Skip ]
-crbug.com/870173 http/tests/security/img-redirect-to-crossorigin-credentials.https.html [ Skip ]
-crbug.com/870173 http/tests/security/script-crossorigin-redirect-credentials.html [ Skip ]
-crbug.com/870173 http/tests/xmlhttprequest/cross-origin-unsupported-url.html [ Skip ]
-crbug.com/870173 http/tests/xmlhttprequest/workers/cross-origin-unsupported-url.html [ Skip ]
-crbug.com/870173 mhtml/cid_in_html_resource.html [ Skip ]
-crbug.com/870173 virtual/omt-worker-fetch/external/wpt/resource-timing/cors-preflight.any.html [ Skip ]
-crbug.com/870173 virtual/omt-worker-fetch/external/wpt/resource-timing/cors-preflight.any.worker.html [ Skip ]
-crbug.com/870173 virtual/omt-worker-fetch/http/tests/workers/worker-redirect.html [ Skip ]
-
 # Windows doesn't have native overlay scrollbar
 [ Win ] virtual/overlay-scrollbar/* [ Skip ]
 

third_party/blink/web_tests/external/wpt/service-workers/service-worker/fetch-request-xhr.https-expected.txt

 This is a testharness.js-based test.
 PASS initialize global state
 FAIL event.request has the expected headers for same-origin GET. promise_test: Unhandled rejection with value: object "Error: assert_array_equals: event.request has the expected headers for same-origin GET. lengths differ, expected 1 got 3"
-FAIL event.request has the expected headers for same-origin POST. promise_test: Unhandled rejection with value: object "Error: assert_array_equals: event.request has the expected headers for same-origin POST. lengths differ, expected 2 got 5"
+FAIL event.request has the expected headers for same-origin POST. promise_test: Unhandled rejection with value: object "Error: assert_array_equals: event.request has the expected headers for same-origin POST. lengths differ, expected 2 got 4"
 FAIL event.request has the expected headers for cross-origin GET. promise_test: Unhandled rejection with value: object "Error: assert_array_equals: event.request has the expected headers for cross-origin GET. lengths differ, expected 1 got 3"
-FAIL event.request has the expected headers for cross-origin POST. promise_test: Unhandled rejection with value: object "Error: assert_array_equals: event.request has the expected headers for cross-origin POST. lengths differ, expected 2 got 5"
+FAIL event.request has the expected headers for cross-origin POST. promise_test: Unhandled rejection with value: object "Error: assert_array_equals: event.request has the expected headers for cross-origin POST. lengths differ, expected 2 got 4"
 PASS FetchEvent#request.body contains XHR request data (string)
 PASS FetchEvent#request.body contains XHR request data (blob)
 PASS FetchEvent#request.method is set to XHR method