47056 - memcache debug option is broken

Status: RESOLVED FIXED

(Core :: Networking: Cache, defect, P3)

Description

[Doug Turner (:dougt)]

The debug preferences panel for mem cache is broken.
The pref string in  /xpfe/components/prefwindow/resources/content/pref-debug.xul

is specified as browser.enable.cache,

<checkbox id="browserEnableCache" value="&debugEnableMemCache.label;"
110               pref="true" preftype="bool" prefstring="browser.enable.cache"
111               prefattribute="checked"/> whereas the actual pref string

defined in all.js for mem cache is browser.cache.enable

pref("browser.cache.enable",                true);

Since this is set to true, mem cache is enabled.

Comment 1

[Gagan]

Neeti didn't we add this to an existing bug on mem cache already? I think I 
did... but I can't remember that bug. 

Comment 2

[neeti]

Yes, this was added to bug 30670

Neeti

Comment 3

[Gagan]

ok then... marking as dup. 

*** This bug has been marked as a duplicate of 30670 ***

Comment 4

[sairuh (rarely reading bugmail)]

vrfy dup.

Comment 5

[neeti]

This is not a duplicate of 30670

Comment 6

[neeti]

*** Bug 44766 has been marked as a duplicate of this bug. ***

Comment 7

[neeti]

*** Bug 44766 has been marked as a duplicate of this bug. ***

Comment 8

[neeti]

Nominating for nsbeta3, since 30670 was nsbeta3+

Comment 9

[Suresh Duddi (gone)]

This is simple. Someone could help us with this.

Comment 10

[Suresh Duddi (gone)]

Debug panel is not in release builds.

Work around is change the pref in the pref file.

Comment 11

[Mike Jaques]

Why do we blow off stuff like this, sure its not major or critical to release 
since debug isn't in the commercial builds, but come on, its  a pref check box, 
how hard can simply fixing the correct status of a pref be?  This I swear is 
symptomatic of one of the few things that p@sses me off about this project, 
lots of piddly little bugs that could be wiped off the charts with ease 
(usually UI polish stuff for the mostpart), but get pushed back, marked future, 
and never dealt with. Give poor Bugzilla a break and wipe out these trivial 
bugs.

I noticed it myself when looking through the pref's of a completely fresh clean 
install. Wish I had code knowledge, then I would go change it myself and submit 
a patch.  Anyways, on a side gripe, if no one has looked recently, the scroll-
bar not appearing in the pref's category scroll-box sometimes when the tree is 
expanded needs to be fixed prior to 1.0, and the layout of a lot of the prefs 
panels themselves in a default profile need to be fixed as well as they do not 
fully appear on the default window dimensions. </endrant>

Comment 12

[gordon]

Currently the memory cache does not respect the enable pref because it is getting 
initialized and used by ImgLib before we have any preferences.  We haven't fix 
this particular bug yet because we've been concentrating on crashers and bugs 
that would appear in optimized builds (not just debug builds).  Setting the 
memory cache size to zero will have nearly (but not quite identical) effects as 
disabling it.  I may be able to fix this in 0.9.1.

Comment 13

[gordon]

This has been fixed.

Comment 14

[benc]

->qa to tom