DevTools: implement proxy server for hosted mode

third_party/WebKit/Source/devtools/hosted_mode/request.js

+// Copyright (c) 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+var http = require("http");
+var https = require("https");
+var parseUrl = require("url").parse;
+
+function request(url)

[lushnikov, 2016/07/22 21:06:04]

can we rename this into fetch? (to avoid conflicting with requestHandler)

[chenwilliam, 2016/07/22 23:20:31]

Done.

+{
+    return new Promise((resolve, reject) => {
+        var request;
+        var protocol = parseUrl(url).protocol;
+
+        if (protocol === "https:")
+            request = https.get(url, handleResponse);
+        else if (protocol === "http:")

[lushnikov, 2016/07/22 21:06:05]

You never have any HTTP urls - let's remove this

[chenwilliam, 2016/07/22 23:20:31]

Can't remove it because we make a request to http://localhost:9222/json/version
(doesn't work over https)

+            request = http.get(url, handleResponse);
+        else
+            reject(new Error(`Invalid protocol for url: ${url}`))

[lushnikov, 2016/07/22 21:06:04]

you want to return here as well, (otherwise, the request is undefined but used
below)

[chenwilliam, 2016/07/22 23:20:32]

Done.

+
+        function handleResponse(response)
+        {
+            if (response.statusCode !== 200) {
+                return reject(new Error(`Request error: + ${response.statusCode}`));

[lushnikov, 2016/07/22 21:06:04]

unfortunately, our style guide does not allow this - let's put return on a
separate line.

[chenwilliam, 2016/07/22 23:20:32]

Done.

+            }
+            var body = "";
+            response.on("data", chunk => body += chunk);
+            response.on("end", () => resolve(body));
+        }
+
+        request.on("error", (err) => reject(err));
+    });
+}
+
+module.exports = request;