[Webkit-unassigned] [Bug 116709] New: Improve loader/go-back-cached-main-resource.html test
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri May 24 01:15:18 PDT 2013
https://bugs.webkit.org/show_bug.cgi?id=116709
Summary: Improve loader/go-back-cached-main-resource.html test
Product: WebKit
Version: 528+ (Nightly build)
Platform: Unspecified
OS/Version: Unspecified
Status: UNCONFIRMED
Severity: Normal
Priority: P2
Component: Tools / Tests
AssignedTo: webkit-unassigned at lists.webkit.org
ReportedBy: rego at igalia.com
This bug is to improve the loader/go-back-cached-main-resource.html test with the ideas from the following comment by Alexey Proskuryakov in bug #116670#c2:
> (From update of attachment 202708 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=202708&action=review
>
> Getting rid of the timeout is a very welcome improvement, so r=me. Please consider the suggestion below.
>
> > LayoutTests/loader/resources/first-page.html:21
> > +<body onpageshow="registerListeners();">
>
> This is quite misleading. We only need to register listeners on first load, they are already present when restoring from cache. But the function does more than its name says, which is why it's necessary to call it every time!
>
> One way to clean this up would be to rename the function to "didShowPage", and use event.persisted property to check if we are handling the initial load. Or just use an anonymous function, and register it as window.onpageshow = function (event) { ... }
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list