Guard against unresolved code (issue 26545)

Guard against unresolved code (issue 26545)

R=jwren@google.com

Committed: https://github.com/dart-lang/sdk/commit/6b04e74aa05768c8d11fd302baf8a486db31d53d

[Brian Wilkerson, 2016-05-26 20:09:17]

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

[Brian Wilkerson, 2016-05-26 20:09:17]

Is it reasonable for the static type to be null at this point? If so, then I
think this fixes one of the reported bugs. If not, then it only mask it and we
ought to look for the root cause.

[jwren, 2016-05-26 22:26:22]

On 2016/05/26 20:09:17, Brian Wilkerson wrote:
> Is it reasonable for the static type to be null at this point? If so, then I
> think this fixes one of the reported bugs. If not, then it only mask it and we
> ought to look for the root cause.

IsExpression has the same structure, so we should cover that case as well.

[Brian Wilkerson, 2016-05-27 16:13:42]

Done. But you didn't answer the question: is this a reasonable fix?

[jwren, 2016-05-27 16:57:24]

On 2016/05/27 16:13:42, Brian Wilkerson wrote:
> Done. But you didn't answer the question: is this a reasonable fix?

Content and discussion here: https://github.com/dart-lang/sdk/issues/26545

[jwren, 2016-05-27 18:02:41]

On 2016/05/27 16:57:24, jwren wrote:
> On 2016/05/27 16:13:42, Brian Wilkerson wrote:
> > Done. But you didn't answer the question: is this a reasonable fix?
> 
> Content and discussion here: https://github.com/dart-lang/sdk/issues/26545

LGTM

[Brian Wilkerson, 2016-05-27 19:14:08]

Description was changed from

==========
Guard against unresolved code (issue 26545)
==========

to

==========
Guard against unresolved code (issue 26545)

R=jwren@google.com

Committed:
https://github.com/dart-lang/sdk/commit/6b04e74aa05768c8d11fd302baf8a486db31d53d
==========

[Brian Wilkerson, 2016-05-27 19:14:09]

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