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

Unified Diff: content/browser/web_contents/web_contents_impl.h

Issue 1934703002: Fix keyboard focus for OOPIF-<webview>. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added test and addressed comments. Created 4 years, 6 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: content/browser/web_contents/web_contents_impl.h
diff --git a/content/browser/web_contents/web_contents_impl.h b/content/browser/web_contents/web_contents_impl.h
index 645b6c7408ed7b6b940cf0b0a7577a276a0b4937..00edcb1c180c1b9907e9c2e0e8064e52b991a431 100644
--- a/content/browser/web_contents/web_contents_impl.h
+++ b/content/browser/web_contents/web_contents_impl.h
@@ -463,6 +463,7 @@ class CONTENT_EXPORT WebContentsImpl
void EnsureOpenerProxiesExist(RenderFrameHost* source_rfh) override;
std::unique_ptr<WebUIImpl> CreateWebUIForRenderFrameHost(
const GURL& url) override;
+ void SetFocusedFrame(FrameTreeNode* node, SiteInstance* source) override;
// RenderViewHostDelegate ----------------------------------------------------
RenderViewHostDelegateView* GetDelegateView() override;
@@ -819,10 +820,14 @@ private:
RenderFrameHostImpl* outer_contents_frame);
WebContentsImpl* outer_web_contents() { return outer_web_contents_; }
- int outer_contents_frame_tree_node_id() {
+ int outer_contents_frame_tree_node_id() const {
return outer_contents_frame_tree_node_id_;
}
+ WebContentsImpl* focused_web_contents() { return focused_web_contents_; }
+
+ void SetFocusedWebContents(WebContentsImpl* web_contents);
+
private:
// The outer WebContents.
WebContentsImpl* outer_web_contents_;
@@ -830,6 +835,9 @@ private:
int outer_contents_frame_tree_node_id_;
// List of inner WebContents that we host.
ChildrenSet inner_web_contents_tree_nodes_;
+ // Only the root node should have this set. This indicates the WebContents
+ // whose frame tree has the focused frame.
+ WebContentsImpl* focused_web_contents_;
};
// See WebContents::Create for a description of these parameters.
@@ -969,6 +977,15 @@ private:
// up at the next animation step if the throbber is going.
void SetNotWaitingForResponse() { waiting_for_response_ = false; }
+ // Returns the focused WebContents.
+ // If there are multiple inner/outer WebContents (when embedding <webview>,
+ // <guestview>, ...) returns the single one containing the currently focused
+ // frame. Otherwise, returns this WebContents.
+ WebContentsImpl* GetFocusedWebContents();
+
+ // Returns the root of the WebContents tree.
+ WebContentsImpl* GetOutermostWebContents();
+
// Navigation helpers --------------------------------------------------------
//
// These functions are helpers for Navigate() and DidNavigate().

Powered by Google App Engine
This is Rietveld 408576698