Closed
Bug 685435
Opened 15 years ago
Closed 15 years ago
Remove some unneeded multithread support
Categories
(Core :: Graphics: ImageLib, defect)
Tracking
()
RESOLVED
FIXED
mozilla9
People
(Reporter: jrmuizel, Assigned: jrmuizel)
Details
Attachments
(1 file)
|
9.34 KB,
patch
|
joe
:
review+
|
Details | Diff | Splinter Review |
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 1•15 years ago
|
||
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+
|
|
||
Updated•15 years ago
|
Assignee: nobody → jmuizelaar
|
||
Comment 2•15 years ago
|
||
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
You need to log in
before you can comment on or make changes to this bug.
Description
•