Confirmed users
291
edits
Gaia/SMS/Scrum/2.1S4: Difference between revisions
→Day 8: 11th September
Steveck324 (talk | contribs) |
Steveck324 (talk | contribs) |
||
| Line 289: | Line 289: | ||
=== Day 8: 11th September === | === Day 8: 11th September === | ||
====Steve==== | |||
* {{Bug|1048841}} - [Messages] investigate performance hit from IndexedDB at startup (eg: Contact lookup) | |||
** Ongoing, but not the top priority | |||
* {{Bug|983172}} - Parsing jpeg header information for downsampling the image for thumbnail | |||
** Some regression reported by Oleg that we might have some problem with generating a thumbnail from cropped image. It might be the same root cause in {{Bug|1059233}}. | |||
**: (Oleg) Can you please help me to understand what's happening here after standup, on IRC maybe? | |||
**: (Steve) Sure, no problem, | |||
**: (Oleg) Thanks! | |||
* {{Bug|1058459}} - [SMS] Data shared from other applications are not shown as Drafts until the app is killed and open again | |||
** Still trying to find better solution than localStorage. Not sure if postMessage could work. | |||
**: (Oleg) That would be great if we can find something other than this, also trying to find out better way.. | |||
* {{Bug|1059233}} - [SMS] User cannot view the cropped image in SMS application and ActivityCanceled error was shown when try to view | |||
** Create a quick patch that store the blob and fetch the blob again to prevent any possible memory access violation. | |||
**: (Oleg) Do you know why Blob.slice doesn't work for this? I thought it was copying original blob, but probably I'm wrong | |||
**: (Steve) Ya.. I though it might work at first, but actually access the blob for further memory copy will make the system(!) crash. I still think it's a bug in gecko side, maybe I will ping David directly on the bug. | |||
**: (Oleg) Yeah, I think it's a way to go, to be honest don't want to create workarounds for not our bugs :) I believe if Gallery or some other app returns some data, then this app should take care about it, or at least we should be able to use blob.slice to copy data.... Don't know :) Gallery currently behaves itself strangely, every 2nd/3rd time I try to select attachment it complains about missing SD card | |||
* {{Bug|1061417}} - [SMS] We don't properly discard the draft in some occasions | |||
** Not much progress right now. | |||
Bunch of review | |||
* R+ deletion patch. | |||
* Some other minor issue reply | |||
Today: | |||
* Figure out the cropped image memory issue before landing thumbnail refactoring. | |||
* More investigation about the across window communication. | |||
* Draft discard issue. | |||
====Oleg==== | |||
* {{Bug|1063970}} - [Messages][Refresh] Update styles of phone and url links located inside messages | |||
** Prepared patch and set for review (in review). | |||
* {{Bug|1058459}} - [SMS] Data shared from other applications are not shown as Drafts until the app is killed and open again | |||
** Not much updates here, waiting for better approach code snippet from Steve. | |||
* {{Bug|918970}} - [B2G][SMS] Conversation history does not appear when sending a SMS through the contacts app | |||
** Probably I was wrong when I was thinking that only one of these 3 fields (sender/receiver/receivers) can be available at the same time, but that's true for all my SIMS though. | |||
**: (Steve) Is there any info available about these fields or maybe you know why sender can be null for outgoing messages for my SIMs, but not for other (like contributor for this bug)? | |||
**: (Steve) Null sender is possible because RIL could not know device phone number(and it very common for the most cases) | |||
**: (Oleg) Ok, got it, so that not always true as I thought | |||
* {{Bug|1053952}} - [Messages][Refactoring] Delete all messages at once when deleting threads | |||
** Fixed review comments and landed (landed). | |||
Other: | |||
* Reviewed Steve's thumbnails patch, looks great, r+'ed! Reviewing cropping issue workaround. | |||
Today: | |||
* Will handle review comments for the patches that currently in review; | |||
* Will probably pick up new blocker | |||
====Julien==== | |||
* PTO until 9/15 | |||
=== Day 9: 12th September === | === Day 9: 12th September === | ||