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

Issue 2167243002: Delete Ozone egltest platform. (Closed)

Created:
4 years, 5 months ago by kylechar
Modified:
4 years, 5 months ago
Reviewers:
jbudorick, spang
CC:
chromium-reviews, kalyank, ozone-reviews_chromium.org, tdresser+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Delete Ozone egltest platform. The Ozone X11 platform implements a more complete X11 backend for Ozone and egltest is unused at this point. Delete the egltest code and all the accompanying parts of GYP and GN build files. BUG=none Committed: https://crrev.com/3f2b800844a8267d98eb2a1bbbcc4e7c499f5654 Cr-Commit-Position: refs/heads/master@{#406918}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -851 lines) Patch
M build/common.gypi View 3 chunks +2 lines, -6 lines 0 comments Download
M build/linux/system.gyp View 1 chunk +1 line, -1 line 0 comments Download
M docs/cast_build_instructions.md View 1 chunk +1 line, -1 line 0 comments Download
M ui/events/events.gyp View 1 chunk +1 line, -1 line 0 comments Download
M ui/ozone/BUILD.gn View 1 chunk +0 lines, -5 lines 0 comments Download
M ui/ozone/ozone.gni View 2 chunks +2 lines, -4 lines 0 comments Download
M ui/ozone/ozone.gyp View 1 chunk +0 lines, -5 lines 0 comments Download
D ui/ozone/platform/egltest/BUILD.gn View 1 chunk +0 lines, -79 lines 0 comments Download
D ui/ozone/platform/egltest/client_native_pixmap_factory_egltest.h View 1 chunk +0 lines, -17 lines 0 comments Download
D ui/ozone/platform/egltest/client_native_pixmap_factory_egltest.cc View 1 chunk +0 lines, -15 lines 0 comments Download
D ui/ozone/platform/egltest/eglplatform_shim.h View 1 chunk +0 lines, -58 lines 0 comments Download
D ui/ozone/platform/egltest/eglplatform_shim_xeleven.cc View 1 chunk +0 lines, -106 lines 0 comments Download
D ui/ozone/platform/egltest/egltest.gypi View 1 chunk +0 lines, -115 lines 0 comments Download
D ui/ozone/platform/egltest/ozone_platform_egltest.h View 1 chunk +0 lines, -17 lines 0 comments Download
D ui/ozone/platform/egltest/ozone_platform_egltest.cc View 1 chunk +0 lines, -421 lines 0 comments Download

Messages

Total messages: 13 (7 generated)
kylechar
+spang for ui/ozone/*, +jbudorick for build/*
4 years, 5 months ago (2016-07-21 17:32:04 UTC) #2
spang
lgtm
4 years, 5 months ago (2016-07-21 17:37:35 UTC) #5
jbudorick
lgtm
4 years, 5 months ago (2016-07-21 18:08:29 UTC) #6
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/2167243002/1
4 years, 5 months ago (2016-07-21 18:32:23 UTC) #10
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 5 months ago (2016-07-21 18:42:40 UTC) #11
commit-bot: I haz the power
4 years, 5 months ago (2016-07-21 18:45:11 UTC) #13
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/3f2b800844a8267d98eb2a1bbbcc4e7c499f5654
Cr-Commit-Position: refs/heads/master@{#406918}

Powered by Google App Engine
This is Rietveld 408576698