<br><br><div class="gmail_quote">On Tue, Dec 21, 2010 at 11:46 AM, Brady Eidson <span dir="ltr">&lt;<a href="mailto:beidson@apple.com">beidson@apple.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div style="word-wrap:break-word"><br><div><div class="im"><div>On Dec 21, 2010, at 11:39 AM, Darin Fisher wrote:</div><br><blockquote type="cite">I&#39;m working on fixing some session history bugs related to a HistoryItem&#39;s URL property changing.<div>
See for example the call to HistoryItem::setURL in HistoryController::updateForReload [1].<br><div><br></div><div>
I&#39;m curious about the platform specific fields on WebCore::HistoryItem.  *** Do any of those need to</div><div>be updated when the URL of the HistoryItem changes? ***</div><div><font color="#000000"><font color="#144FAE"><br>
</font></font></div></div></blockquote><blockquote type="cite"><div><div>Here are the fields I&#39;m referring to:</div>
<div><br></div><div>class HistoryItem ... {</div><div>private:</div><div>    ...</div><div><div>#if PLATFORM(MAC)</div><div>    RetainPtr&lt;id&gt; m_viewState;</div></div></div></blockquote><div><br></div></div>This is used for the Page Cache only.  The URL had sure better not change while the page is cached!</div>
</div></blockquote><div><br></div><div>OK, I will assert that it is 0.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div style="word-wrap:break-word">
<div><br></div><div><div class="im"><blockquote type="cite"><div><div><div>    OwnPtr&lt;HashMap&lt;String, RetainPtr&lt;id&gt; &gt; &gt; m_transientProperties;</div></div></div></blockquote><div><br></div></div>This is to support arbitrary WebKit Mac API and has nothing to do with the URL identity of the item.</div>
</div></blockquote><div><br></div><div>OK, thanks!</div><div><br></div><div>-Darin</div></div>