Add ability to create pending resource hosts.

Add ability to create pending resource hosts.

Allows the host side to register a ResourceHost as pending and send an ID to the plugin, to be connected to a PluginResource at a future time.

BUG=


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

[brettw, 2012-12-03 00:44:59]

I didn't add a unit test for this since just the map lookup seemed silly to have
a test for. Hopefully we can get a good test for the main document loads that
will cover all of this at the same time.

[raymes, 2012-12-03 15:55:18]

lgtm 

Looks good!

https://codereview.chromium.org/11414147/diff/10002/ppapi/host/ppapi_host.h
File ppapi/host/ppapi_host.h (right):

https://codereview.chromium.org/11414147/diff/10002/ppapi/host/ppapi_host.h#n...
ppapi/host/ppapi_host.h:98: void OnHostMsgAttachToPendingHost(PP_Resource
rseource, int pending_host_id);
rseource->resource

https://codereview.chromium.org/11414147/diff/10002/ppapi/proxy/plugin_resour...
File ppapi/proxy/plugin_resource.h (right):

https://codereview.chromium.org/11414147/diff/10002/ppapi/proxy/plugin_resour...
ppapi/proxy/plugin_resource.h:67: // object. The plugin uses this call to
connect the plugin resource with the
call->function

[commit-bot: I haz the power, 2012-12-04 04:34:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/brettw@chromium.org/11414147/5002

[commit-bot: I haz the power, 2012-12-04 04:34:54]

Presubmit check for 11414147-5002 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit Messages **
If this change has an associated bug, add BUG=[bug number].

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    ppapi/proxy/ppapi_messages.h

Presubmit checks took 2.0s to calculate.

[brettw, 2012-12-04 20:38:19]

Tom: fairly straightforward IPC review. Handler is implemented in ppapi_host.cc

[Tom Sepez, 2012-12-04 20:40:43]

LGTM on message security.  Thanks.

[commit-bot: I haz the power, 2012-12-04 20:45:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/brettw@chromium.org/11414147/5002

[commit-bot: I haz the power, 2012-12-05 00:41:21]

Change committed as 171099