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

Issue 2082483003: voice_engine: Removed old variants of Channel constructor and CreateChannel (Closed)

Created:
4 years, 6 months ago by ossu
Modified:
4 years, 4 months ago
Reviewers:
kwiberg-webrtc
CC:
webrtc-reviews_webrtc.org, henrika_webrtc, Andrew MacDonald, tterriberry_mozilla.com, audio-team_agora.io, hlundin-webrtc, peah-webrtc, minyue-webrtc, the sun
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

voice_engine: Removed old variants of Channel constructor and CreateChannel These are no longer used internally and their interface is not to be considered public. They were due to be changed in https://codereview.webrtc.org/1993783002/ but remained due to a misunderstanding. Committed: https://crrev.com/e1f5b4a7fe8c16a891042173c2e357c48fef4ad3 Cr-Commit-Position: refs/heads/master@{#13816}

Patch Set 1 #

Patch Set 2 : Rebase #

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

Messages

Total messages: 17 (12 generated)
ossu
After discussing with Solenberg, it turned out the CreateChannel that needed to remain unchanged was ...
4 years, 6 months ago (2016-06-20 11:09:44 UTC) #2
kwiberg-webrtc
lgtm
4 years, 4 months ago (2016-08-12 08:57:32 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/2082483003/20001
4 years, 4 months ago (2016-08-18 09:03:54 UTC) #14
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 4 months ago (2016-08-18 11:23:02 UTC) #15
commit-bot: I haz the power
4 years, 4 months ago (2016-08-18 11:23:10 UTC) #17
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/e1f5b4a7fe8c16a891042173c2e357c48fef4ad3
Cr-Commit-Position: refs/heads/master@{#13816}

Powered by Google App Engine
This is Rietveld 408576698