That is a much needed big job... look forward to seeing how the details flesh out.<br><br><div class="gmail_quote">On Fri, Sep 25, 2009 at 2:01 PM, Adam Barth <span dir="ltr">&lt;<a href="mailto:abarth@webkit.org">abarth@webkit.org</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">On Fri, Sep 25, 2009 at 1:52 PM, Darin Adler &lt;<a href="mailto:darin@apple.com">darin@apple.com</a>&gt; wrote:<br>

&gt; On Sep 25, 2009, at 1:46 PM, Adam Barth wrote:<br>
&gt;&gt; 1) Separation of concerns. FrameLoader has its fingers in a bunch of<br>
&gt;&gt; different pies. In this phase, I&#39;ll try to break FrameLoader up into a bunch<br>
&gt;&gt; of smaller objects that are in charge of managing different pots of state.<br>
&gt;<br>
&gt; The whole plan sounds great. I’d like to hear more of the details of this<br>
&gt; step.<br>
<br>
</div>I haven&#39;t studied the code in enough detail yet to have many<br>
specifics, but here as some examples:<br>
<br>
A) Scheduled redirections.  There&#39;s a bunch of state associated with<br>
these that seems separable from the concerns of actually performing<br>
the loads.<br>
<br>
B) Methods involving PolicyAction.  Interacting with the<br>
FrameLoadClient seems like a separable issue from advancing the main<br>
state machine.<br>
<br>
C) Form submission.  There seem to be a bunch of special cases having<br>
to do with loading form submissions.  I haven&#39;t looked in detail, but<br>
this also seems like it should be a client of the core machine.<br>
<br>
As we get into the details, I&#39;m sure we&#39;ll find some obvious wins.<br>
<font color="#888888"><br>
Adam<br>
</font><div><div></div><div class="h5">_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
</div></div></blockquote></div><br>