Closed Bug 998851 Opened 11 years ago Closed 11 years ago

[tarako][wifi] wifi got error after monkey test

Categories

(Firefox OS Graveyard :: Wifi, defect)

Product:
Firefox OS Graveyard
Component:
Wifi
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
major

Tracking

(blocking-b2g:1.3T+, firefox29 wontfix, firefox30 wontfix, firefox31 fixed, b2g-v1.3T fixed, b2g-v1.4 fixed, b2g-v2.0 fixed)

Status:
RESOLVED FIXED
Milestone:
2.0 S1 (9may)
Project Flags:
blocking-b2g 1.3T+
Tracking Flags:
Tracking Status
firefox29 --- wontfix
firefox30 --- wontfix
firefox31 --- fixed
b2g-v1.3T --- fixed
b2g-v1.4 --- fixed
b2g-v2.0 --- fixed

People

(Reporter: seinlin, Assigned: seinlin)

References

Details

Attachments

(1 file)

Bind this to callback function
(deleted), patch
vchang
: review+
bajaj
: approval-mozilla-aurora+
Details | Diff | Splinter Review
04-18 21:24:03.090 611 638 W WifiHW : wifi_command: DRIVER SCAN-PASSIVE 04-18 21:24:03.090 611 638 W WifiHW : command is rejected! supplicant is terminating 04-18 21:24:03.090 611 611 E GeckoConsole: [JavaScript Error: "this is undefined" {file: "jar:file:///system/b2g/omni.ja!/components/WifiWorker.js" line: 2493}]
Attached patch Bind this to callback function (deleted) — — Splinter Review
Attachment #8409527 - Flags: review?(vchang)
blocking-b2g: --- → 1.3T?
Comment on attachment 8409527 [details] [diff] [review] Bind this to callback function Review of attachment 8409527 [details] [diff] [review]: ----------------------------------------------------------------- Looks good. Thank you.
Attachment #8409527 - Flags: review?(vchang) → review+
triage: we will take this for 1.3T to improve wifi stability
blocking-b2g: 1.3T? → 1.3T+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/b7d2b4c6a5b0
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.0 S1 (9may)
Should this land on 1.4 too?
status-b2g-v1.4: --- → ?
status-b2g-v2.0: --- → fixed
Flags: needinfo?(kli)
Flags: needinfo?(kli)
Yes, it should also land on 1.4. Thanks!
status-b2g-v1.4: ? → affected
Sorry to be a hassle, but you need to request aurora approval on the patch then. 1.3T+ doesn't grant auto-approval to patches for 1.4 :)
Blocks: dolphin
Comment on attachment 8409527 [details] [diff] [review] Bind this to callback function [Approval Request Comment] Bug caused by (feature/regressing bug #): scanning error in WiFi driver User impact if declined: WiFi worker thread will get error when there is something wrong in scanning Testing completed (on m-c, etc.): Monkey test on v1.3t Risk to taking this patch (and alternatives if risky): low because it is a syntax change to make it work correctly String or IDL/UUID changes made by this patch: none
Attachment #8409527 - Flags: approval-mozilla-aurora?
Comment on attachment 8409527 [details] [diff] [review] Bind this to callback function low risk change, approving on aurora.
Attachment #8409527 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Keywords: checkin-needed
Comment 9 was when B2G v1.4 was still on Aurora. I'm assuming the approval was intended for b2g30_v1_4 where it has been for the past 2 weeks now.
Keywords: checkin-needed
Whiteboard: [checkin-needed-b2g30]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: