diff options
| author | Eugene Ostroukhov <eostroukhov@chromium.org> | 2016-11-10 01:19:35 +0300 |
|---|---|---|
| committer | Eugene Ostroukhov <eostroukhov@chromium.org> | 2016-12-07 01:47:49 +0300 |
| commit | 42da740ed83fc6ba285a056dc9c1764eedde4d6d (patch) | |
| tree | 3b6ae8fb3b6b958dc1931f48978a257cef9ecdff /src/inspector_agent.h | |
| parent | 8ab8363677c4f9440a3c9907eed55ab4a08f2936 (diff) | |
inspector: split HTTP/WS server from the inspector
Both our team experiments and some embedder request indicate a potential
in implementing alternative transport for inspector - e.g. IPC pipes or
custom embedder APIs. This change moves all HTTP specific code into a
separate class and is a first attempt at defining a boundary between the
inspector agent and transport. This API will be refined as new
transports are implemented.
Note that even without considering alternative transports, this change
enables better testing of the HTTP server (Valgrind made it possible to
identify and fix some existing memory leaks).
PR-URL: https://github.com/nodejs/node/pull/9630
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Diffstat (limited to 'src/inspector_agent.h')
| -rw-r--r-- | src/inspector_agent.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/inspector_agent.h b/src/inspector_agent.h index 3607cffba5d..b31c77496b3 100644 --- a/src/inspector_agent.h +++ b/src/inspector_agent.h @@ -1,6 +1,8 @@ #ifndef SRC_INSPECTOR_AGENT_H_ #define SRC_INSPECTOR_AGENT_H_ +#include <stddef.h> + #if !HAVE_INSPECTOR #error("This header can only be used when inspector is enabled") #endif @@ -36,7 +38,6 @@ class Agent { bool IsStarted(); bool IsConnected(); void WaitForDisconnect(); - void FatalException(v8::Local<v8::Value> error, v8::Local<v8::Message> message); private: |