Deflake SandboxedHttpCacheBrowserTests

We expected not to have a response message for CreateSimpleCache when with
invalid path settings, because the network service is killed.

The assumption is invalid. Because the mojo message ordering is not
guaranteed, it is possible that we see a response message in the browser
process. Remove the NOTREACHED statements.

Bug: 1310058
Change-Id: Ifc36b9e5da6e8e26d8a8f8f1800078e2dc450c04
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3552070
Auto-Submit: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Robert Sesek <rsesek@chromium.org>
Commit-Queue: Robert Sesek <rsesek@chromium.org>
Cr-Commit-Position: refs/heads/main@{#985969}

content/browser/net/sandboxed_http_cache_browsertest.cc

@@ -144,20 +144,13 @@ IN_PROC_BROWSER_TEST_F(SandboxedHttpCacheBrowserTest,
   network_service_test()->CreateSimpleCache(
       std::move(factory_remote), path,
       base::BindOnce([](mojo::PendingRemote<SimpleCache> cache) {
-        ADD_FAILURE() << "NOTREACHED";
+        EXPECT_FALSE(cache.is_valid());
       }));
   run_loop.Run();
 }
 
-#if BUILDFLAG(IS_ANDROID)
-#define MAYBE_CreateSimpleCacheWithParentDirectory \
-  DISABLED_CreateSimpleCacheWithParentDirectory
-#else
-#define MAYBE_CreateSimpleCacheWithParentDirectory \
-  CreateSimpleCacheWithParentDirectory
-#endif  // BUILDFLAG(IS_ANDROID)
 IN_PROC_BROWSER_TEST_F(SandboxedHttpCacheBrowserTest,
-                       MAYBE_CreateSimpleCacheWithParentDirectory) {
+                       CreateSimpleCacheWithParentDirectoryTraversal) {
   base::RunLoop run_loop;
 
   const base::FilePath root_path = GetTempDirPath();
@@ -175,7 +168,7 @@ IN_PROC_BROWSER_TEST_F(SandboxedHttpCacheBrowserTest,
   network_service_test()->CreateSimpleCache(
       std::move(factory_remote), path,
       base::BindOnce([](mojo::PendingRemote<SimpleCache> cache) {
-        ADD_FAILURE() << "NOTREACHED";
+        EXPECT_FALSE(cache.is_valid());
       }));
   run_loop.Run();
 }