<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [SOUP] Use libsoup WebSockets API"
   href="https://bugs.webkit.org/show_bug.cgi?id=199151#c7">Comment # 7</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [SOUP] Use libsoup WebSockets API"
   href="https://bugs.webkit.org/show_bug.cgi?id=199151">bug 199151</a>
              from <span class="vcard"><a class="email" href="mailto:cgarcia@igalia.com" title="Carlos Garcia Campos <cgarcia@igalia.com>"> <span class="fn">Carlos Garcia Campos</span></a>
</span></b>
        <pre>(In reply to youenn fablet from <a href="show_bug.cgi?id=199151#c5">comment #5</a>)
<span class="quote">> (In reply to Carlos Garcia Campos from <a href="show_bug.cgi?id=199151#c1">comment #1</a>)
> > Created <span class=""><a href="attachment.cgi?id=372750&action=diff" name="attach_372750" title="WIP">attachment 372750</a> <a href="attachment.cgi?id=372750&action=edit" title="WIP">[details]</a></span>
> > WIP
> > 
> > This is a WIP patch. Old code is used unless WEBKIT_USE_SOUP_WEBSOCKETS env
> > var is present. This is just temporary until the implementation is complete
> > and we make layout tests pass. Most of them are failing with the patch, but
> > in a lot of cases it's because of missing console messages that new code
> > path doesn't generate. I'll add a comment with more information about the
> > layout tests.

> Depending on how you want to do the switch, it might be feasible to
> rename/reuse NSURLSessionWebSocket runtime flag.</span >

We already depend on a lisboup version having the websockets API (although I've fixed several bugs these days while working on it, but mostly edge cases covered by the tests). So, as soon as we have all the basic functionality working we can do the switch unconditionally. I can use platform specific test results until apple switches to use NSURLSession for the tests too. We can probably land this patch using the env var and continue working on the remaining issues in follow up patches.</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>