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

Unified Diff: content/renderer/render_widget.cc

Issue 259523008: Remove RenderWidgetHost::GetSnapshotFromRenderer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: getsnapshotfromrenderer: rm Created 6 years, 8 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
« no previous file with comments | « content/renderer/render_widget.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_widget.cc
diff --git a/content/renderer/render_widget.cc b/content/renderer/render_widget.cc
index 4ab9a5e5cd6b980a9da341c09bfb6ced9c65898b..bc9501ebb785efca5b1a7b4cc0b77380d4cf8638 100644
--- a/content/renderer/render_widget.cc
+++ b/content/renderer/render_widget.cc
@@ -605,7 +605,6 @@ bool RenderWidget::OnMessageReceived(const IPC::Message& message) {
IPC_MESSAGE_HANDLER(ViewMsg_ShowImeIfNeeded, OnShowImeIfNeeded)
IPC_MESSAGE_HANDLER(ViewMsg_ImeEventAck, OnImeEventAck)
#endif
- IPC_MESSAGE_HANDLER(ViewMsg_Snapshot, OnSnapshot)
IPC_MESSAGE_UNHANDLED(handled = false)
IPC_END_MESSAGE_MAP()
return handled;
@@ -1713,50 +1712,6 @@ void RenderWidget::OnImeConfirmComposition(const base::string16& text,
#endif
}
-void RenderWidget::OnSnapshot(const gfx::Rect& src_subrect) {
- SkBitmap snapshot;
-
- if (OnSnapshotHelper(src_subrect, &snapshot)) {
- Send(new ViewHostMsg_Snapshot(routing_id(), true, snapshot));
- } else {
- Send(new ViewHostMsg_Snapshot(routing_id(), false, SkBitmap()));
- }
-}
-
-bool RenderWidget::OnSnapshotHelper(const gfx::Rect& src_subrect,
- SkBitmap* snapshot) {
- base::TimeTicks beginning_time = base::TimeTicks::Now();
-
- if (!webwidget_ || src_subrect.IsEmpty())
- return false;
-
- gfx::Rect viewport_size = gfx::IntersectRects(
- src_subrect, gfx::Rect(physical_backing_size_));
-
- skia::RefPtr<SkCanvas> canvas = skia::AdoptRef(
- skia::CreatePlatformCanvas(viewport_size.width(),
- viewport_size.height(),
- true,
- NULL,
- skia::RETURN_NULL_ON_FAILURE));
- if (!canvas)
- return false;
-
- canvas->save();
- webwidget_->layout();
-
- PaintRect(viewport_size, viewport_size.origin(), canvas.get());
- canvas->restore();
-
- const SkBitmap& bitmap = skia::GetTopDevice(*canvas)->accessBitmap(false);
- if (!bitmap.copyTo(snapshot, kPMColor_SkColorType))
- return false;
-
- UMA_HISTOGRAM_TIMES("Renderer4.Snapshot",
- base::TimeTicks::Now() - beginning_time);
- return true;
-}
-
void RenderWidget::OnRepaint(gfx::Size size_to_paint) {
// During shutdown we can just ignore this message.
if (!webwidget_)
« no previous file with comments | « content/renderer/render_widget.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698