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

Unified Diff: headless/lib/browser/headless_browser_impl.h

Issue 2709433002: Add HeadlessFocusClient to fix document.hasFocus() issues. (Closed)
Patch Set: Set FocusClient and add test Created 3 years, 10 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_browser_impl.h
diff --git a/headless/lib/browser/headless_browser_impl.h b/headless/lib/browser/headless_browser_impl.h
index e622227f65b5f0e13e89224f3cf8d7fa4dc6338b..fbcdb1869b4f648180ca7b54324e95b2af0f3e80 100644
--- a/headless/lib/browser/headless_browser_impl.h
+++ b/headless/lib/browser/headless_browser_impl.h
@@ -21,6 +21,12 @@
#include "headless/lib/browser/headless_window_tree_host.h"
#endif
+namespace aura {
Eric Seckler 2017/02/22 09:00:04 nit: let's move this into the "#if defined(USE_AUR
irisu 2017/02/23 00:02:26 Done.
+namespace client {
+class FocusClient;
+}
+}
+
namespace headless {
class HeadlessBrowserContextImpl;
@@ -81,6 +87,8 @@ class HeadlessBrowserImpl : public HeadlessBrowser {
// is used for all web contents. We should probably use one
// window per web contents, but additional investigation is needed.
std::unique_ptr<HeadlessWindowTreeHost> window_tree_host_;
+ std::unique_ptr<aura::client::FocusClient> focus_client_;
+
Eric Seckler 2017/02/22 09:00:04 nit: move blank line after "#endif"
irisu 2017/02/23 00:02:26 Done.
#endif
base::Callback<void(HeadlessBrowser*)> on_start_callback_;
HeadlessBrowser::Options options_;
« no previous file with comments | « no previous file | headless/lib/browser/headless_browser_impl.cc » ('j') | headless/lib/browser/headless_browser_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698