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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 9139010: Revert 117187 - Adding support for MediaStream and PeerConnection functionality. (Closed)

Created:
8 years, 11 months ago by tommi (sloooow) - chröme
Modified:
8 years, 11 months ago
Reviewers:
Henrik Grunell
CC:
chromium-reviews, hclam+watch_chromium.org, ddorwin+watch_chromium.org, fischman+watch_chromium.org, acolwell+watch_chromium.org, jam, annacc+watch_chromium.org, dpranke-watch+content_chromium.org, joi+watch-content_chromium.org, darin-cc_chromium.org, vrk (LEFT CHROMIUM), scherkus (not reviewing), ihf+watch_chromium.org
Visibility:
Public.

Description

Revert 117187 - Adding support for MediaStream and PeerConnection functionality. BUG= TEST=content_unittests Review URL: http://codereview.chromium.org/8060055 TBR=grunell@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=117191

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+49 lines, -1874 lines) Patch
M content/content_renderer.gypi View 2 chunks +0 lines, -5 lines 0 comments Download
M content/content_tests.gypi View 3 chunks +0 lines, -14 lines 0 comments Download
D content/renderer/media/media_stream_dependency_factory.h View 1 chunk +0 lines, -78 lines 0 comments Download
D content/renderer/media/media_stream_dependency_factory.cc View 1 chunk +0 lines, -72 lines 0 comments Download
M content/renderer/media/media_stream_dispatcher.h View 2 chunks +9 lines, -10 lines 0 comments Download
M content/renderer/media/media_stream_impl.h View 1 chunk +10 lines, -143 lines 0 comments Download
M content/renderer/media/media_stream_impl.cc View 3 chunks +18 lines, -333 lines 0 comments Download
D content/renderer/media/media_stream_impl_unittest.cc View 1 chunk +0 lines, -45 lines 0 comments Download
D content/renderer/media/mock_media_stream_dependency_factory.h View 1 chunk +0 lines, -39 lines 0 comments Download
D content/renderer/media/mock_media_stream_dependency_factory.cc View 1 chunk +0 lines, -50 lines 0 comments Download
D content/renderer/media/mock_media_stream_dispatcher.h View 1 chunk +0 lines, -45 lines 0 comments Download
D content/renderer/media/mock_media_stream_dispatcher.cc View 1 chunk +0 lines, -49 lines 0 comments Download
D content/renderer/media/mock_media_stream_impl.h View 1 chunk +0 lines, -50 lines 0 comments Download
D content/renderer/media/mock_media_stream_impl.cc View 1 chunk +0 lines, -60 lines 0 comments Download
D content/renderer/media/mock_peer_connection_impl.h View 1 chunk +0 lines, -62 lines 0 comments Download
D content/renderer/media/mock_peer_connection_impl.cc View 1 chunk +0 lines, -91 lines 0 comments Download
D content/renderer/media/mock_web_peer_connection_handler_client.h View 1 chunk +0 lines, -42 lines 0 comments Download
D content/renderer/media/mock_web_peer_connection_handler_client.cc View 1 chunk +0 lines, -38 lines 0 comments Download
D content/renderer/media/peer_connection_handler.h View 1 chunk +0 lines, -159 lines 0 comments Download
D content/renderer/media/peer_connection_handler.cc View 1 chunk +0 lines, -274 lines 0 comments Download
D content/renderer/media/peer_connection_handler_unittest.cc View 1 chunk +0 lines, -124 lines 0 comments Download
M content/renderer/p2p/socket_dispatcher.h View 3 chunks +2 lines, -3 lines 0 comments Download
M content/renderer/render_view_impl.h View 7 chunks +1 line, -18 lines 0 comments Download
M content/renderer/render_view_impl.cc View 9 chunks +7 lines, -48 lines 0 comments Download
M content/renderer/renderer_webkitplatformsupport_impl.h View 2 chunks +1 line, -3 lines 0 comments Download
M content/renderer/renderer_webkitplatformsupport_impl.cc View 4 chunks +1 line, -19 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
tommi (sloooow) - chröme
8 years, 11 months ago (2012-01-11 09:17:19 UTC) #1
Henrik Grunell
8 years, 11 months ago (2012-01-11 09:19:30 UTC) #2 
On 2012/01/11 09:17:19, tommi wrote:

LGTM

Powered by Google App Engine
This is Rietveld 408576698