Closed
Bug 1091964
Opened 10 years ago
Closed 10 years ago
[e10s] Don't send messages for isCallable and isConstructor CPOW hooks
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla36
Tracking | Status | |
---|---|---|
e10s | m4+ | --- |
People
(Reporter: billm, Assigned: billm)
References
Details
Attachments
(1 file)
13.88 KB,
patch
|
mrbkap
:
review+
|
Details | Diff | Splinter Review |
Right now, whenever the JS engine does isCallable or isConstructor on a CPOW, we send a message to the other side asking for the answer. However, these results are currently constant for a given object, so it makes sense to send this information when the CPOW is created. At first I came up with a complex scheme for sending data about "new" objects that the other side hadn't seen before. However, I think it makes more sense just to send this data every time we reference a given object. Sending a few extra bytes is cheap, especially compared to what we're saving here, which is an entire sync roundtrip. This reduces the number of CPOW IPC calls in a testcase I have from 19105 to 16837.
Attachment #8514713 -
Flags: review?(mrbkap)
Updated•10 years ago
|
Attachment #8514713 -
Flags: review?(mrbkap) → review+
Updated•10 years ago
|
tracking-e10s:
--- → m4+
Assignee | ||
Comment 1•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/5992aea01aff
Comment 2•10 years ago
|
||
sorry had to back this out in https://treeherder.mozilla.org/ui/#/jobs?repo=mozilla-inbound&revision=541dc2f509e2 since of this changes seems to have caused test failures like https://treeherder.mozilla.org/ui/logviewer.html#?job_id=3605040&repo=mozilla-inbound
Assignee | ||
Comment 3•10 years ago
|
||
The rooting hazards that this patch was implicated in did not go away after the backout. Relanding. https://hg.mozilla.org/integration/mozilla-inbound/rev/054f17d341c1
Assignee | ||
Comment 4•10 years ago
|
||
Apparently it was actually my fault, although the hazard is a false positive. Fixed. https://hg.mozilla.org/integration/mozilla-inbound/rev/37c92f53bfd2
Comment 5•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/054f17d341c1 https://hg.mozilla.org/mozilla-central/rev/37c92f53bfd2
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in
before you can comment on or make changes to this bug.
Description
•