[DO NOT COMMIT] Remove chrome::NOTIFICATION_RETARGETING

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index a97f03a27bbd36c0d69fea2464bbd99f656c3d84..445a6dc543ca023a32d6759bcb9352d7587cc659 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -1118,6 +1118,9 @@ RenderFrameImpl::RenderFrameImpl(const CreateParams& params)
       frame_binding_(this),
       has_accessed_initial_document_(false),
       weak_factory_(this) {
+  LOG(ERROR) << "RF[" << this << "]::RF: "
+             << " routing_id:" << routing_id_;
+
   // We don't have a service_manager::Connection at this point, so use nullptr.
   // TODO(beng): We should fix this, so we can apply policy about which
   //             interfaces get exposed.
@@ -1188,6 +1191,7 @@ RenderFrameImpl::~RenderFrameImpl() {
   render_view_->UnregisterRenderFrame(this);
   g_routing_id_frame_map.Get().erase(routing_id_);
   RenderThread::Get()->RemoveRoute(routing_id_);
+  LOG(ERROR) << "RF[" << this << "]::~RF";
 }
 
 void RenderFrameImpl::BindToWebFrame(blink::WebLocalFrame* web_frame) {
@@ -1784,6 +1788,8 @@ void RenderFrameImpl::OnSwapOut(
 }
 
 void RenderFrameImpl::OnDeleteFrame() {
+  LOG(ERROR) << "RF[" << this << "]::OnDeleteFrame:";
+
   // TODO(nasko): If this message is received right after a commit has
   // swapped a RenderFrameProxy with this RenderFrame, the proxy needs to be
   // recreated in addition to the RenderFrame being deleted.
@@ -2969,6 +2975,9 @@ void RenderFrameImpl::didChangeOpener(blink::WebFrame* opener) {
 
 void RenderFrameImpl::frameDetached(blink::WebLocalFrame* frame,
                                     DetachType type) {
+  LOG(ERROR) << "RF[" << this << "]::frameDetached: "
+             << " type:" << static_cast<int>(type);
+
   // NOTE: This function is called on the frame that is being detached and not
   // the parent frame.  This is different from createChildFrame() which is
   // called on the parent frame.
@@ -3335,6 +3344,9 @@ void RenderFrameImpl::didStartProvisionalLoad(blink::WebLocalFrame* frame,
                "url", ds->request().url().string().utf8());
   DocumentState* document_state = DocumentState::FromDataSource(ds);
 
+  LOG(ERROR) << "RF[" << this << "]::didStartProvisionalLoad: "
+             << " url:" << ds->request().url().string().utf8().substr(0, 250);
+
   // Update the request time if WebKit has better knowledge of it.
   if (document_state->request_time().is_null() &&
           triggering_event_time != 0.0) {
@@ -3375,6 +3387,9 @@ void RenderFrameImpl::didReceiveServerRedirectForProvisionalLoad(
     blink::WebLocalFrame* frame) {
   DCHECK_EQ(frame_, frame);
 
+  LOG(ERROR) << "RF[" << this
+             << "]::didReceiveServerRedirectForProvisionalLoad:";
+
   // We don't use HistoryController in OOPIF enabled modes.
   if (SiteIsolationPolicy::UseSubframeNavigationEntries())
     return;
@@ -3392,6 +3407,9 @@ void RenderFrameImpl::didFailProvisionalLoad(
   WebDataSource* ds = frame->provisionalDataSource();
   DCHECK(ds);
 
+  LOG(ERROR) << "RF[" << this << "]::didFailProvisionalLoad: "
+             << " url:" << ds->request().url().string().utf8().substr(0, 250);
+
   const WebURLRequest& failed_request = ds->request();
 
   // Notify the browser that we failed a provisional load with an error.
@@ -3500,6 +3518,9 @@ void RenderFrameImpl::didCommitProvisionalLoad(
     }
   }
 
+  LOG(ERROR) << "RF[" << this << "]::didCommitProvisionalLoad: "
+             << " url:" << GetLoadingUrl().spec().substr(0, 250);
+
   if (proxy_routing_id_ != MSG_ROUTING_NONE) {
     RenderFrameProxy* proxy =
         RenderFrameProxy::FromRoutingID(proxy_routing_id_);
@@ -3750,6 +3771,10 @@ void RenderFrameImpl::didFinishDocumentLoad(blink::WebLocalFrame* frame) {
   DocumentState* document_state = DocumentState::FromDataSource(ds);
   document_state->set_finish_document_load_time(Time::Now());
 
+  LOG(ERROR) << "RF[" << this << "]::didFinishDocumentLoad: "
+             << " url:"
+             << frame->document().url().string().utf8().substr(0, 250);
+
   Send(new FrameHostMsg_DidFinishDocumentLoad(routing_id_));
 
   for (auto& observer : render_view_->observers())