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

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

Issue 11437038: Revert 171408 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years 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_flash_file_api.h ('k') | ppapi/thunk/ppb_flash_file_modulelocal_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_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_file_api.h" 9 #include "ppapi/thunk/ppb_flash_api.h"
10 #include "ppapi/thunk/ppb_instance_api.h"
10 #include "ppapi/thunk/thunk.h" 11 #include "ppapi/thunk/thunk.h"
11 12
12 namespace ppapi { 13 namespace ppapi {
13 namespace thunk { 14 namespace thunk {
14 15
15 namespace { 16 namespace {
16 17
17 // Returns 0 on failure. 18 // Returns 0 on failure.
18 PP_Instance GetInstanceFromFileRef(PP_Resource file_ref) { 19 PP_Instance GetInstanceFromFileRef(PP_Resource file_ref) {
19 thunk::EnterResource<thunk::PPB_FileRef_API> enter(file_ref, true); 20 thunk::EnterResource<thunk::PPB_FileRef_API> enter(file_ref, true);
20 if (enter.failed()) 21 if (enter.failed())
21 return 0; 22 return 0;
22 return enter.resource()->pp_instance(); 23 return enter.resource()->pp_instance();
23 } 24 }
24 25
25 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) {
26 // TODO(brettw): this function should take an instance. 27 // TODO(brettw): this function should take an instance.
27 // To work around this, use the PP_Instance from the resource. 28 // To work around this, use the PP_Instance from the resource.
28 PP_Instance instance = GetInstanceFromFileRef(file_ref_id); 29 PP_Instance instance = GetInstanceFromFileRef(file_ref_id);
29 EnterInstanceAPI<PPB_Flash_File_API> enter(instance); 30 EnterInstance enter(instance);
30 if (enter.failed()) 31 if (enter.failed())
31 return PP_ERROR_BADARGUMENT; 32 return PP_ERROR_BADARGUMENT;
32 return enter.functions()->OpenFileRef(instance, file_ref_id, mode, file); 33 return enter.functions()->GetFlashAPI()->OpenFileRef(instance, file_ref_id,
34 mode, file);
33 } 35 }
34 36
35 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) {
36 // TODO(brettw): this function should take an instance. 38 // TODO(brettw): this function should take an instance.
37 // To work around this, use the PP_Instance from the resource. 39 // To work around this, use the PP_Instance from the resource.
38 PP_Instance instance = GetInstanceFromFileRef(file_ref_id); 40 PP_Instance instance = GetInstanceFromFileRef(file_ref_id);
39 EnterInstanceAPI<PPB_Flash_File_API> enter(instance); 41 EnterInstance enter(instance);
40 if (enter.failed()) 42 if (enter.failed())
41 return PP_ERROR_BADARGUMENT; 43 return PP_ERROR_BADARGUMENT;
42 return enter.functions()->QueryFileRef(instance, file_ref_id, info); 44 return enter.functions()->GetFlashAPI()->QueryFileRef(instance, file_ref_id,
45 info);
43 } 46 }
44 47
45 const PPB_Flash_File_FileRef g_ppb_flash_file_fileref_thunk = { 48 const PPB_Flash_File_FileRef g_ppb_flash_file_fileref_thunk = {
46 &OpenFile, 49 &OpenFile,
47 &QueryFile 50 &QueryFile
48 }; 51 };
49 52
50 } // namespace 53 } // namespace
51 54
52 const PPB_Flash_File_FileRef* GetPPB_Flash_File_FileRef_Thunk() { 55 const PPB_Flash_File_FileRef* GetPPB_Flash_File_FileRef_Thunk() {
53 return &g_ppb_flash_file_fileref_thunk; 56 return &g_ppb_flash_file_fileref_thunk;
54 } 57 }
55 58
56 } // namespace thunk 59 } // namespace thunk
57 } // namespace ppapi 60 } // namespace ppapi
OLDNEW
« no previous file with comments | « ppapi/thunk/ppb_flash_file_api.h ('k') | ppapi/thunk/ppb_flash_file_modulelocal_thunk.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698