Include 'invokeMethodRef' reference as a linking dependency.

Include 'invokeMethodRef' reference as a linking dependency.

It was crashing with StackOverflow internally. With changes since the
latest internal release it was not crashing, but still failing with
assertion. This makes it fail with a correct error, without throwing
an exception.


R=paulberry@google.com, brianwilkerson@google.com
BUG=

Committed: https://github.com/dart-lang/sdk/commit/03a0ee09436322e2d6609ba61dc88989faa18d60

[Paul Berry, 2017-04-04 21:31:49]

lgtm

https://codereview.chromium.org/2802473002/diff/1/pkg/analyzer/lib/src/summar...
File pkg/analyzer/lib/src/summary/link.dart (right):

https://codereview.chromium.org/2802473002/diff/1/pkg/analyzer/lib/src/summar...
pkg/analyzer/lib/src/summary/link.dart:4888: // TODO(paulberry): if this
reference refers to a variable, should it
We can get rid of this TODO now.

[scheglov, 2017-04-04 21:36:01]

Description was changed from

==========
Include 'invokeMethodRef' reference as a linking dependency.

It was crashing with StackOverflow internally. With changes since the
latest internal release it was not crashing, but still failing with
assertion. This makes it fail with a correct error, without throwing
an exception.


R=brianwilkerson@google.com, paulberry@google.com
BUG=
==========

to

==========
Include 'invokeMethodRef' reference as a linking dependency.

It was crashing with StackOverflow internally. With changes since the
latest internal release it was not crashing, but still failing with
assertion. This makes it fail with a correct error, without throwing
an exception.


R=paulberry@google.com, brianwilkerson@google.com
BUG=

Committed:
https://github.com/dart-lang/sdk/commit/03a0ee09436322e2d6609ba61dc88989faa18d60
==========

[scheglov, 2017-04-04 21:36:02]

Committed patchset #1 (id:1) manually as
03a0ee09436322e2d6609ba61dc88989faa18d60 (presubmit successful).

[Brian Wilkerson, 2017-04-04 21:37:33]

lgtm