Teach XHR how to handle relative urls.

Teach XHR how to handle relative urls.

Hixie says window.URL may not exist yet in the
sky specs, but I'm sure we'll need something like it.

R=ojan@chromium.org, abarth@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/2474837be766176bddae3c17cb7bbf3459f558cf

[ojan, 2014-11-18 01:49:10]

ojan@chromium.org changed reviewers:
+ ojan@chromium.org

[ojan, 2014-11-18 01:49:11]

lgtm

Seems fine. I agree that we'll definitely need some way at getting at your URL
eventually. I was thinking window.location...but it doesn't really matter at the
moment.

[esprehn, 2014-11-18 02:06:24]

esprehn@chromium.org changed reviewers:
+ esprehn@chromium.org

[esprehn, 2014-11-18 02:06:24]

https://codereview.chromium.org/729913003/diff/1/sky/framework/xmlhttprequest...
File sky/framework/xmlhttprequest.sky (right):

https://codereview.chromium.org/729913003/diff/1/sky/framework/xmlhttprequest...
sky/framework/xmlhttprequest.sky:8: var outstandingRequests = new Set();
Why do you need this? I thought the Mojo JS stuff abarth@ and I worked on solved
this.

[abarth-chromium, 2014-11-18 02:11:59]

https://codereview.chromium.org/729913003/diff/1/sky/framework/xmlhttprequest...
File sky/framework/xmlhttprequest.sky (right):

https://codereview.chromium.org/729913003/diff/1/sky/framework/xmlhttprequest...
sky/framework/xmlhttprequest.sky:8: var outstandingRequests = new Set();
On 2014/11/18 at 02:06:24, esprehn wrote:
> Why do you need this? I thought the Mojo JS stuff abarth@ and I worked on
solved this.

It doesn't.  :(

We can make the waiting callbacks strong, which would solve the problem.  (See
https://codereview.chromium.org/734633004/).

Another option is to have the static set at the bindings layer...

[esprehn, 2014-11-18 02:15:49]

https://codereview.chromium.org/729913003/diff/1/sky/framework/xmlhttprequest...
File sky/framework/xmlhttprequest.sky (right):

https://codereview.chromium.org/729913003/diff/1/sky/framework/xmlhttprequest...
sky/framework/xmlhttprequest.sky:8: var outstandingRequests = new Set();
On 2014/11/18 02:11:59, abarth wrote:
> On 2014/11/18 at 02:06:24, esprehn wrote:
> > Why do you need this? I thought the Mojo JS stuff abarth@ and I worked on
> solved this.
> 
> It doesn't.  :(
> 
> We can make the waiting callbacks strong, which would solve the problem.  (See
> https://codereview.chromium.org/734633004/).
> 
> Another option is to have the static set at the bindings layer...

Fixing this with one off Sets seems like a mistake, there's going to be so many
bugs.

[eseidel, 2014-11-18 19:16:00]

Committed patchset #1 (id:1) manually as
2474837be766176bddae3c17cb7bbf3459f558cf (presubmit successful).

[eseidel, 2014-11-18 19:17:01]

On 2014/11/18 02:15:49, esprehn wrote:
>
https://codereview.chromium.org/729913003/diff/1/sky/framework/xmlhttprequest...
> File sky/framework/xmlhttprequest.sky (right):
> 
>
https://codereview.chromium.org/729913003/diff/1/sky/framework/xmlhttprequest...
> sky/framework/xmlhttprequest.sky:8: var outstandingRequests = new Set();
> On 2014/11/18 02:11:59, abarth wrote:
> > On 2014/11/18 at 02:06:24, esprehn wrote:
> > > Why do you need this? I thought the Mojo JS stuff abarth@ and I worked on
> > solved this.
> > 
> > It doesn't.  :(
> > 
> > We can make the waiting callbacks strong, which would solve the problem. 
(See
> > https://codereview.chromium.org/734633004/).
> > 
> > Another option is to have the static set at the bindings layer...
> 
> Fixing this with one off Sets seems like a mistake, there's going to be so
many
> bugs.

We can change them to be strong.  I don't really care.  Landing this for now to
unblock the inspector.