admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
2,178 Commits 1 Branch 0 Tags
Commit Graph

1061 Commits

Author SHA1 Message Date
tina.legrand@webrtc.org
77fd39aa99 ACM PCM16B, fixing a copy-and-paste error. 
Review URL: https://webrtc-codereview.appspot.com/631006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2355 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-06-04 11:47:49 +00:00
phoglund@webrtc.org
e6f235cfa5 Attempt to fix broken encoding. 
TBR=niklase@webrtc.org
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/637004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2353 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-06-04 11:04:05 +00:00
niklas.enbom@webrtc.org
9cf4d72d5d git-svn-id: http://webrtc.googlecode.com/svn/trunk@2352 4adac7df-926f-26a2-2b94-8c16560cd09d 2012-06-04 10:58:43 +00:00
niklas.enbom@webrtc.org
82bf033380 git-svn-id: http://webrtc.googlecode.com/svn/trunk@2351 4adac7df-926f-26a2-2b94-8c16560cd09d 2012-06-04 10:57:51 +00:00
niklas.enbom@webrtc.org
265e38c336 Fixing test gypi for bit rate controller 
Review URL: https://webrtc-codereview.appspot.com/636004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2350 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-06-04 10:12:44 +00:00
marpan@webrtc.org
899baa821b Temporarily disable first partition packet counting to avoid a bug in ProducerFec which doesn't properly handle important packets. 
TBR=stefan@webrtc.org
Review URL: https://webrtc-codereview.appspot.com/631005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2346 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-06-01 18:32:16 +00:00
leozwang@webrtc.org
354b0ed015 Check return result of fwrite [Audio Module] 
Description:
On ChromeOS/ARM, compiler enforces to check return result of a function.
Currently, we don't check return result of fwrite, it causes building errors.

The following files need to patch. The patch should be similar, before I patch all
of them, I will start with 2 files, please take a quick look, if the patch is OK,
I will continue and upload a new patch that covers all of them.
it to all of them.
Review URL: https://webrtc-codereview.appspot.com/566016

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2345 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-06-01 17:46:21 +00:00
kma@webrtc.org
c3b2683bf4 Refactored the pitch filter function in iSAC-fix. One important purpose is to prepare the function for assembly optimization in ARM platforms. 
Note that,
(1) The main change is a new function PitchFilter() replacing a couple of common code blocks. Next step will be the assembly coding of this function in ARM.
(2) Resulted code is not bit exact with the original. The only reason is replacing two saturation blocks (lines 197 and 208) for the case of "type == 2" with the general case (line 147 and 159). The change makes the code more consistent, and I think the original code might just be a bug. I raised the issue in an email to Turaj and Bjorn last week.
Listening test might be needed. I will send the resulted files to Turaj for this purpose.
(3) I used Astyle to make the code more stylish, but didn't try extra effort to correct all the code style details.  Local code style consistency was considered for new code. So this is not a full and final refactor project (will leave that to future refactoring).
Review URL: https://webrtc-codereview.appspot.com/573009

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2344 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-06-01 17:00:07 +00:00
tina.legrand@webrtc.org
5b4f36db88 ACM: Too short char vector 
Revision 2340 failed on Linux Release, and the problem was that we allocated a too short vector for the output file name.

BUG=r2340 failed on Linux release
TEST=audio_coding_module_test

Review URL: https://webrtc-codereview.appspot.com/624006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2343 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-06-01 14:51:28 +00:00
tina.legrand@webrtc.org
4517585db5 Adding separate payload types for stereo modes 
BUG=Issue 452
TEST=audio_coding_test, voe_auto_test, voe_cmd_test

Edit: adding Patrik to review:
src/modules/rtp_rtcp/source/rtp_receiver.cc
...and Shijing to review:
src/voice_engine/main/source/channel.cc
src/voice_engine/main/test/cmd_test/voe_cmd_test.cc

