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

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

Issue 397103006: Revert 283572 "Changes to RenderFrameProxy:" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 5 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
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 9
10 #include "base/auto_reset.h" 10 #include "base/auto_reset.h"
(...skipping 925 matching lines...) Expand 10 before | Expand all | Expand 10 after
936 if (!is_swapped_out_ || !render_view_->is_swapped_out_) { 936 if (!is_swapped_out_ || !render_view_->is_swapped_out_) {
937 // Swap this RenderFrame out so the frame can navigate to a page rendered by 937 // Swap this RenderFrame out so the frame can navigate to a page rendered by
938 // a different process. This involves running the unload handler and 938 // a different process. This involves running the unload handler and
939 // clearing the page. Once WasSwappedOut is called, we also allow this 939 // clearing the page. Once WasSwappedOut is called, we also allow this
940 // process to exit if there are no other active RenderFrames in it. 940 // process to exit if there are no other active RenderFrames in it.
941 941
942 // Send an UpdateState message before we get swapped out. Create the 942 // Send an UpdateState message before we get swapped out. Create the
943 // RenderFrameProxy as well so its routing id is registered for receiving 943 // RenderFrameProxy as well so its routing id is registered for receiving
944 // IPC messages. 944 // IPC messages.
945 render_view_->SyncNavigationState(); 945 render_view_->SyncNavigationState();
946 proxy = RenderFrameProxy::CreateProxyToReplaceFrame(this, 946 proxy = RenderFrameProxy::CreateFrameProxy(proxy_routing_id, routing_id_);
947 proxy_routing_id);
948 947
949 // Synchronously run the unload handler before sending the ACK. 948 // Synchronously run the unload handler before sending the ACK.
950 // TODO(creis): Call dispatchUnloadEvent unconditionally here to support 949 // TODO(creis): Call dispatchUnloadEvent unconditionally here to support
951 // unload on subframes as well. 950 // unload on subframes as well.
952 if (!frame_->parent()) 951 if (!frame_->parent())
953 frame_->dispatchUnloadEvent(); 952 frame_->dispatchUnloadEvent();
954 953
955 // Swap out and stop sending any IPC messages that are not ACKs. 954 // Swap out and stop sending any IPC messages that are not ACKs.
956 if (!frame_->parent()) 955 if (!frame_->parent())
957 render_view_->SetSwappedOut(true); 956 render_view_->SetSwappedOut(true);
(...skipping 2603 matching lines...) Expand 10 before | Expand all | Expand 10 after
3561 3560
3562 #if defined(ENABLE_BROWSER_CDMS) 3561 #if defined(ENABLE_BROWSER_CDMS)
3563 RendererCdmManager* RenderFrameImpl::GetCdmManager() { 3562 RendererCdmManager* RenderFrameImpl::GetCdmManager() {
3564 if (!cdm_manager_) 3563 if (!cdm_manager_)
3565 cdm_manager_ = new RendererCdmManager(this); 3564 cdm_manager_ = new RendererCdmManager(this);
3566 return cdm_manager_; 3565 return cdm_manager_;
3567 } 3566 }
3568 #endif // defined(ENABLE_BROWSER_CDMS) 3567 #endif // defined(ENABLE_BROWSER_CDMS)
3569 3568
3570 } // namespace content 3569 } // namespace content
OLDNEW
« no previous file with comments | « trunk/src/content/renderer/child_frame_compositing_helper.cc ('k') | trunk/src/content/renderer/render_frame_proxy.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698