Confirmed users
820
edits
Gaia/SMS/Scrum/FxOS-S9: Difference between revisions
/* Day 9: 19th October
m (→Demos) |
(/* Day 9: 19th October) |
||
| Line 420: | Line 420: | ||
=== Day 9: 19th October === | === Day 9: 19th October === | ||
====Steve==== | |||
* {{Bug|1211395}} - Performance regression in Message | |||
** Tests added for review and reply the memory usage concerns. | |||
* {{Bug|1192263}} - [Messages] We load Inbox before going to the notification conversation when app is run via notification click | |||
** Give another idea for the notivication removal and message launch handling. | |||
Today: | |||
* Waiting for performce patch review. | |||
* Working on some blockers if they have any progress. | |||
====Julien==== | |||
* Tried to fix {{Bug|1215513}} to minify all JS files, not only the files that are imported from index.html.. but this gave no meaningful improvement. | |||
* looked at {{Bug|1192263}} about the "notification" system message | |||
* commented on {{Bug|1215655}} to explain to Bryant the "unread" vs "draft" indicators behavior | |||
Today: | |||
I want to: | |||
* handle my review queue | |||
* try to wrap up {{Bug|1206727}} to ask for a first feedback | |||
* maybe try to fix some blockers in other apps. | |||
* land some r+ bugs I have somewhere | |||
====Oleg==== | |||
Worked on Saturday instead of Monday, Oct 20-21 - on conference, will be checking email occasionally | |||
* {{Bug|1198266}} - [Messages] Use ConversationService in the application | |||
** No progress (awaiting feedback). | |||
* {{Bug|1213199}} - [Message] Wrong suggestions 'Message saved as draft' pop up after you select delete the MMS. | |||
** Landed (landed). | |||
* {{Bug|1197231}} - Increase in SMS app memory consumption | |||
** Recaptured memory reports with "-m" options, diff between Gecko v2.2, Gaia v2.2 and Gaia v2.2, Gecko v2.5, difference decreased, but still exists, wondering if something related to how GC works changed in Gecko; | |||
** Run raptor-compare for the raptor results I had, it shows statistically significant 1.7ms USS Gecko regression, so Raptor and about:memory show different numbers, Raptor likely doesn't force GC as well as the user when he opens the app; | |||
** So I'd say we need answers from Gecko for SharedWorker consumption (if we can reduce memory footprint) and from Nicholas (but not sure if it's possible to say something using just basic memory reports). Since it will not block release I'd not spent time on crazy workarounds. Still need to keep an eye on this and incorporate easy and safe improvements if we manage to find such. | |||
Other: | |||
* {{Bug|1209419}} - [Messages][Tests] Add basic integration tests for the Conversation view Edit mode | |||
** Was guiding contributor and investigating the issue he was experiecing, see below (in progress). | |||
* {{Bug|1215806}} - Marionette Element.displayed() works inconsistently for the elements that aren't in viewport because of scroll. | |||
** Was investigating this issue that prevents makes integration testing of Edit mode difficult, provided reduced test case and ni?'ed :gaye (wating for ni?) | |||
* {{Bug|1175080}} - AssertionError: expected [Function] to not throw an error but 'Error: Polling socket recv() timeout!' was thrown can happen if you wait for an element to disappear. | |||
** Reviewed that tiny PR that temporarily disabled some Contacts + SMS integration tests (landed). | |||
* {{Bug|1151847}} - [Network Alerts][Refactoring] Use notification "data" attribute instead of icon URL query string parameters | |||
** Reviewed PR, still needs some work (in progress). | |||
* {{Bug|1215471}} - [Contacts] The attachment file name shows "NaN MB" when user attaches a contact in Messages | |||
** Quickly checked that we can't attach Contact from Email as well and invalid blob is returned from contact activity (can't clone like error in logcat), redirected to Contacts, but could be a Gecko regression as well. | |||
* {{Bug|1205988}} - [Messages]Enter Messaging Settings view, then device receives a MMS, but it can't back to Messages view when user taps the notification. | |||
** Quickly verified this bug - works fine from the first look (waiting for QA check). | |||
* Recorded demo for the removed draft save/replace/discard dialog: https://wiki.mozilla.org/Gaia/SMS/Scrum/FxOS-S9#Demos | |||
* {{Bug|1133711}} - Remove dev_apps folder (in background) | |||
** No progress so far, waiting for ni? I've set (waiting for ni?). | |||
Today: | |||
* Will handle review/feedback/need-info requests; | |||
* Will work on review comments and assigned bugs. | |||
=== Day 10: 20th October === | |||
== Demos == | == Demos == | ||
=== {{Bug|1176976}}: Remove the draft saving/replacing action menu === | === {{Bug|1176976}}: Remove the draft saving/replacing action menu === | ||