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

chrome/browser/extensions/api/webstore_private/webstore_private_apitest.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 <vector>
 
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
@@ skipping 26 matching lines @@
 
 namespace extensions {
 
 namespace {
 
 class WebstoreInstallListener : public WebstoreInstaller::Delegate {
  public:
   WebstoreInstallListener()
       : received_failure_(false), received_success_(false), waiting_(false) {}
 
-  void OnExtensionInstallSuccess(const std::string& id) OVERRIDE {
+  virtual void OnExtensionInstallSuccess(const std::string& id) OVERRIDE {
     received_success_ = true;
     id_ = id;
 
     if (waiting_) {
       waiting_ = false;
       MessageLoopForUI::current()->Quit();
     }
   }
 
-  void OnExtensionInstallFailure(
+  virtual void OnExtensionInstallFailure(
       const std::string& id,
       const std::string& error,
       WebstoreInstaller::FailureReason reason) OVERRIDE {
     received_failure_ = true;
     id_ = id;
     error_ = error;
 
     if (waiting_) {
       waiting_ = false;
       MessageLoopForUI::current()->Quit();
@@ skipping 16 matching lines @@
   bool waiting_;
   std::string id_;
   std::string error_;
 };
 
 }  // namespace
 
 // A base class for tests below.
 class ExtensionWebstorePrivateApiTest : public ExtensionApiTest {
  public:
-  void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+  virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
     ExtensionApiTest::SetUpCommandLine(command_line);
     command_line->AppendSwitchASCII(
         switches::kAppsGalleryURL,
         "http://www.example.com/files/extensions/api_test");
     command_line->AppendSwitchASCII(
         switches::kAppsGalleryInstallAutoConfirmForTests, "accept");
   }
 
-  void SetUpInProcessBrowserTestFixture() OVERRIDE {
+  virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {
     // Start up the test server and get us ready for calling the install
     // API functions.
     host_resolver()->AddRule("www.example.com", "127.0.0.1");
     ASSERT_TRUE(test_server()->Start());
     ExtensionInstallUI::DisableFailureUIForTests();
 
     ASSERT_TRUE(tmp_.CreateUniqueTempDirUnderPath(test_data_dir_));
     ASSERT_TRUE(file_util::CreateDirectory(tmp_.path()));
     ASSERT_TRUE(file_util::CopyDirectory(
         test_data_dir_.AppendASCII("webstore_private"),
@@ skipping 35 matching lines @@
   ExtensionService* service() {
     return browser()->profile()->GetExtensionService();
   }
 
   base::ScopedTempDir tmp_;
 };
 
 class ExtensionWebstorePrivateBundleTest
     : public ExtensionWebstorePrivateApiTest {
  public:
-  void SetUpInProcessBrowserTestFixture() OVERRIDE {
+  virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {
     ExtensionWebstorePrivateApiTest::SetUpInProcessBrowserTestFixture();
 
     // The test server needs to have already started, so setup the switch here
     // rather than in SetUpCommandLine.
     CommandLine::ForCurrentProcess()->AppendSwitchASCII(
         switches::kAppsGalleryDownloadURL,
         GetTestServerURL("bundle/%s.crx").spec());
   }
 
-  void TearDownInProcessBrowserTestFixture() OVERRIDE {
+  virtual void TearDownInProcessBrowserTestFixture() OVERRIDE {
     ExtensionWebstorePrivateApiTest::TearDownInProcessBrowserTestFixture();
     for (size_t i = 0; i < test_crx_.size(); ++i)
       ASSERT_TRUE(file_util::Delete(test_crx_[i], false));
   }
 
  protected:
   // Packs the |manifest| file into a CRX using |id|'s PEM key.
   void PackCRX(const std::string& id, const std::string& manifest) {
     // Move the extension to a temporary directory.
     base::ScopedTempDir tmp;
@@ skipping 37 matching lines @@
 
     test_crx_.push_back(crx_path);
   }
 
  private:
   std::vector<FilePath> test_crx_;
 };
 
 class ExtensionWebstoreGetWebGLStatusTest : public InProcessBrowserTest {
  public:
-  void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+  virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
     // In linux, we need to launch GPU process to decide if WebGL is allowed.
     // Run it on top of osmesa to avoid bot driver issues.
 #if defined(OS_LINUX)
     CHECK(test_launcher_utils::OverrideGLImplementation(
         command_line, gfx::kGLImplementationOSMesaName)) <<
         "kUseGL must not be set multiple times!";
 #endif
   }
 
  protected:
@@ skipping 239 matching lines @@
   GpuFeatureType type =
       content::GpuDataManager::GetInstance()->GetBlacklistedFeatures();
   EXPECT_EQ((type & content::GPU_FEATURE_TYPE_WEBGL),
             content::GPU_FEATURE_TYPE_WEBGL);
 
   bool webgl_allowed = false;
   RunTest(webgl_allowed);
 }
 
 }  // namespace extensions