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

Unified Diff: content/renderer/render_frame_proxy.cc

Issue 2383573002: [DO NOT COMMIT] Remove chrome::NOTIFICATION_RETARGETING
Patch Set: Rebase on ToT Created 4 years, 2 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
« no previous file with comments | « content/renderer/render_frame_impl.cc ('k') | content/test/web_contents_observer_sanity_checker.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_frame_proxy.cc
diff --git a/content/renderer/render_frame_proxy.cc b/content/renderer/render_frame_proxy.cc
index 1a833e2fa27d79c0e61392ab96ff95da1947789c..6aa1fe01bc0bff8bfac0307ac16ae3e7c442fc92 100644
--- a/content/renderer/render_frame_proxy.cc
+++ b/content/renderer/render_frame_proxy.cc
@@ -171,6 +171,8 @@ RenderFrameProxy::RenderFrameProxy(int routing_id, int frame_routing_id)
web_frame_(nullptr),
render_view_(nullptr),
render_widget_(nullptr) {
+ LOG(ERROR) << "RFP[" << this << "]::RFP: "
+ << " routing_id:" << routing_id_;
std::pair<RoutingIDProxyMap::iterator, bool> result =
g_routing_id_proxy_map.Get().insert(std::make_pair(routing_id_, this));
CHECK(result.second) << "Inserting a duplicate item.";
@@ -183,6 +185,8 @@ RenderFrameProxy::~RenderFrameProxy() {
CHECK(!web_frame_);
RenderThread::Get()->RemoveRoute(routing_id_);
g_routing_id_proxy_map.Get().erase(routing_id_);
+ LOG(ERROR) << "RFP[" << this << "]::~RFP: "
+ << " routing_id:" << routing_id_;
}
void RenderFrameProxy::Init(blink::WebRemoteFrame* web_frame,
@@ -297,6 +301,7 @@ bool RenderFrameProxy::Send(IPC::Message* message) {
}
void RenderFrameProxy::OnDeleteProxy() {
+ LOG(ERROR) << "RFP[" << this << "]::OnDeleteProxy:";
DCHECK(web_frame_);
web_frame_->detach();
}
@@ -394,6 +399,8 @@ void RenderFrameProxy::OnWillEnterFullscreen() {
}
void RenderFrameProxy::frameDetached(DetachType type) {
+ LOG(ERROR) << "RFP[" << this << "]::frameDetached: "
+ << " type:" << static_cast<int>(type);
if (type == DetachType::Remove && web_frame_->parent()) {
web_frame_->parent()->removeChild(web_frame_);
« no previous file with comments | « content/renderer/render_frame_impl.cc ('k') | content/test/web_contents_observer_sanity_checker.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698