Open Bug 806757 Opened 12 years ago Updated 2 years ago

Finish moving safebrowsing tests

Categories

(Toolkit :: Safe Browsing, defect, P5)

defect

Tracking

()

REOPENED

People

(Reporter: Dolske, Unassigned)

References

Details

Attachments

(1 obsolete file)

Attached patch Patch v.0 (move tests) (obsolete) — Splinter Review
Followup from bug 778608. For some reason moving the safebrowsing tests (from browser/components/safebrowsing/ to browser/base/content/tests) was reliably causing a devtools test to fail on Windows. [See bug 778608 comment 10 for a bit more, I confirmed via extra logging that the parser wasn't creating the condition stuff was looking for.]

It's not at all clear to me _how_ this is happening. Some kind of timing bug involving the order of running tests seems likely, and multiple Try runs seemed to confirm that I could do move of bug 778608 successfully, as long as the safebrowsing tests were left in the same place.

So this bug is spun off as a followup to investigate finishing moving these tests and to investigate what the problem is.
Product: Firefox → Toolkit
Attachment #676492 - Attachment is obsolete: true
Priority: -- → P5
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INACTIVE
Status: RESOLVED → REOPENED
Resolution: INACTIVE → ---
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: