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

Side by Side Diff: content/child/child_message_filter.cc

Issue 504273002: Remove implicit conversions from scoped_refptr to T* in content/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "content/child/child_message_filter.h" 5 #include "content/child/child_message_filter.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/bind_helpers.h" 8 #include "base/bind_helpers.h"
9 #include "base/location.h" 9 #include "base/location.h"
10 #include "base/task_runner.h" 10 #include "base/task_runner.h"
11 #include "content/child/child_thread.h" 11 #include "content/child/child_thread.h"
12 #include "content/child/thread_safe_sender.h" 12 #include "content/child/thread_safe_sender.h"
13 #include "ipc/message_filter.h" 13 #include "ipc/message_filter.h"
14 14
15 namespace content { 15 namespace content {
16 16
17 class ChildMessageFilter::Internal : public IPC::MessageFilter { 17 class ChildMessageFilter::Internal : public IPC::MessageFilter {
18 public: 18 public:
19 explicit Internal(ChildMessageFilter* filter) : filter_(filter) {} 19 explicit Internal(ChildMessageFilter* filter) : filter_(filter) {}
20 20
21 virtual bool OnMessageReceived(const IPC::Message& msg) OVERRIDE { 21 virtual bool OnMessageReceived(const IPC::Message& msg) OVERRIDE {
22 scoped_refptr<base::TaskRunner> runner = 22 scoped_refptr<base::TaskRunner> runner =
23 filter_->OverrideTaskRunnerForMessage(msg); 23 filter_->OverrideTaskRunnerForMessage(msg);
24 if (runner && !runner->RunsTasksOnCurrentThread()) { 24 if (runner.get() && !runner->RunsTasksOnCurrentThread()) {
25 if (!runner->PostTask( 25 if (!runner->PostTask(
26 FROM_HERE, 26 FROM_HERE,
27 base::Bind( 27 base::Bind(
28 base::IgnoreResult(&ChildMessageFilter::OnMessageReceived), 28 base::IgnoreResult(&ChildMessageFilter::OnMessageReceived),
29 filter_, msg))) 29 filter_, msg)))
30 filter_->OnStaleMessageReceived(msg); 30 filter_->OnStaleMessageReceived(msg);
31 return true; 31 return true;
32 } 32 }
33 33
34 return filter_->OnMessageReceived(msg); 34 return filter_->OnMessageReceived(msg);
(...skipping 21 matching lines...) Expand all
56 56
57 ChildMessageFilter::~ChildMessageFilter() {} 57 ChildMessageFilter::~ChildMessageFilter() {}
58 58
59 IPC::MessageFilter* ChildMessageFilter::GetFilter() { 59 IPC::MessageFilter* ChildMessageFilter::GetFilter() {
60 if (!internal_) 60 if (!internal_)
61 internal_ = new Internal(this); 61 internal_ = new Internal(this);
62 return internal_; 62 return internal_;
63 } 63 }
64 64
65 } // namespace content 65 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/web_contents/web_contents_view_aura_browsertest.cc ('k') | content/child/npapi/plugin_url_fetcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698