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

Side by Side Diff: content/public/browser/content_browser_client.h

Issue 2499313002: Don't split off NTP subframes with TDI. (Closed)
Patch Set: rebase Created 4 years, 1 month 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
6 #define CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_ 6 #define CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 9
10 #include <map> 10 #include <map>
(...skipping 224 matching lines...) Expand 10 before | Expand all | Expand 10 after
235 // This also applies in cases where the new URL will open in another process. 235 // This also applies in cases where the new URL will open in another process.
236 virtual bool ShouldAllowOpenURL(SiteInstance* site_instance, const GURL& url); 236 virtual bool ShouldAllowOpenURL(SiteInstance* site_instance, const GURL& url);
237 237
238 // Allows the embedder to override parameters when navigating. Called for both 238 // Allows the embedder to override parameters when navigating. Called for both
239 // opening new URLs and when transferring URLs across processes. 239 // opening new URLs and when transferring URLs across processes.
240 virtual void OverrideNavigationParams(SiteInstance* site_instance, 240 virtual void OverrideNavigationParams(SiteInstance* site_instance,
241 ui::PageTransition* transition, 241 ui::PageTransition* transition,
242 bool* is_renderer_initiated, 242 bool* is_renderer_initiated,
243 content::Referrer* referrer) {} 243 content::Referrer* referrer) {}
244 244
245 // Allows the embedder to override top document isolation for specific frames.
246 // |url| is the URL being loaded in the subframe, and |site_instance| is the
247 // SiteInstance of the parent frame. Called only for subframes and only when
248 // top document isolation mode is enabled.
249 virtual bool ShouldFrameShareParentSiteInstanceDespiteTopDocumentIsolation(
250 const GURL& url,
251 SiteInstance* site_instance);
ncarter (slow) 2016/11/15 21:42:35 WDYT about renaming |site_instance| -> |parent_sit
Avi (use Gerrit) 2016/11/16 20:12:11 Done.
252
245 // Returns whether a new view for a given |site_url| can be launched in a 253 // Returns whether a new view for a given |site_url| can be launched in a
246 // given |process_host|. 254 // given |process_host|.
247 virtual bool IsSuitableHost(RenderProcessHost* process_host, 255 virtual bool IsSuitableHost(RenderProcessHost* process_host,
248 const GURL& site_url); 256 const GURL& site_url);
249 257
250 // Returns whether a new view for a new site instance can be added to a 258 // Returns whether a new view for a new site instance can be added to a
251 // given |process_host|. 259 // given |process_host|.
252 virtual bool MayReuseHost(RenderProcessHost* process_host); 260 virtual bool MayReuseHost(RenderProcessHost* process_host);
253 261
254 // Returns whether a new process should be created or an existing one should 262 // Returns whether a new process should be created or an existing one should
(...skipping 516 matching lines...) Expand 10 before | Expand all | Expand 10 after
771 #endif 779 #endif
772 780
773 // Returns an instance of MemoryCoordinatorDelegate. 781 // Returns an instance of MemoryCoordinatorDelegate.
774 virtual std::unique_ptr<MemoryCoordinatorDelegate> 782 virtual std::unique_ptr<MemoryCoordinatorDelegate>
775 GetMemoryCoordinatorDelegate(); 783 GetMemoryCoordinatorDelegate();
776 }; 784 };
777 785
778 } // namespace content 786 } // namespace content
779 787
780 #endif // CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_ 788 #endif // CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698