881964 - Mac 'Back' gesture animation fails on www.dn.se

Status: RESOLVED WORKSFORME

(Core :: Widget: Cocoa, defect, P2)

Description

[prodigysim]

Attachment: testcase.html

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:23.0) Gecko/20130610 Firefox/23.0 (Nightly/Aurora)
Build ID: 20130610004016

Steps to reproduce:

1. Enable the Mac 'back' gesture animation using browser.snapshots.limit
2. Go to a webpage that calls window.history.pushState()
3. Use the back gesture.
4. Wait a few seconds...


Actual results:

Page appears visually, but cannot be interacted with (selecting text, etc)


Expected results:

Page should both appear and be interactive.

Comment 2

[José Jeria]

Confirming, see this at www.dn.se all the time.

Comment 5

[:Felipe Gomes (needinfo for replies!)]

Also seen on gmail

Comment 6

[Stephen A Pohl [:spohl]]

(In reply to :Felipe Gomes from comment #5)
> Also seen on gmail

GMail is tracked as bug 935258 until we're confident that it is the same issue as reported here. Since history swipe animations remain turned off on the release channel it may not make sense to track this bug for a particular release.

Comment 7

[:Felipe Gomes (needinfo for replies!)]

(In reply to Stephen Pohl [:spohl] from comment #6)
> (In reply to :Felipe Gomes from comment #5)
> Since history swipe animations remain turned off on the release
> channel it may not make sense to track this bug for a particular release.

Ah ok, I didn't realize bug 860493 was #ifndef RELEASE_BUILD

Comment 8

[Florian Bender]

FYI, it's pretty easy to trigger on Github: 

1. Go to https://github.com/mozilla/mozilla-central
2. Click a folder/file (e.g. "accessible").
3. Swipe back. 

--> blank page.


I actually find it confusing to use the swipe animation when the history way manipulated by script. E.g. Github used to do a sliding animation on history.pushState for the part of the UI that was concerned, and that clashed visually with the full page animation Safari did. 

So the animation should be omitted for the history entries that were added through script (and same-page anchor links as well – the animation is really confusing!).

Comment 9

[Stephen A Pohl [:spohl]]

The patch in bug 935258 fixes GitHub reported in comment 8 here, but www.dn.se continues to misbehave. We're not handling any popstate events for www.dn.se at the moment. Either we're not listening for them correctly, or none are being dispatched.

Comment 10

[José Jeria]

Maybe this is unrelated to the swipe gesture since I need to click on back twice with a build without the swipe feature (tested with Firefox 26 beta)

Comment 11

[Stephen A Pohl [:spohl]]

I'm unlikely to get to this in the next two weeks, so unassigning myself.

Comment 12

[Stephen A Pohl [:spohl]]

This appears to be working properly these days.