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

Unified Diff: chrome/browser/background/background_contents_service.cc

Issue 2882513005: Propagate opener to BackgroundsContents. (Closed)
Patch Set: Tweaked the comment in DriveWebContentsManager::ShouldCreateWebContents Created 3 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/background/background_contents_service.cc
diff --git a/chrome/browser/background/background_contents_service.cc b/chrome/browser/background/background_contents_service.cc
index 5e79600f3863595dab59e8ea4b7ea1d20a174011..f871735f7aa5983965c7a56e7b82b31a29082eb7 100644
--- a/chrome/browser/background/background_contents_service.cc
+++ b/chrome/browser/background/background_contents_service.cc
@@ -719,7 +719,7 @@ void BackgroundContentsService::LoadBackgroundContents(
DVLOG(1) << "Loading background content url: " << url;
BackgroundContents* contents = CreateBackgroundContents(
- SiteInstance::CreateForURL(profile, url), MSG_ROUTING_NONE,
+ SiteInstance::CreateForURL(profile, url), nullptr, MSG_ROUTING_NONE,
MSG_ROUTING_NONE, MSG_ROUTING_NONE, profile, frame_name, application_id,
std::string(), NULL);
@@ -728,6 +728,7 @@ void BackgroundContentsService::LoadBackgroundContents(
BackgroundContents* BackgroundContentsService::CreateBackgroundContents(
scoped_refptr<SiteInstance> site,
+ content::RenderFrameHost* opener,
int32_t routing_id,
int32_t main_frame_route_id,
int32_t main_frame_widget_route_id,
@@ -737,9 +738,9 @@ BackgroundContents* BackgroundContentsService::CreateBackgroundContents(
const std::string& partition_id,
content::SessionStorageNamespace* session_storage_namespace) {
BackgroundContents* contents =
- new BackgroundContents(std::move(site), routing_id, main_frame_route_id,
- main_frame_widget_route_id, this, partition_id,
- session_storage_namespace);
+ new BackgroundContents(std::move(site), opener, routing_id,
+ main_frame_route_id, main_frame_widget_route_id,
+ this, partition_id, session_storage_namespace);
// Register the BackgroundContents internally, then send out a notification
// to external listeners.

Powered by Google App Engine
This is Rietveld 408576698