This comment was originally written by @seaneagan

To me it would make more sense in "dart:html", no ?

At least right now, dart:html isn't available on the standalone VM and HTML escaping would definitely be useful there too. Ideally, I would be able to use dart:html on the server too, but that's a much bigger feature request. :)

It definitely belongs in some HTML-or-related library.
The dart:html library is really a document library, more than an HTML library, so it's bound to the browser.
I think we need an environment agnostic helper library for such things. Maybe it should also include HTTPStatus, which currently live in dart:io but is also useful in a browser.

Set owner to @lrhn.

This comment was originally written by jjinux...@google.com

What's the story with this, and how does it relate to:

http://code.google.com/searchframe#TDGadvYaD94/trunk/dart/pkg/htmlescape/lib/htmlescape.dart&q=htmlescape%20package:dart%5C.googlecode%5C.com&ct=rc&cd=1&sq=

I noticed that that library is still using pre-M1 syntax.

This comment was originally written by jjinux...@google.com

Can we up the priority of this? It seems like a critical function. I have several projects that need this functionality, and I've had to copy the same code into each of them.

Removed Priority-Medium label.
Added Priority-High label.

htmlescape went away as a separate library, because a single-function library doesn't make sense.

I agree with Lasse that what we really need is an agnostic library that can be used in all environments, rather than just dropping this into corelib.

This comment was originally written by jjinux...@google.com

I remember when this was added to Dojo. Some guy came along and added escape functions for HTML, JavaScript, etc.

This comment was originally written by jjinux...@google.com

Can we please fix this? Properly escaping HTML is really, really important, and not everyone uses Web UI. I don't see any packages for this on pub.

This isn't going to go into corelib. There's nothing stopping you from pushing this onto pub ;)

This comment was originally written by jjinux...@google.com

This seems like a really critical piece of functionality. For some reason, I just have the feeling that someone more important than me should push the package ;)

Removed Type-Defect label.
Added Type-Enhancement label.

There are 3 copies of this in pub packages (and I just copied one for my own project).

Would be good to DRY these up.

pkg/unittest/lib/html_enhanced_config.dart
pkg/unittest/lib/html_config.dart
pkg/intl/lib/bidi_formatter.dart

This feels like it should be part of dart:convert (or similar) together with all the other converters.

Added this to the M6 milestone.

Sure. Just somewhere shareable between client and server code.

Removed Priority-High label.
Added Priority-Medium label.

Removed the owner.
Removed this from the M6 milestone.
Added Ready-to-implement, Accepted labels.

CL started https://codereview.chromium.org/23492002/

r27211

Removed Ready-to-implement label.
Added Fixed label.