Combine imgRequestProxy::mLocksHeld and imgRequestProxy::mDeferredLocks

RESOLVED FIXED

Status

()

Core
ImageLib
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Bobby Holley (parental leave - send mail for anything urgent), Assigned: Bobby Holley (parental leave - send mail for anything urgent))

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

bz pointed this out over in bug 359608. It complicates code, and adds unnecessary memory footprint. Patch coming right up.
Created attachment 469360 [details] [diff] [review]
patch - v1

Added a patch.
Attachment #469360 - Flags: review?(joe)
pushed this to try for a mochitest-run (linux only): 3b5003a7c070
Comment on attachment 469360 [details] [diff] [review]
patch - v1

Looks like git is only giving me 3 lines of context, instead of the 8 that would make it easier to review :( I MUDDLED THROUGH SOMEHOW
Attachment #469360 - Flags: review?(joe) → review+
Created attachment 469595 [details] [diff] [review]
patch with 8 lines of context

Sorry about that. I've modified git bz to post patches with 8 lines of context. Giving it a test run now...
It works. Go me.

Updated

7 years ago
Blocks: 359608
Pushed to mozilla-central:
http://hg.mozilla.org/mozilla-central/rev/459a07b6bdee

Resolving fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.