Remove some RenderProcessHost::IsGuest call sites

chrome/browser/devtools/devtools_targets_ui.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/devtools/devtools_targets_ui.h"
 
 #include "base/memory/weak_ptr.h"
 #include "base/stl_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/values.h"
@@ skipping 145 matching lines @@
       continue;
 
     DevToolsTargetImpl* target_ptr = target.get();
     targets_[target_ptr->GetId()] = target.release();
     base::DictionaryValue* descriptor = Serialize(*target_ptr);
 
     // TODO (kaznacheev): GetMainFrame() call is a temporary hack.
     // Revisit this when multiple OOP frames are supported.
     RenderFrameHost* rfh = rvh->GetMainFrame();
     rfh_to_descriptor[rfh] = descriptor;
-    if (rvh->GetProcess()->IsGuest() || rfh->IsCrossProcessSubframe()) {
+    content::WebContents* web_contents =
+        content::WebContents::FromRenderViewHost(rvh);
+    if (!!GuestViewBase::FromWebContents(web_contents) ||

[lazyboy, 2014/05/23 15:20:13]

As noted offline, add a more obvious bool GuestViewBase::IsGuest(web_contents)
and use that instead.

[Fady Samuel, 2014/05/23 15:35:59]

Done.

+        rfh->IsCrossProcessSubframe()) {
       nested_frames.push_back(rfh);
     } else {
       list_value->Append(descriptor);
     }
   }
 
   // Add the list of nested targets to each of its owners.
   for (std::vector<RenderFrameHost*>::iterator it(nested_frames.begin());
        it != nested_frames.end(); ++it) {
     RenderFrameHost* rfh = (*it);
@@ skipping 422 matching lines @@
           base::StringPrintf("%d", it->first), it->second);
     }
 
     std::string device_id = base::StringPrintf(
         kAdbDeviceIdFormat,
         sit->first.c_str());
     result.Set(device_id, device_status_dict);
   }
   callback_.Run(result);
 }