diff options
author | Alex Deymo <deymo@chromium.org> | 2015-07-27 14:08:34 -0700 |
---|---|---|
committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2015-07-28 00:59:21 +0000 |
commit | 38c16335b69d8f6611746e6267bc68105e237e97 (patch) | |
tree | e3cd7769413a7d19203a2492cee4409cd24cc94a /chromeos/process_reaper_unittest.cc | |
parent | 471776dcf9a6a861e3f9f63121fd13b1a1b3f180 (diff) | |
download | libbrillo-38c16335b69d8f6611746e6267bc68105e237e97.tar.gz |
libchromeos: Move AsynchronousSignalHandler to an interface.
Both chromeos::AsynchronousSignalHandler and chromeos::Daemon
implemented the same Register/Unregister interface for signal handlers.
This patch moves that functionality to an abstract interface and makes
chromeos::ProcessReapper use that instead of having two methods.
BUG=None
TEST=Unittests still pass.
Change-Id: Ib2aa8c5279b5998e7c88c2211809901fa11a8f0a
Reviewed-on: https://chromium-review.googlesource.com/288752
Trybot-Ready: Alex Deymo <deymo@chromium.org>
Tested-by: Alex Deymo <deymo@chromium.org>
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>
Commit-Queue: Alex Deymo <deymo@chromium.org>
Diffstat (limited to 'chromeos/process_reaper_unittest.cc')
-rw-r--r-- | chromeos/process_reaper_unittest.cc | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/chromeos/process_reaper_unittest.cc b/chromeos/process_reaper_unittest.cc index 44b4048..d49ce37 100644 --- a/chromeos/process_reaper_unittest.cc +++ b/chromeos/process_reaper_unittest.cc @@ -48,8 +48,7 @@ class ProcessReaperTest : public ::testing::Test { void SetUp() override { chromeos_loop_.SetAsCurrent(); async_signal_handler_.Init(); - process_reaper_.RegisterWithAsynchronousSignalHandler( - &async_signal_handler_); + process_reaper_.Register(&async_signal_handler_); } protected: @@ -68,8 +67,7 @@ TEST_F(ProcessReaperTest, UnregisterWhenNotRegistered) { TEST_F(ProcessReaperTest, UnregisterAndReregister) { process_reaper_.Unregister(); - process_reaper_.RegisterWithAsynchronousSignalHandler( - &async_signal_handler_); + process_reaper_.Register(&async_signal_handler_); // This checks that we can unregister the ProcessReaper and then destroy it. process_reaper_.Unregister(); } |