Chromium Code Reviews
Help | Chromium Project | Sign in
(211)

Issue 3020005: Move tests for the layout test plugin to the right fixture. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 10 months ago by jochen
Modified:
4 years ago
CC:
chromium-reviews, Paweł Hajdan Jr.
Visibility:
Public.

Description

Move tests for the layout test plugin to the right fixture. BUG=30702 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=52669

Patch Set 1 #

Patch Set 2 : fix instead of flaky #

Unified diffs Side-by-side diffs Delta from patch set Stats (+62 lines, -55 lines) Patch
M chrome/test/ui/layout_plugin_uitest.cc View 2 chunks +61 lines, -0 lines 0 comments Download
M chrome/test/ui/npapi_uitest.cc View 1 3 chunks +1 line, -55 lines 0 comments Download
Commit: CQ not working?

Messages

Total messages: 8 (0 generated)
jochen
This test fails on my on a fresh build of Chrome w/o patches on Windows. ...
4 years, 10 months ago (2010-07-16 07:33:55 UTC) #1
Bernhard Bauer
On 2010/07/16 07:33:55, jochen wrote: > This test fails on my on a fresh build ...
4 years, 10 months ago (2010-07-16 09:42:56 UTC) #2
jochen
I had a closer look at it and it turns out these tests cannot ever ...
4 years, 10 months ago (2010-07-16 11:09:16 UTC) #3
Bernhard Bauer
On 2010/07/16 11:09:16, jochen wrote: > I had a closer look at it and it ...
4 years, 10 months ago (2010-07-16 11:58:38 UTC) #4
jochen
Evan, seems on Linux, the trybots can't load the layout plugin (the error is that ...
4 years, 10 months ago (2010-07-16 14:14:37 UTC) #5
Evan Martin
I didn't change this code recently -- why'd it break?
4 years, 10 months ago (2010-07-16 14:49:50 UTC) #6
jochen
On 2010/07/16 14:14:37, jochen wrote: > Evan, seems on Linux, the trybots can't load the ...
4 years, 10 months ago (2010-07-16 14:57:19 UTC) #7
jochen
4 years, 10 months ago (2010-07-16 15:40:11 UTC) #8
ok, when i compile statically on linux, it works. i've marked the
selfdeleteplugin test as flaky for now, because it doesn't work on linux
(shlib). I check on monday whether after r52657 this problem is fixed and enable
it again.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld ec887be