Use UTF-8 when serving web pages for analysis server debug.

Use UTF-8 when serving web pages for analysis server debug.

This avoids the need to encode unicode characters using hexadecimal
escape sequences.

(Note that setting the charset using response.headers.contentType
affects the encoding used by both client and server.)

R=brianwilkerson@google.com

Committed: https://code.google.com/p/dart/source/detail?r=43045

[Paul Berry, 2015-01-21 00:14:43]

paulberry@google.com changed reviewers:
+ brianwilkerson@google.com

[Brian Wilkerson, 2015-01-21 00:27:09]

LGTM

https://codereview.chromium.org/855643004/diff/1/pkg/analysis_server/lib/src/...
File pkg/analysis_server/lib/src/get_handler.dart (right):

https://codereview.chromium.org/855643004/diff/1/pkg/analysis_server/lib/src/...
pkg/analysis_server/lib/src/get_handler.dart:356: new ContentType("text",
"html", charset: "utf-8");
Can we use a static const?

[Paul Berry, 2015-01-21 17:57:37]

https://codereview.chromium.org/855643004/diff/1/pkg/analysis_server/lib/src/...
File pkg/analysis_server/lib/src/get_handler.dart (right):

https://codereview.chromium.org/855643004/diff/1/pkg/analysis_server/lib/src/...
pkg/analysis_server/lib/src/get_handler.dart:356: new ContentType("text",
"html", charset: "utf-8");
On 2015/01/21 00:27:09, Brian Wilkerson wrote:
> Can we use a static const?

It looks like the ContentType constructor is non-const, so no.  But I like the
suggestion.  I've used a static final.

[Paul Berry, 2015-01-21 17:58:22]

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