Add EquivalenceVisitor to tools in js_ast

Add EquivalenceVisitor to tools in js_ast

R=sra@google.com

Committed: https://github.com/dart-lang/sdk/commit/09c58282049acca9a411b068360ca733ada7300f

[Johnni Winther, 2017-07-03 12:16:21]

johnniwinther@google.com changed reviewers:
+ sra@google.com

[sra1, 2017-07-05 21:18:33]

lgtm

https://codereview.chromium.org/2967843002/diff/1/pkg/js_ast/lib/js_ast.dart
File pkg/js_ast/lib/js_ast.dart (right):

https://codereview.chromium.org/2967843002/diff/1/pkg/js_ast/lib/js_ast.dart#...
pkg/js_ast/lib/js_ast.dart:15: part 'src/tools.dart';
perhaps call this something like equivalence_visitor.dart

If we add more 'tools', they would be better off in a separate file rather than
mixed in with a 500 line class.

[Johnni Winther, 2017-07-06 08:39:34]

Description was changed from

==========
Add EquivalenceVisitor to tools in js_ast
==========

to

==========
Add EquivalenceVisitor to tools in js_ast

R=sra@google.com

Committed:
https://github.com/dart-lang/sdk/commit/09c58282049acca9a411b068360ca733ada7300f
==========

[Johnni Winther, 2017-07-06 08:39:35]

Committed patchset #2 (id:20001) manually as
09c58282049acca9a411b068360ca733ada7300f (presubmit successful).

[Johnni Winther, 2017-07-06 08:39:39]

https://codereview.chromium.org/2967843002/diff/1/pkg/js_ast/lib/js_ast.dart
File pkg/js_ast/lib/js_ast.dart (right):

https://codereview.chromium.org/2967843002/diff/1/pkg/js_ast/lib/js_ast.dart#...
pkg/js_ast/lib/js_ast.dart:15: part 'src/tools.dart';
On 2017/07/05 21:18:33, sra1 wrote:
> perhaps call this something like equivalence_visitor.dart
> 
> If we add more 'tools', they would be better off in a separate file rather
than
> mixed in with a 500 line class.

Done.