[content shell] implement auto resize related WebTestDelegate interfaces

content/renderer/render_widget.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/render_widget.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
@@ skipping 387 matching lines @@
 
     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.
     webwidget_->resize(new_size);
 
     // Resize should have caused an invalidation of the entire view.
     DCHECK(new_size.IsEmpty() || is_accelerated_compositing_active_ ||
-           paint_aggregator_.HasPendingUpdate());
+           auto_resize_mode_ || paint_aggregator_.HasPendingUpdate());

[piman, 2013/05/14 19:27:37]

This looks wrong. Why wouldn't we get an invalidation if auto_resize_mode_ is
true?

[jochen (gone - plz use gerrit), 2013/05/21 13:49:11]

oh well

because I abused the didAutoResize() call. I cleaned that up and now invoke
setWindowRect() from ForceResizeRenderView. That way didAutoResize doesn't call
to Resize and we don't need this

   } else if (!RenderThreadImpl::current()->short_circuit_size_updates()) {
     resize_ack = NO_RESIZE_ACK;
   }
 
   // Send the Resize_ACK flag once we paint again if requested.
   if (resize_ack == SEND_RESIZE_ACK && !new_size.IsEmpty())
     set_next_paint_is_resize_ack();
 
   if (fullscreen_change)
     DidToggleFullscreen();
@@ skipping 1927 matching lines @@
 
   if (!context->Initialize(
           attributes,
           false /* bind generates resources */,
           CAUSE_FOR_GPU_LAUNCH_WEBGRAPHICSCONTEXT3DCOMMANDBUFFERIMPL_INITIALIZE))
     return NULL;
   return context.release();
 }
 
 }  // namespace content