Use /etc/hosts for URLRequestTest.FileTest on Android

net/url_request/url_request_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <utility>
 
 #include "build/build_config.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
@@ skipping 814 matching lines @@
     EXPECT_EQ(0, r->GetSocketAddress().port());
 
     HttpRequestHeaders headers;
     EXPECT_FALSE(r->GetFullRequestHeaders(&headers));
   }
 }
 
 #if !defined(DISABLE_FILE_SUPPORT)
 TEST_F(URLRequestTest, FileTest) {
   base::FilePath app_path;
+
+#if defined(OS_ANDROID)
+  // Android devices are not guaranteed to be able to read /proc/self/exe
+  // Use /etc/hosts instead
+  app_path = base::FilePath("/etc/hosts");
+#else
+  PathService::Get(base::FILE_EXE, &app_path);
+#endif  // OS_ANDROID
+
   PathService::Get(base::FILE_EXE, &app_path);

[pauljensen, 2016/04/07 17:18:08]

this won't work...you've still got the line here

[kraush, 2016/04/07 17:21:31]

Argh, sorry.
I did run it and it did work, but that's only because PathService::Get fails and
never writes anything into &app_path

I'll remove the second invication to make it cleaner.

   GURL app_url = FilePathToFileURL(app_path);
 
   TestDelegate d;
   {
     scoped_ptr<URLRequest> r(
         default_context_.CreateRequest(app_url, DEFAULT_PRIORITY, &d));
 
     r->Start();
     EXPECT_TRUE(r->is_pending());
 
@@ skipping 9248 matching lines @@
   AddTestInterceptor()->set_main_intercept_job(std::move(job));
 
   req->Start();
   req->Cancel();
   base::RunLoop().RunUntilIdle();
   EXPECT_EQ(URLRequestStatus::CANCELED, req->status().status());
   EXPECT_EQ(0, d.received_redirect_count());
 }
 
 }  // namespace net