Closed Bug 1166207 Opened 10 years ago Closed 9 years ago

Redo preload in the Nuwa process

Categories

(Firefox OS Graveyard :: Performance, defect)

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

Tracking

(feature-b2g:2.5+, firefox43 fixed, b2g-v2.2 wontfix)

Status:
RESOLVED FIXED
Milestone:
FxOS-S6 (04Sep)
Project Flags:
feature-b2g 2.5+
Tracking Flags:
Tracking Status
firefox43 --- fixed
b2g-v2.2 --- wontfix

People

(Reporter: cyu, Assigned: cyu)

References

(Blocks 1 open bug)

Details

(Whiteboard: [caf priority: p2][CR 840327][perf-want])

Attachments

(1 file)

Load preload.js in the Nuwa process
(deleted), patch
khuey
: review+
Details | Diff | Splinter Review
We need to find an alternative to bug 970307 that is less intrusive and fragile.
Assignee: nobody → cyu
Whiteboard: [CR 840327]
Whiteboard: [CR 840327] → [caf priority: p2][CR 840327]
OS: Mac OS X → Gonk (Firefox OS)
Hardware: x86 → ARM
Blocks: 1167703
Offline discussion with Cervantes indicated that this bug will not be fixed on 2.2. m1, can we pull this off the metabug?
status-b2g-v2.2: --- → wontfix
Flags: needinfo?(mvines)
Flags: needinfo?(mvines)
Removing dependency on CAF 2.2, per agreement with partner.
No longer blocks: CAF-v2.2-metabug
No longer blocks: CAF-v2.2-metabug
Blocks: 1180696
feature-b2g: --- → 2.5+
Whiteboard: [caf priority: p2][CR 840327] → [caf priority: p2][CR 840327][perf-want]
(In reply to Cervantes Yu [:cyu] [:cervantes] from comment #8) > https://treeherder.mozilla.org/#/jobs?repo=try&revision=f51a8759ddf6 Tests with BrowserElement APIs finally passed. Continue with other failures on desktop.
We split preload.js into 2 parts: * The "safe part" that doesn't send out IPC messages to the chrome process. This part is loaded in the Nuwa process. * The "unsafe part" that sends out IPC messages to the chrome process. This part is loaded right after we fork. Detection measures in the IPC channel and nsThread are added to debug and test-enabled builds for breakage from future changes.
Attachment #8646272 - Flags: review?(khuey)
Component: General → Performance
url: https://hg.mozilla.org/integration/b2g-inbound/rev/dab5325490951ce5a71b99c366bb902f633893cf changeset: dab5325490951ce5a71b99c366bb902f633893cf user: Cervantes Yu <cyu@mozilla.com> date: Fri Aug 28 17:57:44 2015 +0800 description: Bug 1166207 - Load preload.js in the Nuwa process. r=khuey
https://hg.mozilla.org/mozilla-central/rev/dab532549095
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox43: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → FxOS-S6 (04Sep)
Reopen bug per regression found in bug 1200504. Cervantes, please help to review solution again. Many thanks.
Blocks: 1200504
Status: RESOLVED → REOPENED
Flags: needinfo?(cyu)
Resolution: FIXED → ---
The fix is being reviewed. Clear the ni flag.
Flags: needinfo?(cyu)
Verified in bug 1200504.
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: