Closed Bug 338371 Opened 20 years ago Closed 20 years ago

remove unittests in source code (safebrowsing)

Categories

(Toolkit :: Safe Browsing, defect)

Product:
Toolkit
Component:
Safe Browsing
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: tony, Assigned: tony)

References

Details

(Keywords: fixed1.8.1)

Attachments

(1 file)

v1: defer overlay, remove dead code
34.64 KB, patch
bryner
: review+
Details | Diff | Splinter Review
Similar to bug 338369, we should #ifdef out unittests from safebrowsing code.
Comments out unittests, deletes files we're not using. Also, I'm removing the xul elements from browser.xul and instead do a lazy overlay when the first phishy site is visited.
Attachment #222534 - Flags: review?(bryner)
Attachment #222534 - Flags: review?(bryner) → review+
fixed on SAFEBROWSING_20060516_BRANCH
fixed on trunk and MOZILLA_1_8_BRANCH
Status: NEW → RESOLVED
Closed: 20 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
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: