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

Issue 1177953006: Initial SIE commit: migrating existing code (Closed)

Created:
5 years, 6 months ago by ekm
Modified:
5 years, 6 months ago
Reviewers:
aluebs-webrtc
CC:
webrtc-reviews_webrtc.org, Andrew MacDonald, tterriberry_mozilla.com, hlundin-webrtc, kwiberg-webrtc, aluebs-webrtc, bjornv1
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Initial SIE commit: migrating existing code Moved exact existing intelligibility enhancement implementation into new repository for reference when making further changes. Note: this cl does not add these files to any gyp. Original cl is at https://webrtc-codereview.appspot.com/52719004/ . TBR=aluebs@webrtc.org Committed: https://chromium.googlesource.com/external/webrtc/+/030249dd247444687663c4969ff078dc0a4b24ac

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1131 lines, -0 lines) Patch
A webrtc/modules/audio_processing/intelligibility/intelligibility_enhancer.h View 1 chunk +137 lines, -0 lines 0 comments Download
A webrtc/modules/audio_processing/intelligibility/intelligibility_enhancer.cc View 1 chunk +383 lines, -0 lines 0 comments Download
A webrtc/modules/audio_processing/intelligibility/intelligibility_proc.cc View 1 chunk +187 lines, -0 lines 0 comments Download
A webrtc/modules/audio_processing/intelligibility/intelligibility_utils.h View 1 chunk +137 lines, -0 lines 0 comments Download
A webrtc/modules/audio_processing/intelligibility/intelligibility_utils.cc View 1 chunk +287 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1177953006/1
5 years, 6 months ago (2015-06-15 19:58:51 UTC) #2
commit-bot: I haz the power
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an ...
5 years, 6 months ago (2015-06-15 19:58:53 UTC) #4
ekm
Committed patchset #1 (id:1) manually as 030249dd247444687663c4969ff078dc0a4b24ac (presubmit successful).
5 years, 6 months ago (2015-06-15 20:02:38 UTC) #5
aluebs-webrtc
5 years, 6 months ago (2015-06-15 21:08:03 UTC) #6
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698