Closed Bug 578588 Opened 14 years ago Closed 14 years ago

Test suite may not work correctly on Win64 because navigator.platform is Win64

Categories

(Core :: General, defect)

x86_64
Windows Vista
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla2.0b3

People

(Reporter: m_kato, Assigned: m_kato)

References

Details

Attachments

(1 file)

From neil's commnet of bug 574051.  We should not use "Win32" for Windows.


http://mxr.mozilla.org/mozilla-central/search?string=navigator.platform

/modules/libpr0n/test/reftest/pngsuite-ancillary/qcms-asm-check.js (View Hg log or Hg annotations)

    line 9 -- if (navigator.platform == "MacIntel") {
    line 13 -- if (navigator.platform == "Win32" || navigator.platform == "OS/2") {

/layout/forms/test/test_bug348236.html (View Hg log or Hg annotations)

    line 116 -- if (/OS\/2|Win32/i.test(navigator.platform))
Attached patch patchSplinter Review
Attachment #457289 - Flags: review?(sayrer)
Attachment #457289 - Flags: review?(sayrer) → review+
landed (this is test only fix)
http://hg.mozilla.org/mozilla-central/rev/decac99650a5
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b3
Depends on: 1472618
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: