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

Side by Side Diff: sky/framework/embedder.sky

Issue 875713003: Upgrade Sky's use of the JS bindings (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 years, 10 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 unified diff | Download patch
« no previous file with comments | « no previous file | sky/framework/shell.sky » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <import src="shell.sky" as="shell" /> 1 <import src="/gen/mojo/public/sky/connection.sky" as="connection" />
2 <script> 2 <script>
3 function connectToEmbedderService(service, client) { 3 function connectToEmbedderService(service/*, client*/) {
abarth-chromium 2015/01/28 19:23:17 Can we just delete |client| instead of commenting
hansmuller1 2015/01/28 19:27:05 Oops, sorry about that.
4 var handle = internals.connectToEmbedderService(service.name); 4 var handle = internals.connectToEmbedderService(service.name);
5 return shell.wrapHandle(handle, service, client); 5 return connection.bindHandleToProxy(handle, service);
6 } 6 }
7 7
8 module.exports = { 8 module.exports = {
9 connectToEmbedderService: connectToEmbedderService, 9 connectToEmbedderService: connectToEmbedderService,
10 }; 10 };
11 </script> 11 </script>
OLDNEW
« no previous file with comments | « no previous file | sky/framework/shell.sky » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698