tree 2cc263edc125980dc4b40797083a7609f2a2f98f
parent 0541640e3793cf97a97437075b49e0228c446c80
author Sigmund Cherem <sigmund@google.com> 1549997623 +0000
committer commit-bot@chromium.org <commit-bot@chromium.org> 1549997623 +0000

Fix Js-interop factory calls which broke when eliding optional parameters.

The issue derives from the fact that `parameterStructure` has only the preserved parameters,
however the SSA was emitting values for all arguments.

I fixed this by emitting all parameters. I also noticed that the parameter order should
probably be the native-ordering here as well (it doesn't matter for the factory, but it may
matter for other static native calls, if any)

Bug: https://github.com/dart-lang/sdk/issues/35916
Change-Id: I2e93c7f8e3baf5c9285107d23786bb3bf4f3a511
Reviewed-on: https://dart-review.googlesource.com/c/92626
Commit-Queue: Sigmund Cherem <sigmund@google.com>
Reviewed-by: Stephen Adams <sra@google.com>