Closed Bug 419691 Opened 16 years ago Closed 16 years ago

Get multiword search testcase working on windows

Categories

(Toolkit :: Downloads API, defect)

Product:
Toolkit
Component:
Downloads API
Platform:
All
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.9.1

People

(Reporter: Mardak, Assigned: Mardak)

References

Details

Attachments

(1 file)

v1
2.12 KB, patch
sdwilsh
: review+
Details | Diff | Splinter Review 
The testcase was turned off for windows to fix orange from bug 419403.
Flags: in-testsuite?
Do we have any idea why?
Attached patch v1 — — Splinter Review 
Instead of using file:// directly, get a file path similar to what I did to get cleanUp test working on windows:

http://hg.mozilla.org/cvs-trunk-mirror/index.cgi/rev/e8f7ad982adf#l2.17
Assignee: nobody → edilee
Status: NEW → ASSIGNED
Attachment #323928 - Flags: review?(sdwilsh)
Comment on attachment 323928 [details] [diff] [review]
v1

Yeah, that makes sense.  This, of course, bit rots me horribly in bug 437302 - care to make your patch on top of it's patches or wait until I push?

Either way, r=sdwilsh
Attachment #323928 - Flags: review?(sdwilsh) → review+
http://hg.mozilla.org/mozilla-central/index.cgi/rev/377f2910d180
remote: added 1 changesets with 2 changes to 2 files

changeset:   15293:377f2910d180
tag:         tip
user:        Edward Lee <edward.lee@engineering.uiuc.edu>
date:        Sat Jun 07 16:29:38 2008 -0700
summary:     Bug 419691 - Get multiword search testcase working on windows. r=sdwilsh
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Flags: in-testsuite? → in-testsuite+
Target Milestone: --- → Firefox 3.1
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: