Cleanup unused cache preferences

RESOLVED FIXED in Firefox 17

Status

()

Product:
Toolkit
Component:
Safe Browsing
--
trivial
RESOLVED FIXED
Reported:
5 years ago
Modified:
3 years ago

People

(Reporter: gcp, Assigned: gcp)

Tracking

Version:
13 Branch
Target:
Firefox 17
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Patch 1. Remove unused sqlite3 prefs
4.36 KB, patch
dcamp
: review+
Details | Diff | Splinter Review
Patch. Backout
4.34 KB, patch
akeybl
: approval-mozilla-aurora+
mfinkle
: approval-mozilla-central+
Details | Diff | Splinter Review
(Assignee)

Description

•
5 years ago 
We have some preferences relating to the old sqlite-based urlclassifier left. Clean them up.
(Assignee)

Comment 1

•
5 years ago 
Created attachment 599992 [details] [diff] [review]
Patch 1. Remove unused sqlite3 prefs
Attachment #599992 - Flags: review?(dcamp)
 (Assignee)

Updated

•
5 years ago
Assignee: nobody → gpascutto

Updated

•
5 years ago
Attachment #599992 - Flags: review?(dcamp) → review+
 (Assignee)

Comment 2

•
5 years ago 
https://hg.mozilla.org/integration/mozilla-inbound/rev/1a6d008acb4f

Comment 3

•
5 years ago 
https://hg.mozilla.org/mozilla-central/rev/1a6d008acb4f
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 13
 (Assignee)

Comment 4

•
5 years ago 
Created attachment 616614 [details] [diff] [review]
Patch. Backout

Backout due to bug 744993:

a01cf079ee0b Bug 730247
1a6d008acb4f Bug 729928
f8bf3795b851 Bug 729640
35bf0d62cc30 Bug 726002
a010dcf1a973 Bug 726002
e9291f227d63 Bug 725597
db52b4916cde Bug 673470
173f90d397a8 Bug 673470
Attachment #616614 - Flags: approval-mozilla-central?
Attachment #616614 - Flags: approval-mozilla-aurora?
Attachment #616614 - Flags: approval-mozilla-central? → approval-mozilla-central+
 (Assignee)

Comment 5

•
5 years ago 
https://hg.mozilla.org/integration/mozilla-inbound/rev/936582807ae9

Comment 6

•
5 years ago 
https://hg.mozilla.org/mozilla-central/rev/936582807ae9
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Updated

•
5 years ago
Attachment #616614 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora-

Comment 7

•
5 years ago 
Comment on attachment 616614 [details] [diff] [review]
Patch. Backout

Sorry - thought this bug was reopened because the backout was backed out. My mistake. Approved for Aurora 13 (or Beta 13 if the merge occurs before we land).
Attachment #616614 - Flags: approval-mozilla-aurora- → approval-mozilla-aurora+
 (Assignee)

Comment 8

•
5 years ago 
https://hg.mozilla.org/releases/mozilla-beta/rev/0fa60c5f2288
 (Assignee)

Comment 9

•
5 years ago 
Relanding after fixes in bug 673470 to fix bug 744993.
https://hg.mozilla.org/integration/mozilla-inbound/rev/0636e5319324

Comment 10

•
5 years ago 
https://hg.mozilla.org/mozilla-central/rev/0636e5319324
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago → 5 years ago
Resolution: --- → FIXED
Target Milestone: Firefox 13 → Firefox 17
Component: Phishing Protection → Phishing Protection
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.