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

Unified Diff: content/renderer/devtools/devtools_client.cc

Issue 16026006: Introduce the notion of a "layout test mode" instead of turning individual flags on and off (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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/renderer/devtools/devtools_client.h ('k') | content/renderer/render_thread_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/devtools/devtools_client.cc
diff --git a/content/renderer/devtools/devtools_client.cc b/content/renderer/devtools/devtools_client.cc
index 5f760e690c6cb5e2e196c1bf4b337814940ab7e2..e02c65f51ef8aa6634cd8f0a3182c4d248fa0f3d 100644
--- a/content/renderer/devtools/devtools_client.cc
+++ b/content/renderer/devtools/devtools_client.cc
@@ -23,12 +23,6 @@ using WebKit::WebString;
namespace content {
-namespace {
-
-bool g_devtools_frontend_testing_enabled = false;
-
-} // namespace
-
DevToolsClient::DevToolsClient(RenderViewImpl* render_view)
: RenderViewObserver(render_view) {
const CommandLine& command_line = *CommandLine::ForCurrentProcess();
@@ -116,7 +110,7 @@ void DevToolsClient::removeFileSystem(const WebString& fileSystemPath) {
}
bool DevToolsClient::isUnderTest() {
- return g_devtools_frontend_testing_enabled;
+ return RenderThreadImpl::current()->layout_test_mode();
}
void DevToolsClient::OnDispatchOnInspectorFrontend(const std::string& message) {
@@ -124,9 +118,4 @@ void DevToolsClient::OnDispatchOnInspectorFrontend(const std::string& message) {
WebString::fromUTF8(message));
}
-// static
-void DevToolsClient::EnableDevToolsFrontendTesting() {
- g_devtools_frontend_testing_enabled = true;
-}
-
} // namespace content
« no previous file with comments | « content/renderer/devtools/devtools_client.h ('k') | content/renderer/render_thread_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698