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

Issue 2903373004: WebmMuxer: handle zero-sized encoded frame in OnEncodedVideo() (Closed)

Created:
3 years, 7 months ago by mcasas
Modified:
3 years, 7 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/branch-heads/3071
Project:
chromium
Visibility:
Public.

Description

WebmMuxer: handle zero-sized encoded frame in OnEncodedVideo() This CL teaches WebmMuxer to ignore zero-sized encoded frames in WebmMuxer::OnEncodedVideo(), because these are produced sporadically by the openh264 encoder. Bug: 716451 Change-Id: I06261320e1bc6a7042dc7c6a6917115e6ba5e51a Reviewed-on: https://chromium-review.googlesource.com/514412 Reviewed-by: Emircan Uysaler <emircan@chromium.org>; Commit-Queue: Miguel Casas <mcasas@chromium.org>; Cr-Original-Commit-Position: refs/heads/master@{#474668} Review-Url: https://codereview.chromium.org/2903373004 . Cr-Commit-Position: refs/branch-heads/3071@{#701} Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641} Committed: https://chromium.googlesource.com/chromium/src/+/2ab7d0b017153fb1bde72ebe8f1c865f03a3c249

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M media/muxers/webm_muxer.cc View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
mcasas
3 years, 7 months ago (2017-05-26 17:29:52 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
2ab7d0b017153fb1bde72ebe8f1c865f03a3c249.

Powered by Google App Engine
This is Rietveld 408576698