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

Unified Diff: extensions/browser/api/file_handlers/directory_util.cc

Issue 2685883008: Revert of Move file_handlers API from //chrome to //extensions (Closed)
Patch Set: Created 3 years, 10 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: extensions/browser/api/file_handlers/directory_util.cc
diff --git a/extensions/browser/api/file_handlers/directory_util.cc b/extensions/browser/api/file_handlers/directory_util.cc
deleted file mode 100644
index 516be8f872be520002cb7a3ed3678644bfceda2d..0000000000000000000000000000000000000000
--- a/extensions/browser/api/file_handlers/directory_util.cc
+++ /dev/null
@@ -1,110 +0,0 @@
-// Copyright 2016 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "extensions/browser/api/file_handlers/directory_util.h"
-
-#include "base/files/file_path.h"
-#include "base/files/file_util.h"
-#include "base/threading/thread_task_runner_handle.h"
-#include "content/public/browser/browser_context.h"
-#include "content/public/browser/browser_thread.h"
-#include "net/base/filename_util.h"
-#include "storage/browser/fileapi/file_system_url.h"
-
-#if defined(OS_CHROMEOS)
-#include "extensions/browser/api/extensions_api_client.h"
-#include "extensions/browser/api/file_handlers/non_native_file_system_delegate.h"
-#endif
-
-namespace extensions {
-namespace app_file_handler_util {
-
-namespace {
-
-void GetIsDirectoryFromFileInfo(const base::FilePath& path,
- bool* is_directory) {
- base::File::Info file_info;
- *is_directory = GetFileInfo(path, &file_info) && file_info.is_directory;
-}
-
-void OnGetIsDirectoryFromFileInfoCompleted(
- std::unique_ptr<bool> is_directory,
- const base::Callback<void(bool)>& callback) {
- callback.Run(*is_directory);
-}
-
-// The callback parameter contains the result and is required to support
-// both native local directories to avoid UI thread and non native local
-// path directories for the IsNonNativeLocalPathDirectory API.
-void EntryIsDirectory(content::BrowserContext* context,
- const base::FilePath& path,
- const base::Callback<void(bool)>& callback) {
-#if defined(OS_CHROMEOS)
- NonNativeFileSystemDelegate* delegate =
- ExtensionsAPIClient::Get()->GetNonNativeFileSystemDelegate();
- if (delegate && delegate->IsUnderNonNativeLocalPath(context, path)) {
- delegate->IsNonNativeLocalPathDirectory(context, path, callback);
- return;
- }
-#endif
-
- std::unique_ptr<bool> is_directory(new bool);
- bool* const is_directory_ptr = is_directory.get();
-
- content::BrowserThread::PostBlockingPoolTaskAndReply(
- FROM_HERE,
- base::Bind(&GetIsDirectoryFromFileInfo, path, is_directory_ptr),
- base::Bind(&OnGetIsDirectoryFromFileInfoCompleted,
- base::Passed(&is_directory), callback));
-}
-
-} // namespace
-
-IsDirectoryCollector::IsDirectoryCollector(content::BrowserContext* context)
- : context_(context), left_(0), weak_ptr_factory_(this) {}
-
-IsDirectoryCollector::~IsDirectoryCollector() {}
-
-void IsDirectoryCollector::CollectForEntriesPaths(
- const std::vector<base::FilePath>& paths,
- const CompletionCallback& callback) {
- DCHECK(!callback.is_null());
- paths_ = paths;
- callback_ = callback;
-
- DCHECK(!result_.get());
- result_.reset(new std::set<base::FilePath>());
- left_ = paths.size();
-
- if (!left_) {
- // Nothing to process.
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::Bind(callback_, base::Passed(&result_)));
- callback_ = CompletionCallback();
- return;
- }
-
- for (size_t i = 0; i < paths.size(); ++i) {
- EntryIsDirectory(context_, paths[i],
- base::Bind(&IsDirectoryCollector::OnIsDirectoryCollected,
- weak_ptr_factory_.GetWeakPtr(), i));
- }
-}
-
-void IsDirectoryCollector::OnIsDirectoryCollected(size_t index,
- bool is_directory) {
- if (is_directory)
- result_->insert(paths_[index]);
- if (!--left_) {
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::Bind(callback_, base::Passed(&result_)));
- // Release the callback to avoid a circullar reference in case an instance
- // of this class is a member of a ref counted class, which instance is bound
- // to this callback.
- callback_ = CompletionCallback();
- }
-}
-
-} // namespace app_file_handler_util
-} // namespace extensions
« no previous file with comments | « extensions/browser/api/file_handlers/directory_util.h ('k') | extensions/browser/api/file_handlers/directory_util_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698