Adding UMA stats for Blimp Client

Adding UMA stats for Blimp Client

Added metrics for tracking when blimp is supported, blimp is connected,
blimp tabs are shown or hidden, size of commit messages.

BUG=645545
Committed: https://crrev.com/f769f364f71068930d971dafb86dc1c5f298a8a7
Cr-Commit-Position: refs/heads/master@{#418386}

[shaktisahu, 2016-09-09 19:02:47]

Description was changed from

==========
Adding UMA stats for Blimp Client

Added metrics for tracking when blimp is supported, blimp is connected,
blimp tabs are shown or hidden, size of commit messages.

BUG=645545
==========

to

==========
Adding UMA stats for Blimp Client

Added metrics for tracking when blimp is supported, blimp is connected,
blimp tabs are shown or hidden, size of commit messages.

BUG=645545
==========

[shaktisahu, 2016-09-09 19:02:47]

shaktisahu@chromium.org changed reviewers:
+ dtrainor@chromium.org

[shaktisahu, 2016-09-12 17:50:21]

shaktisahu@chromium.org changed reviewers:
+ nyquist@chromium.org

[shaktisahu, 2016-09-12 17:50:21]

PTAL

[David Trainor- moved to gerrit, 2016-09-13 05:46:16]

lgtm but make sure we understand the implications of the tab visibility UMA. 
Let's talk tomorrow about this before landing thanks!

https://codereview.chromium.org/2326913004/diff/1/blimp/client/core/composito...
File blimp/client/core/compositor/blimp_compositor.cc (right):

https://codereview.chromium.org/2326913004/diff/1/blimp/client/core/composito...
blimp/client/core/compositor/blimp_compositor.cc:138: message->ByteSize());
Isn't this only bytes?

[shaktisahu, 2016-09-13 17:28:19]

shaktisahu@chromium.org changed reviewers:
+ rkaplow@chromium.org

[shaktisahu, 2016-09-13 17:28:20]

rkaplow@ : PTAL

https://codereview.chromium.org/2326913004/diff/1/blimp/client/core/composito...
File blimp/client/core/compositor/blimp_compositor.cc (right):

https://codereview.chromium.org/2326913004/diff/1/blimp/client/core/composito...
blimp/client/core/compositor/blimp_compositor.cc:138: message->ByteSize());
On 2016/09/13 05:46:16, David Trainor wrote:
> Isn't this only bytes?

Done.

[rkaplow, 2016-09-13 19:53:01]

lgtm

[shaktisahu, 2016-09-13 20:29:59]

The CQ bit was unchecked by shaktisahu@chromium.org

[shaktisahu, 2016-09-13 20:30:00]

The CQ bit was checked by shaktisahu@chromium.org

[shaktisahu, 2016-09-13 20:30:00]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtrainor@chromium.org
Link to the patchset: https://codereview.chromium.org/2326913004/#ps20001
(title: "dtrainor comments")

[shaktisahu, 2016-09-13 20:30:17]

The CQ bit was checked by shaktisahu@chromium.org

[commit-bot: I haz the power, 2016-09-13 20:30:22]

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-09-13 22:04:01]

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

[commit-bot: I haz the power, 2016-09-13 22:04:07]

Failed to apply the patch.
On branch working_branch
Your branch is up-to-date with 'origin/refs/pending/heads/master'.
nothing to commit, working tree clean

[commit-bot: I haz the power, 2016-09-13 22:05:05]

Description was changed from

==========
Adding UMA stats for Blimp Client

Added metrics for tracking when blimp is supported, blimp is connected,
blimp tabs are shown or hidden, size of commit messages.

BUG=645545
==========

to

==========
Adding UMA stats for Blimp Client

Added metrics for tracking when blimp is supported, blimp is connected,
blimp tabs are shown or hidden, size of commit messages.

BUG=645545

Committed: https://crrev.com/f769f364f71068930d971dafb86dc1c5f298a8a7
Cr-Commit-Position: refs/heads/master@{#418386}
==========

[commit-bot: I haz the power, 2016-09-13 22:05:07]

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