Rename RenderProcessHost::IsGuest to RenderProcessHost::IsIsolatedGuest

content/public/test/mock_render_process_host.cc

Index: content/public/test/mock_render_process_host.cc
diff --git a/content/public/test/mock_render_process_host.cc b/content/public/test/mock_render_process_host.cc
index 50c19ee849d81fe6876e80482f353b4ed4ce8328..58c57180815d253508380ac3e10ea00af3db67ca 100644
--- a/content/public/test/mock_render_process_host.cc
+++ b/content/public/test/mock_render_process_host.cc
@@ -25,8 +25,7 @@ MockRenderProcessHost::MockRenderProcessHost(BrowserContext* browser_context)
       browser_context_(browser_context),
       prev_routing_id_(0),
       fast_shutdown_started_(false),
-      deletion_callback_called_(false),
-      is_guest_(false) {
+      deletion_callback_called_(false) {
   // Child process security operations can't be unit tested unless we add
   // ourselves as an existing child process.
   ChildProcessSecurityPolicyImpl::GetInstance()->Add(GetID());
@@ -101,8 +100,8 @@ int MockRenderProcessHost::VisibleWidgetCount() const {
   return 1;
 }
 
-bool MockRenderProcessHost::IsGuest() const {
-  return is_guest_;
+bool MockRenderProcessHost::IsIsolatedGuest() const {
+  return false;

[nasko, 2014/05/28 16:29:13]

Is it no longer true that content/ on its own can have isolated guests? It seems
to me that we need a test case inside content/ that ensures the isolation is
working without depending on chrome/ for it.

[fsamuel, 2014/05/28 16:37:01]

Yes, it is true. BrowserPlugin is now inert without chrome.

 }
 
 StoragePartition* MockRenderProcessHost::GetStoragePartition() const {