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

chrome/browser/renderer_host/render_widget_host_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_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_H_
 #define CHROME_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_H_
 
 #include "base/gfx/native_widget_types.h"
 #include "base/shared_memory.h"
 #include "chrome/common/render_messages.h"
 
 namespace gfx {
 class Rect;
 class Size;
 }
 namespace IPC {
 class Message;
 }
 
 class RenderProcessHost;
 class RenderWidgetHost;
 class WebCursor;
 
-///////////////////////////////////////////////////////////////////////////////
-//
-// RenderWidgetHostView
-//
-//  RenderWidgetHostView is an interface implemented by an object that acts as
-//  the "View" portion of a RenderWidgetHost. The RenderWidgetHost and its
-//  associated RenderProcessHost own the "Model" in this case which is the
-//  child renderer process. The View is responsible for receiving events from
-//  the surrounding environment and passing them to the RenderWidgetHost, and
-//  for actually displaying the content of the RenderWidgetHost when it
-//  changes.
-//
-///////////////////////////////////////////////////////////////////////////////
+// RenderWidgetHostView is an interface implemented by an object that acts as
+// the "View" portion of a RenderWidgetHost. The RenderWidgetHost and its
+// associated RenderProcessHost own the "Model" in this case which is the
+// child renderer process. The View is responsible for receiving events from
+// the surrounding environment and passing them to the RenderWidgetHost, and
+// for actually displaying the content of the RenderWidgetHost when it
+// changes.
 class RenderWidgetHostView {
  public:
   // Platform-specific creator. Use this to construct new RenderWidgetHostViews
   // rather than using RenderWidgetHostViewWin & friends.
   //
+  // This function must NOT size it, because the RenderView in the renderer
+  // wounldn't have been created yet. The widget would set its "waiting for
+  // resize ack" flag, and the ack would never come becasue no RenderView
+  // received it.
+  //
   // The RenderWidgetHost must already be created (because we can't know if it's
   // going to be a regular RenderWidgetHost or a RenderViewHost (a subclass).
   static RenderWidgetHostView* CreateViewForWidget(RenderWidgetHost* widget);
 
   // Returns the associated RenderWidgetHost.
   virtual RenderWidgetHost* GetRenderWidgetHost() const = 0;
 
   // Notifies the View that it has become visible.
   virtual void DidBecomeSelected() = 0;
 
@@ skipping 62 matching lines @@
  protected:
   // Interface class only, do not construct.
   RenderWidgetHostView() {}
 
  private:
   DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostView);
 };
 
 #endif  // CHROME_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_H_