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)
Tracking
(blocking-b2g:leo+, 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)
Reporter | ||
Updated•12 years ago
|
Severity: normal → major
Reporter | ||
Updated•12 years ago
|
blocking-b2g: --- → leo?
Whiteboard: [TD]10819 → [TD-10819]
Target Milestone: --- → 1.1 QE2
Reporter | ||
Comment 1•12 years ago
|
||
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)
Comment 3•12 years ago
|
||
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)
Updated•12 years ago
|
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
Comment 7•11 years ago
|
||
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.
Comment 9•11 years ago
|
||
Triage - Leo+, a regression according to comment 6.
blocking-b2g: leo? → leo+
Keywords: regression
Comment 10•11 years ago
|
||
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.
Comment 11•11 years ago
|
||
frsela - are you the right assignee for this leo+ bug?
Assignee: nobody → frsela
Flags: needinfo?(akeybl)
Assignee | ||
Comment 12•11 years ago
|
||
(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
Comment 13•11 years ago
|
||
(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)
Comment 15•11 years ago
|
||
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.
Comment 16•11 years ago
|
||
Updated•11 years ago
|
Flags: needinfo?(pdolanjski)
Assignee | ||
Comment 17•11 years ago
|
||
(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)
Updated•11 years ago
|
Target Milestone: 1.1 QE2 (6jun) → 1.1 QE3 (24jun)
Assignee | ||
Comment 18•11 years ago
|
||
880352 landed in master, can you retest this issue again?
Thanks in advance
Flags: needinfo?(leo.bugzilla.gecko)
Comment 19•11 years ago
|
||
With patch, leo works correctly. Thank you.
Flags: needinfo?(leo.bugzilla.gecko)
Assignee | ||
Comment 20•11 years ago
|
||
\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.
Description
•