[Bug 40254] New: mshtml:script test regression (timeout)

https://bugs.winehq.org/show_bug.cgi?id=40254

Bug ID: 40254
           Summary: mshtml:script test regression (timeout)
           Product: Wine
           Version: 1.7.52
          Hardware: x86
                OS: Windows
            Status: NEW
          Severity: normal
          Priority: P2
         Component: mshtml
          Assignee: [e-mail]
          Reporter: [e-mail]

Since the commit below, the test times out on some Windows XP machines:

script.c:632: Test failed: L"XMLHttpRequest not found in window object\n"
script.c:632: Test failed: L"typeof(XMLHttpRequest) = undefined"
script.c:632: Test failed: L"got exception 'XMLHttpRequest' is undefined"
test failed: timed out

Looking at the test.winehq.org results this seems to specifically affect
pre-Internet Explorer 7 Windows XP configurations such as fgtb-fgtbwinxp,
fg-winxp-1sp and fg-winxp-2sp (for instance none of the fg-winxp-2spie7 and up
VMs is affected).

mshtml:script results on test.winehq.org:
https://test.winehq.org/data/tests/mshtml:script.html

commit c0fda9824e213e5a779bddfba6052dfc20b9c8d3
Author: Jacek Caban 
Date:   Wed Feb 24 15:21:24 2016 +0100

mshtml: Expose XMLHttpRequest constructor to scripts.

[Bug 40254] mshtml:script test regression (timeout)

https://bugs.winehq.org/show_bug.cgi?id=40254

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download, source, testcase
                URL|                            |https://test.winehq.org/dat
                   |                            |a/tests/mshtml:script.html
    Regression SHA1|                            |c0fda9824e213e5a779bddfba60
                   |                            |52dfc20b9c8d3

[Bug 40254] mshtml:script test regression (timeout)

https://bugs.winehq.org/show_bug.cgi?id=40254

whatbug  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #1 from whatbug  ---
If the _Enable native XMLHTTP support” option in IE_ option is disabled
`window.XMLHttpRequest` is undefined.

https://msdn.microsoft.com/en-us/library/ms537505(v=vs.85).aspx#ACTX

[Bug 40254] mshtml:script test regression (timeout)

https://bugs.winehq.org/show_bug.cgi?id=40254

Henri Verbeet  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |regression

[Bug 40254] mshtml:script test regression (timeout)

https://bugs.winehq.org/show_bug.cgi?id=40254

--- Comment #2 from Nikolay Sivov  ---
Looks like it's doing better now. I don't see skips though, so maybe something
else fixed it, but
https://source.winehq.org/git/wine.git/commit/3cb8e4bdc84c3458a0f1979a383266...
helped I think.

[Bug 40254] mshtml:script test regression (timeout)

https://bugs.winehq.org/show_bug.cgi?id=40254

Jacek Caban  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
      Fixed by SHA1|                            |3cb8e4bdc84c3458a0f1979a383
                   |                            |266e02ca79d73
         Resolution|---                         |FIXED
                 CC|                            |[e-mail]

--- Comment #3 from Jacek Caban  ---
It's indeed weird that there is no skip in traces, but the issue seems to be
fixed. Thanks Nikolay.

[Bug 40254] mshtml:script test regression (timeout)

https://bugs.winehq.org/show_bug.cgi?id=40254

--- Comment #4 from Nikolay Sivov  ---
Maybe including winetest.js helped somehow. I noticed that when you run into
undefined objects/functions test simply goes idle and never returns, and
win_skip() was used before my change but was undefined, that could explain that
this line is printed now:

---
script.c:719: Tests skipped: L"postMessage not available"
---

[Bug 40254] mshtml:script test regression (timeout)

https://bugs.winehq.org/show_bug.cgi?id=40254

--- Comment #5 from Jacek Caban  ---
Yeah, that must be it, good catch. Thanks!

[Bug 40254] mshtml:script test regression (timeout)

https://bugs.winehq.org/show_bug.cgi?id=40254

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #6 from Alexandre Julliard  ---
Closing bugs fixed in 3.0-rc6.