Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(321)

Unified Diff: third_party/WebKit/Tools/Scripts/run-blink-websocketserver

Issue 2719633002: Add descriptions with examples to run-blink-* scripts. (Closed)
Patch Set: Add docs for other scripts Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Tools/Scripts/run-blink-websocketserver
diff --git a/third_party/WebKit/Tools/Scripts/run-blink-websocketserver b/third_party/WebKit/Tools/Scripts/run-blink-websocketserver
index 4053e8b89d0a839b7afa55a81f5ac4a45424bdd9..85dcf3511e98ccf21b0baa39c519e3301dbc169b 100755
--- a/third_party/WebKit/Tools/Scripts/run-blink-websocketserver
+++ b/third_party/WebKit/Tools/Scripts/run-blink-websocketserver
@@ -27,9 +27,20 @@
# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+"""Runs a WebSocket server for WebSocket tests.
+
+Some tests require both an HTTP server and WebSocket server. You can start
+both servers by running both run-blink-httpd and run-blink-websocketserver.
+
+Tests served by the HTTP server have paths relative to LayoutTests/http/tests/.
+For example, to run a test http/tests/websocket/binary-type.html which depends
+on WebSocket, you can navigate to:
+ http://localhost:8000/websocket/close-unref-websocket.html
dcheng 2017/02/25 00:06:24 I'm not as familiar with these, but maybe this sho
qyearsley 2017/02/26 19:21:44 Done
+"""
+
import webkitpy.common.version_check
from webkitpy.layout_tests.servers import cli_wrapper
from webkitpy.layout_tests.servers import pywebsocket
-cli_wrapper.main(pywebsocket.PyWebSocket)
+cli_wrapper.main(pywebsocket.PyWebSocket, description)

Powered by Google App Engine
This is Rietveld 408576698