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

Side by Side 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef CONTENT_PUBLIC_TEST_CONTENT_BROWSER_SANITY_CHECKER_H_
6 #define CONTENT_PUBLIC_TEST_CONTENT_BROWSER_SANITY_CHECKER_H_
7
8 #include "base/basictypes.h"
9 #include "base/callback.h"
10
11 namespace content {
12
13 class WebContents;
14
15 // While an instance of this class exists, a bunch of sanity-checks are enabled
16 // to validate the correctness of the browser side of the content layer
17 // implementation. It's good to enable these in both unit tests and browser
18 // tests, as a means of detecting bugs in the implementation of the content api.
19 //
20 // These checks should typically be enabled by the base class of any test
21 // framework that would instantiate a WebContents -- ContentBrowserTest::SetUp,
22 // RenderViewHostTestHarness::SetUp, and so forth. Individual tests won't
23 // typically need to enable them.
24 //
25 // For the nuts and bolts of what the checks enforce, see the implementation.
26 class ContentBrowserSanityChecker {
27 public:
28 ContentBrowserSanityChecker();
29 ~ContentBrowserSanityChecker();
30
31 private:
32 void OnWebContentsCreated(WebContents* web_contents);
33
34 // The callback needs to be cached so that it can be unregistered.
35 base::Callback<void(WebContents*)> creation_hook_;
36
37 DISALLOW_COPY_AND_ASSIGN(ContentBrowserSanityChecker);
38 };
39
40 } // namespace content
41
42 #endif // CONTENT_PUBLIC_TEST_CONTENT_BROWSER_SANITY_CHECKER_H_
OLDNEW
« 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