Update nested OOPIF's screen rects when parent is repositioned in page.

content/browser/frame_host/frame_tree.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_FRAME_HOST_FRAME_TREE_H_
 #define CONTENT_BROWSER_FRAME_HOST_FRAME_TREE_H_
 
 #include <stdint.h>
 
 #include <iterator>
@@ skipping 57 matching lines @@
   };
 
   class CONTENT_EXPORT NodeRange {
    public:
     NodeIterator begin();
     NodeIterator end();
 
    private:
     friend class FrameTree;
 
-    NodeRange(FrameTree* tree, FrameTreeNode* node_to_skip);
+    NodeRange(FrameTreeNode* root, FrameTreeNode* node_to_skip);
 
-    FrameTree* const tree_;
+    FrameTreeNode* const root_;
     FrameTreeNode* const node_to_skip_;
   };
 
   class CONTENT_EXPORT ConstNodeIterator {
    public:
     ~ConstNodeIterator();
 
     ConstNodeIterator& operator++();
 
     bool operator==(const ConstNodeIterator& rhs) const;
@@ skipping 13 matching lines @@
   };
 
   class CONTENT_EXPORT ConstNodeRange {
    public:
     ConstNodeIterator begin();
     ConstNodeIterator end();
 
    private:
     friend class FrameTree;
 
-    ConstNodeRange(const FrameTree* tree);
+    ConstNodeRange(const FrameTreeNode* root);
 
-    const FrameTree* const tree_;
+    const FrameTreeNode* const root_;
   };
 
   // Each FrameTreeNode will default to using the given |navigator| for
   // navigation tasks in the frame.
   // A set of delegates are remembered here so that we can create
   // RenderFrameHostManagers.
   // TODO(creis): This set of delegates will change as we move things to
   // Navigator.
   FrameTree(Navigator* navigator,
             RenderFrameHostDelegate* render_frame_delegate,
@@ skipping 14 matching lines @@
   // Returns the first frame in this tree with the given |name|, or the main
   // frame if |name| is empty.
   // Note that this does NOT support pseudo-names like _self, _top, and _blank,
   // nor searching other FrameTrees (unlike blink::WebView::findFrameByName).
   FrameTreeNode* FindByName(const std::string& name);
 
   // Returns a range to iterate over all FrameTreeNodes in the frame tree in
   // breadth-first traversal order.
   NodeRange Nodes();
 
+  // Returns a range to iterall over all FrameTreeNodes in a subtree of the

[nasko, 2016/03/22 13:56:54]

s/iterall/iterate/? here and below.

[kenrb, 2016/03/22 15:40:20]

Done.

+  // frame tree, starting from |subtree_root|.
+  NodeRange SubtreeNodes(FrameTreeNode* subtree_root);
+
   // Returns a range to iterate over all FrameTreeNodes in the frame tree in
   // breadth-first traversal order. All FrameTreeNodes returned will be const.
   ConstNodeRange ConstNodes() const;
 
+  // Returns a range to iterall over all FrameTreeNodes in a subtree of the
+  // frame tree, starting from |subtree_root|. All FrameTreeNodes returned
+  // will be const.
+  ConstNodeRange ConstSubtreeNodes(FrameTreeNode* subtree_root);

[nasko, 2016/03/22 13:56:54]

Nick wanted to get rid of the const version. Maybe no need to add one for
subtrees?

[kenrb, 2016/03/22 15:40:20]

Done.

+
   // Adds a new child frame to the frame tree. |process_id| is required to
   // disambiguate |new_routing_id|, and it must match the process of the
   // |parent| node. Otherwise no child is added and this method returns false.
   bool AddFrame(FrameTreeNode* parent,
                 int process_id,
                 int new_routing_id,
                 blink::WebTreeScopeType scope,
                 const std::string& frame_name,
                 const std::string& frame_unique_name,
                 blink::WebSandboxFlags sandbox_flags,
@@ skipping 125 matching lines @@
 
   // Overall load progress.
   double load_progress_;
 
   DISALLOW_COPY_AND_ASSIGN(FrameTree);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_FRAME_TREE_H_