Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(67)

Side by Side Diff: chrome/browser/chromeos/extensions/file_manager/open_with_browser.h

Issue 23463007: file_manager: Fix a bug where hosted documents could not be opened without active browser window. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // This file provides utilities for opening files with the browser. 5 // This file provides utilities for opening files with the browser.
6 6
7 #ifndef CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_OPEN_WITH_BROWSER_H_ 7 #ifndef CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_OPEN_WITH_BROWSER_H_
8 #define CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_OPEN_WITH_BROWSER_H_ 8 #define CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_OPEN_WITH_BROWSER_H_
9 9
10 #include "base/files/file_path.h" 10 #include "base/files/file_path.h"
11 11
12 class Browser; 12 class Browser;
13 class Profile; 13 class Profile;
14 14
15 namespace file_manager { 15 namespace file_manager {
16 namespace util { 16 namespace util {
17 17
18 // Opens the file specified by |file_path| with the browser. This function takes 18 // Opens the file specified by |file_path| with the browser for
19 // care of the following intricacies: 19 // |profile|. This function takes care of the following intricacies:
20 // 20 //
21 // - If there is no active browser window, open it.
21 // - If the file is a Drive hosted document, the hosted document will be 22 // - If the file is a Drive hosted document, the hosted document will be
22 // opened in the browser by extracting the right URL for the file. 23 // opened in the browser by extracting the right URL for the file.
23 // - If the file is a CRX file, the CRX file will be installed. 24 // - If the file is a CRX file, the CRX file will be installed.
24 // - If the file is on Drive, the file will be downloaded from Drive as 25 // - If the file is on Drive, the file will be downloaded from Drive as
25 // needed. 26 // needed.
26 // 27 //
27 // Returns false if failed to open. This happens if the file type is unknown. 28 // Returns false if failed to open. This happens if the file type is unknown.
28 bool OpenFileWithBrowser(Browser* browser, const base::FilePath& file_path); 29 bool OpenFileWithBrowser(Profile* profile, const base::FilePath& file_path);
29 30
30 // Checks whether a pepper plugin for |file_extension| is enabled. 31 // Checks whether a pepper plugin for |file_extension| is enabled.
31 bool ShouldBeOpenedWithPlugin( 32 bool ShouldBeOpenedWithPlugin(
32 Profile* profile, 33 Profile* profile,
33 const base::FilePath::StringType& file_extension); 34 const base::FilePath::StringType& file_extension);
34 35
35 } // namespace util 36 } // namespace util
36 } // namespace file_manager 37 } // namespace file_manager
37 38
38 #endif // CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_OPEN_WITH_BROWSER_H_ 39 #endif // CHROME_BROWSER_CHROMEOS_EXTENSIONS_FILE_MANAGER_OPEN_WITH_BROWSER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698