|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
The AudioClassifier class seems to be dead code, and was removed upstream per https://bugs.chromium.org/p/webrtc/issues/detail?id=5676 I'd like to remove it in Gecko as well.
Comment on attachment 8877203 [details] Bug 1372622 - webrtc: Remove AudioClassifier. https://reviewboard.mozilla.org/r/148582/#review152980 NB this just removes the code. I didn't attempt to patch the update script. It looks like we haven't pulled from upstream in a while, and if we do, it should already contain this change, so hopefully that's ok.
Comment on attachment 8877203 [details] Bug 1372622 - webrtc: Remove AudioClassifier. https://reviewboard.mozilla.org/r/148582/#review152988 It's always good to delete code which don't need any more :-)
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/e9227c17f3bc webrtc: Remove AudioClassifier. r=drno
I had to back this out because it's causing conflicts for me trying to merge mozilla-central over to autoland after bug 1341285 landed. Feel free to rebase and reland. https://hg.mozilla.org/integration/autoland/rev/64d17537e11f
Ok, thanks for the update. Jesup's patch was more important.
I'm going to go ahead and close this. Bug 1341285 put back the neteq_tests.gypi entry, but apparently it's dead code since the implementation file is still removed. Otherwise the pull from upstream did the same thing this change did.