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

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

Issue 1836973003: Move download messages from Renderer to Frame filter. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_frame_impl.h" 5 #include "content/renderer/render_frame_impl.h"
6 6
7 #include <map> 7 #include <map>
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 2778 matching lines...) Expand 10 before | Expand all | Expand 10 after
2789 routing_id_, static_cast<int32_t>(log_severity), message.text, 2789 routing_id_, static_cast<int32_t>(log_severity), message.text,
2790 static_cast<int32_t>(source_line), source_name)); 2790 static_cast<int32_t>(source_line), source_name));
2791 } 2791 }
2792 2792
2793 void RenderFrameImpl::loadURLExternally(const blink::WebURLRequest& request, 2793 void RenderFrameImpl::loadURLExternally(const blink::WebURLRequest& request,
2794 blink::WebNavigationPolicy policy, 2794 blink::WebNavigationPolicy policy,
2795 const blink::WebString& suggested_name, 2795 const blink::WebString& suggested_name,
2796 bool should_replace_current_entry) { 2796 bool should_replace_current_entry) {
2797 Referrer referrer(RenderViewImpl::GetReferrerFromRequest(frame_, request)); 2797 Referrer referrer(RenderViewImpl::GetReferrerFromRequest(frame_, request));
2798 if (policy == blink::WebNavigationPolicyDownload) { 2798 if (policy == blink::WebNavigationPolicyDownload) {
2799 render_view_->Send(new ViewHostMsg_DownloadUrl(render_view_->GetRoutingID(), 2799 Send(new FrameHostMsg_DownloadUrl(render_view_->GetRoutingID(),
2800 GetRoutingID(), 2800 GetRoutingID(), request.url(), referrer,
2801 request.url(), referrer, 2801 suggested_name));
2802 suggested_name));
2803 } else { 2802 } else {
2804 OpenURL(request.url(), referrer, policy, should_replace_current_entry, 2803 OpenURL(request.url(), referrer, policy, should_replace_current_entry,
2805 false); 2804 false);
2806 } 2805 }
2807 } 2806 }
2808 2807
2809 blink::WebHistoryItem RenderFrameImpl::historyItemForNewChildFrame() { 2808 blink::WebHistoryItem RenderFrameImpl::historyItemForNewChildFrame() {
2810 // OOPIF enabled modes will punt this navigation to the browser in 2809 // OOPIF enabled modes will punt this navigation to the browser in
2811 // decidePolicyForNavigation. 2810 // decidePolicyForNavigation.
2812 if (SiteIsolationPolicy::UseSubframeNavigationEntries()) 2811 if (SiteIsolationPolicy::UseSubframeNavigationEntries())
(...skipping 3198 matching lines...) Expand 10 before | Expand all | Expand 10 after
6011 int match_count, 6010 int match_count,
6012 int ordinal, 6011 int ordinal,
6013 const WebRect& selection_rect, 6012 const WebRect& selection_rect,
6014 bool final_status_update) { 6013 bool final_status_update) {
6015 Send(new FrameHostMsg_Find_Reply(routing_id_, request_id, match_count, 6014 Send(new FrameHostMsg_Find_Reply(routing_id_, request_id, match_count,
6016 selection_rect, ordinal, 6015 selection_rect, ordinal,
6017 final_status_update)); 6016 final_status_update));
6018 } 6017 }
6019 6018
6020 } // namespace content 6019 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698