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

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

Issue 1019393004: [MemSheriff] Disable a flaky CrOS test under MSan. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 | « no previous file | 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) 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 1256 matching lines...) Expand 10 before | Expand all | Expand 10 after
1267 MAYBE_BasicDownloads) { 1267 MAYBE_BasicDownloads) {
1268 AddAllUsers(); 1268 AddAllUsers();
1269 1269
1270 // Sanity check that normal operations work in multi-profile setting as well. 1270 // Sanity check that normal operations work in multi-profile setting as well.
1271 set_test_case_name("keyboardCopyDownloads"); 1271 set_test_case_name("keyboardCopyDownloads");
1272 StartTest(); 1272 StartTest();
1273 } 1273 }
1274 1274
1275 // Slow tests are disabled on debug build. http://crbug.com/327719 1275 // Slow tests are disabled on debug build. http://crbug.com/327719
1276 // Fails on official build. http://crbug.com/429294 1276 // Fails on official build. http://crbug.com/429294
1277 #if !defined(NDEBUG) || defined(OFFICIAL_BUILD) 1277 // Disabled under MSAN as well. http://crbug.com/468982.
1278 #if !defined(NDEBUG) || defined(OFFICIAL_BUILD) || defined(MEMORY_SANITIZER)
1278 #define MAYBE_PRE_BasicDrive DISABLED_PRE_BasicDrive 1279 #define MAYBE_PRE_BasicDrive DISABLED_PRE_BasicDrive
1279 #define MAYBE_BasicDrive DISABLED_BasicDrive 1280 #define MAYBE_BasicDrive DISABLED_BasicDrive
1280 #else 1281 #else
1281 #define MAYBE_PRE_BasicDrive PRE_BasicDrive 1282 #define MAYBE_PRE_BasicDrive PRE_BasicDrive
1282 #define MAYBE_BasicDrive BasicDrive 1283 #define MAYBE_BasicDrive BasicDrive
1283 #endif 1284 #endif
1284 IN_PROC_BROWSER_TEST_F(MultiProfileFileManagerBrowserTest, 1285 IN_PROC_BROWSER_TEST_F(MultiProfileFileManagerBrowserTest,
1285 MAYBE_PRE_BasicDrive) { 1286 MAYBE_PRE_BasicDrive) {
1286 AddAllUsers(); 1287 AddAllUsers();
1287 } 1288 }
(...skipping 195 matching lines...) Expand 10 before | Expand all | Expand 10 after
1483 StartTest(); 1484 StartTest();
1484 } 1485 }
1485 1486
1486 IN_PROC_BROWSER_TEST_F(VideoPlayerBrowserTest, OpenSingleVideoOnDrive) { 1487 IN_PROC_BROWSER_TEST_F(VideoPlayerBrowserTest, OpenSingleVideoOnDrive) {
1487 set_test_case_name("openSingleVideoOnDrive"); 1488 set_test_case_name("openSingleVideoOnDrive");
1488 StartTest(); 1489 StartTest();
1489 } 1490 }
1490 1491
1491 } // namespace 1492 } // namespace
1492 } // namespace file_manager 1493 } // namespace file_manager
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698