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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 163883008: Revert of Cast: Re-enable End2EndTest.StartSenderBeforeReceiver (Closed)

Created:
6 years, 10 months ago by pkotwicz
Modified:
6 years, 10 months ago
Reviewers:
hubbe, miu
CC:
chromium-reviews, hclam+watch_chromium.org, mikhal+watch_chromium.org, hguihot+watch_chromium.org, jasonroberts+watch_google.com, pwestin+watch_google.com, feature-media-reviews_chromium.org, hubbe+watch_chromium.org, miu+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Revert of Cast: Re-enable End2EndTest.StartSenderBeforeReceiver (https://codereview.chromium.org/175213004/) Reason for revert: The test is failing on all of the windows bots on the Chrome waterfall. http://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/builds/27822/steps/cast_unittests/logs/StartSenderBeforeReceiver Original issue's description: > Cast: Re-enable End2EndTest.StartSenderBeforeReceiver > > I was not able to re-produce the non-determism. > > BUG=314233 > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=252761 TBR=miu@chromium.org,hubbe@chromium.org NOTREECHECKS=true NOTRY=true BUG=314233 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=252803

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -1 line) Patch
M media/cast/test/end2end_unittest.cc View 1 chunk +4 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
pkotwicz
Created Revert of Cast: Re-enable End2EndTest.StartSenderBeforeReceiver
6 years, 10 months ago (2014-02-22 16:57:39 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/163883008/1
6 years, 10 months ago (2014-02-22 16:57:50 UTC) #2
commit-bot: I haz the power
6 years, 10 months ago (2014-02-22 16:58:12 UTC) #3
Message was sent while issue was closed. 
Change committed as 252803

Powered by Google App Engine
This is Rietveld 408576698