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

Issue 2279593004: Delete gyp files from media/ (Closed)

Created:
4 years, 3 months ago by watk
Modified:
4 years, 3 months ago
CC:
Aaron Boodman, abarth-chromium, alokp+watch_chromium.org, avayvod+watch_chromium.org, chromium-reviews, darin (slow to review), eme-reviews_chromium.org, feature-media-reviews_chromium.org, imcheng+watch_chromium.org, isheriff+watch_chromium.org, jasonroberts+watch_google.com, mcasas+watch+vc_chromium.org, miu+watch_chromium.org, piman+watch_chromium.org, posciak+watch_chromium.org, qsr+mojo_chromium.org, toyoshim+midi_chromium.org, viettrungluu+watch_chromium.org, xjz+watch_chromium.org, yzshen+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Delete gyp files from media/ After this CL there are no more references to gyp anywhere in media/. BUG=641113 Committed: https://crrev.com/cd8f9885e1a55d7388056632f12723481181dcfb Cr-Commit-Position: refs/heads/master@{#414923}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -4800 lines) Patch
M media/BUILD.gn View 2 chunks +2 lines, -2 lines 0 comments Download
D media/base/ipc/media_base_ipc.gyp View 1 chunk +0 lines, -26 lines 0 comments Download
D media/blink/media_blink.gyp View 1 chunk +0 lines, -181 lines 0 comments Download
D media/capture/capture.gyp View 1 chunk +0 lines, -308 lines 0 comments Download
D media/cast/cast.gyp View 1 chunk +0 lines, -273 lines 0 comments Download
D media/cast/cast_testing.gypi View 1 chunk +0 lines, -201 lines 0 comments Download
D media/cast/cast_testing_tools.gypi View 1 chunk +0 lines, -202 lines 0 comments Download
M media/cast/test/proto/BUILD.gn View 1 chunk +0 lines, -1 line 0 comments Download
M media/cdm/cdm_paths.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/cdm/ppapi/BUILD.gn View 1 chunk +0 lines, -3 lines 0 comments Download
M media/cdm/ppapi/external_clear_key/cdm_video_decoder.h View 1 chunk +3 lines, -3 lines 0 comments Download
M media/cdm/ppapi/ppapi_cdm_adapter.gni View 1 chunk +1 line, -3 lines 0 comments Download
D media/cdm_paths.gypi View 1 chunk +0 lines, -58 lines 0 comments Download
M media/ffmpeg/ffmpeg_common.h View 1 chunk +2 lines, -2 lines 0 comments Download
D media/gpu/ipc/media_ipc.gyp View 1 chunk +0 lines, -109 lines 0 comments Download
D media/media.gyp View 1 chunk +0 lines, -2326 lines 0 comments Download
D media/media_cdm.gypi View 1 chunk +0 lines, -200 lines 0 comments Download
D media/media_cdm_adapter.gyp View 1 chunk +0 lines, -75 lines 0 comments Download
D media/media_gpu.gypi View 1 chunk +0 lines, -379 lines 0 comments Download
D media/media_nacl.gyp View 1 chunk +0 lines, -47 lines 0 comments Download
D media/media_variables.gypi View 1 chunk +0 lines, -19 lines 0 comments Download
D media/midi/midi.gyp View 1 chunk +0 lines, -251 lines 0 comments Download
D media/mojo/interfaces/mojo_bindings.gyp View 1 chunk +0 lines, -104 lines 0 comments Download
D media/shared_memory_support.gypi View 1 chunk +0 lines, -26 lines 0 comments Download

Messages

Total messages: 22 (15 generated)
watk
4 years, 3 months ago (2016-08-26 19:13:21 UTC) #9
DaleCurtis
lgtm
4 years, 3 months ago (2016-08-26 19:15:13 UTC) #12
DaleCurtis
lgtm (┛◉Д◉)┛彡┻━┻
4 years, 3 months ago (2016-08-26 19:15:39 UTC) #14
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/2279593004/1
4 years, 3 months ago (2016-08-26 23:22:10 UTC) #17
mcasas
On 2016/08/26 23:22:10, commit-bot: I haz the power wrote: > CQ is trying da patch. ...
4 years, 3 months ago (2016-08-26 23:37:14 UTC) #18
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 3 months ago (2016-08-27 11:08:15 UTC) #20
commit-bot: I haz the power
4 years, 3 months ago (2016-08-27 11:11:04 UTC) #22
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/cd8f9885e1a55d7388056632f12723481181dcfb
Cr-Commit-Position: refs/heads/master@{#414923}

Powered by Google App Engine
This is Rietveld 408576698