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

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

Issue 2437253002: Improve TDI render process naming in the task manager. (Closed)
Patch Set: ncarter 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_SITE_INSTANCE_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_SITE_INSTANCE_H_
6 #define CONTENT_PUBLIC_BROWSER_SITE_INSTANCE_H_ 6 #define CONTENT_PUBLIC_BROWSER_SITE_INSTANCE_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 virtual bool IsRelatedSiteInstance(const SiteInstance* instance) = 0; 123 virtual bool IsRelatedSiteInstance(const SiteInstance* instance) = 0;
124 124
125 // Returns the total active WebContents count for this SiteInstance and all 125 // Returns the total active WebContents count for this SiteInstance and all
126 // related SiteInstances in the same BrowsingInstance. 126 // related SiteInstances in the same BrowsingInstance.
127 virtual size_t GetRelatedActiveContentsCount() = 0; 127 virtual size_t GetRelatedActiveContentsCount() = 0;
128 128
129 // Returns true if this SiteInstance is for a site that requires a dedicated 129 // Returns true if this SiteInstance is for a site that requires a dedicated
130 // process. This only returns true under the "site per process" process model. 130 // process. This only returns true under the "site per process" process model.
131 virtual bool RequiresDedicatedProcess() = 0; 131 virtual bool RequiresDedicatedProcess() = 0;
132 132
133 // Returns true if this SiteInstance is used as the default SiteInstance for
134 // cross-site subframes. This only returns true if "top document isolation" is
135 // used.
136 virtual bool IsDefaultSubframeSiteInstance() const = 0;
137
133 // Factory method to create a new SiteInstance. This will create a new 138 // Factory method to create a new SiteInstance. This will create a new
134 // new BrowsingInstance, so it should only be used when creating a new tab 139 // new BrowsingInstance, so it should only be used when creating a new tab
135 // from scratch (or similar circumstances). 140 // from scratch (or similar circumstances).
136 // 141 //
137 // The render process host factory may be nullptr. See SiteInstance 142 // The render process host factory may be nullptr. See SiteInstance
138 // constructor. 143 // constructor.
139 static scoped_refptr<SiteInstance> Create( 144 static scoped_refptr<SiteInstance> Create(
140 content::BrowserContext* browser_context); 145 content::BrowserContext* browser_context);
141 146
142 // Factory method to get the appropriate SiteInstance for the given URL, in 147 // Factory method to get the appropriate SiteInstance for the given URL, in
(...skipping 24 matching lines...) Expand all
167 protected: 172 protected:
168 friend class base::RefCounted<SiteInstance>; 173 friend class base::RefCounted<SiteInstance>;
169 174
170 SiteInstance() {} 175 SiteInstance() {}
171 virtual ~SiteInstance() {} 176 virtual ~SiteInstance() {}
172 }; 177 };
173 178
174 } // namespace content. 179 } // namespace content.
175 180
176 #endif // CONTENT_PUBLIC_BROWSER_SITE_INSTANCE_H_ 181 #endif // CONTENT_PUBLIC_BROWSER_SITE_INSTANCE_H_
OLDNEW
« no previous file with comments | « content/browser/site_instance_impl_unittest.cc ('k') | content/test/content_browser_test_utils_internal.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698