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

Issue 6026013: Revert 70267 - Refactor PipelineImpl to use CompositeFilter to manage Filter ... (Closed)

Created:
9 years, 12 months ago by acolwell GONE FROM CHROMIUM
Modified:
9 years, 6 months ago
CC:
chromium-reviews, vrk (LEFT CHROMIUM), sjl, Alpha Left Google, ddorwin+watch_chromium.org, acolwell GONE FROM CHROMIUM, annacc, darin-cc_chromium.org, awong, Paweł Hajdan Jr., scherkus (not reviewing)
Visibility:
Public.

Description

Revert 70267 - Refactor PipelineImpl to use CompositeFilter to manage Filter state transitions. BUG=54110 TEST=media_unittests CompositeFilterTest.* Review URL: http://codereview.chromium.org/5744002 TBR=acolwell@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=70275

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+121 lines, -1709 lines) Patch
M chrome/browser/media_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
D media/base/composite_filter.h View 1 chunk +0 lines, -165 lines 0 comments Download
D media/base/composite_filter.cc View 1 chunk +0 lines, -603 lines 0 comments Download
D media/base/composite_filter_unittest.cc View 1 chunk +0 lines, -809 lines 0 comments Download
M media/base/mock_filters.h View 4 chunks +5 lines, -41 lines 0 comments Download
M media/base/mock_filters.cc View 1 chunk +0 lines, -19 lines 0 comments Download
M media/base/pipeline.h View 1 chunk +0 lines, -2 lines 0 comments Download
M media/base/pipeline_impl.h View 4 chunks +14 lines, -6 lines 0 comments Download
M media/base/pipeline_impl.cc View 19 chunks +101 lines, -58 lines 0 comments Download
M media/media.gyp View 2 chunks +0 lines, -3 lines 0 comments Download
M webkit/glue/webmediaplayer_impl.cc View 1 chunk +0 lines, -2 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
acolwell GONE FROM CHROMIUM
9 years, 12 months ago (2010-12-29 20:56:48 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698