Bug 1610193 followup - update test for fixed behavior a=pascalc
authorSteve Fink <sfink@mozilla.com>
Tue, 11 Feb 2020 20:08:29 +0200
changeset 575721 997fc6647f1e2966d1d457cde1bef0d42c22806b
parent 575720 54b5788b78a9820d1fbebcd25a437433f51da0aa
child 575722 64dab3403abc272111d36fb8e9f7c37d6058ea05
push id12699
push useropoprus@mozilla.com
push dateFri, 14 Feb 2020 00:45:20 +0000
treeherdermozilla-beta@997fc6647f1e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerspascalc
bugs1610193
milestone74.0
Bug 1610193 followup - update test for fixed behavior a=pascalc Differential Revision: https://phabricator.services.mozilla.com/D62469
js/src/jit-test/tests/gc/weak-marking-03.js
--- a/js/src/jit-test/tests/gc/weak-marking-03.js
+++ b/js/src/jit-test/tests/gc/weak-marking-03.js
@@ -547,19 +547,19 @@ function blackDuringGray() {
   assertEq(marks[1], "unmarked", "key is not marked yet");
   assertEq(marks[2], "black", "delegate is black");
   assertEq(marks[3], "unmarked", "values is not marked yet");
 
   finishgc();
   showmarks();
   marks = getMarks();
   assertEq(marks[0], "gray", "map is gray");
-  assertEq(marks[1], "black", "key inherits black even though in gray marking");
+  assertEq(marks[1], "gray", "delegate marks key but map is gray, so key gets gray");
   assertEq(marks[2], "black", "delegate is still black");
-  assertEq(marks[3], "gray", "gray map + black key => gray value");
+  assertEq(marks[3], "gray", "gray map + gray key => gray value");
 
   clearMarkQueue();
   clearMarkObservers();
   grayRoot().length = 0;
   g.eval("grayRoot().length = 0");
 }
 
 if (this.enqueueMark)