PPAPI/NaCl: Restrict ppapi messages to unblock only ppapi

PPAPI/NaCl: Restrict ppapi messages to unblock only ppapi

Currently, we don't tell the HostDispatcher to only unblock pepper messages when an incoming "unblock" pepper message comes in. This results in us unblocking non-pepper messages improperly (such as GPU messages).

BUG=116317
TEST=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=149451

[dmichael (off chromium), 2012-07-31 22:26:32]

Should be easy...  basically just something we forgot to copy/paste from
pepper_plugin_delegate_impl.cc, though I added a comment that wasn't there in
the original.

[brettw, 2012-07-31 22:30:16]

lgtm

[commit-bot: I haz the power, 2012-07-31 22:34:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dmichael@chromium.org/10843017/1

[commit-bot: I haz the power, 2012-07-31 22:34:15]

Presubmit check for 10843017-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
You added one or more #includes that violate checkdeps rules.
  chrome/renderer/pepper/ppb_nacl_private_impl.cc
    Illegal include: "content/renderer/renderer_restrict_dispatch_group.h"
    Because of no rule applying

Presubmit checks took 3.6s to calculate.

[dmichael(do not use this one), 2012-07-31 22:40:50]

On Tue, Jul 31, 2012 at 4:34 PM, <commit-bot@chromium.org> wrote:

> Presubmit check for 10843017-1 failed and returned exit status 1.
>
> Running presubmit commit checks ...
>
> ** Presubmit ERRORS **
> You added one or more #includes that violate checkdeps rules.
>   chrome/renderer/pepper/ppb_**nacl_private_impl.cc
>     Illegal include: "content/renderer/renderer_**
> restrict_dispatch_group.h"
>
brettw: Should I move that class to content/public/renderer, and add an
appropriate DEPS rule?

I'm not too familiar with the dependency limitations here, so want to make
sure I get it right.

    Because of no rule applying
>
> Presubmit checks took 3.6s to calculate.
>
>
>
>
https://chromiumcodereview.**appspot.com/10843017/<https://chromiumcodereview...
>

[bbudge, 2012-07-31 22:42:06]

LGTM. Brett is fixing his machine so you might want to TBR him.

[brettw, 2012-07-31 22:50:54]

Talked with John and he's OK moving this file to public.

[commit-bot: I haz the power, 2012-08-01 15:51:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dmichael@chromium.org/10843017/8001

[commit-bot: I haz the power, 2012-08-01 18:35:02]

Change committed as 149451