Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update [a-f]* from monorail results #809

Merged
merged 4 commits into from
Feb 3, 2021
Merged

Update [a-f]* from monorail results #809

merged 4 commits into from
Feb 3, 2021

Conversation

stephenmcgruer
Copy link
Contributor

See #798 for methodology

@stephenmcgruer stephenmcgruer added the do not merge yet Disable auto-merge label Feb 1, 2021
ambient-light/META.yml Outdated Show resolved Hide resolved
audio-output/META.yml Outdated Show resolved Hide resolved
background-fetch/META.yml Show resolved Hide resolved
bluetooth/idl/META.yml Outdated Show resolved Hide resolved
bluetooth/requestDevice/META.yml Outdated Show resolved Hide resolved
feature-policy/experimental-features/META.yml Outdated Show resolved Hide resolved
feature-policy/experimental-features/META.yml Outdated Show resolved Hide resolved
feature-policy/reporting/META.yml Outdated Show resolved Hide resolved
@@ -0,0 +1,5 @@
links:
- product: chrome
url: https://crbug.com/1033852
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is probably unrelated, but leaving for now

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://bugs.chromium.org/p/chromium/issues/detail?id=836697 would be the one to link here. I'll do just that. Linking this one too seems fine, but not super useful for the purposes of "what should we fix to improve interop?"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

fetch/api/policies/META.yml Outdated Show resolved Hide resolved
@stephenmcgruer
Copy link
Contributor Author

@foolip - PTAL.

This was a rough one for the methodology - I think I reverted like 75%+ of the added links -_-. So many meta tracking bugs and random CLs touching the tests.

@stephenmcgruer stephenmcgruer changed the title Update [a-m]* from monorail results Update [a-f]* from monorail results Feb 1, 2021
- product: chrome
url: https://crbug.com/1168974
results:
- test: delete.html?1-1000
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

None of the failures in this subset seem to be about tables. Can you trim this list to just the ones listed in the bug description?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@@ -0,0 +1,5 @@
links:
- product: chrome
url: https://crbug.com/1033852
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://bugs.chromium.org/p/chromium/issues/detail?id=836697 would be the one to link here. I'll do just that. Linking this one too seems fine, but not super useful for the purposes of "what should we fix to improve interop?"

@foolip foolip merged commit 35069bd into master Feb 3, 2021
@foolip foolip deleted the smcgruer/a-m-monorail branch February 3, 2021 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge yet Disable auto-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants