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

tests/compiler/dart2js/serialization/model_test_helper.dart

Index: tests/compiler/dart2js/serialization/model_test_helper.dart
diff --git a/tests/compiler/dart2js/serialization/model_test_helper.dart b/tests/compiler/dart2js/serialization/model_test_helper.dart
index ec6f85e14b7cc9146fed26a245c74286a704412b..007e90ac6dc156c1e86fc7fa942c11da04bcf394 100644
--- a/tests/compiler/dart2js/serialization/model_test_helper.dart
+++ b/tests/compiler/dart2js/serialization/model_test_helper.dart
@@ -264,8 +264,6 @@ bool areClosureScopesEquivalent(ClosureScope a, ClosureScope b) {
       '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;
 }