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

Side by Side Diff: chrome/browser/about_flags.cc

Issue 1488303005: Replace multi-value keyboard flags with single value flag types. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add new flags to histograms. Created 5 years 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/app/generated_resources.grd ('k') | tools/metrics/histograms/histograms.xml » ('j') | 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 #include "chrome/browser/about_flags.h" 5 #include "chrome/browser/about_flags.h"
6 6
7 #include <iterator> 7 #include <iterator>
8 #include <map> 8 #include <map>
9 #include <set> 9 #include <set>
10 #include <utility> 10 #include <utility>
(...skipping 378 matching lines...) Expand 10 before | Expand all | Expand 10 after
389 { IDS_GENERIC_EXPERIMENT_CHOICE_DEFAULT, "", "" }, 389 { IDS_GENERIC_EXPERIMENT_CHOICE_DEFAULT, "", "" },
390 { IDS_GENERIC_EXPERIMENT_CHOICE_ENABLED, 390 { IDS_GENERIC_EXPERIMENT_CHOICE_ENABLED,
391 chromeos::switches::kEnableDataSaverPrompt, "" }, 391 chromeos::switches::kEnableDataSaverPrompt, "" },
392 { IDS_GENERIC_EXPERIMENT_CHOICE_DISABLED, 392 { IDS_GENERIC_EXPERIMENT_CHOICE_DISABLED,
393 chromeos::switches::kDisableDataSaverPrompt, "" }, 393 chromeos::switches::kDisableDataSaverPrompt, "" },
394 { IDS_FLAGS_DATASAVER_PROMPT_DEMO_MODE, 394 { IDS_FLAGS_DATASAVER_PROMPT_DEMO_MODE,
395 chromeos::switches::kEnableDataSaverPrompt, 395 chromeos::switches::kEnableDataSaverPrompt,
396 chromeos::switches::kDataSaverPromptDemoMode }, 396 chromeos::switches::kDataSaverPromptDemoMode },
397 }; 397 };
398 398
399 const FeatureEntry::Choice kFloatingVirtualKeyboardChoices[] = {
400 { IDS_GENERIC_EXPERIMENT_CHOICE_DEFAULT, "", "" },
401 { IDS_GENERIC_EXPERIMENT_CHOICE_DISABLED,
402 keyboard::switches::kFloatingVirtualKeyboard,
403 keyboard::switches::kFloatingVirtualKeyboardDisabled},
404 { IDS_GENERIC_EXPERIMENT_CHOICE_ENABLED,
405 keyboard::switches::kFloatingVirtualKeyboard,
406 keyboard::switches::kFloatingVirtualKeyboardEnabled},
407 };
408
409 const FeatureEntry::Choice kSmartVirtualKeyboardChoices[] = {
410 { IDS_GENERIC_EXPERIMENT_CHOICE_DEFAULT, "", "" },
411 { IDS_GENERIC_EXPERIMENT_CHOICE_DISABLED,
412 keyboard::switches::kSmartVirtualKeyboard,
413 keyboard::switches::kSmartVirtualKeyboardDisabled},
414 { IDS_GENERIC_EXPERIMENT_CHOICE_ENABLED,
415 keyboard::switches::kSmartVirtualKeyboard,
416 keyboard::switches::kSmartVirtualKeyboardEnabled},
417 };
418
419 const FeatureEntry::Choice kGestureTypingChoices[] = {
420 { IDS_GENERIC_EXPERIMENT_CHOICE_DEFAULT, "", "" },
421 { IDS_GENERIC_EXPERIMENT_CHOICE_DISABLED,
422 keyboard::switches::kGestureTyping,
423 keyboard::switches::kGestureTypingDisabled},
424 { IDS_GENERIC_EXPERIMENT_CHOICE_ENABLED,
425 keyboard::switches::kGestureTyping,
426 keyboard::switches::kGestureTypingEnabled},
427 };
428
429 const FeatureEntry::Choice kGestureEditingChoices[] = {
430 { IDS_GENERIC_EXPERIMENT_CHOICE_DEFAULT, "", "" },
431 { IDS_GENERIC_EXPERIMENT_CHOICE_DISABLED,
432 keyboard::switches::kGestureEditing,
433 keyboard::switches::kGestureEditingDisabled},
434 { IDS_GENERIC_EXPERIMENT_CHOICE_ENABLED,
435 keyboard::switches::kGestureEditing,
436 keyboard::switches::kGestureEditingEnabled},
437 };
438
439 const FeatureEntry::Choice kDownloadNotificationChoices[] = { 399 const FeatureEntry::Choice kDownloadNotificationChoices[] = {
440 { IDS_GENERIC_EXPERIMENT_CHOICE_DEFAULT, "", "" }, 400 { IDS_GENERIC_EXPERIMENT_CHOICE_DEFAULT, "", "" },
441 { IDS_GENERIC_EXPERIMENT_CHOICE_ENABLED, 401 { IDS_GENERIC_EXPERIMENT_CHOICE_ENABLED,
442 switches::kEnableDownloadNotification, 402 switches::kEnableDownloadNotification,
443 "enabled" }, 403 "enabled" },
444 { IDS_GENERIC_EXPERIMENT_CHOICE_DISABLED, 404 { IDS_GENERIC_EXPERIMENT_CHOICE_DISABLED,
445 switches::kEnableDownloadNotification, 405 switches::kEnableDownloadNotification,
446 "disabled" } 406 "disabled" }
447 }; 407 };
448 #endif 408 #endif
(...skipping 730 matching lines...) Expand 10 before | Expand all | Expand 10 after
1179 IDS_FLAGS_GESTURE_REQUIREMENT_FOR_MEDIA_PLAYBACK_DESCRIPTION, 1139 IDS_FLAGS_GESTURE_REQUIREMENT_FOR_MEDIA_PLAYBACK_DESCRIPTION,
1180 kOsAll, 1140 kOsAll,
1181 SINGLE_DISABLE_VALUE_TYPE( 1141 SINGLE_DISABLE_VALUE_TYPE(
1182 switches::kDisableGestureRequirementForMediaPlayback)}, 1142 switches::kDisableGestureRequirementForMediaPlayback)},
1183 #if defined(OS_CHROMEOS) 1143 #if defined(OS_CHROMEOS)
1184 {"enable-virtual-keyboard", 1144 {"enable-virtual-keyboard",
1185 IDS_FLAGS_VIRTUAL_KEYBOARD_NAME, 1145 IDS_FLAGS_VIRTUAL_KEYBOARD_NAME,
1186 IDS_FLAGS_VIRTUAL_KEYBOARD_DESCRIPTION, 1146 IDS_FLAGS_VIRTUAL_KEYBOARD_DESCRIPTION,
1187 kOsCrOS, 1147 kOsCrOS,
1188 SINGLE_VALUE_TYPE(keyboard::switches::kEnableVirtualKeyboard)}, 1148 SINGLE_VALUE_TYPE(keyboard::switches::kEnableVirtualKeyboard)},
1189 {"enable-virtual-keyboard-overscroll", 1149 {"virtual-keyboard-overscroll",
1190 IDS_FLAGS_VIRTUAL_KEYBOARD_OVERSCROLL_NAME, 1150 IDS_FLAGS_VIRTUAL_KEYBOARD_OVERSCROLL_NAME,
1191 IDS_FLAGS_VIRTUAL_KEYBOARD_OVERSCROLL_DESCRIPTION, 1151 IDS_FLAGS_VIRTUAL_KEYBOARD_OVERSCROLL_DESCRIPTION,
1192 kOsCrOS, 1152 kOsCrOS,
1193 ENABLE_DISABLE_VALUE_TYPE( 1153 SINGLE_DISABLE_VALUE_TYPE(
1194 keyboard::switches::kEnableVirtualKeyboardOverscroll,
1195 keyboard::switches::kDisableVirtualKeyboardOverscroll)}, 1154 keyboard::switches::kDisableVirtualKeyboardOverscroll)},
1196 {"enable-input-view", 1155 {"input-view",
1197 IDS_FLAGS_INPUT_VIEW_NAME, 1156 IDS_FLAGS_INPUT_VIEW_NAME,
1198 IDS_FLAGS_INPUT_VIEW_DESCRIPTION, 1157 IDS_FLAGS_INPUT_VIEW_DESCRIPTION,
1199 kOsCrOS, 1158 kOsCrOS,
1200 ENABLE_DISABLE_VALUE_TYPE(keyboard::switches::kEnableInputView, 1159 SINGLE_DISABLE_VALUE_TYPE(keyboard::switches::kDisableInputView)},
1201 keyboard::switches::kDisableInputView)},
1202 {"disable-new-korean-ime", 1160 {"disable-new-korean-ime",
1203 IDS_FLAGS_NEW_KOREAN_IME_NAME, 1161 IDS_FLAGS_NEW_KOREAN_IME_NAME,
1204 IDS_FLAGS_NEW_KOREAN_IME_DESCRIPTION, 1162 IDS_FLAGS_NEW_KOREAN_IME_DESCRIPTION,
1205 kOsCrOS, 1163 kOsCrOS,
1206 SINGLE_DISABLE_VALUE_TYPE(chromeos::switches::kDisableNewKoreanIme)}, 1164 SINGLE_DISABLE_VALUE_TYPE(chromeos::switches::kDisableNewKoreanIme)},
1207 {"enable-physical-keyboard-autocorrect", 1165 {"enable-physical-keyboard-autocorrect",
1208 IDS_FLAGS_PHYSICAL_KEYBOARD_AUTOCORRECT_NAME, 1166 IDS_FLAGS_PHYSICAL_KEYBOARD_AUTOCORRECT_NAME,
1209 IDS_FLAGS_PHYSICAL_KEYBOARD_AUTOCORRECT_DESCRIPTION, 1167 IDS_FLAGS_PHYSICAL_KEYBOARD_AUTOCORRECT_DESCRIPTION,
1210 kOsCrOS, 1168 kOsCrOS,
1211 ENABLE_DISABLE_VALUE_TYPE( 1169 ENABLE_DISABLE_VALUE_TYPE(
1212 chromeos::switches::kEnablePhysicalKeyboardAutocorrect, 1170 chromeos::switches::kEnablePhysicalKeyboardAutocorrect,
1213 chromeos::switches::kDisablePhysicalKeyboardAutocorrect)}, 1171 chromeos::switches::kDisablePhysicalKeyboardAutocorrect)},
1214 {"disable-voice-input", 1172 {"disable-voice-input",
1215 IDS_FLAGS_VOICE_INPUT_NAME, 1173 IDS_FLAGS_VOICE_INPUT_NAME,
1216 IDS_FLAGS_VOICE_INPUT_DESCRIPTION, 1174 IDS_FLAGS_VOICE_INPUT_DESCRIPTION,
1217 kOsCrOS, 1175 kOsCrOS,
1218 SINGLE_DISABLE_VALUE_TYPE(keyboard::switches::kDisableVoiceInput)}, 1176 SINGLE_DISABLE_VALUE_TYPE(keyboard::switches::kDisableVoiceInput)},
1219 {"enable-experimental-input-view-features", 1177 {"experimental-input-view-features",
1220 IDS_FLAGS_EXPERIMENTAL_INPUT_VIEW_FEATURES_NAME, 1178 IDS_FLAGS_EXPERIMENTAL_INPUT_VIEW_FEATURES_NAME,
1221 IDS_FLAGS_EXPERIMENTAL_INPUT_VIEW_FEATURES_DESCRIPTION, 1179 IDS_FLAGS_EXPERIMENTAL_INPUT_VIEW_FEATURES_DESCRIPTION,
1222 kOsCrOS, 1180 kOsCrOS,
1223 SINGLE_VALUE_TYPE( 1181 SINGLE_VALUE_TYPE(
1224 keyboard::switches::kEnableExperimentalInputViewFeatures)}, 1182 keyboard::switches::kEnableExperimentalInputViewFeatures)},
1225 {"floating-virtual-keyboard", 1183 {"floating-virtual-keyboard",
1226 IDS_FLAGS_FLOATING_VIRTUAL_KEYBOARD_NAME, 1184 IDS_FLAGS_FLOATING_VIRTUAL_KEYBOARD_NAME,
1227 IDS_FLAGS_FLOATING_VIRTUAL_KEYBOARD_DESCRIPTION, 1185 IDS_FLAGS_FLOATING_VIRTUAL_KEYBOARD_DESCRIPTION,
1228 kOsCrOS, 1186 kOsCrOS,
1229 MULTI_VALUE_TYPE(kFloatingVirtualKeyboardChoices)}, 1187 SINGLE_VALUE_TYPE(keyboard::switches::kEnableFloatingVirtualKeyboard)},
1230 {"smart-virtual-keyboard", 1188 {"smart-virtual-keyboard",
1231 IDS_FLAGS_SMART_VIRTUAL_KEYBOARD_NAME, 1189 IDS_FLAGS_SMART_VIRTUAL_KEYBOARD_NAME,
1232 IDS_FLAGS_SMART_VIRTUAL_KEYBOARD_DESCRIPTION, 1190 IDS_FLAGS_SMART_VIRTUAL_KEYBOARD_DESCRIPTION,
1233 kOsCrOS, 1191 kOsCrOS,
1234 MULTI_VALUE_TYPE(kSmartVirtualKeyboardChoices)}, 1192 SINGLE_DISABLE_VALUE_TYPE(
1193 keyboard::switches::kDisableSmartVirtualKeyboard)},
1235 {"gesture-typing", 1194 {"gesture-typing",
1236 IDS_FLAGS_GESTURE_TYPING_NAME, 1195 IDS_FLAGS_GESTURE_TYPING_NAME,
1237 IDS_FLAGS_GESTURE_TYPING_DESCRIPTION, 1196 IDS_FLAGS_GESTURE_TYPING_DESCRIPTION,
1238 kOsCrOS, 1197 kOsCrOS,
1239 MULTI_VALUE_TYPE(kGestureTypingChoices)}, 1198 SINGLE_DISABLE_VALUE_TYPE(keyboard::switches::kDisableGestureTyping)},
1240 {"gesture-editing", 1199 {"gesture-editing",
1241 IDS_FLAGS_GESTURE_EDITING_NAME, 1200 IDS_FLAGS_GESTURE_EDITING_NAME,
1242 IDS_FLAGS_GESTURE_EDITING_DESCRIPTION, 1201 IDS_FLAGS_GESTURE_EDITING_DESCRIPTION,
1243 kOsCrOS, 1202 kOsCrOS,
1244 MULTI_VALUE_TYPE(kGestureEditingChoices)}, 1203 SINGLE_DISABLE_VALUE_TYPE(keyboard::switches::kDisableGestureEditing)},
1245 {"enable-fullscreen-app-list", 1204 {"enable-fullscreen-app-list",
1246 IDS_FLAGS_FULLSCREEN_APP_LIST_NAME, 1205 IDS_FLAGS_FULLSCREEN_APP_LIST_NAME,
1247 IDS_FLAGS_FULLSCREEN_APP_LIST_DESCRIPTION, 1206 IDS_FLAGS_FULLSCREEN_APP_LIST_DESCRIPTION,
1248 kOsCrOS, 1207 kOsCrOS,
1249 SINGLE_VALUE_TYPE(ash::switches::kAshEnableFullscreenAppList)}, 1208 SINGLE_VALUE_TYPE(ash::switches::kAshEnableFullscreenAppList)},
1250 #endif 1209 #endif
1251 {"enable-simple-cache-backend", 1210 {"enable-simple-cache-backend",
1252 IDS_FLAGS_SIMPLE_CACHE_BACKEND_NAME, 1211 IDS_FLAGS_SIMPLE_CACHE_BACKEND_NAME,
1253 IDS_FLAGS_SIMPLE_CACHE_BACKEND_DESCRIPTION, 1212 IDS_FLAGS_SIMPLE_CACHE_BACKEND_DESCRIPTION,
1254 kOsWin | kOsMac | kOsLinux | kOsCrOS, 1213 kOsWin | kOsMac | kOsLinux | kOsCrOS,
(...skipping 1054 matching lines...) Expand 10 before | Expand all | Expand 10 after
2309 const base::HistogramBase::Sample kBadSwitchFormatHistogramId = 0; 2268 const base::HistogramBase::Sample kBadSwitchFormatHistogramId = 0;
2310 2269
2311 const FeatureEntry* GetFeatureEntries(size_t* count) { 2270 const FeatureEntry* GetFeatureEntries(size_t* count) {
2312 *count = arraysize(kFeatureEntries); 2271 *count = arraysize(kFeatureEntries);
2313 return kFeatureEntries; 2272 return kFeatureEntries;
2314 } 2273 }
2315 2274
2316 } // namespace testing 2275 } // namespace testing
2317 2276
2318 } // namespace about_flags 2277 } // namespace about_flags
OLDNEW
« no previous file with comments | « chrome/app/generated_resources.grd ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698