Adam, something else that imho must be considered ( while refactoring
the state machine ) is adding a "load type" that specifically does not
touch session and global history, and avoid "abusing" some of the
existent load types like below:

<abuse>
    // FIXME: This seems like a dangerous overloading of the meaning
of "FrameLoadTypeReload" ...
    // shouldn't a more explicit type of reload be defined, that means roughly
    // "load without affecting history" ?
    if (shouldReloadToHandleUnreachableURL(newDocumentLoader)) {
        ASSERT(type == FrameLoadTypeStandard);
        type = FrameLoadTypeReload;
    }
</abuse>


great effort so far , btw

-- 
--Antonio Gomes
_______________________________________________
webkit-dev mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev

Reply via email to