Reload: Have dangling extracted properties raise NoSuchMethod.

Reload: Have dangling extracted properties raise NoSuchMethod.

Closes #26995

R=johnmccutchan@google.com

Committed: https://github.com/dart-lang/sdk/commit/5f4a685cd7fe55d4d15e7e37eec905d999d43e08

[rmacnak, 2016-08-03 23:08:33]

Description was changed from

==========
.


.

BUG=
==========

to

==========
Reload: Have dangling extracted properties raise NoSuchMethod.

Closes #26995
==========

[rmacnak, 2016-08-03 23:08:57]

Description was changed from

==========
Reload: Have dangling extracted properties raise NoSuchMethod.

Closes #26995
==========

to

==========
Reload: Have dangling extracted properties raise NoSuchMethod.

Closes #26995
==========

[rmacnak, 2016-08-04 00:03:42]

rmacnak@google.com changed reviewers:
+ johnmccutchan@google.com

[Cutch, 2016-08-04 00:38:05]

LGTM with a question

https://codereview.chromium.org/2204593011/diff/60001/runtime/vm/parser.cc
File runtime/vm/parser.cc (right):

https://codereview.chromium.org/2204593011/diff/60001/runtime/vm/parser.cc#ne...
runtime/vm/parser.cc:1555: &Symbols::Value(),
why drop the field type here?

[rmacnak, 2016-08-04 16:19:31]

https://chromiumcodereview.appspot.com/2204593011/diff/60001/runtime/vm/parse...
File runtime/vm/parser.cc (right):

https://chromiumcodereview.appspot.com/2204593011/diff/60001/runtime/vm/parse...
runtime/vm/parser.cc:1555: &Symbols::Value(),
On 2016/08/04 00:38:05, Cutch wrote:
> why drop the field type here?

The lookup can fail if the target no longer exists. It's not needed because it
is redundant with the parameter check in the target.

[Cutch, 2016-08-04 16:23:07]

LGTM

[rmacnak, 2016-08-04 16:54:45]

Description was changed from

==========
Reload: Have dangling extracted properties raise NoSuchMethod.

Closes #26995
==========

to

==========
Reload: Have dangling extracted properties raise NoSuchMethod.

Closes #26995

R=johnmccutchan@google.com

Committed:
https://github.com/dart-lang/sdk/commit/5f4a685cd7fe55d4d15e7e37eec905d999d43e08
==========

[rmacnak, 2016-08-04 16:54:50]

Committed patchset #4 (id:60001) manually as
5f4a685cd7fe55d4d15e7e37eec905d999d43e08 (presubmit successful).