Initial TestExpectations vs. WPTServeExpectations cleanup

Initial TestExpectations vs. WPTServeExpectations cleanup

Now that --enable-wptserve is on by default we don't need
an override expectations file.

BUG=666111
R=tkent@chromium.org

Committed: https://crrev.com/e831429bc4c93f3ff505ce56f1835babac6eef9d
Cr-Commit-Position: refs/heads/master@{#433041}

[jsbell, 2016-11-17 00:17:44]

tkent@ - I had this underway when I saw your note on crbug.com/666111 about
picking this up.

It might be useful to take this CL as the basis for your change, or you may be
ahead of me already. I'll hold off on doing anything more here.

[tkent, 2016-11-17 00:24:25]

On 2016/11/17 at 00:17:44, jsbell wrote:
> tkent@ - I had this underway when I saw your note on crbug.com/666111 about
picking this up.
> 
> It might be useful to take this CL as the basis for your change, or you may be
ahead of me already. I'll hold off on doing anything more here.

I haven't started it yet.  So, please go ahead :)

[tkent, 2016-11-17 00:25:10]

lgtm

https://codereview.chromium.org/2509753003/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/WPTServeExpectations (left):

https://codereview.chromium.org/2509753003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/WPTServeExpectations:1: # Overriden expectations
for imported tests when run with --enable-wptserve
Is it safe to remove this file without removing the code to read
WPTServeExpectation?

[jsbell, 2016-11-17 00:26:16]

On 2016/11/17 00:25:10, tkent wrote:
> lgtm
> 
>
https://codereview.chromium.org/2509753003/diff/1/third_party/WebKit/LayoutTe...
> File third_party/WebKit/LayoutTests/WPTServeExpectations (left):
> 
>
https://codereview.chromium.org/2509753003/diff/1/third_party/WebKit/LayoutTe...
> third_party/WebKit/LayoutTests/WPTServeExpectations:1: # Overriden
expectations
> for imported tests when run with --enable-wptserve
> Is it safe to remove this file without removing the code to read
> WPTServeExpectation?

It didn't fail - certainly we should remove the code that attempts to use it.
I'll put a dummy file in for now.

[jsbell, 2016-11-17 00:30:08]

Description was changed from

==========
Initial TestExpectations vs. WPTServeExpectations cleanup

BUG=666111
R=tkent@chromium.org
==========

to

==========
Initial TestExpectations vs. WPTServeExpectations cleanup

Now that --enable-wptserve is on by default we don't need
an override expectations file.

BUG=666111
R=tkent@chromium.org
==========

[jsbell, 2016-11-17 00:30:15]

The CQ bit was checked by jsbell@chromium.org

[jsbell, 2016-11-17 00:30:15]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2509753003/#ps20001
(title: "Add dummy WPTServeExpectations")

[commit-bot: I haz the power, 2016-11-17 00:33:44]

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-17 00:39:28]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-17 00:39:29]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[jsbell, 2016-11-17 01:08:08]

The CQ bit was checked by jsbell@chromium.org

[jsbell, 2016-11-17 01:08:09]

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

[commit-bot: I haz the power, 2016-11-17 01:09:08]

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-17 02:51:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-17 02:51:42]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[jsbell, 2016-11-17 04:25:24]

On 2016/11/17 02:51:42, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

Looks like these fail with output that's platform specific, so we should mark
[Failure] for now:

imported/wpt/service-workers/service-worker/fetch-header-visibility.https.html
imported/wpt/service-workers/service-worker/request-end-to-end.https.html

And this times out:

imported/wpt/html/browsers/origin/cross-origin-objects/cross-origin-objects.html

tkent@ - I'm offline now; I'll poke at this tomorrow (Pacific Time), but if this
will unblock you please feel free to take over this CL.

[jsbell, 2016-11-17 20:39:43]

The CQ bit was checked by jsbell@chromium.org

[jsbell, 2016-11-17 20:39:44]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2509753003/#ps100001
(title: "Add expectation for cross-origin-objects.html")

[commit-bot: I haz the power, 2016-11-17 20:40:45]

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-17 21:52:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-17 21:52:23]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[jsbell, 2016-11-17 23:34:32]

The CQ bit was checked by jsbell@chromium.org

[commit-bot: I haz the power, 2016-11-17 23:37:10]

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-18 00:57:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-18 00:58:00]

Failed to apply the patch.

[jsbell, 2016-11-18 01:11:12]

The CQ bit was checked by jsbell@chromium.org

[jsbell, 2016-11-18 01:11:13]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2509753003/#ps120001
(title: "rebased")

[commit-bot: I haz the power, 2016-11-18 01:12:52]

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-18 03:07:44]

Description was changed from

==========
Initial TestExpectations vs. WPTServeExpectations cleanup

Now that --enable-wptserve is on by default we don't need
an override expectations file.

BUG=666111
R=tkent@chromium.org
==========

to

==========
Initial TestExpectations vs. WPTServeExpectations cleanup

Now that --enable-wptserve is on by default we don't need
an override expectations file.

BUG=666111
R=tkent@chromium.org
==========

[commit-bot: I haz the power, 2016-11-18 03:07:45]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-11-18 03:10:55]

Description was changed from

==========
Initial TestExpectations vs. WPTServeExpectations cleanup

Now that --enable-wptserve is on by default we don't need
an override expectations file.

BUG=666111
R=tkent@chromium.org
==========

to

==========
Initial TestExpectations vs. WPTServeExpectations cleanup

Now that --enable-wptserve is on by default we don't need
an override expectations file.

BUG=666111
R=tkent@chromium.org

Committed: https://crrev.com/e831429bc4c93f3ff505ce56f1835babac6eef9d
Cr-Commit-Position: refs/heads/master@{#433041}
==========

[commit-bot: I haz the power, 2016-11-18 03:10:57]

Patchset 7 (id:??) landed as
https://crrev.com/e831429bc4c93f3ff505ce56f1835babac6eef9d
Cr-Commit-Position: refs/heads/master@{#433041}