In wpt-update-expectations, ignore tests that unexpectedly pass.

In wpt-update-expectations, ignore tests that unexpectedly pass.

In general, when the auto-importer is updating expectations and baselines, if a test unexpectedly passes, then it should be OK to just leave it alone and not update anything.

If the pass was a flaky pass, then the previous behavior would have been to try to rebaseline it, which would remove the expectation, which would later cause trouble if the test is flaky.

BUG=687442
Review-Url: https://codereview.chromium.org/2750273003
Cr-Commit-Position: refs/heads/master@{#457647}
Committed: https://chromium.googlesource.com/chromium/src/+/fb5a41a23dda1f926f1d724650ad63d22bb823ac

[qyearsley, 2017-03-16 23:53:05]

Description was changed from

==========
In wpt-update-expectations, ignore tests that unexpectedly pass.

BUG=687442
==========

to

==========
In wpt-update-expectations, ignore tests that unexpectedly pass.

In general, when the auto-importer is updating expectations and baselines, if a
test unexpectedly passes, then it should be OK to just leave it alone and not
update anything.

If the pass was a flaky pass, then the previous behavior would have been to try
to rebaseline it, which would remove the expectation, which would later cause
trouble if the test is flaky.

BUG=687442
==========

[qyearsley, 2017-03-16 23:53:05]

qyearsley@chromium.org changed reviewers:
+ tkent@chromium.org

[tkent, 2017-03-16 23:55:39]

lgtm

[qyearsley, 2017-03-17 00:10:52]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2017-03-17 00:11:39]

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-03-17 01:29:57]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489709452733970, "parent_rev":
"386d195b0c127db5f738102781d92c163ac7f9b4", "commit_rev":
"fb5a41a23dda1f926f1d724650ad63d22bb823ac"}

[commit-bot: I haz the power, 2017-03-17 01:30:44]

Description was changed from

==========
In wpt-update-expectations, ignore tests that unexpectedly pass.

In general, when the auto-importer is updating expectations and baselines, if a
test unexpectedly passes, then it should be OK to just leave it alone and not
update anything.

If the pass was a flaky pass, then the previous behavior would have been to try
to rebaseline it, which would remove the expectation, which would later cause
trouble if the test is flaky.

BUG=687442
==========

to

==========
In wpt-update-expectations, ignore tests that unexpectedly pass.

In general, when the auto-importer is updating expectations and baselines, if a
test unexpectedly passes, then it should be OK to just leave it alone and not
update anything.

If the pass was a flaky pass, then the previous behavior would have been to try
to rebaseline it, which would remove the expectation, which would later cause
trouble if the test is flaky.

BUG=687442

Review-Url: https://codereview.chromium.org/2750273003
Cr-Commit-Position: refs/heads/master@{#457647}
Committed:
https://chromium.googlesource.com/chromium/src/+/fb5a41a23dda1f926f1d724650ad...
==========

[commit-bot: I haz the power, 2017-03-17 01:30:44]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/fb5a41a23dda1f926f1d724650ad...