Clean up the Windows-ness in the sizing of new views. Write a bunch of commen...

chrome/browser/tab_contents/web_contents_view.h

 // Copyright (c) 2006-2008 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 CHROME_BROWSER_TAB_CONTENTS_WEB_CONTENTS_VIEW_H_
 #define CHROME_BROWSER_TAB_CONTENTS_WEB_CONTENTS_VIEW_H_
 
 #include <map>
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/gfx/native_widget_types.h"
 #include "base/gfx/rect.h"
 #include "base/gfx/size.h"
 #include "chrome/browser/renderer_host/render_view_host_delegate.h"
 
 class Browser;
 class RenderViewHost;
 class RenderWidgetHost;
 class RenderWidgetHostView;
-class RenderWidgetHostViewWin;  // TODO(brettw) this should not be necessary.
 class WebContents;
 struct WebDropData;
 class WebKeyboardEvent;
 
 namespace base {
 class WaitableEvent;
 }
 
 // The WebContentsView is an interface that is implemented by the platform-
 // dependent web contents views. The WebContents uses this interface to talk to
 // them. View-related messages will also get forwarded directly to this class
 // from RenderViewHost via RenderViewHostDelegate::View.
 //
 // It contains a small amount of logic with respect to creating new sub-view
 // that should be the same for all platforms.
 class WebContentsView : public RenderViewHostDelegate::View {
  public:
   virtual ~WebContentsView() {}
 
   virtual WebContents* GetWebContents() = 0;
 
   virtual void CreateView() = 0;
 
   // Sets up the View that holds the rendered web page, receives messages for
-  // it and contains page plugins.
-  // TODO(brettw) make this so we don't need to return the Win version (see the
-  // caller in WebContents).
-  virtual RenderWidgetHostViewWin* CreateViewForWidget(
+  // it and contains page plugins. The host view should be sized to the current
+  // size of the WebContents.
+  virtual RenderWidgetHostView* CreateViewForWidget(
       RenderWidgetHost* render_widget_host) = 0;
 
   // Returns the native widget that contains the contents of the tab.
   virtual gfx::NativeView GetNativeView() const = 0;
 
   // Returns the native widget with the main content of the tab (i.e. the main
   // render view host, though there may be many popups in the tab as children of
   // the container).
   virtual gfx::NativeView GetContentNativeView() const = 0;
 
@@ skipping 129 matching lines @@
 
   // These maps hold on to the widgets that we created on behalf of the
   // renderer that haven't shown yet.
   typedef std::map<int, RenderWidgetHostView*> PendingWidgetViews;
   PendingWidgetViews pending_widget_views_;
 
   DISALLOW_COPY_AND_ASSIGN(WebContentsView);
 };
 
 #endif  // CHROME_BROWSER_TAB_CONTENTS_WEB_CONTENTS_VIEW_H_