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

Unified Diff: headless/lib/browser/headless_devtools_manager_delegate.cc

Issue 2755973003: DevTools: android shouldn't depend on devtools frontend (Closed)
Patch Set: fixup Created 3 years, 9 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
Index: headless/lib/browser/headless_devtools_manager_delegate.cc
diff --git a/headless/lib/browser/headless_devtools_manager_delegate.cc b/headless/lib/browser/headless_devtools_manager_delegate.cc
index 3ecf23d9a473e4f2f30696383b7d91ba89ee068b..108ec6b201a2fff60c74225a4227657868b9efbc 100644
--- a/headless/lib/browser/headless_devtools_manager_delegate.cc
+++ b/headless/lib/browser/headless_devtools_manager_delegate.cc
@@ -9,7 +9,6 @@
#include "content/public/browser/browser_thread.h"
#include "content/public/browser/devtools_agent_host.h"
-#include "content/public/browser/devtools_frontend_host.h"
#include "content/public/browser/web_contents.h"
#include "headless/grit/headless_lib_resources.h"
#include "headless/lib/browser/headless_browser_context_impl.h"
@@ -18,6 +17,10 @@
#include "headless/public/devtools/domains/target.h"
#include "ui/base/resource/resource_bundle.h"
+#if !defined(OS_ANDROID)
chenwilliam 2017/03/20 23:59:40 @altimin Is headless only for desktop? If so, I'll
altimin 2017/03/21 00:12:10 Yes, headless is desktop-only.
+#include "content/public/browser/devtools_frontend_host.h"
+#endif
+
namespace headless {
namespace {
« content/public/browser/devtools_frontend_host.h ('K') | « headless/lib/browser/headless_devtools.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698