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

Issue 505001: Assigning bugs to excluded test groups. (Closed)

Created:
11 years ago by oshima
Modified:
9 years, 6 months ago
Reviewers:
viettrungluu
CC:
chromium-reviews_googlegroups.com, Timur Iskhodzhanov, stuartmorgan, pam+watch_chromium.org, dank, not_the_right_glider, Paweł Hajdan Jr., brettw+cc_chromium.org, jam, darin (slow to review), Erik does not do reviews, Aaron Boodman
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : " #

Patch Set 3 : " #

Total comments: 3

Patch Set 4 : " #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -8 lines) Patch
M chrome/test/data/valgrind/ui_tests.gtest.txt View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/data/valgrind/ui_tests.gtest_mac.txt View 1 2 3 1 chunk +6 lines, -3 lines 0 comments Download
M chrome/test/data/valgrind/unit_tests.gtest_mac.txt View 1 2 3 1 chunk +17 lines, -4 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
oshima
no change to tests. just updating comments.
11 years ago (2009-12-17 01:08:40 UTC) #1
viettrungluu
Other than nitpicking about (some other) comments, LGTM.
11 years ago (2009-12-17 01:32:23 UTC) #2
oshima
I don't see your comments. I guess you just did "reply" instead of sending comments? ...
11 years ago (2009-12-17 01:35:33 UTC) #3
viettrungluu
You were entirely correct. Sorry about that. http://codereview.chromium.org/505001/diff/2008/3002 File chrome/test/data/valgrind/ui_tests.gtest_mac.txt (right): http://codereview.chromium.org/505001/diff/2008/3002#newcode27 chrome/test/data/valgrind/ui_tests.gtest_mac.txt:27: # Following ...
11 years ago (2009-12-17 01:41:08 UTC) #4
oshima
all done. On 2009/12/17 01:41:08, viettrungluu wrote: > You were entirely correct. Sorry about that. ...
11 years ago (2009-12-17 01:51:18 UTC) #5
viettrungluu
11 years ago (2009-12-17 02:06:50 UTC) #6
Great. LGTM.

Powered by Google App Engine
This is Rietveld 408576698