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

Unified Diff: content/browser/web_contents/navigation_controller_impl.cc

Issue 11635059: navigation: Retain a screenshot of the page before it unloads (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years 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/web_contents/navigation_controller_impl.cc
diff --git a/content/browser/web_contents/navigation_controller_impl.cc b/content/browser/web_contents/navigation_controller_impl.cc
index 9481ed69c3db1a195ad77c892530b855e293adf8..bd6971646b8d428103ed5fcb13eb1b9d226ebf1e 100644
--- a/content/browser/web_contents/navigation_controller_impl.cc
+++ b/content/browser/web_contents/navigation_controller_impl.cc
@@ -5,6 +5,7 @@
#include "content/browser/web_contents/navigation_controller_impl.h"
#include "base/bind.h"
+#include "base/command_line.h"
#include "base/file_util.h"
#include "base/logging.h"
#include "base/string_number_conversions.h" // Temporary
@@ -28,15 +29,20 @@
#include "content/public/browser/navigation_details.h"
#include "content/public/browser/notification_service.h"
#include "content/public/browser/notification_types.h"
+#include "content/public/browser/render_widget_host.h"
+#include "content/public/browser/render_widget_host_view.h"
#include "content/public/browser/storage_partition.h"
#include "content/public/browser/user_metrics.h"
#include "content/public/browser/web_contents_delegate.h"
#include "content/public/common/content_client.h"
#include "content/public/common/content_constants.h"
+#include "content/public/common/content_switches.h"
#include "content/public/common/url_constants.h"
#include "net/base/escape.h"
#include "net/base/mime_util.h"
#include "net/base/net_util.h"
+#include "skia/ext/platform_canvas.h"
+#include "ui/gfx/codec/png_codec.h"
#include "webkit/glue/glue_serialize.h"
namespace content {
@@ -470,6 +476,62 @@ int NavigationControllerImpl::GetIndexForOffset(int offset) const {
return GetCurrentEntryIndex() + offset;
}
+void NavigationControllerImpl::TakeScreenshot() {
+ static bool overscroll_enabled = CommandLine::ForCurrentProcess()->
+ HasSwitch(switches::kEnableOverscrollHistoryNavigation);
+ if (!overscroll_enabled)
+ return;
+
+ NavigationEntryImpl* entry =
+ NavigationEntryImpl::FromNavigationEntry(GetLastCommittedEntry());
+ if (!entry)
+ return;
+
+ RenderViewHost* render_view_host = web_contents_->GetRenderViewHost();
+ content::RenderWidgetHostView* view = render_view_host->GetView();
+ if (!view)
+ return;
+
+ skia::PlatformBitmap* temp_bitmap = new skia::PlatformBitmap;
+ render_view_host->CopyFromBackingStore(gfx::Rect(),
+ view->GetViewBounds().size(),
+ base::Bind(&NavigationControllerImpl::OnScreenshotTaken,
+ base::Unretained(this),
+ entry->GetUniqueID(),
+ base::Owned(temp_bitmap)),
+ temp_bitmap);
+}
+
+void NavigationControllerImpl::OnScreenshotTaken(
+ int unique_id,
+ skia::PlatformBitmap* bitmap,
+ bool success) {
+ if (!success) {
+ LOG(ERROR) << "Taking snapshot was unsuccessful for "
+ << unique_id;
+ return;
+ }
+
+ NavigationEntryImpl* entry = NULL;
+ for (NavigationEntries::iterator i = entries_.begin();
+ i != entries_.end();
+ ++i) {
+ if ((*i)->GetUniqueID() == unique_id) {
+ entry = (*i).get();
+ break;
+ }
+ }
+
+ if (!entry) {
+ LOG(ERROR) << "Invalid entry with unique id: " << unique_id;
+ return;
+ }
+
+ std::vector<unsigned char> data;
+ if (gfx::PNGCodec::EncodeBGRASkBitmap(bitmap->GetBitmap(), true, &data))
+ entry->SetScreenshotPNGData(data);
+}
+
bool NavigationControllerImpl::CanGoBack() const {
return entries_.size() > 1 && GetCurrentEntryIndex() > 0;
}
@@ -690,6 +752,11 @@ void NavigationControllerImpl::DocumentLoadedInFrame() {
bool NavigationControllerImpl::RendererDidNavigate(
const ViewHostMsg_FrameNavigate_Params& params,
LoadCommittedDetails* details) {
+ // When navigating away from the current page, take a screenshot of it in the
+ // current state so that it can be used during an overscroll-navigation
+ // gesture.
+ if (details->is_main_frame)
+ TakeScreenshot();
// Save the previous state before we clobber it.
if (GetLastCommittedEntry()) {
« no previous file with comments | « content/browser/web_contents/navigation_controller_impl.h ('k') | content/browser/web_contents/navigation_entry_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698