Media Galleries: Prune uninteresting folders when scanning.

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/file_util.h"
 #include "base/logging.h"
 #include "base/mac/bundle_locations.h"
 #include "base/path_service.h"
@@ skipping 441 matching lines @@
         return false;
       cur = cur.Append(FILE_PATH_LITERAL("Managed Preferences"));
       char* login = getlogin();
       if (!login)
         return false;
       cur = cur.AppendASCII(login);
       if (!base::PathExists(cur))  // We don't want to create this.
         return false;
       break;
     }
+    case chrome::DIR_USER_LIBRARY {
+      if (!GetLocalLibraryDirectory(&cur))

[Mark Mentovai, 2014/02/25 14:02:50]

The comment in the .h said that you wanted the user’s ~/Library, but this gets
the root /Library.

+        return false;
+      if (!base::PathExists(cur))  // We don't want to create this.
+        return false;
+      break;
+    }
 #endif
 #if defined(OS_CHROMEOS) || (defined(OS_MACOSX) && !defined(OS_IOS))
     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
 #if defined(OS_LINUX)
@@ skipping 73 matching lines @@
   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