Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1742)

Unified Diff: content/browser/renderer_host/render_widget_host_view_win.cc

Issue 10815070: Support copying a partial rectangle region from the compositing surface on Aura and GTK. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address comments Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/render_widget_host_view_win.cc
diff --git a/content/browser/renderer_host/render_widget_host_view_win.cc b/content/browser/renderer_host/render_widget_host_view_win.cc
index c375154a775c79addd934041d5485957659144b8..f08543709e5d8531def7bc8feb277fd6db2cd31e 100644
--- a/content/browser/renderer_host/render_widget_host_view_win.cc
+++ b/content/browser/renderer_host/render_widget_host_view_win.cc
@@ -1197,21 +1197,29 @@ BackingStore* RenderWidgetHostViewWin::AllocBackingStore(
}
void RenderWidgetHostViewWin::CopyFromCompositingSurface(
- const gfx::Size& size,
+ const gfx::Rect& src_subrect,
+ const gfx::Size& dst_size,
const base::Callback<void(bool)>& callback,
skia::PlatformCanvas* output) {
base::ScopedClosureRunner scoped_callback_runner(base::Bind(callback, false));
+ // TODO(mazda): Support copying a partial rectangle from the compositing
+ // surface with |src_subrect| (http://crbug.com/118571).
+ if (!src_subrect.IsEmpty()) {
+ NOTIMPLEMENTED();
+ return;
+ }
+
if (!accelerated_surface_.get())
return;
- if (size.IsEmpty())
+ if (dst_size.IsEmpty())
return;
- if (!output->initialize(size.width(), size.height(), true))
+ if (!output->initialize(dst_size.width(), dst_size.height(), true))
return;
const bool result = accelerated_surface_->CopyTo(
- size, output->getTopDevice()->accessBitmap(true).getPixels());
+ dst_size, output->getTopDevice()->accessBitmap(true).getPixels());
scoped_callback_runner.Release();
callback.Run(result);
}
« no previous file with comments | « content/browser/renderer_host/render_widget_host_view_win.h ('k') | content/browser/renderer_host/test_render_view_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698