Associate variation IDs with uniformity trials.

Associate variation IDs with uniformity trials. 

BUG=124391
TEST=Run with -v=1 and make sure the correct field trial names and group names are being outputted by the log. Ensure that once we start transmitting variation IDs, these IDs begin appearing on the server.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=135353

[SteveT, 2012-05-04 02:18:44]

PTAL.

[MAD, 2012-05-04 12:32:54]

Cool stuff... LGTM with one small nit/question...

Thanks!

BYE
MAD...

http://codereview.chromium.org/10317011/diff/4002/chrome/common/metrics/varia...
File chrome/common/metrics/variation_ids.h (right):

http://codereview.chromium.org/10317011/diff/4002/chrome/common/metrics/varia...
chrome/common/metrics/variation_ids.h:48: kUniformity1PercentBase  = 3300000,
Why not use kMinimumID here?

[SteveT, 2012-05-04 13:20:36]

Response to MAD's nit.

http://codereview.chromium.org/10317011/diff/4002/chrome/common/metrics/varia...
File chrome/common/metrics/variation_ids.h (right):

http://codereview.chromium.org/10317011/diff/4002/chrome/common/metrics/varia...
chrome/common/metrics/variation_ids.h:48: kUniformity1PercentBase  = 3300000,
On 2012/05/04 12:32:54, MAD wrote:
> Why not use kMinimumID here?

You're right - should have done that. I will make this change in a small
follow-up patch for you.


Steve

[SteveT, 2012-05-04 13:29:01]

Oops - meant to say that I would do it now. For a second I thought this was a
different, closed patch. I have too many git branches :)

[SteveT, 2012-05-04 13:29:01]

Oops - meant to say that I would do it now. For a second I thought this was a
different, closed patch. I have too many git branches :)

[commit-bot: I haz the power, 2012-05-04 14:09:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/stevet@chromium.org/10317011/12001

[commit-bot: I haz the power, 2012-05-04 15:57:44]

Change committed as 135353