tree 497c1022b107963a5a1513b99dfb9e2be92c07ef
parent 74b767777c302677b03c3be09d9c5f9af8d6095c
author Kevin Lubick <kjlubick@google.com> 1546629379 -0500
committer Kevin Lubick <kjlubick@google.com> 1546631299 +0000

[canvaskit] Add .ready() which is a real promise

Prior to this, CanvasKit was unintentionally a thenable, which means if
it was used to resolve a Promise, certain logic would be called which
would put it into an infinite loop. By adding a .ready() which returns
a proper Promise and removes the .then(), this should make the CanvasKit
libary safer.

For now, the .then() still exists unless .ready() is called. .then will
be removed unconditionally in 0.4.0 which will be out after all known
clients (e.g. jsfiddle.skia.org) are changed.

See: https://github.com/kripken/emscripten/issues/5820

Bug: skia:
Change-Id: Ie4093f6b0ce03070ef737941693b06dfff93f61c
Reviewed-on: https://skia-review.googlesource.com/c/181177
Reviewed-by: Joe Gregorio <jcgregorio@google.com>