Removing Windows specific code from interstitial pages.

BUG=None
TEST=Navigate to bad https URL, make sure interstitial page still shows up and works as expected.
Review URL: http://codereview.chromium.org/40311

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@11277 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 33 deletions

diff --git a/chrome/browser/tab_contents/interstitial_page.cc b/chrome/browser/tab_contents/interstitial_page.cc
index adfe270..f49a4f0 100644
--- a/chrome/browser/tab_contents/interstitial_page.cc
+++ b/chrome/browser/tab_contents/interstitial_page.cc
@@ -9,23 +9,16 @@
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_list.h"
 #include "chrome/browser/dom_operation_notification_details.h"
+#include "chrome/browser/renderer_host/render_widget_host_view.h"
 #include "chrome/browser/tab_contents/navigation_controller.h"
 #include "chrome/browser/tab_contents/navigation_entry.h"
 #include "chrome/browser/tab_contents/web_contents.h"
+#include "chrome/browser/tab_contents/web_contents_view.h"
 #include "chrome/common/notification_service.h"
 #include "chrome/views/window_delegate.h"
 #include "grit/browser_resources.h"
 #include "net/base/escape.h"
 
-#if defined(OS_WIN)
-#include "chrome/browser/renderer_host/render_widget_host_view_win.h"
-#include "chrome/browser/tab_contents/web_contents_view_win.h"
-#include "chrome/views/window.h"
-#else
-#include "chrome/browser/renderer_host/render_widget_host_view.h"
-#include "chrome/browser/tab_contents/web_contents_view.h"
-#endif
-
 namespace {
 
 class ResourceRequestTask : public Task {
@@ -248,29 +241,6 @@ void InterstitialPage::Observe(NotificationType type,
 }
 
 RenderViewHost* InterstitialPage::CreateRenderViewHost() {
-#if defined(OS_WIN)
-  RenderViewHost* render_view_host = new RenderViewHost(
-      SiteInstance::CreateSiteInstance(tab()->profile()),
-      this, MSG_ROUTING_NONE, NULL);
-  RenderWidgetHostViewWin* view =
-      new RenderWidgetHostViewWin(render_view_host);
-  render_view_host->set_view(view);
-  view->Create(tab_->GetContentNativeView());
-  view->set_parent_hwnd(tab_->GetContentNativeView());
-  WebContentsViewWin* web_contents_view =
-      static_cast<WebContentsViewWin*>(tab_->view());
-  render_view_host->AllowDomAutomationBindings();
-  render_view_host->CreateRenderView();
-  // SetSize must be called after CreateRenderView or the HWND won't show.
-  view->SetSize(web_contents_view->GetContainerSize());
-
-  return render_view_host;
-#else
-  // It is untested, whether this code is sufficiently generic that it
-  // works with Windows, and thus obsoletes the special-cased code above.
-  // If it does work, don't forget to also clean up the include statements!
-  NOTIMPLEMENTED();
-
   RenderViewHost* render_view_host = new RenderViewHost(
       SiteInstance::CreateSiteInstance(tab()->profile()),
       this, MSG_ROUTING_NONE, NULL);
@@ -281,8 +251,9 @@ RenderViewHost* InterstitialPage::CreateRenderViewHost() {
   render_view_host->AllowDomAutomationBindings();
   render_view_host->CreateRenderView();
   view->SetSize(web_contents_view->GetContainerSize());
+  // Don't show the interstitial until we have navigated to it.
+  view->Hide();
   return render_view_host;
-#endif
 }
 
 void InterstitialPage::Proceed() {