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

Side by Side Diff: chrome/common/variations/experiment_labels.cc

Issue 412943002: Move variations component code to variations namespace. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 6 years, 4 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #include "chrome/common/variations/experiment_labels.h" 5 #include "chrome/common/variations/experiment_labels.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/strings/string_number_conversions.h" 10 #include "base/strings/string_number_conversions.h"
11 #include "base/strings/string_split.h" 11 #include "base/strings/string_split.h"
12 #include "base/strings/string_util.h" 12 #include "base/strings/string_util.h"
13 #include "base/strings/utf_string_conversions.h" 13 #include "base/strings/utf_string_conversions.h"
14 #include "chrome/installer/util/google_update_experiment_util.h" 14 #include "chrome/installer/util/google_update_experiment_util.h"
15 #include "components/variations/variations_associated_data.h" 15 #include "components/variations/variations_associated_data.h"
16 16
17 namespace chrome_variations { 17 namespace chrome_variations {
18 18
19 namespace { 19 namespace {
20 20
21 const char kVariationPrefix[] = "CrVar"; 21 const char kVariationPrefix[] = "CrVar";
22 22
23 // This method builds a single experiment label for a Chrome Variation, 23 // This method builds a single experiment label for a Chrome Variation,
24 // including a timestamp that is a year in the future from |current_time|. Since 24 // including a timestamp that is a year in the future from |current_time|. Since
25 // multiple headers can be transmitted, |count| is a number that is appended 25 // multiple headers can be transmitted, |count| is a number that is appended
26 // after the label key to differentiate the labels. 26 // after the label key to differentiate the labels.
27 base::string16 CreateSingleExperimentLabel(int count, VariationID id, 27 base::string16 CreateSingleExperimentLabel(int count,
28 variations::VariationID id,
28 const base::Time& current_time) { 29 const base::Time& current_time) {
29 // Build the parts separately so they can be validated. 30 // Build the parts separately so they can be validated.
30 const base::string16 key = 31 const base::string16 key =
31 base::ASCIIToUTF16(kVariationPrefix) + base::IntToString16(count); 32 base::ASCIIToUTF16(kVariationPrefix) + base::IntToString16(count);
32 DCHECK_LE(key.size(), 8U); 33 DCHECK_LE(key.size(), 8U);
33 const base::string16 value = base::IntToString16(id); 34 const base::string16 value = base::IntToString16(id);
34 DCHECK_LE(value.size(), 8U); 35 DCHECK_LE(value.size(), 8U);
35 base::string16 label(key); 36 base::string16 label(key);
36 label += base::ASCIIToUTF16("="); 37 label += base::ASCIIToUTF16("=");
37 label += value; 38 label += value;
38 label += base::ASCIIToUTF16("|"); 39 label += base::ASCIIToUTF16("|");
39 label += installer::BuildExperimentDateString(current_time); 40 label += installer::BuildExperimentDateString(current_time);
40 return label; 41 return label;
41 } 42 }
42 43
43 } // namespace 44 } // namespace
44 45
45 base::string16 BuildGoogleUpdateExperimentLabel( 46 base::string16 BuildGoogleUpdateExperimentLabel(
46 const base::FieldTrial::ActiveGroups& active_groups) { 47 const base::FieldTrial::ActiveGroups& active_groups) {
47 base::string16 experiment_labels; 48 base::string16 experiment_labels;
48 int counter = 0; 49 int counter = 0;
49 50
50 const base::Time current_time(base::Time::Now()); 51 const base::Time current_time(base::Time::Now());
51 52
52 // Find all currently active VariationIDs associated with Google Update. 53 // Find all currently active VariationIDs associated with Google Update.
53 for (base::FieldTrial::ActiveGroups::const_iterator it = 54 for (base::FieldTrial::ActiveGroups::const_iterator it =
54 active_groups.begin(); it != active_groups.end(); ++it) { 55 active_groups.begin(); it != active_groups.end(); ++it) {
55 const VariationID id = GetGoogleVariationID(GOOGLE_UPDATE_SERVICE, 56 const variations::VariationID id =
56 it->trial_name, it->group_name); 57 variations::GetGoogleVariationID(variations::GOOGLE_UPDATE_SERVICE,
58 it->trial_name, it->group_name);
57 59
58 if (id == EMPTY_ID) 60 if (id == variations::EMPTY_ID)
59 continue; 61 continue;
60 62
61 if (!experiment_labels.empty()) 63 if (!experiment_labels.empty())
62 experiment_labels += google_update::kExperimentLabelSeparator; 64 experiment_labels += google_update::kExperimentLabelSeparator;
63 experiment_labels += CreateSingleExperimentLabel(++counter, id, 65 experiment_labels += CreateSingleExperimentLabel(++counter, id,
64 current_time); 66 current_time);
65 } 67 }
66 68
67 return experiment_labels; 69 return experiment_labels;
68 } 70 }
(...skipping 30 matching lines...) Expand all
99 DCHECK(!EndsWith(other_labels, separator, false)); 101 DCHECK(!EndsWith(other_labels, separator, false));
100 // Note that if either label is empty, a separator is not necessary. 102 // Note that if either label is empty, a separator is not necessary.
101 base::string16 combined_labels = other_labels; 103 base::string16 combined_labels = other_labels;
102 if (!other_labels.empty() && !variation_labels.empty()) 104 if (!other_labels.empty() && !variation_labels.empty())
103 combined_labels += google_update::kExperimentLabelSeparator; 105 combined_labels += google_update::kExperimentLabelSeparator;
104 combined_labels += variation_labels; 106 combined_labels += variation_labels;
105 return combined_labels; 107 return combined_labels;
106 } 108 }
107 109
108 } // namespace chrome_variations 110 } // namespace chrome_variations
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698