Issue 23694. Include 'docSummary' and 'docComplete' into CompletionSuggestion.

Issue 23694. Include 'docSummary' and 'docComplete' into CompletionSuggestion.

R=brianwilkerson@google.com
BUG= https://github.com/dart-lang/sdk/issues/23694

Committed: https://github.com/dart-lang/sdk/commit/1af7b154af904cf0627708818eb15e9fb5066f8a

[Brian Wilkerson, 2015-10-01 22:04:34]

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

[Brian Wilkerson, 2015-10-01 22:04:34]

LGTM. Do we know what kind of impact this has on the performance of code
completions?

[scheglov, 2015-10-01 22:24:29]

On 2015/10/01 22:04:34, Brian Wilkerson wrote:
> LGTM. Do we know what kind of impact this has on the performance of code
> completions?

For the first time when we complete top-level elements, such as "Str^" it takes
about 500-600 ms vs. 300 ms without documentation.
For the second and subsequent completions difference is not so high, but still
exists.

For prefixed completions difference is pretty high - 15 ms vs. 3 ms for
stringLiteral.^ completion.
I.e. documentation makes it 5 times more expensive.

I'm not sure that we want to take such performance hit.

[Brian Wilkerson, 2015-10-01 23:48:43]

> I'm not sure that we want to take such performance hit.

I agree. Let's talk about it tomorrow.

[scheglov, 2015-10-04 23:16:51]

Committed patchset #2 (id:20001) manually as
1af7b154af904cf0627708818eb15e9fb5066f8a (presubmit successful).