Closed Bug 811754 Opened 12 years ago Closed 7 years ago

B2G RIL: clean up ril_worker

Categories

(Firefox OS Graveyard :: RIL, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(tracking-b2g:backlog)

RESOLVED WONTFIX
tracking-b2g backlog

People

(Reporter: vicamo, Unassigned)

References

Details

(Whiteboard: [grooming])

Attachments

(2 files)

ril_worker.js is now 7000+ lines of code. After bug 798491 landed, we should now split it into several smaller scripts. For example: === In ril_worker_icc.js === this.ICC = { // All ICC related stuff }; // ICC-specific parcel handlers this.ICC[REQUEST_SIM_IO] = function REQUEST_SIM_IO(length, options) { } // Allow this file to be imported via Components.utils.import(). this.EXPORTED_SYMBOLS = Object.keys(this); === In ril_worker.js === importScripts("ril_consts.js", "systemlibs.js", "ril_worker_icc.js"); ... RIL[REQUEST_SIM_IO] = ICC[REQUEST_SIM_IO].bind(ICC);
TODO: fix test scripts
Attached patch Part 2: Clean up SMS functions (deleted) — Splinter Review
TODO: fix test cases
Depends on: 905109
Component: DOM: Device Interfaces → RIL
Product: Core → Boot2Gecko
Move it to backlog.
blocking-b2g: --- → backlog
Depends on: 964644
Whiteboard: [grooming]
Depends on: 1123201
blocking-b2g: backlog → ---
No longer depends on: 1146799
Depends on: 1146799
Depends on: 999300
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: