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

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

Issue 15975004: Replace sets with vectors when storing file handlers. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed comments. Created 7 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 | Annotate | Revision Log
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 775 matching lines...) Expand 10 before | Expand all | Expand 10 after
786 IN_PROC_BROWSER_TEST_P(FileManagerBrowserLocalTest, TestFileDisplay) { 786 IN_PROC_BROWSER_TEST_P(FileManagerBrowserLocalTest, TestFileDisplay) {
787 PrepareVolume(); 787 PrepareVolume();
788 DoTestFileDisplay(&volume_); 788 DoTestFileDisplay(&volume_);
789 } 789 }
790 790
791 IN_PROC_BROWSER_TEST_P(FileManagerBrowserLocalTest, TestGalleryOpen) { 791 IN_PROC_BROWSER_TEST_P(FileManagerBrowserLocalTest, TestGalleryOpen) {
792 PrepareVolume(); 792 PrepareVolume();
793 DoTestGalleryOpen(&volume_); 793 DoTestGalleryOpen(&volume_);
794 } 794 }
795 795
796 // Disabled temporarily since fails on Linux Chromium OS ASAN Tests (2). 796 IN_PROC_BROWSER_TEST_P(FileManagerBrowserDriveTest, TestGalleryOpen) {
797 // TODO(mtomasz): crbug.com/243611.
798 IN_PROC_BROWSER_TEST_P(FileManagerBrowserDriveTest, DISABLED_TestGalleryOpen) {
799 PrepareVolume(); 797 PrepareVolume();
800 DoTestGalleryOpen(&volume_); 798 DoTestGalleryOpen(&volume_);
801 } 799 }
802 800
803 IN_PROC_BROWSER_TEST_P(FileManagerBrowserDriveTest, TestKeyboardCopy) { 801 IN_PROC_BROWSER_TEST_P(FileManagerBrowserDriveTest, TestKeyboardCopy) {
804 PrepareVolume(); 802 PrepareVolume();
805 DoTestKeyboardCopy(&volume_); 803 DoTestKeyboardCopy(&volume_);
806 } 804 }
807 805
808 IN_PROC_BROWSER_TEST_P(FileManagerBrowserDriveTest, TestKeyboardDelete) { 806 IN_PROC_BROWSER_TEST_P(FileManagerBrowserDriveTest, TestKeyboardDelete) {
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
899 // TODO(hirono): Bring back the offline feature. http://crbug.com/238545 897 // TODO(hirono): Bring back the offline feature. http://crbug.com/238545
900 IN_PROC_BROWSER_TEST_P(FileManagerBrowserTransferTest, 898 IN_PROC_BROWSER_TEST_P(FileManagerBrowserTransferTest,
901 DISABLED_TransferFromOfflineToDrive) { 899 DISABLED_TransferFromOfflineToDrive) {
902 PrepareVolume(); 900 PrepareVolume();
903 ResultCatcher catcher; 901 ResultCatcher catcher;
904 StartTest("transferFromOfflineToDrive"); 902 StartTest("transferFromOfflineToDrive");
905 ASSERT_TRUE(catcher.GetNextResult()) << catcher.message(); 903 ASSERT_TRUE(catcher.GetNextResult()) << catcher.message();
906 } 904 }
907 905
908 } // namespace 906 } // namespace
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698