From ab84272272b1a9104480c00bc84c7c6bceaf28f7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sami=20Kalliom=C3=A4ki?= Date: Thu, 1 Jun 2017 13:38:27 +0200 Subject: [PATCH] Remove deprecation warning from JVM::Initialize with the context parameter. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Decision was made to keep this API for the time being. Bug: webrtc:7710 Change-Id: Ief41ffb2ec2345e3a74fc72927d038be1ff5941c No-Try: True Reviewed-on: https://chromium-review.googlesource.com/521085 Reviewed-by: Henrik Andreasson Commit-Queue: Sami Kalliomäki Cr-Commit-Position: refs/heads/master@{#18377} --- webrtc/modules/utility/include/jvm_android.h | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/webrtc/modules/utility/include/jvm_android.h b/webrtc/modules/utility/include/jvm_android.h index 60e15e57ed..fcdb1baca7 100644 --- a/webrtc/modules/utility/include/jvm_android.h +++ b/webrtc/modules/utility/include/jvm_android.h @@ -147,10 +147,9 @@ class JVM { // Stores global handles to the Java VM interface. // Should be called once on a thread that is attached to the JVM. static void Initialize(JavaVM* jvm); - // TODO(sakal): Remove once downstream dependencies have been updated. - // DEPRECATED (crbug.com/webrtc/7710): Old signature taking the Android - // context as a parameter. Currently passes in the context to the ContextUtils - // class. + // Like the method above but also passes the context to the ContextUtils + // class. This method should be used by pure-C++ Android users that can't call + // ContextUtils.initialize directly. static void Initialize(JavaVM* jvm, jobject context); // Clears handles stored in Initialize(). Must be called on same thread as // Initialize().