Hopefully the last bit of restructuring between closture classes and loop boxing, etc.

tests/compiler/dart2js/serialization/model_test_helper.dart

 // Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library dart2js.serialization_model_test;
 
 import 'dart:async';
 import 'package:async_helper/async_helper.dart';
 import 'package:expect/expect.dart';
 import 'package:compiler/src/closure.dart';
@@ skipping 246 matching lines @@
   if (a == null || b == null) return false;
   if (!areLocalsEquivalent(a.boxElement, b.boxElement)) {
     return false;
   }
   checkMaps(
       a.capturedVariables,
       b.capturedVariables,
       'ClosureScope.capturedVariables',
       areLocalsEquivalent,
       areElementsEquivalent);
-  checkSets(a.boxedLoopVariables, b.boxedLoopVariables,

[Siggi Cherem (dart-lang), 2017/06/26 16:54:42]

should we replace this somehow?

One option would be to create 2 maps locally and then compare them:

var aBoxed = {};
a.forEachBoxedVariable((k, v) => aBoxed[k] = v);
var bBoxed = {};
b.forEachBoxedVariable((k, v) => bBoxed[k] = v);

checkMaps(aBoxed, bBoxed, ...)

[Emily Fortuna, 2017/06/26 21:57:20]

Done.

-      'ClosureScope.boxedLoopVariables', areElementsEquivalent);
   return true;
 }
 
 String nodeToString(Node node) {
   String text = '$node';
   if (text.length > 40) {
     return '(${node.runtimeType}) ${text.substring(0, 37)}...';
   }
   return '(${node.runtimeType}) $text';
 }
@@ skipping 13 matching lines @@
   check(outputUnit1, outputUnit2, 'OutputUnit.isMainOutput $message',
       outputUnit1.isMainOutput, outputUnit2.isMainOutput);
   checkSetEquivalence(
       outputUnit1,
       outputUnit2,
       'OutputUnit.imports $message',
       outputUnit1.imports,
       outputUnit2.imports,
       (a, b) => areElementsEquivalent(a.declaration, b.declaration));
 }