[Calendar] move logic from bridge.js into backend/calendar_service.js

RESOLVED INVALID

Status

Firefox OS
Gaia::Calendar
RESOLVED INVALID
3 years ago
2 years ago

People

(Reporter: millermedeiros, Assigned: millermedeiros)

Tracking

(Blocks: 2 bugs)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
right now the "bridge.js" still does all the work in the main thread, we need to convert it into a real "bridge" and use the [threads.js](https://github.com/gaia-components/threads) lib for the worker communication (getting rid of the FakeClientStream implementation)

PS: we should also delete the "js/store" and "js/provider" folders after doing that (move them into the "backend" folder) since they should not be needed anymore - please make sure they are up-to-date, not sure if anything changed since we created the backend folder.
(Assignee)

Updated

3 years ago
Depends on: 1162592, 1163178, 1163179
(Assignee)

Updated

3 years ago
Blocks: 1169790
(Assignee)

Updated

3 years ago
Assignee: nobody → mmedeiros
Depends on: 1177542
(Assignee)

Comment 1

3 years ago
Created attachment 8626360 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/30716
Attachment #8626360 - Flags: review?(gaye)
This looks good, but we shouldn't do anything here until we can run and test our code without crashing gecko.
(Assignee)

Comment 3

2 years ago
Comment on attachment 8626360 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/30716

this was actually uplifted to the Bug 1162586 PR, so I'm going to ignore this PR.
Attachment #8626360 - Flags: review?(gaye)
(Assignee)

Comment 4

2 years ago
let's handle this at once on Bug 1162586 - since we have a pull request there that does the same thing.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.