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

Side by Side Diff: net/dns/notify_watcher_mac.h

Issue 667923003: Standardize usage of virtual/override/final in net/ (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 | « net/dns/mock_host_resolver.h ('k') | net/dns/serial_worker_unittest.cc » ('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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #ifndef NET_DNS_NOTIFY_WATCHER_MAC_H_ 5 #ifndef NET_DNS_NOTIFY_WATCHER_MAC_H_
6 #define NET_DNS_NOTIFY_WATCHER_MAC_H_ 6 #define NET_DNS_NOTIFY_WATCHER_MAC_H_
7 7
8 #include "base/callback.h" 8 #include "base/callback.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 10
11 namespace net { 11 namespace net {
12 12
13 // Watches for notifications from Libnotify and delivers them to a Callback. 13 // Watches for notifications from Libnotify and delivers them to a Callback.
14 // After failure the watch is cancelled and will have to be restarted. 14 // After failure the watch is cancelled and will have to be restarted.
15 class NotifyWatcherMac : public base::MessageLoopForIO::Watcher { 15 class NotifyWatcherMac : public base::MessageLoopForIO::Watcher {
16 public: 16 public:
17 // Called on received notification with true on success and false on error. 17 // Called on received notification with true on success and false on error.
18 typedef base::Callback<void(bool succeeded)> CallbackType; 18 typedef base::Callback<void(bool succeeded)> CallbackType;
19 19
20 NotifyWatcherMac(); 20 NotifyWatcherMac();
21 21
22 // When deleted, automatically cancels. 22 // When deleted, automatically cancels.
23 virtual ~NotifyWatcherMac(); 23 ~NotifyWatcherMac() override;
24 24
25 // Registers for notifications for |key|. Returns true if succeeds. If so, 25 // Registers for notifications for |key|. Returns true if succeeds. If so,
26 // will deliver asynchronous notifications and errors to |callback|. 26 // will deliver asynchronous notifications and errors to |callback|.
27 bool Watch(const char* key, const CallbackType& callback); 27 bool Watch(const char* key, const CallbackType& callback);
28 28
29 // Cancels the watch. 29 // Cancels the watch.
30 void Cancel(); 30 void Cancel();
31 31
32 private: 32 private:
33 // MessageLoopForIO::Watcher: 33 // MessageLoopForIO::Watcher:
34 virtual void OnFileCanReadWithoutBlocking(int fd) override; 34 void OnFileCanReadWithoutBlocking(int fd) override;
35 virtual void OnFileCanWriteWithoutBlocking(int fd) override {} 35 void OnFileCanWriteWithoutBlocking(int fd) override {}
36 36
37 int notify_fd_; 37 int notify_fd_;
38 int notify_token_; 38 int notify_token_;
39 CallbackType callback_; 39 CallbackType callback_;
40 base::MessageLoopForIO::FileDescriptorWatcher watcher_; 40 base::MessageLoopForIO::FileDescriptorWatcher watcher_;
41 41
42 DISALLOW_COPY_AND_ASSIGN(NotifyWatcherMac); 42 DISALLOW_COPY_AND_ASSIGN(NotifyWatcherMac);
43 }; 43 };
44 44
45 } // namespace net 45 } // namespace net
46 46
47 #endif // NET_DNS_NOTIFY_WATCHER_MAC_H_ 47 #endif // NET_DNS_NOTIFY_WATCHER_MAC_H_
OLDNEW
« no previous file with comments | « net/dns/mock_host_resolver.h ('k') | net/dns/serial_worker_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698