Issue 2848323003: Remove unnecessary runtime API-level checks.

Issue 2848323003: Remove unnecessary runtime API-level checks. (Closed)

Created:
3 years, 7 months ago by sakal

Modified:
3 years, 7 months ago

Reviewers:
magjed_webrtc, henrika_webrtc

CC:
webrtc-reviews_webrtc.org, tterriberry_mozilla.com, audio-team_agora.io, sdk-team_agora.io, peah-webrtc, kjellander_webrtc

Target Ref:
refs/heads/master

Project:
webrtc

Visibility:
Public.

Remove unnecessary runtime API-level checks. APIs below level 16 are not supported by WebRTC. BUG=webrtc:7549 TBR=henrika@webrtc.org Review-Url: https://codereview.webrtc.org/2848323003 Cr-Commit-Position: refs/heads/master@{#17980} Committed: https://chromium.googlesource.com/external/webrtc/+/bee4ff8d825eb7d3fabd1a6fd5bb1a0c494f329d

Created: 3 years, 7 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+8 lines, -42 lines)			Patch
M	webrtc/examples/androidapp/src/org/appspot/apprtc/AppRTCProximitySensor.java	View	1	3 chunks	+2 lines, -7 lines	0 comments	Download
M	webrtc/modules/audio_device/android/java/src/org/webrtc/voiceengine/WebRtcAudioEffects.java	View		5 chunks	+3 lines, -10 lines	0 comments	Download
M	webrtc/modules/audio_device/android/java/src/org/webrtc/voiceengine/WebRtcAudioManager.java	View		2 chunks	+1 line, -8 lines	0 comments	Download
M	webrtc/modules/audio_device/android/java/src/org/webrtc/voiceengine/WebRtcAudioUtils.java	View		2 chunks	+1 line, -13 lines	0 comments	Download
M	webrtc/sdk/android/api/org/webrtc/SurfaceTextureHelper.java	View		2 chunks	+1 line, -4 lines	0 comments	Download

Dependent Patchsets:

Total messages: 12 (7 generated)

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/2848323003/20001

3 years, 7 months ago (2017-05-02 14:50:14 UTC) #8

commit-bot: I haz the power

Committed patchset #2 (id:20001) as https://chromium.googlesource.com/external/webrtc/+/bee4ff8d825eb7d3fabd1a6fd5bb1a0c494f329d

3 years, 7 months ago (2017-05-02 15:33:56 UTC) #11

Message was sent while issue was closed.

LGTM