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

Issue 2812037: Adding FAILS_ for failing worker tests (Closed)

Created:
10 years, 5 months ago by kinuko
Modified:
9 years, 7 months ago
CC:
chromium-reviews, jam, Paweł Hajdan Jr.
Visibility:
Public.

Description

Adding FAILS_ for failing worker tests BUG=48082 TBR=yurys Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=51378

Patch Set 1 #

Patch Set 2 : remove WorkerMessagePort #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M chrome/worker/worker_uitest.cc View 1 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Andrew T Wilson (Slow)
It looks like the webkit roll causing this was rolled back. Can you roll this ...
10 years, 5 months ago (2010-07-01 18:45:26 UTC) #1
kinuko
10 years, 5 months ago (2010-07-01 19:13:38 UTC) #2
Yep, it was reverted after the rollback.  Thanks for the headsup.

http://codereview.chromium.org/2878019

On Thu, Jul 1, 2010 at 11:45 AM, <atwilson@chromium.org> wrote:

> It looks like the webkit roll causing this was rolled back. Can you roll
> this
> back also?
>
> http://codereview.chromium.org/2812037/show
>

Powered by Google App Engine
This is Rietveld 408576698