Fixing resize for HW accelerated compositing.

content/renderer/browser_plugin/browser_plugin.cc

 // Copyright (c) 2012 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 "content/renderer/browser_plugin/browser_plugin.h"
 
 #include "base/json/json_string_value_serializer.h"
 #include "base/message_loop.h"
 #include "base/string_number_conversions.h"
 #include "base/string_util.h"
@@ skipping 409 matching lines @@
     SwapDamageBuffers();
     use_new_damage_buffer = true;
   }
 
   if (params.is_resize_ack || !UsesDamageBuffer(params))
     resize_ack_received_ = true;
 
   if ((!auto_size_ &&
        (width() != params.view_size.width() ||
         height() != params.view_size.height())) ||
       (auto_size_ && (!InAutoSizeBounds(params.view_size)))) {

[Cris Neckar, 2013/01/08 23:22:35]

Not part of your CL but it feels like InAutoSizeBounds() should probably account
for negative height/width.

+    // We are HW accelerated, render widget does not expect an ack,
+    // but we still need to update the size.
+    if (!params.needs_ack) {
+      UpdateGuestAutoSizeState();
+      return;
+    }
+
     if (!resize_ack_received_) {
       // The guest has not yet responded to the last resize request, and
       // so we don't want to do anything at this point other than ACK the guest.
       PopulateAutoSizeParameters(&auto_size_params);
     } else {
       // If we have no pending damage buffer, then the guest has not caught up
       // with the BrowserPlugin container. We now tell the guest about the new
       // container size.
       GetDamageBufferWithSizeParams(&auto_size_params,
                                     &resize_guest_params);
@@ skipping 598 matching lines @@
                                                  void* notify_data) {
 }
 
 void BrowserPlugin::didFailLoadingFrameRequest(
     const WebKit::WebURL& url,
     void* notify_data,
     const WebKit::WebURLError& error) {
 }
 
 }  // namespace content