GitLab

Before this CL, when we failed to extract a topic name from the response
to a per-user-topic subscription request, we'd record *both* kSuccess
and kParsingFailure in the FCMInvalidations.SubscriptionRequestStatus
histogram.
This CL fixes this by recording kSuccess only on actual success.
This probably makes little or no difference in practice, but let's be
consistent.

Bug: 1023813
Change-Id: I16ea38e4df5783455450e250ee0fd732ed0aa4f5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911751Reviewed-by: Tatiana Gornak <melandory@chromium.org>
Commit-Queue: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714878}