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

Issue 2058943002: Remove //mojo/tests. (Closed)

Created:
4 years, 6 months ago by viettrungluu
Modified:
4 years, 6 months ago
Reviewers:
vardhan
CC:
mojo-reviews_chromium.org, gregsimon, qsr+mojo_chromium.org, viettrungluu+watch_chromium.org, abarth-chromium, Aaron Boodman, darin (slow to review), ben+mojo_chromium.org
Base URL:
https://github.com/domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

Remove //mojo/tests. It wasn't being built, and the only thing it has is a perftest for stuff that's already been deleted. R=vardhan@google.com Committed: https://chromium.googlesource.com/external/mojo/+/03c1866cce775b14196e73bfda49ce49afffe591

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -164 lines) Patch
D mojo/tests/BUILD.gn View 1 chunk +0 lines, -30 lines 0 comments Download
D mojo/tests/task_tracker_perftest.cc View 1 chunk +0 lines, -134 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
viettrungluu
4 years, 6 months ago (2016-06-10 21:54:02 UTC) #1
vardhan
lgtm
4 years, 6 months ago (2016-06-10 22:00:09 UTC) #2
viettrungluu
4 years, 6 months ago (2016-06-10 22:24:55 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
03c1866cce775b14196e73bfda49ce49afffe591 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698