Remove visitLabelElement() and visitLocalVariableElement().

Remove visitLabelElement() and visitLocalVariableElement().

These elements are not parts of the bigger element model, so it
does not make sense to implement these visit methods.

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

Committed: https://github.com/dart-lang/sdk/commit/2c064df1451568d4b1187b9a54d0ad7cc383612e

[Brian Wilkerson, 2017-07-06 22:42:52]

lgtm

https://codereview.chromium.org/2971253002/diff/1/pkg/analyzer/lib/src/dart/e...
File pkg/analyzer/lib/src/dart/element/element.dart (right):

https://codereview.chromium.org/2971253002/diff/1/pkg/analyzer/lib/src/dart/e...
pkg/analyzer/lib/src/dart/element/element.dart:5564: T
accept<T>(ElementVisitor<T> visitor) => null;
It would be better to throw a not-implemented exception, just to make it
explicit that the visitor won't be called.

[scheglov, 2017-07-07 15:05:00]

Description was changed from

==========
Remove visitLabelElement() and visitLocalVariableElement().

These elements are not parts of the bigger element model, so it
does not make sense to implement these visit methods.

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

to

==========
Remove visitLabelElement() and visitLocalVariableElement().

These elements are not parts of the bigger element model, so it
does not make sense to implement these visit methods.

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

Committed:
https://github.com/dart-lang/sdk/commit/2c064df1451568d4b1187b9a54d0ad7cc383612e
==========

[scheglov, 2017-07-07 15:05:01]

Committed patchset #1 (id:1) manually as
2c064df1451568d4b1187b9a54d0ad7cc383612e (presubmit successful).

[scheglov, 2017-07-07 15:05:33]

https://codereview.chromium.org/2971253002/diff/1/pkg/analyzer/lib/src/dart/e...
File pkg/analyzer/lib/src/dart/element/element.dart (right):

https://codereview.chromium.org/2971253002/diff/1/pkg/analyzer/lib/src/dart/e...
pkg/analyzer/lib/src/dart/element/element.dart:5564: T
accept<T>(ElementVisitor<T> visitor) => null;
On 2017/07/06 22:42:52, Brian Wilkerson wrote:
> It would be better to throw a not-implemented exception, just to make it
> explicit that the visitor won't be called.

Done.