Closed Bug 871402 Opened 12 years ago Closed 11 years ago

[STK] launch browser doesn't work properly

Categories

(Firefox OS Graveyard :: Gaia::Settings, defect, P1)

ARM
Gonk (Firefox OS)

Tracking

(blocking-b2g:leo+, b2g18 fixed, b2g-v1.1hd fixed)

RESOLVED FIXED
1.1 QE3 (26jun)
blocking-b2g leo+
Tracking Status
b2g18 --- fixed
b2g-v1.1hd --- fixed

People

(Reporter: leo.bugzilla.gecko, Assigned: frsela)

References

Details

(Keywords: regression, Whiteboard: [TD-10819])

Attachments

(2 files)

From attached logs, when the launch browser command is executed, it seems that browser is launched. However, settings app is immediately brought again to foreground. (may be due to some STK commands, not sure though)
Severity: normal → major
blocking-b2g: --- → leo?
Whiteboard: [TD]10819 → [TD-10819]
Target Milestone: --- → 1.1 QE2
if you want to take a test about this case, you must have VIVO operator sim card. After insert simcard and finish booting, you can see STK menu in setting app.
(In reply to leo.bugzilla.gecko from comment #1) > if you want to take a test about this case, you must have VIVO operator sim > card. > After insert simcard and finish booting, you can see STK menu in setting app. After finish booting, in the idle mode - settings - Movistar - launch browser - browser basico ( it is located in setting menu)
New feature, unimplemented requirement, or regression? needinfo ffos-product
Flags: needinfo?(ffos-product)
when we confirm this issue, we found additional information. <gaia/apps/settings/js/icc.js> function init() { ...... icc.addEventListener('stkcommand', function do_handleSTKCmd(event) { handleSTKCommand(event.command); }); window.addEventListener('stkasynccommand', function do_handleAsyncSTKCmd(event) { handleSTKCommand(event.detail.command); }); ......
when we confirm this issue, we found additional information. <gaia/apps/settings/js/icc.js> function init() { ...... icc.addEventListener('stkcommand', function do_handleSTKCmd(event) { handleSTKCommand(event.command); }); window.addEventListener('stkasynccommand', function do_handleAsyncSTKCmd(event) { handleSTKCommand(event.detail.command); }); ...... When the Launch Browser Command executed, the handleSTKCommand function is called twice.(icc EventListener and window EventListner).
Flags: needinfo?(akeybl)
Flags: needinfo?(ffos-product) → needinfo?(pdolanjski)
this is regression issue. when we take a test about this issue, it operated properly in qe1 version. but the problem has occured in qe2 version. <QE1 version> gaia : 663101b6eb809383e5882d9bc3868a923a57998a gecko : 0b83d5e6a54df9bfccdbb5ba662527bf657f5381 <QE2 version> gaia : d5f7b2d3f7acc04c35abda8b001262d8912f9dfe gecko : 0ab3418551bfcefe1e14633f8f2b0422707a4c2f
Is this a dupe of 865985?
No, it is different 865985. although mobile phone has implented launch browser in STK menu, mobile phone still shows stk menu. In my opinion, ME should show browser app and stk menu is located in background. plz confirm attached video clip.
Triage - Leo+, a regression according to comment 6.
blocking-b2g: leo? → leo+
Keywords: regression
When we analyze icc.js, icc_menu.js, icc_cache.js in setting app's code, there is another problem. Although browser is implemented one time, browser tab is three except for basic tab one. I think that browser is actually implemented twice. UICC --------------------> Mobile phone proactive command(launch browser) icc_cache.js receive stkcommand and send stkcommand event to icc.js. And launch browser is implemented. However, icc_menu.js also send stkasynccommand event to icc.js. Therefore, this is the reason launch browser is implemented twice.
frsela - are you the right assignee for this leo+ bug?
Assignee: nobody → frsela
Flags: needinfo?(akeybl)
(In reply to Alex Keybl [:akeybl] from comment #11) > frsela - are you the right assignee for this leo+ bug? I'll check why the message is processed twice
(In reply to Fernando R. Sela [:frsela] from comment #12) > (In reply to Alex Keybl [:akeybl] from comment #11) > > frsela - are you the right assignee for this leo+ bug? > > I'll check why the message is processed twice How does it proceed about checking this problem?? I know that you have plan to code refactoring about simtoolkit code.(bugzilla : 875679) is it possible to fix this problem, when you proceed code refactoring??
Flags: needinfo?(frsela)
AU_LINUX_GECKO_ICS_STRAWBERRY.01.01.00.019.108 Mozilla build ID: 20130526070207 FFOS os version : 1.1.0.0 I have attached log file.(Log_enable_debug.7z) you can check more detail log, because it is enabled debug value from system app (icc_cache.js file DEBUG variable) and settings app (utils.js DEBUG variable). plz confirm this log.
Flags: needinfo?(pdolanjski)
(In reply to Leo from comment #13) > (In reply to Fernando R. Sela [:frsela] from comment #12) > > (In reply to Alex Keybl [:akeybl] from comment #11) > > > frsela - are you the right assignee for this leo+ bug? > > > > I'll check why the message is processed twice > > How does it proceed about checking this problem?? > I know that you have plan to code refactoring about simtoolkit > code.(bugzilla : 875679) > is it possible to fix this problem, when you proceed code refactoring?? Sure, we've it in our radar ;)
Flags: needinfo?(frsela)
Depends on: 880352
No longer depends on: 875679
Target Milestone: 1.1 QE2 (6jun) → 1.1 QE3 (24jun)
880352 landed in master, can you retest this issue again? Thanks in advance
Flags: needinfo?(leo.bugzilla.gecko)
With patch, leo works correctly. Thank you.
Flags: needinfo?(leo.bugzilla.gecko)
\o/ solved with 880352
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: