<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] URI Fragment is lost after redirect"
href="https://bugs.webkit.org/show_bug.cgi?id=170058#c11">Comment # 11</a>
on <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - [SOUP] URI Fragment is lost after redirect"
href="https://bugs.webkit.org/show_bug.cgi?id=170058">bug 170058</a>
from <span class="vcard"><a class="email" href="mailto:svillar@igalia.com" title="Sergio Villar Senin <svillar@igalia.com>"> <span class="fn">Sergio Villar Senin</span></a>
</span></b>
<pre>Comment on <span class=""><a href="attachment.cgi?id=306070&action=diff" name="attach_306070" title="Patch">attachment 306070</a> <a href="attachment.cgi?id=306070&action=edit" title="Patch">[details]</a></span>
Patch
View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=306070&action=review">https://bugs.webkit.org/attachment.cgi?id=306070&action=review</a>
<span class="quote">> LayoutTests/http/tests/navigation/redirect-preserves-fragment.html:3
> +<head></span >
No need for <html>, <head> and <body> tags when specifying the DOCTYPE. Same for the other tests.
<span class="quote">> Source/WebKit2/NetworkProcess/soup/NetworkDataTaskSoup.cpp:107
> + m_currentRequest = WTFMove(request);</span >
Not a big fan of adding another element to the internal state of the object.
Also we're doing additional changes of behaviour here in case of multiple redirections because you are no longer using the first request for several checks.</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>