This is essentially a bug fix so no review needed.
The feature changes the behavior of FileReader API so that the result attribute is only set right before the Load event is fired. Motivation: This brings the chromium FileReader implementation in line with the FileAPI spec. The existing implementation was identified to possibly cause memory leaks but that scenario was very rarely seen, as evidenced by this experiment: https://www.chromestatus.com/metrics/feature/timeline/popularity/2158.
Link to “Intent to Prototype” blink-dev discussion
Interoperability and Compatibility
Very minimal risk as the scenario it removes is super rarely used as can be seen from https://www.chromestatus.com/metrics/feature/timeline/popularity/2158 .
Gecko: Shipped/Shipping Already fixed
WebKit: No signal
Web developers: No signals
Will this feature be supported on all six Blink platforms (Windows, Mac, Linux, Chrome OS, Android, and Android WebView)?
Is this feature fully tested by web-platform-tests?
Link to entry on the Chrome Platform Status
You received this message because you are subscribed to the Google Groups "blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+...@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/ed9bd689-29e3-4d04-af9c-0ab041eab02en%40chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw9PVr1M9KX2%2Bka_AMXoz0UW2miwDYRjLncg%3DwqcBZikwQ%40mail.gmail.com.