Make the PPP wrappers take an Instance*.

Make the PPP wrappers take an Instance*.

I changed these to take InstanceHandles but this doesn't actually work. If you
use them the recommended way, the module->instance mapping isn't set up by the
time the constructors are called, so doing AddPerInstanceObject fails.

Since these objects are all called on the main thread, there is no race
condition with pp::Instance destruction and the implementation using a pointer
is safe.

BUG=
TEST=

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=123828

[jam, 2012-02-27 21:57:01]

lgtm

[jam, 2012-02-28 00:41:54]

in debug builds when navigating away from a pdf, i hit

PP_DCHECK(interface_name_to_objects_.empty());

there are 6 interfaces in that map

On Mon, Feb 27, 2012 at 1:57 PM, <jam@chromium.org> wrote:

> lgtm
>
>
>
>
https://chromiumcodereview.**appspot.com/9481015/<https://chromiumcodereview....
>

[brettw, 2012-02-28 04:47:23]

On Mon, Feb 27, 2012 at 4:41 PM, John Abd-El-Malek <jam@chromium.org> wrote:
> in debug builds when navigating away from a pdf, i hit
>
> PP_DCHECK(interface_name_to_objects_.empty());
>
> there are 6 interfaces in that map

Thanks, I'll fix that. It should be harmless in the meantime.

Brett