add comments documenting the new callbacks

Author: petekanev

runtime/src/main/jni/NetworkDomainCallbackHandlers.h

@@ -17,6 +17,10 @@ namespace NetworkDomainCallbackHandlers {
 static const char* FrameId = "NSFrameIdentifier";
 static const char* LoaderId = "NSLoaderIdentifier";
 
+/*
+ * Fired when device is about to send HTTP request.
+ * @param args - JS object : { requestId, timestamp, type, response }
+ */
 static void ResponseReceivedCallback(const v8::FunctionCallbackInfo<v8::Value>& args) {
     try {
         auto networkAgentInstance = V8NetworkAgentImpl::Instance;
@@ -90,6 +94,10 @@ static void ResponseReceivedCallback(const v8::FunctionCallbackInfo<v8::Value>&
     }
 }
 
+/*
+ * Fired when HTTP response is available.
+ * @param args - JS object { requestId, url, request, timestamp, type }
+ */
 static void RequestWillBeSentCallback(const v8::FunctionCallbackInfo<v8::Value>& args) {
     try {
         auto networkAgentInstance = V8NetworkAgentImpl::Instance;
@@ -164,6 +172,10 @@ static void RequestWillBeSentCallback(const v8::FunctionCallbackInfo<v8::Value>&
     }
 }
 
+/*
+ * Called after response is received to save the response data for the provided requestId
+ * @param args - JS object { requestId, data, hasTextContent }
+ */
 static void DataForRequestId(const v8::FunctionCallbackInfo<v8::Value>& args) {
     try {
         auto networkAgentInstance = V8NetworkAgentImpl::Instance;
@@ -229,6 +241,10 @@ static void DataForRequestId(const v8::FunctionCallbackInfo<v8::Value>& args) {
     }
 }
 
+/*
+ * Fired when HTTP request has finished loading.
+ * @param args - JS object { requestId, timestamp }
+ */
 static void LoadingFinishedCallback(const v8::FunctionCallbackInfo<v8::Value>& args) {
     try {
         auto networkAgentInstance = V8NetworkAgentImpl::Instance;

runtime/src/main/jni/v8_inspector/src/inspector/v8-network-agent-impl.h

@@ -26,9 +26,24 @@ class V8NetworkAgentImpl : public protocol::Network::Backend {
         void enable(ErrorString*) override;
         void disable(ErrorString*) override;
 
+        /*
+         * NOT supported
+         */
         void setExtraHTTPHeaders(ErrorString*, std::unique_ptr<protocol::Network::Headers> in_headers) override;
+
+        /*
+         * Returns content served for the given request.
+         * The content of each response is stored in `m_responses` when a request completes.
+         * @param in_requestId
+         */
         void getResponseBody(ErrorString*, const String& in_requestId, String* out_body, bool* out_base64Encoded) override;
+        /*
+         * NOT supported
+         */
         void setCacheDisabled(ErrorString*, bool in_cacheDisabled) override;
+        /*
+         * NOT supported
+         */
         void loadResource(const String& in_frameId, const String& in_url, std::unique_ptr<LoadResourceCallback> callback) override;
 
         const bool enabled() {