Skip to content

Commit

Permalink
Remove DEV permission requirement on PPB_MouseCursor::SetCursor()
Browse files Browse the repository at this point in the history
The permission-check was accidentally added in r171921, because the API's proxy implementation is shared with PPB_CursorControl_Dev.

BUG=174792

Review URL: https://chromiumcodereview.appspot.com/12207054

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181396 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
wez@chromium.org committed Feb 8, 2013
1 parent a0a1493 commit e65bf5b
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions ppapi/proxy/ppb_instance_proxy.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1140,13 +1140,12 @@ void PPB_Instance_Proxy::OnHostMsgDeliverSamples(
enter.functions()->DeliverSamples(instance, audio_frames, &block_info);
}

void PPB_Instance_Proxy::OnHostMsgSetCursor(
void PPB_Instance_Proxy::OnHostMsgSetCursor(
PP_Instance instance,
int32_t type,
const ppapi::HostResource& custom_image,
const PP_Point& hot_spot) {
if (!dispatcher()->permissions().HasPermission(PERMISSION_DEV))
return;
// This API serves PPB_CursorControl_Dev and PPB_MouseCursor, so is public.
EnterInstanceNoLock enter(instance);
if (enter.succeeded()) {
enter.functions()->SetCursor(
Expand Down

0 comments on commit e65bf5b

Please sign in to comment.