Clean up code

This commit is contained in:
Cheng Zhao 2015-06-03 14:53:19 +08:00
parent 95a8f3fc70
commit 3a3b05b2f0

View file

@ -31,22 +31,23 @@ namespace atom {
namespace { namespace {
// The default routing id of WebContents, since in Electron a WebContents // The default routing id of WebContents.
// always owns its own RenderProcessHost, this ID is same for every WebContents. // In Electron each RenderProcessHost only has one WebContents, so this ID is
// same for every WebContents.
int kDefaultRoutingID = 2; int kDefaultRoutingID = 2;
// Next navigation should not restart renderer process. // Next navigation should not restart renderer process.
bool g_suppress_renderer_process_restart = false; bool g_suppress_renderer_process_restart = false;
// Find out the owner of the child process according to child_process_id. // Find out the owner of the child process according to |process_id|.
enum ProcessOwner { enum ProcessOwner {
OWNER_NATIVE_WINDOW, OWNER_NATIVE_WINDOW,
OWNER_GUEST_WEB_CONTENTS, OWNER_GUEST_WEB_CONTENTS,
OWNER_NONE, // it might be devtools though. OWNER_NONE, // it might be devtools though.
}; };
ProcessOwner GetProcessOwner(int process_id, ProcessOwner GetProcessOwner(int process_id,
NativeWindow** window, NativeWindow** window,
WebViewManager::WebViewInfo* info) { WebViewManager::WebViewInfo* info) {
auto web_contents = content::WebContents::FromRenderViewHost( auto web_contents = content::WebContents::FromRenderViewHost(
content::RenderViewHost::FromID(process_id, kDefaultRoutingID)); content::RenderViewHost::FromID(process_id, kDefaultRoutingID));
if (!web_contents) if (!web_contents)