GCM: Handle non-Strings in incoming message Bundle

GCM: Handle non-Strings in incoming message Bundle

Google Cloud Messaging recently started including a "google.sent_time"
Long in the incoming message Bundle from Google Play Services. This
patch filters out non-String values to prevent warnings about
ClassCastExceptions.

BUG=639313
Committed: https://crrev.com/6688926c9db1c186b24b729fe8c8588f747a1a4a
Cr-Commit-Position: refs/heads/master@{#413435}

[johnme, 2016-08-19 15:46:29]

Description was changed from

==========
GCM: Handle non-Strings in incoming message Bundle

Google Cloud Messaging recently started including a "google.sent_time"
Long in the incoming message Bundle from Google Play Services. This
patch filters out non-String values to prevent warnings about
ClassCastExceptions.

BUG=639313
==========

to

==========
GCM: Handle non-Strings in incoming message Bundle

Google Cloud Messaging recently started including a "google.sent_time"
Long in the incoming message Bundle from Google Play Services. This
patch filters out non-String values to prevent warnings about
ClassCastExceptions.

BUG=639313
==========

[johnme, 2016-08-19 15:46:29]

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

[johnme, 2016-08-19 16:40:22]

The CQ bit was checked by johnme@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-19 16:40:59]

Dry run: 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-08-19 17:23:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-19 17:23:04]

Dry run: This issue passed the CQ dry run.

[Peter Beverloo, 2016-08-21 21:04:26]

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

[Peter Beverloo, 2016-08-21 21:04:27]

lgtm (note that you didn't send this out)

[johnme, 2016-08-22 12:41:34]

The CQ bit was checked by johnme@chromium.org

[johnme, 2016-08-22 12:41:39]

johnme@chromium.org changed reviewers:
- zea@chromium.org

[commit-bot: I haz the power, 2016-08-22 12:41:44]

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-08-22 13:15:36]

Description was changed from

==========
GCM: Handle non-Strings in incoming message Bundle

Google Cloud Messaging recently started including a "google.sent_time"
Long in the incoming message Bundle from Google Play Services. This
patch filters out non-String values to prevent warnings about
ClassCastExceptions.

BUG=639313
==========

to

==========
GCM: Handle non-Strings in incoming message Bundle

Google Cloud Messaging recently started including a "google.sent_time"
Long in the incoming message Bundle from Google Play Services. This
patch filters out non-String values to prevent warnings about
ClassCastExceptions.

BUG=639313
==========

[commit-bot: I haz the power, 2016-08-22 13:15:37]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-22 13:17:30]

Description was changed from

==========
GCM: Handle non-Strings in incoming message Bundle

Google Cloud Messaging recently started including a "google.sent_time"
Long in the incoming message Bundle from Google Play Services. This
patch filters out non-String values to prevent warnings about
ClassCastExceptions.

BUG=639313
==========

to

==========
GCM: Handle non-Strings in incoming message Bundle

Google Cloud Messaging recently started including a "google.sent_time"
Long in the incoming message Bundle from Google Play Services. This
patch filters out non-String values to prevent warnings about
ClassCastExceptions.

BUG=639313

Committed: https://crrev.com/6688926c9db1c186b24b729fe8c8588f747a1a4a
Cr-Commit-Position: refs/heads/master@{#413435}
==========

[commit-bot: I haz the power, 2016-08-22 13:17:31]

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