Skip to content
This repository has been archived by the owner on Mar 13, 2018. It is now read-only.

utlise proxy on systems where Proxy is supported #115

Open
capaj opened this issue Nov 6, 2015 · 1 comment
Open

utlise proxy on systems where Proxy is supported #115

capaj opened this issue Nov 6, 2015 · 1 comment

Comments

@capaj
Copy link
Contributor

capaj commented Nov 6, 2015

As recent development caused O.o strawman proposal to be withdrawn, I think it should be only sensible that observe-js would shift from using native O.o to utilising it's own omplementation of O.o in raw JS utilising proxy- something very similar to https://github.com/tvcutsem/harmony-reflect/blob/master/examples/observer.js

This way, dirty checking is really needed only for older browsers without O.o and without proxies.
Anybody feels up for making this PR? I will probably have time to do this in January, not sooner.

@capaj capaj changed the title utlise proxy on systems where utlise proxy on systems where Proxy is supported Nov 16, 2015
@capaj
Copy link
Contributor Author

capaj commented Nov 16, 2015

I got a little time to play around with observe-js repo and it seems like I can't even run the tests after cloning- I get:

grunt test
>> Tasks directory "../tools/tasks" not found.
Warning: Task "override-chrome-launcher" not found. Use --force to continue.

Aborted due to warnings.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant