Last modified: 2013-03-23 20:09:19 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T36820, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 34820 - [Regression] jquery.textSelection fail
[Regression] jquery.textSelection fail
Status: REOPENED
Product: MediaWiki
Classification: Unclassified
JavaScript (Other open bugs)
1.19
All All
: Normal normal (vote)
: Future release
Assigned To: Krinkle
: code-update-regression
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-02-29 23:46 UTC by Krinkle
Modified: 2013-03-23 20:09 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Krinkle 2012-02-29 23:46:53 UTC
For several weeks now some unti tests continue to fail and pass at random.

Initially we were seeing some IE VMs causing test failures due to what I think was a DLL error on the VMs side, however those have been solved.
Comment 1 Krinkle 2012-02-29 23:48:38 UTC
This is not a bug on our side.

When an IE browser window doesn't have focus (i.e. is not in the fore ground. Either minimized or another window in front of it) text can't be selected.

Normally this isn't an issue (since a user can't click in a window that he can't see), but when testing en mass people often have a swarm client in the background.

This can't be fixed on our end, we need to make sure people supporting the swarm with IE only do so with dedicated VMs or machines, not in the background.
Comment 2 Krinkle 2012-03-05 05:47:01 UTC
Hm.. appears aside from the test failing if the window doesn't have focus, it also fails on the first "cold" (uncached) hit.

I'm consistently seeing that when the jquery.textSelection test is loaded in IE6-8 it downloads a bunch of stuff and fails. Then when I refresh (and get lucky to get the same test again, TestSwarm tries up to 4 times to get a pass), it does pass.

Re-opening, we really need to get this sorted out. It might be a valid failure.
Comment 3 Krinkle 2012-03-06 20:47:45 UTC
Marking as blocking for 1.19.0, either this module is broken (in which case we have real problem for IE6-8 in WikiEditor etc.), or the unit test needs to be fixed.
Comment 4 Krinkle 2012-03-07 02:49:30 UTC
Disabled the test in question in r113213, to be re-enabled when this bug is fixed (cc-ing brion since he knows more about the bug this test is associated with).

Now we can at least use TestSwarm again in a normal way for IE (so far all IE columns were red that made it hard to detect a real regression).
Comment 5 Krinkle 2012-07-03 21:53:25 UTC
Latest version appears to fail only in IE9 and IE10 (IE6-8 seem to have recovered).
Comment 6 Mark A. Hershberger 2012-09-30 16:54:04 UTC
Any relation to bug #40598 ?? (also, bumping to future).
Comment 7 Krinkle 2012-10-02 18:33:52 UTC
(In reply to comment #6)
> Any relation to bug #40598 ?? (also, bumping to future).

No, unlikely to be related (other than that both are an exposition of Microsoft's talent to implement annoying bugs and keep them around for years)
Comment 8 Krinkle 2012-11-10 12:41:13 UTC
In Opera 12 as well.
Comment 9 Krinkle 2012-11-10 12:41:28 UTC
*** Bug 41931 has been marked as a duplicate of this bug. ***
Comment 10 Bartosz Dziewoński 2012-11-10 17:32:07 UTC
I'm pretty sure that the Opera thing is not a dupe; I've only noticed it happening on Opera 12.10 (newest version, just released), and it seems to be caused by special-casing Opera in the tests (grep for "window.opera") - the failure is not a randomly thrown error, but wrong output (the insertion point seems to be shifted by one position, resulting in weirdness).
Comment 11 Krinkle 2012-11-13 01:01:26 UTC
I know it isn't the same original issue (since Opera 12 is a recent version). However it exhibits the same behaviour.
Comment 12 Bartosz Dziewoński 2013-03-23 20:09:19 UTC
(I looked into it and unmarked that bug as dupe.)

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links