Media Galleries: Always call MediaFileSystemRegistry::GetMediaFileSystemsForExtension() before show…

chrome/browser/extensions/api/media_galleries/media_galleries_api.h

 // 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.
 
 // Defines the Chrome Extensions Media Galleries API functions for accessing
 // user's media files, as specified in the extension API IDL.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_MEDIA_GALLERIES_MEDIA_GALLERIES_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_MEDIA_GALLERIES_MEDIA_GALLERIES_API_H_
 
 #include <vector>
 
 #include "chrome/browser/extensions/extension_function.h"
 #include "chrome/browser/media_gallery/media_file_system_registry.h"
 
 namespace extensions {
 
 class MediaGalleriesGetMediaFileSystemsFunction
     : public AsyncExtensionFunction {
  public:
   DECLARE_EXTENSION_FUNCTION("mediaGalleries.getMediaFileSystems",
                              MEDIAGALLERIES_GETMEDIAFILESYSTEMS)
 
  protected:
   virtual ~MediaGalleriesGetMediaFileSystemsFunction();
   virtual bool RunImpl() OVERRIDE;
 
  private:
+  // Always show the dialog.
+  void AlwaysShowDialog(

[vandebo (ex-Chrome), 2013/01/30 17:47:30]

nit: ShowDialog(

[Lei Zhang, 2013/01/30 21:00:47]

Name's already taken on line 46.

+      const std::vector<chrome::MediaFileSystemInfo>& filesystems);
+
   // If no galleries are found, show the dialog, otherwise return them.
   void ShowDialogIfNoGalleries(
       const std::vector<chrome::MediaFileSystemInfo>& filesystems);
 
   // Grabs galleries from the media file system registry and passes them to
   // |ReturnGalleries|.
   void GetAndReturnGalleries();
 
   // Returns galleries to the caller.
   void ReturnGalleries(
@@ skipping 10 matching lines @@
       EXPERIMENTAL_MEDIAGALLERIES_ASSEMBLEMEDIAFILE)
 
  protected:
   virtual ~MediaGalleriesAssembleMediaFileFunction();
   virtual bool RunImpl() OVERRIDE;
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_MEDIA_GALLERIES_MEDIA_GALLERIES_API_H_