Fix asserts in gcm_client_impl_unittest

Fix asserts in gcm_client_impl_unittest

Many of the tests were calling helper methods that ASSERT, without
wrapping those calls with ASSERT_NO_FATAL_FAILURE to propagate any
assertion failures to the test. This patch wraps the calls.

BUG=none

Committed: https://crrev.com/0fb2fc0f442e3f9390d0521b19603b8095fecb23
Cr-Commit-Position: refs/heads/master@{#405494}

[johnme, 2016-07-14 11:47:18]

johnme@chromium.org changed reviewers:
+ peter@chromium.org

[Peter Beverloo, 2016-07-14 13:52:40]

lgtm

[johnme, 2016-07-14 14:13:27]

The CQ bit was checked by johnme@chromium.org

[commit-bot: I haz the power, 2016-07-14 14:13:47]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-14 14:47:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-14 14:48:04]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-14 14:49:27]

Description was changed from

==========
Fix asserts in gcm_client_impl_unittest

Many of the tests were calling helper methods that ASSERT, without
wrapping those calls with ASSERT_NO_FATAL_FAILURE to propagate any
assertion failures to the test. This patch wraps the calls.

BUG=none
==========

to

==========
Fix asserts in gcm_client_impl_unittest

Many of the tests were calling helper methods that ASSERT, without
wrapping those calls with ASSERT_NO_FATAL_FAILURE to propagate any
assertion failures to the test. This patch wraps the calls.

BUG=none

Committed: https://crrev.com/0fb2fc0f442e3f9390d0521b19603b8095fecb23
Cr-Commit-Position: refs/heads/master@{#405494}
==========

[commit-bot: I haz the power, 2016-07-14 14:49:28]

Patchset 1 (id:??) landed as
https://crrev.com/0fb2fc0f442e3f9390d0521b19603b8095fecb23
Cr-Commit-Position: refs/heads/master@{#405494}