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

Unified Diff: remoting/host/posix/signal_handler.h

Issue 10905081: Pass Me2Me config via stdin. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed #include guard footer. Created 8 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 side-by-side diff with in-line comments
Download patch
Index: remoting/host/posix/signal_handler.h
diff --git a/remoting/host/posix/sighup_listener.h b/remoting/host/posix/signal_handler.h
similarity index 68%
rename from remoting/host/posix/sighup_listener.h
rename to remoting/host/posix/signal_handler.h
index 48140a39f9a4460bcf877dae949822ec64f2a3cb..1845aef2d689a86eaf91dcfc1a45cbb29404b364 100644
--- a/remoting/host/posix/sighup_listener.h
+++ b/remoting/host/posix/signal_handler.h
@@ -6,17 +6,19 @@
// and trigger the specified callback. It is used on Linux and Mac in order to
// reload the me2me host configuration.
-#ifndef REMOTING_HOST_POSIX_SIGHUP_LISTENER_H_
-#define REMOTING_HOST_POSIX_SIGHUP_LISTENER_H_
+#ifndef REMOTING_HOST_POSIX_SIGNAL_HANDLER_H_
+#define REMOTING_HOST_POSIX_SIGNAL_HANDLER_H_
#include "base/callback_forward.h"
namespace remoting {
+typedef base::Callback<void(int)> SignalHandler;
+
// Register for SIGHUP notifications on the current thread, which must have
alexeypa (please no reviews) 2012/09/05 21:41:25 nit: SIGHUP -> |signum|
Jamie 2012/09/06 00:18:50 Done.
// an associated MessageLoopForIO.
-bool RegisterHupSignalHandler(const base::Closure& callback);
+bool RegisterSignalHandler(int signum, const SignalHandler& handler);
Lambros 2012/09/06 00:47:19 nit: signal or signal_number perhaps?
} // namespace remoting
-#endif // REMOTING_HOST_POSIX_SIGHUP_LISTENER_H_
+#endif // REMOTING_HOST_POSIX_SIGNAL_HANDLER_H_

Powered by Google App Engine
This is Rietveld 408576698