Closed Bug 570258 Opened 10 years ago Closed 10 years ago

Some more atom usage cleanup

Categories

(Core :: XPCOM, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: sicking, Assigned: sicking)

Details

(Keywords: perf)

Attachments

(1 file)

Attached patch Patch to fixSplinter Review
I had this sitting in my tree since the last round of nsIAtom cleanup. All this does is fix a few more location which either uses atoms poorly, or that are optimized for utf8 atoms.

An interesting effect is that nsIAtom::EqualsUTF8 is now a totally unused API (and it's not scriptable). I've still left it in though as it might be useful. We can always nuke it if there are still no callers down the road.
Attachment #449384 - Flags: review?(jst)
Attachment #449384 - Flags: review?(jst) → review+
Checked in

http://hg.mozilla.org/mozilla-central/rev/861329598eed
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.