[Webkit-unassigned] [Bug 60412] Page::goToItem doesn't work while loading is deferred

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=60412


Adam Barth <abarth at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #92862|review?, commit-queue?      |review-
               Flag|                            |




--- Comment #14 from Adam Barth <abarth at webkit.org>  2011-05-11 20:51:48 PST ---
(From update of attachment 92862)
View in context: https://bugs.webkit.org/attachment.cgi?id=92862&action=review

> Source/WebCore/page/Page.h:407
> +        RefPtr<HistoryItem> m_deferredItem;
> +        FrameLoadType m_deferredFrameLoadType;

I suspect that this isn't the best place for this data.  Generally, the large "hub" objects like Frame and Page shouldn't hold much data themselves because they become "dumping grounds" for lots of unrelated stuff.  I haven't fully understood what your patch does, but these seem related to history, so perhaps the HistoryController is a more appropriate place?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.