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

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

Issue 11359097: Refactored the PPB_Flash_File_ModuleLocal/FileRef to the new ppapi resource model (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years, 1 month 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 (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_errors.h" 5 #include "ppapi/c/pp_errors.h"
6 #include "ppapi/c/private/ppb_flash_file.h" 6 #include "ppapi/c/private/ppb_flash_file.h"
7 #include "ppapi/thunk/enter.h" 7 #include "ppapi/thunk/enter.h"
8 #include "ppapi/thunk/ppb_file_ref_api.h" 8 #include "ppapi/thunk/ppb_file_ref_api.h"
9 #include "ppapi/thunk/ppb_flash_api.h" 9 #include "ppapi/thunk/ppb_flash_file_api.h"
10 #include "ppapi/thunk/ppb_instance_api.h" 10 #include "ppapi/thunk/ppb_instance_api.h"
11 #include "ppapi/thunk/thunk.h" 11 #include "ppapi/thunk/thunk.h"
12 12
13 namespace ppapi { 13 namespace ppapi {
14 namespace thunk { 14 namespace thunk {
15 15
16 namespace { 16 namespace {
17 17
18 // Returns 0 on failure. 18 // Returns 0 on failure.
19 PP_Instance GetInstanceFromFileRef(PP_Resource file_ref) { 19 PP_Instance GetInstanceFromFileRef(PP_Resource file_ref) {
20 thunk::EnterResource<thunk::PPB_FileRef_API> enter(file_ref, true); 20 thunk::EnterResource<thunk::PPB_FileRef_API> enter(file_ref, true);
21 if (enter.failed()) 21 if (enter.failed())
22 return 0; 22 return 0;
23 return enter.resource()->pp_instance(); 23 return enter.resource()->pp_instance();
24 } 24 }
25 25
26 int32_t OpenFile(PP_Resource file_ref_id, int32_t mode, PP_FileHandle* file) { 26 int32_t OpenFile(PP_Resource file_ref_id, int32_t mode, PP_FileHandle* file) {
27 // TODO(brettw): this function should take an instance. 27 // TODO(brettw): this function should take an instance.
28 // To work around this, use the PP_Instance from the resource. 28 // To work around this, use the PP_Instance from the resource.
29 PP_Instance instance = GetInstanceFromFileRef(file_ref_id); 29 PP_Instance instance = GetInstanceFromFileRef(file_ref_id);
30 EnterInstance enter(instance); 30 EnterInstance enter(instance);
31 if (enter.failed()) 31 if (enter.failed())
32 return PP_ERROR_BADARGUMENT; 32 return PP_ERROR_BADARGUMENT;
33 return enter.functions()->GetFlashAPI()->OpenFileRef(instance, file_ref_id, 33 return enter.functions()->GetFlashFileAPI(instance)->OpenFileRef(
34 mode, file); 34 instance, file_ref_id, mode, file);
35 } 35 }
36 36
37 int32_t QueryFile(PP_Resource file_ref_id, struct PP_FileInfo* info) { 37 int32_t QueryFile(PP_Resource file_ref_id, struct PP_FileInfo* info) {
38 // TODO(brettw): this function should take an instance. 38 // TODO(brettw): this function should take an instance.
39 // To work around this, use the PP_Instance from the resource. 39 // To work around this, use the PP_Instance from the resource.
40 PP_Instance instance = GetInstanceFromFileRef(file_ref_id); 40 PP_Instance instance = GetInstanceFromFileRef(file_ref_id);
41 EnterInstance enter(instance); 41 EnterInstance enter(instance);
42 if (enter.failed()) 42 if (enter.failed())
43 return PP_ERROR_BADARGUMENT; 43 return PP_ERROR_BADARGUMENT;
44 return enter.functions()->GetFlashAPI()->QueryFileRef(instance, file_ref_id, 44 return enter.functions()->GetFlashFileAPI(instance)->QueryFileRef(
45 info); 45 instance, file_ref_id, info);
46 } 46 }
47 47
48 const PPB_Flash_File_FileRef g_ppb_flash_file_fileref_thunk = { 48 const PPB_Flash_File_FileRef g_ppb_flash_file_fileref_thunk = {
49 &OpenFile, 49 &OpenFile,
50 &QueryFile 50 &QueryFile
51 }; 51 };
52 52
53 } // namespace 53 } // namespace
54 54
55 const PPB_Flash_File_FileRef* GetPPB_Flash_File_FileRef_Thunk() { 55 const PPB_Flash_File_FileRef* GetPPB_Flash_File_FileRef_Thunk() {
56 return &g_ppb_flash_file_fileref_thunk; 56 return &g_ppb_flash_file_fileref_thunk;
57 } 57 }
58 58
59 } // namespace thunk 59 } // namespace thunk
60 } // namespace ppapi 60 } // namespace ppapi
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698