Make downloaded WebAPK world readable

chrome/browser/android/webapk/webapk_installer.cc

 // Copyright 2016 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 "chrome/browser/android/webapk/webapk_installer.h"
 
 #include "base/android/build_info.h"
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/android/path_utils.h"
@@ skipping 66 matching lines @@
     return "";
 
   SkColor sk_color = reinterpret_cast<uint32_t&>(color);
   int r = SkColorGetR(sk_color);
   int g = SkColorGetG(sk_color);
   int b = SkColorGetB(sk_color);
   double a = SkColorGetA(sk_color) / 255.0;
   return base::StringPrintf("rgba(%d,%d,%d,%.2f)", r, g, b, a);
 }
 
+// Returns task runner for running background tasks.
+scoped_refptr<base::TaskRunner> GetBackgroundTaskRunner() {
+  return content::BrowserThread::GetBlockingPool()
+      ->GetTaskRunnerWithShutdownBehavior(
+          base::SequencedWorkerPool::SKIP_ON_SHUTDOWN);
+}
+
 }  // anonymous namespace
 
 WebApkInstaller::WebApkInstaller(const ShortcutInfo& shortcut_info,
                                  const SkBitmap& shortcut_icon)
     : shortcut_info_(shortcut_info),
       shortcut_icon_(shortcut_icon),
       webapk_download_url_timeout_ms_(kWebApkDownloadUrlTimeoutMs),
       download_timeout_ms_(kDownloadTimeoutMs),
       weak_ptr_factory_(this) {
   base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
@@ skipping 13 matching lines @@
 }
 
 void WebApkInstaller::InstallAsyncWithURLRequestContextGetter(
     net::URLRequestContextGetter* request_context_getter,
     const FinishCallback& finish_callback) {
   request_context_getter_ = request_context_getter;
   finish_callback_ = finish_callback;
 
   // base::Unretained() is safe because WebApkInstaller owns itself and does not
   // start the timeout timer till after SendCreateWebApkRequest() is called.
-  scoped_refptr<base::TaskRunner> background_task_runner =
-      content::BrowserThread::GetBlockingPool()
-          ->GetTaskRunnerWithShutdownBehavior(
-              base::SequencedWorkerPool::SKIP_ON_SHUTDOWN);
   base::PostTaskAndReplyWithResult(
-      background_task_runner.get(), FROM_HERE,
+      GetBackgroundTaskRunner().get(), FROM_HERE,
       base::Bind(&WebApkInstaller::BuildWebApkProtoInBackground,
                  base::Unretained(this)),
       base::Bind(&WebApkInstaller::SendCreateWebApkRequest,
                  base::Unretained(this)));
 }
 
 void WebApkInstaller::SetTimeoutMs(int timeout_ms) {
   webapk_download_url_timeout_ms_ = timeout_ms;
   download_timeout_ms_ = timeout_ms;
 }
@@ skipping 71 matching lines @@
 void WebApkInstaller::OnWebApkDownloaded(const base::FilePath& file_path,
                                          const std::string& package_name,
                                          FileDownloader::Result result) {
   timer_.Stop();
 
   if (result != FileDownloader::DOWNLOADED) {
     OnFailure();
     return;
   }
 
+  int posix_permissions = base::FILE_PERMISSION_READ_BY_USER |
+                          base::FILE_PERMISSION_WRITE_BY_USER |
+                          base::FILE_PERMISSION_READ_BY_GROUP |
+                          base::FILE_PERMISSION_READ_BY_OTHERS;
+  base::PostTaskAndReplyWithResult(
+      GetBackgroundTaskRunner().get(), FROM_HERE,
+      base::Bind(&base::SetPosixFilePermissions, file_path, posix_permissions),
+      base::Bind(&WebApkInstaller::OnWebApkMadeWorldReadable,
+                 weak_ptr_factory_.GetWeakPtr(), file_path, package_name));
+}
+
+void WebApkInstaller::OnWebApkMadeWorldReadable(
+    const base::FilePath& file_path,
+    const std::string& package_name,
+    bool change_permission_success) {
+  if (!change_permission_success) {
+    OnFailure();
+    return;
+  }
+
   JNIEnv* env = base::android::AttachCurrentThread();
   base::android::ScopedJavaLocalRef<jstring> java_file_path =
       base::android::ConvertUTF8ToJavaString(env, file_path.value());
   base::android::ScopedJavaLocalRef<jstring> java_package_name =
       base::android::ConvertUTF8ToJavaString(env, package_name);
   bool success =
       StartDownloadedWebApkInstall(env, java_file_path, java_package_name);
   if (success)
     OnSuccess();
   else
@@ skipping 44 matching lines @@
 
 void WebApkInstaller::OnSuccess() {
   finish_callback_.Run(true);
   delete this;
 }
 
 void WebApkInstaller::OnFailure() {
   finish_callback_.Run(false);
   delete this;
 }