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

Side by Side Diff: content/renderer/render_view_impl.cc

Issue 10640019: Remove the HANDLED_EXTERNALLY status code. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: fix more nits Created 8 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/renderer/render_view_impl.h" 5 #include "content/renderer/render_view_impl.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <cmath> 8 #include <cmath>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 3116 matching lines...) Expand 10 before | Expand all | Expand 10 after
3127 &params.error_description); 3127 &params.error_description);
3128 params.url = error.unreachableURL; 3128 params.url = error.unreachableURL;
3129 params.showing_repost_interstitial = show_repost_interstitial; 3129 params.showing_repost_interstitial = show_repost_interstitial;
3130 Send(new ViewHostMsg_DidFailProvisionalLoadWithError( 3130 Send(new ViewHostMsg_DidFailProvisionalLoadWithError(
3131 routing_id_, params)); 3131 routing_id_, params));
3132 3132
3133 // Don't display an error page if this is simply a cancelled load. Aside 3133 // Don't display an error page if this is simply a cancelled load. Aside
3134 // from being dumb, WebCore doesn't expect it and it will cause a crash. 3134 // from being dumb, WebCore doesn't expect it and it will cause a crash.
3135 if (error.reason == net::ERR_ABORTED) 3135 if (error.reason == net::ERR_ABORTED)
3136 return; 3136 return;
3137 // Don't display an error message if the request was handled by an
3138 // external protocol handler.
3139 if (error.reason == net::ERR_UNKNOWN_URL_SCHEME)
3140 return;
3141 3137
3142 // Make sure we never show errors in view source mode. 3138 // Make sure we never show errors in view source mode.
3143 frame->enableViewSourceMode(false); 3139 frame->enableViewSourceMode(false);
3144 3140
3145 DocumentState* document_state = DocumentState::FromDataSource(ds); 3141 DocumentState* document_state = DocumentState::FromDataSource(ds);
3146 NavigationState* navigation_state = document_state->navigation_state(); 3142 NavigationState* navigation_state = document_state->navigation_state();
3147 3143
3148 // If this is a failed back/forward/reload navigation, then we need to do a 3144 // If this is a failed back/forward/reload navigation, then we need to do a
3149 // 'replace' load. This is necessary to avoid messing up session history. 3145 // 'replace' load. This is necessary to avoid messing up session history.
3150 // Otherwise, we do a normal load, which simulates a 'go' navigation as far 3146 // Otherwise, we do a normal load, which simulates a 'go' navigation as far
(...skipping 2894 matching lines...) Expand 10 before | Expand all | Expand 10 after
6045 6041
6046 updating_frame_tree_ = true; 6042 updating_frame_tree_ = true;
6047 active_frame_id_map_.clear(); 6043 active_frame_id_map_.clear();
6048 6044
6049 target_process_id_ = process_id; 6045 target_process_id_ = process_id;
6050 target_routing_id_ = route_id; 6046 target_routing_id_ = route_id;
6051 CreateFrameTree(webview()->mainFrame(), frames); 6047 CreateFrameTree(webview()->mainFrame(), frames);
6052 6048
6053 updating_frame_tree_ = false; 6049 updating_frame_tree_ = false;
6054 } 6050 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698