Log into the user-data-dir instead of Chrome's bundle.

chrome/common/chrome_paths.cc

 // Copyright (c) 2012 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/common/chrome_paths.h"
 
 #include "base/files/file_util.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "base/path_service.h"
@@ skipping 122 matching lines @@
 }  // namespace
 
 namespace chrome {
 
 bool PathProvider(int key, base::FilePath* result) {
   // Some keys are just aliases...
   switch (key) {
     case chrome::DIR_APP:
       return PathService::Get(base::DIR_MODULE, result);
     case chrome::DIR_LOGS:
 #ifdef NDEBUG

[Nico, 2017/02/10 21:36:18]

i wondered why we even make this differentiation and found (internal only,
sorry) https://b/853431 /
https://chrome-internal.googlesource.com/chrome/chrome-history/+/f4d4aae13fc9...

It sounds like we could probably just always use the user data dir

       // Release builds write to the data dir
       return PathService::Get(chrome::DIR_USER_DATA, result);
 #else
       // Debug builds write next to the binary (in the build tree)
 #if defined(OS_MACOSX)
-      if (!PathService::Get(base::DIR_EXE, result))
-        return false;
+      // Apps may not write into their own bundle.
       if (base::mac::AmIBundled()) {
-        // If we're called from chrome, dump it beside the app (outside the
-        // app bundle), if we're called from a unittest, we'll already
-        // outside the bundle so use the exe dir.
-        // exe_dir gave us .../Chromium.app/Contents/MacOS/Chromium.
-        *result = result->DirName();
-        *result = result->DirName();
-        *result = result->DirName();
+        return PathService::Get(chrome::DIR_USER_DATA, result);
       }
-      return true;
+      return PathService::Get(base::DIR_EXE, result);
 #else
       return PathService::Get(base::DIR_EXE, result);
 #endif  // defined(OS_MACOSX)
 #endif  // NDEBUG
     case chrome::FILE_RESOURCE_MODULE:
       return PathService::Get(base::FILE_MODULE, result);
   }
 
   // Assume that we will not need to create the directory if it does not exist.
   // This flag can be set to true for the cases where we want to create it.
@@ skipping 429 matching lines @@
 
 void SetInvalidSpecifiedUserDataDir(const base::FilePath& user_data_dir) {
   g_invalid_specified_user_data_dir.Get() = user_data_dir;
 }
 
 const base::FilePath& GetInvalidSpecifiedUserDataDir() {
   return g_invalid_specified_user_data_dir.Get();
 }
 
 }  // namespace chrome