commit 5a9e718e5299c72094d1bcfd988d7e44831e9d07
author Alex Vakulenko <avakulenko@chromium.org> Mon Aug 11 15:59:58 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Tue Aug 12 23:10:06 2014 +0000
tree 6d4b31b79ff40484a77e111bfe9f7863f8ef7c6e
parent e23caf25571af9ca44bf7f2b9e2d06a66d515100

platform2: sweep the lint errors identified by the updated linter

cpplint.py has been updated and identified new issues in existing
code. Stuff like overridden functions that specify 'override' should
not be marked as 'virtual', and constructors with no parameters
should not be marked as 'explicit'.

BUG=None
TEST=cpplint.py `find ./platform2 -name *.cc -or -name *.h`

Change-Id: Ibb9de43286d874d076ffd5ebb1b13c36ec794f01
Reviewed-on: https://chromium-review.googlesource.com/211950
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>
Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>
Tested-by: Alex Vakulenko <avakulenko@chromium.org>

buffet/commands/command_queue_unittest.cc

diff --git a/buffet/commands/command_queue_unittest.cc b/buffet/commands/command_queue_unittest.cc
index 117d7bc..2c0e01b 100644
--- a/buffet/commands/command_queue_unittest.cc
+++ b/buffet/commands/command_queue_unittest.cc
@@ -25,7 +25,7 @@
 // Aborts if duplicate commands are added or non-existent commands are removed.
 class FakeDispatchInterface : public buffet::CommandDispachInterface {
  public:
-  virtual void OnCommandAdded(
+  void OnCommandAdded(
       const std::string& command_id,
       const buffet::CommandInstance* command_instance) override {
     CHECK(ids_.insert(command_id).second)
@@ -34,7 +34,7 @@
         << "Command instance already exists";
   }
 
-  virtual void OnCommandRemoved(
+  void OnCommandRemoved(
       const std::string& command_id,
       const buffet::CommandInstance* command_instance) override {
     CHECK_EQ(1, ids_.erase(command_id))