buffet: Cleaned up the remaining linter warnings

Fixed the remaining warnings from cpplint. So now every file
in platform2/buffet conforms to the default linter settings.

BUG=None
TEST=Code still compiles and unit tests pass

Change-Id: I094d06789590c6c76679c1e1c758525898e86923
Reviewed-on: https://chromium-review.googlesource.com/199414
Tested-by: Alex Vakulenko <avakulenko@chromium.org>
Reviewed-by: Christopher Wiley <wiley@chromium.org>
Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>
diff --git a/buffet/async_event_sequencer.cc b/buffet/async_event_sequencer.cc
index b447c70..bf08d14 100644
--- a/buffet/async_event_sequencer.cc
+++ b/buffet/async_event_sequencer.cc
@@ -54,10 +54,10 @@
     const std::string& expected_method_name,
     const std::string& actual_interface_name,
     const std::string& actual_method_name, bool success) {
-  CHECK(expected_method_name == actual_method_name)
+  CHECK_EQ(expected_method_name, actual_method_name)
       << "Exported DBus method '" << actual_method_name << "' "
       << "but expected '" << expected_method_name << "'";
-  CHECK(expected_interface_name == actual_interface_name)
+  CHECK_EQ(expected_interface_name, actual_interface_name)
       << "Exported method DBus interface '" << actual_interface_name << "' "
       << "but expected '" << expected_interface_name << "'";
   finish_handler.Run(success);
@@ -67,7 +67,7 @@
 void AsyncEventSequencer::RetireRegistration(int registration_number) {
   const size_t handlers_retired = outstanding_registrations_.erase(
       registration_number);
-  CHECK(handlers_retired == 1)
+  CHECK_EQ(1, handlers_retired)
       << "Tried to retire invalid handler " << registration_number << ")";
 }