Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3412)

Unified Diff: chrome/renderer/extensions/render_view_observer_natives.cc

Issue 235943018: Move extensions bindings code out of //chrome (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: nit Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/extensions/render_view_observer_natives.cc
diff --git a/chrome/renderer/extensions/render_view_observer_natives.cc b/chrome/renderer/extensions/render_view_observer_natives.cc
deleted file mode 100644
index f13f3a2a273e3a4dde5e70408b7eb4e8ba0c619a..0000000000000000000000000000000000000000
--- a/chrome/renderer/extensions/render_view_observer_natives.cc
+++ /dev/null
@@ -1,80 +0,0 @@
-// 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/renderer/extensions/render_view_observer_natives.h"
-
-#include "chrome/renderer/extensions/dispatcher.h"
-#include "content/public/renderer/render_view.h"
-#include "content/public/renderer/render_view_observer.h"
-#include "extensions/common/extension_api.h"
-#include "extensions/renderer/script_context.h"
-#include "third_party/WebKit/public/web/WebFrame.h"
-#include "third_party/WebKit/public/web/WebScopedMicrotaskSuppression.h"
-
-namespace extensions {
-
-namespace {
-
-// Deletes itself when done.
-class LoadWatcher : public content::RenderViewObserver {
- public:
- LoadWatcher(ScriptContext* context,
- content::RenderView* view,
- v8::Handle<v8::Function> cb)
- : content::RenderViewObserver(view), context_(context), callback_(cb) {}
-
- virtual void DidCreateDocumentElement(blink::WebLocalFrame* frame) OVERRIDE {
- CallbackAndDie(true);
- }
-
- virtual void DidFailProvisionalLoad(
- blink::WebLocalFrame* frame,
- const blink::WebURLError& error) OVERRIDE {
- CallbackAndDie(false);
- }
-
- private:
- void CallbackAndDie(bool succeeded) {
- v8::Isolate* isolate = context_->isolate();
- v8::HandleScope handle_scope(isolate);
- v8::Handle<v8::Value> args[] = { v8::Boolean::New(isolate, succeeded) };
- context_->CallFunction(callback_.NewHandle(isolate), 1, args);
- delete this;
- }
-
- ScriptContext* context_;
- ScopedPersistent<v8::Function> callback_;
- DISALLOW_COPY_AND_ASSIGN(LoadWatcher);
-};
-} // namespace
-
-
-RenderViewObserverNatives::RenderViewObserverNatives(Dispatcher* dispatcher,
- ChromeV8Context* context)
- : ChromeV8Extension(dispatcher, context) {
- RouteFunction("OnDocumentElementCreated",
- base::Bind(&RenderViewObserverNatives::OnDocumentElementCreated,
- base::Unretained(this)));
-}
-
-void RenderViewObserverNatives::OnDocumentElementCreated(
- const v8::FunctionCallbackInfo<v8::Value>& args) {
- CHECK(args.Length() == 2);
- CHECK(args[0]->IsInt32());
- CHECK(args[1]->IsFunction());
-
- int view_id = args[0]->Int32Value();
-
- content::RenderView* view = content::RenderView::FromRoutingID(view_id);
- if (!view) {
- LOG(WARNING) << "No render view found to register LoadWatcher.";
- return;
- }
-
- new LoadWatcher(context(), view, args[1].As<v8::Function>());
-
- args.GetReturnValue().Set(true);
-}
-
-} // namespace extensions
« no previous file with comments | « chrome/renderer/extensions/render_view_observer_natives.h ('k') | chrome/renderer/extensions/send_request_natives.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698