Add external extensions json source in proper mac location.

chrome/common/chrome_paths.cc

 // Copyright (c) 2011 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/command_line.h"
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/path_service.h"
@@ skipping 309 matching lines @@
 #if defined(OS_CHROMEOS)
     case chrome::DIR_USER_EXTERNAL_EXTENSIONS: {
       if (!PathService::Get(chrome::DIR_USER_DATA, &cur))
         return false;
       cur = cur.Append(FILE_PATH_LITERAL("External Extensions"));
       break;
     }
 #endif
     case chrome::DIR_EXTERNAL_EXTENSIONS:
 #if defined(OS_MACOSX)
-      if (!PathService::Get(base::DIR_EXE, &cur))
-        return false;
-
-      // On Mac, built-in extensions are in Contents/Extensions, a sibling of
-      // the App dir. If there are none, it may not exist.
-      // TODO(skerner): Reading external extensions from a file inside the
-      // app budle causes several problems.  Change this path to be outside
-      // the app bundle.  crbug/67203
-      cur = cur.DirName();
-      cur = cur.Append(FILE_PATH_LITERAL("Extensions"));
+      cur = FilePath(
+          FILE_PATH_LITERAL("/Library/Application Support/Google/Chrome"));

[TVL, 2011/08/30 15:32:40]

This should probably use
base::mac::GetLocalDirectory(NSApplicationSupportDirectory, ...) (in
foundation_utils) and then append "Google" and "Chrome" to that.

[Sam Kerner (Chrome), 2011/09/16 18:12:59]

Done.

+      cur = cur.Append("External Extensions");
       create_dir = false;
 #else
       if (!PathService::Get(base::DIR_MODULE, &cur))
         return false;
 
       cur = cur.Append(FILE_PATH_LITERAL("extensions"));
       create_dir = true;
 #endif
       break;
+
+#if defined(OS_MACOSX)
+    case DIR_DEPRICATED_EXTERNAL_EXTENSIONS:
+      // TODO(skerner): Reading external extensions from a file inside the
+      // app budle causes several problems.  Once users have a chance to
+      // migrate, remove this path.  crbug/67203
+      if (!PathService::Get(base::DIR_EXE, &cur))
+        return false;
+
+      cur = cur.DirName();
+      cur = cur.Append(FILE_PATH_LITERAL("Extensions"));
+      create_dir = false;
+
+      break;
+#endif
+
     default:
       return false;
   }
 
   if (create_dir && !file_util::PathExists(cur) &&
       !file_util::CreateDirectory(cur))
     return false;
 
   *result = cur;
   return true;
 }
 
 // This cannot be done as a static initializer sadly since Visual Studio will
 // eliminate this object file if there is no direct entry point into it.
 void RegisterPathProvider() {
   PathService::RegisterProvider(PathProvider, PATH_START, PATH_END);
 }
 
 }  // namespace chrome