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.h b/buffet/async_event_sequencer.h
index 105be6d..ab5bc1a 100644
--- a/buffet/async_event_sequencer.h
+++ b/buffet/async_event_sequencer.h
@@ -19,7 +19,7 @@
 
 // A helper class for coordinating the multiple async tasks.  A consumer
 // may grab any number of callbacks via Get*Handler() and schedule a list
-// of completion actions to take.  When all handlers obtained bia Get*Handler()
+// of completion actions to take.  When all handlers obtained via Get*Handler()
 // have been called, the AsyncEventSequencer will call its CompletionActions.
 //
 // Usage:
@@ -79,11 +79,11 @@
                        const std::string& error_message);
   void PossiblyRunCompletionActions();
 
-  bool started_{false};
-  int registration_counter_{0};
+  bool started_{false};          // NOLINT - initializer list
+  int registration_counter_{0};  // NOLINT - initializer list
   std::set<int> outstanding_registrations_;
   std::vector<CompletionAction> completion_actions_;
-  bool had_failures_{false};
+  bool had_failures_{false};     // NOLINT - initializer list
   // Ref counted objects have private destructors.
   ~AsyncEventSequencer();
   friend class base::RefCounted<AsyncEventSequencer>;