Gaia/SMS/Scrum/2.1S5

List of bugs

SMS issues handled by the SMS subteam (blocks the sprint bug 1067820)

Bugzilla link

Full Query

ID	Assigned to	Summary	Blocking b2g	Feature-b2g	Whiteboard	Resolution
903763		[Messages] avoid css recalculations while rendering a thread	---	No cf_feature-b2g	[c=progress p= s= u=][p(2.1S5)=1]	DUPLICATE
1003843	Oleg Zasypkin [:azasypkin]	Follow up to Bug 951676 - [Messages][Refresh] Add specific image in case of more than one recipient in the list threads	---	No cf_feature-b2g	groupmms [p(2.1S6)=1][p(2.1S5)=1]	FIXED
1021608	Steve Chung [:steveck]	[Messages] Report panel visual refresh	---	No cf_feature-b2g	[sms-most-wanted][p(2.1S7)=1][p(2.1S6)=2][p(2.1S5)=2]	FIXED
1053708	Julien Wajsberg [:julienw]	Make SMS main User Interface RTL-friendly	---	No cf_feature-b2g	[p(2.1S6)=1][p(2.1S5)=2]	FIXED
1054989	Julien Wajsberg [:julienw]	[Messages][DSDS] Send button refresh	---	No cf_feature-b2g	[p=1]	FIXED
1061417	Julien Wajsberg [:julienw]	[SMS] We don't properly discard the draft in some occasions	2.1+	No cf_feature-b2g	[g+][LibGLA,TD92485,QE1, B][p=1]	FIXED
1067228	Oleg Zasypkin [:azasypkin]	[Messages][Refactoring] Move subject management to a separate component	---	No cf_feature-b2g	[p=2]	FIXED

7 Total; 7 Open (100%); 0 Resolved (0%); 0 Verified (0%);

Remaining points and burndown chart

google chart api url for Sprint 2.1S5

Burndown chart

	Remaining points
Start	10
Day 2	10
Day 3	10
Day 4	9
Day 5
Day 6
Day 7
Day 8
Day 9
Day 10
End

SMS issues handled by the SMS subteam outside of the sprint (contains whiteboard "sms-sprint-2.1S5")

Full Query

ID	Assigned to	Summary	Blocking b2g	Feature-b2g	Whiteboard	Resolution
1052447	Julien Wajsberg [:julienw]	[Messages] when we're in an activity and we delete all messages in the thread, we should close the activity	2.0+	No cf_feature-b2g	[g+][LibGLA, Dev, B][sms-papercuts][sms-sprint-2.1S5]	FIXED
1061215	Oleg Zasypkin [:azasypkin]	[Messages][Refactoring] Improve handling of remaining chars counter	---	No cf_feature-b2g	[sms-papercuts][sms-sprint-2.1S5]	FIXED
1070890	Oleg Zasypkin [:azasypkin]	[Messages][Refactoring] Make use of EventDispatcher in Compose	---	No cf_feature-b2g	[sms-sprint-2.1S5]	FIXED

3 Total; 3 Open (100%); 0 Resolved (0%); 0 Verified (0%);

All SMS issues tracked for this sprint (target milestone)

Bugzilla link

Full Query

ID	Assigned to	Summary	Blocking b2g	Feature b2g	Resolution
836690	Jorge Prudencio [:jorgep]	[SMS] Implement temporary in App message informing the user that they have started another SMS packet	---	---	FIXED
1052447	Julien Wajsberg [:julienw]	[Messages] when we're in an activity and we delete all messages in the thread, we should close the activity	2.0+	---	FIXED
1054989	Julien Wajsberg [:julienw]	[Messages][DSDS] Send button refresh	---	2.2+	FIXED
1061417	Julien Wajsberg [:julienw]	[SMS] We don't properly discard the draft in some occasions	2.1+	---	FIXED
1067228	Oleg Zasypkin [:azasypkin]	[Messages][Refactoring] Move subject management to a separate component	---	---	FIXED
1067820		[meta] SMS subteam sprint 2.1S5	---	---	WONTFIX

6 Total; 6 Open (100%); 0 Resolved (0%); 0 Verified (0%);

Sprint planning

Minutes are on a separate page.

Daily meetings

Day 2: 17th September

Julien

• I finished reading my mails and handling some bugs
• created the Scrum page; I updated the Template page (the graph URL was incorrect when the points were not 11). Maybe we could easily convert this to a script instead, now.

Today:

• will start working on the blocker bug 1061417

Oleg

• bug 1067228 - [Messages][Refactoring] Move subject management to a separate component
  • Almost finished with the main parts, today will tune and work on tests (in progress);
  • Just realized that this bug depends on "bug 1061215 - [Messages][Refactoring] Improve handling of remaining chars counter" that is currently in r? for Steve. Will rebase on master and redirect to Julien as Steve is on PTO.

Other:

• Investigated and dig through bugzilla about iframe vs unload/beforeunload. Alive asked to redirect that question to platform as it's Gecko who doesn't support that.

  (Julien) did we file a bug for having a "onclose" event for a MessagePort ?

  (Oleg) No, does spec contain that event?

  (Julien) nothing in the spec, but from what we saw we need it... we need to evolve the spec :)

  (Oleg) Ok, let's wait for Andrea reply first as start point, he is working on MessagePort and BroadcastChannel (MessageChannel too :)).

  (Julien) if we don't have an answer today, maybe send a mail to dev-webapi. Andrea can be busy :)

  (Oleg) Okay, thanks for dev-webapi reference :) I need to subscribe :)

• Added few demos to previous sprint page.

Today:

• Will handle review comments for the patches that currently in review;

Steve

• Absent/no report

Luke

• Absent/no report

Day 3: 18th September

Julien

