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

Issue 7552042: ui_tests is needed by chromium performance testers (Closed)

Created:
9 years, 4 months ago by M-A Ruel
Modified:
9 years, 4 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

ui_tests is needed by chromium performance testers TBR=chase@chromium.org BUG= TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=96185

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M build/all.gyp View 1 chunk +1 line, -0 lines 1 comment Download

Messages

Total messages: 4 (0 generated)
M-A Ruel
Linux Builder builds the target chromium_builder_perf. http://build.chromium.org/p/chromium.perf/builders/Linux%20Builder/builds/19946/steps/compile/logs/stdio ui_tests is not generated, not archived, http://build.chromium.org/p/chromium.perf/builders/Linux%20Builder/builds/19946/steps/package_build/logs/stdio ui_tests ...
9 years, 4 months ago (2011-08-10 15:30:17 UTC) #1
M-A Ruel
100% not tested by the way
9 years, 4 months ago (2011-08-10 15:30:40 UTC) #2
Mattias Nissler (ping if slow)
LGTM
9 years, 4 months ago (2011-08-10 15:35:25 UTC) #3
cmp
9 years, 4 months ago (2011-08-10 17:31:26 UTC) #4
lgtm with nit

http://codereview.chromium.org/7552042/diff/1/build/all.gyp
File build/all.gyp (right):

http://codereview.chromium.org/7552042/diff/1/build/all.gyp#newcode278
build/all.gyp:278: '../chrome/chrome.gyp:ui_tests',
btw, these are supposed to remain in alphabetic order

Powered by Google App Engine
This is Rietveld 408576698