Admixtures in BrowserTestBase

content/public/test/browser_test_base.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_TEST_BROWSER_TEST_BASE_H_
 #define CONTENT_PUBLIC_TEST_BROWSER_TEST_BASE_H_
 
 #include "base/callback.h"
 #include "base/compiler_specific.h"
+#include "base/memory/scoped_vector.h"
 #include "base/threading/thread.h"
 #include "net/test/spawned_test_server/spawned_test_server.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace base {
 class CommandLine;
 class FilePath;
 }
 
 namespace net {
 namespace test_server {
 class EmbeddedTestServer;
 }
 
 class RuleBasedHostResolverProc;
 }  // namespace net
 
 namespace content {
 
 class BrowserTestBase : public testing::Test {
  public:
+  // Class that can be used to add some features not related directly
+  // to the testing process and do set ups needed for them in proper time.
+  // To do this you need to derive Admixture, implement
+  // all the methods you need there and (if needed) reload set ups and tear
+  // downs
+  // for things that you would usually do in those of the test, but which are
+  // connected
+  // to the admixture, not to the test itself.
+  // Finally, AddAdmixture() should be called in the test's constructor for each
+  // Admixture.
+  class Admixture {
+   public:
+    Admixture() {}
+    virtual ~Admixture() {}
+    virtual void SetUpCommandLine(base::CommandLine* command_line) {}
+    virtual void SetUpOnMainThread() {}
+    virtual void SetUpInProcessBrowserTestFixture() {}
+    virtual void TearDownOnMainThread() {}
+    virtual void TearDownInProcessBrowserTestFixture() {}

[dzhioev (left Google), 2014/08/18 12:11:46]

You forgot to add TearDown* calls in appropriate places of BrowserTestBase and
InProcessBrowserTest.

[Lisa Ignatyeva, 2014/08/18 12:20:41]

Done.

+  };
+
   BrowserTestBase();
   virtual ~BrowserTestBase();
 
   // We do this so we can be used in a Task.
   void AddRef() {}
   void Release() {}
 
   // Configures everything for an in process browser test, then invokes
   // BrowserMain. BrowserMain ends up invoking RunTestOnMainThreadLoop.
   virtual void SetUp() OVERRIDE;
@@ skipping 82 matching lines @@
   // Call this before SetUp() to cause the test to generate pixel output.
   void EnablePixelOutput();
 
   // Call this before SetUp() to not use GL, but use software compositing
   // instead.
   void UseSoftwareCompositing();
 
   // Returns true if the test will be using GL acceleration via OSMesa.
   bool UsingOSMesa() const;
 
+  // Adds |admixture| as an admixture for this test, passing ownership
+  // for it to BrowserTestBase.
+  // Should be called in constructor of the test (should be already completed
+  // before running set ups).
+  void AddAdmixture(Admixture* admixture);
+
+  // Returns admixtures for this test.
+  std::vector<BrowserTestBase::Admixture*> GetAdmixtures();
+
+  // Passes admixtures ownership back.
+  void ReturnAdmixtures(ScopedVector<BrowserTestBase::Admixture> arg);

[dzhioev (left Google), 2014/08/18 12:11:46]

Remove this declaration.

[Lisa Ignatyeva, 2014/08/18 12:20:41]

Done.

+
  private:
   void ProxyRunTestOnMainThreadLoop();
 
   // Testing server, started on demand.
   scoped_ptr<net::SpawnedTestServer> test_server_;
 
   // Embedded test server, cheap to create, started on demand.
   scoped_ptr<net::test_server::EmbeddedTestServer> embedded_test_server_;
 
   // Host resolver used during tests.
   scoped_refptr<net::RuleBasedHostResolverProc> rule_based_resolver_;
 
+  // Holds admixtures for this test.
+  ScopedVector<Admixture> admixtures_;
+
   // Expected exit code (default is 0).
   int expected_exit_code_;
 
   // When true, the compositor will produce pixel output that can be read back
   // for pixel tests.
   bool enable_pixel_output_;
 
   // When true, do compositing with the software backend instead of using GL.
   bool use_software_compositing_;
 
 #if defined(OS_POSIX)
   bool handle_sigterm_;
 #endif
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_TEST_BROWSER_TEST_BASE_H_