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

Issue 23125008: Revert 217691 "Merge 217276 "Add media::VideoEncodeAccelerator w..." (Closed)

Created:
7 years, 4 months ago by kareng
Modified:
7 years, 4 months ago
Reviewers:
karen
CC:
chromium-reviews, jbauman+watch_chromium.org, nkostylev+watch_chromium.org, jam, apatrick_chromium, sievers+watch_chromium.org, joi+watch-content_chromium.org, feature-media-reviews_chromium.org, darin-cc_chromium.org, oshima+watch_chromium.org, piman+watch_chromium.org, danakj+watch_chromium.org, stevenjb+watch_chromium.org, davemoore+watch_chromium.org
Visibility:
Public.

Description

Revert 217691 "Merge 217276 "Add media::VideoEncodeAccelerator w..." > Merge 217276 "Add media::VideoEncodeAccelerator with WebRTC inte..." > > > Add media::VideoEncodeAccelerator with WebRTC integration > > > > * Adds media::VideoEncodeAccelerator class. > > * Add GpuVideoEncodeAccelerator{,Host} classes and appropriate IPC. > > * Integrates into WebRTC stack with RTCVideoEncoderFactory/RTCVideoEncoder. > > * Rename media::GpuVideoDecodeFactories -> media::GpuVideoAcceleratorFactories > > and generalize for use by the encode accelerator implementations as well. > > > > BUG=260210 > > BUG=170345 > > TEST=local build, run on CrOS snow; local build, unittests on desktop Linux > > > > Review URL: https://chromiumcodereview.appspot.com/20632002 > > TBR=sheu@chromium.org > > Review URL: https://codereview.chromium.org/22875014 TBR=karen@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=218294

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+84 lines, -2721 lines) Patch
M chrome/browser/chromeos/login/chrome_restart_request.cc View 1 chunk +0 lines, -4 lines 0 comments Download
M content/browser/renderer_host/render_process_host_impl.cc View 1 chunk +0 lines, -1 line 0 comments Download
M content/common/gpu/DEPS View 1 chunk +0 lines, -1 line 0 comments Download
M content/common/gpu/client/gpu_channel_host.h View 2 chunks +0 lines, -5 lines 0 comments Download
M content/common/gpu/client/gpu_channel_host.cc View 2 chunks +0 lines, -16 lines 0 comments Download
D content/common/gpu/client/gpu_video_encode_accelerator_host.h View 1 chunk +0 lines, -111 lines 0 comments Download
D content/common/gpu/client/gpu_video_encode_accelerator_host.cc View 1 chunk +0 lines, -217 lines 0 comments Download
M content/common/gpu/gpu_channel.h View 4 chunks +1 line, -8 lines 0 comments Download
M content/common/gpu/gpu_channel.cc View 3 chunks +2 lines, -26 lines 0 comments Download
M content/common/gpu/gpu_messages.h View 4 chunks +0 lines, -73 lines 0 comments Download
D content/common/gpu/media/gpu_video_encode_accelerator.h View 1 chunk +0 lines, -98 lines 0 comments Download
D content/common/gpu/media/gpu_video_encode_accelerator.cc View 1 chunk +0 lines, -232 lines 0 comments Download
M content/content_common.gypi View 2 chunks +0 lines, -4 lines 0 comments Download
M content/content_renderer.gypi View 2 chunks +2 lines, -6 lines 0 comments Download
M content/public/common/content_switches.h View 1 chunk +0 lines, -1 line 0 comments Download
M content/public/common/content_switches.cc View 1 chunk +0 lines, -3 lines 0 comments Download
M content/renderer/media/media_stream_dependency_factory.cc View 3 chunks +11 lines, -17 lines 0 comments Download
D content/renderer/media/renderer_gpu_video_accelerator_factories.h View 1 chunk +0 lines, -149 lines 0 comments Download
D content/renderer/media/renderer_gpu_video_accelerator_factories.cc View 1 chunk +0 lines, -428 lines 0 comments Download
A + content/renderer/media/renderer_gpu_video_decoder_factories.h View 0 chunks +-1 lines, --1 lines 0 comments Download
A + content/renderer/media/renderer_gpu_video_decoder_factories.cc View 0 chunks +-1 lines, --1 lines 0 comments Download
M content/renderer/media/rtc_video_decoder.h View 4 chunks +4 lines, -4 lines 0 comments Download
M content/renderer/media/rtc_video_decoder.cc View 5 chunks +6 lines, -6 lines 0 comments Download
M content/renderer/media/rtc_video_decoder_factory.h View 2 chunks +7 lines, -3 lines 0 comments Download
M content/renderer/media/rtc_video_decoder_factory.cc View 2 chunks +4 lines, -3 lines 0 comments Download
M content/renderer/media/rtc_video_decoder_unittest.cc View 4 chunks +5 lines, -5 lines 0 comments Download
D content/renderer/media/rtc_video_encoder.h View 1 chunk +0 lines, -104 lines 0 comments Download
D content/renderer/media/rtc_video_encoder.cc View 1 chunk +0 lines, -658 lines 0 comments Download
D content/renderer/media/rtc_video_encoder_factory.h View 1 chunk +0 lines, -48 lines 0 comments Download
D content/renderer/media/rtc_video_encoder_factory.cc View 1 chunk +0 lines, -110 lines 0 comments Download
M content/renderer/media/webmediaplayer_impl.h View 2 chunks +3 lines, -3 lines 0 comments Download
M content/renderer/media/webmediaplayer_impl.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/renderer/media/webmediaplayer_params.h View 4 chunks +4 lines, -5 lines 0 comments Download
M content/renderer/media/webmediaplayer_params.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M content/renderer/render_thread_impl.h View 3 chunks +5 lines, -3 lines 0 comments Download
M content/renderer/render_thread_impl.cc View 2 chunks +9 lines, -7 lines 0 comments Download
MM content/renderer/render_view_impl.cc View 2 chunks +2 lines, -2 lines 0 comments Download
D media/filters/gpu_video_accelerator_factories.h View 1 chunk +0 lines, -73 lines 0 comments Download
D media/filters/gpu_video_accelerator_factories.cc View 1 chunk +0 lines, -11 lines 0 comments Download
M media/filters/gpu_video_decoder.h View 3 chunks +3 lines, -3 lines 0 comments Download
M media/filters/gpu_video_decoder.cc View 4 chunks +4 lines, -5 lines 0 comments Download
A + media/filters/gpu_video_decoder_factories.h View 0 chunks +-1 lines, --1 lines 0 comments Download
A + media/filters/gpu_video_decoder_factories.cc View 0 chunks +-1 lines, --1 lines 0 comments Download
D media/filters/mock_gpu_video_accelerator_factories.h View 1 chunk +0 lines, -70 lines 0 comments Download
D media/filters/mock_gpu_video_accelerator_factories.cc View 1 chunk +0 lines, -28 lines 0 comments Download
A + media/filters/mock_gpu_video_decoder_factories.h View 0 chunks +-1 lines, --1 lines 0 comments Download
A + media/filters/mock_gpu_video_decoder_factories.cc View 0 chunks +-1 lines, --1 lines 0 comments Download
M media/media.gyp View 8 chunks +15 lines, -17 lines 0 comments Download
D media/video/video_encode_accelerator.h View 1 chunk +0 lines, -145 lines 0 comments Download
D media/video/video_encode_accelerator.cc View 1 chunk +0 lines, -11 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
kareng
7 years, 4 months ago (2013-08-19 19:11:25 UTC) #1
kareng
7 years, 4 months ago (2013-08-19 19:11:50 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r218294.

Powered by Google App Engine
This is Rietveld 408576698