Review URL: https://webrtc-codereview.appspot.com/540004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2340 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-06-01 09:27:35 +00:00
pwestin@webrtc.org
c2722a0e68 Fixed compiler warning 
Review URL: https://webrtc-codereview.appspot.com/624005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2339 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-06-01 08:56:42 +00:00
stefan@webrtc.org
f5d934dfd8 Upgrade libvpx to cab6ac16 (v. 1.1.1 pre-point-release). 
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/619004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2337 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-06-01 07:43:02 +00:00
andrew@webrtc.org
7d8c567982 Ignore return value of fwrites. 
The removed error return was of course failing in the void ProcessBlock.
Ignored the returns of the remaining fwrites as well for consistency.

TBR=leozwang@webrtc.org
BUG=none
TEST=run audioproc with debug enabled

Review URL: https://webrtc-codereview.appspot.com/623004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2336 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-06-01 02:41:14 +00:00
kjellander@webrtc.org
2e84c112f5 Updating bitrate controller tests to test naming conventions. 
The test is now named 'bitrate_controller_unittests'.
This CL also enables it on the bots. The test is excluded on ASAN since
it fails when compiled with projects generated with GYP_DEFINES='asan=1' (see issue 555).

BUG=None
TEST=bitrate_controller_unittests was tested in Debug+Release on Linux, Mac and Windows + TSAN and memcheck.

Review URL: https://webrtc-codereview.appspot.com/612004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2333 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-31 13:55:01 +00:00
phoglund@webrtc.org
baaf2434a7 Extracted a method for sending padded data. 
BUG=
TEST=Ran vie_auto_test and voe_auto_test standard tests.

Review URL: https://webrtc-codereview.appspot.com/605004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2332 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-31 10:47:35 +00:00
andrew@webrtc.org
36ccce4f58 Remove documentation folders. 
Review URL: https://webrtc-codereview.appspot.com/606007

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2329 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-30 17:28:24 +00:00
leozwang@webrtc.org
e7e64e3468 Fix compilation errors on ChromeOS 
Description:
This cl fixes two compilation errors on ChromeOS/ARM, it could
also be reproduced by gcc 4.5+.

I also add comments about error message and how I solve them.

BUG=webrtc issue 554
TEST=try bots and build on chromeos arm
Review URL: https://webrtc-codereview.appspot.com/611006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2327 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-30 16:46:09 +00:00
niklas.enbom@webrtc.org
0cb79cc851 Fixing gyp bug in https://webrtc-codereview.appspot.com/599006 
Review URL: https://webrtc-codereview.appspot.com/609006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2325 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-30 14:32:42 +00:00
stefan@webrtc.org
dc257b5781 Add option to configure error concealment and disable by default. 
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/597005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2324 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-30 11:25:00 +00:00
mflodman@webrtc.org
327ada1cb0 Refactored IncomingVideoStream and VideoRenderFrame, to get code in better shape when hunting BUG=481. 
BUG=481
TEST=Compiles on all platformas and autotest passes.

Review URL: https://webrtc-codereview.appspot.com/608005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2323 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-30 10:45:18 +00:00
bjornv@webrtc.org
281b7983db Resolved Coverity warnings. 
This CL includes changes to resolve Coverity warnings 14086, 14110 and 14111.

Tested with trybots and audioproc_unittests.

BUG=None
TEST=None

Review URL: https://webrtc-codereview.appspot.com/606004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2321 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-30 07:41:57 +00:00
leozwang@webrtc.org
b5ea03adbb Add print out stats summary to integrationtest.cc 
Stats summary prints out cpu usage.

BUG=
TEST=test on linux
Review URL: https://webrtc-codereview.appspot.com/602004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2320 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-30 00:34:50 +00:00
andrew@webrtc.org
459955f821 Move audio_frame_operations to the utility module. 
TBR=henrika@webrtc.org
BUG=issue551
TEST=voe_auto_test, webrtc_utility_unittest, trybots

Review URL: https://webrtc-codereview.appspot.com/599006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2318 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-29 22:13:14 +00:00
andrew@webrtc.org
5f23d64cf2 Set the stream delay to zero if too low. 
- Return a stream warning instead of an error.
- Add a few delay offset tests.

BUG=
TEST=audioproc_unittest

Review URL: https://webrtc-codereview.appspot.com/607004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2316 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-29 21:14:06 +00:00
leozwang@webrtc.org
2fc6e388c0 Check return value of fwrite. [Video Module] 
Description:
On ChromeOS/ARM, compiler enforces to check return result of a function.
Currently, we don't check return result of fwrite, it causes building errors.

The following files need to patch. The patch should be similar, before I patch all
of them, I will start with 3 files, once we agree upon the solution, we will expand
it to all of them.

The question is should we do 
1. if (error) { return -1;} 
or 
2. if (error) { /*ignor the error*/ }

I took "return -1" in this patch, but I'm OK with either. Please let me know your
thoughts and I will upload a new patch.
Review URL: https://webrtc-codereview.appspot.com/583010

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2315 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-29 17:33:13 +00:00
pwestin@webrtc.org
1eef9c16ff Bitrate bugfixes 
Review URL: https://webrtc-codereview.appspot.com/609005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2313 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-29 09:28:43 +00:00
stefan@webrtc.org
5abab0b1b5 Revert 2311 - Disable error concealment. 
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/609004

TBR=stefan@webrtc.org
Review URL: https://webrtc-codereview.appspot.com/604006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2312 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-29 09:04:40 +00:00
stefan@webrtc.org
3348b2990b Disable error concealment. 
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/609004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2311 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-29 08:44:00 +00:00
tina.legrand@webrtc.org
0de1ee3830 NetEQ: Remove an unnecessary condition, to fix a clang warning 
This is a duplicate of issue 606005: https://webrtc-codereview.appspot.com/606005/

BUG=
TEST=neteq_unittests

Review URL: https://webrtc-codereview.appspot.com/605005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2305 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-28 11:37:50 +00:00
kma@webrtc.org
0d321da7e1 Refactored ARM specific code in Noise Suppression. Bit exact. 
Review URL: https://webrtc-codereview.appspot.com/459006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2303 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-26 01:05:27 +00:00
leozwang@webrtc.org
1755a57cbc Check return result of fwrite, [APM] 
Description:
This cl added checking return result of fwrite which makes it compile
on ChromeOS/ARM.

BUG=issue:541
TEST=Build on all platforms
Review URL: https://webrtc-codereview.appspot.com/583009

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2302 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-25 19:20:35 +00:00
turaj@webrtc.org
10d3b5239b I haven't done any refactoring here. 
Resolve coverity warnings.

14305.

The warning is not really valid. The 'decode' function should be called with a 'mode' variable, where inside the function it is assumed that mode is either zero or one. If mode is taking other values some varibles are used uninitialized. However, this is an internal function and it is always called with either ZERO or ONE. Therefore, the code operates correctly. I made small changes as I beleive it is a bit nicer way. 

In ACM:
- Conditions on 'mode' is changed.


Tested with trybots.

BUG=None
TEST=None
Review URL: https://webrtc-codereview.appspot.com/564014

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2297 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-24 21:20:25 +00:00
stefan@webrtc.org
42e78ac087 Disable normal_async_test and rename tests to vp8_integrationtests. 
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/598004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2294 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-24 14:36:34 +00:00
mflodman@webrtc.org
6af9594d71 Added gyp variable to include/exclude all tests. 
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/597004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2292 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-24 13:23:35 +00:00
pwestin@webrtc.org
5c3a400fae Re-added ChangeUniqueId temporary for chrome. 
Review URL: https://webrtc-codereview.appspot.com/594004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2287 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-24 09:52:19 +00:00
turaj@webrtc.org
ea0aa13aa8 I haven't done any refactoring here. 
Resolve coverity warnings.

14240, 14241.

In ACM:
- NULL pointer sanity checks corrected.

Tested with trybots.

BUG=None
TEST=None
Review URL: https://webrtc-codereview.appspot.com/571012

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2281 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-23 15:43:51 +00:00
andrew@webrtc.org
9dc45dad1b Move trunk/test/data -> trunk/data 
BUG=
TEST=all trybot test failures passed locally

