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

Side by Side Diff: chrome/browser/chromeos/file_system_provider/service_unittest.cc

Issue 1137383002: Show the eject button only for removabled and file handlers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed tests. Created 5 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #include "chrome/browser/chromeos/file_system_provider/service.h" 5 #include "chrome/browser/chromeos/file_system_provider/service.h"
6 6
7 #include <string> 7 #include <string>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/files/file.h" 10 #include "base/files/file.h"
11 #include "base/memory/ref_counted.h" 11 #include "base/memory/ref_counted.h"
12 #include "base/memory/scoped_ptr.h" 12 #include "base/memory/scoped_ptr.h"
13 #include "base/strings/string_number_conversions.h" 13 #include "base/strings/string_number_conversions.h"
14 #include "chrome/browser/chromeos/file_system_provider/fake_provided_file_system .h" 14 #include "chrome/browser/chromeos/file_system_provider/fake_provided_file_system .h"
15 #include "chrome/browser/chromeos/file_system_provider/mount_path_util.h" 15 #include "chrome/browser/chromeos/file_system_provider/mount_path_util.h"
16 #include "chrome/browser/chromeos/file_system_provider/observer.h" 16 #include "chrome/browser/chromeos/file_system_provider/observer.h"
17 #include "chrome/browser/chromeos/file_system_provider/provided_file_system_info .h" 17 #include "chrome/browser/chromeos/file_system_provider/provided_file_system_info .h"
18 #include "chrome/browser/chromeos/file_system_provider/registry_interface.h" 18 #include "chrome/browser/chromeos/file_system_provider/registry_interface.h"
19 #include "chrome/browser/chromeos/login/users/fake_chrome_user_manager.h" 19 #include "chrome/browser/chromeos/login/users/fake_chrome_user_manager.h"
20 #include "chrome/browser/chromeos/login/users/scoped_user_manager_enabler.h" 20 #include "chrome/browser/chromeos/login/users/scoped_user_manager_enabler.h"
21 #include "chrome/common/extensions/api/file_system_provider_capabilities/file_sy stem_provider_capabilities_handler.h"
21 #include "chrome/test/base/testing_browser_process.h" 22 #include "chrome/test/base/testing_browser_process.h"
22 #include "chrome/test/base/testing_pref_service_syncable.h" 23 #include "chrome/test/base/testing_pref_service_syncable.h"
23 #include "chrome/test/base/testing_profile.h" 24 #include "chrome/test/base/testing_profile.h"
24 #include "chrome/test/base/testing_profile_manager.h" 25 #include "chrome/test/base/testing_profile_manager.h"
25 #include "components/user_prefs/user_prefs.h" 26 #include "components/user_prefs/user_prefs.h"
26 #include "content/public/test/test_browser_thread_bundle.h" 27 #include "content/public/test/test_browser_thread_bundle.h"
27 #include "extensions/browser/extension_registry.h" 28 #include "extensions/browser/extension_registry.h"
28 #include "extensions/common/extension.h" 29 #include "extensions/common/extension.h"
29 #include "extensions/common/manifest_constants.h" 30 #include "extensions/common/manifest_constants.h"
30 #include "storage/browser/fileapi/external_mount_points.h" 31 #include "storage/browser/fileapi/external_mount_points.h"
(...skipping 377 matching lines...) Expand 10 before | Expand all | Expand 10 after
408 } 409 }
409 410
410 TEST_F(FileSystemProviderServiceTest, RestoreFileSystem_OnExtensionLoad) { 411 TEST_F(FileSystemProviderServiceTest, RestoreFileSystem_OnExtensionLoad) {
411 LoggingObserver observer; 412 LoggingObserver observer;
412 service_->AddObserver(&observer); 413 service_->AddObserver(&observer);
413 414
414 // Remember a fake file system first in order to be able to restore it. 415 // Remember a fake file system first in order to be able to restore it.
415 MountOptions options(kFileSystemId, kDisplayName); 416 MountOptions options(kFileSystemId, kDisplayName);
416 options.supports_notify_tag = true; 417 options.supports_notify_tag = true;
417 ProvidedFileSystemInfo file_system_info( 418 ProvidedFileSystemInfo file_system_info(
418 kExtensionId, options, base::FilePath(FILE_PATH_LITERAL("/a/b/c"))); 419 kExtensionId, options, base::FilePath(FILE_PATH_LITERAL("/a/b/c")),
420 false /* configurable */, extensions::SOURCE_FILE);
419 Watchers fake_watchers; 421 Watchers fake_watchers;
420 fake_watchers[WatcherKey(fake_watcher_.entry_path, fake_watcher_.recursive)] = 422 fake_watchers[WatcherKey(fake_watcher_.entry_path, fake_watcher_.recursive)] =
421 fake_watcher_; 423 fake_watcher_;
422 registry_->RememberFileSystem(file_system_info, fake_watchers); 424 registry_->RememberFileSystem(file_system_info, fake_watchers);
423 425
424 EXPECT_EQ(0u, observer.mounts.size()); 426 EXPECT_EQ(0u, observer.mounts.size());
425 427
426 // Directly call the observer's method. 428 // Directly call the observer's method.
427 service_->OnExtensionLoaded(profile_, extension_.get()); 429 service_->OnExtensionLoaded(profile_, extension_.get());
428 430
(...skipping 109 matching lines...) Expand 10 before | Expand all | Expand 10 after
538 EXPECT_EQ(1u, observer.unmounts.size()); 540 EXPECT_EQ(1u, observer.unmounts.size());
539 EXPECT_FALSE(registry_->file_system_info()); 541 EXPECT_FALSE(registry_->file_system_info());
540 EXPECT_FALSE(registry_->watchers()); 542 EXPECT_FALSE(registry_->watchers());
541 } 543 }
542 544
543 service_->RemoveObserver(&observer); 545 service_->RemoveObserver(&observer);
544 } 546 }
545 547
546 } // namespace file_system_provider 548 } // namespace file_system_provider
547 } // namespace chromeos 549 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698