Last Comment Bug 696201 - Switch content lists to nsTArray
: Switch content lists to nsTArray
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: DOM (show other bugs)
: Trunk
: All All
: P1 normal (vote)
: mozilla10
Assigned To: Boris Zbarsky [:bz] (TPAC)
:
Mentors:
Depends on: 696195
Blocks:
  Show dependency tree / graph
 
Reported: 2011-10-20 13:44 PDT by Boris Zbarsky [:bz] (TPAC)
Modified: 2011-11-01 07:41 PDT (History)
3 users (show)
bzbarsky: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Attachments
Switch nsContentList to using an nsTArray as the element store. (13.17 KB, patch)
2011-10-20 13:48 PDT, Boris Zbarsky [:bz] (TPAC) 		jonas: review+
Details | Diff | Splinter Review
View All Add an attachment (proposed patch, testcase, etc.)

Description Boris Zbarsky [:bz] (TPAC) 2011-10-20 13:44:50 PDT 
The nsCOMArray thing was annoying me.
Comment 1 Boris Zbarsky [:bz] (TPAC) 2011-10-20 13:48:30 PDT 
Created attachment 568501 [details] [diff] [review]
Switch nsContentList to using an nsTArray as the element store.
Comment 2 Jonas Sicking (:sicking) No longer reading bugmail consistently 2011-10-31 16:55:50 PDT 
Comment on attachment 568501 [details] [diff] [review]
Switch nsContentList to using an nsTArray as the element store.

Sweet! Those non-inlined functions always annoyed me :)
Comment 3 Boris Zbarsky [:bz] (TPAC) 2011-10-31 19:51:21 PDT 
http://hg.mozilla.org/integration/mozilla-inbound/rev/74e1804b91de
Comment 4 Kyle Huey [:khuey] (Exited; not receiving bugmail, email if necessary) 2011-11-01 07:41:10 PDT 
https://hg.mozilla.org/mozilla-central/rev/74e1804b91de
Note You need to log in before you can comment on or make changes to this bug.