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

Side by Side Diff: content/browser/renderer_host/render_view_host_impl.h

Issue 50223002: Prevent modal dialogs when preparing to swap out. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 years, 1 month 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
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 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_ 5 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_
6 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_ 6 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 285 matching lines...) Expand 10 before | Expand all | Expand 10 after
296 // handler and determine whether a transfer to a different RenderViewHost is 296 // handler and determine whether a transfer to a different RenderViewHost is
297 // needed. 297 // needed.
298 void OnCrossSiteResponse( 298 void OnCrossSiteResponse(
299 const GlobalRequestID& global_request_id, 299 const GlobalRequestID& global_request_id,
300 bool is_transfer, 300 bool is_transfer,
301 const std::vector<GURL>& transfer_url_chain, 301 const std::vector<GURL>& transfer_url_chain,
302 const Referrer& referrer, 302 const Referrer& referrer,
303 PageTransition page_transition, 303 PageTransition page_transition,
304 int64 frame_id); 304 int64 frame_id);
305 305
306 // Tells the renderer that this RenderView will soon be swapped out, and thus
307 // not to create any new modal dialogs until it happens. This must be done
308 // separately so that the PageGroupLoadDeferrers of any current dialogs are no
309 // longer on the stack when we attempt to swap it out.
310 void SuppressDialogsUntilSwapOut();
311
306 // Tells the renderer that this RenderView is being swapped out for one in a 312 // Tells the renderer that this RenderView is being swapped out for one in a
307 // different renderer process. It should run its unload handler and move to 313 // different renderer process. It should run its unload handler and move to
308 // a blank document. The renderer should preserve the Frame object until it 314 // a blank document. The renderer should preserve the Frame object until it
309 // exits, in case we come back. The renderer can exit if it has no other 315 // exits, in case we come back. The renderer can exit if it has no other
310 // active RenderViews, but not until WasSwappedOut is called (when it is no 316 // active RenderViews, but not until WasSwappedOut is called (when it is no
311 // longer visible). 317 // longer visible).
312 void SwapOut(); 318 void SwapOut();
313 319
314 // Called when either the SwapOut request has been acknowledged or has timed 320 // Called when either the SwapOut request has been acknowledged or has timed
315 // out. 321 // out.
(...skipping 417 matching lines...) Expand 10 before | Expand all | Expand 10 after
733 DISALLOW_COPY_AND_ASSIGN(RenderViewHostImpl); 739 DISALLOW_COPY_AND_ASSIGN(RenderViewHostImpl);
734 }; 740 };
735 741
736 #if defined(COMPILER_MSVC) 742 #if defined(COMPILER_MSVC)
737 #pragma warning(pop) 743 #pragma warning(pop)
738 #endif 744 #endif
739 745
740 } // namespace content 746 } // namespace content
741 747
742 #endif // CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_ 748 #endif // CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_
OLDNEW
« no previous file with comments | « content/browser/frame_host/render_view_host_manager.cc ('k') | content/browser/renderer_host/render_view_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698