1557506 - Remove [array] use in xpidl from chat/

Status: RESOLVED FIXED

(Chat Core :: General, task)

Description

[Magnus Melin [:mkmelin]]

Per bug 1551704, replace usage of [array] from chat/

These:

• https://searchfox.org/comm-central/search?q=retval%2C+array%2C+size_is&case=false&regexp=false&path=chat
• https://searchfox.org/comm-central/search?q=array%2C+size_is(aCount)%2C+retval&case=false&regexp=false&path=chat
• https://searchfox.org/comm-central/search?q=%5Barray%2C+size_is(aCount)%5D&case=false&regexp=false&path=chat

Might be best with one patch per interface.

Comment 1

[Patrick Cloke [:clokep]]

Attachment: Patch v1

I believe this makes all the required changes. I tested this with the following:

1. Create an IRC account on moznet.
2. Join a channel (and also join that channel from another client).
3. Type "/<tab><tab>" should show a list of commands that are accessible.
4. Type "/list<enter>" and you should have another conversation open with a list of all channels.
5. Hover over a participants name in the conversation UI and more information should pop up.
6. Restart your Thunderbird.
7. Rejoin the same channel.
8. Ensure you can view the historical conversation from the logs.
9. In the account list, right click on one and click "Copy Debug Log".
10. Paste this somewhere and ensure there's content.
11. Add someone to your contact list and ensure status, etc. is working properly for them.
12. Double click on the person in your contact list and a conversation should open.
13. Type a message that's too long for a single IRC message (you'll see the counter appear in the bottom right when you get close).
14. Send the message and ensure both clients see the same content (and that it is properly broken up).

Comment 2

[Magnus Melin [:mkmelin]]

Comment on attachment 9092699 [details] [diff] [review]
Patch v1

Review of attachment 9092699 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM, this is pretty nice cleanup! r=mkmelin

::: chat/components/public/imIConversationsService.idl
@@ +93,5 @@
>  [scriptable, uuid(3f88cc5c-6940-4eb5-a576-c65770f49ce9)]
>  interface imIMessage: prplIMessage {
> +  attribute boolean cancelled;
> +  // Holds the sender color for Chats.
> +  // Empty string by default, it is set by the conversation binding.

For these, please update to jsdoc style documentation comments. /** */

Comment 3

[Patrick Cloke [:clokep]]

(In reply to Magnus Melin [:mkmelin] from comment #2)

For these, please update to jsdoc style documentation comments. /** */

Filed bug 1581202 as a follow-up to do this.

Comment 4

[Jorg K (CEST = GMT+2)]

I'll land it with the next lot.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/a3a6442dde2f
Remove [array] use in xpidl from chat/. r=mkmelin