Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(296)

Unified Diff: chrome/test/base/chrome_render_view_host_test_harness.h

Issue 7892007: Add ChromeRenderViewHostTestHarness to get rid of the dependency from RVHTH to profile (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: updates Created 9 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/base/chrome_render_view_host_test_harness.h
diff --git a/chrome/test/base/chrome_render_view_host_test_harness.h b/chrome/test/base/chrome_render_view_host_test_harness.h
new file mode 100644
index 0000000000000000000000000000000000000000..7f148f9170d4f65a6b5f1868d7f659d1a54f98f7
--- /dev/null
+++ b/chrome/test/base/chrome_render_view_host_test_harness.h
@@ -0,0 +1,32 @@
+// Copyright (c) 2011 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 CHROME_TEST_BASE_CHROME_RENDER_VIEW_HOST_TEST_HARNESS_H_
+#define CHROME_TEST_BASE_CHROME_RENDER_VIEW_HOST_TEST_HARNESS_H_
+#pragma once
+
+#include "base/compiler_specific.h"
+#include "content/browser/renderer_host/test_render_view_host.h"
+
+class TestingProfile;
+
+// Wrapper around RenderViewHostTestHarness that uses a TestingProfile as
+// browser context instead of a TestBrowserContext.
+class ChromeRenderViewHostTestHarness : public RenderViewHostTestHarness {
+ public:
+ ChromeRenderViewHostTestHarness();
+ virtual ~ChromeRenderViewHostTestHarness();
+
+ TestingProfile* profile();
+
+ protected:
+ // testing::Test
+ virtual void SetUp() OVERRIDE;
+ virtual void TearDown() OVERRIDE;
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(ChromeRenderViewHostTestHarness);
Paweł Hajdan Jr. 2011/09/13 19:16:36 nit: No need for that for test fixtures.
+};
+
+#endif // CHROME_TEST_BASE_CHROME_RENDER_VIEW_HOST_TEST_HARNESS_H_

Powered by Google App Engine
This is Rietveld 408576698