Fix media player painting bug.

webkit/glue/plugins/webplugin_impl.cc

 // Copyright (c) 2010 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 "base/logging.h"
 #include "base/message_loop.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "gfx/rect.h"
 #include "net/base/escape.h"
@@ skipping 222 matching lines @@
 void WebPluginImpl::destroy() {
   SetContainer(NULL);
   MessageLoop::current()->DeleteSoon(FROM_HERE, this);
 }
 
 NPObject* WebPluginImpl::scriptableObject() {
   return delegate_->GetPluginScriptableObject();
 }
 
 void WebPluginImpl::paint(WebCanvas* canvas, const WebRect& paint_rect) {
-  if (!delegate_)
+  if (!delegate_ || !container_)
     return;
 
+#if defined(OS_WIN)
+  // Force a geometry update if needed to allow plugins like media player
+  // which defer the initial geometry update to work.
+  container_->reportGeometry();
+#endif  // OS_WIN
+
   // Note that |canvas| is only used when in windowless mode.
   delegate_->Paint(canvas, paint_rect);
 }
 
 void WebPluginImpl::updateGeometry(
     const WebRect& window_rect, const WebRect& clip_rect,
     const WebVector<WebRect>& cutout_rects, bool is_visible) {
   WebPluginGeometry new_geometry;
   new_geometry.window = window_;
   new_geometry.window_rect = window_rect;
@@ skipping 28 matching lines @@
       // receiving valid plugin geometry. By valid geometry we mean the
       // geometry received by a call to setFrameRect in the Webkit
       // layout code path. To workaround this issue we download the
       // plugin source url on a timer.
       MessageLoop::current()->PostDelayedTask(
           FROM_HERE, method_factory_.NewRunnableMethod(
               &WebPluginImpl::OnDownloadPluginSrcUrl), 0);
     }
   }
 
+#if defined(OS_WIN)
+  // Don't cache the geometry during the first geometry update. The first
+  // geometry update sequence is received when Widget::setParent is called.
+  // For plugins like media player which have a bug where they only honor
+  // the first geometry update, we have a quirk which ignores the first
+  // geometry update. To ensure that these plugins work correctly in cases
+  // where we receive only one geometry update from webkit, we also force
+  // a geometry update during paint which should go out correctly as the
+  // initial geometry update was not cached.
+  if (!first_geometry_update_)
+    geometry_ = new_geometry;
+#else  // OS_WIN
+  geometry_ = new_geometry;
+#endif  // OS_WIN
   first_geometry_update_ = false;
-  geometry_ = new_geometry;
 }
 
 void WebPluginImpl::updateFocus(bool focused) {
   if (accepts_input_events_)
     delegate_->SetFocus(focused);
 }
 
 void WebPluginImpl::updateVisibility(bool visible) {
   if (!window_ || !page_delegate_)
     return;
@@ skipping 926 matching lines @@
 WebDevToolsAgent* WebPluginImpl::GetDevToolsAgent() {
   if (!webframe_)
     return NULL;
   WebView* view = webframe_->view();
   if (!view)
     return NULL;
   return view->devToolsAgent();
 }
 
 }  // namespace webkit_glue