Index: content/browser/renderer_host/render_widget_host.cc |
diff --git a/content/browser/renderer_host/render_widget_host.cc b/content/browser/renderer_host/render_widget_host.cc |
index 4d1f8570bb0074567650f2957769052f712e0c03..69da891a805a2ade69bda1c7ec53e569553f97df 100644 |
--- a/content/browser/renderer_host/render_widget_host.cc |
+++ b/content/browser/renderer_host/render_widget_host.cc |
@@ -890,17 +890,7 @@ bool RenderWidgetHostImpl::IsFullscreen() const { |
} |
void RenderWidgetHostImpl::SetShouldAutoResize(bool enable) { |
- // Note if this switches from true to false then one has to verify that the |
- // mechanics about all the messaging works. For example, what happens to a |
- // update message rect that was in progress from the render widget. Perhaps, |
- // on a transition to false, this should do a WasResized, but what if that |
- // will not trigger a resize message...etc. Due to these complications it is |
- // fitting that this method doesn't look like a simple set method. |
- DCHECK(enable); |
- |
- // TODO: Change this to enable instead of true when this supports turning |
- // off auto-resize. |
- should_auto_resize_ = true; |
+ should_auto_resize_ = enable; |
} |
void RenderWidgetHostImpl::Destroy() { |
@@ -1026,12 +1016,10 @@ void RenderWidgetHostImpl::OnMsgUpdateRect( |
// resize_ack_pending_ needs to be cleared before we call DidPaintRect, since |
// that will end up reaching GetBackingStore. |
- if (is_resize_ack || should_auto_resize_) { |
- if (is_resize_ack) { |
- DCHECK(resize_ack_pending_); |
- resize_ack_pending_ = false; |
- in_flight_size_.SetSize(0, 0); |
- } |
+ if (is_resize_ack) { |
+ DCHECK(resize_ack_pending_); |
+ resize_ack_pending_ = false; |
+ in_flight_size_.SetSize(0, 0); |
} |
bool is_repaint_ack = |