Fix RenderFrameHostChanged WebContentsObserver methods

content/browser/frame_host/render_frame_host_manager.h

 // Copyright 2013 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_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
 
 #include <list>
 
 #include "base/basictypes.h"
@@ skipping 119 matching lines @@
     virtual void BeforeUnloadFiredFromRenderManager(
         bool proceed, const base::TimeTicks& proceed_time,
         bool* proceed_to_fire_unload) = 0;
     virtual void RenderProcessGoneFromRenderManager(
         RenderViewHost* render_view_host) = 0;
     virtual void UpdateRenderViewSizeForRenderManager() = 0;
     virtual void CancelModalDialogsForRenderManager() = 0;
     virtual void NotifySwappedFromRenderManager(RenderFrameHost* old_host,
                                                 RenderFrameHost* new_host,
                                                 bool is_main_frame) = 0;
+    // TODO(nasko): This should be removed once extensions no longer use
+    // NotificationService. See https://crbug.com/462682

[Charlie Reis, 2015/03/02 18:38:26]

nit: End with period.

[nasko, 2015/03/02 19:30:54]

Done.

+    virtual void NotifyMainFrameSwappedFromRenderManager(
+        RenderFrameHost* old_host,
+        RenderFrameHost* new_host) = 0;

[Charlie Reis, 2015/03/02 18:38:26]

I feel like it's worth using RVH here instead of RFH.  That will make it more
obvious what it's used for and why it must go away.

[nasko, 2015/03/02 19:30:54]

Done.

     virtual NavigationControllerImpl&
         GetControllerForRenderManager() = 0;
 
     // Create swapped out RenderViews in the given SiteInstance for each tab in
     // the opener chain of this tab, if any.  This allows the current tab to
     // make cross-process script calls to its opener(s).  Returns the route ID
     // of the immediate opener, if one exists (otherwise MSG_ROUTING_NONE).
     virtual int CreateOpenerRenderViewsForRenderManager(
         SiteInstance* instance) = 0;
 
@@ skipping 538 matching lines @@
   bool should_reuse_web_ui_;
 
   base::WeakPtrFactory<RenderFrameHostManager> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_