1817041 - Make JSValidatorParent to send the data over in OnStopRequest rather than OnDataAvailable

Status: NEW

(Core :: JavaScript Engine, task, P2)

Description

[Sean Feng [:sefeng]]

This way, we only need to create Shmem once and avoid unnecessary IPC message for OnDataAvailable between JSVadliatorParent and JSValidatorChild.

Comment 1

[Sean Feng [:sefeng]]

Attachment: Bug 1817041 - Tweak the data sharing code for ORB r=farre,smaug

This patch has two changes

• Instead of sending the data over from parent to utility in
  OnDataAvailable, we cache in JSValidatorParent and only
  send it over once in OnStopRequest.

  • So we only need to create the shared memory once and sending
    the data once, should be an improvement.
  • This also helps bug 1804086 to only decoding data once.

• After the validation, reuse the same data that JSValidatorParent
  caches, instead of using whatever the utility process send over.

  • This change is needed because we are ensuring utility process
    only receives UTF8 encoded data in bug 1804086, so the content
    it receives can be modified and we don't want to use modified
    data after the JavaScript validation.
  • This also removes one shared memory allocation, should
    also be an improvement.