Linux/ChromeOS Chromium style checker cleanup, chrome/browser edition.

chrome/browser/download/save_page_browsertest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/files/scoped_temp_dir.h"
@@ skipping 68 matching lines @@
   bool WaitForPersisted() {
     if (persisted_)
       return true;
     waiting_ = true;
     content::RunMessageLoop();
     waiting_ = false;
     return persisted_;
   }
 
   virtual void OnDownloadStored(DownloadItem* item,
-                                const history::DownloadRow& info) {
+                                const history::DownloadRow& info) OVERRIDE {
     persisted_ = filter_.Run(item, info);
     if (persisted_ && waiting_)
       MessageLoopForUI::current()->Quit();
   }
 
  private:
   Profile* profile_;
   DownloadItem* item_;
   PersistedFilter filter_;
   bool waiting_;
@@ skipping 16 matching lines @@
   bool WaitForRemoved() {
     if (removed_)
       return true;
     waiting_ = true;
     content::RunMessageLoop();
     waiting_ = false;
     return removed_;
   }
 
   virtual void OnDownloadStored(DownloadItem* item,
-                                const history::DownloadRow& info) {
+                                const history::DownloadRow& info) OVERRIDE {
   }
 
-  virtual void OnDownloadsRemoved(const DownloadHistory::IdSet& ids) {
+  virtual void OnDownloadsRemoved(const DownloadHistory::IdSet& ids) OVERRIDE {
     removed_ = ids.find(download_id_) != ids.end();
     if (removed_ && waiting_)
       MessageLoopForUI::current()->Quit();
   }
 
  private:
   bool removed_;
   bool waiting_;
   int32 download_id_;
 
@@ skipping 48 matching lines @@
 #endif
 
 // Loosely based on logic in DownloadTestObserver.
 class DownloadItemCreatedObserver : public DownloadManager::Observer {
  public:
   explicit DownloadItemCreatedObserver(DownloadManager* manager)
       : waiting_(false), manager_(manager) {
     manager->AddObserver(this);
   }
 
-  ~DownloadItemCreatedObserver() {
+  virtual ~DownloadItemCreatedObserver() {
     if (manager_)
       manager_->RemoveObserver(this);
   }
 
   // Wait for the first download item created after object creation.
   // Note that this class provides no protection against the download
   // being destroyed between creation and return of WaitForNewDownloadItem();
   // the caller must guarantee that in some other fashion.
   void WaitForDownloadItem(std::vector<DownloadItem*>* items_seen) {
     if (!manager_) {
@@ skipping 68 matching lines @@
 
   DISALLOW_COPY_AND_ASSIGN(SavePackageFinishedObserver);
 };
 
 class SavePageBrowserTest : public InProcessBrowserTest {
  public:
   SavePageBrowserTest() {}
   virtual ~SavePageBrowserTest();
 
  protected:
-  void SetUp() OVERRIDE {
+  virtual void SetUp() OVERRIDE {
     ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_dir_));
     ASSERT_TRUE(save_dir_.CreateUniqueTempDir());
     InProcessBrowserTest::SetUp();
   }
 
-  void SetUpOnMainThread() OVERRIDE {
+  virtual void SetUpOnMainThread() OVERRIDE {
     browser()->profile()->GetPrefs()->SetFilePath(
         prefs::kDownloadDefaultDirectory, save_dir_.path());
     BrowserThread::PostTask(
         BrowserThread::IO, FROM_HERE,
         base::Bind(&chrome_browser_net::SetUrlRequestMocksEnabled, true));
   }
 
   GURL NavigateToMockURL(const std::string& prefix) {
     GURL url = URLRequestMockHTTPJob::GetMockUrl(
         FilePath(kTestDir).AppendASCII(prefix + ".htm"));
@@ skipping 465 matching lines @@
   persisted.WaitForPersisted();
 
   EXPECT_TRUE(file_util::PathExists(full_file_name));
   int64 actual_file_size = -1;
   EXPECT_TRUE(file_util::GetFileSize(full_file_name, &actual_file_size));
   EXPECT_LE(kFileSizeMin, actual_file_size);
 }
 
 }  // namespace