Gaia/SMS/Scrum/FxOS-S9

List of bugs

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

Bugzilla link

Full Query

ID	Assigned to	Summary	Blocking b2g	Feature-b2g	Whiteboard	Resolution
1192263	Steve Chung [:steveck]	[Messages] We load Inbox before going to the notification conversation when app is run via notification click.	2.5+	No cf_feature-b2g	[p=2]	FIXED
1197231		Increase in SMS app memory consumption	---	No cf_feature-b2g	[p=2]	WONTFIX
1206727	Julien Wajsberg [:julienw]	[Messages][Inbox] Don't reload the conversation node after saving draft	---	No cf_feature-b2g	[sms-papercuts][p=1]	FIXED
1211395	Steve Chung [:steveck]	Performance regression in Message	2.5+	No cf_feature-b2g	[p=2]	FIXED

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

Remaining points and burndown chart

google chart api url for Sprint FxOS-S9

Burndown chart

	Remaining points
Start	7
Day 2	7
Day 3
Day 4
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-FxOS-S9")

Full Query

ID	Assigned to	Summary	Blocking b2g	Feature-b2g	Whiteboard	Resolution
1213199	Oleg Zasypkin [:azasypkin]	[Messages] 'Message saved as draft' notification is displayed even if draft has been removed afterwards before going to Inbox	2.5+	No cf_feature-b2g	[2.5-aries-test-run-3][sms-sprint-FxOS-S9]	FIXED
1216617	Oleg Zasypkin [:azasypkin]	[Messages] Disable timeout for ActivityClient	2.5+	No cf_feature-b2g	[sms-sprint-FxOS-S9]	FIXED
1218091	Oleg Zasypkin [:azasypkin]	[Messages][Performance] Use optimized Bridge libraries	---	No cf_feature-b2g	[sms-sprint-FxOS-S9]	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
1037620	Stéphane Roucheray	[v2.1] Feature Proposal: Late Arrival Notice in Messages App	---	---	FIXED
1181936		(Gaia RTL 2.5) Support RTL for Messages/SMS App	---	2.5+	WONTFIX
1192263	Steve Chung [:steveck]	[Messages] We load Inbox before going to the notification conversation when app is run via notification click.	2.5+	---	FIXED
1212336		SMS sprint FxOS-S9	---	---	WONTFIX

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

Sprint planning

Minutes are on a separate page.

Daily meetings

Day 2: 8th October

Steve

• National holiday on Friday
• bug 1211395 - Performance regression in Message
  • Profiling the number for coldlaunch time, will compare the result in each stage to see which stage is the bottoleneck

    (Oleg) I'll post my numbers into the bug, will be nice to compare with yours later then :)

• bug 1192263 - [Messages] We load Inbox before going to the notification conversation when app is run via notification click
  • Cearte a small patch for early feeback
• Reviewing the systemMessage handler(Still in unit test part)

Today:

• Coldlaunch profiling.
• Give some feedback for Oleg's patch about the system message.
• Working on some blockers if they have any progress.

Julien

Nothing really exciting. I reviewed patches that are out of the SMS codebase:

• bug 1209041 is a patch for gaia-header but that affected our RTL support
• bug 1196268 is a patch to action_menu.js that will fix the "cancel" button in all our menus

I also had a look to the NI in bug 1206844 about emojis support. And I also looked at bug 1190980 as discussed yesterday, to see if I could reproduce (copy/paste issue in To recipient panel). I also started my experiment with async scripts, but will stop now and will resume later tonight or tomorrow. Today: I want to:

• handle review/feedback/NI queue that's still huge
• handle my bugmail that's still huge too
• land some r+ bugs I have somewhere

Oleg

• I'll be on PTO on Friday.
• bug 1198266 - [Messages] Use ConversationService in the application
  • No progress (awaiting feedback).
• bug 1208532 - [Messages][Refactoring] Extract SystemMessageHandler from ActivityHandler
  • No progress (in review).
• bug 1197231 - Increase in SMS app memory consumption
  • Started to dig into it from re-measuring v2.2, but figured out that raptor doesn't work with v2.2 anymore, talked with Eli and bisect that it has been broken between 1.4.4 and 1.4.5 (current is 2.0.3), filed "bug 1212454 - [Raptor] Raptor fails to run coldlaunch tests for v2.2 branch".
  • So if you need v2.2 numbers use @mozilla/raptor@1.4.4 for now.
  • Also tried to test v2.5 gaia on v2.2 gecko - no luck, it doesn't work for Messages app - Intl api is not available, and other not-really-clear exceptions are thrown in shims. So I measure gaia v2.2/gecko v2.2, gaia master/gecko master and will try gaia v2.2 (well entire gaia doesn't work, so will try only v2.2 Messages app in v2.5 gaia)/gecko master. (in progress)

    (Steve) I saw the empty homscreen after make raptor on 2.2, is that the issue you said here? So using raptor v1.4 should be working on 2.2?

    (Oleg) gaia v2.2 on gecko v2.5 - yes, empty home screen, gaia v2.5 on gecko v2.2 - sms doesn't work, but homescreen works :) Yep v1.4.4 for v2.2 branch (1.4.3 doesn't work, 1.4.5 doesn't work as well but with different error). I've just was able to run gaia v2.5 + sms v2.2 on gecko v2.5 - aaaaaand I'm confused :) _Memory_ numbers are very close to sms v2.5 - digging.

    (Steve) ok :/ We want to reduce the variable as less as possible but it seems not gonna happen

Other:

• bug 1207094 - [Messages] Disable appropriate controls when in low storage condition
  • Left feedback.
• bug 1133711 - Remove dev_apps folder (in background)
  • Discussing with Eli and Kevin where and how we can put performance benchmarking apps into gaia, things are going very slow, maybe I've involved not the right people into discussion - so no decision yet :/
  • Tried to preload outoftree apps with preload script like Spark does, looks like a viable solution.
  • Ideally I'd include various benchmarking app to have _pure_ performance numbers for the API we use, ServiceWorker, maybe mozMobileMessage - to easier and earlier spot Gecko (or System app at least) regressions.
• Guided contributor on "bug 1209419 - [Messages][Tests] Add basic integration tests for the Conversation view Edit mode";
• Finished with bugmail.

Today:

• Will handle review/feedback/need-info requests;
• Will work on review comments and assigned bugs.

Day 3: 9th October

Julien

• Reviewed bug 1192263 about the bug that clicking the notification is loading the inbox
• Answered the question in bug 1190613 about automatically canceling current inline activities when the user displays an app's existing instance from the homescreen
• Looked at bug 864790 about warning about deleting unread messages
• Had a deep look at the issue in bug 1210201 in Calendar, about a change in event fluffing that benefitted the SMS app
• Had a look at bug 1212875 about RTL issues in empty contenteditable elements
• Reviewed bug 1196268 about the cancel button in action menus
• Looked at bug 1180470 about the send report configuration

Today: I want to:

• handle review/feedback/NI queue that's still huge
• handle my bugmail that's still huge too
• land some r+ bugs I have somewhere

Steve

National Holiday

Oleg

PTO