Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(210)

Issue 2287743002: Revert of Adding Site Isolation Win bot filtering for WindowOpenPanelTest.WindowOpenFocus. (Closed)

Created:
4 years, 3 months ago by Łukasz Anforowicz
Modified:
4 years, 3 months ago
Reviewers:
Charlie Reis
CC:
chromium-reviews, chromium-apps-reviews_chromium.org, extensions-reviews_chromium.org, alexmos, site-isolation-reviews_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Adding Site Isolation Win bot filtering for WindowOpenPanelTest.WindowOpenFocus. (patchset #1 id:1 of https://codereview.chromium.org/2269523003/ ) Reason for revert: The filter file is apparently not needed anymore: despite the fact that the test got renamed in r413759, we are not seeing failures in the WindowOpenFocus test in the latest builds on Site Isolation Win bot. Original issue's description: > Adding Site Isolation Win bot filtering for WindowOpenPanelTest.WindowOpenFocus. > > BUG=639510 > > Committed: https://crrev.com/a98073ac85514395939268f9d71a5d609867d450 > Cr-Commit-Position: refs/heads/master@{#413773} TBR=creis@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=639510 Committed: https://crrev.com/92c4a9fcfb32947cc40a9a4e3125d2e1992616f5 Cr-Commit-Position: refs/heads/master@{#414857}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -5 lines) Patch
M testing/buildbot/chromium.fyi.json View 1 chunk +1 line, -2 lines 0 comments Download
D testing/buildbot/filters/isolate-extensions.interactive_ui_tests.filter View 1 chunk +0 lines, -3 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
Łukasz Anforowicz
Created Revert of Adding Site Isolation Win bot filtering for WindowOpenPanelTest.WindowOpenFocus.
4 years, 3 months ago (2016-08-26 21:54:25 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2287743002/1
4 years, 3 months ago (2016-08-26 21:54:53 UTC) #3
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 3 months ago (2016-08-26 23:37:35 UTC) #4
commit-bot: I haz the power
4 years, 3 months ago (2016-08-26 23:39:11 UTC) #6
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/92c4a9fcfb32947cc40a9a4e3125d2e1992616f5
Cr-Commit-Position: refs/heads/master@{#414857}

Powered by Google App Engine
This is Rietveld 408576698