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

Side by Side Diff: headless/test/headless_browser_test.h

Issue 2978473002: Disable headless TargetDomain tests on Windows. (Closed)
Patch Set: add disabling macro Created 3 years, 5 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
« no previous file with comments | « headless/lib/headless_devtools_client_browsertest.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef HEADLESS_TEST_HEADLESS_BROWSER_TEST_H_ 5 #ifndef HEADLESS_TEST_HEADLESS_BROWSER_TEST_H_
6 #define HEADLESS_TEST_HEADLESS_BROWSER_TEST_H_ 6 #define HEADLESS_TEST_HEADLESS_BROWSER_TEST_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <string> 9 #include <string>
10 10
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
87 // Returns the options used by the browser. Modify with caution, since some 87 // Returns the options used by the browser. Modify with caution, since some
88 // options only take effect if they were set before browser creation. 88 // options only take effect if they were set before browser creation.
89 HeadlessBrowser::Options* options() const; 89 HeadlessBrowser::Options* options() const;
90 90
91 private: 91 private:
92 std::unique_ptr<base::RunLoop> run_loop_; 92 std::unique_ptr<base::RunLoop> run_loop_;
93 93
94 DISALLOW_COPY_AND_ASSIGN(HeadlessBrowserTest); 94 DISALLOW_COPY_AND_ASSIGN(HeadlessBrowserTest);
95 }; 95 };
96 96
97 // TODO(eseckler): Make macro more sheriff-friendly.
97 #define HEADLESS_ASYNC_DEVTOOLED_TEST_F(TEST_FIXTURE_NAME) \ 98 #define HEADLESS_ASYNC_DEVTOOLED_TEST_F(TEST_FIXTURE_NAME) \
98 IN_PROC_BROWSER_TEST_F(TEST_FIXTURE_NAME, RunAsyncTest) { RunTest(); } \ 99 IN_PROC_BROWSER_TEST_F(TEST_FIXTURE_NAME, RunAsyncTest) { RunTest(); } \
99 class AsyncHeadlessBrowserTestNeedsSemicolon##TEST_FIXTURE_NAME {} 100 class AsyncHeadlessBrowserTestNeedsSemicolon##TEST_FIXTURE_NAME {}
100 101
101 #define HEADLESS_ASYNC_DEVTOOLED_TEST_P(TEST_FIXTURE_NAME) \ 102 #define HEADLESS_ASYNC_DEVTOOLED_TEST_P(TEST_FIXTURE_NAME) \
102 IN_PROC_BROWSER_TEST_P(TEST_FIXTURE_NAME, RunAsyncTest) { RunTest(); } \ 103 IN_PROC_BROWSER_TEST_P(TEST_FIXTURE_NAME, RunAsyncTest) { RunTest(); } \
103 class AsyncHeadlessBrowserTestNeedsSemicolon##TEST_FIXTURE_NAME {} 104 class AsyncHeadlessBrowserTestNeedsSemicolon##TEST_FIXTURE_NAME {}
104 105
106 #define DISABLED_HEADLESS_ASYNC_DEVTOOLED_TEST_F(TEST_FIXTURE_NAME) \
107 IN_PROC_BROWSER_TEST_F(TEST_FIXTURE_NAME, DISABLED_RunAsyncTest) { \
108 RunTest(); \
109 } \
110 class AsyncHeadlessBrowserTestNeedsSemicolon##TEST_FIXTURE_NAME {}
111
112 #define DISABLED_HEADLESS_ASYNC_DEVTOOLED_TEST_P(TEST_FIXTURE_NAME) \
113 IN_PROC_BROWSER_TEST_P(TEST_FIXTURE_NAME, DISABLED_RunAsyncTest) { \
114 RunTest(); \
115 } \
116 class AsyncHeadlessBrowserTestNeedsSemicolon##TEST_FIXTURE_NAME {}
117
105 // Base class for tests that require access to a DevToolsClient. Subclasses 118 // Base class for tests that require access to a DevToolsClient. Subclasses
106 // should override the RunDevTooledTest() method, which is called asynchronously 119 // should override the RunDevTooledTest() method, which is called asynchronously
107 // when the DevToolsClient is ready. 120 // when the DevToolsClient is ready.
108 class HeadlessAsyncDevTooledBrowserTest : public HeadlessBrowserTest, 121 class HeadlessAsyncDevTooledBrowserTest : public HeadlessBrowserTest,
109 public HeadlessWebContents::Observer { 122 public HeadlessWebContents::Observer {
110 public: 123 public:
111 HeadlessAsyncDevTooledBrowserTest(); 124 HeadlessAsyncDevTooledBrowserTest();
112 ~HeadlessAsyncDevTooledBrowserTest() override; 125 ~HeadlessAsyncDevTooledBrowserTest() override;
113 126
114 // HeadlessWebContentsObserver implementation: 127 // HeadlessWebContentsObserver implementation:
(...skipping 23 matching lines...) Expand all
138 HeadlessBrowserContext* browser_context_; // Not owned. 151 HeadlessBrowserContext* browser_context_; // Not owned.
139 HeadlessWebContents* web_contents_; 152 HeadlessWebContents* web_contents_;
140 std::unique_ptr<HeadlessDevToolsClient> devtools_client_; 153 std::unique_ptr<HeadlessDevToolsClient> devtools_client_;
141 std::unique_ptr<HeadlessDevToolsClient> browser_devtools_client_; 154 std::unique_ptr<HeadlessDevToolsClient> browser_devtools_client_;
142 bool render_process_exited_; 155 bool render_process_exited_;
143 }; 156 };
144 157
145 } // namespace headless 158 } // namespace headless
146 159
147 #endif // HEADLESS_TEST_HEADLESS_BROWSER_TEST_H_ 160 #endif // HEADLESS_TEST_HEADLESS_BROWSER_TEST_H_
OLDNEW
« no previous file with comments | « headless/lib/headless_devtools_client_browsertest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698