• bug 1061417 (draft is not discarded in some occasions): it was easier than expected, and a long-standing bug probably existing since v1.4 (in review)
• resumed working on bug 1054989 (DSDS send button refresh), should have a patch ready today
• did some reviews
• looked into the strange blob-related issue that appeared while I was in holidays (bug 1059233); should have been fixed in gecko... (bug 1063658 might be this)

Other:

• resumed work on bug 874510 (upgrade mocha); one piece was missing (bug 1035133) that I added and is in review now.

Today:

• will continue working on bug 1054989
• will dig in some issues I left from yesterday

Oleg

• bug 1067228 - [Messages][Refactoring] Move subject management to a separate component
  • Still in progress (in progress);
• bug 1061215 - [Messages][Refactoring] Improve handling of remaining chars counter
  • Got first review feedback, working on it (in progress).

    (Julien) do you need something from me here? I saw a question but I don't know if you still need the answer :)

    (Oleg) I guess no :) Will ping you on IRC if have one :)

    (Julien) ok great !

• bug 1067712 - [MADAI] [SMS] Message Apps cannot remove surrogate pair character correctly
  • Investigated and provided reduced test case, now we have Core->Editor bug instead

    (Julien) I hope we don't need an uplift :/

    (Oleg) Yeah, I believe it was always there

Today:

• Will handle review comments for the patches that currently in review;
• Will handle review requests.
• Will see what events are fired on iframe in Firefox when it's removed from DOM to be sure that's not only gaia problem.

Steve

• Absent/no report

Luke

• Absent/no report

Day 4: 19th September

Steve

• bug 1058459 - [SMS] Data shared from other applications are not shown as Drafts until the app is killed and open again
  • Reviewing the SharedWorker patch(and it's brilliant!) will leave some comments later and give it a try
• bug 1059233 - [SMS] User cannot view the cropped image in SMS application and ActivityCanceled error was shown when try to view
  • Having a integration tests for this case might be an idea to avoid possible regression(removing the workaround without gecko fixed first) like this. I've checked 1.4 and this issue exist since workaround was removed from v1.4 and later.

    (Julien) yeah it's strange that the branch checks said that it works in v1.4. I'll look closer today...

• bug 1067232 - [SMS] Avoid to make a private memory copy for file based blob
  • No update yet.

    (Julien) please wait that I sort out the issue in the platform; I think I'll just back out the workaround in SMS and we'll either add it back in Gallery or find the fix in the platform. :asuth thinks that even with the current workaround we can fail in some cases.

    (Steve) Hmm, not sure which case will make the blob copy workaround failed, I thought both workaround have similar approach.

    (Julien) not sure either. Nope, the gallery workaround is: copy the blob to a temporary file so that we get a file-backed blob and do not have the issue in SMS.

    (Steve) Ah, I see. Just check the gallery's patch.

    (Julien) I haven't looked at the patch though, I may be wrong here, I just take what djf told me :) Anyway we'll know more today !

    (Steve) I could agree it's better to workaround in gallery then in messaging in general like asuth said, if they(gallery) agrees to do so. I just confused that why we need messaging workaround in 1.2f and 1.3t if gallery already got the workaround as well...

    (Julien) the gallery workaround is in v1.3t only, as far as I know.

• bug 1021608 - [Messages] Consider adding a "resend" button in the message report page, if there is an error, and other visual refresh
  • Start to implement report panel refresh.
• bug 1068564 - When CMAS alert are deleted in notifications panel, device closes utility try
  • Checking with current master and other notification case. It's weird and seems not a CMAS app wise issue.

    (Oleg) It looks like what I saw previously https://bugzilla.mozilla.org/show_bug.cgi?id=1051793#c8

    (Steve) Thanks! I almost forgot this. It was worked on emulator, but I haven't revisit again

    (Oleg) Maybe I'm wrong but as I remember when I swiped notification it got activated and our app reacted... don't remember exactly :)

    (Julien) it's probably a system front-end bug. (but probably from my code from bug 1051788 :D)

    (Steve) Hmm, will take a look about this then, thanks too!

Today:

• Investigate bug 1068564 and start bug 1021608

Julien

• bug 1061417 (draft is not discarded in some occasions): landed
• bug 1054989 (DSDS send button refresh):
  • got a patch, waiting for fang's ui-review and maybe image updates (because the new ones are larger for apparently no reason)
  • also added a missing l10n string
  • might ask for an approval to land on v2.1 if not too late
• did some reviews for Oleg especially :)
• looked into the strange blob-related issue that appeared while I was in holidays (bug 1059233); should have been fixed in gecko... (bug 1063658 might be this)
  • looks like there is something deeper than just SMS. Will look further today, especially see how it works in the various versions. The regression window just points to the workaround removal in Gallery which does _and_ doesn't make sense ;)

Other:

• resumed work on bug 874510 (upgrade mocha) (my "subway" project :) ); fixed some unit tests but there are more failing.

Today:

• will continue working on bug 1059233 for the blob related issue
• will do reviews
• will start another sprint bug (don't know which one yet)

Oleg

• bug 1067228 - [Messages][Refactoring] Move subject management to a separate component
  • Almost finished unit tests, will wait for bug 1061215 first(in progress, almost finished);
• bug 1061215 - [Messages][Refactoring] Improve handling of remaining chars counter
  • Got second round of review comments, fixed, will ask review in a few! (in progress).
• bug 1050416 - [Messages][Build] Add build script to ignore the desktop-only folder for production build
  • Got first round of review comments, working on it (in progress).

Other:

• Reviewed Luke's patch to fix cursor glitch, left some suggestion on selection managment and unit test;
• Reviewed small patch on draft discard.

Today:

• Will handle review comments for the patches that currently in review;
• Will handle review requests.