Closed Bug 811754 Opened 12 years ago Closed 6 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
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]
See Also: → 1123066
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: 6 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: