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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1692543005: Remove no-op VideoDecoder::Reset implementation. (Closed)

Created:
4 years, 10 months ago by pbos-webrtc
Modified:
4 years, 10 months ago
Reviewers:
mflodman
CC:
webrtc-reviews_webrtc.org, interface-changes_webrtc.org, rwolff_gocast.it, video-team_agora.io, yujie_mao (webrtc), zhengzhonghou_agora.io, stefan-webrtc, tterriberry_mozilla.com, qiang.lu, niklas.enbom, andresp, peah-webrtc, the sun, perkj_webrtc, mflodman
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Remove no-op VideoDecoder::Reset implementation. No longer used in Chromium and should be removable now. BUG=webrtc:5475 TBR=mflodman@webrtc.org Committed: https://crrev.com/c90e9b64d5446a9cc5f183e830eb088770392676 Cr-Commit-Position: refs/heads/master@{#11602}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -8 lines) Patch
M webrtc/video/video_decoder.cc View 1 chunk +0 lines, -4 lines 0 comments Download
M webrtc/video_decoder.h View 2 chunks +0 lines, -4 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
pbos-webrtc
Committed patchset #1 (id:1) manually as c90e9b64d5446a9cc5f183e830eb088770392676 (presubmit successful).
4 years, 10 months ago (2016-02-12 14:16:35 UTC) #2
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/c90e9b64d5446a9cc5f183e830eb088770392676 Cr-Commit-Position: refs/heads/master@{#11602}
4 years, 10 months ago (2016-02-12 14:16:40 UTC) #4
mflodman
4 years, 10 months ago (2016-02-12 14:39:44 UTC) #5
Message was sent while issue was closed. 
lgtm

Powered by Google App Engine
This is Rietveld 408576698