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

Side by Side Diff: chrome/browser/renderer_host/render_view_host.cc

Issue 246080: Prevents a duplicate beforeunload dialog after window.close().... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 years, 2 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 | Annotate | Revision Log
« no previous file with comments | « chrome/browser/browser_browsertest.cc ('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 (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 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 "chrome/browser/renderer_host/render_view_host.h" 5 #include "chrome/browser/renderer_host/render_view_host.h"
6 6
7 #include <string> 7 #include <string>
8 #include <vector> 8 #include <vector>
9 9
10 #include "app/l10n_util.h" 10 #include "app/l10n_util.h"
(...skipping 977 matching lines...) Expand 10 before | Expand all | Expand 10 after
988 const SkBitmap& bitmap) { 988 const SkBitmap& bitmap) {
989 delegate_->UpdateThumbnail(url, bitmap, score); 989 delegate_->UpdateThumbnail(url, bitmap, score);
990 } 990 }
991 991
992 void RenderViewHost::OnUpdateInspectorSettings( 992 void RenderViewHost::OnUpdateInspectorSettings(
993 const std::wstring& raw_settings) { 993 const std::wstring& raw_settings) {
994 delegate_->UpdateInspectorSettings(raw_settings); 994 delegate_->UpdateInspectorSettings(raw_settings);
995 } 995 }
996 996
997 void RenderViewHost::OnMsgClose() { 997 void RenderViewHost::OnMsgClose() {
998 delegate_->Close(this); 998 // If the renderer is telling us to close, it has already run the unload
999 // events, and we can take the fast path.
1000 ClosePageIgnoringUnloadEvents();
999 } 1001 }
1000 1002
1001 void RenderViewHost::OnMsgRequestMove(const gfx::Rect& pos) { 1003 void RenderViewHost::OnMsgRequestMove(const gfx::Rect& pos) {
1002 delegate_->RequestMove(pos); 1004 delegate_->RequestMove(pos);
1003 Send(new ViewMsg_Move_ACK(routing_id())); 1005 Send(new ViewMsg_Move_ACK(routing_id()));
1004 } 1006 }
1005 1007
1006 void RenderViewHost::OnMsgDidRedirectProvisionalLoad(int32 page_id, 1008 void RenderViewHost::OnMsgDidRedirectProvisionalLoad(int32 page_id,
1007 const GURL& source_url, 1009 const GURL& source_url,
1008 const GURL& target_url) { 1010 const GURL& target_url) {
(...skipping 676 matching lines...) Expand 10 before | Expand all | Expand 10 after
1685 #endif 1687 #endif
1686 } 1688 }
1687 1689
1688 void RenderViewHost::OnCSSInserted() { 1690 void RenderViewHost::OnCSSInserted() {
1689 delegate_->DidInsertCSS(); 1691 delegate_->DidInsertCSS();
1690 } 1692 }
1691 1693
1692 void RenderViewHost::UpdateBrowserWindowId(int window_id) { 1694 void RenderViewHost::UpdateBrowserWindowId(int window_id) {
1693 Send(new ViewMsg_UpdateBrowserWindowId(routing_id(), window_id)); 1695 Send(new ViewMsg_UpdateBrowserWindowId(routing_id(), window_id));
1694 } 1696 }
OLDNEW
« no previous file with comments | « chrome/browser/browser_browsertest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698