Image decoding prefs aren't reloaded when they change

RESOLVED FIXED in Firefox 12

Status

()

RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: justin.lebar+bug, Assigned: justin.lebar+bug)

Tracking

(Blocks: 1 bug)

unspecified
mozilla12
Points:
---

Firefox Tracking Flags

(firefox12 fixed)

Details

(Whiteboard: [qa?])

Attachments

(1 attachment)

There's no good reason for this; just an omission in the pref tracking code.

I'll rewrite the code to use the new pref caches.  It's much cleaner.
(Assignee)

Updated

7 years ago
Blocks: 676270
Created attachment 583825 [details] [diff] [review]
Patch v1

Review: joedrew!
(Assignee)

Updated

7 years ago
Attachment #583825 - Flags: review?(joe)
> Review: joedrew!

Hm, that didn't work quite as I'd intended.  :)
Comment on attachment 583825 [details] [diff] [review]
Patch v1

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

DiscardTracker.cpp |   14 ++++++-
 RasterImage.cpp    |   39 ++++++++++++---------
 RasterImage.h      |    5 --
 imgRequest.cpp     |   94 ++++++-----------------------------------------------
 4 files changed, 45 insertions(+), 107 deletions(-)

Yes, this'll do.

::: image/src/RasterImage.cpp
@@ +86,5 @@
>  #endif
>  
> +// Tweakable progressive decoding parameters.  These are initialized to 0 here
> +// because otherwise, we have to initialize them in a static initializer, which
> +// makes us slower to start up.

All globals are initialized to zero as they're in the .bss section, right?
Attachment #583825 - Flags: review?(joe) → review+
> All globals are initialized to zero as they're in the .bss section, right?

Yes.  And because the whole page comes in as zeroed, this is faster than initializing to something else.
Thanks, Joe.

https://hg.mozilla.org/integration/mozilla-inbound/rev/e251ac6325b4
Assignee: nobody → justin.lebar+bug
status-firefox12: --- → fixed
https://hg.mozilla.org/mozilla-central/rev/e251ac6325b4
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
Is there something QA can do to test this?
Whiteboard: [qa?]
You need to log in before you can comment on or make changes to this bug.