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

Issue 586133002: Disable MediaGalleriesPlatformAppBrowserTest.MediaGalleriesRead as its flaking. (Closed)

Created:
6 years, 3 months ago by jam
Modified:
6 years, 3 months ago
Reviewers:
Lei Zhang
CC:
chromium-reviews, extensions-reviews_chromium.org, vandebo (ex-Chrome), Lei Zhang, tommycli, Greg Billock, chromium-apps-reviews_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Disable MediaGalleriesPlatformAppBrowserTest.MediaGalleriesRead as its flaking. BUG=415792 TBR=thestig@chromium.org Committed: https://crrev.com/bbc867aed92512d525f7115bef77bab74b41079d Cr-Commit-Position: refs/heads/master@{#295845}

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -6 lines) Patch
M chrome/browser/extensions/api/media_galleries/media_galleries_apitest.cc View 1 1 chunk +1 line, -6 lines 0 comments Download

Messages

Total messages: 8 (1 generated)
jam
6 years, 3 months ago (2014-09-19 20:38:13 UTC) #1
vandebo (ex-Chrome)
6 years, 3 months ago (2014-09-19 20:48:18 UTC) #3
Lei Zhang
This has already happened in another CL: https://codereview.chromium.org/588663003
6 years, 3 months ago (2014-09-19 21:02:15 UTC) #4
jam
On 2014/09/19 21:02:15, Lei Zhang wrote: > This has already happened in another CL: > ...
6 years, 3 months ago (2014-09-20 02:13:44 UTC) #5
jam
Committed patchset #2 (id:20001) manually as bbc867a (presubmit successful).
6 years, 3 months ago (2014-09-20 02:17:26 UTC) #6
commit-bot: I haz the power
Patchset 2 (id:??) landed as https://crrev.com/bbc867aed92512d525f7115bef77bab74b41079d Cr-Commit-Position: refs/heads/master@{#295845}
6 years, 3 months ago (2014-09-20 02:17:27 UTC) #7
Lei Zhang
6 years, 3 months ago (2014-09-20 02:17:49 UTC) #8
Message was sent while issue was closed.
Ah, lgtm then

Powered by Google App Engine
This is Rietveld 408576698