Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 1 | // Copyright 2013 The Chromium Authors. All rights reserved. |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
[email protected] | d4a8ca48 | 2013-10-30 21:06:40 | [diff] [blame] | 5 | #include "content/browser/frame_host/frame_tree.h" |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 6 | |
avi | b734894 | 2015-12-25 20:57:10 | [diff] [blame] | 7 | #include <stddef.h> |
| 8 | |
| 9 | #include "base/macros.h" |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 10 | #include "base/run_loop.h" |
[email protected] | 7cc7ebd | 2013-10-08 00:59:00 | [diff] [blame] | 11 | #include "base/strings/string_number_conversions.h" |
[email protected] | 6ea6bdf | 2013-12-06 13:35:01 | [diff] [blame] | 12 | #include "content/browser/frame_host/navigator_impl.h" |
[email protected] | 2a18ee22 | 2013-11-21 07:52:44 | [diff] [blame] | 13 | #include "content/browser/frame_host/render_frame_host_factory.h" |
[email protected] | d4a8ca48 | 2013-10-30 21:06:40 | [diff] [blame] | 14 | #include "content/browser/frame_host/render_frame_host_impl.h" |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 15 | #include "content/browser/renderer_host/render_view_host_impl.h" |
[email protected] | 94d0cc1 | 2013-12-18 00:07:41 | [diff] [blame] | 16 | #include "content/browser/web_contents/web_contents_impl.h" |
nasko | b985af1 | 2015-02-06 04:15:33 | [diff] [blame] | 17 | #include "content/common/frame_messages.h" |
raymes | 3145780 | 2016-07-20 06:08:09 | [diff] [blame] | 18 | #include "content/common/frame_owner_properties.h" |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 19 | #include "content/public/browser/web_contents_observer.h" |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 20 | #include "content/public/test/mock_render_process_host.h" |
| 21 | #include "content/public/test/test_browser_context.h" |
| 22 | #include "content/public/test/test_browser_thread_bundle.h" |
[email protected] | 6b50e36 | 2014-08-15 05:15:59 | [diff] [blame] | 23 | #include "content/test/test_render_frame_host.h" |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 24 | #include "content/test/test_render_view_host.h" |
| 25 | #include "content/test/test_web_contents.h" |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 26 | #include "testing/gtest/include/gtest/gtest.h" |
Blink Reformat | a30d423 | 2018-04-07 15:31:06 | [diff] [blame] | 27 | #include "third_party/blink/public/common/frame/frame_policy.h" |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 28 | |
| 29 | namespace content { |
dgrogan | fb22f9a | 2014-10-20 21:32:32 | [diff] [blame] | 30 | |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 31 | namespace { |
| 32 | |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 33 | // Appends a description of the structure of the frame tree to |result|. |
| 34 | void AppendTreeNodeState(FrameTreeNode* node, std::string* result) { |
| 35 | result->append( |
| 36 | base::Int64ToString(node->current_frame_host()->GetRoutingID())); |
schenney | 6408fed2 | 2015-04-17 17:44:57 | [diff] [blame] | 37 | if (!node->current_frame_host()->IsRenderFrameLive()) |
| 38 | result->append("*"); // Asterisk next to dead frames. |
| 39 | |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 40 | if (!node->frame_name().empty()) { |
| 41 | result->append(" '"); |
| 42 | result->append(node->frame_name()); |
| 43 | result->append("'"); |
| 44 | } |
| 45 | result->append(": ["); |
| 46 | const char* separator = ""; |
| 47 | for (size_t i = 0; i < node->child_count(); i++) { |
| 48 | result->append(separator); |
| 49 | AppendTreeNodeState(node->child_at(i), result); |
| 50 | separator = ", "; |
| 51 | } |
| 52 | result->append("]"); |
| 53 | } |
| 54 | |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 55 | service_manager::mojom::InterfaceProviderRequest |
| 56 | CreateStubInterfaceProviderRequest() { |
| 57 | return TestRenderFrameHost::CreateStubInterfaceProviderRequest(); |
| 58 | } |
| 59 | |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 60 | // Logs calls to WebContentsObserver along with the state of the frame tree, |
| 61 | // for later use in EXPECT_EQ(). |
| 62 | class TreeWalkingWebContentsLogger : public WebContentsObserver { |
| 63 | public: |
| 64 | explicit TreeWalkingWebContentsLogger(WebContents* web_contents) |
| 65 | : WebContentsObserver(web_contents) {} |
| 66 | |
dcheng | c2282aa | 2014-10-21 12:07:58 | [diff] [blame] | 67 | ~TreeWalkingWebContentsLogger() override { |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 68 | EXPECT_EQ("", log_) << "Activity logged that was not expected"; |
| 69 | } |
| 70 | |
| 71 | // Gets and resets the log, which is a string of what happened. |
| 72 | std::string GetLog() { |
| 73 | std::string result = log_; |
| 74 | log_.clear(); |
| 75 | return result; |
| 76 | } |
| 77 | |
| 78 | // content::WebContentsObserver implementation. |
dcheng | c2282aa | 2014-10-21 12:07:58 | [diff] [blame] | 79 | void RenderFrameCreated(RenderFrameHost* render_frame_host) override { |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 80 | LogWhatHappened("RenderFrameCreated", render_frame_host); |
| 81 | } |
| 82 | |
dcheng | c2282aa | 2014-10-21 12:07:58 | [diff] [blame] | 83 | void RenderFrameHostChanged(RenderFrameHost* old_host, |
| 84 | RenderFrameHost* new_host) override { |
[email protected] | 02d7b6e | 2014-06-24 21:01:50 | [diff] [blame] | 85 | if (old_host) |
nasko | f5940b9f | 2015-03-02 23:04:05 | [diff] [blame] | 86 | LogWhatHappened("RenderFrameHostChanged(old)", old_host); |
| 87 | LogWhatHappened("RenderFrameHostChanged(new)", new_host); |
[email protected] | 02d7b6e | 2014-06-24 21:01:50 | [diff] [blame] | 88 | } |
| 89 | |
dcheng | c2282aa | 2014-10-21 12:07:58 | [diff] [blame] | 90 | void RenderFrameDeleted(RenderFrameHost* render_frame_host) override { |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 91 | LogWhatHappened("RenderFrameDeleted", render_frame_host); |
| 92 | } |
| 93 | |
dcheng | c2282aa | 2014-10-21 12:07:58 | [diff] [blame] | 94 | void RenderProcessGone(base::TerminationStatus status) override { |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 95 | LogWhatHappened("RenderProcessGone"); |
| 96 | } |
| 97 | |
| 98 | private: |
| 99 | void LogWhatHappened(const std::string& event_name) { |
| 100 | if (!log_.empty()) { |
| 101 | log_.append("\n"); |
| 102 | } |
| 103 | log_.append(event_name + " -> "); |
| 104 | AppendTreeNodeState( |
| 105 | static_cast<WebContentsImpl*>(web_contents())->GetFrameTree()->root(), |
| 106 | &log_); |
| 107 | } |
| 108 | |
| 109 | void LogWhatHappened(const std::string& event_name, RenderFrameHost* rfh) { |
| 110 | LogWhatHappened( |
| 111 | base::StringPrintf("%s(%d)", event_name.c_str(), rfh->GetRoutingID())); |
| 112 | } |
| 113 | |
| 114 | std::string log_; |
| 115 | |
| 116 | DISALLOW_COPY_AND_ASSIGN(TreeWalkingWebContentsLogger); |
| 117 | }; |
| 118 | |
dgrogan | fb22f9a | 2014-10-20 21:32:32 | [diff] [blame] | 119 | } // namespace |
| 120 | |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 121 | class FrameTreeTest : public RenderViewHostImplTestHarness { |
[email protected] | 7cc7ebd | 2013-10-08 00:59:00 | [diff] [blame] | 122 | protected: |
| 123 | // Prints a FrameTree, for easy assertions of the tree hierarchy. |
| 124 | std::string GetTreeState(FrameTree* frame_tree) { |
| 125 | std::string result; |
[email protected] | fa944cb8 | 2013-11-15 17:51:21 | [diff] [blame] | 126 | AppendTreeNodeState(frame_tree->root(), &result); |
[email protected] | 7cc7ebd | 2013-10-08 00:59:00 | [diff] [blame] | 127 | return result; |
| 128 | } |
nick | 4ed97029 | 2016-01-20 21:46:45 | [diff] [blame] | 129 | |
| 130 | std::string GetTraversalOrder(FrameTree* frame_tree, |
Alex Moshchuk | 27caae8 | 2017-09-11 23:11:18 | [diff] [blame] | 131 | FrameTreeNode* subtree_to_skip) { |
nick | 4ed97029 | 2016-01-20 21:46:45 | [diff] [blame] | 132 | std::string result; |
Alex Moshchuk | 27caae8 | 2017-09-11 23:11:18 | [diff] [blame] | 133 | for (FrameTreeNode* node : |
| 134 | frame_tree->NodesExceptSubtree(subtree_to_skip)) { |
dcheng | 57e39e2 | 2016-01-21 00:25:38 | [diff] [blame] | 135 | if (!result.empty()) |
| 136 | result += " "; |
| 137 | result += base::Int64ToString(node->current_frame_host()->GetRoutingID()); |
| 138 | } |
nick | 4ed97029 | 2016-01-20 21:46:45 | [diff] [blame] | 139 | return result; |
| 140 | } |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 141 | }; |
| 142 | |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 143 | // Exercise tree manipulation routines. |
| 144 | // - Add a series of nodes and verify tree structure. |
| 145 | // - Remove a series of nodes and verify tree structure. |
nasko | 9f2261b | 2015-07-02 11:40:26 | [diff] [blame] | 146 | TEST_F(FrameTreeTest, Shape) { |
nick | 8814e65 | 2015-12-18 01:44:12 | [diff] [blame] | 147 | main_test_rfh()->InitializeRenderFrameIfNeeded(); |
| 148 | |
[email protected] | 94d0cc1 | 2013-12-18 00:07:41 | [diff] [blame] | 149 | // Use the FrameTree of the WebContents so that it has all the delegates it |
| 150 | // needs. We may want to consider a test version of this. |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 151 | FrameTree* frame_tree = contents()->GetFrameTree(); |
[email protected] | 58faf94 | 2014-02-20 21:03:58 | [diff] [blame] | 152 | FrameTreeNode* root = frame_tree->root(); |
[email protected] | 190b8c5 | 2013-11-09 01:35:44 | [diff] [blame] | 153 | |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 154 | std::string no_children_node("no children node"); |
| 155 | std::string deep_subtree("node with deep subtree"); |
dgrogan | fb22f9a | 2014-10-20 21:32:32 | [diff] [blame] | 156 | int process_id = root->current_frame_host()->GetProcess()->GetID(); |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 157 | |
nasko | 9f2261b | 2015-07-02 11:40:26 | [diff] [blame] | 158 | // Do not navigate each frame separately, since that will clutter the test |
| 159 | // itself. Instead, leave them in "not live" state, which is indicated by the |
| 160 | // * after the frame id, since this test cares about the shape, not the |
nick | 8814e65 | 2015-12-18 01:44:12 | [diff] [blame] | 161 | // frame liveness. |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 162 | EXPECT_EQ("3: []", GetTreeState(frame_tree)); |
[email protected] | 7cc7ebd | 2013-10-08 00:59:00 | [diff] [blame] | 163 | |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 164 | // Simulate attaching a series of frames to build the frame tree. |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 165 | frame_tree->AddFrame(root, process_id, 14, |
| 166 | CreateStubInterfaceProviderRequest(), |
| 167 | blink::WebTreeScopeType::kDocument, std::string(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 168 | "uniqueName0", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 169 | blink::FramePolicy(), FrameOwnerProperties(), false); |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 170 | frame_tree->AddFrame(root, process_id, 15, |
| 171 | CreateStubInterfaceProviderRequest(), |
| 172 | blink::WebTreeScopeType::kDocument, std::string(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 173 | "uniqueName1", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 174 | blink::FramePolicy(), FrameOwnerProperties(), false); |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 175 | frame_tree->AddFrame(root, process_id, 16, |
| 176 | CreateStubInterfaceProviderRequest(), |
| 177 | blink::WebTreeScopeType::kDocument, std::string(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 178 | "uniqueName2", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 179 | blink::FramePolicy(), FrameOwnerProperties(), false); |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 180 | frame_tree->AddFrame(root->child_at(0), process_id, 244, |
| 181 | CreateStubInterfaceProviderRequest(), |
| 182 | blink::WebTreeScopeType::kDocument, std::string(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 183 | "uniqueName3", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 184 | blink::FramePolicy(), FrameOwnerProperties(), false); |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 185 | frame_tree->AddFrame(root->child_at(1), process_id, 255, |
| 186 | CreateStubInterfaceProviderRequest(), |
| 187 | blink::WebTreeScopeType::kDocument, no_children_node, |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 188 | "uniqueName4", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 189 | blink::FramePolicy(), FrameOwnerProperties(), false); |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 190 | frame_tree->AddFrame(root->child_at(0), process_id, 245, |
| 191 | CreateStubInterfaceProviderRequest(), |
| 192 | blink::WebTreeScopeType::kDocument, std::string(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 193 | "uniqueName5", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 194 | blink::FramePolicy(), FrameOwnerProperties(), false); |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 195 | |
dcheng | 3ce04b6 | 2015-10-26 23:30:55 | [diff] [blame] | 196 | EXPECT_EQ( |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 197 | "3: [14: [244: [], 245: []], " |
nick | 8814e65 | 2015-12-18 01:44:12 | [diff] [blame] | 198 | "15: [255 'no children node': []], " |
| 199 | "16: []]", |
dcheng | 3ce04b6 | 2015-10-26 23:30:55 | [diff] [blame] | 200 | GetTreeState(frame_tree)); |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 201 | |
[email protected] | 58faf94 | 2014-02-20 21:03:58 | [diff] [blame] | 202 | FrameTreeNode* child_16 = root->child_at(2); |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 203 | frame_tree->AddFrame(child_16, process_id, 264, |
| 204 | CreateStubInterfaceProviderRequest(), |
| 205 | blink::WebTreeScopeType::kDocument, std::string(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 206 | "uniqueName6", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 207 | blink::FramePolicy(), FrameOwnerProperties(), false); |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 208 | frame_tree->AddFrame(child_16, process_id, 265, |
| 209 | CreateStubInterfaceProviderRequest(), |
| 210 | blink::WebTreeScopeType::kDocument, std::string(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 211 | "uniqueName7", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 212 | blink::FramePolicy(), FrameOwnerProperties(), false); |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 213 | frame_tree->AddFrame(child_16, process_id, 266, |
| 214 | CreateStubInterfaceProviderRequest(), |
| 215 | blink::WebTreeScopeType::kDocument, std::string(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 216 | "uniqueName8", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 217 | blink::FramePolicy(), FrameOwnerProperties(), false); |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 218 | frame_tree->AddFrame(child_16, process_id, 267, |
| 219 | CreateStubInterfaceProviderRequest(), |
| 220 | blink::WebTreeScopeType::kDocument, deep_subtree, |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 221 | "uniqueName9", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 222 | blink::FramePolicy(), FrameOwnerProperties(), false); |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 223 | frame_tree->AddFrame(child_16, process_id, 268, |
| 224 | CreateStubInterfaceProviderRequest(), |
| 225 | blink::WebTreeScopeType::kDocument, std::string(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 226 | "uniqueName10", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 227 | blink::FramePolicy(), FrameOwnerProperties(), false); |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 228 | |
[email protected] | 58faf94 | 2014-02-20 21:03:58 | [diff] [blame] | 229 | FrameTreeNode* child_267 = child_16->child_at(3); |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 230 | frame_tree->AddFrame(child_267, process_id, 365, |
| 231 | CreateStubInterfaceProviderRequest(), |
| 232 | blink::WebTreeScopeType::kDocument, std::string(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 233 | "uniqueName11", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 234 | blink::FramePolicy(), FrameOwnerProperties(), false); |
dcheng | 860817a | 2015-05-22 03:16:56 | [diff] [blame] | 235 | frame_tree->AddFrame(child_267->child_at(0), process_id, 455, |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 236 | CreateStubInterfaceProviderRequest(), |
Blink Reformat | 1c4d759e | 2017-04-09 16:34:54 | [diff] [blame] | 237 | blink::WebTreeScopeType::kDocument, std::string(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 238 | "uniqueName12", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 239 | blink::FramePolicy(), FrameOwnerProperties(), false); |
dgrogan | fb22f9a | 2014-10-20 21:32:32 | [diff] [blame] | 240 | frame_tree->AddFrame(child_267->child_at(0)->child_at(0), process_id, 555, |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 241 | CreateStubInterfaceProviderRequest(), |
Blink Reformat | 1c4d759e | 2017-04-09 16:34:54 | [diff] [blame] | 242 | blink::WebTreeScopeType::kDocument, std::string(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 243 | "uniqueName13", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 244 | blink::FramePolicy(), FrameOwnerProperties(), false); |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 245 | frame_tree->AddFrame(child_267->child_at(0)->child_at(0)->child_at(0), |
| 246 | process_id, 655, CreateStubInterfaceProviderRequest(), |
| 247 | blink::WebTreeScopeType::kDocument, std::string(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 248 | "uniqueName14", false, base::UnguessableToken::Create(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 249 | blink::FramePolicy(), FrameOwnerProperties(), false); |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 250 | |
[email protected] | 7cc7ebd | 2013-10-08 00:59:00 | [diff] [blame] | 251 | // Now that's it's fully built, verify the tree structure is as expected. |
dcheng | 3ce04b6 | 2015-10-26 23:30:55 | [diff] [blame] | 252 | EXPECT_EQ( |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 253 | "3: [14: [244: [], 245: []], " |
nick | 8814e65 | 2015-12-18 01:44:12 | [diff] [blame] | 254 | "15: [255 'no children node': []], " |
| 255 | "16: [264: [], 265: [], 266: [], " |
| 256 | "267 'node with deep subtree': " |
| 257 | "[365: [455: [555: [655: []]]]], 268: []]]", |
dcheng | 3ce04b6 | 2015-10-26 23:30:55 | [diff] [blame] | 258 | GetTreeState(frame_tree)); |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 259 | |
nick | 4ed97029 | 2016-01-20 21:46:45 | [diff] [blame] | 260 | // Verify that traversal order is breadth first, even if we skip a subtree. |
| 261 | FrameTreeNode* child_14 = root->child_at(0); |
| 262 | FrameTreeNode* child_15 = root->child_at(1); |
| 263 | FrameTreeNode* child_244 = child_14->child_at(0); |
| 264 | FrameTreeNode* child_245 = child_14->child_at(1); |
[email protected] | 58faf94 | 2014-02-20 21:03:58 | [diff] [blame] | 265 | FrameTreeNode* child_555 = child_267->child_at(0)->child_at(0)->child_at(0); |
nick | 4ed97029 | 2016-01-20 21:46:45 | [diff] [blame] | 266 | FrameTreeNode* child_655 = child_555->child_at(0); |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 267 | EXPECT_EQ("3 14 15 16 244 245 255 264 265 266 267 268 365 455 555 655", |
nick | 4ed97029 | 2016-01-20 21:46:45 | [diff] [blame] | 268 | GetTraversalOrder(frame_tree, nullptr)); |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 269 | EXPECT_EQ("3", GetTraversalOrder(frame_tree, root)); |
| 270 | EXPECT_EQ("3 14 15 16 255 264 265 266 267 268 365 455 555 655", |
nick | 4ed97029 | 2016-01-20 21:46:45 | [diff] [blame] | 271 | GetTraversalOrder(frame_tree, child_14)); |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 272 | EXPECT_EQ("3 14 15 16 244 245 255 264 265 266 267 268 365 455 555 655", |
nick | 4ed97029 | 2016-01-20 21:46:45 | [diff] [blame] | 273 | GetTraversalOrder(frame_tree, child_244)); |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 274 | EXPECT_EQ("3 14 15 16 244 245 255 264 265 266 267 268 365 455 555 655", |
nick | 4ed97029 | 2016-01-20 21:46:45 | [diff] [blame] | 275 | GetTraversalOrder(frame_tree, child_245)); |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 276 | EXPECT_EQ("3 14 15 16 244 245 264 265 266 267 268 365 455 555 655", |
nick | 4ed97029 | 2016-01-20 21:46:45 | [diff] [blame] | 277 | GetTraversalOrder(frame_tree, child_15)); |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 278 | EXPECT_EQ("3 14 15 16 244 245 255 264 265 266 267 268", |
nick | 4ed97029 | 2016-01-20 21:46:45 | [diff] [blame] | 279 | GetTraversalOrder(frame_tree, child_267)); |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 280 | EXPECT_EQ("3 14 15 16 244 245 255 264 265 266 267 268 365 455 555", |
Alex Moshchuk | 27caae8 | 2017-09-11 23:11:18 | [diff] [blame] | 281 | GetTraversalOrder(frame_tree, child_555)); |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 282 | EXPECT_EQ("3 14 15 16 244 245 255 264 265 266 267 268 365 455 555 655", |
nick | 4ed97029 | 2016-01-20 21:46:45 | [diff] [blame] | 283 | GetTraversalOrder(frame_tree, child_655)); |
| 284 | |
[email protected] | 58faf94 | 2014-02-20 21:03:58 | [diff] [blame] | 285 | frame_tree->RemoveFrame(child_555); |
dcheng | 3ce04b6 | 2015-10-26 23:30:55 | [diff] [blame] | 286 | EXPECT_EQ( |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 287 | "3: [14: [244: [], 245: []], " |
nick | 8814e65 | 2015-12-18 01:44:12 | [diff] [blame] | 288 | "15: [255 'no children node': []], " |
| 289 | "16: [264: [], 265: [], 266: [], " |
| 290 | "267 'node with deep subtree': " |
| 291 | "[365: [455: []]], 268: []]]", |
dcheng | 3ce04b6 | 2015-10-26 23:30:55 | [diff] [blame] | 292 | GetTreeState(frame_tree)); |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 293 | |
[email protected] | 58faf94 | 2014-02-20 21:03:58 | [diff] [blame] | 294 | frame_tree->RemoveFrame(child_16->child_at(1)); |
dcheng | 3ce04b6 | 2015-10-26 23:30:55 | [diff] [blame] | 295 | EXPECT_EQ( |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 296 | "3: [14: [244: [], 245: []], " |
nick | 8814e65 | 2015-12-18 01:44:12 | [diff] [blame] | 297 | "15: [255 'no children node': []], " |
| 298 | "16: [264: [], 266: [], " |
| 299 | "267 'node with deep subtree': " |
| 300 | "[365: [455: []]], 268: []]]", |
dcheng | 3ce04b6 | 2015-10-26 23:30:55 | [diff] [blame] | 301 | GetTreeState(frame_tree)); |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 302 | |
[email protected] | 58faf94 | 2014-02-20 21:03:58 | [diff] [blame] | 303 | frame_tree->RemoveFrame(root->child_at(1)); |
dcheng | 3ce04b6 | 2015-10-26 23:30:55 | [diff] [blame] | 304 | EXPECT_EQ( |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 305 | "3: [14: [244: [], 245: []], " |
nick | 8814e65 | 2015-12-18 01:44:12 | [diff] [blame] | 306 | "16: [264: [], 266: [], " |
| 307 | "267 'node with deep subtree': " |
| 308 | "[365: [455: []]], 268: []]]", |
dcheng | 3ce04b6 | 2015-10-26 23:30:55 | [diff] [blame] | 309 | GetTreeState(frame_tree)); |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 310 | } |
| 311 | |
creis | 6a93a81 | 2015-04-24 23:13:17 | [diff] [blame] | 312 | // Ensure frames can be found by frame_tree_node_id, routing ID, or name. |
Charlie Reis | b140562 | 2018-04-02 22:52:39 | [diff] [blame] | 313 | TEST_F(FrameTreeTest, FindFrames) { |
lfg | 269b702f | 2015-06-08 19:28:19 | [diff] [blame] | 314 | main_test_rfh()->InitializeRenderFrameIfNeeded(); |
| 315 | |
creis | 6a93a81 | 2015-04-24 23:13:17 | [diff] [blame] | 316 | // Add a few child frames to the main frame. |
| 317 | FrameTree* frame_tree = contents()->GetFrameTree(); |
| 318 | FrameTreeNode* root = frame_tree->root(); |
lfg | 269b702f | 2015-06-08 19:28:19 | [diff] [blame] | 319 | |
lukasza | 464d869 | 2016-02-22 19:26:32 | [diff] [blame] | 320 | main_test_rfh()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 321 | 22, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 322 | blink::WebTreeScopeType::kDocument, "child0", "uniqueName0", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 323 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 324 | FrameOwnerProperties()); |
lukasza | 464d869 | 2016-02-22 19:26:32 | [diff] [blame] | 325 | main_test_rfh()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 326 | 23, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 327 | blink::WebTreeScopeType::kDocument, "child1", "uniqueName1", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 328 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 329 | FrameOwnerProperties()); |
lukasza | 464d869 | 2016-02-22 19:26:32 | [diff] [blame] | 330 | main_test_rfh()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 331 | 24, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 332 | blink::WebTreeScopeType::kDocument, std::string(), "uniqueName2", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 333 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 334 | FrameOwnerProperties()); |
creis | 6a93a81 | 2015-04-24 23:13:17 | [diff] [blame] | 335 | FrameTreeNode* child0 = root->child_at(0); |
| 336 | FrameTreeNode* child1 = root->child_at(1); |
creis | 6a93a81 | 2015-04-24 23:13:17 | [diff] [blame] | 337 | FrameTreeNode* child2 = root->child_at(2); |
| 338 | |
| 339 | // Add one grandchild frame. |
dcheng | 860817a | 2015-05-22 03:16:56 | [diff] [blame] | 340 | child1->current_frame_host()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 341 | 33, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 342 | blink::WebTreeScopeType::kDocument, "grandchild", "uniqueName3", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 343 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 344 | FrameOwnerProperties()); |
creis | 6a93a81 | 2015-04-24 23:13:17 | [diff] [blame] | 345 | FrameTreeNode* grandchild = child1->child_at(0); |
| 346 | |
| 347 | // Ensure they can be found by FTN id. |
| 348 | EXPECT_EQ(root, frame_tree->FindByID(root->frame_tree_node_id())); |
| 349 | EXPECT_EQ(child0, frame_tree->FindByID(child0->frame_tree_node_id())); |
| 350 | EXPECT_EQ(child1, frame_tree->FindByID(child1->frame_tree_node_id())); |
| 351 | EXPECT_EQ(child2, frame_tree->FindByID(child2->frame_tree_node_id())); |
| 352 | EXPECT_EQ(grandchild, frame_tree->FindByID(grandchild->frame_tree_node_id())); |
| 353 | EXPECT_EQ(nullptr, frame_tree->FindByID(-1)); |
| 354 | |
| 355 | // Ensure they can be found by routing id. |
| 356 | int process_id = main_test_rfh()->GetProcess()->GetID(); |
| 357 | EXPECT_EQ(root, frame_tree->FindByRoutingID(process_id, |
| 358 | main_test_rfh()->GetRoutingID())); |
| 359 | EXPECT_EQ(child0, frame_tree->FindByRoutingID(process_id, 22)); |
| 360 | EXPECT_EQ(child1, frame_tree->FindByRoutingID(process_id, 23)); |
| 361 | EXPECT_EQ(child2, frame_tree->FindByRoutingID(process_id, 24)); |
| 362 | EXPECT_EQ(grandchild, frame_tree->FindByRoutingID(process_id, 33)); |
| 363 | EXPECT_EQ(nullptr, frame_tree->FindByRoutingID(process_id, 37)); |
| 364 | |
| 365 | // Ensure they can be found by name, if they have one. |
| 366 | EXPECT_EQ(root, frame_tree->FindByName(std::string())); |
| 367 | EXPECT_EQ(child0, frame_tree->FindByName("child0")); |
| 368 | EXPECT_EQ(child1, frame_tree->FindByName("child1")); |
| 369 | EXPECT_EQ(grandchild, frame_tree->FindByName("grandchild")); |
| 370 | EXPECT_EQ(nullptr, frame_tree->FindByName("no such frame")); |
| 371 | } |
| 372 | |
paulmeyer | 322777fb | 2016-05-16 23:15:39 | [diff] [blame] | 373 | // Check that PreviousSibling() and NextSibling() are retrieved correctly. |
| 374 | TEST_F(FrameTreeTest, GetSibling) { |
lfg | 269b702f | 2015-06-08 19:28:19 | [diff] [blame] | 375 | main_test_rfh()->InitializeRenderFrameIfNeeded(); |
| 376 | |
alexmos | 9f8705a | 2015-05-06 19:58:59 | [diff] [blame] | 377 | // Add a few child frames to the main frame. |
| 378 | FrameTree* frame_tree = contents()->GetFrameTree(); |
| 379 | FrameTreeNode* root = frame_tree->root(); |
lukasza | 464d869 | 2016-02-22 19:26:32 | [diff] [blame] | 380 | main_test_rfh()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 381 | 22, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 382 | blink::WebTreeScopeType::kDocument, "child0", "uniqueName0", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 383 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 384 | FrameOwnerProperties()); |
lukasza | 464d869 | 2016-02-22 19:26:32 | [diff] [blame] | 385 | main_test_rfh()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 386 | 23, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 387 | blink::WebTreeScopeType::kDocument, "child1", "uniqueName1", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 388 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 389 | FrameOwnerProperties()); |
lukasza | 464d869 | 2016-02-22 19:26:32 | [diff] [blame] | 390 | main_test_rfh()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 391 | 24, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 392 | blink::WebTreeScopeType::kDocument, "child2", "uniqueName2", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 393 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 394 | FrameOwnerProperties()); |
alexmos | 9f8705a | 2015-05-06 19:58:59 | [diff] [blame] | 395 | FrameTreeNode* child0 = root->child_at(0); |
| 396 | FrameTreeNode* child1 = root->child_at(1); |
| 397 | FrameTreeNode* child2 = root->child_at(2); |
| 398 | |
| 399 | // Add one grandchild frame. |
dcheng | 860817a | 2015-05-22 03:16:56 | [diff] [blame] | 400 | child1->current_frame_host()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 401 | 33, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 402 | blink::WebTreeScopeType::kDocument, "grandchild", "uniqueName3", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 403 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 404 | FrameOwnerProperties()); |
alexmos | 9f8705a | 2015-05-06 19:58:59 | [diff] [blame] | 405 | FrameTreeNode* grandchild = child1->child_at(0); |
| 406 | |
paulmeyer | 322777fb | 2016-05-16 23:15:39 | [diff] [blame] | 407 | // Test PreviousSibling(). |
alexmos | 9f8705a | 2015-05-06 19:58:59 | [diff] [blame] | 408 | EXPECT_EQ(nullptr, root->PreviousSibling()); |
| 409 | EXPECT_EQ(nullptr, child0->PreviousSibling()); |
| 410 | EXPECT_EQ(child0, child1->PreviousSibling()); |
| 411 | EXPECT_EQ(child1, child2->PreviousSibling()); |
| 412 | EXPECT_EQ(nullptr, grandchild->PreviousSibling()); |
paulmeyer | 322777fb | 2016-05-16 23:15:39 | [diff] [blame] | 413 | |
| 414 | // Test NextSibling(). |
| 415 | EXPECT_EQ(nullptr, root->NextSibling()); |
| 416 | EXPECT_EQ(child1, child0->NextSibling()); |
| 417 | EXPECT_EQ(child2, child1->NextSibling()); |
| 418 | EXPECT_EQ(nullptr, child2->NextSibling()); |
| 419 | EXPECT_EQ(nullptr, grandchild->NextSibling()); |
alexmos | 9f8705a | 2015-05-06 19:58:59 | [diff] [blame] | 420 | } |
| 421 | |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 422 | // Do some simple manipulations of the frame tree, making sure that |
| 423 | // WebContentsObservers see a consistent view of the tree as we go. |
| 424 | TEST_F(FrameTreeTest, ObserverWalksTreeDuringFrameCreation) { |
| 425 | TreeWalkingWebContentsLogger activity(contents()); |
schenney | 6408fed2 | 2015-04-17 17:44:57 | [diff] [blame] | 426 | contents()->NavigateAndCommit(GURL("http://www.google.com")); |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 427 | EXPECT_EQ("RenderFrameCreated(3) -> 3: []", activity.GetLog()); |
schenney | 6408fed2 | 2015-04-17 17:44:57 | [diff] [blame] | 428 | |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 429 | FrameTree* frame_tree = contents()->GetFrameTree(); |
| 430 | FrameTreeNode* root = frame_tree->root(); |
| 431 | |
| 432 | // Simulate attaching a series of frames to build the frame tree. |
lukasza | 464d869 | 2016-02-22 19:26:32 | [diff] [blame] | 433 | main_test_rfh()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 434 | 14, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 435 | blink::WebTreeScopeType::kDocument, std::string(), "uniqueName0", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 436 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 437 | FrameOwnerProperties()); |
nasko | f5940b9f | 2015-03-02 23:04:05 | [diff] [blame] | 438 | EXPECT_EQ( |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 439 | "RenderFrameHostChanged(new)(14) -> 3: []\n" |
| 440 | "RenderFrameCreated(14) -> 3: [14: []]", |
nasko | f5940b9f | 2015-03-02 23:04:05 | [diff] [blame] | 441 | activity.GetLog()); |
lukasza | 464d869 | 2016-02-22 19:26:32 | [diff] [blame] | 442 | main_test_rfh()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 443 | 18, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 444 | blink::WebTreeScopeType::kDocument, std::string(), "uniqueName1", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 445 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 446 | FrameOwnerProperties()); |
nasko | f5940b9f | 2015-03-02 23:04:05 | [diff] [blame] | 447 | EXPECT_EQ( |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 448 | "RenderFrameHostChanged(new)(18) -> 3: [14: []]\n" |
| 449 | "RenderFrameCreated(18) -> 3: [14: [], 18: []]", |
nasko | f5940b9f | 2015-03-02 23:04:05 | [diff] [blame] | 450 | activity.GetLog()); |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 451 | frame_tree->RemoveFrame(root->child_at(0)); |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 452 | EXPECT_EQ("RenderFrameDeleted(14) -> 3: [18: []]", activity.GetLog()); |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 453 | frame_tree->RemoveFrame(root->child_at(0)); |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 454 | EXPECT_EQ("RenderFrameDeleted(18) -> 3: []", activity.GetLog()); |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 455 | } |
| 456 | |
| 457 | // Make sure that WebContentsObservers see a consistent view of the tree after |
| 458 | // recovery from a render process crash. |
| 459 | TEST_F(FrameTreeTest, ObserverWalksTreeAfterCrash) { |
| 460 | TreeWalkingWebContentsLogger activity(contents()); |
schenney | 6408fed2 | 2015-04-17 17:44:57 | [diff] [blame] | 461 | contents()->NavigateAndCommit(GURL("http://www.google.com")); |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 462 | EXPECT_EQ("RenderFrameCreated(3) -> 3: []", activity.GetLog()); |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 463 | |
lukasza | 464d869 | 2016-02-22 19:26:32 | [diff] [blame] | 464 | main_test_rfh()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 465 | 22, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 466 | blink::WebTreeScopeType::kDocument, std::string(), "uniqueName0", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 467 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 468 | FrameOwnerProperties()); |
nasko | f5940b9f | 2015-03-02 23:04:05 | [diff] [blame] | 469 | EXPECT_EQ( |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 470 | "RenderFrameHostChanged(new)(22) -> 3: []\n" |
| 471 | "RenderFrameCreated(22) -> 3: [22: []]", |
nasko | f5940b9f | 2015-03-02 23:04:05 | [diff] [blame] | 472 | activity.GetLog()); |
lukasza | 464d869 | 2016-02-22 19:26:32 | [diff] [blame] | 473 | main_test_rfh()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 474 | 23, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 475 | blink::WebTreeScopeType::kDocument, std::string(), "uniqueName1", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 476 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 477 | FrameOwnerProperties()); |
nasko | f5940b9f | 2015-03-02 23:04:05 | [diff] [blame] | 478 | EXPECT_EQ( |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 479 | "RenderFrameHostChanged(new)(23) -> 3: [22: []]\n" |
| 480 | "RenderFrameCreated(23) -> 3: [22: [], 23: []]", |
nasko | f5940b9f | 2015-03-02 23:04:05 | [diff] [blame] | 481 | activity.GetLog()); |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 482 | |
| 483 | // Crash the renderer |
nick | 16b0765 | 2015-04-18 02:35:31 | [diff] [blame] | 484 | main_test_rfh()->GetProcess()->SimulateCrash(); |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 485 | EXPECT_EQ( |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 486 | "RenderProcessGone -> 3*: [22*: [], 23*: []]\n" |
| 487 | "RenderFrameDeleted(23) -> 3*: [22*: [], 23*: []]\n" |
| 488 | "RenderFrameDeleted(22) -> 3*: [22*: [], 23*: []]\n" |
| 489 | "RenderFrameDeleted(3) -> 3*: []", |
[email protected] | 1426607 | 2014-04-19 00:35:20 | [diff] [blame] | 490 | activity.GetLog()); |
| 491 | } |
| 492 | |
dgrogan | fb22f9a | 2014-10-20 21:32:32 | [diff] [blame] | 493 | // Ensure that frames are not added to the tree, if the process passed in |
| 494 | // is different than the process of the parent node. |
| 495 | TEST_F(FrameTreeTest, FailAddFrameWithWrongProcessId) { |
schenney | 6408fed2 | 2015-04-17 17:44:57 | [diff] [blame] | 496 | contents()->NavigateAndCommit(GURL("http://www.google.com")); |
dgrogan | fb22f9a | 2014-10-20 21:32:32 | [diff] [blame] | 497 | FrameTree* frame_tree = contents()->GetFrameTree(); |
| 498 | FrameTreeNode* root = frame_tree->root(); |
| 499 | int process_id = root->current_frame_host()->GetProcess()->GetID(); |
| 500 | |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 501 | ASSERT_EQ("3: []", GetTreeState(frame_tree)); |
dgrogan | fb22f9a | 2014-10-20 21:32:32 | [diff] [blame] | 502 | |
| 503 | // Simulate attaching a frame from mismatched process id. |
dcheng | 5f60abb | 2015-05-28 01:39:36 | [diff] [blame] | 504 | ASSERT_FALSE(frame_tree->AddFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 505 | root, process_id + 1, 1, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 506 | blink::WebTreeScopeType::kDocument, std::string(), "uniqueName0", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 507 | base::UnguessableToken::Create(), blink::FramePolicy(), |
Shubhie Panicker | ddf2a4e | 2018-03-06 00:09:06 | [diff] [blame] | 508 | FrameOwnerProperties(), false)); |
Albert J. Wong | cb00463 | 2018-07-10 22:58:25 | [diff] [blame^] | 509 | ASSERT_EQ("3: []", GetTreeState(frame_tree)); |
dgrogan | fb22f9a | 2014-10-20 21:32:32 | [diff] [blame] | 510 | } |
| 511 | |
nasko | aeca57b | 2015-02-13 00:50:46 | [diff] [blame] | 512 | // Ensure that frames removed while a process has crashed are not preserved in |
| 513 | // the global map of id->frame. |
| 514 | TEST_F(FrameTreeTest, ProcessCrashClearsGlobalMap) { |
lfg | 269b702f | 2015-06-08 19:28:19 | [diff] [blame] | 515 | main_test_rfh()->InitializeRenderFrameIfNeeded(); |
| 516 | |
nasko | aeca57b | 2015-02-13 00:50:46 | [diff] [blame] | 517 | // Add a couple child frames to the main frame. |
| 518 | FrameTreeNode* root = contents()->GetFrameTree()->root(); |
| 519 | |
lukasza | 464d869 | 2016-02-22 19:26:32 | [diff] [blame] | 520 | main_test_rfh()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 521 | 22, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 522 | blink::WebTreeScopeType::kDocument, std::string(), "uniqueName0", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 523 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 524 | FrameOwnerProperties()); |
lukasza | 464d869 | 2016-02-22 19:26:32 | [diff] [blame] | 525 | main_test_rfh()->OnCreateChildFrame( |
Balazs Engedy | ba034e7 | 2017-10-27 22:26:28 | [diff] [blame] | 526 | 23, CreateStubInterfaceProviderRequest(), |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 527 | blink::WebTreeScopeType::kDocument, std::string(), "uniqueName1", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 528 | base::UnguessableToken::Create(), blink::FramePolicy(), |
| 529 | FrameOwnerProperties()); |
nasko | aeca57b | 2015-02-13 00:50:46 | [diff] [blame] | 530 | |
dmazzoni | e950ea23 | 2015-03-13 21:39:45 | [diff] [blame] | 531 | // Add one grandchild frame. |
| 532 | RenderFrameHostImpl* child1_rfh = root->child_at(0)->current_frame_host(); |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 533 | child1_rfh->OnCreateChildFrame(33, CreateStubInterfaceProviderRequest(), |
| 534 | blink::WebTreeScopeType::kDocument, |
Lukasz Anforowicz | 7bfb2e9 | 2017-11-22 17:19:45 | [diff] [blame] | 535 | std::string(), "uniqueName2", false, |
Luna Lu | c3fdacdf | 2017-11-08 04:48:53 | [diff] [blame] | 536 | base::UnguessableToken::Create(), |
| 537 | blink::FramePolicy(), FrameOwnerProperties()); |
dmazzoni | e950ea23 | 2015-03-13 21:39:45 | [diff] [blame] | 538 | |
nasko | aeca57b | 2015-02-13 00:50:46 | [diff] [blame] | 539 | // Ensure they can be found by id. |
vishal.b | 782eb5d | 2015-04-29 12:22:57 | [diff] [blame] | 540 | int id1 = root->child_at(0)->frame_tree_node_id(); |
| 541 | int id2 = root->child_at(1)->frame_tree_node_id(); |
| 542 | int id3 = root->child_at(0)->child_at(0)->frame_tree_node_id(); |
dmazzoni | e950ea23 | 2015-03-13 21:39:45 | [diff] [blame] | 543 | EXPECT_TRUE(FrameTreeNode::GloballyFindByID(id1)); |
| 544 | EXPECT_TRUE(FrameTreeNode::GloballyFindByID(id2)); |
| 545 | EXPECT_TRUE(FrameTreeNode::GloballyFindByID(id3)); |
nasko | aeca57b | 2015-02-13 00:50:46 | [diff] [blame] | 546 | |
| 547 | // Crash the renderer. |
nick | 16b0765 | 2015-04-18 02:35:31 | [diff] [blame] | 548 | main_test_rfh()->GetProcess()->SimulateCrash(); |
nasko | aeca57b | 2015-02-13 00:50:46 | [diff] [blame] | 549 | |
| 550 | // Ensure they cannot be found by id after the process has crashed. |
dmazzoni | e950ea23 | 2015-03-13 21:39:45 | [diff] [blame] | 551 | EXPECT_FALSE(FrameTreeNode::GloballyFindByID(id1)); |
| 552 | EXPECT_FALSE(FrameTreeNode::GloballyFindByID(id2)); |
| 553 | EXPECT_FALSE(FrameTreeNode::GloballyFindByID(id3)); |
nasko | aeca57b | 2015-02-13 00:50:46 | [diff] [blame] | 554 | } |
| 555 | |
[email protected] | 9b159a5 | 2013-10-03 17:24:55 | [diff] [blame] | 556 | } // namespace content |