Implement SetSize for valid size requests....

chrome/browser/renderer_host/render_widget_host_view_mac.mm

 // 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.
 
 #include "chrome/browser/renderer_host/render_widget_host_view_mac.h"
 
 #include "base/histogram.h"
 #include "base/sys_string_conversions.h"
 #include "chrome/browser/browser_trial.h"
 #import "chrome/browser/cocoa/rwhvm_editcommand_helper.h"
@@ skipping 99 matching lines @@
   // If we have a renderer, then inform it that we are being hidden so it can
   // reduce its resource utilization.
   render_widget_host_->WasHidden();
 }
 
 void RenderWidgetHostViewMac::SetSize(const gfx::Size& size) {
   if (is_hidden_)
     return;
 
   // TODO(avi): the TabContents object uses this method to size the newly
-  // created widget to the correct size. At the time of this call, we're not yet
-  // in the view hierarchy so |size| ends up being 0x0. However, this works for
-  // us because we're using the Cocoa view struture and resizer flags to fix
-  // things up as soon as the view gets added to the hierarchy. Figure out if we
-  // want to keep this flow or switch back to the flow Windows uses which sets
-  // the size upon creation. http://crbug.com/8285.
+  // created widget to the correct size. For that instance, we're not yet in the
+  // view hierarchy so |size| ends up being 0x0. However, this works for us
+  // because we're using the Cocoa view struture and resizer flags to fix things
+  // up as soon as the view gets added to the hierarchy. Figure out if we want
+  // to keep this flow or switch back to the flow Windows uses which sets the
+  // size upon creation. <http://crbug.com/8285>. However, if the size is not
+  // 0x0, this is a valid request.
+  if (size.IsEmpty())
+    return;
+
+  // Do conversions to upper-left origin, as "set size" means keep the
+  // upper-left corner pinned. If the new size is valid, this is a popup whose
+  // superview is another RenderWidgetHostViewCocoa, but even if it's directly
+  // in a TabContentsViewCocoa, they're both BaseViews.
+  gfx::Rect rect =
+      [(BaseView*)[cocoa_view_ superview] NSRectToRect:[cocoa_view_ frame]];
+  rect.set_width(size.width());
+  rect.set_height(size.height());
+  [cocoa_view_ setFrame:[(BaseView*)[cocoa_view_ superview] RectToNSRect:rect]];
 }
 
 gfx::NativeView RenderWidgetHostViewMac::GetNativeView() {
   return native_view();
 }
 
 void RenderWidgetHostViewMac::MovePluginWindows(
     const std::vector<WebPluginGeometry>& plugin_window_moves) {
   // All plugin stuff is TBD. TODO(avi,awalker): fill in
   // http://crbug.com/8192
@@ skipping 621 matching lines @@
 // NSView calls this to get the text when displaying the tooltip.
 - (NSString *)view:(NSView *)view
   stringForToolTip:(NSToolTipTag)tag
              point:(NSPoint)point
           userData:(void *)data {
   return [[toolTip_ copy] autorelease];
 }
 
 @end