Intent to implement and ship: URLSearchParams.sort()

[Takeshi Yoshino]

Contact emails

tyos...@chromium.org

Spec

https://url.spec.whatwg.org/#dom-urlsearchparams-sort

TAG review: Skipped. Small change.

Summary / Motivation

Allows URLSearchParams to sort the queries in it easily. It's useful for getting higher cache hit rate as discussed in https://github.com/whatwg/url/issues/26.

Interoperability and Compatibility risk

Firefox:

Implemented and already shipped in Firefox 54

https://bugzilla.mozilla.org/show_bug.cgi?id=1331864

WebKit:

Implemented

https://bugs.webkit.org/show_bug.cgi?id=167154

Will this feature be supported on all six Blink platforms?

Yes

Demo link

See the WPT file below.

Debuggability

N/A

Is this feature fully tested by web-platform-tests?

Yes, by url/urlsearchparams-sort.html

Ongoing technical constraints

None

OWP launch tracking bug

As it's a small change (just 1 method), there's only a normal bug.

https://bugs.chromium.org/p/chromium/issues/detail?id=682143

Link to entry on the feature dashboard

https://www.chromestatus.com/feature/5632900720492544

Requesting approval to ship?

Yes

[Rick Byers]

LGTM1

--
You received this message because you are subscribed to the Google Groups "blink-dev" group.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAH9hSJYcgsqw0MtH7mQL%2B_mK83wpjDQL_JCmZv%2Bn8N07ee2giQ%40mail.gmail.com.

[Mike West]

LGTM2

To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFUtAY-865zxRMVE80_MRt0LhezngwZvUd0giu74MQgc88H6wg%40mail.gmail.com.

--

-mike

[TAMURA, Kent]

LGTM3.

To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAKXHy%3DeziheU4PcoGavAK74zVEgQEFg2qYpt1vAhawvnatYheg%40mail.gmail.com.

--

TAMURA Kent
Software Engineer, Google

[Takeshi Yoshino]

Thanks for reviewing. With 3 LGTMs, I'm shipping it at https://chromium-review.googlesource.com/c/515243/.