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

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

Issue 2394343002: Removing ShouldSwapProcessesForRedirect (using DoesSiteRequireDedicatedProcess).
Patch Set: Rebasing... 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 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 275 matching lines...) Expand 10 before | Expand all | Expand 10 after
286 // Returns true if for the navigation from |current_url| to |new_url| 286 // Returns true if for the navigation from |current_url| to |new_url|
287 // in |site_instance|, a new SiteInstance and BrowsingInstance should be 287 // in |site_instance|, a new SiteInstance and BrowsingInstance should be
288 // created (even if we are in a process model that doesn't usually swap.) 288 // created (even if we are in a process model that doesn't usually swap.)
289 // This forces a process swap and severs script connections with existing 289 // This forces a process swap and severs script connections with existing
290 // tabs. 290 // tabs.
291 virtual bool ShouldSwapBrowsingInstancesForNavigation( 291 virtual bool ShouldSwapBrowsingInstancesForNavigation(
292 SiteInstance* site_instance, 292 SiteInstance* site_instance,
293 const GURL& current_url, 293 const GURL& current_url,
294 const GURL& new_url); 294 const GURL& new_url);
295 295
296 // Returns true if the given navigation redirect should cause a renderer
297 // process swap.
298 virtual bool ShouldSwapProcessesForRedirect(BrowserContext* browser_context,
299 const GURL& current_url,
300 const GURL& new_url);
301
302 // Returns true if the passed in URL should be assigned as the site of the 296 // Returns true if the passed in URL should be assigned as the site of the
303 // current SiteInstance, if it does not yet have a site. 297 // current SiteInstance, if it does not yet have a site.
304 virtual bool ShouldAssignSiteForURL(const GURL& url); 298 virtual bool ShouldAssignSiteForURL(const GURL& url);
305 299
306 // Allows the embedder to pass extra command line flags. 300 // Allows the embedder to pass extra command line flags.
307 // switches::kProcessType will already be set at this point. 301 // switches::kProcessType will already be set at this point.
308 virtual void AppendExtraCommandLineSwitches(base::CommandLine* command_line, 302 virtual void AppendExtraCommandLineSwitches(base::CommandLine* command_line,
309 int child_process_id) {} 303 int child_process_id) {}
310 304
311 // Returns the locale used by the application. 305 // Returns the locale used by the application.
(...skipping 520 matching lines...) Expand 10 before | Expand all | Expand 10 after
832 virtual bool ShouldRedirectDOMStorageTaskRunner(); 826 virtual bool ShouldRedirectDOMStorageTaskRunner();
833 827
834 // If this returns true, all BrowserThreads (but UI/IO) that support it on 828 // If this returns true, all BrowserThreads (but UI/IO) that support it on
835 // this platform will experimentally be redirected to TaskScheduler. 829 // this platform will experimentally be redirected to TaskScheduler.
836 virtual bool RedirectNonUINonIOBrowserThreadsToTaskScheduler(); 830 virtual bool RedirectNonUINonIOBrowserThreadsToTaskScheduler();
837 }; 831 };
838 832
839 } // namespace content 833 } // namespace content
840 834
841 #endif // CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_ 835 #endif // CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
OLDNEW
« no previous file with comments | « content/browser/service_worker/service_worker_browsertest.cc ('k') | content/public/browser/content_browser_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698