Remove some unneeded multithread support

RESOLVED FIXED in mozilla9

Status

()

Core
ImageLib
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jrmuizel, Assigned: jrmuizel)

Tracking

unspecified
mozilla9
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 559054 [details] [diff] [review]
Remove mCacheId and associated code

imglib should only be called from the main thread which implies that we shouldn't need this stuff.

I'll follow up with a patch that removes IsReusable()
Attachment #559054 - Flags: review?(joe)
Comment on attachment 559054 [details] [diff] [review]
Remove mCacheId and associated code

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

::: modules/libpr0n/src/imgLoader.cpp
@@ +1062,3 @@
>  
>      // If the existing request is currently loading, or loading on a different
>      // thread, we'll leave it be, and not put this new entry into the cache.

There's comments about "loading on a different thread" or "different event queue" left over here. They should all be removed.
Attachment #559054 - Flags: review?(joe) → review+
Assignee: nobody → jmuizelaar
https://hg.mozilla.org/mozilla-central/rev/b80b4d52cd14
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
You need to log in before you can comment on or make changes to this bug.