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

Unified Diff: content/public/test/content_browser_sanity_checker.h

Issue 1137083005: Move ContentBrowserSanityChecker and WebContentsObserverSanityChecker from content/public/test to c… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 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
« no previous file with comments | « content/public/test/browser_test_base.cc ('k') | content/public/test/content_browser_sanity_checker.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/test/content_browser_sanity_checker.h
diff --git a/content/public/test/content_browser_sanity_checker.h b/content/public/test/content_browser_sanity_checker.h
deleted file mode 100644
index e227c2c5b1d0e61ac08af63765e6db233f2319b7..0000000000000000000000000000000000000000
--- a/content/public/test/content_browser_sanity_checker.h
+++ /dev/null
@@ -1,42 +0,0 @@
-// Copyright 2014 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_CONTENT_BROWSER_SANITY_CHECKER_H_
-#define CONTENT_PUBLIC_TEST_CONTENT_BROWSER_SANITY_CHECKER_H_
-
-#include "base/basictypes.h"
-#include "base/callback.h"
-
-namespace content {
-
-class WebContents;
-
-// While an instance of this class exists, a bunch of sanity-checks are enabled
-// to validate the correctness of the browser side of the content layer
-// implementation. It's good to enable these in both unit tests and browser
-// tests, as a means of detecting bugs in the implementation of the content api.
-//
-// These checks should typically be enabled by the base class of any test
-// framework that would instantiate a WebContents -- ContentBrowserTest::SetUp,
-// RenderViewHostTestHarness::SetUp, and so forth. Individual tests won't
-// typically need to enable them.
-//
-// For the nuts and bolts of what the checks enforce, see the implementation.
-class ContentBrowserSanityChecker {
- public:
- ContentBrowserSanityChecker();
- ~ContentBrowserSanityChecker();
-
- private:
- void OnWebContentsCreated(WebContents* web_contents);
-
- // The callback needs to be cached so that it can be unregistered.
- base::Callback<void(WebContents*)> creation_hook_;
-
- DISALLOW_COPY_AND_ASSIGN(ContentBrowserSanityChecker);
-};
-
-} // namespace content
-
-#endif // CONTENT_PUBLIC_TEST_CONTENT_BROWSER_SANITY_CHECKER_H_
« no previous file with comments | « content/public/test/browser_test_base.cc ('k') | content/public/test/content_browser_sanity_checker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698