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

Revert "[CSS Env Vars] Add some WPT tests" #11644

Merged
merged 1 commit into from Jun 24, 2018

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

This reverts commit 981a543f9e743001a6e492a006873de3a5953747.

Reason for revert:
These tests have been failing since they were added:
https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Linux%20Trusty%20Leak/20683

Original change's description:

[CSS Env Vars] Add some WPT tests

Adds some WPT tests to test env() with @supports as well
as fallback values.

BUG=825890

Change-Id: I071f5c2582f8056fe39b4a50ab62140d579c577d
Reviewed-on: https://chromium-review.googlesource.com/1111023
Reviewed-by: Rune Lillesveen futhark@chromium.org
Reviewed-by: Mounir Lamouri mlamouri@chromium.org
Commit-Queue: Becca Hughes beccahughes@chromium.org
Cr-Commit-Position: refs/heads/master@{#569852}

TBR=mlamouri@chromium.org,beccahughes@chromium.org,futhark@chromium.org

Change-Id: I32fbe3682558bdd2d117c8f647750e5d8bcbf740
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 825890
Reviewed-on: https://chromium-review.googlesource.com/1112982
Reviewed-by: Nico Weber thakis@chromium.org
Commit-Queue: Nico Weber thakis@chromium.org
Cr-Commit-Position: refs/heads/master@{#569916}

This reverts commit 981a543f9e743001a6e492a006873de3a5953747.

Reason for revert:
These tests have been failing since they were added:
https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Linux%20Trusty%20Leak/20683

Original change's description:
> [CSS Env Vars] Add some WPT tests
>
> Adds some WPT tests to test env() with @supports as well
> as fallback values.
>
> BUG=825890
>
> Change-Id: I071f5c2582f8056fe39b4a50ab62140d579c577d
> Reviewed-on: https://chromium-review.googlesource.com/1111023
> Reviewed-by: Rune Lillesveen <futhark@chromium.org>
> Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
> Commit-Queue: Becca Hughes <beccahughes@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#569852}

TBR=mlamouri@chromium.org,beccahughes@chromium.org,futhark@chromium.org

Change-Id: I32fbe3682558bdd2d117c8f647750e5d8bcbf740
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 825890
Reviewed-on: https://chromium-review.googlesource.com/1112982
Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569916}
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

Already reviewed downstream.

@chromium-wpt-export-bot chromium-wpt-export-bot merged commit 86e620c into master Jun 24, 2018
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-a3ef4dca85 branch June 24, 2018 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants