Add analysis server API specification and related tools.

Add analysis server API specification and related tools.

The source for the API specification is in
pkg/analysis_server/spec/spec_input.html.  This file uses custom HTML
elements to declare the machine-readable parts of the specification.

The script pkg/analysis_server/spec/generate_files processes that file
and creates two output files:

- pkg/analysis_server/spec/api.html, the human-readable analysis
  server spec.

- pkg/analysis_server/test/integration/protocol_matchers.dart, which
  is used by integration tests to check the data returned by the
  analysis server.

R=brianwilkerson@google.com, jwren@google.com, scheglov@google.com

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

[Brian Wilkerson, 2014-08-06 18:12:06]

The code LGTM, but I think the structure needs to be changed. According to the
package layout conventions
(https://www.dartlang.org/tools/pub/package-layout.html) the specification (at
least the final readable form) ought to be in a 'doc' directory and the tool
used to generate it should be in a 'tool' directory (in a 'spec' subdirectory
might be nice).

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/test/int...
File pkg/analysis_server/test/integration/protocol_matchers.dart (right):

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/test/int...
pkg/analysis_server/test/integration/protocol_matchers.dart:1: // Copyright (c)
2014, the Dart project authors.  Please see the AUTHORS file');
Remove the trailing ');

[jwren, 2014-08-06 18:12:52]

LGTM  This is great!

I might have nits later when I see the look of the html, but those will just be
nits.

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/test/int...
File pkg/analysis_server/test/integration/protocol_matchers.dart (right):

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/test/int...
pkg/analysis_server/test/integration/protocol_matchers.dart:1: // Copyright (c)
2014, the Dart project authors.  Please see the AUTHORS file');
');   ..  typo?  Doesn't look generated

[scheglov, 2014-08-06 18:20:39]

LGTM

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/spec/api...
File pkg/analysis_server/spec/api.dart (right):

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/spec/api...
pkg/analysis_server/spec/api.dart:26: * Dispatch the given [node] to the
visitor.
[node] -> [type]

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/spec/api...
pkg/analysis_server/spec/api.dart:139: * Base class for objects in the API model
.

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/spec/cod...
File pkg/analysis_server/spec/codegen_matchers.dart (right):

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/spec/cod...
pkg/analysis_server/spec/codegen_matchers.dart:8: library CodegenMatchers;
AFAIK library names should be all lower case with dots.

[Paul Berry, 2014-08-06 18:32:22]

On 2014/08/06 18:12:06, Brian Wilkerson wrote:
> The code LGTM, but I think the structure needs to be changed. According to the
> package layout conventions
> (https://www.dartlang.org/tools/pub/package-layout.html) the specification (at
> least the final readable form) ought to be in a 'doc' directory and the tool
> used to generate it should be in a 'tool' directory (in a 'spec' subdirectory
> might be nice).

Done.

[Paul Berry, 2014-08-06 18:32:43]

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/spec/api...
File pkg/analysis_server/spec/api.dart (right):

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/spec/api...
pkg/analysis_server/spec/api.dart:26: * Dispatch the given [node] to the
visitor.
On 2014/08/06 18:20:39, scheglov wrote:
> [node] -> [type]

Done.

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/spec/api...
pkg/analysis_server/spec/api.dart:139: * Base class for objects in the API model
On 2014/08/06 18:20:39, scheglov wrote:
> .

Done.

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/spec/cod...
File pkg/analysis_server/spec/codegen_matchers.dart (right):

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/spec/cod...
pkg/analysis_server/spec/codegen_matchers.dart:8: library CodegenMatchers;
On 2014/08/06 18:20:39, scheglov wrote:
> AFAIK library names should be all lower case with dots.

Done.

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/test/int...
File pkg/analysis_server/test/integration/protocol_matchers.dart (right):

https://codereview.chromium.org/443873006/diff/1/pkg/analysis_server/test/int...
pkg/analysis_server/test/integration/protocol_matchers.dart:1: // Copyright (c)
2014, the Dart project authors.  Please see the AUTHORS file');
On 2014/08/06 18:12:05, Brian Wilkerson wrote:
> Remove the trailing ');

Done.

[Paul Berry, 2014-08-06 18:34:33]

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