IE hangs when switching between multiple editors / refocusing browser window

[nkurdybakha]

Steps to reproduce:

1. Open: ​http://download.dojotoolkit.org/release-1.8.3/dojo-release-1.8.3/dijit/tests/editor/test_Editor.html
2. Click editor 'No plugin, initially empty'
3. Click editor 'Created from div'
4. Click Desktop or other app (so editor loose focus)
5. Repeat steps 2-4 several times

Result: IE hangs. At the point IE hangs, both of editors endlessly receive a lot of 'focus' and 'blur' events.

Reproducible: Browsers: IE9 and IE10 OS: Win 7 x64 (host machine, 4 processors), Win 8 x32 (VM, 2 processors)

[nkurdybakha]

See recorded video for details: ​http://youtu.be/5KgV6kZlUWQ

[bill]

Presumably fixed already by [30984] but need to test to confirm.

[bill]

Hmm, actually it's still broken on trunk, perhaps even worse, seems to hang just by opening the Editor (testing on Win8 / IE10).

[bill]

It's hard to debug this problem since IE hangs as soon as the problem occurs, but I can reproduce it by:

1. clicking editor1
2. click outside the browser
3. click editor0

One thing I've noticed is that even on Chrome/mac, if you focus editor1, then click outside the browser, and then click back in the browser *but not on editor1*, editor1 still gets focus. AFAICT there's no dijit code calling focus() on anything.

The problems on IE9+ are possibly exacerbated by the fact that the focusout event on the iframe comes *after* the focusin event on the main window, see #16928. But (as an experiment) I modified dijit/focus to do nothing on the focusin/focusout events, and the problem still happens.

[bill]

In [31073]:

Avoid 100% CPU spin on IE9+ when the browser window is defocused while an Editor instance has focus, and then a different Editor is clicked to bring the browser window back into focus. Fixes #16939 !strict.

[mahays0 <[email protected]…>]

In 5158c16c642d55510af504894c2627b9c1e8d6c5/dijit:

Error: Processor CommitTicketReference failed

Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled? 

[Ed Chatelain]

This also hangs on IE8, it is an easy fix to change the test for the fix added into _onBlur from if(has("ie") >= 9){ to if(has("ie")){

[Ed Chatelain]

This also fails with IE11, so the new check will be if(has("ie") || has("trident")){

[Ed Chatelain <[email protected]…>]

In 95cc48933bef6ed4217a527dae4eef7a83f71059/dijit:

Error: Processor CommitTicketReference failed

Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled? 

[Ed Chatelain <[email protected]…>]

In 06f348a8ecb9e9d9c212e90406068528b6e2e795/dijit:

Error: Processor CommitTicketReference failed

Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled? 

[Ed Chatelain <[email protected]…>]

In 4b181d1df4a457284297ab76ab70091f8af55d60/dijit:

Error: Processor CommitTicketReference failed

Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled? 

[Ed Chatelain]

I am not sure why the links to the changesets above did not work, but here are the links: ​https://github.com/dojo/dijit/commit/95cc48933bef6ed4217a527dae4eef7a83f71059

​https://github.com/dojo/dijit/commit/06f348a8ecb9e9d9c212e90406068528b6e2e795

​https://github.com/dojo/dijit/commit/4b181d1df4a457284297ab76ab70091f8af55d60

[bill]

Mark - You should add this to the list of IE11 fixes that you backport to 1.7.