<html>

    <head>

      <base href="https://bugs.webkit.org/" />

    </head>

    <body>

      <p>

        <div>

            <b><a class="bz_bug_link 

          bz_status_NEW "

   title="NEW - Improve binding of JSBuiltinConstructor classes"

   href="https://bugs.webkit.org/show_bug.cgi?id=149556#c2">Comment # 2</a>

              on <a class="bz_bug_link 

          bz_status_NEW "

   title="NEW - Improve binding of JSBuiltinConstructor classes"

   href="https://bugs.webkit.org/show_bug.cgi?id=149556">bug 149556</a>

              from <span class="vcard"><a class="email" href="mailto:youennf&#64;gmail.com" title="youenn fablet &lt;youennf&#64;gmail.com&gt;"> <span class="fn">youenn fablet</span></a>

</span></b>

        <pre>(In reply to <a href="show_bug.cgi?id=149556#c1">comment #1</a>)

<span class="quote">&gt; &gt; It requires bypassing JSDOMGlobalObject wrapper cache, which is another

&gt; &gt; efficiency bonus.

&gt; 

&gt; My proposal above does not require bypassing the wrapper cache. The

&gt; singleton wrapper will simply be found in the cache (many times over).</span >

I may be missing something here.

My understanding is that, if using a singleton, toJS(XX::create()) will always return the same JSXX object. This is not what we want, we want a new object each time. 

Hence why I think we should bypass the cache and remove the generation of the toJS(XX) function. Once done, the singleton class might be made private to the JS class or applicable to all classes.

<span class="quote">&gt; We may not need an object at all if we are changing things like the wrapper

&gt; class. I think we should create a JavaScript object that doesn’t wrap a DOM

&gt; object at all. And I’m not even sure it’s all that challenging to do that. I

&gt; might take a crack at it.</span >

I agree, that is the idea behind removing toJS, releaseImpl and so on...

I haven't done the exercise so cannot comment on how much complexity it will add to the binding generator.

You are probably right about the limited complexity it will need.

But the binding generator seems already a bit messy to me.

Do not hesitate to move that forward directly.</pre>

        </div>

      </p>

      <hr>

      <span>You are receiving this mail because:</span>

      <ul>

          <li>You are the assignee for the bug.</li>

      </ul>

    </body>

</html>