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

Unified Diff: chrome/browser/chromeos/file_system_provider/operations/open_file_unittest.cc

Issue 1221093002: Add support for refreshing contents of providers which don't support watchers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed tests. Created 5 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/file_system_provider/operations/open_file_unittest.cc
diff --git a/chrome/browser/chromeos/file_system_provider/operations/open_file_unittest.cc b/chrome/browser/chromeos/file_system_provider/operations/open_file_unittest.cc
index f6bd6d9ba99be15d2fb37b2a216ccdb7390ac3fb..725054770b67f4e461bf199789269f592add961f 100644
--- a/chrome/browser/chromeos/file_system_provider/operations/open_file_unittest.cc
+++ b/chrome/browser/chromeos/file_system_provider/operations/open_file_unittest.cc
@@ -75,7 +75,8 @@ class FileSystemProviderOperationsOpenFileTest : public testing::Test {
void SetUp() override {
file_system_info_ = ProvidedFileSystemInfo(
kExtensionId, MountOptions(kFileSystemId, "" /* display_name */),
- base::FilePath(), false /* configurable */, extensions::SOURCE_FILE);
+ base::FilePath(), false /* configurable */, true /* watchable */,
+ extensions::SOURCE_FILE);
}
ProvidedFileSystemInfo file_system_info_;
@@ -139,7 +140,7 @@ TEST_F(FileSystemProviderOperationsOpenFileTest, Execute_ReadOnly) {
const ProvidedFileSystemInfo read_only_file_system_info(
kExtensionId, MountOptions(kFileSystemId, "" /* display_name */),
base::FilePath() /* mount_path */, false /* configurable */,
- extensions::SOURCE_FILE);
+ true /* watchable */, extensions::SOURCE_FILE);
// Opening for read on a read-only file system is allowed.
{

Powered by Google App Engine
This is Rietveld 408576698