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

Side by Side Diff: chrome/browser/chromeos/file_manager/file_manager_browsertest.cc

Issue 287083003: Multi-profile flag cleanup (Closed) Base URL: https://chromium.googlesource.com/chromium/src
Patch Set: Created 6 years, 7 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 (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 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 // Browser test for basic Chrome OS file manager functionality: 5 // Browser test for basic Chrome OS file manager functionality:
6 // - The file list is updated when a file is added externally to the Downloads 6 // - The file list is updated when a file is added externally to the Downloads
7 // folder. 7 // folder.
8 // - Selecting a file and copy-pasting it with the keyboard copies the file. 8 // - Selecting a file and copy-pasting it with the keyboard copies the file.
9 // - Selecting a file and pressing delete deletes it. 9 // - Selecting a file and pressing delete deletes it.
10 10
(...skipping 889 matching lines...) Expand 10 before | Expand all | Expand 10 after
900 {"bob@invalid.domain", "hashbob", "Bob"}, 900 {"bob@invalid.domain", "hashbob", "Bob"},
901 {"charlie@invalid.domain", "hashcharlie", "Charlie"}, 901 {"charlie@invalid.domain", "hashcharlie", "Charlie"},
902 }; 902 };
903 903
904 // Test fixture class for testing multi-profile features. 904 // Test fixture class for testing multi-profile features.
905 class MultiProfileFileManagerBrowserTest : public FileManagerBrowserTestBase { 905 class MultiProfileFileManagerBrowserTest : public FileManagerBrowserTestBase {
906 protected: 906 protected:
907 // Enables multi-profiles. 907 // Enables multi-profiles.
908 virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE { 908 virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
909 FileManagerBrowserTestBase::SetUpCommandLine(command_line); 909 FileManagerBrowserTestBase::SetUpCommandLine(command_line);
910 command_line->AppendSwitch(switches::kMultiProfiles);
911 // Logs in to a dummy profile (For making MultiProfileWindowManager happy; 910 // Logs in to a dummy profile (For making MultiProfileWindowManager happy;
912 // browser test creates a default window and the manager tries to assign a 911 // browser test creates a default window and the manager tries to assign a
913 // user for it, and we need a profile connected to a user.) 912 // user for it, and we need a profile connected to a user.)
914 command_line->AppendSwitchASCII(chromeos::switches::kLoginUser, 913 command_line->AppendSwitchASCII(chromeos::switches::kLoginUser,
915 kTestAccounts[DUMMY_ACCOUNT_INDEX].email); 914 kTestAccounts[DUMMY_ACCOUNT_INDEX].email);
916 command_line->AppendSwitchASCII(chromeos::switches::kLoginProfile, 915 command_line->AppendSwitchASCII(chromeos::switches::kLoginProfile,
917 kTestAccounts[DUMMY_ACCOUNT_INDEX].hash); 916 kTestAccounts[DUMMY_ACCOUNT_INDEX].hash);
918 } 917 }
919 918
920 // Logs in to the primary profile of this test. 919 // Logs in to the primary profile of this test.
(...skipping 143 matching lines...) Expand 10 before | Expand all | Expand 10 after
1064 // (bob and charlie) are added in the test. Thus the existing test verifies 1063 // (bob and charlie) are added in the test. Thus the existing test verifies
1065 // that the feature is effectively working with lastly logged in users. 1064 // that the feature is effectively working with lastly logged in users.
1066 AddExtraUsersForStressTesting(); 1065 AddExtraUsersForStressTesting();
1067 1066
1068 set_test_case_name("multiProfileVisitDesktopMenu"); 1067 set_test_case_name("multiProfileVisitDesktopMenu");
1069 StartTest(); 1068 StartTest();
1070 } 1069 }
1071 1070
1072 } // namespace 1071 } // namespace
1073 } // namespace file_manager 1072 } // namespace file_manager
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698