Browse Source

Fix polluting devtools in some cases.

Cheng Zhao 11 years ago
parent
commit
7d93b4a48f
1 changed files with 4 additions and 3 deletions
  1. 4 3
      renderer/atom_renderer_client.cc

+ 4 - 3
renderer/atom_renderer_client.cc

@@ -155,12 +155,13 @@ bool AtomRendererClient::ShouldFork(WebKit::WebFrame* frame,
 bool AtomRendererClient::IsNodeBindingEnabled(WebKit::WebFrame* frame) {
   if (node_integration_ == DISABLE)
     return false;
+  // Do not pollute devtools.
+  else if (frame != NULL &&
+           GURL(frame->document().url()).SchemeIs(kChromeDevToolsScheme))
+    return false;
   // Node integration is enabled in main frame unless explictly disabled.
   else if (frame == main_frame_)
     return true;
-  // Do not pollute devtools.
-  else if (GURL(frame->document().url()).SchemeIs(kChromeDevToolsScheme))
-    return false;
   else if (node_integration_ == MANUAL_ENABLE_IFRAME &&
            frame != NULL &&
            frame->uniqueName().utf8().find(kEnableNodeIntegration)