Last Comment Bug 718297 - Add EnumerateAll to nsPropertyTable
: Add EnumerateAll to nsPropertyTable
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: DOM (show other bugs)
: 12 Branch
: All All
: -- normal (vote)
: ---
Assigned To: Olli Pettay [:smaug] (high review load, please consider other reviewers)
:
Mentors:
Depends on:
Blocks: 705582
  Show dependency treegraph
 
Reported: 2012-01-15 08:17 PST by Olli Pettay [:smaug] (high review load, please consider other reviewers)
Modified: 2012-01-15 11:23 PST (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (2.73 KB, patch)
2012-01-15 08:17 PST, Olli Pettay [:smaug] (high review load, please consider other reviewers)
jst: review+
Details | Diff | Review

Description Olli Pettay [:smaug] (high review load, please consider other reviewers) 2012-01-15 08:17:48 PST
Created attachment 588742 [details] [diff] [review]
patch

Spitting parts from by main CC optimization patch.
This part is needed so that userdata and userdata handlers can be enumerated
and marked black in certain cases.
Coding and comment styles should be similar what nsPropertyTable has now.
EnumerateAll works almost like Enumerate, but doesn't take the owner as parameter, 
so that all the owners are enumerated.
Comment 1 Olli Pettay [:smaug] (high review load, please consider other reviewers) 2012-01-15 11:23:08 PST
https://hg.mozilla.org/mozilla-central/rev/047c8ba7d2e4

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