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

Side by Side Diff: content/browser/frame_host/render_frame_host_manager.h

Issue 616133002: Make RenderFrame(Host) own a RenderWidget(Host). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Test update, RWH lifetime fix Created 5 years, 11 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
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 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
6 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
7 7
8 #include <list> 8 #include <list>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 451 matching lines...) Expand 10 before | Expand all | Expand 10 after
462 // |pending_render_frame_host_| while respecting the opener route if needed 462 // |pending_render_frame_host_| while respecting the opener route if needed
463 // and stores it in pending_render_frame_host_. 463 // and stores it in pending_render_frame_host_.
464 void CreatePendingRenderFrameHost(SiteInstance* old_instance, 464 void CreatePendingRenderFrameHost(SiteInstance* old_instance,
465 SiteInstance* new_instance, 465 SiteInstance* new_instance,
466 bool is_main_frame); 466 bool is_main_frame);
467 467
468 // Ensure that we have created RFHs for the new RFH's opener chain if 468 // Ensure that we have created RFHs for the new RFH's opener chain if
469 // we are staying in the same BrowsingInstance. This allows the new RFH 469 // we are staying in the same BrowsingInstance. This allows the new RFH
470 // to send cross-process script calls to its opener(s). Returns the opener 470 // to send cross-process script calls to its opener(s). Returns the opener
471 // route ID to be used for the new RenderView to be created. 471 // route ID to be used for the new RenderView to be created.
472 // |create_render_frame_flags| allows the method to set additional flags.
472 int CreateOpenerRenderViewsIfNeeded(SiteInstance* old_instance, 473 int CreateOpenerRenderViewsIfNeeded(SiteInstance* old_instance,
473 SiteInstance* new_instance); 474 SiteInstance* new_instance,
475 int* create_render_frame_flags);
474 476
475 // Creates a RenderFrameHost and corresponding RenderViewHost if necessary. 477 // Creates a RenderFrameHost and corresponding RenderViewHost if necessary.
476 scoped_ptr<RenderFrameHostImpl> CreateRenderFrameHost(SiteInstance* instance, 478 scoped_ptr<RenderFrameHostImpl> CreateRenderFrameHost(SiteInstance* instance,
477 int view_routing_id, 479 int view_routing_id,
478 int frame_routing_id, 480 int frame_routing_id,
479 int flags); 481 int flags);
480 482
481 // Sets up the necessary state for a new RenderViewHost with the given opener, 483 // Sets up the necessary state for a new RenderViewHost with the given opener,
482 // if necessary. It creates a RenderFrameProxy in the target renderer process 484 // if necessary. It creates a RenderFrameProxy in the target renderer process
483 // with the given |proxy_routing_id|, which is used to route IPC messages when 485 // with the given |proxy_routing_id|, which is used to route IPC messages when
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
618 NotificationRegistrar registrar_; 620 NotificationRegistrar registrar_;
619 621
620 base::WeakPtrFactory<RenderFrameHostManager> weak_factory_; 622 base::WeakPtrFactory<RenderFrameHostManager> weak_factory_;
621 623
622 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager); 624 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager);
623 }; 625 };
624 626
625 } // namespace content 627 } // namespace content
626 628
627 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_ 629 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698