Implementation for 'analysis.getHover'.

Implementation for 'analysis.getHover'.

R=brianwilkerson@google.com
BUG=

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

[Brian Wilkerson, 2014-07-02 13:58:06]

LGTM, but there is *way* too much churn. Also, I thought Dan was working on this
support.

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

https://codereview.chromium.org/362893004/diff/1/pkg/analysis_server/lib/src/...
pkg/analysis_server/lib/src/analysis_server.dart:90: final Map<Folder,
AnalysisContext> folderMap = new HashMap<Folder,
You probably have no control, but I liked the old formatting better.

https://codereview.chromium.org/362893004/diff/1/pkg/analysis_server/lib/src/...
pkg/analysis_server/lib/src/analysis_server.dart:130: PackageMapProvider
packageMapProvider, this.index, {this.rethrowExceptions:
I liked the old formatting better here too.

https://codereview.chromium.org/362893004/diff/1/pkg/analysis_server/test/ana...
File pkg/analysis_server/test/analysis_hover_test.dart (right):

https://codereview.chromium.org/362893004/diff/1/pkg/analysis_server/test/ana...
pkg/analysis_server/test/analysis_hover_test.dart:68: expect(hover.dartDoc,
'''/// doc aaa\n/// doc bbb''');
I think I need to update the spec. because I think I wrote that the comment
delimiters were removed, but that appears to not be the case.

[scheglov, 2014-07-02 17:33:10]

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

https://codereview.chromium.org/362893004/diff/1/pkg/analysis_server/lib/src/...
pkg/analysis_server/lib/src/analysis_server.dart:90: final Map<Folder,
AnalysisContext> folderMap = new HashMap<Folder,
On 2014/07/02 13:58:06, Brian Wilkerson wrote:
> You probably have no control, but I liked the old formatting better.

Yes, we need to improve formatter.
Rolled back for now.

https://codereview.chromium.org/362893004/diff/1/pkg/analysis_server/test/ana...
File pkg/analysis_server/test/analysis_hover_test.dart (right):

https://codereview.chromium.org/362893004/diff/1/pkg/analysis_server/test/ana...
pkg/analysis_server/test/analysis_hover_test.dart:68: expect(hover.dartDoc,
'''/// doc aaa\n/// doc bbb''');
On 2014/07/02 13:58:06, Brian Wilkerson wrote:
> I think I need to update the spec. because I think I wrote that the comment
> delimiters were removed, but that appears to not be the case.

No need to change the spec.
I will change the implementation for conform the spec.

[scheglov, 2014-07-02 17:33:28]

Committed patchset #2 manually as r37952 (presubmit successful).