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

Side by Side Diff: components/variations/proto/study.proto

Issue 370463003: Add support for variations controlled overriding UI strings to the variations service. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: asvitkine's first round of comments Created 6 years, 5 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 (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 syntax = "proto2"; 5 syntax = "proto2";
6 6
7 option optimize_for = LITE_RUNTIME; 7 option optimize_for = LITE_RUNTIME;
8 8
9 package chrome_variations; 9 package chrome_variations;
10 10
(...skipping 25 matching lines...) Expand all
36 // Ex: PERMANENT 36 // Ex: PERMANENT
37 optional Consistency consistency = 7 [default = SESSION]; 37 optional Consistency consistency = 7 [default = SESSION];
38 38
39 // Name of the experiment that gets the default experience. This experiment 39 // Name of the experiment that gets the default experience. This experiment
40 // must be included in the list below. 40 // must be included in the list below.
41 // Ex: "default" 41 // Ex: "default"
42 optional string default_experiment_name = 8; 42 optional string default_experiment_name = 8;
43 43
44 // An experiment within the study. 44 // An experiment within the study.
45 // 45 //
46 // Next tag: 9 46 // Next tag: 10
47 message Experiment { 47 message Experiment {
48 // A named parameter value for this experiment. 48 // A named parameter value for this experiment.
49 // 49 //
50 // Next tag: 3 50 // Next tag: 3
51 message Param { 51 message Param {
52 // The name of the parameter. 52 // The name of the parameter.
53 optional string name = 1; 53 optional string name = 1;
54 54
55 // The value of the parameter. 55 // The value of the parameter.
56 optional string value = 2; 56 optional string value = 2;
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
100 KILL_BEST_EFFORT = 2; 100 KILL_BEST_EFFORT = 2;
101 101
102 // This is a kill-switch group that should be killed with "critical" 102 // This is a kill-switch group that should be killed with "critical"
103 // priority. Depending on platform this may result in showing a 103 // priority. Depending on platform this may result in showing a
104 // non-dismissible restart prompt with a timer. This should only be used 104 // non-dismissible restart prompt with a timer. This should only be used
105 // in very serious emergency circumstances. The experiment must have a 105 // in very serious emergency circumstances. The experiment must have a
106 // probability_weight of 0. 106 // probability_weight of 0.
107 KILL_CRITICAL = 3; 107 KILL_CRITICAL = 3;
108 } 108 }
109 optional Type type = 7 [default = NORMAL]; 109 optional Type type = 7 [default = NORMAL];
110
111 // A UI string to override, and the new value to use. |name_hash| is the
112 // hash of the resource name to be overridden.
113 message OverrideUIString {
114 // The first 32 bits of the MD5 hash digest of the resource name to
115 // override.
116 optional fixed32 name_hash = 1; // e.g. Hash("IDS_BOOKMARK_BAR_UNDO")
Alexei Svitkine (slow) 2014/07/15 16:44:52 Nit: Move this line comment to be at the end of th
jwd 2014/07/16 21:33:05 Done.
117 optional string value = 2; // e.g. "Undo"
Alexei Svitkine (slow) 2014/07/15 16:44:52 Nit: Put comment above the field instead with a sh
jwd 2014/07/16 21:33:05 Done.
118 }
119 repeated OverrideUIString override_ui_string = 9;
110 } 120 }
111 121
112 // List of experiments in this study. This list should include the default / 122 // List of experiments in this study. This list should include the default /
113 // control experiment. 123 // control experiment.
114 // 124 //
115 // For example, to specify that 99% of users get the default behavior, while 125 // For example, to specify that 99% of users get the default behavior, while
116 // 0.5% of users get experience "A" and 0.5% of users get experience "B", 126 // 0.5% of users get experience "A" and 0.5% of users get experience "B",
117 // specify the values below. 127 // specify the values below.
118 // Ex: { "default": 990, "A": 5, "B": 5 } 128 // Ex: { "default": 990, "A": 5, "B": 5 }
119 repeated Experiment experiment = 9; 129 repeated Experiment experiment = 9;
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
223 // This is recommended for most studies that include client code. 233 // This is recommended for most studies that include client code.
224 ACTIVATION_EXPLICIT = 0; 234 ACTIVATION_EXPLICIT = 0;
225 // The study will be automatically activated when it is created. This 235 // The study will be automatically activated when it is created. This
226 // is recommended for studies that do not have any client logic. 236 // is recommended for studies that do not have any client logic.
227 ACTIVATION_AUTO = 1; 237 ACTIVATION_AUTO = 1;
228 } 238 }
229 239
230 // Activation type for this study. Defaults to ACTIVATION_EXPLICIT if omitted. 240 // Activation type for this study. Defaults to ACTIVATION_EXPLICIT if omitted.
231 optional ActivationType activation_type = 12; 241 optional ActivationType activation_type = 12;
232 } 242 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698