Fix transparent windowless plugin background painting. This would manifest...

chrome/plugin/webplugin_proxy.cc

 // Copyright (c) 2009 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 "chrome/plugin/webplugin_proxy.h"
 
 #include "build/build_config.h"
 
 #include "app/gfx/canvas.h"
 #if defined(OS_WIN)
@@ skipping 358 matching lines @@
     return;
 #endif
 
   // Clear the damaged area so that if the plugin doesn't paint there we won't
   // end up with the old values.
   gfx::Rect offset_rect = rect;
   offset_rect.Offset(delegate_->GetRect().origin());
 #if defined(OS_WIN) || defined(OS_LINUX)
   windowless_canvas_->save();
 
-  // The given clip rect is in global coordinates, so install it before the
-  // transformation is done for the coordinate system.
+  // The given clip rect is relative to the plugin coordinate system.
   SkRect sk_rect = { SkIntToScalar(rect.x()),
                      SkIntToScalar(rect.y()),
                      SkIntToScalar(rect.right()),
                      SkIntToScalar(rect.bottom()) };
   windowless_canvas_->clipRect(sk_rect);
-  windowless_canvas_->translate(SkIntToScalar(-delegate_->GetRect().x()),
-                                SkIntToScalar(-delegate_->GetRect().y()));
 
   // Setup the background.
-  if (!background_canvas_.get()) {
+  if (background_canvas_.get()) {
+    // When a background canvas is given, we're in transparent mode. This means
+    // the plugin wants to have the image of the page in the canvas it's drawing
+    // into (which is windowless_canvas_) so it can do blending. So we copy the
+    // background bitmap into the windowless_canvas_.
+    const SkBitmap& background_bitmap =
+        background_canvas_->getTopPlatformDevice().accessBitmap(false);
+    windowless_canvas_->drawBitmap(background_bitmap, 0, 0);
+  } else {
+    // In non-transparent mode, the plugin doesn't care what's underneath, so we
+    // can just give it black.
     SkPaint black_fill_paint;
     black_fill_paint.setARGB(0xFF, 0x00, 0x00, 0x00);
     windowless_canvas_->drawPaint(black_fill_paint);
-  } else {
-    SkIRect src_rect = { rect.x(), rect.y(),
-                         rect.x() + offset_rect.width(),
-                         rect.y() + offset_rect.height() };
+  }
 
-    SkRect dest_rect = { SkIntToScalar(offset_rect.x()),
-                         SkIntToScalar(offset_rect.y()),
-                         SkIntToScalar(offset_rect.right()),
-                         SkIntToScalar(offset_rect.bottom()) };
-    const SkBitmap& background_bitmap =
-        background_canvas_->getTopPlatformDevice().accessBitmap(false);
-    windowless_canvas_->drawBitmapRect(background_bitmap, &src_rect, dest_rect);
-  }
+  // Bring the windowless_canvas_ into the window coordinate system, which is
+  // how the plugin expects to draw (since the windowless API was originally
+  // designed just for scribbling over the web page).
+  windowless_canvas_->translate(SkIntToScalar(-delegate_->GetRect().x()),
+                                SkIntToScalar(-delegate_->GetRect().y()));
 
   // Before we send the invalidate, paint so that renderer uses the updated
   // bitmap.
   delegate_->Paint(windowless_canvas_.get(), offset_rect);
 
   windowless_canvas_->restore();
 #elif defined(OS_MACOSX)
   CGContextSaveGState(windowless_context_);
   if (!background_context_.get()) {
     CGContextSetFillColorWithColor(windowless_context_,
@@ skipping 166 matching lines @@
   while (index != resource_clients_.end()) {
     WebPluginResourceClient* client = (*index).second;
 
     if (client == resource_client) {
       resource_clients_.erase(index++);
     } else {
       index++;
     }
   }
 }