Add CHECK in case PathProviderAndroid is failing.

base/base_paths_android.cc

Index: base/base_paths_android.cc
diff --git a/base/base_paths_android.cc b/base/base_paths_android.cc
index ca58179b6782fdc4d9ec283df14b4d97b5e688dc..4ba1e1ab85f997a145c4984361db5566305a31e1 100644
--- a/base/base_paths_android.cc
+++ b/base/base_paths_android.cc
@@ -24,7 +24,11 @@ bool PathProviderAndroid(int key, FilePath* result) {
       char bin_dir[PATH_MAX + 1];
       int bin_dir_size = readlink(kProcSelfExe, bin_dir, PATH_MAX);
       if (bin_dir_size < 0 || bin_dir_size > PATH_MAX) {
-        NOTREACHED() << "Unable to resolve " << kProcSelfExe << ".";
+        // TODO(falken): This CHECK is for debugging crbug.com/600226.
+        // Revert to NOTREACHED when the cause of the bug is understood.
+        CHECK(false) << "Unable to resolve " << kProcSelfExe
+                     << ". bin_dir_size=" << bin_dir_size
+                     << ", errno=" << errno;

[Torne, 2016/04/25 09:41:54]

Use PCHECK here instead of including errno this way - it interprets errno and
gives you a meaningful string, making the log easier to understand (also works
on windows by using GetLastError, just for reference).

I know this is just a temporary debugging thing, but it's a good general habit
;)

[falken, 2016/04/25 12:52:58]

Ah, didn't know that existed. Done. I also removed the unreachable return
statement.

         return false;
       }
       bin_dir[bin_dir_size] = 0;