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

Side by Side Diff: ppapi/thunk/ppb_file_system_thunk.cc

Issue 9113044: Convert to new enter method (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: PATCH DESCRIPTIONS ARE A STUPID WASTE OF TIME Created 8 years, 10 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
« no previous file with comments | « ppapi/thunk/ppb_file_ref_thunk.cc ('k') | ppapi/thunk/ppb_flash_clipboard_thunk.cc » ('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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "ppapi/c/pp_completion_callback.h" 5 #include "ppapi/c/pp_completion_callback.h"
6 #include "ppapi/c/pp_errors.h" 6 #include "ppapi/c/pp_errors.h"
7 #include "ppapi/c/ppb_file_system.h" 7 #include "ppapi/c/ppb_file_system.h"
8 #include "ppapi/thunk/common.h"
9 #include "ppapi/thunk/enter.h" 8 #include "ppapi/thunk/enter.h"
10 #include "ppapi/thunk/thunk.h" 9 #include "ppapi/thunk/thunk.h"
11 #include "ppapi/thunk/ppb_file_system_api.h" 10 #include "ppapi/thunk/ppb_file_system_api.h"
12 #include "ppapi/thunk/resource_creation_api.h" 11 #include "ppapi/thunk/resource_creation_api.h"
13 12
14 namespace ppapi { 13 namespace ppapi {
15 namespace thunk { 14 namespace thunk {
16 15
17 namespace { 16 namespace {
18 17
18 typedef EnterResource<PPB_FileSystem_API> EnterFileSystem;
19
19 PP_Resource Create(PP_Instance instance, PP_FileSystemType type) { 20 PP_Resource Create(PP_Instance instance, PP_FileSystemType type) {
20 EnterFunction<ResourceCreationAPI> enter(instance, true); 21 EnterResourceCreation enter(instance);
21 if (enter.failed()) 22 if (enter.failed())
22 return 0; 23 return 0;
23 return enter.functions()->CreateFileSystem(instance, type); 24 return enter.functions()->CreateFileSystem(instance, type);
24 } 25 }
25 26
26 PP_Bool IsFileSystem(PP_Resource resource) { 27 PP_Bool IsFileSystem(PP_Resource resource) {
27 EnterResource<PPB_FileSystem_API> enter(resource, false); 28 EnterFileSystem enter(resource, false);
28 return PP_FromBool(enter.succeeded()); 29 return PP_FromBool(enter.succeeded());
29 } 30 }
30 31
31 int32_t Open(PP_Resource file_system, 32 int32_t Open(PP_Resource file_system,
32 int64 expected_size, 33 int64 expected_size,
33 PP_CompletionCallback callback) { 34 PP_CompletionCallback callback) {
34 EnterResource<PPB_FileSystem_API> enter(file_system, true); 35 EnterFileSystem enter(file_system, callback, true);
35 if (enter.failed()) 36 if (enter.failed())
36 return MayForceCallback(callback, PP_ERROR_BADRESOURCE); 37 return enter.retval();
37 int32_t result = enter.object()->Open(expected_size, callback); 38 return enter.SetResult(enter.object()->Open(expected_size, callback));
38 return MayForceCallback(callback, result);
39 } 39 }
40 40
41 PP_FileSystemType GetType(PP_Resource file_system) { 41 PP_FileSystemType GetType(PP_Resource file_system) {
42 EnterResource<PPB_FileSystem_API> enter(file_system, true); 42 EnterFileSystem enter(file_system, true);
43 if (enter.failed()) 43 if (enter.failed())
44 return PP_FILESYSTEMTYPE_INVALID; 44 return PP_FILESYSTEMTYPE_INVALID;
45 return enter.object()->GetType(); 45 return enter.object()->GetType();
46 } 46 }
47 47
48 const PPB_FileSystem g_ppb_file_system_thunk = { 48 const PPB_FileSystem g_ppb_file_system_thunk = {
49 &Create, 49 &Create,
50 &IsFileSystem, 50 &IsFileSystem,
51 &Open, 51 &Open,
52 &GetType 52 &GetType
53 }; 53 };
54 54
55 } // namespace 55 } // namespace
56 56
57 const PPB_FileSystem_1_0* GetPPB_FileSystem_1_0_Thunk() { 57 const PPB_FileSystem_1_0* GetPPB_FileSystem_1_0_Thunk() {
58 return &g_ppb_file_system_thunk; 58 return &g_ppb_file_system_thunk;
59 } 59 }
60 60
61 } // namespace thunk 61 } // namespace thunk
62 } // namespace ppapi 62 } // namespace ppapi
OLDNEW
« no previous file with comments | « ppapi/thunk/ppb_file_ref_thunk.cc ('k') | ppapi/thunk/ppb_flash_clipboard_thunk.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698