1050038 - [Monitor][Message]The message interface display error when click "back" options quickly

Status: RESOLVED WORKSFORME

Description

[sync-1]

Firefox OS v1.3
 Mozilla build ID:20140422024003
 
 +++ This bug was initially created as a clone of Bug #738206 +++
 DEFECT DESCRIPTION:
 The message interface display erro when click "back" options quickly
 
  REPRODUCING PROCEDURES:
 PRE:have some message in the DUT.
 1.Enter into message
 2.click a conversations quickly,click "back"option,quickly
 3.repeat setp 2 some times
 4.the message niterface display erro--K.O
 
  EXPECTED BEHAVIOUR:
 K.O--message interface should display correct
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:medium
 
  REPRODUCING RATE:1/6
 
  For FT PR, Please list reference mobile's behavior:
 ++++++++++ end of initial bug #738206 description ++++++++++

Comment 1

[sync-1]

Created attachment 8468967 [details]
jrdlog

Comment 2

[sync-1]

Created attachment 8468968 [details]
jrdlog

Comment 3

[sync-1]

Created attachment 8468969 [details]
screen short

Comment 4

[sync-1]

Created attachment 8468970 [details]
screen short

Comment 5

[sync-1]

Created attachment 8468971 [details]
Video

Comment 6

[Julien Wajsberg [:julienw]]

Can you please check on a more recent version? I think we fixed this.

The issue is probably that we call scrollIntoView on the last SMS when we're not in the thread panel.

Comment 7

[田旻]

Dear Julien,

I don't know which build did you fixed this bug, because we just have build:20140422024003.

Thus can you give the patch to me?

Thanks a lot.

Comment 8

[田旻]

(In reply to Julien Wajsberg [:julienw] from comment #6)
> Can you please check on a more recent version? I think we fixed this.
> 
> The issue is probably that we call scrollIntoView on the last SMS when we're
> not in the thread panel.

No, I don't think you are right, because this bug can still be reproduced even if it dosen't scroll  at all in a very short sms.

Comment 9

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

qawanted to check if latest master has this bug

Comment 10

[Julien Wajsberg [:julienw]]

(In reply to 田旻 from comment #8)
> (In reply to Julien Wajsberg [:julienw] from comment #6)
> > Can you please check on a more recent version? I think we fixed this.
> > 
> > The issue is probably that we call scrollIntoView on the last SMS when we're
> > not in the thread panel.
> 
> No, I don't think you are right, because this bug can still be reproduced
> even if it dosen't scroll  at all in a very short sms.

I'll check this but I think we call this function all the time even when the thread is not scrollable. The result is that if the element is not displayed (because we aleady left the panel) we get the behavior you see.

Comment 11

[Julien Wajsberg [:julienw]]

I think this has been fixed by the navigation refactoring in bug 881469 and its follow-up bug 1022755.

If you want to fix it in v1.3 or v1.4, this is IMO the easiest:

* in ThreadUI.scrollViewToBottom: check that the hash starts with "#thread="

Comment 12

[Cody Roesch [:croesch]]

Unable to reproduce this bug on Flame 2.1 with the variables below.

Actual Results: No error or broken UI is seen when going in and out of a message thread many times quickly.

Repro Rate: 0/30

Environmental Variables:
Device: Flame Master
Build ID: 20140807084328
Gaia: 54c3c19d439f7dbafda5c6cc3b4850b545a068ba
Gecko: 2f198e81ed98
Version: 34.0a1 (Master)
Firmware Version: v123

Comment 13

[Joshua Mitchell (Inactive)]

This is certainly fixed in the latest. Seems like a works-for-me issue but I'll leave it alone because I'm not sure if you guys are trying to fix issues in 1.3?

Comment 14

[Julien Wajsberg [:julienw]]

Let's close it, we won't provide a patch for this.

If partners want to fix this, it should be easy to do that by themselves following comment 11.

Thanks Joshua !

Comment 15

[田旻]

Dear Julien,

I think this problem is lead by the press of the button on the header again during its page slide from right to right and before it on a stable status.  It should be disable during this time.

Comment 16

[田旻]

The back button will slide from left to right, click on a message on the thread_list_ui will slide from right to left. If we press them in very short slot, they will collision. This will lead the bug.

Comment 17

[Julien Wajsberg [:julienw]]

Dear Tianm, there are a several fixes to this. What you say is not false, but I think it's a wrong fix. If a user wants to move back to the Thread List as soon as he enters a Thread, I think we should not prevent it.

As I said, the easiest fix is comment 11.