Bug 820310 - [B2G SMS] Multiple markMessageRead() calls causes significa... #7149

Merged
merged 1 commit into from Dec 28, 2012

Projects

None yet

2 participants

@ferjm
Mozilla-B2G member

...nt performance issues. r=borjasalguero

Fixing typo

@borjasalguero
Mozilla-B2G member

https://github.com/ferjm/gaia/blob/88328d80a77130d586ff0dd171431d206c5e036b/apps/sms/js/sms.js#L43

You should modify this line to MessageManager.markMessagesRead([event.message.id],.... due to markMessagesRead does not exist anymore.

@ferjm ferjm Bug 820310 - [B2G SMS] Multiple markMessageRead() calls causes signif…
…icant performance issues. r=borjasalguero

Fixing typo

Addressing feedback and marking thread as read
b9358df
@borjasalguero
Mozilla-B2G member

Great patch! Simple change that makes SMS App works so good! R+. Thanks @ferjm ! Enjoy your Holidays! Gracias ;)

@borjasalguero borjasalguero merged commit cec87e8 into mozilla-b2g:master Dec 28, 2012
@ferjm ferjm deleted the ferjm:820310.markmessageread branch Jan 8, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment