Closed
Bug 635492
Opened 14 years ago
Closed 14 years ago
Remove addressbook throbber broadcaster/observer
Categories
(SeaMonkey :: MailNews: Address Book & Contacts, defect)
SeaMonkey
MailNews: Address Book & Contacts
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.1b3
People
(Reporter: iannbugzilla, Assigned: iannbugzilla)
Details
Attachments
(1 file, 1 obsolete file)
1.51 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
As I was working on another bug I noticed that there was a typo in the broadcaster / observer for addressbook's throbber. Its id is broadcaster-throbber in one and broadcaster_throbber in the other.
Attachment #513748 -
Flags: review?(neil)
Comment 1•14 years ago
|
||
Comment on attachment 513748 [details] [diff] [review]
Fix Typo
It's not as if we use either id anywhere else anyway, maybe we should just remove the broadcaster?
Attachment #513748 -
Flags: review?(neil) → review+
Changes since last version:
* Just remove broadcaster/observer combo as it is not used.
Attachment #513748 -
Attachment is obsolete: true
Attachment #513760 -
Flags: review?(neil)
Updated•14 years ago
|
Attachment #513760 -
Flags: review?(neil) → review+
Summary: Fix addressbook throbber → Remove addressbook throbber broadcaster/observer
Comment on attachment 513760 [details] [diff] [review]
Remove broadcaster and observer for throbber [Checked in: Comment 3]
http://hg.mozilla.org/comm-central/rev/841ae1b1b62c
Attachment #513760 -
Attachment description: Remove broadcaster and observer for throbber → Remove broadcaster and observer for throbber [Checked in: Comment 3]
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b3
Updated•14 years ago
|
Flags: in-testsuite-
You need to log in
before you can comment on or make changes to this bug.
Description
•