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

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

Issue 671663002: Standardize usage of virtual/override/final in content/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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
« no previous file with comments | « content/child/child_message_filter.h ('k') | content/child/child_resource_message_filter.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 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.get() && !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);
35 } 35 }
36 36
37 private: 37 private:
38 virtual ~Internal() {} 38 ~Internal() override {}
39 scoped_refptr<ChildMessageFilter> filter_; 39 scoped_refptr<ChildMessageFilter> filter_;
40 40
41 DISALLOW_COPY_AND_ASSIGN(Internal); 41 DISALLOW_COPY_AND_ASSIGN(Internal);
42 }; 42 };
43 43
44 bool ChildMessageFilter::Send(IPC::Message* message) { 44 bool ChildMessageFilter::Send(IPC::Message* message) {
45 return thread_safe_sender_->Send(message); 45 return thread_safe_sender_->Send(message);
46 } 46 }
47 47
48 base::TaskRunner* ChildMessageFilter::OverrideTaskRunnerForMessage( 48 base::TaskRunner* ChildMessageFilter::OverrideTaskRunnerForMessage(
49 const IPC::Message& msg) { 49 const IPC::Message& msg) {
50 return NULL; 50 return NULL;
51 } 51 }
52 52
53 ChildMessageFilter::ChildMessageFilter() 53 ChildMessageFilter::ChildMessageFilter()
54 : internal_(NULL), 54 : internal_(NULL),
55 thread_safe_sender_(ChildThread::current()->thread_safe_sender()) {} 55 thread_safe_sender_(ChildThread::current()->thread_safe_sender()) {}
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/child/child_message_filter.h ('k') | content/child/child_resource_message_filter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698