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

Side by Side Diff: content/browser/frame_host/frame_tree_node.cc

Issue 1390053002: Trace FrameTreeNode Snapshots (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: TraceSnapshot only in FrameTree Created 5 years, 1 month 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/frame_host/frame_tree_node.h" 5 #include "content/browser/frame_host/frame_tree_node.h"
6 6
7 #include <queue> 7 #include <queue>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/profiler/scoped_tracker.h" 10 #include "base/profiler/scoped_tracker.h"
(...skipping 174 matching lines...) Expand 10 before | Expand all | Expand 10 after
185 if (!opener_observer_) 185 if (!opener_observer_)
186 opener_observer_ = make_scoped_ptr(new OpenerDestroyedObserver(this)); 186 opener_observer_ = make_scoped_ptr(new OpenerDestroyedObserver(this));
187 opener_->AddObserver(opener_observer_.get()); 187 opener_->AddObserver(opener_observer_.get());
188 } 188 }
189 } 189 }
190 190
191 void FrameTreeNode::SetCurrentURL(const GURL& url) { 191 void FrameTreeNode::SetCurrentURL(const GURL& url) {
192 if (!has_committed_real_load_ && url != GURL(url::kAboutBlankURL)) 192 if (!has_committed_real_load_ && url != GURL(url::kAboutBlankURL))
193 has_committed_real_load_ = true; 193 has_committed_real_load_ = true;
194 current_url_ = url; 194 current_url_ = url;
195 frame_tree_->TraceSnapshot();
nasko 2015/10/28 20:17:55 Keep in mind that since the tracing here, it will
benjhayden 2015/10/28 20:50:50 Good to know! If it turns out that these snapshots
195 } 196 }
196 197
197 void FrameTreeNode::SetCurrentOrigin(const url::Origin& origin) { 198 void FrameTreeNode::SetCurrentOrigin(const url::Origin& origin) {
198 if (!origin.IsSameOriginWith(replication_state_.origin)) 199 if (!origin.IsSameOriginWith(replication_state_.origin))
199 render_manager_.OnDidUpdateOrigin(origin); 200 render_manager_.OnDidUpdateOrigin(origin);
200 replication_state_.origin = origin; 201 replication_state_.origin = origin;
201 } 202 }
202 203
203 void FrameTreeNode::SetFrameName(const std::string& name) { 204 void FrameTreeNode::SetFrameName(const std::string& name) {
204 if (name != replication_state_.name) 205 if (name != replication_state_.name)
(...skipping 172 matching lines...) Expand 10 before | Expand all | Expand 10 after
377 // TODO(nasko): see if child frames should send IPCs in site-per-process 378 // TODO(nasko): see if child frames should send IPCs in site-per-process
378 // mode. 379 // mode.
379 if (!IsMainFrame()) 380 if (!IsMainFrame())
380 return true; 381 return true;
381 382
382 render_manager_.Stop(); 383 render_manager_.Stop();
383 return true; 384 return true;
384 } 385 }
385 386
386 } // namespace content 387 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698