Revert "Adapt isolated-code-cache web tests to partitioned HTTP Cache"

This reverts commit 3a9215b3.

Reason for revert:
virtual/split-http-cache-not-site-per-process/http/tests/devtools/isolated-code-cache/same-origin-test.js
virtual/split-http-cache-not-site-per-process/http/tests/devtools/isolated-code-cache/same-origin-module-test.js are flaky on WebKit Linux MSAN.
https://ci.chromium.org/p/chromium/builders/ci/WebKit%20Linux%20MSAN

Original change's description:
> Adapt isolated-code-cache web tests to partitioned HTTP Cache
> 
> The not-site-per-process isolated-code-cache web tests currently assume
> that cross-origin resources are cached when code cache site isolation is
> not active. This causes them to fail when the HTTP Cache is partitioned
> by Network Isolation Key. We thus disable the partitioned HTTP Cache for
> those tests and add a new test for when site isolation is not active,
> but the HTTP Cache is still partitioned.
> 
> Bug: 1009628
> Change-Id: I0d5d438d9564d3f659b93484a5bcc8aa56d2cd26
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1865421
> Reviewed-by: Shivani Sharma <shivanisha@chromium.org>
> Reviewed-by: Nico Weber <thakis@chromium.org>
> Commit-Queue: Alex Turner <alexmt@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#707360}

TBR=thakis@chromium.org,shivanisha@chromium.org,alexmt@chromium.org

Change-Id: I8ebebb654e02d4d3c5d0c7e946ea70710b0b5df7
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1009628
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1869477Reviewed-by: Lan Wei <lanwei@chromium.org>
Commit-Queue: Lan Wei <lanwei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#707457}

third_party/blink/web_tests/VirtualTestSuites

@@ -746,17 +746,10 @@
              "--disable-features=WebAssemblyBaseline",
              "--site-per-process"]
   },
-  {
-    "prefix": "split-http-cache-not-site-per-process",
-    "base": "http/tests/devtools/isolated-code-cache",
-    "args": ["--enable-features=SplitCacheByNetworkIsolationKey",
-             "--disable-site-isolation-trials"]
-  },
   {
     "prefix": "not-site-per-process",
     "base": "http/tests/devtools/isolated-code-cache",
-    "args": ["--disable-site-isolation-trials",
-             "--disable-features=SplitCacheByNetworkIsolationKey"]
+    "args": ["--disable-site-isolation-trials"]
   },
   {
     "prefix": "not-site-per-process",

third_party/blink/web_tests/virtual/not-site-per-process/README.md

@@ -15,11 +15,7 @@ behave in presence of out-of-process iframes.
 Tests under `virtual/not-site-per-process` are run with
 `--disable-site-isolation-trials` cmdline flag which turns off site
 isolation.  This is needed to preserve test coverage provided by around
-60 tests that fail when run with site isolation. `isolated-code-cache` tests are
-also run with `--disable-features=SplitCacheByNetworkIsolationKey` which turns
-off HTTP cache partitioning. This is needed as a test expects cross-origin
-resources to be cached. Equivalent tests with the feature enabled can be found
-under `virtual/split-http-cache-not-site-per-process`.
+60 tests that fail when run with site isolation.
 
 When modifying the list of files that behave differently with and without
 OOPIFs, please consider modifying all the locations below:

third_party/blink/web_tests/virtual/split-http-cache-not-site-per-process/http/tests/devtools/isolated-code-cache/README.txt

-# This suite runs the tests in http/tests/devtools/isolated-code-cache with
-# --enable-features=SplitCacheByNetworkIsolationKey
-# --disable-site-isolation-trials
-# This feature partitions the HTTP cache by network isolation key for improved
-# security. Tracking bug: crbug.com/910708. This test disables site isolation,
-# which would cause similar results.

third_party/blink/web_tests/virtual/split-http-cache-not-site-per-process/http/tests/devtools/isolated-code-cache/cross-origin-test-expected.txt

-Tests V8 code cache for javascript resources
-
----First navigation - produce and consume code cache ------
-
-v8.compile Properties:
-{
-    data : {
-        columnNumber : 0
-        lineNumber : 0
-        notStreamedReason : "script too small"
-        streamed : <boolean>
-        url : .../devtools/resources/v8-cache-script.js
-    }
-    endTime : <number>
-    startTime : <number>
-    type : "v8.compile"
-}
-Text details for v8.compile: v8-cache-script.js:1
-v8.compile Properties:
-{
-    data : {
-        columnNumber : 0
-        lineNumber : 0
-        notStreamedReason : "already used streamed data"
-        streamed : <boolean>
-        url : .../devtools/resources/v8-cache-script.js
-    }
-    endTime : <number>
-    startTime : <number>
-    type : "v8.compile"
-}
-Text details for v8.compile: v8-cache-script.js:1
-v8.compile Properties:
-{
-    data : {
-        columnNumber : 0
-        lineNumber : 0
-        notStreamedReason : "already used streamed data"
-        producedCacheSize : <number>
-        streamed : <boolean>
-        url : .../devtools/resources/v8-cache-script.js
-    }
-    endTime : <number>
-    startTime : <number>
-    type : "v8.compile"
-}
-Text details for v8.compile: v8-cache-script.js:1
-v8.compile Properties:
-{
-    data : {
-        cacheConsumeOptions : "code"
-        cacheRejected : false
-        columnNumber : 0
-        consumedCacheSize : <number>
-        lineNumber : 0
-        notStreamedReason : "already used streamed data"
-        streamed : <boolean>
-        url : .../devtools/resources/v8-cache-script.js
-    }
-    endTime : <number>
-    startTime : <number>
-    type : "v8.compile"
-}
-Text details for v8.compile: v8-cache-script.js:1
-
---- Second navigation - from a different origin ------
-
-v8.compile Properties:
-{
-    data : {
-        columnNumber : 0
-        lineNumber : 0
-        notStreamedReason : "script too small"
-        streamed : <boolean>
-        url : .../devtools/resources/v8-cache-script.js
-    }
-    endTime : <number>
-    startTime : <number>
-    type : "v8.compile"
-}
-Text details for v8.compile: v8-cache-script.js:1
-