diff options
Diffstat (limited to 'chrome/renderer')
-rw-r--r-- | chrome/renderer/render_view.cc | 10 | ||||
-rw-r--r-- | chrome/renderer/render_view.h | 3 |
2 files changed, 6 insertions, 7 deletions
diff --git a/chrome/renderer/render_view.cc b/chrome/renderer/render_view.cc index a2a37e1..f41e007 100644 --- a/chrome/renderer/render_view.cc +++ b/chrome/renderer/render_view.cc @@ -2736,8 +2736,7 @@ void RenderView::OnMsgShouldClose() { Send(new ViewHostMsg_ShouldClose_ACK(routing_id_, should_close)); } -void RenderView::OnClosePage(int new_render_process_host_id, - int new_request_id) { +void RenderView::OnClosePage(const ViewMsg_ClosePage_Params& params) { // TODO(creis): We'd rather use webview()->Close() here, but that currently // sets the WebView's delegate_ to NULL, preventing any JavaScript dialogs // in the onunload handler from appearing. For now, we're bypassing that and @@ -2751,14 +2750,13 @@ void RenderView::OnClosePage(int new_render_process_host_id, // TODO(davemoore) this code should be removed once WillCloseFrame() gets // called when a page is destroyed. DumpLoadHistograms() is safe to call // multiple times for the same frame, but it will simplify things. - if (url.SchemeIs("http") || url.SchemeIs("https")) + if (url.SchemeIs(chrome::kHttpScheme) || url.SchemeIs(chrome::kHttpsScheme)) DumpLoadHistograms(); main_frame->ClosePage(); } - Send(new ViewHostMsg_ClosePage_ACK(routing_id_, - new_render_process_host_id, - new_request_id)); + // Just echo back the params in the ACK. + Send(new ViewHostMsg_ClosePage_ACK(routing_id_, params)); } void RenderView::OnThemeChanged() { diff --git a/chrome/renderer/render_view.h b/chrome/renderer/render_view.h index d7288e6..7aa408029b 100644 --- a/chrome/renderer/render_view.h +++ b/chrome/renderer/render_view.h @@ -56,6 +56,7 @@ class WebPluginDelegateProxy; class WebDevToolsAgentDelegate; struct ContextMenuMediaParams; struct ThumbnailScore; +struct ViewMsg_ClosePage_Params; struct ViewMsg_Navigate_Params; struct ViewMsg_UploadFile_Params; struct WebDropData; @@ -552,7 +553,7 @@ class RenderView : public RenderWidget, // Runs the onunload handler and closes the page, replying with ClosePage_ACK // (with the given RPH and request IDs, to help track the request). - void OnClosePage(int new_render_process_host_id, int new_request_id); + void OnClosePage(const ViewMsg_ClosePage_Params& params); // Notification about ui theme changes. void OnThemeChanged(); |