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

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

Issue 1777233002: Top document isolation mode prototype (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Doghouse SiteInstance, with bugs Created 4 years, 9 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_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 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
118 virtual bool IsRelatedSiteInstance(const SiteInstance* instance) = 0; 118 virtual bool IsRelatedSiteInstance(const SiteInstance* instance) = 0;
119 119
120 // Returns the total active WebContents count for this SiteInstance and all 120 // Returns the total active WebContents count for this SiteInstance and all
121 // related SiteInstances in the same BrowsingInstance. 121 // related SiteInstances in the same BrowsingInstance.
122 virtual size_t GetRelatedActiveContentsCount() = 0; 122 virtual size_t GetRelatedActiveContentsCount() = 0;
123 123
124 // Returns true if this SiteInstance is for a site that requires a dedicated 124 // Returns true if this SiteInstance is for a site that requires a dedicated
125 // process. This only returns true under the "site per process" process model. 125 // process. This only returns true under the "site per process" process model.
126 virtual bool RequiresDedicatedProcess() = 0; 126 virtual bool RequiresDedicatedProcess() = 0;
127 127
128 // TODO(xiaochengh): Say something...
129 virtual bool IsForDoghouse() const = 0;
130
128 // Factory method to create a new SiteInstance. This will create a new 131 // Factory method to create a new SiteInstance. This will create a new
129 // new BrowsingInstance, so it should only be used when creating a new tab 132 // new BrowsingInstance, so it should only be used when creating a new tab
130 // from scratch (or similar circumstances). Callers should ensure that 133 // from scratch (or similar circumstances). Callers should ensure that
131 // this SiteInstance becomes ref counted, by storing it in a scoped_refptr. 134 // this SiteInstance becomes ref counted, by storing it in a scoped_refptr.
132 // 135 //
133 // The render process host factory may be nullptr. See SiteInstance 136 // The render process host factory may be nullptr. See SiteInstance
134 // constructor. 137 // constructor.
135 // 138 //
136 // TODO(creis): This may be an argument to build a pass_refptr<T> class, as 139 // TODO(creis): This may be an argument to build a pass_refptr<T> class, as
137 // Darin suggests. 140 // Darin suggests.
(...skipping 26 matching lines...) Expand all
164 protected: 167 protected:
165 friend class base::RefCounted<SiteInstance>; 168 friend class base::RefCounted<SiteInstance>;
166 169
167 SiteInstance() {} 170 SiteInstance() {}
168 virtual ~SiteInstance() {} 171 virtual ~SiteInstance() {}
169 }; 172 };
170 173
171 } // namespace content. 174 } // namespace content.
172 175
173 #endif // CONTENT_PUBLIC_BROWSER_SITE_INSTANCE_H_ 176 #endif // CONTENT_PUBLIC_BROWSER_SITE_INSTANCE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698