Remove the retargeting notification

content/public/browser/web_contents_observer.h

 // Copyright (c) 2012 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_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
 
 #include <stdint.h>
 
 #include "base/macros.h"
@@ skipping 257 matching lines @@
   // This method is invoked when a new WebContents was created in response to
   // an action in the observed WebContents, e.g. a link with target=_blank was
   // clicked. The |source_render_frame_host| is the frame in which the action
   // took place.
   virtual void DidOpenRequestedURL(WebContents* new_contents,
                                    RenderFrameHost* source_render_frame_host,
                                    const GURL& url,
                                    const Referrer& referrer,
                                    WindowOpenDisposition disposition,
                                    ui::PageTransition transition,
-                                   bool started_from_context_menu) {}
+                                   bool started_from_context_menu,
+                                   bool not_yet_in_tabstrip) {}

[nasko, 2017/03/01 00:48:38]

Technically, TabStrip is chrome/ concept and content/ on its own doesn't have
such a thing. Can we rename this to reflect better how it is set? It is true for
renderer-initiated WebContents creations and it is set to false for
browser-initiated ones. So renderer_initiated?

[Patrick Noland, 2017/03/01 23:58:59]

Done.

 
   // This method is invoked when the renderer process has completed its first
   // paint after a non-empty layout.
   virtual void DidFirstVisuallyNonEmptyPaint() {}
 
   // This method is invoked when the main frame in the renderer process performs
   // the first paint after a navigation.
   virtual void DidFirstPaintAfterLoad(RenderWidgetHost* render_widget_host) {}
 
   // When WebContents::Stop() is called, the WebContents stops loading and then
@@ skipping 167 matching lines @@
   void ResetWebContents();
 
   WebContentsImpl* web_contents_;
 
   DISALLOW_COPY_AND_ASSIGN(WebContentsObserver);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_