Chromium Code Reviews Chromium Code Reviews
Issue 109653014:
Allow RenderFrames to swap out for subframe navigations. (Closed)
Base URL: svn://svn.chromium.org/chrome/trunk/src| OLD | NEW |
|---|---|
| 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 #include "content/common/swapped_out_messages.h" | 5 #include "content/common/swapped_out_messages.h" |
| 6 | 6 |
| 7 #include "content/common/accessibility_messages.h" | 7 #include "content/common/accessibility_messages.h" |
| 8 #include "content/common/frame_messages.h" | 8 #include "content/common/frame_messages.h" |
| 9 #include "content/common/input_messages.h" | 9 #include "content/common/input_messages.h" |
| 10 #include "content/common/view_messages.h" | 10 #include "content/common/view_messages.h" |
| (...skipping 18 matching lines...) Expand all Loading... | |
| 29 case ViewHostMsg_ShouldClose_ACK::ID: | 29 case ViewHostMsg_ShouldClose_ACK::ID: |
| 30 case ViewHostMsg_SwapOut_ACK::ID: | 30 case ViewHostMsg_SwapOut_ACK::ID: |
| 31 case ViewHostMsg_ClosePage_ACK::ID: | 31 case ViewHostMsg_ClosePage_ACK::ID: |
| 32 case ViewHostMsg_DomOperationResponse::ID: | 32 case ViewHostMsg_DomOperationResponse::ID: |
| 33 case ViewHostMsg_SwapCompositorFrame::ID: | 33 case ViewHostMsg_SwapCompositorFrame::ID: |
| 34 case ViewHostMsg_UpdateIsDelayed::ID: | 34 case ViewHostMsg_UpdateIsDelayed::ID: |
| 35 case ViewHostMsg_DidActivateAcceleratedCompositing::ID: | 35 case ViewHostMsg_DidActivateAcceleratedCompositing::ID: |
| 36 // Allow cross-process JavaScript calls. | 36 // Allow cross-process JavaScript calls. |
| 37 case ViewHostMsg_RouteCloseEvent::ID: | 37 case ViewHostMsg_RouteCloseEvent::ID: |
| 38 case ViewHostMsg_RouteMessageEvent::ID: | 38 case ViewHostMsg_RouteMessageEvent::ID: |
| 39 // Handled by RenderFrame. | |
|
nasko
2013/12/20 22:59:51
nit: Technically it is RenderFrameHost, isn't it?
Charlie Reis
2013/12/21 00:26:42
Yes, that's right. Fixed for the others above as
| |
| 40 case FrameHostMsg_SwapOut_ACK::ID: | |
| 39 // Frame detach must occur after the RenderView has swapped out. | 41 // Frame detach must occur after the RenderView has swapped out. |
| 40 case FrameHostMsg_Detach::ID: | 42 case FrameHostMsg_Detach::ID: |
| 41 return true; | 43 return true; |
| 42 default: | 44 default: |
| 43 break; | 45 break; |
| 44 } | 46 } |
| 45 | 47 |
| 46 // Check with the embedder as well. | 48 // Check with the embedder as well. |
| 47 ContentClient* client = GetContentClient(); | 49 ContentClient* client = GetContentClient(); |
| 48 return client->CanSendWhileSwappedOut(msg); | 50 return client->CanSendWhileSwappedOut(msg); |
| (...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 88 default: | 90 default: |
| 89 break; | 91 break; |
| 90 } | 92 } |
| 91 | 93 |
| 92 // Check with the embedder as well. | 94 // Check with the embedder as well. |
| 93 ContentClient* client = GetContentClient(); | 95 ContentClient* client = GetContentClient(); |
| 94 return client->CanHandleWhileSwappedOut(msg); | 96 return client->CanHandleWhileSwappedOut(msg); |
| 95 } | 97 } |
| 96 | 98 |
| 97 } // namespace content | 99 } // namespace content |
| OLD | NEW |
