Chromium Code Reviews

Issue 2358943002: [inspector] added inspector protocol test runner (Closed)

Created:
4 years, 3 months ago by kozy
Modified:
4 years, 2 months ago
Reviewers:
alph, dgozman
CC:
v8-reviews_googlegroups.com
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

[inspector] added inspector protocol test runner To run tests: tools/run-tests.py inspector-protocol --shell-dir=out/Release BUG=chromium:645640 R=dgozman@chromium.org,alph@chromium.org

Patch Set 1 #

Patch Set 2 : a #

Patch Set 3 : fix build without inspector #

Patch Set 4 : more DISALLOW_COPY_AND_ASSIGN #

Patch Set 5 : removed most tests for now #

Patch Set 6 : a #

Total comments: 3
Unified diffs Side-by-side diffs Stats (+1739 lines, --1 lines)
M test/BUILD.gn View 2 chunks +5 lines, -0 lines 0 comments
A test/inspector-protocol/BUILD.gn View 1 chunk +53 lines, -0 lines 0 comments
A test/inspector-protocol/DEPS View 1 chunk +11 lines, -0 lines 0 comments
A test/inspector-protocol/OWNERS View 1 chunk +9 lines, -0 lines 0 comments
A test/inspector-protocol/channel-impl.h View 1 chunk +38 lines, -0 lines 0 comments
A test/inspector-protocol/channel-impl.cc View 1 chunk +51 lines, -0 lines 1 comment
A test/inspector-protocol/cpu-profiler/record-cpu-profile.js View 1 chunk +48 lines, -0 lines 0 comments
A test/inspector-protocol/cpu-profiler/record-cpu-profile-expected.txt View 1 chunk +7 lines, -0 lines 0 comments
A test/inspector-protocol/debugger/stepping-with-blackboxed-ranges.js View 1 chunk +126 lines, -0 lines 0 comments
A test/inspector-protocol/debugger/stepping-with-blackboxed-ranges-expected.txt View 1 chunk +54 lines, -0 lines 0 comments
A test/inspector-protocol/inspector-client-impl.h View 1 chunk +67 lines, -0 lines 0 comments
A test/inspector-protocol/inspector-client-impl.cc View 1 chunk +90 lines, -0 lines 0 comments
A test/inspector-protocol/inspector-extension.h View 1 chunk +54 lines, -0 lines 0 comments
A test/inspector-protocol/inspector-extension.cc View 1 chunk +70 lines, -0 lines 0 comments
A test/inspector-protocol/inspector-protocol.cc View 1 chunk +84 lines, -0 lines 0 comments
A test/inspector-protocol/inspector-protocol.gyp View 1 chunk +56 lines, -0 lines 0 comments
A + test/inspector-protocol/inspector-protocol.status View 0 chunks +-1 lines, --1 lines 0 comments
A test/inspector-protocol/json-parse.js View 1 chunk +13 lines, -0 lines 0 comments
A test/inspector-protocol/json-parse-expected.txt View 1 chunk +10 lines, -0 lines 0 comments
A test/inspector-protocol/load-extension.h View 1 chunk +28 lines, -0 lines 1 comment
A test/inspector-protocol/load-extension.cc View 1 chunk +57 lines, -0 lines 0 comments
A test/inspector-protocol/protocol-test.js View 1 chunk +209 lines, -0 lines 0 comments
A test/inspector-protocol/set-timeout-extension.h View 1 chunk +29 lines, -0 lines 0 comments
A test/inspector-protocol/set-timeout-extension.cc View 1 chunk +59 lines, -0 lines 0 comments
A test/inspector-protocol/task.h View 1 chunk +85 lines, -0 lines 0 comments
A test/inspector-protocol/task.cc View 1 chunk +76 lines, -0 lines 0 comments
A test/inspector-protocol/task-queue.h View 1 chunk +38 lines, -0 lines 0 comments
A test/inspector-protocol/task-queue.cc View 1 chunk +51 lines, -0 lines 0 comments
A test/inspector-protocol/task-runner.h View 1 chunk +45 lines, -0 lines 0 comments
A test/inspector-protocol/task-runner.cc View 1 chunk +42 lines, -0 lines 0 comments
A test/inspector-protocol/testcfg.py View 1 chunk +77 lines, -0 lines 1 comment
A test/inspector-protocol/utils-extension.h View 1 chunk +27 lines, -0 lines 0 comments
A test/inspector-protocol/utils-extension.cc View 1 chunk +67 lines, -0 lines 0 comments
M tools/verify_source_deps.py View 4 chunks +4 lines, -0 lines 0 comments

Messages

Total messages: 12 (9 generated)
kozy
Dmitry, please take a look. Roadmap: - improve gyp and gn files for inspector-protocol test ...
4 years, 3 months ago (2016-09-21 19:10:00 UTC) #2
dgozman
Let's split this into three parts: - task handling with task runner, queue and task; ...
4 years, 3 months ago (2016-09-22 17:44:49 UTC) #11
kozy
4 years, 2 months ago (2016-09-30 02:14:04 UTC) #12

Powered by Google App Engine