PPAPI: Add more error testing for postMessageAndAwaitResponse

PPAPI: Add more error testing for postMessageAndAwaitResponse

Add a test that tries using postMessageAndAwaitResponse in ways that should throw an exception, and makes sure it actually throws.

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

[dmichael (off chromium), 2014-09-26 16:52:39]

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

[dmichael (off chromium), 2014-09-26 18:38:17]

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

[teravest, 2014-09-26 19:28:50]

teravest@chromium.org changed reviewers:
+ teravest@chromium.org

[teravest, 2014-09-26 19:28:50]

https://codereview.chromium.org/608843002/diff/1/ppapi/tests/test_message_han...
File ppapi/tests/test_message_handler.cc (right):

https://codereview.chromium.org/608843002/diff/1/ppapi/tests/test_message_han...
ppapi/tests/test_message_handler.cc:316: ASSERT_EQ("SUCCESS", last_message_);
If this assertion fails, will testing_exceptions_ get set back to false? I
thought a string was returned on assert failures, preventing the rest of the
test case from running.

[dmichael (off chromium), 2014-09-26 20:12:43]

Took a slightly different approach. Now HandleMessage can tell which test is
running by asking the TestingInstance.

https://codereview.chromium.org/608843002/diff/1/ppapi/tests/test_message_han...
File ppapi/tests/test_message_handler.cc (right):

https://codereview.chromium.org/608843002/diff/1/ppapi/tests/test_message_han...
ppapi/tests/test_message_handler.cc:316: ASSERT_EQ("SUCCESS", last_message_);
On 2014/09/26 19:28:50, teravest wrote:
> If this assertion fails, will testing_exceptions_ get set back to false? I
> thought a string was returned on assert failures, preventing the rest of the
> test case from running.

Good catch, thanks.

[teravest, 2014-09-26 20:14:31]

lgtm

[dmichael (off chromium), 2014-09-26 20:15:06]

The CQ bit was checked by dmichael@chromium.org

[commit-bot: I haz the power, 2014-09-26 20:16:35]

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

[commit-bot: I haz the power, 2014-09-26 20:34:51]

Committed patchset #2 (id:20001) as c001780da861b64f221ba70d21f38e9e93d7aea5

[commit-bot: I haz the power, 2014-09-26 20:36:02]

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