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

Issue 2445323002: Ozone: Remove the caca platform from the tree (Closed)

Created:
4 years, 1 month ago by fwang
Modified:
4 years, 1 month ago
Reviewers:
rjkroege
CC:
chromium-reviews, kalyank, ozone-reviews_chromium.org, tonikitoo, spang, dnicoara
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Ozone: Remove the caca platform from the tree This Ozone platform is not built by default, is not handled by the try bots, does not have the required dependencies installed in the sysroot and does not have any unit test. It is also really only useful as a demo so instead of maintaining it, we can just update the documentation to explain how to build the latest revision where it is known to work. BUG=None Committed: https://crrev.com/e7dc921a4088b26b8108032968e1c11819895e3e Cr-Commit-Position: refs/heads/master@{#427635}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -981 lines) Patch
M ui/ozone/BUILD.gn View 1 chunk +0 lines, -5 lines 0 comments Download
M ui/ozone/ozone.gni View 2 chunks +3 lines, -4 lines 0 comments Download
D ui/ozone/platform/caca/BUILD.gn View 1 chunk +0 lines, -43 lines 0 comments Download
D ui/ozone/platform/caca/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
D ui/ozone/platform/caca/caca_event_source.h View 1 chunk +0 lines, -45 lines 0 comments Download
D ui/ozone/platform/caca/caca_event_source.cc View 1 chunk +0 lines, -234 lines 0 comments Download
D ui/ozone/platform/caca/caca_window.h View 1 chunk +0 lines, -101 lines 0 comments Download
D ui/ozone/platform/caca/caca_window.cc View 1 chunk +0 lines, -157 lines 0 comments Download
D ui/ozone/platform/caca/caca_window_manager.h View 1 chunk +0 lines, -49 lines 0 comments Download
D ui/ozone/platform/caca/caca_window_manager.cc View 1 chunk +0 lines, -135 lines 0 comments Download
D ui/ozone/platform/caca/client_native_pixmap_factory_caca.h View 1 chunk +0 lines, -17 lines 0 comments Download
D ui/ozone/platform/caca/client_native_pixmap_factory_caca.cc View 1 chunk +0 lines, -15 lines 0 comments Download
D ui/ozone/platform/caca/ozone_platform_caca.h View 1 chunk +0 lines, -17 lines 0 comments Download
D ui/ozone/platform/caca/ozone_platform_caca.cc View 1 chunk +0 lines, -101 lines 0 comments Download
D ui/ozone/platform/caca/scoped_caca_types.h View 1 chunk +0 lines, -34 lines 0 comments Download
D ui/ozone/platform/caca/scoped_caca_types.cc View 1 chunk +0 lines, -23 lines 0 comments Download

Messages

Total messages: 8 (3 generated)
fwang
PTAL
4 years, 1 month ago (2016-10-25 10:00:00 UTC) #2
rjkroege
lgtm
4 years, 1 month ago (2016-10-25 23:08:34 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2445323002/1
4 years, 1 month ago (2016-10-26 07:34:08 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 1 month ago (2016-10-26 08:33:27 UTC) #6
commit-bot: I haz the power
4 years, 1 month ago (2016-10-26 08:36:05 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/e7dc921a4088b26b8108032968e1c11819895e3e
Cr-Commit-Position: refs/heads/master@{#427635}

Powered by Google App Engine
This is Rietveld 408576698