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

Issue 10616: Roll back Rietveld issue 8778 (r5183, appending stderr to layout-test output)... (Closed)

Created:
12 years, 1 month ago by Pam (message me for reviews)
Modified:
9 years, 5 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Roll back Rietveld issue 8778 (r5183, appending stderr to layout-test output) to see if it makes the layout tests more stable. TBR=pinkerton BUG=none TEST=covered by layout tests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=5229

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -244 lines) Patch
M webkit/tools/layout_tests/layout_package/test_shell_thread.py View 2 chunks +1 line, -19 lines 0 comments Download
M webkit/tools/layout_tests/test_lists/win/tests_fixable.txt View 2 chunks +0 lines, -189 lines 0 comments Download
M webkit/tools/test_shell/mac/test_webview_delegate.mm View 2 chunks +2 lines, -14 lines 0 comments Download
M webkit/tools/test_shell/test_shell_win.cc View 2 chunks +3 lines, -8 lines 0 comments Download
M webkit/tools/test_shell/test_webview_delegate.cc View 2 chunks +2 lines, -14 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Pam (message me for reviews)
12 years, 1 month ago (2008-11-11 23:48:35 UTC) #1
pink (ping after 24hrs)
12 years, 1 month ago (2008-11-12 14:09:19 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698