[Webkit-unassigned] [Bug 34773] [GTK] Hits assertion on history back, with page cache enabled, in specific conditions

[bugzilla-daemon at webkit.org]

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





--- Comment #3 from Xan Lopez <xan.lopez at gmail.com>  2010-02-10 07:10:55 PST ---
(In reply to comment #2)

Just to update this after some chat through jabber.

> - I don't think the test really needs to open new windows? I certainly got the
> crashes without doing that.

Apparently this was only done to organize the test, but it's not needed for it
to crash. Guess we can try to get rid of it.

> - Do you really need the fourth page just for the PASS? Can't you just print
> that  when you are back at the beginning or something? Or just get rid of it
> completely, since the tests passes if you don't crash anyway.

Didn't discuss this :P

> - Do we really need the bit about saving the values in the adjustments?
> Shouldn't that be set by the view when it's restored automatically through the
> usual path? Or was that dependent on a new view being created?

This one seems to be needed because although the view indeed restores the
values it does it before we are able to restore our adjustments, so it's
useless. If this is the case there should be some FIXME/comment in the code
about it.

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