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

Unified Diff: third_party/WebKit/LayoutTests/fast/filesystem/cross-filesystem-op.html

Issue 2537193003: Make fast/filesystem/cross-filesystem-op.html insensitive to test order (Closed)
Patch Set: Make a few more tests clean up when done Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/fast/filesystem/cross-filesystem-op.html
diff --git a/third_party/WebKit/LayoutTests/fast/filesystem/cross-filesystem-op.html b/third_party/WebKit/LayoutTests/fast/filesystem/cross-filesystem-op.html
index 5c1f8f753406f903f284af8a82b6021a9b80ae85..63aca2967ab6683f38518fedfe5a43e25166b005 100644
--- a/third_party/WebKit/LayoutTests/fast/filesystem/cross-filesystem-op.html
+++ b/third_party/WebKit/LayoutTests/fast/filesystem/cross-filesystem-op.html
@@ -1,5 +1,6 @@
<html>
<body>
+<script src="resources/fs-test-util.js"></script>
<div>This tests basic cross-filesystem operations.</div>
<div id="destination" style="min-height:100px; border: solid 1px black">Drop files here if you test this manually</div>
@@ -58,14 +59,16 @@ function errorCallback(msg)
function setupDestFileSystem(successCallback) {
webkitRequestFileSystem(window.TEMPORARY, 1024, function (fs) {
- fs.root.getDirectory(
- destDirectoryPath,
- {create:true},
- function (entry) {
- destTestDirectory = entry;
- successCallback();
- },
- errorCallback('dest: createDirectory(' + destDirectoryPath + ')'));
+ removeAllInDirectory(fs.root, function() {
+ fs.root.getDirectory(
+ destDirectoryPath,
+ {create:true},
+ function (entry) {
+ destTestDirectory = entry;
+ successCallback();
+ },
+ errorCallback('dest: createDirectory(' + destDirectoryPath + ')'));
jeffcarp 2016/11/29 21:38:28 Just curious, are ES2015 template strings OK for l
jsbell 2016/11/29 21:41:15 Sure.
+ }, errorCallback('dest: removeAllInDirectory'));
}, errorCallback('dest: requestFileSystem for PERSISTENT'));
}

Powered by Google App Engine
This is Rietveld 408576698