Properly send URL parameters in all requests

webrtc/examples/androidapp/src/org/appspot/apprtc/WebSocketRTCClient.java

 /*
  *  Copyright 2014 The WebRTC Project Authors. All rights reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 122 matching lines @@
   private String getConnectionUrl(RoomConnectionParameters connectionParameters) {
     String url = connectionParameters.roomUrl + "/" + ROOM_JOIN + "/" + connectionParameters.roomId;
     if (connectionParameters.urlParameters != null) {
       url += "?" + connectionParameters.urlParameters;
     }
     return url;
   }
 
   private String getMessageUrl(
       RoomConnectionParameters connectionParameters, SignalingParameters signalingParameters) {
-    return connectionParameters.roomUrl + "/" + ROOM_MESSAGE + "/" + connectionParameters.roomId
-        + "/" + signalingParameters.clientId;
+    String url = connectionParameters.roomUrl + "/" + ROOM_MESSAGE + "/"
+        + connectionParameters.roomId + "/" + signalingParameters.clientId;
+    if (connectionParameters.urlParameters != null) {

[kjellander_webrtc, 2017/05/02 11:52:56]

I think the code duplication pattern calls for introducing a private helper
function to call instead.

[oprypin_webrtc, 2017/05/02 12:15:03]

Done.

+      url += "?" + connectionParameters.urlParameters;
+    }
+    return url;
   }
 
   private String getLeaveUrl(
       RoomConnectionParameters connectionParameters, SignalingParameters signalingParameters) {
-    return connectionParameters.roomUrl + "/" + ROOM_LEAVE + "/" + connectionParameters.roomId + "/"
-        + signalingParameters.clientId;
+    String url = connectionParameters.roomUrl + "/" + ROOM_LEAVE + "/"
+        + connectionParameters.roomId + "/" + signalingParameters.clientId;
+    if (connectionParameters.urlParameters != null) {
+      url += "?" + connectionParameters.urlParameters;
+    }
+    return url;
   }
 
   // Callback issued when room parameters are extracted. Runs on local
   // looper thread.
   private void signalingParametersReady(final SignalingParameters signalingParameters) {
     Log.d(TAG, "Room connection completed.");
     if (connectionParameters.loopback
         && (!signalingParameters.initiator || signalingParameters.offerSdp != null)) {
       reportError("Loopback room is busy.");
       return;
@@ skipping 253 matching lines @@
     jsonPut(json, "candidate", candidate.sdp);
     return json;
   }
 
   // Converts a JSON candidate to a Java object.
   IceCandidate toJavaCandidate(JSONObject json) throws JSONException {
     return new IceCandidate(
         json.getString("id"), json.getInt("label"), json.getString("candidate"));
   }
 }