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

classic Classic list List threaded Threaded
9 messages Options
Reply | Threaded
Open this post in threaded view
|

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

Wine - Bugs mailing list
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: [hidden email]
          Reporter: [hidden email]

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 <[hidden email]>
Date:   Wed Feb 24 15:21:24 2016 +0100

    mshtml: Expose XMLHttpRequest constructor to scripts.

--
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.


Reply | Threaded
Open this post in threaded view
|

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

Wine - Bugs mailing list
https://bugs.winehq.org/show_bug.cgi?id=40254

François Gouget <[hidden email]> changed:

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

--
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.

Reply | Threaded
Open this post in threaded view
|

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

Wine - Bugs mailing list
In reply to this post by Wine - Bugs mailing list
https://bugs.winehq.org/show_bug.cgi?id=40254

whatbug <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]

--- Comment #1 from whatbug <[hidden email]> ---
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

--
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.

Reply | Threaded
Open this post in threaded view
|

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

Wine - Bugs mailing list
In reply to this post by Wine - Bugs mailing list
https://bugs.winehq.org/show_bug.cgi?id=40254

Henri Verbeet <[hidden email]> changed:

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

--
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.


Reply | Threaded
Open this post in threaded view
|

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

Wine - Bugs mailing list
In reply to this post by Wine - Bugs mailing list
https://bugs.winehq.org/show_bug.cgi?id=40254

--- Comment #2 from Nikolay Sivov <[hidden email]> ---
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/3cb8e4bdc84c3458a0f1979a383266e02ca79d73
helped I think.

--
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.


Reply | Threaded
Open this post in threaded view
|

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

Wine - Bugs mailing list
In reply to this post by Wine - Bugs mailing list
https://bugs.winehq.org/show_bug.cgi?id=40254

Jacek Caban <[hidden email]> changed:

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

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

--
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.


Reply | Threaded
Open this post in threaded view
|

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

Wine - Bugs mailing list
In reply to this post by Wine - Bugs mailing list
https://bugs.winehq.org/show_bug.cgi?id=40254

--- Comment #4 from Nikolay Sivov <[hidden email]> ---
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"
---

--
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.


Reply | Threaded
Open this post in threaded view
|

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

Wine - Bugs mailing list
In reply to this post by Wine - Bugs mailing list
https://bugs.winehq.org/show_bug.cgi?id=40254

--- Comment #5 from Jacek Caban <[hidden email]> ---
Yeah, that must be it, good catch. Thanks!

--
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.


Reply | Threaded
Open this post in threaded view
|

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

Wine - Bugs mailing list
In reply to this post by Wine - Bugs mailing list
https://bugs.winehq.org/show_bug.cgi?id=40254

Alexandre Julliard <[hidden email]> changed:

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

--- Comment #6 from Alexandre Julliard <[hidden email]> ---
Closing bugs fixed in 3.0-rc6.

--
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.