Invalid FileSystemURL should not be given to FileUtils

webkit/fileapi/isolated_file_util_unittest.cc

Index: webkit/fileapi/isolated_file_util_unittest.cc
diff --git a/webkit/fileapi/isolated_file_util_unittest.cc b/webkit/fileapi/isolated_file_util_unittest.cc
index d3e6303aab73e8323ce15579644f4036de4e4f23..03953c87dcb83fc5f11c395967ec694134c5a12b 100644
--- a/webkit/fileapi/isolated_file_util_unittest.cc
+++ b/webkit/fileapi/isolated_file_util_unittest.cc
@@ -20,6 +20,7 @@
 #include "webkit/fileapi/file_util_helper.h"
 #include "webkit/fileapi/isolated_context.h"
 #include "webkit/fileapi/isolated_file_util.h"
+#include "webkit/fileapi/local_file_system_operation.h"
 #include "webkit/fileapi/local_file_system_test_helper.h"
 #include "webkit/fileapi/local_file_util.h"
 #include "webkit/fileapi/mock_file_system_options.h"
@@ -299,13 +300,11 @@ TEST_F(IsolatedFileUtilTest, UnregisteredPathsTest) {
     const test::TestCaseRecord& test_case = kUnregisteredCases[i];
     FileSystemURL url = GetFileSystemURL(FilePath(test_case.path));
 
-    // This should fail as the paths in kUnregisteredCases are not included
-    // in the dropped files (i.e. the regular test cases).
-    base::PlatformFileInfo info;
-    FilePath platform_path;
-    FileSystemOperationContext context(file_system_context());
-    ASSERT_EQ(base::PLATFORM_FILE_ERROR_NOT_FOUND,
-              file_util()->GetFileInfo(&context, url, &info, &platform_path));
+    // We should not be able to get the valid URL for unregistered files.
+    ASSERT_FALSE(url.is_valid());
+
+    // We should not be able to create a new operation for an invalid URL.
+    ASSERT_EQ(NULL, file_system_context()->CreateFileSystemOperation(url));
   }
 }