delta/node-new.git/src/node_platform.cc, branch try_Python3_on_macOS

src: unimplement deprecated v8-platform methods

2019-05-30T13:51:37+00:00

This removes the implementations of NodePlatform::CallOnForegroundThread
and NodePlatform::CallDelayedOnForegroundThread and updates the
test_platform cctest to stop using them.

PR-URL: https://github.com/nodejs/node/pull/27872
Reviewed-By: Gus Caplan 
Reviewed-By: Refael Ackermann 
Reviewed-By: Rich Trott 
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig

src: remove util-inl.h from header files

2019-05-13T20:43:26+00:00

Its intended that *-inl.h header files are only included into the src
files that call the inline methods. Explicitly include it into the files
that need it.

PR-URL: https://github.com/nodejs/node/pull/27631
Reviewed-By: Joyee Cheung 
Reviewed-By: James M Snell 
Reviewed-By: Rich Trott 
Reviewed-By: Colin Ihrig

src: enable non-nestable V8 platform tasks

2019-04-18T10:50:52+00:00

We never execute tasks in a nested fashion, so enabling them should
be as simple as forwarding tasks to the existing `Post*` methods.

PR-URL: https://github.com/nodejs/node/pull/27252
Reviewed-By: Ben Noordhuis 
Reviewed-By: James M Snell 
Reviewed-By: Daniel Bevenius

src: apply clang-tidy rule bugprone-incorrect-roundings

2019-04-03T16:19:44+00:00

PR-URL: https://github.com/nodejs/node/pull/26885
Reviewed-By: James M Snell 
Reviewed-By: Minwoo Jung 
Reviewed-By: Refael Ackermann

src: elevate v8::Task namespace

2019-03-30T12:17:43+00:00

PR-URL: https://github.com/nodejs/node/pull/26909
Reviewed-By: Michaël Zasso 
Reviewed-By: Colin Ihrig 
Reviewed-By: Ruben Bridgewater

src: clean up MultiIsolatePlatform interface

2019-03-13T00:12:56+00:00

- Since this was introduced, V8 has effectively started requiring
  that the platform knows of the `Isolate*` before we (or an embedder)
  create our `IsolateData` structure; therefore, (un)registering it
  from the `IsolateData` constructor/destructor doesn’t make much
  sense anymore.
- Instead, we can require that the register/unregister functions
  are only called once, simplifying the implementation a bit.
- Add a callback that we can use to know when the platform has
  cleaned up its resources associated with a given `Isolate`.
  In particular, this means that in the Worker code, we don’t need
  to rely on what are essentially guesses about the number of event
  loop turns that we need in order to have everything cleaned up.

PR-URL: https://github.com/nodejs/node/pull/26384
Reviewed-By: Daniel Bevenius 
Reviewed-By: James M Snell 
Reviewed-By: Matteo Collina 
Reviewed-By: Joyee Cheung

src: apply clang-tidy rule modernize-make-unique

2019-03-12T19:34:36+00:00

PR-URL: https://github.com/nodejs/node/pull/26493
Reviewed-By: Anna Henningsen 
Reviewed-By: Refael Ackermann 
Reviewed-By: Ben Noordhuis

src: merge debug-only `SealHandleScope`s

2019-03-08T23:27:29+00:00

Instead of repeating the same `#ifdef DEBUG` + `SealHandleScope`
pattern over and over, create an utility that does this for us.

PR-URL: https://github.com/nodejs/node/pull/26459
Reviewed-By: Ben Noordhuis 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Colin Ihrig 
Reviewed-By: Joyee Cheung

src: forbid handle allocations from Platform tasks

2019-03-05T21:40:04+00:00

Platform tasks should have their own handle scopes, rather than
leak into outer ones.

PR-URL: https://github.com/nodejs/node/pull/26376
Reviewed-By: Gireesh Punathil 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell

src: allow running tasks without `Environment`

2019-03-05T21:40:02+00:00

There is no real reason to assume that V8 tasks would have
to run in a Node.js `Context`.

PR-URL: https://github.com/nodejs/node/pull/26376
Reviewed-By: Gireesh Punathil 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell