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

Unified Diff: content/renderer/render_frame_impl.cc

Issue 2600163002: SharedWorker: Clarify that SharedWorkerRepository is owned by RenderFrameImpl (Closed)
Patch Set: rebase Created 4 years 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/renderer/render_frame_impl.h ('k') | content/renderer/shared_worker_repository.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 0d01a92eb3fdff12f0849cacbca2bc6b140e74ea..c424e70f7b106477e6c5b59c00aa17763921813e 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -1237,7 +1237,9 @@ void RenderFrameImpl::Initialize() {
#if BUILDFLAG(ENABLE_PLUGINS)
new PepperBrowserConnection(this);
#endif
- new SharedWorkerRepository(this);
+ shared_worker_repository_ = base::MakeUnique<SharedWorkerRepository>(this);
+ GetWebFrame()->setSharedWorkerRepositoryClient(
+ shared_worker_repository_.get());
if (IsLocalRoot()) {
// DevToolsAgent is a RenderFrameObserver, and will destruct itself
« no previous file with comments | « content/renderer/render_frame_impl.h ('k') | content/renderer/shared_worker_repository.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698