Tweak magic number to fix bad behavior in Firefox, which was caused by an earlier fix to bad behavior in Chromium.

Browsers are crap.
This commit is contained in:
Joey Hess 2013-12-09 17:44:45 -04:00
parent f8e1d2fd09
commit 4a5ab9b1e7
4 changed files with 13 additions and 6 deletions

2
debian/changelog vendored
View file

@ -19,6 +19,8 @@ git-annex (5.20131131) UNRELEASED; urgency=low
* import: better handling of overwriting an existing file/directory/broken
link when importing
* Windows: assistant and webapp work! (very experimental)
* Fix bad behavior in Firefox, which was caused by an earlier fix to
bad behavior in Chromium.
-- Joey Hess <joeyh@debian.org> Sun, 01 Dec 2013 13:57:58 -0400

View file

@ -18,3 +18,9 @@ Ubuntu 13.04, firefox 25.0.1
--Walter
> [[fixed|done]]; increased number of longpolling failures allowed before
> it enters the chromium back button bugfix workaround to 12.
> Should be more than enough for 3 or 4 long polling elements on a page.
> (Considered only running the chromium bugfix on chromium, but I don't
> want to get into browser detection hacks.) --[[Joey]]

View file

@ -18,10 +18,5 @@ now! --[[Joey]]
repo creation, autostart, restarting, etc, as --force is not propigated.)
* `git annex assistant` has not been tested, is probably quite incomplete.
Is known to not download any files. (transferrer doesn't built yet)
* webapp: Odd problem when using Firefox: Some links don't seem to load.
Clicking on such a link repeatedly will eventually load the right page.
Network console shows web browser is requesting page "/"
despite a link to another page being clicked on. Workaround: Use Chrome
instead; no such problem there.
* No XMPP support (needs a lot of C libraries which are available in
cygwin, but pkg-config does not list them once installed).

View file

@ -16,7 +16,11 @@ function longpoll_div(url, divid, cont, fail) {
},
'error': function(jqxhr, msg, e) {
connfails=connfails+1;
if (connfails > 3) {
// It's normal to get 1 failure per longpolling
// element when navigating away from a page.
// So 12 allows up to 4 longpolling elements per
// page.
if (connfails > 12) {
fail();
}
else {