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

Unified Diff: content/renderer/render_view_impl.cc

Issue 10316020: Remove WebContentsImpl::OnDidRedirectProvisionalLoad. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Don't break unittest Created 8 years, 7 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/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index b31576c3632fc9d1c526ccfd650151124d2c93e5..0cedfe2cfc2be90ad167328d2d1d1d4f2cf7e310 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -2870,25 +2870,6 @@ void RenderViewImpl::didStartProvisionalLoad(WebFrame* frame) {
ds->request().url()));
}
-void RenderViewImpl::didReceiveServerRedirectForProvisionalLoad(
- WebFrame* frame) {
- if (frame->parent())
- return;
- // Received a redirect on the main frame.
- WebDataSource* data_source = frame->provisionalDataSource();
- if (!data_source) {
- // Should only be invoked when we have a data source.
- NOTREACHED();
- return;
- }
- std::vector<GURL> redirects;
- GetRedirectChain(data_source, &redirects);
- if (redirects.size() >= 2) {
- Send(new ViewHostMsg_DidRedirectProvisionalLoad(routing_id_, page_id_,
- GetOpenerUrl(), redirects[redirects.size() - 2], redirects.back()));
- }
-}
-
void RenderViewImpl::didFailProvisionalLoad(WebFrame* frame,
const WebURLError& error) {
// Notify the browser that we failed a provisional load with an error.

Powered by Google App Engine
This is Rietveld 408576698