Closed Bug 521285 Opened 15 years ago Closed 14 years ago

remove app-wide microsummary generator support

Categories

(Firefox Graveyard :: Microsummaries, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: dietrich, Assigned: dietrich)

Details

(Keywords: perf, Whiteboard: [tsnap])

Attachments

(1 file, 1 obsolete file)

we've never used it, and it makes various syscalls when the microsummary service inits, checking for directories, etc.
Whiteboard: [tsnap]
OS: Windows NT → All
Hardware: x86 → All
Attached patch v1 (obsolete) — Splinter Review
Assignee: nobody → dietrich
Attachment #405362 - Flags: review?(mak77)
Comment on attachment 405362 [details] [diff] [review]
v1

the bug title is confusing, it should probably be "remove app-wide microsummary generators support".
Attachment #405362 - Flags: review?(mak77) → review+
ps: this bug does not look Tsnap related
Summary: remove app-wide microsummary support → remove app-wide microsummary generator support
(In reply to comment #3)
> ps: this bug does not look Tsnap related
See comment 0
Indeed comment 0 says "when the service inits"... it's this a single-instance-tsnap win?
Attached patch v2Splinter Review
Attachment #422877 - Flags: review?(mak77)
Attachment #405362 - Attachment is obsolete: true
Attachment #422877 - Flags: review?(mak77) → review+
Comment on attachment 422877 [details] [diff] [review]
v2

ops, the test!

looks correct, i can't find other hits for this particular folder in mxr.
Keywords: dev-doc-needed
Not sure if this is documented already or not, but we should note that the functionality no longer exists.
Looked through MDC and the wiki and Myk's posts, and could not find a single reference to this behavior.
http://hg.mozilla.org/mozilla-central/rev/b99be55d43f7
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: