* Don't use JSON to send the result of `ipcRenderer.sendSync`.
- Change the return type of AtomViewHostMsg_Message_Sync from `base::string16`
to `base::ListValue`
- Adjust lib/browser/api/web-contents.js and /lib/renderer/api/ipc-renderer.js
to wrap/unwrap return values to/from array, instead of
serializing/deserializing JSON.
This change can greatly improve `ipcRenderer.sendSync` calls where the return
value contains Buffer instances, because those are converted to Array before
being serialized to JSON(which has no efficient way of representing byte
arrays).
A simple benchmark where remote.require('fs') was used to read a 16mb file got
at least 5x faster, not to mention it used a lot less memory. This difference
tends increases with larger buffers.
* Don't base64 encode Buffers
* Don't allocate V8ValueConverter on the heap
* Replace hidden global.sandbox with NodeBindings::IsInitialized()
* Refactoring: check NodeBindings::IsInitialized() in V8ValueConverter
* Refactor problematic test to make it more reliable
* Add tests for NaN and Infinity
* ensure all uv handles are closed before ending worker's loop
* add DCHECK to test that the Worker loop is finished
* don't call deprecated uv_loop_new(), uv_loop_delete()
* make cpplint happy
* fix comment error
* empty commit for CI
* tweak DCHECK expression
* extract-method: stop_and_close_uv_loop()
* fix DCHECK oops
Thiago de Arruda
Milan Burda
Jeremy Apthorp
Shelley Vohr
shelley vohr
Charles Kerr
Cheng Zhao
Kevin Sawicki
Paul Betts