Removes RenderViewHostDelegate::GetBrowserWindowID. Instead the places

content/browser/tab_contents/interstitial_page.cc

 // Copyright (c) 2011 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.
 
 #include "content/browser/tab_contents/interstitial_page.h"
 
 #include <vector>
 
 #include "base/compiler_specific.h"
 #include "base/message_loop.h"
@@ skipping 701 matching lines @@
 }
 
 void InterstitialPage::InterstitialPageRVHViewDelegate::HandleMouseActivate() {
 }
 
 void InterstitialPage::InterstitialPageRVHViewDelegate::OnFindReply(
     int request_id, int number_of_matches, const gfx::Rect& selection_rect,
     int active_match_ordinal, bool final_update) {
 }
 
-int InterstitialPage::GetBrowserWindowID() const {

[Matt Perry, 2011/06/08 18:18:42]

does the InterstitialPage need to send the UpdateBrowserId message?

[jam, 2011/06/08 18:30:08]

based on inspection, it didn't seem that this matters?  i.e. this is ultimately
used in the extension code so that script can enumerate all the tabs/background
pages etc, right?  If so, then it seems that interstitial pages should just be
an internal chrome implementation and not visible?

-  return tab_->GetBrowserWindowID();
-}
-
 void InterstitialPage::UpdateInspectorSetting(const std::string& key,
                                               const std::string& value) {
   RenderViewHostDelegateHelper::UpdateInspectorSetting(
       tab_->profile(), key, value);
 }
 
 void InterstitialPage::ClearInspectorSettings() {
   RenderViewHostDelegateHelper::ClearInspectorSettings(tab_->profile());
 }