Add isVisible parameter to WebPlugin::updateGeometry.

This allows for geometry updates while the plugin is hidden so that when it
becomes visible, the plugin is properly positioned.

In this change, I also revised RenderWidget::SchedulePluginMove to properly
merge a move that only updates the visible state, and I fixed up TestShell
to properly handle moves that only update the visible state.

R=jam
BUG=19888,20260
TEST=covered by layout tests

Review URL: http://codereview.chromium.org/174513

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24457 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 2 deletions

diff --git a/webkit/glue/webplugin_impl.cc b/webkit/glue/webplugin_impl.cc
index 397fbd2..d10cf6c 100644
--- a/webkit/glue/webplugin_impl.cc
+++ b/webkit/glue/webplugin_impl.cc
@@ -285,7 +285,7 @@ void WebPluginImpl::paint(WebCanvas* canvas, const WebRect& paint_rect) {
 
 void WebPluginImpl::updateGeometry(
     const WebRect& window_rect, const WebRect& clip_rect,
-    const WebVector<WebRect>& cutout_rects) {
+    const WebVector<WebRect>& cutout_rects, bool is_visible) {
   if (window_) {
     WebViewDelegate* view_delegate = GetWebViewDelegate();
     if (view_delegate) {
@@ -299,7 +299,7 @@ void WebPluginImpl::updateGeometry(
       for (size_t i = 0; i < cutout_rects.size(); ++i)
         move.cutout_rects.push_back(cutout_rects[i]);
       move.rects_valid = true;
-      move.visible = true;  // Assume visible or else we wouldn't be here.
+      move.visible = is_visible;
 
       view_delegate->DidMovePlugin(move);
     }