1032166 - Follow up bug 959201 adding test

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::SMS, defect)

Description

[lana]

+++ This bug was initially created as a clone of Bug #959201 +++

Wrong Cursor position in the message compose of a Draft

**PATH**
1. open message composer
2. select new message
3. type something in the message field (and also the ‘to field’ if you wish)
4. select the back button
5. select ‘save as draft’
6. Open the thread with the saved draft

**EXPECTED**
The cursor should be at the end of the message compose to continue writing the message

**ACTUAL**
The cursor is at the beginning of the message compose

Build info:

unagi-ICS.
user
master.
B-39.
Gecko-12dd20e.
Gaia-6bb1b52

Comment 1

[Julien Wajsberg [:julienw]]

(This bug is to contribute tests for the code that was done in bug 959201)

Comment 2

[lana]

Attachment: Pull request

Comment 3

[Steve Chung [:steveck]]

Comment on attachment 8447985 [details] [review]
Pull request

Some suggestions on the github, looks great and congrats on your first unit test patch for the bonus :)

Comment 4

[lana]

That was with some help from Etienne. I didn't want to do half of the work.
You can see on github I made the modifications you suggested.

Comment 5

[Steve Chung [:steveck]]

Comment on attachment 8447985 [details] [review]
Pull request

r=me, let's wait for the travis or gaia-try for landing

Comment 6

[Steve Chung [:steveck]]

in master: ab9d962b6b4cc7a3f03de6573e9c5ebfa2c6a865

Comment 7

[Shally Li（leave from Mozilla）]

This issue has been successfully verified on Flame v2.1.
See attachment: verified_v2.1.mp4
Reproduce rate: 0/5

Flame 2.1 build:
Gaia-Rev        ccb49abe412c978a4045f0c75abff534372716c4
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/18fb67530b22
Build-ID        20141202001201
Version         34.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141202.034824
FW-Date         Tue Dec  2 03:48:34 EST 2014
Bootloader      L1TC00011880

Comment 8

[Shally Li（leave from Mozilla）]

Attachment: verified.MP4