Add mayHaveChangedRenderedSize callback.

content/renderer/render_view.cc

 // Copyright (c) 2011 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_view.h"
 
 #include <algorithm>
 #include <cmath>
 #include <string>
 #include <vector>
@@ skipping 2697 matching lines @@
       RenderViewObserver,
       observers_,
       LogCrossFramePropertyAccess(
           frame, target, cross_origin, property_name, event_id));
 }
 
 void RenderView::didChangeContentsSize(WebFrame* frame, const WebSize& size) {
   CheckPreferredSize();
 }
 
+void RenderView::mayHaveChangedRenderedSize(WebFrame* frame) {
+  if (!send_preferred_size_changes_ || !webview())
+    return;
+  // If we hit this code path, then stop the deprecated timer.
+  check_preferred_size_timer_.Stop();
+  preferred_size_change_timer_.Stop();
+  preferred_size_change_timer_.Start(TimeDelta::FromMilliseconds(10), this,

[jamesr, 2011/05/24 22:53:33]

based on what base/timer.h says i think this Stop/Start() should be a Reset()

[levin, 2011/05/24 23:00:23]

Good catch. I missed that call actually, but I don't think it works in this
case.

It appears to only works if the timer is running. (Also this is called the first
time the timer is set-up.)

+                                     &RenderView::CheckPreferredSize);
+}
+
 void RenderView::CheckPreferredSize() {
   // We don't always want to send the change messages over IPC, only if we've
   // be put in that mode by getting a |ViewMsg_EnablePreferredSizeChangedMode|
   // message.
   if (!send_preferred_size_changes_ || !webview())
     return;
 
   // WebCore likes to tell us things have changed even when they haven't, so
   // cache the width and height and only send the IPC message when we're sure
   // they're different.
@@ skipping 1451 matching lines @@
     const webkit_glue::CustomContextMenuContext& custom_context) {
   if (custom_context.is_pepper_menu)
     pepper_delegate_.OnContextMenuClosed(custom_context);
   else
     context_menu_node_.reset();
 }
 
 void RenderView::OnNetworkStateChanged(bool online) {
   WebNetworkStateNotifier::setOnLine(online);
 }