VOOZH about

URL: https://bugzilla.mozilla.org/show_bug.cgi?id=1961385

⇱ 1961385 - Move getTabContext and getUserlistContext into menus.js for cZ


Closed Bug 1961385 Opened 1 year ago Closed 1 year ago

Move getTabContext and getUserlistContext into menus.js for cZ

Move getTabContext and getUserlistContext into menus.js for cZ
SeaMonkey
Chat
Trunk
All
All
RESOLVED FIXED
RESOLVED
FIXED
Future
Tracking Status
seamonkey2.53 + fixed
Tracking Status
seamonkey2.53
seamonkey2.57esr
---
SM2.53.22
QA Whiteboard:
---
Change Request:
---
Bug Flags:
Signature:
None
This bug is publicly visible.

 
Assignee

Description

1 year ago

The only caller for getTabContext and getUserlistContext is now in menus.js, so might as well move these helpers into that file. Do some tidy up of their code whilst here.

Assignee

Comment 1

1 year ago

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: none
Testing completed (on m-c, etc.): 2.53.22pre
Risk to taking this patch (and alternatives if risky): low
String changes made by this patch: none

Attachment #9479910 - Flags: review?(frgrahl)
Attachment #9479910 - Flags: approval-comm-release?

Comment on attachment 9479910 [details] [diff] [review]
1961385-irc-context-move-25322.patch

LGTM r/a+

Attachment #9479910 - Flags: review?(frgrahl)
Attachment #9479910 - Flags: review+
Attachment #9479910 - Flags: approval-comm-release?
Attachment #9479910 - Flags: approval-comm-release+

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/59d287b3617d
Move getTabContext and getUserlistContext into menus.js for cZ. r=frg DONTBUILD

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED

https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/b7ea5118ca10f41bb8fc1610b42360146c824345
Move getTabContext and getUserlistContext into menus.js for cZ. r=frg a=frg

Target Milestone: --- → Future
You need to log in before you can comment on or make changes to this bug.