Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Use handleEvent in PopupNotifications

RESOLVED FIXED in Firefox 14

Status

()

Firefox
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: tetsuharu, Assigned: tetsuharu)

Tracking

Trunk
Firefox 14
x86_64
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [doorhanger])

Attachments

(1 attachment, 1 obsolete attachment)

Use handleEvent in PopupNotifications.
(Assignee)

Updated

5 years ago
Depends on: 745524
Whiteboard: [doorhanger]
Created attachment 615123 [details] [diff] [review]
proposed patch
Attachment #615123 - Flags: review?(dolske)
This seems roughly ok from a quick skim, but what's the reason for making this change? Just code cleanup?
(In reply to Justin Dolske [:Dolske] from comment #2)
> This seems roughly ok from a quick skim, but what's the reason for making
> this change? Just code cleanup?

Yes. This change purposes to clean up.
Using handleEvent can decrease the part of using Function.bind() or making functions for listener only.
Comment on attachment 615123 [details] [diff] [review]
proposed patch

r+ assuming this passes tests -- I didn't test it. :)
Attachment #615123 - Flags: review?(dolske) → review+
Created attachment 615202 [details] [diff] [review]
proposed patch

Rebase on the latest mozilla-central.
I can't Mochitest on all-platform because I don't have the permission to access to try-server.
But I have tried this patch, this patch worked good.
Attachment #615123 - Attachment is obsolete: true
Attachment #615202 - Flags: review?(dolske)
Comment on attachment 615202 [details] [diff] [review]
proposed patch

Would you like tryserver access? Looks like you've been busy with lots of Mozilla stuff, so I'd be happy to vouch for you. See http://www.mozilla.org/hacking/commit-access-policy/ and http://www.mozilla.org/hacking/committer/ -- CC me if you file a bug for access so I can vouch.
Attachment #615202 - Flags: review?(dolske) → review+

Comment 7

5 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/9a2f5f601616
Assignee: nobody → saneyuki.s.snyk
Target Milestone: --- → Firefox 14
https://hg.mozilla.org/mozilla-central/rev/9a2f5f601616
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.