Add an integration test for analysis.highlights.

Add an integration test for analysis.highlights.

Also fix sort order in the API definition of HighlightRegionType.

R=brianwilkerson@google.com

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

[Paul Berry, 2014-08-11 21:11:19]

Brian--

I wasn't able to figure out how to get all of the possible highlight types to
occur (see TODO comments in highlights_test.dart).  Is this due to bugs in the
highlight generation, or was I simply not creative enough in creating my test
code?

Paul

[scheglov, 2014-08-12 02:28:47]

https://codereview.chromium.org/454843004/ will fix most of these TODO problems.

https://codereview.chromium.org/463643002/diff/1/pkg/analysis_server/test/int...
File pkg/analysis_server/test/integration/analysis/highlights_test.dart (right):

https://codereview.chromium.org/463643002/diff/1/pkg/analysis_server/test/int...
pkg/analysis_server/test/integration/analysis/highlights_test.dart:98: //
TODO(paulberry): When is the 'DIRECTIVE' type used?
It's a bug in the highlights computer.

https://codereview.chromium.org/463643002/diff/1/pkg/analysis_server/test/int...
pkg/analysis_server/test/integration/analysis/highlights_test.dart:101: //
TODO(paulberry): Why is "override" annotated as FIELD_STATIC?
It's a bug in the highlights computer.

https://codereview.chromium.org/463643002/diff/1/pkg/analysis_server/test/int...
pkg/analysis_server/test/integration/analysis/highlights_test.dart:107: //
TODO(paulberry): When is the 'IDENTIFIER_DEFAULT' type used?
For unresolved identifiers: aaa, bbb, CCC.

main() {
  aaa = 42;
  bbb(84);
  CCC ccc;
}

https://codereview.chromium.org/463643002/diff/1/pkg/analysis_server/test/int...
pkg/analysis_server/test/integration/analysis/highlights_test.dart:109: //
TODO(paulberry): When is the 'KEYWORD' type used?
It's a bug in the highlights computer.

https://codereview.chromium.org/463643002/diff/1/pkg/analysis_server/test/int...
pkg/analysis_server/test/integration/analysis/highlights_test.dart:114: //
TODO(paulberry): When is the 'LITERAL_MAP' type used?
These are bugs in the highlights computer.

[Brian Wilkerson, 2014-08-12 13:29:14]

LGTM

https://codereview.chromium.org/463643002/diff/1/pkg/analysis_server/tool/spe...
File pkg/analysis_server/tool/spec/spec_input.html (right):

https://codereview.chromium.org/463643002/diff/1/pkg/analysis_server/tool/spe...
pkg/analysis_server/tool/spec/spec_input.html:8: <h1
style="color:#999999">WORKING DRAFT - Version <version>0.3</version></h1>
Do we need to update the version with each change?

[Paul Berry, 2014-08-12 17:40:02]

https://codereview.chromium.org/463643002/diff/1/pkg/analysis_server/tool/spe...
File pkg/analysis_server/tool/spec/spec_input.html (right):

https://codereview.chromium.org/463643002/diff/1/pkg/analysis_server/tool/spe...
pkg/analysis_server/tool/spec/spec_input.html:8: <h1
style="color:#999999">WORKING DRAFT - Version <version>0.3</version></h1>
On 2014/08/12 13:29:14, Brian Wilkerson wrote:
> Do we need to update the version with each change?

Given that:

- the last version of the analysis server that we distributed (through pub) was
really early in development and didn't support a lot of this API

- in the long term we're leaning towards distributing the analysis server with
the SDK, but we haven't set up that distribution mechanism yet

I don't think it matters very much at the moment.  I think all that matters is
that we rev this version number when we distribute the next version of the
analysis server.

I'm open to other arguments, though.

[Paul Berry, 2014-08-12 18:16:42]

Committed patchset #1 manually as r39152 (presubmit successful).