Change the meaning of SelectFileDialog.support_drive: it means Drive-aware callers.

chrome/browser/chromeos/extensions/file_manager/private_api_util.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.
 //
 // This file provides utility functions for fileBrowserPrivate API.
 
 #ifndef CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_PRIVATE_API_UTIL_H_
 #define CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_PRIVATE_API_UTIL_H_
 
 #include "base/callback_forward.h"
@@ skipping 43 matching lines @@
 
 // The callback type is used for GetSelectedFileInfo().
 typedef base::Callback<void(const std::vector<ui::SelectedFileInfo>&)>
     GetSelectedFileInfoCallback;
 
 // Gets the information for |file_urls|.
 void GetSelectedFileInfo(content::RenderViewHost* render_view_host,
                          Profile* profile,
                          const std::vector<GURL>& file_urls,
                          bool for_opening,
+                         bool need_local_path,

[satorux1, 2013/08/06 02:07:17]

Maybe use a bitmask? Then, the callers would be clearer.

enum GetSelectedFileInfoOptions {
  OPTION_NONE = 0,
  OPTION_FOR_OPENING = 1,
  OPTION_NEED_LOCAL_PATH = 2,
}

[kinaba, 2013/08/06 06:20:56]

Done.
I changed to a simple enum rather than a bitmask,
since when |need_local_path==false| the other bit becomes irrelevant.

                          GetSelectedFileInfoCallback callback);
 
 }  // namespace util
 }  // namespace file_manager
 
 #endif  // CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_PRIVATE_API_UTIL_H_