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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/public/browser/web_contents.h

Issue 2882513005: Propagate opener to BackgroundsContents. (Closed)
Patch Set: Added (and re-enabled) tests. Suppressed opener propagation in one case. Created 3 years, 7 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
« chrome/browser/page_load_metrics/metrics_web_contents_observer.cc ('K') | « content/browser/web_contents/web_contents_impl.cc ('k') | content/public/browser/web_contents_delegate.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/browser/web_contents.h
diff --git a/content/public/browser/web_contents.h b/content/public/browser/web_contents.h
index 2d788807cbf73cab3dab325e127db1e55237678e..cf1b7f2d609e852e1165af07cde018090da62bc5 100644
--- a/content/public/browser/web_contents.h
+++ b/content/public/browser/web_contents.h
@@ -662,7 +662,7 @@ class WebContents : public PageNavigator,
virtual bool HasOpener() const = 0;
// Returns the opener if HasOpener() is true, or nullptr otherwise.
- virtual WebContents* GetOpener() const = 0;
+ virtual RenderFrameHost* GetOpener() const = 0;
// Returns true if this WebContents was opened by another WebContents, even
// if the opener was suppressed. In contrast to HasOpener/GetOpener, the
@@ -672,7 +672,7 @@ class WebContents : public PageNavigator,
// Returns the original opener if HasOriginalOpener() is true, or nullptr
// otherwise.
- virtual WebContents* GetOriginalOpener() const = 0;
+ virtual RenderFrameHost* GetOriginalOpener() const = 0;
typedef base::Callback<void(
int, /* id */
« chrome/browser/page_load_metrics/metrics_web_contents_observer.cc ('K') | « content/browser/web_contents/web_contents_impl.cc ('k') | content/public/browser/web_contents_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698