Chromium Code Reviews
Help | Chromium Project | Sign in
(8)

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
5 months ago by fwang
Modified:
5 months 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
Commit queue not available (can’t edit this change).

Messages

Total messages: 8 (3 generated)
fwang
PTAL
5 months ago (2016-10-25 10:00:00 UTC) #2
rjkroege
lgtm
5 months 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
5 months ago (2016-10-26 07:34:08 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 months ago (2016-10-26 08:33:27 UTC) #6
commit-bot: I haz the power
5 months 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}
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld cc6ac46