Don't set DynamicElementImpl as staticElement for unresolved types.

Don't set DynamicElementImpl as staticElement for unresolved types.

R=brianwilkerson@google.com
BUG=

Committed: https://github.com/dart-lang/sdk/commit/22b9e79090e6ee6faa56a3a4bcf4131e46aaa17f

[Brian Wilkerson, 2016-02-16 19:32:19]

LGTM

https://codereview.chromium.org/1702713003/diff/1/pkg/analyzer/lib/src/genera...
File pkg/analyzer/lib/src/generated/resolver.dart (right):

https://codereview.chromium.org/1702713003/diff/1/pkg/analyzer/lib/src/genera...
pkg/analyzer/lib/src/generated/resolver.dart:12137: void _setElement(Identifier
typeName, Element element) {
It's kind of ugly (and probably ought to be documented) that this method has a
side-effect of setting the element associated with the prefix if the typeName is
a prefixed identifier.

[scheglov, 2016-02-16 19:56:13]

https://codereview.chromium.org/1702713003/diff/1/pkg/analyzer/lib/src/genera...
File pkg/analyzer/lib/src/generated/resolver.dart (right):

https://codereview.chromium.org/1702713003/diff/1/pkg/analyzer/lib/src/genera...
pkg/analyzer/lib/src/generated/resolver.dart:12137: void _setElement(Identifier
typeName, Element element) {
On 2016/02/16 19:32:18, Brian Wilkerson wrote:
> It's kind of ugly (and probably ought to be documented) that this method has a
> side-effect of setting the element associated with the prefix if the typeName
is
> a prefixed identifier.

Done.

[scheglov, 2016-02-16 19:58:19]

Description was changed from

==========
Don't set DynamicElementImpl as staticElement for unresolved types.

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

to

==========
Don't set DynamicElementImpl as staticElement for unresolved types.

R=brianwilkerson@google.com
BUG=

Committed:
https://github.com/dart-lang/sdk/commit/22b9e79090e6ee6faa56a3a4bcf4131e46aaa17f
==========

[scheglov, 2016-02-16 19:58:21]

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