<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:darin@apple.com" title="Darin Adler <darin@apple.com>"> <span class="fn">Darin Adler</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - Pending API Request URL is wrong after reloading"
href="https://bugs.webkit.org/show_bug.cgi?id=139342">bug 139342</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Attachment #242716 Flags</td>
<td>review?
</td>
<td>review+
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Pending API Request URL is wrong after reloading"
href="https://bugs.webkit.org/show_bug.cgi?id=139342#c3">Comment # 3</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Pending API Request URL is wrong after reloading"
href="https://bugs.webkit.org/show_bug.cgi?id=139342">bug 139342</a>
from <span class="vcard"><a class="email" href="mailto:darin@apple.com" title="Darin Adler <darin@apple.com>"> <span class="fn">Darin Adler</span></a>
</span></b>
<pre>Comment on <span class=""><a href="attachment.cgi?id=242716&action=diff" name="attach_242716" title="Patch">attachment 242716</a> <a href="attachment.cgi?id=242716&action=edit" title="Patch">[details]</a></span>
Patch
View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=242716&action=review">https://bugs.webkit.org/attachment.cgi?id=242716&action=review</a>
I’m saying review+ but I am not 100% sure this is right.
<span class="quote">> Source/WebKit2/UIProcess/WebPageProxy.cpp:902
> + String url = m_pageLoadState.activeURL();
> + if (url.isEmpty() && m_backForwardList->currentItem())
> + url = m_backForwardList->currentItem()->url();
> +
> + if (!url.isEmpty()) {</span >
Seems a little peculiar here to use isEmpty rather than isNull.
I’m generally concerned that this code is trying to figure out what URL the other process will reload; we are not sending the URL over, so this is trying to predict what the other process will do? If so, then it seems that to be correct this has to match the logic in FrameLoader::reload and other loader logic inside WebCore, and it seems to me that adding this one thing it’s still far from matching it! It’s nice to fix this one particular case, but what about:
- The case where there is an unreachableURL.
- If a window is created by javascript, its main frame can have an empty but non-nil URL. We need to test that case.
I’d like to hear Anders’s opinion.</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>