Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Eliminate lazy atoms

RESOLVED FIXED in mozilla15

Status

()

Core
JavaScript Engine
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: billm, Assigned: billm)

Tracking

unspecified
mozilla15
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 613451 [details] [diff] [review]
patch

Maybe I'm missing something, but doing lazy initialization of ~20 atoms per runtime seems pointless. It's simpler to eagerly initialize them.
Attachment #613451 - Flags: review?(jwalden+bmo)
Attachment #613451 - Flags: review?(jwalden+bmo) → review+
(Assignee)

Comment 1

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/b627e60ea09b
Target Milestone: --- → mozilla15
(Assignee)

Comment 2

5 years ago
Had to back this out:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0ec98b564d59

and re-land:
https://hg.mozilla.org/integration/mozilla-inbound/rev/99d00468a0dc
Original landing and backout:
http://hg.mozilla.org/mozilla-central/rev/b627e60ea09b
http://hg.mozilla.org/mozilla-central/rev/0ec98b564d59

Re-landing:
http://hg.mozilla.org/mozilla-central/rev/99d00468a0dc
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.