iTunes file util and data provider for media galleries

chrome/browser/media_galleries/imported_media_gallery_registry.h

 // 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.
 
 #ifndef CHROME_BROWSER_MEDIA_GALLERIES_IMPORTED_MEDIA_GALLERY_REGISTRY_H_
 #define CHROME_BROWSER_MEDIA_GALLERIES_IMPORTED_MEDIA_GALLERY_REGISTRY_H_
 
+#include <set>
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/files/file_path.h"
 #include "base/lazy_instance.h"
 #include "base/memory/scoped_ptr.h"
 
+namespace itunes {
+class ITunesDataProvider;
+}
+
 namespace picasa {
 class PicasaDataProvider;
 }
 
 namespace chrome {
 
 // This class lives on the MediaTaskRunner thread. It has some static
 // methods which are called on the UI thread.
 //
 // MediaTaskRunner is not guaranteed to be one thread, but it is guaranteed
 // to be a series of sequential calls. See SequencedTaskRunner for details.
 class ImportedMediaGalleryRegistry {
  public:
   static ImportedMediaGalleryRegistry* GetInstance();
 
   // Should be called on the UI thread only.
-  static std::string RegisterPicasaFilesystemOnUIThread(
+  std::string RegisterPicasaFilesystemOnUIThread(
       const base::FilePath& database_path);
-  static bool RevokePicasaFilesystemOnUIThread(const std::string& fsid);
+
+  std::string RegisterITunesFilesystemOnUIThread(
+      const base::FilePath& xml_library_path);
+
+  bool RevokeImportedFilesystemOnUIThread(const std::string& fsid);
 
   // Should be called on the MediaTaskRunner thread only.
   static picasa::PicasaDataProvider* picasa_data_provider();
 
+  static itunes::ITunesDataProvider* itunes_data_provider();
+
  private:
   friend struct base::DefaultLazyInstanceTraits<ImportedMediaGalleryRegistry>;
 
   ImportedMediaGalleryRegistry();
   virtual ~ImportedMediaGalleryRegistry();
 
   void RegisterPicasaFileSystem(const base::FilePath& database_path);
   void RevokePicasaFileSystem();
 
+  void RegisterITunesFileSystem(const base::FilePath& xml_library_path);
+  void RevokeITunesFileSystem();
+
   scoped_ptr<picasa::PicasaDataProvider> picasa_data_provider_;

[Lei Zhang, 2013/06/06 03:48:47]

You should mention which variables are accessed on which thread.

[vandebo (ex-Chrome), 2013/06/06 19:49:18]

Done.

+  std::set<std::string> picasa_fsids_;
 
-  int picasa_filesystems_count_;
+  scoped_ptr<itunes::ITunesDataProvider> itunes_data_provider_;
+  std::set<std::string> itunes_fsids_;
+
+#ifndef NDEBUG
+  std::string picasa_database_path_;

[Lei Zhang, 2013/06/06 03:48:47]

These need to be FilePaths to compile x-platform.

[vandebo (ex-Chrome), 2013/06/06 19:49:18]

Done.

+  std::string itunes_xml_library_path_;
+#endif
 
   DISALLOW_COPY_AND_ASSIGN(ImportedMediaGalleryRegistry);
 };
 
 }  // namespace chrome
 
 #endif  // CHROME_BROWSER_MEDIA_GALLERIES_IMPORTED_MEDIA_GALLERY_REGISTRY_H_