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

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

Issue 2862093002: Remove DCHECK that crashes.
Patch Set: Created 3 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 unified diff | Download patch
« no previous file with comments | « content/common/frame_messages.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 4261 matching lines...) Expand 10 before | Expand all | Expand 10 after
4272 bool RenderFrameImpl::RunModalBeforeUnloadDialog(bool is_reload) { 4272 bool RenderFrameImpl::RunModalBeforeUnloadDialog(bool is_reload) {
4273 // Don't allow further dialogs if we are waiting to swap out, since the 4273 // Don't allow further dialogs if we are waiting to swap out, since the
4274 // ScopedPageLoadDeferrer in our stack prevents it. 4274 // ScopedPageLoadDeferrer in our stack prevents it.
4275 if (suppress_further_dialogs_) 4275 if (suppress_further_dialogs_)
4276 return false; 4276 return false;
4277 4277
4278 bool success = false; 4278 bool success = false;
4279 // This is an ignored return value, but is included so we can accept the same 4279 // This is an ignored return value, but is included so we can accept the same
4280 // response as RunJavaScriptDialog. 4280 // response as RunJavaScriptDialog.
4281 base::string16 ignored_result; 4281 base::string16 ignored_result;
4282 Send(new FrameHostMsg_RunBeforeUnloadConfirm( 4282 Send(new FrameHostMsg_RunBeforeUnloadConfirm(routing_id_, is_reload, &success,
4283 routing_id_, frame_->GetDocument().Url(), is_reload, &success, 4283 &ignored_result));
4284 &ignored_result));
4285 return success; 4284 return success;
4286 } 4285 }
4287 4286
4288 bool RenderFrameImpl::RunFileChooser( 4287 bool RenderFrameImpl::RunFileChooser(
4289 const blink::WebFileChooserParams& params, 4288 const blink::WebFileChooserParams& params,
4290 blink::WebFileChooserCompletion* chooser_completion) { 4289 blink::WebFileChooserCompletion* chooser_completion) {
4291 FileChooserParams ipc_params; 4290 FileChooserParams ipc_params;
4292 if (params.directory) 4291 if (params.directory)
4293 ipc_params.mode = FileChooserParams::UploadFolder; 4292 ipc_params.mode = FileChooserParams::UploadFolder;
4294 else if (params.multi_select) 4293 else if (params.multi_select)
(...skipping 2801 matching lines...) Expand 10 before | Expand all | Expand 10 after
7096 policy(info.default_policy), 7095 policy(info.default_policy),
7097 replaces_current_history_item(info.replaces_current_history_item), 7096 replaces_current_history_item(info.replaces_current_history_item),
7098 history_navigation_in_new_child_frame( 7097 history_navigation_in_new_child_frame(
7099 info.is_history_navigation_in_new_child_frame), 7098 info.is_history_navigation_in_new_child_frame),
7100 client_redirect(info.is_client_redirect), 7099 client_redirect(info.is_client_redirect),
7101 cache_disabled(info.is_cache_disabled), 7100 cache_disabled(info.is_cache_disabled),
7102 form(info.form), 7101 form(info.form),
7103 source_location(info.source_location) {} 7102 source_location(info.source_location) {}
7104 7103
7105 } // namespace content 7104 } // namespace content
OLDNEW
« no previous file with comments | « content/common/frame_messages.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698