Closed Bug 1091964 Opened 5 years ago Closed 5 years ago
[e10s] Don't send messages for is
Callable and is Constructor CPOW hooks
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)
Attachment #8514713 - Flags: review?(mrbkap) → review+
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
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
Apparently it was actually my fault, although the hazard is a false positive. Fixed. https://hg.mozilla.org/integration/mozilla-inbound/rev/37c92f53bfd2
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.