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

Unified Diff: chrome/browser/file_system/file_system_dispatcher_host.cc

Issue 5698008: Switch a bunch of remaining filters to derive from BrowserMessageFilters so t... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years 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
« no previous file with comments | « chrome/browser/file_system/file_system_dispatcher_host.h ('k') | chrome/browser/mime_registry_dispatcher.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/file_system/file_system_dispatcher_host.cc
===================================================================
--- chrome/browser/file_system/file_system_dispatcher_host.cc (revision 68877)
+++ chrome/browser/file_system/file_system_dispatcher_host.cc (working copy)
@@ -10,7 +10,6 @@
#include "base/file_path.h"
#include "base/thread.h"
#include "base/time.h"
-#include "chrome/browser/browser_thread.h"
#include "chrome/browser/content_settings/host_content_settings_map.h"
#include "chrome/browser/file_system/browser_file_system_context.h"
#include "chrome/browser/net/chrome_url_request_context.h"
@@ -82,37 +81,25 @@
int request_id_;
};
-FileSystemDispatcherHost::FileSystemDispatcherHost(
- IPC::Message::Sender* sender, Profile* profile)
- : message_sender_(sender),
- process_handle_(0),
- shutdown_(false),
- context_(profile->GetFileSystemContext()),
+FileSystemDispatcherHost::FileSystemDispatcherHost(Profile* profile)
+ : context_(profile->GetFileSystemContext()),
host_content_settings_map_(profile->GetHostContentSettingsMap()),
request_context_getter_(profile->GetRequestContext()) {
- DCHECK(message_sender_);
}
FileSystemDispatcherHost::FileSystemDispatcherHost(
- IPC::Message::Sender* sender, ChromeURLRequestContext* context)
- : message_sender_(sender),
- process_handle_(0),
- shutdown_(false),
- context_(context->browser_file_system_context()),
+ ChromeURLRequestContext* context)
+ : context_(context->browser_file_system_context()),
host_content_settings_map_(context->host_content_settings_map()),
request_context_(context) {
- DCHECK(message_sender_);
}
FileSystemDispatcherHost::~FileSystemDispatcherHost() {
}
-void FileSystemDispatcherHost::Init(base::ProcessHandle process_handle) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
- DCHECK(!shutdown_);
- DCHECK(!process_handle_);
- DCHECK(process_handle);
- process_handle_ = process_handle;
+void FileSystemDispatcherHost::OnChannelConnected(int32 peer_pid) {
+ BrowserMessageFilter::OnChannelConnected(peer_pid);
+
if (request_context_getter_.get()) {
DCHECK(!request_context_.get());
request_context_ = request_context_getter_->GetURLRequestContext();
@@ -120,14 +107,8 @@
DCHECK(request_context_.get());
}
-void FileSystemDispatcherHost::Shutdown() {
- message_sender_ = NULL;
- shutdown_ = true;
-}
-
bool FileSystemDispatcherHost::OnMessageReceived(
const IPC::Message& message, bool* message_was_ok) {
- DCHECK(!shutdown_);
*message_was_ok = true;
bool handled = true;
IPC_BEGIN_MESSAGE_MAP_EX(FileSystemDispatcherHost, message, *message_was_ok)
@@ -251,14 +232,6 @@
}
}
-void FileSystemDispatcherHost::Send(IPC::Message* message) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
- if (!shutdown_ && message_sender_)
- message_sender_->Send(message);
- else
- delete message;
-}
-
SandboxedFileSystemOperation* FileSystemDispatcherHost::GetNewOperation(
int request_id) {
BrowserFileSystemCallbackDispatcher* dispatcher =
« no previous file with comments | « chrome/browser/file_system/file_system_dispatcher_host.h ('k') | chrome/browser/mime_registry_dispatcher.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698