[Android] Do not warm up access to the profile directory in isolated services.

chrome/common/chrome_paths_android.cc

Index: chrome/common/chrome_paths_android.cc
diff --git a/chrome/common/chrome_paths_android.cc b/chrome/common/chrome_paths_android.cc
index 1cee68a53904e67157d59aa3d7d814e489875742..edbac864540f8d8f7f730e6ad992a91e9210a0e3 100644
--- a/chrome/common/chrome_paths_android.cc
+++ b/chrome/common/chrome_paths_android.cc
@@ -7,6 +7,7 @@
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/path_service.h"
+#include "content/public/common/content_switches.h"
 
 namespace chrome {
 
@@ -50,6 +51,10 @@ bool GetUserVideosDirectory(base::FilePath* result) {
 }
 
 bool ProcessNeedsProfileDir(const std::string& process_type) {
+  // SELinux prohibits accessing the data directory for isolated services.
+  if (process_type == switches::kRendererProcess)

[klobag.chromium, 2014/10/17 17:41:14]

How about the other child process type?

Should we check empty string for process_type to ensure it only returns true for
browser process?

https://code.google.com/p/chromium/codesearch#chromium/src/chrome/app/chrome_...

[Robert Sesek, 2014/10/17 18:00:32]

I wanted to keep the change scoped small, since I wasn't sure what other process
types Chrome used. Would you prefer to only return true if process_type.empty()
to match the CHECK? If so, I'll put together a CL to do that.

+    return false;
+
   return true;
 }