[DevTools] Remove workspace-test.js part1.

[DevTools] Remove workspace-test.js part1.

This test harness is slowing us down, so we are migrating away from it.

BUG=none

Committed: https://crrev.com/2a2b2c36e69180578a1c8e5a4348977f553e9426
Cr-Commit-Position: refs/heads/master@{#434840}

[dgozman, 2016-11-29 00:11:04]

dgozman@chromium.org changed reviewers:
+ lushnikov@chromium.org

[dgozman, 2016-11-29 00:11:05]

Take a look please.

[lushnikov, 2016-11-29 00:27:21]

lgtm!

Didn't you forget to actually remove workspace-test.js?

https://codereview.chromium.org/2533073003/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/http/tests/inspector/inspector-test.js
(right):

https://codereview.chromium.org/2533073003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/inspector-test.js:445: {
capabilities = capabilities || Object.values(SDK.Target.Capability).reduce((a,b)
=> a | b)

This will allow you to drop the AllForTests constant

[dgozman, 2016-11-29 00:33:43]

> Didn't you forget to actually remove workspace-test.js?
I'm not done yet, hence the part1.

https://codereview.chromium.org/2533073003/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/http/tests/inspector/inspector-test.js
(right):

https://codereview.chromium.org/2533073003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/inspector-test.js:445: {
On 2016/11/29 00:27:21, lushnikov wrote:
> capabilities = capabilities ||
Object.values(SDK.Target.Capability).reduce((a,b)
> => a | b)
> 
> This will allow you to drop the AllForTests constant

I like the constant more :-)

[dgozman, 2016-11-29 00:33:49]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-11-29 00:34:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-29 02:41:51]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480379629364370, "parent_rev":
"de64de05df1202f7299306cb3056bfafb2fccaf3", "commit_rev":
"af922897b066e368c55400107d127eb61cfd9aa4"}

[commit-bot: I haz the power, 2016-11-29 02:42:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-29 02:44:27]

Description was changed from

==========
[DevTools] Remove workspace-test.js part1.

This test harness is slowing us down, so we are migrating away from it.

BUG=none
==========

to

==========
[DevTools] Remove workspace-test.js part1.

This test harness is slowing us down, so we are migrating away from it.

BUG=none

Committed: https://crrev.com/2a2b2c36e69180578a1c8e5a4348977f553e9426
Cr-Commit-Position: refs/heads/master@{#434840}
==========

[commit-bot: I haz the power, 2016-11-29 02:44:28]

Patchset 1 (id:??) landed as
https://crrev.com/2a2b2c36e69180578a1c8e5a4348977f553e9426
Cr-Commit-Position: refs/heads/master@{#434840}