Closed Bug 635492 Opened 13 years ago Closed 13 years ago

Remove addressbook throbber broadcaster/observer

Categories

(SeaMonkey :: MailNews: Address Book & Contacts, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1b3

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Fix Typo (obsolete) — 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)
Assignee: nobody → iann_bugzilla
Status: NEW → ASSIGNED
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)
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: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b3
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: