Removes RenderViewHostDelegate::GetBrowserWindowID. Instead the places

chrome/browser/ui/browser.cc

Index: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index a86afce9005afc41e24958a2d856d5ec2cdbbe3f..74d7ea21af87607912a60ec88454cd4f7aa767c4 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -2859,6 +2859,7 @@ void Browser::TabInsertedAt(TabContentsWrapper* contents,
 
   // Extension code in the renderer holds the ID of the window that hosts it.
   // Notify it that the window ID changed.
+  // TODO(sky): move this to a better place.
   contents->render_view_host()->Send(new ExtensionMsg_UpdateBrowserWindowId(
       contents->render_view_host()->routing_id(),
       contents->controller().window_id().id()));
@@ -3436,6 +3437,14 @@ content::JavaScriptDialogCreator* Browser::GetJavaScriptDialogCreator() {
   return GetJavaScriptDialogCreatorInstance();
 }
 
+void Browser::RenderViewCreated(TabContents* source, RenderViewHost* host) {
+  // TODO(sky): move this to a TabContentsObserver hung off TabContentsWrapper,

[jam, 2011/06/08 17:09:30]

you can send this now in ExtensionTabHelper which seems like a decent place
since it's an extension message

[sky, 2011/06/08 17:19:43]

That will work now because the window id is in the NavigationController, but it
won't be like that in my next patch.

I plan on creating a RestoreTabHelper that contains the session and window ids.
I'll move this code there (as well as 2863).

+  // then nuke this method.
+  host->Send(new ExtensionMsg_UpdateBrowserWindowId(
+                 host->routing_id(),
+                 source->controller().window_id().id()));
+}
+
 ///////////////////////////////////////////////////////////////////////////////
 // Browser, TabContentsWrapperDelegate implementation: