Fix for setting the os listed in metrics reporting for Blimp.

Fix for setting the os listed in metrics reporting for Blimp.

Metrics uploads from blimp engine are including OS = Linux in the System Proto instead of the desired "Blimp".

This is because the defines isn't getting set in BUILD.gn until after metrics_log.cc is built.

This change will ensure the blimp engine is uploading logs with the desired system information so they can be separate from other linux metrics logs.

BUG=613680
Committed: https://crrev.com/d599415d4e9fbb458090f9cd544a17dcd0a90a6c
Cr-Commit-Position: refs/heads/master@{#395170}

[Jess, 2016-05-20 20:04:56]

jessicag@chromium.org changed reviewers:
+ asvitkine@chromium.org

[Jess, 2016-05-20 20:04:56]

Short change to fix a BUILD file error that is a root cause of Blimp upload not
showing up.

[Jess, 2016-05-20 20:04:58]

Short change to fix a BUILD file error that is a root cause of Blimp upload not
showing up.

[Alexei Svitkine (slow), 2016-05-20 20:06:12]

I think you need to sync your client as bots show patch failure.

[Jess, 2016-05-20 20:13:21]

On 2016/05/20 20:06:12, Alexei Svitkine (OOO til Tues) wrote:
> I think you need to sync your client as bots show patch failure.

You are correct. Sorry for the premature send.  Synced and new patch out to
trybots.

[Alexei Svitkine (slow), 2016-05-20 20:15:03]

lgtm

[Jess, 2016-05-20 21:01:25]

The CQ bit was checked by jessicag@chromium.org

[commit-bot: I haz the power, 2016-05-20 21:02:33]

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

[commit-bot: I haz the power, 2016-05-20 21:27:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-20 21:28:38]

Description was changed from

==========
Fix for setting the os listed in metrics reporting for Blimp.

Metrics uploads from blimp engine are including OS = Linux in the System Proto
instead of the desired "Blimp".

This is because the defines isn't getting set in BUILD.gn until after
metrics_log.cc is built.

This change will ensure the blimp engine is uploading logs with the desired
system information so they can be separate from other linux metrics logs.

BUG=613680
==========

to

==========
Fix for setting the os listed in metrics reporting for Blimp.

Metrics uploads from blimp engine are including OS = Linux in the System Proto
instead of the desired "Blimp".

This is because the defines isn't getting set in BUILD.gn until after
metrics_log.cc is built.

This change will ensure the blimp engine is uploading logs with the desired
system information so they can be separate from other linux metrics logs.

BUG=613680

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

[commit-bot: I haz the power, 2016-05-20 21:28:39]

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