ppapi/thunk/interfaces_ppb_private_no_permissions.h - Issue 24039002: Pepper API implementation for output protection.

Side by Side Diff: ppapi/thunk/interfaces_ppb_private_no_permissions.h

Issue 24039002: Pepper API implementation for output protection. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master

Patch Set: avoid PostTask methods in dtor Created 7 years, 3 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

« ppapi/proxy/resource_creation_proxy.cc ('K') | « ppapi/tests/all_c_includes.h ('k') | ppapi/thunk/ppb_output_protection_api.h » ('j') | no next file with comments »

OLD	NEW
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 // Please see inteface_ppb_public_stable for the documentation on the format of	5 // Please see inteface_ppb_public_stable for the documentation on the format of

6 // this file.	6 // this file.

7	7

8 #include "ppapi/thunk/interfaces_preamble.h"	8 #include "ppapi/thunk/interfaces_preamble.h"

9	9

10 // These interfaces don't require private permissions. However, they only work	10 // These interfaces don't require private permissions. However, they only work

(...skipping 35 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
46 PROXIED_IFACE(NoAPIName, PPB_EXT_CRXFILESYSTEM_PRIVATE_INTERFACE_0_1,	46 PROXIED_IFACE(NoAPIName, PPB_EXT_CRXFILESYSTEM_PRIVATE_INTERFACE_0_1,

47 PPB_Ext_CrxFileSystem_Private_0_1)	47 PPB_Ext_CrxFileSystem_Private_0_1)

48 PROXIED_IFACE(NoAPIName, PPB_FILEIO_PRIVATE_INTERFACE_0_1,	48 PROXIED_IFACE(NoAPIName, PPB_FILEIO_PRIVATE_INTERFACE_0_1,

49 PPB_FileIO_Private_0_1)	49 PPB_FileIO_Private_0_1)

50	50

51 PROXIED_IFACE(NoAPIName, PPB_VIDEODESTINATION_PRIVATE_INTERFACE_0_1,	51 PROXIED_IFACE(NoAPIName, PPB_VIDEODESTINATION_PRIVATE_INTERFACE_0_1,

52 PPB_VideoDestination_Private_0_1)	52 PPB_VideoDestination_Private_0_1)

53 PROXIED_IFACE(NoAPIName, PPB_VIDEOSOURCE_PRIVATE_INTERFACE_0_1,	53 PROXIED_IFACE(NoAPIName, PPB_VIDEOSOURCE_PRIVATE_INTERFACE_0_1,

54 PPB_VideoSource_Private_0_1)	54 PPB_VideoSource_Private_0_1)

55	55

	56 PROXIED_IFACE(NoAPIName, PPB_OUTPUTPROTECTION_PRIVATE_INTERFACE_0_1,

	57 PPB_OutputProtection_Private_0_1)
	dmichael (off chromium) 2013/09/09 20:37:15 See the comment on 10 and 11... is that how we wa See the comment on 10 and 11... is that how we want this to work? That this is only available for whitelisted origins/plugins? I think you probably want this in interfaces_ppb_private.h. Otherwise the check you have in the host-side factory is inconsistent. (That, or the host-side factory should check that the plugin is in a whitelist, instead of looking at PRIVATE permissions.) kcwu 2013/09/10 12:50:21 Good question. I'm not sure how CDM will use this Show quoted text On 2013/09/09 20:37:15, dmichael wrote: > See the comment on 10 and 11... is that how we want this to work? That this is > only available for whitelisted origins/plugins? > > I think you probably want this in interfaces_ppb_private.h. Otherwise the check > you have in the host-side factory is inconsistent. (That, or the host-side > factory should check that the plugin is in a whitelist, instead of looking at > PRIVATE permissions.) Good question. I'm not sure how CDM will use this api. dalecurtis@, could you give comment on this? DaleCurtis 2013/09/12 18:32:38 Per ddorwin@ the CDM has private permissions, so i Show quoted text On 2013/09/10 12:50:21, kcwu wrote: > On 2013/09/09 20:37:15, dmichael wrote: > > See the comment on 10 and 11... is that how we want this to work? That this > is > > only available for whitelisted origins/plugins? > > > > I think you probably want this in interfaces_ppb_private.h. Otherwise the > check > > you have in the host-side factory is inconsistent. (That, or the host-side > > factory should check that the plugin is in a whitelist, instead of looking at > > PRIVATE permissions.) > > Good question. I'm not sure how CDM will use this api. > dalecurtis@, could you give comment on this? Per ddorwin@ the CDM has private permissions, so it should go under private. kcwu 2013/09/16 11:57:37 Done. Show quoted text On 2013/09/12 18:32:38, DaleCurtis wrote: > On 2013/09/10 12:50:21, kcwu wrote: > > On 2013/09/09 20:37:15, dmichael wrote: > > > See the comment on 10 and 11... is that how we want this to work? That this > > is > > > only available for whitelisted origins/plugins? > > > > > > I think you probably want this in interfaces_ppb_private.h. Otherwise the > > check > > > you have in the host-side factory is inconsistent. (That, or the host-side > > > factory should check that the plugin is in a whitelist, instead of looking > at > > > PRIVATE permissions.) > > > > Good question. I'm not sure how CDM will use this api. > > dalecurtis@, could you give comment on this? > > Per ddorwin@ the CDM has private permissions, so it should go under private. Done.
	58

56 #include "ppapi/thunk/interfaces_postamble.h"	59 #include "ppapi/thunk/interfaces_postamble.h"

OLD	NEW