Handle resize corner layout entirely in the platform BrowserWindow*/BrowserView* code...

chrome/renderer/render_widget.cc

 // Copyright (c) 2010 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/renderer/render_widget.h"
 
 #include "app/surface/transport_dib.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
@@ skipping 85 matching lines @@
   DCHECK(opener_id != MSG_ROUTING_NONE);
   scoped_refptr<RenderWidget> widget(new RenderWidget(render_thread,
                                                         popup_type));
   widget->Init(opener_id);  // adds reference
   return widget;
 }
 
 // static
 WebWidget* RenderWidget::CreateWebWidget(RenderWidget* render_widget) {
   switch (render_widget->popup_type_) {
-    case WebKit::WebPopupTypeNone: // Nothing to create.
+    case WebKit::WebPopupTypeNone:  // Nothing to create.
       break;
     case WebKit::WebPopupTypeSelect:
     case WebKit::WebPopupTypeSuggestion:
       return WebPopupMenu::create(render_widget);
     default:
       NOTREACHED();
   }
   return NULL;
 }
 
@@ skipping 108 matching lines @@
   // Balances the AddRef taken when we called AddRoute.
   Release();
 }
 
 void RenderWidget::OnResize(const gfx::Size& new_size,
                             const gfx::Rect& resizer_rect) {
   // During shutdown we can just ignore this message.
   if (!webwidget_)
     return;
 
+  // We shouldn't be asked to resize to our current size.
+  DCHECK(size_ != new_size || resizer_rect_ != resizer_rect);
+
   // Remember the rect where the resize corner will be drawn.
   resizer_rect_ = resizer_rect;
 
+  if (size_ == new_size)
+    return;
+
   // TODO(darin): We should not need to reset this here.
   SetHidden(false);
   needs_repainting_on_restore_ = false;
 
-  // We shouldn't be asked to resize to our current size.
-  DCHECK(size_ != new_size);
   size_ = new_size;
 
   // We should not be sent a Resize message if we have not ACK'd the previous
   DCHECK(!next_paint_is_resize_ack());
 
   paint_aggregator_.ClearPendingUpdate();
 
   // When resizing, we want to wait to paint before ACK'ing the resize.  This
   // ensures that we only resize as fast as we can paint.  We only need to send
   // an ACK if we are resized to a non-empty rect.
@@ skipping 304 matching lines @@
   if (is_gpu_rendering_active_) {
     // If painting is done via the gpu process then we clear out all damage
     // rects to save the browser process from doing unecessary work.
     params.scroll_rect = gfx::Rect();
     params.copy_rects.clear();
   } else {
     params.scroll_rect = update.scroll_rect;
     params.copy_rects.swap(copy_rects);  // TODO(darin): clip to bounds?
   }
   params.view_size = size_;
+  params.resizer_rect = resizer_rect_;
   params.plugin_window_moves.swap(plugin_window_moves_);
   params.flags = next_paint_flags_;
 
   update_reply_pending_ = true;
   Send(new ViewHostMsg_UpdateRect(routing_id_, params));
   next_paint_flags_ = 0;
 
   UpdateInputMethod();
 
   // Let derived classes know we've painted.
@@ skipping 76 matching lines @@
       this, &RenderWidget::CallDoDeferredUpdate));
 }
 
 void RenderWidget::scheduleComposite() {
   // TODO(nduca): replace with something a little less hacky.  The reason this
   // hack is still used is because the Invalidate-DoDeferredUpdate loop
   // contains a lot of host-renderer synchronization logic that is still
   // important for the accelerated compositing case. The option of simply
   // duplicating all that code is less desirable than "faking out" the
   // invalidation path using a magical damage rect.
-  didInvalidateRect(WebRect(0,0,1,1));
+  didInvalidateRect(WebRect(0, 0, 1, 1));
 }
 
 void RenderWidget::didChangeCursor(const WebCursorInfo& cursor_info) {
   // TODO(darin): Eliminate this temporary.
   WebCursor cursor(cursor_info);
 
   // Only send a SetCursor message if we need to make a change.
   if (!current_cursor_.IsEqual(cursor)) {
     current_cursor_ = cursor;
     Send(new ViewHostMsg_SetCursor(routing_id_, cursor));
@@ skipping 334 matching lines @@
 
 void RenderWidget::CleanupWindowInPluginMoves(gfx::PluginWindowHandle window) {
   for (WebPluginGeometryVector::iterator i = plugin_window_moves_.begin();
        i != plugin_window_moves_.end(); ++i) {
     if (i->window == window) {
       plugin_window_moves_.erase(i);
       break;
     }
   }
 }