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

Issue 297963003: PPAPI: Remove clang plugins from ppapi/tests/clang (Closed)

Created:
6 years, 7 months ago by dmichael (off chromium)
Modified:
6 years, 7 months ago
Reviewers:
teravest
CC:
chromium-reviews
Visibility:
Public.

Description

PPAPI: Remove clang plugins from ppapi/tests/clang We don't use these anymore, and they have bitrotted and probably don't even build with recent Clang. If we want them back, we can get them back out of SVN/git. R=teravest@chromium.org BUG= Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=272639

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -448 lines) Patch
D ppapi/tests/clang/Makefile View 1 chunk +0 lines, -67 lines 0 comments Download
D ppapi/tests/clang/README View 1 chunk +0 lines, -57 lines 0 comments Download
D ppapi/tests/clang/find_affected_interfaces.cc View 1 chunk +0 lines, -154 lines 0 comments Download
D ppapi/tests/clang/print_names_and_sizes.cc View 1 chunk +0 lines, -170 lines 0 comments Download

Messages

Total messages: 10 (0 generated)
dmichael (off chromium)
6 years, 7 months ago (2014-05-23 16:18:32 UTC) #1
teravest
lgtm
6 years, 7 months ago (2014-05-23 16:19:24 UTC) #2
dmichael (off chromium)
The CQ bit was checked by dmichael@chromium.org
6 years, 7 months ago (2014-05-23 16:43:22 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dmichael@chromium.org/297963003/1
6 years, 7 months ago (2014-05-23 16:44:54 UTC) #4
commit-bot: I haz the power
FYI, CQ is re-trying this CL (attempt #1). Please consider checking whether the failures are ...
6 years, 7 months ago (2014-05-23 18:24:19 UTC) #5
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 7 months ago (2014-05-23 20:22:29 UTC) #6
commit-bot: I haz the power
Try jobs failed on following builders: mac_chromium_rel on tryserver.chromium (http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/builds/32152)
6 years, 7 months ago (2014-05-23 20:22:29 UTC) #7
dmichael (off chromium)
The CQ bit was checked by dmichael@chromium.org
6 years, 7 months ago (2014-05-23 20:26:37 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dmichael@chromium.org/297963003/1
6 years, 7 months ago (2014-05-23 20:28:18 UTC) #9
commit-bot: I haz the power
6 years, 7 months ago (2014-05-23 23:48:41 UTC) #10
Message was sent while issue was closed.
Change committed as 272639

Powered by Google App Engine
This is Rietveld 408576698