add parameter for WebFrame on context creation

rogerwang · rogerwang · commit 8ce23553624a · 2013-01-28T16:20:23.000+08:00
we want to operate on main frame in the future
diff --git a/src/renderer/shell_content_renderer_client.cc b/src/renderer/shell_content_renderer_client.cc
@@ -144,7 +144,7 @@ void ShellContentRendererClient::DidCreateScriptContext(
   ProxyBypassRules rules;
   rules.ParseFromString(rv->renderer_preferences_.nw_remote_page_rules);
   bool force_on = rules.Matches(url);
-  InstallNodeSymbols(context, url, force_on);
+  InstallNodeSymbols(frame, context, url, force_on);
 }
 
 bool ShellContentRendererClient::WillSetSecurityToken(
@@ -157,6 +157,7 @@ bool ShellContentRendererClient::WillSetSecurityToken(
 }
 
 void ShellContentRendererClient::InstallNodeSymbols(
+    WebKit::WebFrame* frame,
     v8::Handle<v8::Context> context,
     const GURL& url,
     bool force_on) {
diff --git a/src/renderer/shell_content_renderer_client.h b/src/renderer/shell_content_renderer_client.h
@@ -52,7 +52,8 @@ class ShellContentRendererClient : public ContentRendererClient {
   scoped_ptr<ShellRenderProcessObserver> shell_observer_;
   scoped_ptr<api::WindowBindings> window_bindings_;;
 
-  void InstallNodeSymbols(v8::Handle<v8::Context> context, const GURL& url,
+  void InstallNodeSymbols(WebKit::WebFrame* frame,
+                          v8::Handle<v8::Context> context, const GURL& url,
                           bool force_on = false);
 
   // Catch node uncaughtException.

Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,7 @@ void ShellContentRendererClient::DidCreateScriptContext(`
`144`	`144`	`ProxyBypassRules rules;`
`145`	`145`	`rules.ParseFromString(rv->renderer_preferences_.nw_remote_page_rules);`
`146`	`146`	`bool force_on = rules.Matches(url);`
`147`		`- InstallNodeSymbols(context, url, force_on);`
	`147`	`+ InstallNodeSymbols(frame, context, url, force_on);`
`148`	`148`	`}`
`149`	`149`
`150`	`150`	`bool ShellContentRendererClient::WillSetSecurityToken(`
`@@ -157,6 +157,7 @@ bool ShellContentRendererClient::WillSetSecurityToken(`
`157`	`157`	`}`
`158`	`158`
`159`	`159`	`void ShellContentRendererClient::InstallNodeSymbols(`
	`160`	`+ WebKit::WebFrame* frame,`
`160`	`161`	`v8::Handle<v8::Context> context,`
`161`	`162`	`const GURL& url,`
`162`	`163`	`bool force_on) {`