PPAPI: Add C++ wrapper for MessageHandler stuff.

PPAPI: Add C++ wrapper for MessageHandler stuff.

BUG=367896
Committed: https://crrev.com/b8737917043275c52ae682cddc6604c495d2e73f
Cr-Commit-Position: refs/heads/master@{#295495}

[dmichael (off chromium), 2014-09-17 22:57:03]

dmichael@chromium.org changed reviewers:
+ bbudge@chromium.org

[bbudge, 2014-09-18 12:41:17]

LGTM w / comments

https://codereview.chromium.org/318763003/diff/20001/ppapi/cpp/instance.h
File ppapi/cpp/instance.h (right):

https://codereview.chromium.org/318763003/diff/20001/ppapi/cpp/instance.h#new...
ppapi/cpp/instance.h:505: /// postMessageAndAwaitResponse will be dispatched to
<code>handler</code>.
s/handler/message_handler

https://codereview.chromium.org/318763003/diff/20001/ppapi/cpp/instance.h#new...
ppapi/cpp/instance.h:526: /// @param[in] handler The plugin-provided object for
handling messages. The
s/handler/message_handler

https://codereview.chromium.org/318763003/diff/20001/ppapi/cpp/instance.h#new...
ppapi/cpp/instance.h:539: /// further messages after that point. After that
point, all messages sent
nit: I would eliminate the first "after that point" as redundant with the
sentence beginning.

https://codereview.chromium.org/318763003/diff/20001/ppapi/cpp/message_handler.h
File ppapi/cpp/message_handler.h (right):

https://codereview.chromium.org/318763003/diff/20001/ppapi/cpp/message_handle...
ppapi/cpp/message_handler.h:11: /// implement if it wants to receive messages
from JavaScript (postMessage() or
the first ( doesn't seem to have a matching )

https://codereview.chromium.org/318763003/diff/20001/ppapi/cpp/message_handle...
ppapi/cpp/message_handler.h:38: /// @param[in] message is a copy of the
parameter that JavaScript provided
s/message/message_data
This should be identical to line 23 above.

[dmichael (off chromium), 2014-09-18 15:50:09]

dmichael@chromium.org changed reviewers:
+ binji@chromium.org

[dmichael (off chromium), 2014-09-18 15:50:10]

+binji for library.dsc

[binji, 2014-09-18 16:33:38]

native_client_sdk lgtm

[dmichael (off chromium), 2014-09-18 16:35:13]

The CQ bit was checked by dmichael@chromium.org

[commit-bot: I haz the power, 2014-09-18 16:36:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/318763003/60001

[commit-bot: I haz the power, 2014-09-18 18:00:41]

Committed patchset #4 (id:60001) as 44f524b61a1b7cdd0c63ab9a5c90656b7475478d

[commit-bot: I haz the power, 2014-09-18 18:01:16]

Patchset 4 (id:??) landed as
https://crrev.com/b8737917043275c52ae682cddc6604c495d2e73f
Cr-Commit-Position: refs/heads/master@{#295495}