<html>
    <head>
      <base href="https://bugs.webkit.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - [SOUP] Simplify cookie storage handling"
   href="https://bugs.webkit.org/show_bug.cgi?id=166967#c11">Comment # 11</a>
              on <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - [SOUP] Simplify cookie storage handling"
   href="https://bugs.webkit.org/show_bug.cgi?id=166967">bug 166967</a>
              from <span class="vcard"><a class="email" href="mailto:mcatanzaro&#64;igalia.com" title="Michael Catanzaro &lt;mcatanzaro&#64;igalia.com&gt;"> <span class="fn">Michael Catanzaro</span></a>
</span></b>
        <pre>(In reply to <a href="show_bug.cgi?id=166967#c9">comment #9</a>)
<span class="quote">&gt; &gt; It's true that for every
&gt; &gt; session, if the cookie storage is asked before the soup session is created,
&gt; &gt; when the soup session is created we have two references of the cookie jar,
&gt; &gt; we could probably clear the NetworkStorageSession one in that case. That can
&gt; &gt; only happen for the default session, all others are created with a
&gt; &gt; SoupNetworkSession.
&gt; 
&gt; OK, I think you should indeed clear the NetworkStorageSession jar and have
&gt; it return the SoupNetworkSession jar instead. Even if it's the same thing,
&gt; that would make the code clearer and more robust.</span >

Um, I see the current code already returns the SoupNetworkSession's pointer to the jar. I think it should also clear the NetworkStorageSession's pointer to the jar, for clarity.</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>