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

Unified Diff: chrome/browser/browsing_data_file_system_helper.cc

Issue 10805015: Move browsing_data_helper files into a separate directory. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix chrome_frame build Created 8 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/browsing_data_file_system_helper.cc
diff --git a/chrome/browser/browsing_data_file_system_helper.cc b/chrome/browser/browsing_data_file_system_helper.cc
deleted file mode 100644
index 2f4e2ed5e3e78a13753e2b2b23b39f3bd0063d0b..0000000000000000000000000000000000000000
--- a/chrome/browser/browsing_data_file_system_helper.cc
+++ /dev/null
@@ -1,283 +0,0 @@
-// Copyright (c) 2012 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 "chrome/browser/browsing_data_file_system_helper.h"
-
-#include "base/bind.h"
-#include "base/compiler_specific.h"
-#include "base/file_util.h"
-#include "base/memory/scoped_ptr.h"
-#include "base/message_loop.h"
-#include "base/string_util.h"
-#include "base/utf_string_conversions.h"
-#include "chrome/browser/browsing_data_helper.h"
-#include "chrome/browser/profiles/profile.h"
-#include "content/public/browser/browser_thread.h"
-#include "webkit/fileapi/file_system_context.h"
-#include "webkit/fileapi/file_system_quota_util.h"
-#include "webkit/fileapi/file_system_types.h"
-#include "webkit/fileapi/sandbox_mount_point_provider.h"
-
-using content::BrowserContext;
-using content::BrowserThread;
-
-namespace {
-
-// An implementation of the BrowsingDataFileSystemHelper interface that pulls
-// data from a given |profile| and returns a list of FileSystemInfo items to a
-// client.
-class BrowsingDataFileSystemHelperImpl : public BrowsingDataFileSystemHelper {
- public:
- // BrowsingDataFileSystemHelper implementation
- explicit BrowsingDataFileSystemHelperImpl(Profile* profile);
- virtual void StartFetching(const base::Callback<
- void(const std::list<FileSystemInfo>&)>& callback) OVERRIDE;
- virtual void DeleteFileSystemOrigin(const GURL& origin) OVERRIDE;
-
- private:
- virtual ~BrowsingDataFileSystemHelperImpl();
-
- // Enumerates all filesystem files, storing the resulting list into
- // file_system_file_ for later use. This must be called on the FILE thread.
- void FetchFileSystemInfoInFileThread();
-
- // Triggers the success callback as the end of a StartFetching workflow. This
- // must be called on the UI thread.
- void NotifyOnUIThread();
-
- // Deletes all file systems associated with |origin|. This must be called on
- // the FILE thread.
- void DeleteFileSystemOriginInFileThread(const GURL& origin);
-
- // Keep a reference to the FileSystemContext object for the current profile
- // for use on the FILE thread.
- scoped_refptr<fileapi::FileSystemContext> filesystem_context_;
-
- // Holds the current list of file systems returned to the client after
- // StartFetching is called. Access to |file_system_info_| is triggered
- // indirectly via the UI thread and guarded by |is_fetching_|. This means
- // |file_system_info_| is only accessed while |is_fetching_| is true. The
- // flag |is_fetching_| is only accessed on the UI thread. In the context of
- // this class |file_system_info_| only mutates on the FILE thread.
- std::list<FileSystemInfo> file_system_info_;
-
- // Holds the callback passed in at the beginning of the StartFetching workflow
- // so that it can be triggered via NotifyOnUIThread. This only mutates on the
- // UI thread.
- base::Callback<void(const std::list<FileSystemInfo>&)> completion_callback_;
-
- // Indicates whether or not we're currently fetching information: set to true
- // when StartFetching is called on the UI thread, and reset to false when
- // NotifyOnUIThread triggers the success callback.
- // This property only mutates on the UI thread.
- bool is_fetching_;
-
- DISALLOW_COPY_AND_ASSIGN(BrowsingDataFileSystemHelperImpl);
-};
-
-BrowsingDataFileSystemHelperImpl::BrowsingDataFileSystemHelperImpl(
- Profile* profile)
- : filesystem_context_(BrowserContext::GetFileSystemContext(profile)),
- is_fetching_(false) {
- DCHECK(filesystem_context_);
-}
-
-BrowsingDataFileSystemHelperImpl::~BrowsingDataFileSystemHelperImpl() {
-}
-
-void BrowsingDataFileSystemHelperImpl::StartFetching(
- const base::Callback<void(const std::list<FileSystemInfo>&)>& callback) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- DCHECK(!is_fetching_);
- DCHECK_EQ(false, callback.is_null());
- is_fetching_ = true;
- completion_callback_ = callback;
- BrowserThread::PostTask(
- BrowserThread::FILE, FROM_HERE,
- base::Bind(
- &BrowsingDataFileSystemHelperImpl::FetchFileSystemInfoInFileThread,
- this));
-}
-
-void BrowsingDataFileSystemHelperImpl::DeleteFileSystemOrigin(
- const GURL& origin) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- BrowserThread::PostTask(
- BrowserThread::FILE, FROM_HERE,
- base::Bind(
- &BrowsingDataFileSystemHelperImpl::DeleteFileSystemOriginInFileThread,
- this, origin));
-}
-
-void BrowsingDataFileSystemHelperImpl::FetchFileSystemInfoInFileThread() {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
- scoped_ptr<fileapi::SandboxMountPointProvider::OriginEnumerator>
- origin_enumerator(filesystem_context_->
- sandbox_provider()->CreateOriginEnumerator());
-
- // We don't own this pointer; it's a magic singleton generated by the
- // profile's FileSystemContext. Deleting it would be a bad idea.
- fileapi::FileSystemQuotaUtil* quota_util =
- filesystem_context_->GetQuotaUtil(fileapi::kFileSystemTypeTemporary);
-
- GURL current;
-
- while (!(current = origin_enumerator->Next()).is_empty()) {
- if (!BrowsingDataHelper::HasWebScheme(current))
- continue; // Non-websafe state is not considered browsing data.
-
- // We can call these synchronous methods as we've already verified that
- // we're running on the FILE thread.
- int64 persistent_usage = quota_util->GetOriginUsageOnFileThread(
- filesystem_context_, current,
- fileapi::kFileSystemTypePersistent);
- int64 temporary_usage = quota_util->GetOriginUsageOnFileThread(
- filesystem_context_, current,
- fileapi::kFileSystemTypeTemporary);
- file_system_info_.push_back(
- FileSystemInfo(
- current,
- origin_enumerator->HasFileSystemType(
- fileapi::kFileSystemTypePersistent),
- origin_enumerator->HasFileSystemType(
- fileapi::kFileSystemTypeTemporary),
- persistent_usage,
- temporary_usage));
- }
-
- BrowserThread::PostTask(
- BrowserThread::UI, FROM_HERE,
- base::Bind(&BrowsingDataFileSystemHelperImpl::NotifyOnUIThread, this));
-}
-
-void BrowsingDataFileSystemHelperImpl::NotifyOnUIThread() {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- DCHECK(is_fetching_);
- completion_callback_.Run(file_system_info_);
- completion_callback_.Reset();
- is_fetching_ = false;
-}
-
-void BrowsingDataFileSystemHelperImpl::DeleteFileSystemOriginInFileThread(
- const GURL& origin) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
- filesystem_context_->DeleteDataForOriginOnFileThread(origin);
-}
-
-} // namespace
-
-BrowsingDataFileSystemHelper::FileSystemInfo::FileSystemInfo(
- const GURL& origin,
- bool has_persistent,
- bool has_temporary,
- int64 usage_persistent,
- int64 usage_temporary)
- : origin(origin),
- has_persistent(has_persistent),
- has_temporary(has_temporary),
- usage_persistent(usage_persistent),
- usage_temporary(usage_temporary) {
-}
-
-BrowsingDataFileSystemHelper::FileSystemInfo::~FileSystemInfo() {}
-
-// static
-BrowsingDataFileSystemHelper* BrowsingDataFileSystemHelper::Create(
- Profile* profile) {
- return new BrowsingDataFileSystemHelperImpl(profile);
-}
-
-CannedBrowsingDataFileSystemHelper::CannedBrowsingDataFileSystemHelper(
- Profile* profile)
- : is_fetching_(false) {
-}
-
-CannedBrowsingDataFileSystemHelper::CannedBrowsingDataFileSystemHelper()
- : is_fetching_(false) {
-}
-
-CannedBrowsingDataFileSystemHelper::~CannedBrowsingDataFileSystemHelper() {}
-
-CannedBrowsingDataFileSystemHelper*
- CannedBrowsingDataFileSystemHelper::Clone() {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- CannedBrowsingDataFileSystemHelper* clone =
- new CannedBrowsingDataFileSystemHelper();
- // This list only mutates on the UI thread, so it's safe to work with it here
- // (given the DCHECK above).
- clone->file_system_info_ = file_system_info_;
- return clone;
-}
-
-void CannedBrowsingDataFileSystemHelper::AddFileSystem(
- const GURL& origin, const fileapi::FileSystemType type, const int64 size) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- // This canned implementation of AddFileSystem uses an O(n^2) algorithm; which
- // is fine, as it isn't meant for use in a high-volume context. If it turns
- // out that we want to start using this in a context with many, many origins,
- // we should think about reworking the implementation.
- bool duplicate_origin = false;
- for (std::list<FileSystemInfo>::iterator
- file_system = file_system_info_.begin();
- file_system != file_system_info_.end();
- ++file_system) {
- if (file_system->origin == origin) {
- if (type == fileapi::kFileSystemTypePersistent) {
- file_system->has_persistent = true;
- file_system->usage_persistent = size;
- } else {
- file_system->has_temporary = true;
- file_system->usage_temporary = size;
- }
- duplicate_origin = true;
- break;
- }
- }
- if (duplicate_origin)
- return;
-
- if (!BrowsingDataHelper::HasWebScheme(origin))
- return; // Non-websafe state is not considered browsing data.
-
- file_system_info_.push_back(FileSystemInfo(
- origin,
- (type == fileapi::kFileSystemTypePersistent),
- (type == fileapi::kFileSystemTypeTemporary),
- (type == fileapi::kFileSystemTypePersistent) ? size : 0,
- (type == fileapi::kFileSystemTypeTemporary) ? size : 0));
-}
-
-void CannedBrowsingDataFileSystemHelper::Reset() {
- file_system_info_.clear();
-}
-
-bool CannedBrowsingDataFileSystemHelper::empty() const {
- return file_system_info_.empty();
-}
-
-size_t CannedBrowsingDataFileSystemHelper::GetFileSystemCount() const {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- return file_system_info_.size();
-}
-
-void CannedBrowsingDataFileSystemHelper::StartFetching(
- const base::Callback<void(const std::list<FileSystemInfo>&)>& callback) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- DCHECK(!is_fetching_);
- DCHECK_EQ(false, callback.is_null());
- is_fetching_ = true;
- completion_callback_ = callback;
-
- BrowserThread::PostTask(
- BrowserThread::UI, FROM_HERE,
- base::Bind(&CannedBrowsingDataFileSystemHelper::NotifyOnUIThread, this));
-}
-
-void CannedBrowsingDataFileSystemHelper::NotifyOnUIThread() {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- DCHECK(is_fetching_);
- completion_callback_.Run(file_system_info_);
- completion_callback_.Reset();
- is_fetching_ = false;
-}
« no previous file with comments | « chrome/browser/browsing_data_file_system_helper.h ('k') | chrome/browser/browsing_data_file_system_helper_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698