Remove addressbook throbber broadcaster/observer

RESOLVED FIXED in seamonkey2.1b3

Status

SeaMonkey
MailNews: Address Book & Contacts
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Ian Neal, Assigned: Ian Neal)

Tracking

Trunk
seamonkey2.1b3
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

1.51 KB, patch
neil@parkwaycc.co.uk
: review+
Details | Diff | Splinter Review
(Assignee)

Description

7 years ago
Created attachment 513748 [details] [diff] [review]
Fix Typo

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)

Updated

7 years ago
Assignee: nobody → iann_bugzilla
(Assignee)

Updated

7 years ago
Status: NEW → ASSIGNED

Comment 1

7 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+
(Assignee)

Comment 2

7 years ago
Created attachment 513760 [details] [diff] [review]
Remove broadcaster and observer for throbber [Checked in: Comment 3]

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

7 years ago
Attachment #513760 - Flags: review?(neil) → review+
(Assignee)

Updated

7 years ago
Summary: Fix addressbook throbber → Remove addressbook throbber broadcaster/observer
(Assignee)

Comment 3

7 years ago
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]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 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.