Last Comment Bug 652052 - don't create |nsImageLoader|s for null border-images
: don't create |nsImageLoader|s for null border-images
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Layout (show other bugs)
: Trunk
: All All
: P5 normal (vote)
: mozilla6
Assigned To: Nobody; OK to take it and work on it
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-04-21 23:22 PDT by David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch)
Modified: 2011-04-22 19:17 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (2.00 KB, patch)
2011-04-21 23:22 PDT, David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch)
bzbarsky: review+
Details | Diff | Review

Description David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2011-04-21 23:22:02 PDT
Created attachment 527725 [details] [diff] [review]
patch

I've had this patch in my tree for ages -- at some point I stuck some assertions in nsImageLoader and discovered that there are cases where we create image loaders for null images, which seems silly.  This fixes that.
Comment 1 Boris Zbarsky [:bz] 2011-04-22 10:42:02 PDT
Comment on attachment 527725 [details] [diff] [review]
patch

r=me
Comment 2 David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2011-04-22 19:17:13 PDT
https://hg.mozilla.org/mozilla-central/rev/5e93d760e31d

Note You need to log in before you can comment on or make changes to this bug.