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

Side by Side Diff: chrome/browser/extensions/api/sync_file_system/sync_file_system_api_helpers.h

Issue 2777063003: Stop passing raw pointers to base::Value API in c/b/chromeos and c/b/extensions (Closed)
Patch Set: Just rebased Created 3 years, 8 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
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 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_BROWSER_EXTENSIONS_API_SYNC_FILE_SYSTEM_SYNC_FILE_SYSTEM_API_HELP ERS_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_API_SYNC_FILE_SYSTEM_SYNC_FILE_SYSTEM_API_HELP ERS_H_
6 #define CHROME_BROWSER_EXTENSIONS_API_SYNC_FILE_SYSTEM_SYNC_FILE_SYSTEM_API_HELP ERS_H_ 6 #define CHROME_BROWSER_EXTENSIONS_API_SYNC_FILE_SYSTEM_SYNC_FILE_SYSTEM_API_HELP ERS_H_
7 7
8 #include <memory>
9
8 #include "chrome/browser/sync_file_system/conflict_resolution_policy.h" 10 #include "chrome/browser/sync_file_system/conflict_resolution_policy.h"
9 #include "chrome/browser/sync_file_system/sync_action.h" 11 #include "chrome/browser/sync_file_system/sync_action.h"
10 #include "chrome/browser/sync_file_system/sync_direction.h" 12 #include "chrome/browser/sync_file_system/sync_direction.h"
11 #include "chrome/browser/sync_file_system/sync_file_status.h" 13 #include "chrome/browser/sync_file_system/sync_file_status.h"
12 #include "chrome/browser/sync_file_system/sync_file_type.h" 14 #include "chrome/browser/sync_file_system/sync_file_type.h"
13 #include "chrome/browser/sync_file_system/sync_service_state.h" 15 #include "chrome/browser/sync_file_system/sync_service_state.h"
14 #include "chrome/common/extensions/api/sync_file_system.h" 16 #include "chrome/common/extensions/api/sync_file_system.h"
15 17
16 namespace storage { 18 namespace storage {
17 class FileSystemURL; 19 class FileSystemURL;
(...skipping 27 matching lines...) Expand all
45 ExtensionEnumToConflictResolutionPolicy( 47 ExtensionEnumToConflictResolutionPolicy(
46 api::sync_file_system::ConflictResolutionPolicy); 48 api::sync_file_system::ConflictResolutionPolicy);
47 49
48 // Creates a dictionary for FileSystem Entry from given |url|. 50 // Creates a dictionary for FileSystem Entry from given |url|.
49 // This will create a dictionary which has 'fileSystemType', 'fileSystemName', 51 // This will create a dictionary which has 'fileSystemType', 'fileSystemName',
50 // 'rootUrl', 'filePath' and 'isDirectory' fields. 52 // 'rootUrl', 'filePath' and 'isDirectory' fields.
51 // The returned dictionary is supposed to be interpreted 53 // The returned dictionary is supposed to be interpreted
52 // in the renderer's customer binding to create a FileEntry object. 54 // in the renderer's customer binding to create a FileEntry object.
53 // This returns NULL if the given |url| is not valid or |file_type| is 55 // This returns NULL if the given |url| is not valid or |file_type| is
54 // SYNC_FILE_TYPE_UNKNOWN. 56 // SYNC_FILE_TYPE_UNKNOWN.
55 base::DictionaryValue* CreateDictionaryValueForFileSystemEntry( 57 std::unique_ptr<base::DictionaryValue> CreateDictionaryValueForFileSystemEntry(
56 const storage::FileSystemURL& url, 58 const storage::FileSystemURL& url,
57 sync_file_system::SyncFileType file_type); 59 sync_file_system::SyncFileType file_type);
58 60
59 } // namespace extensions 61 } // namespace extensions
60 62
61 #endif // CHROME_BROWSER_EXTENSIONS_API_SYNC_FILE_SYSTEM_SYNC_FILE_SYSTEM_API_H ELPERS_H_ 63 #endif // CHROME_BROWSER_EXTENSIONS_API_SYNC_FILE_SYSTEM_SYNC_FILE_SYSTEM_API_H ELPERS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698