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

Unified Diff: ppapi/proxy/ppb_testing_proxy.cc

Issue 11471011: Add PPAPI permissions for the testing interface (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 side-by-side diff with in-line comments
Download patch
Index: ppapi/proxy/ppb_testing_proxy.cc
diff --git a/ppapi/proxy/ppb_testing_proxy.cc b/ppapi/proxy/ppb_testing_proxy.cc
index c52528b612e5683be1c5d639439dbe2885cc61ee..167a91e843bb09789a1fcafe48a44d53668d3912 100644
--- a/ppapi/proxy/ppb_testing_proxy.cc
+++ b/ppapi/proxy/ppb_testing_proxy.cc
@@ -156,6 +156,9 @@ const InterfaceProxy::Info* PPB_Testing_Proxy::GetInfo() {
}
bool PPB_Testing_Proxy::OnMessageReceived(const IPC::Message& msg) {
+ if (!dispatcher()->permissions().HasPermission(PERMISSION_TESTING))
bbudge 2012/12/07 22:23:41 Should we also allow this for the 'DEV' permission
+ return false;
+
bool handled = true;
IPC_BEGIN_MESSAGE_MAP(PPB_Testing_Proxy, msg)
IPC_MESSAGE_HANDLER(PpapiHostMsg_PPBTesting_ReadImageData,

Powered by Google App Engine
This is Rietveld 408576698