Re: [heycam/webidl] Modernize invoking user code (#113)

> -                    <li>Set <var>R</var> to the result of invoking the <span class='prop'>[[Call]]</span> method of <var>F</var>, providing the <a class='dfnref' href='#dfn-callback-this-value'>callback this value</a> as the <span class='esvalue'>this</span> value and <var>values</var> as the argument values.</li>
> -                  </ol>
> -                  And then, whether or not an exception was thrown:
> -                  <ol>
> -                    <li>Pop <var>script</var> off the <a class='dfnref external' href='http://www.whatwg.org/specs/web-apps/current-work/multipage/webappapis.html'>stack of incumbent scripts</a>.</li>
> -                    <li>If an exception was thrown, end these steps, and allow it to propagate.</li>
> -                  </ol>
> -                </li>
> -                </ol>
> -                </li>
> -                <li>If the callback function’s return type is <span class='idltype'>void</span>, return.</li>
> -                <li>
> -                  Return the result of <a class='dfnref' href='#dfn-convert-ecmascript-to-idl-value'>converting</a>
> -                  <var>R</var> to an IDL value of the same type as the callback function’s return type.
> -                </li>
> +                <li>If the callback function's return type is <span class='idltype'>void</span>,

This was here before, but it might use some explanation in a note...

The only way to end up with a non-callable here is to be dealing with a callback function which came from an attribute setter marked `[TreatNonObjectAsNull]`.  And in that case we just want to no-op the function call.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/heycam/webidl/pull/113/files/6a54acae5c1ff4199b5bf85b61bc4681addbafb7#r63775592