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

Issue 11778010: Re-enable the messaging/connect extension API test. (Closed)

Created:
7 years, 11 months ago by not at google - send to devlin
Modified:
7 years, 11 months ago
Reviewers:
Yoyo Zhou
CC:
chromium-reviews, Aaron Boodman, chromium-apps-reviews_chromium.org
Visibility:
Public.

Description

Re-enable ExtensionApiTest.Messaging. It was disabled due to flakiness over a year ago, but it's our only test for some features of extension messaging, and we've recently been changing that. If ExtensionApiTest.Messaging starts failing, this will be why. However, don't get it confused with PlatformAppApiTest.Messaging. Those failures are unrelated. BUG=96725 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=175458

Patch Set 1 #

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

Messages

Total messages: 5 (0 generated)
not at google - send to devlin
I want to see if this is still flaky. Will monitor it as it goes.
7 years, 11 months ago (2013-01-07 19:26:18 UTC) #1
Yoyo Zhou
On 2013/01/07 19:26:18, kalman wrote: > I want to see if this is still flaky. ...
7 years, 11 months ago (2013-01-07 19:27:15 UTC) #2
not at google - send to devlin
heh sure.
7 years, 11 months ago (2013-01-07 19:27:57 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kalman@chromium.org/11778010/1
7 years, 11 months ago (2013-01-07 19:35:57 UTC) #4
commit-bot: I haz the power
7 years, 11 months ago (2013-01-07 21:42:22 UTC) #5
Retried try job too often on win_aura for step(s) interactive_ui_tests

Powered by Google App Engine
This is Rietveld 408576698