Remove BrowserPlugin's -internal-attach method

extensions/renderer/script_context.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/script_context.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "base/values.h"
 #include "content/public/common/url_constants.h"
+#include "content/public/renderer/render_frame.h"
 #include "content/public/renderer/render_view.h"
 #include "content/public/renderer/v8_value_converter.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/extension_api.h"
 #include "extensions/common/extension_urls.h"
 #include "extensions/common/features/base_feature_provider.h"
 #include "gin/per_context_data.h"
 #include "third_party/WebKit/public/web/WebDataSource.h"
 #include "third_party/WebKit/public/web/WebDocument.h"
 #include "third_party/WebKit/public/web/WebFrame.h"
@@ skipping 43 matching lines @@
   return extension_.get() ? extension_->id() : base::EmptyString();
 }
 
 content::RenderView* ScriptContext::GetRenderView() const {
   if (web_frame_ && web_frame_->view())
     return content::RenderView::FromWebView(web_frame_->view());
   else
     return NULL;
 }
 
+content::RenderFrame* ScriptContext::GetRenderFrame() const {
+  if (web_frame_)
+    return content::RenderFrame::FromWebFrame(web_frame_);
+  else
+    return NULL;

[not at google - send to devlin, 2014/08/13 23:40:16]

No else after return. Mind fixing line 74 as well?

[Fady Samuel, 2014/08/14 19:41:18]

Done.

+}
+
 v8::Local<v8::Value> ScriptContext::CallFunction(
     v8::Handle<v8::Function> function,
     int argc,
     v8::Handle<v8::Value> argv[]) const {
   v8::EscapableHandleScope handle_scope(isolate());
   v8::Context::Scope scope(v8_context());
 
   blink::WebScopedMicrotaskSuppression suppression;
   if (!is_valid()) {
     return handle_scope.Escape(
@@ skipping 152 matching lines @@
                                           v8::Handle<v8::Value> argv[]) {
   return CallFunction(function, argc, argv);
 }
 
 gin::ContextHolder* ScriptContext::GetContextHolder() {
   v8::HandleScope handle_scope(isolate());
   return gin::PerContextData::From(v8_context())->context_holder();
 }
 
 }  // namespace extensions