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

Side by Side Diff: chrome/installer/util/master_preferences_unittest.cc

Issue 12316097: Added master_preferences to control shortcuts on windows. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: fixed nested conditional Created 7 years, 9 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/installer/util/master_preferences_constants.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 // Unit tests for master preferences related methods. 5 // Unit tests for master preferences related methods.
6 6
7 #include "base/file_util.h" 7 #include "base/file_util.h"
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #include "base/path_service.h" 9 #include "base/path_service.h"
10 #include "base/stringprintf.h" 10 #include "base/stringprintf.h"
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 TEST_F(MasterPreferencesTest, ParseDistroParams) { 49 TEST_F(MasterPreferencesTest, ParseDistroParams) {
50 const char text[] = 50 const char text[] =
51 "{ \n" 51 "{ \n"
52 " \"distribution\": { \n" 52 " \"distribution\": { \n"
53 " \"show_welcome_page\": true,\n" 53 " \"show_welcome_page\": true,\n"
54 " \"import_search_engine\": true,\n" 54 " \"import_search_engine\": true,\n"
55 " \"import_history\": true,\n" 55 " \"import_history\": true,\n"
56 " \"import_bookmarks\": true,\n" 56 " \"import_bookmarks\": true,\n"
57 " \"import_bookmarks_from_file\": \"c:\\\\foo\",\n" 57 " \"import_bookmarks_from_file\": \"c:\\\\foo\",\n"
58 " \"import_home_page\": true,\n" 58 " \"import_home_page\": true,\n"
59 " \"do_not_create_any_shortcuts\": true,\n"
59 " \"do_not_create_desktop_shortcut\": true,\n" 60 " \"do_not_create_desktop_shortcut\": true,\n"
60 " \"do_not_create_quick_launch_shortcut\": true,\n" 61 " \"do_not_create_quick_launch_shortcut\": true,\n"
62 " \"do_not_create_taskbar_shortcut\": true,\n"
61 " \"do_not_launch_chrome\": true,\n" 63 " \"do_not_launch_chrome\": true,\n"
62 " \"make_chrome_default\": true,\n" 64 " \"make_chrome_default\": true,\n"
63 " \"make_chrome_default_for_user\": true,\n" 65 " \"make_chrome_default_for_user\": true,\n"
64 " \"system_level\": true,\n" 66 " \"system_level\": true,\n"
65 " \"verbose_logging\": true,\n" 67 " \"verbose_logging\": true,\n"
66 " \"require_eula\": true,\n" 68 " \"require_eula\": true,\n"
67 " \"alternate_shortcut_text\": true,\n" 69 " \"alternate_shortcut_text\": true,\n"
68 " \"chrome_shortcut_icon_index\": 1,\n" 70 " \"chrome_shortcut_icon_index\": 1,\n"
69 " \"ping_delay\": 40\n" 71 " \"ping_delay\": 40\n"
70 " },\n" 72 " },\n"
71 " \"blah\": {\n" 73 " \"blah\": {\n"
72 " \"import_history\": false\n" 74 " \"import_history\": false\n"
73 " }\n" 75 " }\n"
74 "} \n"; 76 "} \n";
75 77
76 EXPECT_TRUE(file_util::WriteFile(prefs_file(), text, strlen(text))); 78 EXPECT_TRUE(file_util::WriteFile(prefs_file(), text, strlen(text)));
77 installer::MasterPreferences prefs(prefs_file()); 79 installer::MasterPreferences prefs(prefs_file());
78 EXPECT_TRUE(prefs.read_from_file()); 80 EXPECT_TRUE(prefs.read_from_file());
79 81
80 const char* expected_true[] = { 82 const char* expected_true[] = {
81 installer::master_preferences::kDistroImportSearchPref, 83 installer::master_preferences::kDistroImportSearchPref,
82 installer::master_preferences::kDistroImportHistoryPref, 84 installer::master_preferences::kDistroImportHistoryPref,
83 installer::master_preferences::kDistroImportBookmarksPref, 85 installer::master_preferences::kDistroImportBookmarksPref,
84 installer::master_preferences::kDistroImportHomePagePref, 86 installer::master_preferences::kDistroImportHomePagePref,
87 installer::master_preferences::kDoNotCreateAnyShortcuts,
85 installer::master_preferences::kDoNotCreateDesktopShortcut, 88 installer::master_preferences::kDoNotCreateDesktopShortcut,
86 installer::master_preferences::kDoNotCreateQuickLaunchShortcut, 89 installer::master_preferences::kDoNotCreateQuickLaunchShortcut,
90 installer::master_preferences::kDoNotCreateTaskbarShortcut,
87 installer::master_preferences::kDoNotLaunchChrome, 91 installer::master_preferences::kDoNotLaunchChrome,
88 installer::master_preferences::kMakeChromeDefault, 92 installer::master_preferences::kMakeChromeDefault,
89 installer::master_preferences::kMakeChromeDefaultForUser, 93 installer::master_preferences::kMakeChromeDefaultForUser,
90 installer::master_preferences::kSystemLevel, 94 installer::master_preferences::kSystemLevel,
91 installer::master_preferences::kVerboseLogging, 95 installer::master_preferences::kVerboseLogging,
92 installer::master_preferences::kRequireEula, 96 installer::master_preferences::kRequireEula,
93 installer::master_preferences::kAltShortcutText, 97 installer::master_preferences::kAltShortcutText,
94 }; 98 };
95 99
96 for (int i = 0; i < arraysize(expected_true); ++i) { 100 for (int i = 0; i < arraysize(expected_true); ++i) {
(...skipping 254 matching lines...) Expand 10 before | Expand all | Expand 10 after
351 "{" 355 "{"
352 " \"distribution\": {" 356 " \"distribution\": {"
353 " \"create_all_shortcuts\": false" 357 " \"create_all_shortcuts\": false"
354 " }" 358 " }"
355 "}"; 359 "}";
356 360
357 installer::MasterPreferences prefs(kCreateAllShortcutsFalsePrefs); 361 installer::MasterPreferences prefs(kCreateAllShortcutsFalsePrefs);
358 362
359 bool do_not_create_desktop_shortcut = false; 363 bool do_not_create_desktop_shortcut = false;
360 bool do_not_create_quick_launch_shortcut = false; 364 bool do_not_create_quick_launch_shortcut = false;
365 bool do_not_create_taskbar_shortcut = false;
361 prefs.GetBool( 366 prefs.GetBool(
362 installer::master_preferences::kDoNotCreateDesktopShortcut, 367 installer::master_preferences::kDoNotCreateDesktopShortcut,
363 &do_not_create_desktop_shortcut); 368 &do_not_create_desktop_shortcut);
364 prefs.GetBool( 369 prefs.GetBool(
365 installer::master_preferences::kDoNotCreateQuickLaunchShortcut, 370 installer::master_preferences::kDoNotCreateQuickLaunchShortcut,
366 &do_not_create_quick_launch_shortcut); 371 &do_not_create_quick_launch_shortcut);
372 prefs.GetBool(
373 installer::master_preferences::kDoNotCreateTaskbarShortcut,
374 &do_not_create_taskbar_shortcut);
375 // create_all_shortcuts is a legacy preference that should only enforce
376 // do_not_create_desktop_shortcut and do_not_create_quick_launch_shortcut
377 // when set to false.
367 EXPECT_TRUE(do_not_create_desktop_shortcut); 378 EXPECT_TRUE(do_not_create_desktop_shortcut);
368 EXPECT_TRUE(do_not_create_quick_launch_shortcut); 379 EXPECT_TRUE(do_not_create_quick_launch_shortcut);
380 EXPECT_FALSE(do_not_create_taskbar_shortcut);
369 } 381 }
370 382
371 TEST_F(MasterPreferencesTest, DontEnforceLegacyCreateAllShortcutsTrue) { 383 TEST_F(MasterPreferencesTest, DontEnforceLegacyCreateAllShortcutsTrue) {
372 static const char kCreateAllShortcutsFalsePrefs[] = 384 static const char kCreateAllShortcutsFalsePrefs[] =
373 "{" 385 "{"
374 " \"distribution\": {" 386 " \"distribution\": {"
375 " \"create_all_shortcuts\": true" 387 " \"create_all_shortcuts\": true"
376 " }" 388 " }"
377 "}"; 389 "}";
378 390
379 installer::MasterPreferences prefs(kCreateAllShortcutsFalsePrefs); 391 installer::MasterPreferences prefs(kCreateAllShortcutsFalsePrefs);
380 392
381 bool do_not_create_desktop_shortcut = false; 393 bool do_not_create_desktop_shortcut = false;
382 bool do_not_create_quick_launch_shortcut = false; 394 bool do_not_create_quick_launch_shortcut = false;
395 bool do_not_create_taskbar_shortcut = false;
383 prefs.GetBool( 396 prefs.GetBool(
384 installer::master_preferences::kDoNotCreateDesktopShortcut, 397 installer::master_preferences::kDoNotCreateDesktopShortcut,
385 &do_not_create_desktop_shortcut); 398 &do_not_create_desktop_shortcut);
386 prefs.GetBool( 399 prefs.GetBool(
387 installer::master_preferences::kDoNotCreateQuickLaunchShortcut, 400 installer::master_preferences::kDoNotCreateQuickLaunchShortcut,
388 &do_not_create_quick_launch_shortcut); 401 &do_not_create_quick_launch_shortcut);
402 prefs.GetBool(
403 installer::master_preferences::kDoNotCreateTaskbarShortcut,
404 &do_not_create_taskbar_shortcut);
389 EXPECT_FALSE(do_not_create_desktop_shortcut); 405 EXPECT_FALSE(do_not_create_desktop_shortcut);
390 EXPECT_FALSE(do_not_create_quick_launch_shortcut); 406 EXPECT_FALSE(do_not_create_quick_launch_shortcut);
407 EXPECT_FALSE(do_not_create_taskbar_shortcut);
391 } 408 }
392 409
393 TEST_F(MasterPreferencesTest, DontEnforceLegacyCreateAllShortcutsNotSpecified) { 410 TEST_F(MasterPreferencesTest, DontEnforceLegacyCreateAllShortcutsNotSpecified) {
394 static const char kCreateAllShortcutsFalsePrefs[] = 411 static const char kCreateAllShortcutsFalsePrefs[] =
395 "{" 412 "{"
396 " \"distribution\": {" 413 " \"distribution\": {"
397 " \"some_other_pref\": true" 414 " \"some_other_pref\": true"
398 " }" 415 " }"
399 "}"; 416 "}";
400 417
401 installer::MasterPreferences prefs(kCreateAllShortcutsFalsePrefs); 418 installer::MasterPreferences prefs(kCreateAllShortcutsFalsePrefs);
402 419
403 bool do_not_create_desktop_shortcut = false; 420 bool do_not_create_desktop_shortcut = false;
404 bool do_not_create_quick_launch_shortcut = false; 421 bool do_not_create_quick_launch_shortcut = false;
422 bool do_not_create_taskbar_shortcut = false;
405 prefs.GetBool( 423 prefs.GetBool(
406 installer::master_preferences::kDoNotCreateDesktopShortcut, 424 installer::master_preferences::kDoNotCreateDesktopShortcut,
407 &do_not_create_desktop_shortcut); 425 &do_not_create_desktop_shortcut);
408 prefs.GetBool( 426 prefs.GetBool(
409 installer::master_preferences::kDoNotCreateQuickLaunchShortcut, 427 installer::master_preferences::kDoNotCreateQuickLaunchShortcut,
410 &do_not_create_quick_launch_shortcut); 428 &do_not_create_quick_launch_shortcut);
429 prefs.GetBool(
430 installer::master_preferences::kDoNotCreateTaskbarShortcut,
431 &do_not_create_taskbar_shortcut);
411 EXPECT_FALSE(do_not_create_desktop_shortcut); 432 EXPECT_FALSE(do_not_create_desktop_shortcut);
412 EXPECT_FALSE(do_not_create_quick_launch_shortcut); 433 EXPECT_FALSE(do_not_create_quick_launch_shortcut);
434 EXPECT_FALSE(do_not_create_taskbar_shortcut);
413 } 435 }
OLDNEW
« no previous file with comments | « chrome/installer/util/master_preferences_constants.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698