Add more logs in ServicificationBackgroundService.

Adds more logs to investigate the failures on test-go-phone.

Bug: 1096833
Change-Id: Ic067a93ddb87c6afb314f415f69307fe31794394
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2300269Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Commit-Queue: Xi Han <hanxi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#788760}

chrome/browser/android/servicification_background_service_jni.cc

@@ -25,25 +25,34 @@ JNI_ServicificationBackgroundService_TestPersistentHistogramsOnDiskSystemProfile
     JNIEnv* env) {
   base::GlobalHistogramAllocator* allocator =
       base::GlobalHistogramAllocator::Get();
-  if (!allocator)
+  if (!allocator) {
+    LOG(ERROR) << "The GlobalHistogramAllocator is null!";
     return false;
+  }
 
   const base::FilePath& persistent_file_path =
       allocator->GetPersistentLocation();
-  if (persistent_file_path.empty())
+  if (persistent_file_path.empty()) {
+    LOG(ERROR) << "The path of the persistent file is empty!";
     return false;
+  }
 
   base::File file(persistent_file_path,
                   base::File::FLAG_OPEN | base::File::FLAG_READ);
-  if (!file.IsValid())
+  if (!file.IsValid()) {
+    LOG(ERROR) << "The persistent file isn't valid!";
     return false;
+  }
 
   std::unique_ptr<base::MemoryMappedFile> mapped(new base::MemoryMappedFile());
-  if (!mapped->Initialize(std::move(file), base::MemoryMappedFile::READ_ONLY))
+  if (!mapped->Initialize(std::move(file), base::MemoryMappedFile::READ_ONLY)) {
+    LOG(ERROR) << "Fails to Initialize the memory mapped file!";
     return false;
+  }
 
   if (!base::FilePersistentMemoryAllocator::IsFileAcceptable(
           *mapped, true /* read_only */)) {
+    LOG(ERROR) << "The memory mapped file isn't acceptable!";
     return false;
   }