Send extension console messages to the right WebFrame.

extensions/renderer/console.cc

 // Copyright 2014 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 "extensions/renderer/console.h"
 
 #include "base/compiler_specific.h"
 #include "base/debug/alias.h"
 #include "base/lazy_instance.h"
 #include "base/strings/string_util.h"
@@ skipping 21 matching lines @@
     content::RenderView::ForEach(&finder);
     return finder.found_;
   }
 
  private:
   explicit ByContextFinder(v8::Handle<v8::Context> context)
       : context_(context), found_(NULL) {}
 
   bool Visit(content::RenderView* render_view) override {
     ExtensionHelper* helper = ExtensionHelper::Get(render_view);
-    if (helper &&
-        helper->dispatcher()->script_context_set().GetByV8Context(context_)) {
-      found_ = render_view;
+    if (helper) {
+      ScriptContext* script_context =
+          helper->dispatcher()->script_context_set().GetByV8Context(context_);
+      if (script_context &&
+          content::RenderView::FromWebView(
+              script_context->web_frame()->view()) == render_view) {

[not at google - send to devlin, 2014/11/07 16:24:03]

You should just be able to do "script_context->GetRenderView() == render_view".

+        found_ = render_view;
+      }
     }
     return !found_;
   }
 
   v8::Handle<v8::Context> context_;
   content::RenderView* found_;
 
   DISALLOW_COPY_AND_ASSIGN(ByContextFinder);
 };
 
@@ skipping 124 matching lines @@
   v8::Local<v8::Object> console_object = v8::Object::New(isolate);
   BindLogMethod(isolate, console_object, "debug", &Debug);
   BindLogMethod(isolate, console_object, "log", &Log);
   BindLogMethod(isolate, console_object, "warn", &Warn);
   BindLogMethod(isolate, console_object, "error", &Error);
   return handle_scope.Escape(console_object);
 }
 
 }  // namespace console
 }  // namespace extensions