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/common/extensions/api/file_browser_handlers/file_browser_handler.h

Issue 423483006: Cleanup return values in chrome/common/extensions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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
« no previous file with comments | « no previous file | chrome/common/extensions/manifest_handlers/mime_types_handler.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 #ifndef CHROME_COMMON_EXTENSIONS_API_FILE_BROWSER_HANDLERS_FILE_BROWSER_HANDLER_ H_ 5 #ifndef CHROME_COMMON_EXTENSIONS_API_FILE_BROWSER_HANDLERS_FILE_BROWSER_HANDLER_ H_
6 #define CHROME_COMMON_EXTENSIONS_API_FILE_BROWSER_HANDLERS_FILE_BROWSER_HANDLER_ H_ 6 #define CHROME_COMMON_EXTENSIONS_API_FILE_BROWSER_HANDLERS_FILE_BROWSER_HANDLER_ H_
7 7
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 30 matching lines...) Expand all
41 41
42 // File schema URL patterns. 42 // File schema URL patterns.
43 const extensions::URLPatternSet& file_url_patterns() const { 43 const extensions::URLPatternSet& file_url_patterns() const {
44 return url_set_; 44 return url_set_;
45 } 45 }
46 void AddPattern(const URLPattern& pattern); 46 void AddPattern(const URLPattern& pattern);
47 bool MatchesURL(const GURL& url) const; 47 bool MatchesURL(const GURL& url) const;
48 void ClearPatterns(); 48 void ClearPatterns();
49 49
50 // Action icon path. 50 // Action icon path.
51 const std::string icon_path() const { return default_icon_path_; } 51 const std::string& icon_path() const { return default_icon_path_; }
52 void set_icon_path(const std::string& path) { 52 void set_icon_path(const std::string& path) {
53 default_icon_path_ = path; 53 default_icon_path_ = path;
54 } 54 }
55 55
56 // File access permissions. 56 // File access permissions.
57 // Adjusts file_access_permission_flags_ to allow specified permission. 57 // Adjusts file_access_permission_flags_ to allow specified permission.
58 bool AddFileAccessPermission(const std::string& permission_str); 58 bool AddFileAccessPermission(const std::string& permission_str);
59 // Checks that specified file access permissions are valid (all set 59 // Checks that specified file access permissions are valid (all set
60 // permissions are valid and there is no other permission specified with 60 // permissions are valid and there is no other permission specified with
61 // "create") 61 // "create")
(...skipping 30 matching lines...) Expand all
92 virtual ~FileBrowserHandlerParser(); 92 virtual ~FileBrowserHandlerParser();
93 93
94 virtual bool Parse(extensions::Extension* extension, 94 virtual bool Parse(extensions::Extension* extension,
95 base::string16* error) OVERRIDE; 95 base::string16* error) OVERRIDE;
96 96
97 private: 97 private:
98 virtual const std::vector<std::string> Keys() const OVERRIDE; 98 virtual const std::vector<std::string> Keys() const OVERRIDE;
99 }; 99 };
100 100
101 #endif // CHROME_COMMON_EXTENSIONS_API_FILE_BROWSER_HANDLERS_FILE_BROWSER_HANDL ER_H_ 101 #endif // CHROME_COMMON_EXTENSIONS_API_FILE_BROWSER_HANDLERS_FILE_BROWSER_HANDL ER_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/common/extensions/manifest_handlers/mime_types_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698