DevTools: skip specific tests for backwards compatibility testing

DevTools: skip specific tests for backwards compatibility testing

Created a new TestExpectations file for the backwards compatibility tests
with specific reasons why certain types of tests are skipped.

BUG=682078
Review-Url: https://codereview.chromium.org/2661573002
Cr-Commit-Position: refs/heads/master@{#446826}
Committed: https://chromium.googlesource.com/chromium/src/+/45d7c03cf6c50bf18b9d518b42b76b3b47308682

[chenwilliam, 2017-01-27 02:34:52]

chenwilliam@chromium.org changed reviewers:
+ dgozman@chromium.org, pfeldman@chromium.org

[chenwilliam, 2017-01-27 02:34:53]

ptal. there's a few tests (~4) that still aren't running properly (crash /
timeout) but this addresses the vast majority of timeouts. I'll f/u with
rebaselining the remaining failing tests (over a few CLs).

[pfeldman, 2017-01-27 18:38:01]

https://codereview.chromium.org/2661573002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/tests/TestExpectations (right):

https://codereview.chromium.org/2661573002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/tests/TestExpectations:38: # Skipping async
await tests b/c 1.2 content shell (DevTools embedder) does not
Why is devtools embedder important? Is it rather runtime not supporting it?

[chenwilliam, 2017-01-27 20:00:26]

https://codereview.chromium.org/2661573002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/tests/TestExpectations (right):

https://codereview.chromium.org/2661573002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/tests/TestExpectations:38: # Skipping async
await tests b/c 1.2 content shell (DevTools embedder) does not
On 2017/01/27 at 18:38:00, pfeldman wrote:
> Why is devtools embedder important? Is it rather runtime not supporting it?

OK changed. That's what I meant.

[dgozman, 2017-01-27 20:06:47]

lgtm

https://codereview.chromium.org/2661573002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/tests/TestExpectations (right):

https://codereview.chromium.org/2661573002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/tests/TestExpectations:1: # Skipping service
worker tests because it's an experimental domain
I think this is due to similar reason as worker tests - we migrated from
ServiceWorker domain to Target domain.

[chenwilliam, 2017-01-27 21:53:33]

https://codereview.chromium.org/2661573002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/tests/TestExpectations (right):

https://codereview.chromium.org/2661573002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/tests/TestExpectations:1: # Skipping service
worker tests because it's an experimental domain
On 2017/01/27 20:06:47, dgozman wrote:
> I think this is due to similar reason as worker tests - we migrated from
> ServiceWorker domain to Target domain.

Done.

[chenwilliam, 2017-01-27 21:53:37]

The CQ bit was checked by chenwilliam@chromium.org

[chenwilliam, 2017-01-27 21:53:37]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2661573002/#ps40001
(title: "cl comment")

[commit-bot: I haz the power, 2017-01-27 21:54:30]

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

[chenwilliam, 2017-01-27 21:56:10]

The CQ bit was unchecked by chenwilliam@chromium.org

[chenwilliam, 2017-01-27 22:00:08]

The CQ bit was checked by chenwilliam@chromium.org

[chenwilliam, 2017-01-27 22:00:09]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2661573002/#ps60001
(title: "add skip expectations for inspector-unit")

[commit-bot: I haz the power, 2017-01-27 22:01:18]

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, 2017-01-27 23:34:13]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1485554408939490,
"parent_rev": "82d19a580c72707597f9e3e60d9f55f271c51317", "commit_rev":
"45d7c03cf6c50bf18b9d518b42b76b3b47308682"}

[commit-bot: I haz the power, 2017-01-27 23:34:44]

Description was changed from

==========
DevTools: skip specific tests for backwards compatibility testing

Created a new TestExpectations file for the backwards compatibility tests
with specific reasons why certain types of tests are skipped.

BUG=682078
==========

to

==========
DevTools: skip specific tests for backwards compatibility testing

Created a new TestExpectations file for the backwards compatibility tests
with specific reasons why certain types of tests are skipped.

BUG=682078

Review-Url: https://codereview.chromium.org/2661573002
Cr-Commit-Position: refs/heads/master@{#446826}
Committed:
https://chromium.googlesource.com/chromium/src/+/45d7c03cf6c50bf18b9d518b42b7...
==========

[commit-bot: I haz the power, 2017-01-27 23:34:45]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/45d7c03cf6c50bf18b9d518b42b7...