fix invocation completion with trailing stmt

fix invocation completion with trailing stmt

BUG=dartbug.com/22365
R=scheglov@google.com

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

[danrubel, 2015-02-27 16:41:02]

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

[scheglov, 2015-02-27 17:58:07]

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

[scheglov, 2015-02-27 17:59:14]

I'm not aware if there was a change to assign propagated types in these cases.
If it works, LGTM.

https://codereview.chromium.org/965753003/diff/1/pkg/analysis_server/lib/src/...
File pkg/analysis_server/lib/src/services/completion/invocation_computer.dart
(right):

https://codereview.chromium.org/965753003/diff/1/pkg/analysis_server/lib/src/...
pkg/analysis_server/lib/src/services/completion/invocation_computer.dart:149:
finished = true;
BTW, it might be slightly more efficient to throw an exception and stop visiting
the whole sub-tree.

[danrubel, 2015-02-27 18:42:00]

https://codereview.chromium.org/965753003/diff/1/pkg/analysis_server/lib/src/...
File pkg/analysis_server/lib/src/services/completion/invocation_computer.dart
(right):

https://codereview.chromium.org/965753003/diff/1/pkg/analysis_server/lib/src/...
pkg/analysis_server/lib/src/services/completion/invocation_computer.dart:149:
finished = true;
On 2015/02/27 17:59:13, scheglov wrote:
> BTW, it might be slightly more efficient to throw an exception and stop
visiting
> the whole sub-tree.

Good point. I'll look at that in a subsequent CL.

[danrubel, 2015-02-27 18:42:51]

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

[danrubel, 2015-02-27 21:05:09]

https://codereview.chromium.org/965753003/diff/1/pkg/analysis_server/lib/src/...
File pkg/analysis_server/lib/src/services/completion/invocation_computer.dart
(right):

https://codereview.chromium.org/965753003/diff/1/pkg/analysis_server/lib/src/...
pkg/analysis_server/lib/src/services/completion/invocation_computer.dart:149:
finished = true;
On 2015/02/27 18:42:00, danrubel wrote:
> On 2015/02/27 17:59:13, scheglov wrote:
> > BTW, it might be slightly more efficient to throw an exception and stop
> visiting
> > the whole sub-tree.
> 
> Good point. I'll look at that in a subsequent CL.

https://codereview.chromium.org/967643002/