1117705 - [FFOS7715 v2.1][STK]USAT case 27.22.7.5.1 USER ACTIVITY EVENT SEQ 1.1 fail，Press any key，no ENVELOPE EVENT DOWNLOAD -USER ACTIVITY

Status: RESOLVED DUPLICATE of bug 1126687

(Firefox OS Graveyard :: RIL, defect)

Description

[jinghua.xing]

[Testing Steps     ]:USAT case 27.22.7.5.1 USER ACTIVITY EVENT SEQ 1.1  EVENT DOWNLOAD -USER ACTIVITY

[Expected Result  ]:case pass

[Test Result      ]:Press any key，no ENVELOPE EVENT DOWNLOAD -USER ACTIVITY

Comment 1

[jinghua.xing]

In 3GPP TS 51.010-4
Case request return： D6 07 19 01 04 82 02 82 81
Truly return:         D6 07 82 02 82 81 99 01 04

But in ETSI TS 101 267, it also say the Event list tag can be 99 or 19.

What should we do with this case?

Comment 2

[jinghua.xing]

And does the difference between the tags' order of the truly returned and the requested can cause the case failed?

Comment 3

[jinghua.xing]

As in the protocol ETSI TS 101 267, in the ENVELOPE EVENT DOWNLOAD returned, the Event list tag should be placed before the Device identities tag. Do I understand this right?

Comment 4

[jinghua.xing]

Attachment: stk_event.patch

I have made a patch for this issue. Can you help me review it?

Thank you.

Comment 5

[shawn ku [:sku]]

Comment on attachment 8544439 [details] [diff] [review]
stk_event.patch

Review of attachment 8544439 [details] [diff] [review]:
-----------------------------------------------------------------

Change to proper owner for review.

Comment 6

[shawn ku [:sku]]

Hi Jinghua:
 After checking 3GPP TS 11.14, event list should come before device identities (see section 11).
But for tag part, both 19 or 99 are okay for Event list tag.

Hence, I think you patch should be able to work properly.


Hi Edgar,
 Since this involves the change in ril_worker.js, could you please help review it?

Comment 7

[Edgar Chen [:edgar]]

Change to proper component.

Comment 8

[Edgar Chen [:edgar]]

(In reply to shawn ku [:sku] (OOO 01/06 ~ 01/09) from comment #6)
> Hi Jinghua:
>  After checking 3GPP TS 11.14, event list should come before device
> identities (see section 11).
> But for tag part, both 19 or 99 are okay for Event list tag.
> 
> Hence, I think you patch should be able to work properly.
> 
> 
> Hi Edgar,
>  Since this involves the change in ril_worker.js, could you please help
> review it?

Sure I will take a look. Thank you.

Comment 9

[Edgar Chen [:edgar]]

Comment on attachment 8544439 [details] [diff] [review]
stk_event.patch

Review of attachment 8544439 [details] [diff] [review]:
-----------------------------------------------------------------

Patch looks good, but please help to generate a HG patch by following below link,
https://developer.mozilla.org/en-US/docs/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F

Thank you. :)

Comment 10

[Edgar Chen [:edgar]]

Hi Jinghua, I assume you will handle this bug, otherwise please feel free to kick back to me, thank you.