Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(115)

Side by Side Diff: chrome/common/metrics/variation_ids.h

Issue 10317011: Associate variation IDs with uniformity trials. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: kUniformityBase = kMinID Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « chrome/browser/chrome_browser_main.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_COMMON_METRICS_VARIATION_IDS_H_ 5 #ifndef CHROME_COMMON_METRICS_VARIATION_IDS_H_
6 #define CHROME_COMMON_METRICS_VARIATION_IDS_H_ 6 #define CHROME_COMMON_METRICS_VARIATION_IDS_H_
7 #pragma once 7 #pragma once
8 8
9 namespace chrome_variations { 9 namespace chrome_variations {
10 10
(...skipping 27 matching lines...) Expand all
38 // please ensure that they are within the range of the min and max values. 38 // please ensure that they are within the range of the min and max values.
39 enum ID { 39 enum ID {
40 // Used to represent no associated Chrome variation ID. 40 // Used to represent no associated Chrome variation ID.
41 kEmptyID = 0, 41 kEmptyID = 0,
42 42
43 // The smallest possible Chrome Variation ID in the reserved range. The 43 // The smallest possible Chrome Variation ID in the reserved range. The
44 // first 10,000 values are reserved for internal variations infrastructure 44 // first 10,000 values are reserved for internal variations infrastructure
45 // use. Please do not use values in this range. 45 // use. Please do not use values in this range.
46 kMinimumID = 3300000, 46 kMinimumID = 3300000,
47 47
48 kUniformity1PercentBase = kMinimumID,
49 kUniformity5PercentBase = kUniformity1PercentBase + 100,
50 kUniformity10PercentBase = kUniformity5PercentBase + 20,
51 kUniformity20PercentBase = kUniformity10PercentBase + 10,
52 kUniformity50PercentBase = kUniformity20PercentBase + 5,
53 kUniformityTrialsMax = 3300136,
54
48 // Some values reserved for unit and integration tests. 55 // Some values reserved for unit and integration tests.
49 kTestValueA = 3300200, 56 kTestValueA = 3300200,
50 kTestValueB = 3300201, 57 kTestValueB = 3300201,
51 58
52 // USABLE IDs BEGIN HERE. 59 // USABLE IDs BEGIN HERE.
53 // 60 //
54 // The smallest possible Chrome Variation ID for use in real FieldTrials. If 61 // The smallest possible Chrome Variation ID for use in real FieldTrials. If
55 // you are defining variation IDs for your own FieldTrials, NEVER use a value 62 // you are defining variation IDs for your own FieldTrials, NEVER use a value
56 // lower than this. 63 // lower than this.
57 kMinimumUserID = 3310000, 64 kMinimumUserID = 3310000,
58 65
59 // Add new variation IDs below. 66 // Add new variation IDs below.
60 67
61 // USABLE IDs END HERE. 68 // USABLE IDs END HERE.
62 // 69 //
63 // The largest possible Chrome variation ID in the reserved range. When 70 // The largest possible Chrome variation ID in the reserved range. When
64 // defining your variation IDs, DO NOT exceed this value. 71 // defining your variation IDs, DO NOT exceed this value.
65 kMaximumID = 3399999, 72 kMaximumID = 3399999,
66 }; 73 };
67 74
68 } // namespace chrome_variations 75 } // namespace chrome_variations
69 76
70 #endif // CHROME_COMMON_METRICS_VARIATION_IDS_H_ 77 #endif // CHROME_COMMON_METRICS_VARIATION_IDS_H_
OLDNEW
« no previous file with comments | « chrome/browser/chrome_browser_main.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698