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

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

Issue 14161017: Pepper: Simplify idl_thunk implementation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: on_failure comment nit Created 7 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 | Annotate | Revision Log
« no previous file with comments | « ppapi/thunk/ppb_url_loader_thunk.cc ('k') | ppapi/thunk/ppb_widget_dev_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 // From trusted/ppb_url_loader_trusted.idl modified Wed Apr 17 09:21:10 2013. 5 // From trusted/ppb_url_loader_trusted.idl modified Wed Apr 17 11:16:00 2013.
6 6
7 #include "ppapi/c/pp_errors.h" 7 #include "ppapi/c/pp_errors.h"
8 #include "ppapi/c/trusted/ppb_url_loader_trusted.h" 8 #include "ppapi/c/trusted/ppb_url_loader_trusted.h"
9 #include "ppapi/shared_impl/tracked_callback.h" 9 #include "ppapi/shared_impl/tracked_callback.h"
10 #include "ppapi/thunk/enter.h" 10 #include "ppapi/thunk/enter.h"
11 #include "ppapi/thunk/ppb_instance_api.h" 11 #include "ppapi/thunk/ppb_instance_api.h"
12 #include "ppapi/thunk/ppb_url_loader_api.h" 12 #include "ppapi/thunk/ppb_url_loader_api.h"
13 #include "ppapi/thunk/resource_creation_api.h" 13 #include "ppapi/thunk/resource_creation_api.h"
14 #include "ppapi/thunk/thunk.h" 14 #include "ppapi/thunk/thunk.h"
15 15
16 namespace ppapi { 16 namespace ppapi {
17 namespace thunk { 17 namespace thunk {
18 18
19 namespace { 19 namespace {
20 20
21 void GrantUniversalAccess(PP_Resource loader) { 21 void GrantUniversalAccess(PP_Resource loader) {
22 VLOG(4) << "PPB_URLLoaderTrusted::GrantUniversalAccess()"; 22 VLOG(4) << "PPB_URLLoaderTrusted::GrantUniversalAccess()";
23 EnterResource<PPB_URLLoader_API> enter(loader, true); 23 EnterResource<PPB_URLLoader_API> enter(loader, true);
24 if (enter.succeeded()) 24 if (enter.failed())
25 enter.object()->GrantUniversalAccess(); 25 return;
26 enter.object()->GrantUniversalAccess();
26 } 27 }
27 28
28 void RegisterStatusCallback(PP_Resource loader, 29 void RegisterStatusCallback(PP_Resource loader,
29 PP_URLLoaderTrusted_StatusCallback cb) { 30 PP_URLLoaderTrusted_StatusCallback cb) {
30 VLOG(4) << "PPB_URLLoaderTrusted::RegisterStatusCallback()"; 31 VLOG(4) << "PPB_URLLoaderTrusted::RegisterStatusCallback()";
31 EnterResource<PPB_URLLoader_API> enter(loader, true); 32 EnterResource<PPB_URLLoader_API> enter(loader, true);
32 if (enter.succeeded()) 33 if (enter.failed())
33 enter.object()->RegisterStatusCallback(cb); 34 return;
35 enter.object()->RegisterStatusCallback(cb);
34 } 36 }
35 37
36 const PPB_URLLoaderTrusted_0_3 g_ppb_urlloadertrusted_thunk_0_3 = { 38 const PPB_URLLoaderTrusted_0_3 g_ppb_urlloadertrusted_thunk_0_3 = {
37 &GrantUniversalAccess, 39 &GrantUniversalAccess,
38 &RegisterStatusCallback 40 &RegisterStatusCallback
39 }; 41 };
40 42
41 } // namespace 43 } // namespace
42 44
43 const PPB_URLLoaderTrusted_0_3* GetPPB_URLLoaderTrusted_0_3_Thunk() { 45 const PPB_URLLoaderTrusted_0_3* GetPPB_URLLoaderTrusted_0_3_Thunk() {
44 return &g_ppb_urlloadertrusted_thunk_0_3; 46 return &g_ppb_urlloadertrusted_thunk_0_3;
45 } 47 }
46 48
47 } // namespace thunk 49 } // namespace thunk
48 } // namespace ppapi 50 } // namespace ppapi
OLDNEW
« no previous file with comments | « ppapi/thunk/ppb_url_loader_thunk.cc ('k') | ppapi/thunk/ppb_widget_dev_thunk.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698