Support browser side thumbnailing for GPU composited pages on Windows.

content/browser/renderer_host/render_widget_host_impl.cc

 // Copyright (c) 2012 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/browser/renderer_host/render_widget_host_impl.h"
 
 #include "base/auto_reset.h"
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
@@ skipping 14 matching lines @@
 #include "content/common/gpu/gpu_messages.h"
 #include "content/common/view_messages.h"
 #include "content/port/browser/render_widget_host_view_port.h"
 #include "content/public/browser/browser_accessibility_state.h"
 #include "content/public/browser/native_web_keyboard_event.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/browser/user_metrics.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/common/result_codes.h"
+#include "skia/ext/platform_canvas.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebCompositionUnderline.h"
 #include "ui/base/keycodes/keyboard_codes.h"
 #include "webkit/glue/webcursor.h"
 #include "webkit/glue/webpreferences.h"
 #include "webkit/plugins/npapi/webplugin.h"
 
 #if defined(TOOLKIT_GTK)
 #include "content/browser/renderer_host/backing_store_gtk.h"
 #elif defined(OS_MACOSX)
 #include "content/browser/renderer_host/backing_store_mac.h"
@@ skipping 440 matching lines @@
   if (!view_)
     return;
   view_->SetIsLoading(is_loading);
 }
 
 bool RenderWidgetHostImpl::CopyFromBackingStore(skia::PlatformCanvas* output) {
   BackingStore* backing_store = GetBackingStore(false);
   if (!backing_store)
     return false;
 
+  TRACE_EVENT0("browser", "RenderWidgetHostImpl::CopyFromBackingStore");
   return backing_store->CopyFromBackingStore(
       gfx::Rect(backing_store->size()), output);
 }
 
+bool RenderWidgetHostImpl::CopyFromCompositingSurface(
+    const gfx::Size& size,
+    skia::PlatformCanvas* output) {
+  if (!is_accelerated_compositing_active_)
+    return false;
+
+  TRACE_EVENT0("browser", "RenderWidgetHostImpl::CopyFromCompositingSurface");
+  std::vector<unsigned char> pixels(4 * size.GetArea());
+  if(!GpuSurfaceTracker::Get()->CopySurface(surface_id_,
+                                            size,
+                                            &pixels))

[apatrick_chromium, 2012/03/07 20:39:37]

Is it possible to copy the pixels directly into the SkBitmap rather than copy
them from the vector into the SkBitmap later?

[mazda, 2012/03/08 13:14:28]

Actually SkBitmap::setPixels does not copy the data to SkBitmap and just sets
the pointer, so there should be little difference (I also noticed that
allocPixels is not necessary).
SkCanvas::writePixels does copy the data, but there's no way to write the pixels
directly to SkCanvas with DirectX as far as I know.

[apatrick_chromium, 2012/03/08 20:15:47]

I think you could call SkBitmap::allocPixels before CopySurface, call
SkBitmap::getPixels to get the address of the allocated buffer and pass that as
the pixels argument to CopySurface. That would mean changing the the type of the
pixels argument from std::vector* to void* and the code in AcceleratedPresenter
would just assume that the output buffer was exacly the size implied by the size
argument.

[mazda, 2012/03/10 07:51:37]

Thanks. I changed the AcceleratedSurface::CopyTo to take void*.

+    return false;
+
+  SkBitmap bitmap;
+  bitmap.setConfig(SkBitmap::kARGB_8888_Config, size.width(), size.height());
+  bitmap.allocPixels();
+  bitmap.setPixels(&pixels[0]);

[apatrick_chromium, 2012/03/07 20:39:37]

This won't work if the area of the surface is zero because pixels will not have
an element at index zero to take the address of and will assert in debug builds.

[mazda, 2012/03/08 13:14:28]

I added a check for |size| and also made AcceleratorSurface::CopyTo fail if the
area of surface is zero.

+
+  if (!output->initialize(size.width(), size.height(), true))
+    return false;
+  output->writePixels(bitmap, 0, 0);
+  return true;
+}
+
 #if defined(TOOLKIT_GTK)
 bool RenderWidgetHostImpl::CopyFromBackingStoreToGtkWindow(
     const gfx::Rect& dest_rect, GdkWindow* target) {
   BackingStore* backing_store = GetBackingStore(false);
   if (!backing_store)
     return false;
   (static_cast<BackingStoreGtk*>(backing_store))->PaintToRect(
       dest_rect, target);
   return true;
 }
@@ skipping 1153 matching lines @@
     ui_shim->Send(new AcceleratedSurfaceMsg_BuffersSwappedACK(route_id));
 }
 
 // static
 void RenderWidgetHostImpl::AcknowledgePostSubBuffer(int32 route_id,
                                                     int gpu_host_id) {
   GpuProcessHostUIShim* ui_shim = GpuProcessHostUIShim::FromID(gpu_host_id);
   if (ui_shim)
     ui_shim->Send(new AcceleratedSurfaceMsg_PostSubBufferACK(route_id));
 }