Review URL: https://webrtc-codereview.appspot.com/583007

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2280 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-23 15:39:01 +00:00
stefan@webrtc.org
78d8d99180 Fixes the vp8_test on Linux. 
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/579015

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2277 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-23 14:18:56 +00:00
marpan@webrtc.org
747cd87da1 FEC: For multi-frame FEC, allow for the size of the code to be increased, 
under certain conditions. This generally improves the FEC recovery for 
     bursty loss under medium-high protection level.
Review URL: https://webrtc-codereview.appspot.com/566012

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2271 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-22 16:50:00 +00:00
astor@webrtc.org
e231c61ba7 Style guide reformatting of overuse_detector.{h,cc} 
Review URL: https://webrtc-codereview.appspot.com/575006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2268 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-22 12:24:00 +00:00
andrew@webrtc.org
294be77c2e Permit mixing mono and stereo streams. 
Add mixing tests based on older ones from the extended tests.

BUG=issue534
TEST=manual, voe_auto_test

Review URL: https://webrtc-codereview.appspot.com/576014

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2265 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-22 03:28:41 +00:00
pwestin@webrtc.org
b2179c20f0 Bugfix issue 533. Client does not handle NACK or PLI requests received from far end if a RTCP report from it has not been processed when RFC 5506 is enabled. 
Review URL: https://webrtc-codereview.appspot.com/569020

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2263 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-21 12:00:49 +00:00
stefan@webrtc.org
1bca6d2437 Fixes coverity warnings. 
BUG=
TEST=

Review URL: https://webrtc-codereview.appspot.com/566014

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2262 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-21 07:42:12 +00:00
andrew@webrtc.org
b3bea2eb3e Fix build errors on OS X Lion. 
TBR=henrika@webrtc.org
TEST=build on Lion, trybots
Review URL: https://webrtc-codereview.appspot.com/583005

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2261 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-18 19:02:42 +00:00
leozwang@webrtc.org
ada5db4e75 Enable aecm neon optimized code in android build 
Description:
This CL is a follow up of http://review.webrtc.org/575008/ and r2241.
Because of r2243, r2241 is messed up and reverted, I'm going to commit it again.
This is exact same to the last patch in 575008, just want to inform you and have
your double check before I commit it.

The original description
This cl enables cpu detection and aecm optimized code in android build.

BUG=
TEST=build on all platforms
Review URL: https://webrtc-codereview.appspot.com/568006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2259 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-18 16:48:45 +00:00
leozwang@webrtc.org
00c07e66a7 Fix a random crash because of NULL point 
Description:
This cl fixes NULL point crash problem which was not detected runtime.
Also, small reformats are added too.

BUG=
TEST=
Review URL: https://webrtc-codereview.appspot.com/579009

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2254 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-16 16:57:28 +00:00
leozwang@webrtc.org
4d7d23cde4 Refactor OpenGL Code 
Description:
This CL refactored OpenGL java code. Most of code referenced sample code.
And bug-fix and reformats are added into this cl too.

BUG=
TEST=build and test on android
Review URL: https://webrtc-codereview.appspot.com/583004

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2253 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-16 16:57:15 +00:00
hta@webrtc.org
3a698651d7 Coverity issue 14317 (uninitialized status may influence execution flow) 
Solution: restructure the flow when AquireSocket fails

BUG=coverity:14317
TEST=trybot

Review URL: https://webrtc-codereview.appspot.com/573006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2252 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-16 14:28:52 +00:00
kjellander@webrtc.org
7e4a72a78a Revert 2241 - Enable aecm neon optimized code in android build 
Description:
This cl enables cpu detection and aecm optimized code in android build.

BUG=
TEST=build on all platforms
Review URL: https://webrtc-codereview.appspot.com/575008

TBR=leozwang@webrtc.org
Review URL: https://webrtc-codereview.appspot.com/566013

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2249 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-16 11:49:45 +00:00
hta@webrtc.org
d429086f62 Deleted udp_socket_windows and all references to it. 
This fixes a couple of Coverity issues, and doesn't impact any tested
platforms.

BUG=Coverity:14423
TEST=trybots

Review URL: https://webrtc-codereview.appspot.com/564012

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2247 4adac7df-926f-26a2-2b94-8c16560cd09d
 2012-05-16 09:32:14 +00:00