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

Issue 2306233002: Revert of Add testing config for OverlayFullscreenVideo. (Closed)

Created:
4 years, 3 months ago by sandersd (OOO until July 31)
Modified:
4 years, 3 months ago
CC:
chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Add testing config for OverlayFullscreenVideo. (patchset #1 id:1 of https://codereview.chromium.org/2291413002/ ) Reason for revert: Turns out we shouldn't do this for optimization experiments, since we don't actually want this enabled for all dev builds. Original issue's description: > Add testing config for OverlayFullscreenVideo. > > BUG=642571 > > Committed: https://crrev.com/99fce27763b037227011e357502b1c700477d153 > Cr-Commit-Position: refs/heads/master@{#415825} TBR=asvitkine@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=642571 Committed: https://crrev.com/4e36c9cd407f42368f7d2f5df5806a33303187b8 Cr-Commit-Position: refs/heads/master@{#416397}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -8 lines) Patch
M testing/variations/fieldtrial_testing_config_android.json View 1 chunk +0 lines, -8 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
sandersd (OOO until July 31)
Created Revert of Add testing config for OverlayFullscreenVideo.
4 years, 3 months ago (2016-09-02 23:01:23 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/2306233002/1
4 years, 3 months ago (2016-09-02 23:02:14 UTC) #3
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 3 months ago (2016-09-02 23:51:31 UTC) #4
commit-bot: I haz the power
4 years, 3 months ago (2016-09-02 23:53:25 UTC) #6
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/4e36c9cd407f42368f7d2f5df5806a33303187b8
Cr-Commit-Position: refs/heads/master@{#416397}

Powered by Google App Engine
This is Rietveld 408576698