Remove WebWidgetDelegate::GetContainingView since it is not needed.

TEST=none
BUG=16234
R=jam

Review URL: http://codereview.chromium.org/155243

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

7 files changed, 3 insertions, 23 deletions

diff --git a/webkit/glue/chrome_client_impl.cc b/webkit/glue/chrome_client_impl.cc
index d843b0f..53aca2e 100644
--- a/webkit/glue/chrome_client_impl.cc
+++ b/webkit/glue/chrome_client_impl.cc
@@ -479,8 +479,7 @@ WebCore::IntRect ChromeClientImpl::windowToScreen(
 }
 
 PlatformWidget ChromeClientImpl::platformWindow() const {
-  WebViewDelegate* delegate = webview_->delegate();
-  return delegate ? delegate->GetContainingView(webview_) : NULL;
+  return NULL;
 }
 
 void ChromeClientImpl::contentsSizeChanged(WebCore::Frame* frame, const
diff --git a/webkit/glue/webwidget_delegate.h b/webkit/glue/webwidget_delegate.h
index faca629..824b665 100644
--- a/webkit/glue/webwidget_delegate.h
+++ b/webkit/glue/webwidget_delegate.h
@@ -7,7 +7,6 @@
 
 #include <vector>
 
-#include "base/gfx/native_widget_types.h"
 #include "base/string16.h"
 #include "webkit/glue/window_open_disposition.h"
 
@@ -38,9 +37,6 @@ struct WebMenuItem {
 
 class WebWidgetDelegate {
  public:
-  // Returns the view in which the WebWidget is embedded.
-  virtual gfx::NativeViewId GetContainingView(WebWidget* webwidget) = 0;
-
   // Called when a region of the WebWidget needs to be re-painted.
   virtual void DidInvalidateRect(WebWidget* webwidget,
                                  const WebKit::WebRect& rect) = 0;
diff --git a/webkit/glue/webwidget_impl.cc b/webkit/glue/webwidget_impl.cc
index c738cdf..cb3a0c9 100644
--- a/webkit/glue/webwidget_impl.cc
+++ b/webkit/glue/webwidget_impl.cc
@@ -271,9 +271,7 @@ WebCore::IntRect WebWidgetImpl::windowToScreen(
 }
 
 PlatformWidget WebWidgetImpl::platformWindow() const {
-  if (!delegate_)
-    return NULL;
-  return delegate_->GetContainingView(const_cast<WebWidgetImpl*>(this));
+  return NULL;
 }
 
 void WebWidgetImpl::scrollRectIntoView(
diff --git a/webkit/glue/webworker_impl.cc b/webkit/glue/webworker_impl.cc
index bfe27ac..d1ba068 100644
--- a/webkit/glue/webworker_impl.cc
+++ b/webkit/glue/webworker_impl.cc
@@ -56,9 +56,6 @@ class WorkerWebViewDelegate : public WebViewDelegate {
   virtual void DidScrollRect(WebWidget *webwidget, int dx, int dy,
                              const WebKit::WebRect &clip_rect) { }
   virtual void Focus(WebWidget *webwidget) { }
-  virtual gfx::NativeViewId GetContainingView(WebWidget *webwidget) {
-    return gfx::NativeViewId();
-  }
   virtual void GetRootWindowRect(WebWidget *webwidget,
                                  WebKit::WebRect *rect) { }
   virtual void GetRootWindowResizerRect(WebWidget *webwidget,
diff --git a/webkit/tools/test_shell/test_webview_delegate.cc b/webkit/tools/test_shell/test_webview_delegate.cc
index b97c923..d78971b 100644
--- a/webkit/tools/test_shell/test_webview_delegate.cc
+++ b/webkit/tools/test_shell/test_webview_delegate.cc
@@ -744,15 +744,6 @@ void TestWebViewDelegate::SetUserStyleSheetLocation(const GURL& location) {
 
 // WebWidgetDelegate ---------------------------------------------------------
 
-gfx::NativeViewId TestWebViewDelegate::GetContainingView(WebWidget* webwidget) {
-  // For test shell, we pack a NativeView pointer into the NativeViewId since
-  // everything is single process.
-  if (WebWidgetHost* host = GetHostForWidget(webwidget))
-    return gfx::IdFromNativeView(host->view_handle());
-
-  return NULL;
-}
-
 void TestWebViewDelegate::DidInvalidateRect(WebWidget* webwidget,
                                             const WebRect& rect) {
   if (WebWidgetHost* host = GetHostForWidget(webwidget))
diff --git a/webkit/tools/test_shell/test_webview_delegate.h b/webkit/tools/test_shell/test_webview_delegate.h
index 4c0f079..f463c22 100644
--- a/webkit/tools/test_shell/test_webview_delegate.h
+++ b/webkit/tools/test_shell/test_webview_delegate.h
@@ -222,7 +222,6 @@ class TestWebViewDelegate : public base::RefCounted<TestWebViewDelegate>,
   virtual int GetHistoryForwardListCount();
 
   // WebWidgetDelegate
-  virtual gfx::NativeViewId GetContainingView(WebWidget* webwidget);
   virtual void DidInvalidateRect(WebWidget* webwidget,
                                  const WebKit::WebRect& rect);
   virtual void DidScrollRect(WebWidget* webwidget, int dx, int dy,
diff --git a/webkit/tools/test_shell/test_webview_delegate_win.cc b/webkit/tools/test_shell/test_webview_delegate_win.cc
index c0dbc825..dc16544 100644
--- a/webkit/tools/test_shell/test_webview_delegate_win.cc
+++ b/webkit/tools/test_shell/test_webview_delegate_win.cc
@@ -50,7 +50,7 @@ WebPluginDelegate* TestWebViewDelegate::CreatePluginDelegate(
     const std::string& mime_type,
     const std::string& clsid,
     std::string* actual_mime_type) {
-  HWND hwnd = gfx::NativeViewFromId(GetContainingView(webview));
+  HWND hwnd = shell_->webViewHost()->view_handle();
   if (!hwnd)
     return NULL;