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/storage_impls.h

diff --git a/buffet/storage_impls.h b/buffet/storage_impls.h
index 6e69084..156f946 100644
--- a/buffet/storage_impls.h
+++ b/buffet/storage_impls.h
@@ -18,8 +18,8 @@
  public:
   explicit FileStorage(const base::FilePath& file_path);
   virtual ~FileStorage() = default;
-  virtual std::unique_ptr<base::Value> Load() override;
-  virtual bool Save(const base::Value* config) override;
+  std::unique_ptr<base::Value> Load() override;
+  bool Save(const base::Value* config) override;
 
  private:
   base::FilePath file_path_;
@@ -31,8 +31,8 @@
  public:
   MemStorage() = default;
   virtual ~MemStorage() = default;
-  virtual std::unique_ptr<base::Value> Load() override;
-  virtual bool Save(const base::Value* config) override;
+  std::unique_ptr<base::Value> Load() override;
+  bool Save(const base::Value* config) override;
   int save_count() { return save_count_; }
   void reset_save_count() { save_count_ = 0; }