logPrivate API: prevent log dump from trying to use a non-native path

chrome/browser/extensions/api/log_private/log_private_api_chromeos.cc

 // Copyright 2013 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/extensions/api/log_private/log_private_api.h"
 
 #include <memory>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 21 matching lines @@
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/render_process_host.h"
 #include "extensions/browser/api/file_handlers/app_file_handler_util.h"
 #include "extensions/browser/event_router.h"
 #include "extensions/browser/extension_function.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/browser/granted_file_entry.h"
 #include "net/log/net_log_entry.h"
 
 #if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/file_manager/filesystem_api_util.h"
 #include "chrome/browser/chromeos/system_logs/debug_log_writer.h"
 #endif
 
 using content::BrowserThread;
 
 namespace events {
 const char kOnCapturedEvents[] = "logPrivate.onCapturedEvents";
 }  // namespace events
 
 namespace extensions {
@@ skipping 52 matching lines @@
 // logPrivate.startEventRecorder() calls - /home/chronos/user/log/apps
 base::FilePath GetAppLogDirectory() {
   return logging::GetSessionLogDir(*base::CommandLine::ForCurrentProcess())
       .Append(kAppLogsSubdir);
 }
 
 // Returns directory location where logs dumps initiated with chrome.dumpLogs
 // will be stored - /home/chronos/<user_profile_dir>/Downloads/log_dumps
 base::FilePath GetLogDumpDirectory(content::BrowserContext* context) {
   const DownloadPrefs* const prefs = DownloadPrefs::FromBrowserContext(context);
-  return prefs->DownloadPath().Append(kLogDumpsSubdir);
+  base::FilePath path = prefs->DownloadPath();
+
+#if defined(OS_CHROMEOS)
+  Profile* profile = Profile::FromBrowserContext(context);
+  if (file_manager::util::IsUnderNonNativeLocalPath(profile, path))
+    path = prefs->GetDefaultDownloadDirectoryForProfile();
+#endif
+
+  return path.Append(kLogDumpsSubdir);
 }
 
 // Removes direcotry content of |logs_dumps| and |app_logs_dir| (only for the
 // primary profile).
 void CleanUpLeftoverLogs(bool is_primary_profile,
                          const base::FilePath& app_logs_dir,
                          const base::FilePath& logs_dumps) {
   LOG(WARNING) << "Deleting " << app_logs_dir.value();
   LOG(WARNING) << "Deleting " << logs_dumps.value();
 
@@ skipping 421 matching lines @@
   entry->SetBoolean("isDirectory", false);
   auto entry_list = base::MakeUnique<base::ListValue>();
   entry_list->Append(std::move(entry));
   response->Set("entries", std::move(entry_list));
   response->SetBoolean("multiple", false);
   SetResult(std::move(response));
   SendResponse(succeeded);
 }
 
 }  // namespace extensions