Opened 9 years ago
Closed 9 years ago
#14717 closed bug (cantfix)
onbeforeunload doesn't work on iOS 6-7
Reported by: | m_gol | Owned by: | m_gol |
---|---|---|---|
Priority: | low | Milestone: | |
Component: | event | Version: | 2.1.0-rc1 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
iOS doesn't seem to support the onbeforeunload
event but window.onbeforeunload
is null
which makes our test suite believe it's supported and run the test. :(
This is the only test that's failing in iOS6, fixing/workarounding it would make it possible to register iOS6 in TestSwarm. (iOS7 fails more tests so that would come later).
I have no idea how to blacklist this test in Mobile Safari without resorting to UA sniffing.
Change History (6)
comment:5 Changed 9 years ago by
Component: | unfiled → event |
---|---|
Milestone: | None → 1.11/2.1 |
Owner: | set to m_gol |
Priority: | undecided → low |
Status: | new → assigned |
comment:3 Changed 9 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Tests: Blacklist beforeunload test in iOS
iOS has the window.onbeforeunload field but doesn't support the beforeunload handler making it impossible to feature-detect the support.
Fixes #14717 Closes gh-1496
Changeset: 988d99ad278c2c9dc16c68f86bddc4df99dcd928
comment:4 Changed 9 years ago by
I'm thinking we may want to close this actual ticket as cantfix
and we probably need somewhere to document this. That part is stumping me though, because a note in the API docs is likely to be overlooked. Perhaps Google will find this ticket.
comment:5 Changed 9 years ago by
Milestone: | 1.11/2.1 |
---|---|
Resolution: | fixed |
Status: | closed → reopened |
comment:6 Changed 9 years ago by
Resolution: | → cantfix |
---|---|
Status: | reopened → closed |
You're right, it's not fixed per se, I just blacklisted a failing test so that the build passes. Definitely cantfix.
The problem is a support test would have to effectively trigger
unload
and ifbeforeunload
is not supported there's no way to cancel the unload. :/