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

Unified Diff: content/renderer/browser_plugin/browser_plugin.cc

Issue 11028019: Browser plugin: Implement loadRedirect event. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix merge conflicts. Created 8 years, 2 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: content/renderer/browser_plugin/browser_plugin.cc
diff --git a/content/renderer/browser_plugin/browser_plugin.cc b/content/renderer/browser_plugin/browser_plugin.cc
index 6e720e0535a1a3923adaa6431871de3feeaa3951..acdea505531f3c687f84dc5f68770e90ef04c081 100644
--- a/content/renderer/browser_plugin/browser_plugin.cc
+++ b/content/renderer/browser_plugin/browser_plugin.cc
@@ -45,8 +45,11 @@ namespace {
const char kCrashEventName[] = "crash";
const char kIsTopLevel[] = "isTopLevel";
const char kLoadAbortEventName[] = "loadAbort";
+const char kLoadRedirectEventName[] = "loadRedirect";
const char kLoadStartEventName[] = "loadStart";
const char kNavigationEventName[] = "navigation";
+const char kNewURL[] = "newUrl";
+const char kOldURL[] = "oldUrl";
const char kPartitionAttribute[] = "partition";
const char kPersistPrefix[] = "persist:";
const char kSrcAttribute[] = "src";
@@ -415,6 +418,40 @@ void BrowserPlugin::LoadAbort(const GURL& url,
}
}
+void BrowserPlugin::LoadRedirect(const GURL& old_url,
+ const GURL& new_url,
+ bool is_top_level) {
+ if (!HasListeners(kLoadRedirectEventName))
+ return;
+
+ EventListeners& listeners = event_listener_map_[kLoadRedirectEventName];
+ EventListeners::iterator it = listeners.begin();
+
+ v8::Context::Scope context_scope(v8::Context::New());
+ v8::HandleScope handle_scope;
+
+ // Construct the loadRedirect event object.
+ v8::Local<v8::Value> event =
+ v8::Local<v8::Object>::New(v8::Object::New());
+ v8::Local<v8::Object>::Cast(event)->Set(
+ v8::String::New(kOldURL, sizeof(kOldURL) - 1),
+ v8::String::New(old_url.spec().c_str(), old_url.spec().size()));
+ v8::Local<v8::Object>::Cast(event)->Set(
+ v8::String::New(kNewURL, sizeof(kNewURL) - 1),
+ v8::String::New(new_url.spec().c_str(), new_url.spec().size()));
+ v8::Local<v8::Object>::Cast(event)->Set(
+ v8::String::New(kIsTopLevel, sizeof(kIsTopLevel) - 1),
+ v8::Boolean::New(is_top_level));
+ for (; it != listeners.end(); ++it) {
+ // Fire the event listener.
+ container()->element().document().frame()->
+ callFunctionEvenIfScriptDisabled(*it,
+ v8::Object::New(),
+ 1,
+ &event);
+ }
+}
+
void BrowserPlugin::AdvanceFocus(bool reverse) {
// We do not have a RenderView when we are testing.
if (render_view_)
« no previous file with comments | « content/renderer/browser_plugin/browser_plugin.h ('k') | content/renderer/browser_plugin/browser_plugin_manager_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698