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

Side by Side Diff: components/variations/variations_experiment_util.cc

Issue 1291763002: Componentize kExperimentLabelSeparator & BuildExperimentDateString. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Try moving kExperimentLabels into the component to fix the build on Windows Created 5 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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/installer/util/google_update_experiment_util.h" 5 #include "components/variations/variations_experiment_util.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/stringprintf.h" 10 #include "base/strings/stringprintf.h"
11 #include "base/strings/utf_string_conversions.h" 11 #include "base/strings/utf_string_conversions.h"
12 #include "base/time/time.h" 12 #include "base/time/time.h"
13 13
14 namespace google_update { 14 namespace variations {
15 15
16 #if defined(OS_WIN) 16 #if defined(OS_WIN)
17 const wchar_t kExperimentLabels[] = L"experiment_labels"; 17 const wchar_t kExperimentLabels[] = L"experiment_labels";
18 #endif 18 #endif
19 19
20 const base::char16 kExperimentLabelSeparator = ';'; 20 const base::char16 kExperimentLabelSeparator = ';';
21 21
22 } // namespace google_update
23
24 namespace installer {
25
26 namespace { 22 namespace {
27 23
28 const char* const kDays[] = 24 const char* const kDays[] =
29 { "Sun", "Mon", "Tue", "Wed", "Thu", "Fri", "Sat" }; 25 { "Sun", "Mon", "Tue", "Wed", "Thu", "Fri", "Sat" };
30 26
31 const char* const kMonths[] = 27 const char* const kMonths[] =
32 { "Jan", "Feb", "Mar", "Apr", "May", "Jun", "Jul", "Aug", "Sep", 28 { "Jan", "Feb", "Mar", "Apr", "May", "Jun", "Jul", "Aug", "Sep",
33 "Oct", "Nov", "Dec"}; 29 "Oct", "Nov", "Dec"};
34 30
35 } 31 } // namespace
36 32
37 base::string16 BuildExperimentDateString(const base::Time& current_time) { 33 base::string16 BuildExperimentDateString(const base::Time& current_time) {
38 // The Google Update experiment_labels timestamp format is: 34 // The Google Update experiment_labels timestamp format is:
39 // "DAY, DD0 MON YYYY HH0:MI0:SE0 TZ" 35 // "DAY, DD0 MON YYYY HH0:MI0:SE0 TZ"
40 // DAY = 3 character day of week, 36 // DAY = 3 character day of week,
41 // DD0 = 2 digit day of month, 37 // DD0 = 2 digit day of month,
42 // MON = 3 character month of year, 38 // MON = 3 character month of year,
43 // YYYY = 4 digit year, 39 // YYYY = 4 digit year,
44 // HH0 = 2 digit hour, 40 // HH0 = 2 digit hour,
45 // MI0 = 2 digit minute, 41 // MI0 = 2 digit minute,
46 // SE0 = 2 digit second, 42 // SE0 = 2 digit second,
47 // TZ = 3 character timezone 43 // TZ = 3 character timezone
48 base::Time::Exploded then = {}; 44 base::Time::Exploded then = {};
49 current_time.UTCExplode(&then); 45 current_time.UTCExplode(&then);
50 then.year += 1; 46 then.year += 1;
51 DCHECK(then.HasValidValues()); 47 DCHECK(then.HasValidValues());
52 48
53 return base::UTF8ToUTF16( 49 return base::UTF8ToUTF16(
54 base::StringPrintf("%s, %02d %s %d %02d:%02d:%02d GMT", 50 base::StringPrintf("%s, %02d %s %d %02d:%02d:%02d GMT",
55 kDays[then.day_of_week], 51 kDays[then.day_of_week],
56 then.day_of_month, 52 then.day_of_month,
57 kMonths[then.month - 1], 53 kMonths[then.month - 1],
58 then.year, 54 then.year,
59 then.hour, 55 then.hour,
60 then.minute, 56 then.minute,
61 then.second)); 57 then.second));
62 } 58 }
63 59
64 } // namespace installer 60 } // namespace variations
65
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698