<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 - CookieManager only works with the default session"
href="https://bugs.webkit.org/show_bug.cgi?id=168229#c18">Comment # 18</a>
on <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - CookieManager only works with the default session"
href="https://bugs.webkit.org/show_bug.cgi?id=168229">bug 168229</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 <a href="show_bug.cgi?id=168229#c16">comment #16</a>)
<span class="quote">> (In reply to <a href="show_bug.cgi?id=168229#c15">comment #15</a>)
> > (In reply to <a href="show_bug.cgi?id=168229#c11">comment #11</a>)
> > > (In reply to <a href="show_bug.cgi?id=168229#c5">comment #5</a>)
> > > > Comment on <span class="bz_obsolete"><a href="attachment.cgi?id=301344&action=diff" name="attach_301344" title="Patch">attachment 301344</a> <a href="attachment.cgi?id=301344&action=edit" title="Patch">[details]</a></span>
> > > > Patch
> > > >
> > > > I like the direction this code is going in. Definitely fix the Windows
> > > > build. WTF::Functions are not copyable, and that seems preferable in this
> > > > case. Can this be tested?
> > >
> > > I started using Function but had to change it to std::function to be able to
> > > use it in HashMap value.
> >
> > Nothing about a Function precludes it from being a HashMap value; We use
> > Functions in HashMap values all the time.
>
> For hints, search WebKit2 for:
> `HashMap<uint64_t, Function`</span >
I did it when I found the issue, and all those were cases where function is added once and then taken, like completion handlers. In that case there's no problem because take doesn't need to copy, it just moves. Here we need to set once and get multiple times. We could take and set on every get, but I don't think it's worth it.</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>