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

Unified Diff: webkit/browser/fileapi/copy_or_move_file_validator_unittest.cc

Issue 15729007: Cleanup: Remove fileapi::FileSystemMountPointProvider::InitializeCopyOrMoveFileValidatorFactory() a… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 7 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 side-by-side diff with in-line comments
Download patch
Index: webkit/browser/fileapi/copy_or_move_file_validator_unittest.cc
===================================================================
--- webkit/browser/fileapi/copy_or_move_file_validator_unittest.cc (revision 202646)
+++ webkit/browser/fileapi/copy_or_move_file_validator_unittest.cc (working copy)
@@ -81,10 +81,9 @@
void SetMediaCopyOrMoveFileValidatorFactory(
scoped_ptr<CopyOrMoveFileValidatorFactory> factory) {
- FileSystemMountPointProvider* mount_point_provider =
- file_system_context_->GetMountPointProvider(kWithValidatorType);
- mount_point_provider->InitializeCopyOrMoveFileValidatorFactory(
- kWithValidatorType, factory.Pass());
+ TestMountPointProvider* provider = static_cast<TestMountPointProvider*>(
+ file_system_context_->GetMountPointProvider(kWithValidatorType));
+ provider->InitializeCopyOrMoveFileValidatorFactory(factory.Pass());
}
void CopyTest(base::PlatformFileError expected) {
@@ -173,7 +172,8 @@
: public CopyOrMoveFileValidatorFactory {
public:
// A factory that creates validators that accept everything or nothing.
- TestCopyOrMoveFileValidatorFactory(bool all_valid) : all_valid_(all_valid) {}
+ explicit TestCopyOrMoveFileValidatorFactory(bool all_valid)
+ : all_valid_(all_valid) {}
virtual ~TestCopyOrMoveFileValidatorFactory() {}
virtual CopyOrMoveFileValidator* CreateCopyOrMoveFileValidator(
@@ -185,7 +185,7 @@
private:
class TestCopyOrMoveFileValidator : public CopyOrMoveFileValidator {
public:
- TestCopyOrMoveFileValidator(bool all_valid)
+ explicit TestCopyOrMoveFileValidator(bool all_valid)
: result_(all_valid ? base::PLATFORM_FILE_OK
: base::PLATFORM_FILE_ERROR_SECURITY) {
}

Powered by Google App Engine
This is Rietveld 408576698