<webview>: Correctly shift focus between WebContents.

content/browser/frame_host/frame_tree.cc

Index: content/browser/frame_host/frame_tree.cc
diff --git a/content/browser/frame_host/frame_tree.cc b/content/browser/frame_host/frame_tree.cc
index 370946e428b8d77be1edc2b815a36d53b1b0ce29..5e074d7450bd8231d111b0aed2c91c3ea04693d1 100644
--- a/content/browser/frame_host/frame_tree.cc
+++ b/content/browser/frame_host/frame_tree.cc
@@ -264,18 +264,6 @@ void FrameTree::SetFocusedFrame(FrameTreeNode* node, SiteInstance* source) {
   if (node == GetFocusedFrame())
     return;
 
-  if (!node) {
-    // TODO(avallee): https://crbug.com/614463 Notify proxies here once
-    // <webview> supports oopifs inside itself.
-    if (GetFocusedFrame())
-      GetFocusedFrame()->current_frame_host()->ClearFocusedFrame();
-    focused_frame_tree_node_id_ = FrameTreeNode::kFrameTreeNodeInvalidId;
-
-    // TODO(avallee): https://crbug.com/610795 This line is not sufficient to
-    // make the test pass. There seems to be no focus change events generated.
-    root()->current_frame_host()->UpdateAXTreeData();

[alexmos, 2016/11/09 02:47:54]

Does this need to be called somewhere else, now that this path is gone?  (In any
case, sounds like it's a separate issue for a future followup.)

[avallee, 2016/11/16 05:38:09]

This was for the clear frame case, this was essentially a dupe of what happens
at the bottom of this function.

-    return;
-  }
 
   std::set<SiteInstance*> frame_tree_site_instances =
       CollectSiteInstances(this);