Title: [211514] branches/safari-603-branch/Source/_javascript_Core
- Revision
- 211514
- Author
- matthew_han...@apple.com
- Date
- 2017-02-01 13:26:25 -0800 (Wed, 01 Feb 2017)
Log Message
Merge r211300. rdar://problem/30135571
Modified Paths
Diff
Modified: branches/safari-603-branch/Source/_javascript_Core/API/JSVirtualMachine.mm (211513 => 211514)
--- branches/safari-603-branch/Source/_javascript_Core/API/JSVirtualMachine.mm 2017-02-01 21:26:23 UTC (rev 211513)
+++ branches/safari-603-branch/Source/_javascript_Core/API/JSVirtualMachine.mm 2017-02-01 21:26:25 UTC (rev 211514)
@@ -326,6 +326,8 @@
}
[externalRememberedSet removeAllObjects];
}
+
+ visitor.mergeIfNecessary();
}
#endif // JSC_OBJC_API_ENABLED
Modified: branches/safari-603-branch/Source/_javascript_Core/ChangeLog (211513 => 211514)
--- branches/safari-603-branch/Source/_javascript_Core/ChangeLog 2017-02-01 21:26:23 UTC (rev 211513)
+++ branches/safari-603-branch/Source/_javascript_Core/ChangeLog 2017-02-01 21:26:25 UTC (rev 211514)
@@ -1,3 +1,25 @@
+2017-01-31 Matthew Hanson <matthew_han...@apple.com>
+
+ Merge r211300. rdar://problem/30135571
+
+ 2017-01-27 Filip Pizlo <fpi...@apple.com>
+
+ scanExternalRememberedSet needs to mergeIfNecessary
+ https://bugs.webkit.org/show_bug.cgi?id=167523
+
+ Reviewed by Keith Miller.
+
+ The protocol for opaque roots is that if you add to them outside of draining, then you need to call
+ mergeIfNecessary.
+
+ This means that every MarkingConstraint that adds opaque roots needs to mergeIfNecessary after.
+
+ scanExternalRememberedSet transitively calls addOpaqueRoot, is called from a MarkingConstraint, and
+ was missing a call to mergeIfNecessary. This fixes it.
+
+ * API/JSVirtualMachine.mm:
+ (scanExternalRememberedSet):
+
2017-01-27 Matthew Hanson <matthew_han...@apple.com>
Merge r211194. rdar://problem/30201008
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes