507930 - Workers: Implement WorkerLocation

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Ben Turner (not reading bugmail, use the needinfo flag!)]

This was added to the spec and we should get it in.

Comment 1

[Ben Turner (not reading bugmail, use the needinfo flag!)]

http://www.whatwg.org/specs/web-workers/current-work/#workerlocation

Comment 2

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Attachment: Patch, v1

Comment 3

[Johnny Stenback (:jst)]

Comment on attachment 392325 [details] [diff] [review]
Patch, v1

+interface nsIWorkerLocation : nsISupports
+{
+  readonly attribute DOMString href;
...

Since the objects that implement this interface are created eagerly per worker, and since the code that does that need to convert from UTF8 to UTF16, you could change this interface to use AUTF8String instead of DOMString and save a bit of memory and push the cost of conversion to the users of this (of which there will be pretty few I'd argue).

r+sr=jst either way.

Comment 4

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Pushed changeset b9cee22bad2e to mozilla-central.