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

Issue 2800523003: Properly handle track status changes in FLUSHED state (Closed)

Created:
3 years, 8 months ago by servolk
Modified:
3 years, 8 months ago
Reviewers:
xhwang
CC:
chromium-reviews, feature-media-reviews_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Properly handle track status changes in FLUSHED state BUG=

Patch Set 1 #

Patch Set 2 : Adjust test expectations for track switches after Flush #

Patch Set 3 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+38 lines, -35 lines) Patch
M media/renderers/renderer_impl.cc View 1 2 5 chunks +34 lines, -13 lines 0 comments Download
M media/renderers/renderer_impl_unittest.cc View 1 2 2 chunks +4 lines, -22 lines 0 comments Download

Depends on Patchset:

Messages

Total messages: 15 (14 generated)
servolk
3 years, 8 months ago (2017-04-06 22:59:36 UTC) #13
Now that the RendererImpl has the new FLUSHED state, I have rebased the media
track switch CL and this CL to demonstrate the changes necessary to properly
handle track switching during Flush operation. PTAL.
PTAL. If this CL looks good to you, then I merge it into the bigger media track
switch CL.

Powered by Google App Engine
This is Rietveld 408576698