Closed
Bug 873156
Opened 12 years ago
Closed 8 years ago
Rename profile-before-change2 to something better
Categories
(Core :: XPCOM, defect, P5)
Tracking
()
RESOLVED
INVALID
People
(Reporter: ehsan.akhgari, Assigned: froydnj)
Details
Like, profile-before-change-last-chance, or really, anything which doesn't have a number in it!
Nathan, wanna take this on please?
Comment 1•12 years ago
|
||
I suggested the current name; not that I really care, but I'm not sure why this name is so terrible.
Priority: -- → P5
Reporter | ||
Comment 2•12 years ago
|
||
(In reply to comment #1)
> I suggested the current name; not that I really care, but I'm not sure why this
> name is so terrible.
Because it doesn't communicate anything about what the notification is supposed to do.
Comment 3•12 years ago
|
||
did we really introduce such thing? I know it's likely not the right place to open such discussion, though if we start introducing multipe layers to notifications it's likely better to keep the current naming, and likely someone will add profile-before-change3 (not that I think it's a good idea, but having to remember many different names is not any better).
FWIW, why not using profile-change-teardown and profile-before-change when in need of synchronized work, or firing a custom notification at profile-before-change for syncing with specific consumers (like on p-b-c serviceA fires serviceA-shutdown and other services depending on that listen to servicesA-shutdown) instead of layering p-b-c?
Comment 4•8 years ago
|
||
Looks like profile-before-change2 has been renamed and split into profile-before-change-qm, profile-before-change-telemetry
so this is no more valid.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•