typo in sanitize.js pref call (nsIBranch instead of nsIPrefBranch from bug 284086)

RESOLVED FIXED

Status

()

RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: rstrong, Assigned: rstrong)

Tracking

({fixed1.8})

1.5.0.x Branch
fixed1.8
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 198551 [details] [diff] [review]
fix

Fixes the typo and wraps the call to evictEntries in a try catch (like
everything else there) to prevent the following exception when the deletion is
successful so the button will disable. This was also happening before the typo
was fixed.
Error: uncaught exception: [Exception... "Component returned failure code:
0x80520006 (NS_ERROR_FILE_TARGET_DOES_NOT_EXIST)
[nsICacheService.evictEntries]"  nsresult: "0x80520006
(NS_ERROR_FILE_TARGET_DOES_NOT_EXIST)"	location: "JS frame ::
chrome://browser/content/sanitize.js :: anonymous :: line 104"	data: no]
Assignee: nobody → robert.bugzilla
Status: NEW → ASSIGNED
Attachment #198551 - Flags: review?(mconnor)
This will fix also forced deletion failing on a custom cache directory, as
reported here:
https://bugzilla.mozilla.org/show_bug.cgi?id=296256#c9

Thanks!
I decided to test sanitize on shutdown with this patch and experienced crashes
about 1 out of every 3 tries when either clicking the cancel or the clear
private data buttons. So, I backed out this patch and I STILL experienced a
crash about 1 out of every 3 tries - so, this crash condition pre-exists this
change.

without the patch
http://talkback-public.mozilla.org/talkback/fastfind.jsp?search=2&type=iid&id=TB10226811X

with the patch
http://talkback-public.mozilla.org/talkback/fastfind.jsp?search=2&type=iid&id=TB10226811X

So, I think the patch is still the right thing to do and there are other
problems causing the crash. I'll look for a bug   tomorrow, file one if
necessary, and try to debug it if no one else beats me to it (please beat me to
it :/ ). Good night.
Forgot to mention in case talkback is still backed up
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8b5) Gecko/20051004 Firefox/1.4.1
ID:2005100404
doh! the links above are the same and both without this patch - this is one with
the patch
http://talkback-public.mozilla.org/talkback/fastfind.jsp?search=2&type=iid&id=TB10226190W

g'night gracie
Attachment #198551 - Flags: review?(mconnor) → review+
Checked in on trunk

Checking in mozilla/browser/base/content/sanitize.js;
/cvsroot/mozilla/browser/base/content/sanitize.js,v  <--  sanitize.js
new revision: 1.10; previous revision: 1.9
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Comment on attachment 198551 [details] [diff] [review]
fix

Fixes a typo that prevents the sanitize clear cache code from doing its job
properly and also fixes a condition where when it does clear the cache the code
throws an exception which prevents updating of the widget state. This is a safe
/ sane fix.
Attachment #198551 - Flags: approval1.8rc1?
Flags: blocking1.8rc1?

Updated

13 years ago
Attachment #198551 - Flags: approval1.8rc1? → approval1.8rc1+
Checked in on MOZILLA_1_8_BRANCH

Checking in mozilla/browser/base/content/sanitize.js;
/cvsroot/mozilla/browser/base/content/sanitize.js,v  <--  sanitize.js
new revision: 1.5.2.5; previous revision: 1.5.2.4
done
Flags: blocking1.8rc1?
Keywords: fixed1.8
You need to log in before you can comment on or make changes to this bug.