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

Unified Diff: headless/test/headless_browser_test.h

Issue 2509813006: HeadlessWebContents:Observer to observe render process exit status (Closed)
Patch Set: Make it safe to call Shutdown() Created 4 years, 1 month 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: headless/test/headless_browser_test.h
diff --git a/headless/test/headless_browser_test.h b/headless/test/headless_browser_test.h
index c787859eb5b84c5f4a03f41809cef22bae8c37b5..3db973b02e5490ca3c4a7107a56b9a2a168ee461 100644
--- a/headless/test/headless_browser_test.h
+++ b/headless/test/headless_browser_test.h
@@ -105,6 +105,8 @@ class HeadlessAsyncDevTooledBrowserTest : public HeadlessBrowserTest,
// HeadlessWebContentsObserver implementation:
void DevToolsTargetReady() override;
+ void RenderProcessExited(base::TerminationStatus status,
+ int exit_code) override;
// Implemented by tests and used to send request(s) to DevTools. Subclasses
// need to ensure that FinishAsynchronousTest() is called after response(s)
@@ -117,6 +119,7 @@ class HeadlessAsyncDevTooledBrowserTest : public HeadlessBrowserTest,
HeadlessBrowserContext* browser_context_; // Not owned.
HeadlessWebContents* web_contents_;
std::unique_ptr<HeadlessDevToolsClient> devtools_client_;
+ bool render_process_exited_;
};
} // namespace headless

Powered by Google App Engine
This is Rietveld 408576698