951157 - Clear seer DB when clearing history

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: Clear seer DB v0.1

The seer DB is not cleared when clearing the history via [Clear private data]. It's because the sanitizer for Android was not updated when the desktop version was updated in bug 881804. This patch just adds the same code to the Android version of Sanitizer.jsm

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: Remove desktop code v0.1

A followup patch to remove desktop specific code from the Android sanitizer:
* We don't have a searchbar in XUL
* We don't use the Open Location dialog or preference

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 8348757 [details] [diff] [review]
Remove desktop code v0.1

Wes is probably a better reviewer for this chunk

Comment 3

[Wesley Johnston (:wesj)]

Comment on attachment 8348757 [details] [diff] [review]
Remove desktop code v0.1

Review of attachment 8348757 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/android/modules/Sanitizer.jsm
@@ -149,5 @@
>          catch (e) { }
>  
> -        // Clear last URL of the Open Web Location dialog
> -        try {
> -          Services.prefs.clearUserPref("general.open_location.last_url");

It doesn't look like Desktop uses this anymore either :)

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

remote:   https://hg.mozilla.org/integration/fx-team/rev/084f1f645c2d
remote:   https://hg.mozilla.org/integration/fx-team/rev/26e97bf73504

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/084f1f645c2d
https://hg.mozilla.org/mozilla-central/rev/26e97bf73504