Android Chromoting: Don't include API key with Directory request

remoting/android/java/src/org/chromium/chromoting/HostListLoader.java

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chromoting;
 
 import android.os.Handler;
 import android.os.HandlerThread;
 import android.os.Looper;
 import android.util.Log;
@@ skipping 24 matching lines @@
     }
 
     /** Callback for receiving the host list, or getting notified of an error. */
     public interface Callback {
         void onHostListReceived(HostInfo[] hosts);
         void onError(Error error);
     }
 
     /** Path from which to download a user's host list JSON object. */
     private static final String HOST_LIST_PATH =
-            "https://www.googleapis.com/chromoting/v1/@me/hosts?key=";
+            "https://www.googleapis.com/chromoting/v1/@me/hosts";
 
     /** Callback handler to be used for network operations. */
     private Handler mNetworkThread;
 
     /** Handler for main thread. */
     private Handler mMainThread;
 
     public HostListLoader() {
         // Thread responsible for downloading the host list.
 
@@ skipping 23 matching lines @@
             public void run() {
                 doRetrieveHostList(authTokenFinal, callbackFinal);
             }
         });
     }
 
     private void doRetrieveHostList(String authToken, Callback callback) {
         HttpURLConnection link = null;
         String response = null;
         try {
-            link = (HttpURLConnection)
-                    new URL(HOST_LIST_PATH + JniInterface.nativeGetApiKey()).openConnection();
+            link = (HttpURLConnection) new URL(HOST_LIST_PATH).openConnection();
             link.addRequestProperty("client_id", JniInterface.nativeGetClientId());

[Lambros, 2014/10/06 23:03:21]

Maybe we should remove these lines as well (client_id and client_secret)?

[rmsousa, 2014/10/06 23:13:25]

Yes, we should, I don't think they're currently doing anything.

[Lambros, 2014/10/07 00:11:01]

Done.

             link.addRequestProperty("client_secret", JniInterface.nativeGetClientSecret());
             link.setRequestProperty("Authorization", "OAuth " + authToken);
 
             // Listen for the server to respond.
             int status = link.getResponseCode();
             switch (status) {
                 case HttpURLConnection.HTTP_OK:  // 200
                     break;
                 case HttpURLConnection.HTTP_UNAUTHORIZED:  // 401
                     postError(callback, Error.AUTH_FAILED);
@@ skipping 86 matching lines @@
                     return a.isOnline ? -1 : 1;
                 }
                 String aName = a.name.toUpperCase(Locale.getDefault());
                 String bName = b.name.toUpperCase(Locale.getDefault());
                 return aName.compareTo(bName);
             }
         };
         Collections.sort(hosts, hostComparator);
     }
 }