1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

inspector: fixed V8InspectorClient::currentTimeMS

Browse Source 
On inspector side inside V8 we assume that this method should
return number of ms since epoch.

PR-URL: https://github.com/nodejs/node/pull/21917
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
This commit is contained in:
Aleksey Kozyatinskiy 2018-07-21 00:52:48 -07:00
parent fc6f49a704
commit 0ed164e082
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/inspector_agent.cc
View File

@ -187,8 +187,6 @@ static int StartDebugSignalHandler() {
#endif // _WIN32 #endif // _WIN32
// Used in NodeInspectorClient::currentTimeMS() below.
const int NANOS_PER_MSEC = 1000000;
const int CONTEXT_GROUP_ID = 1; const int CONTEXT_GROUP_ID = 1;
class ChannelImpl final : public v8_inspector::V8Inspector::Channel, class ChannelImpl final : public v8_inspector::V8Inspector::Channel,
@ -593,7 +591,7 @@ class NodeInspectorClient : public V8InspectorClient {
} }
double currentTimeMS() override { double currentTimeMS() override {
return uv_hrtime() * 1.0 / NANOS_PER_MSEC; return env_->isolate_data()->platform()->CurrentClockTimeMillis();
} }
node::Environment* env_; node::Environment* env_;