Use new Physical Web service on Google infrastructure

chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PwsClient.java

Index: chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PwsClient.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PwsClient.java b/chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PwsClient.java
index 5e661ea50c285335b6ff932ebf51dbbb93f17b31..a93fa80aa34032bd40429b5d0d9d755e9a71f30c 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PwsClient.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PwsClient.java
@@ -9,6 +9,8 @@ import android.os.AsyncTask;
 
 import org.chromium.base.Log;
 import org.chromium.base.ThreadUtils;
+import org.chromium.chrome.GoogleAPIKeys;
+import org.chromium.chrome.browser.ChromeVersionInfo;
 
 import org.json.JSONArray;
 import org.json.JSONException;
@@ -23,7 +25,7 @@ import java.util.Collection;
  */
 class PwsClient {
     private static final String TAG = "PhysicalWeb";
-    private static final String ENDPOINT_URL = "https://url-caster.appspot.com/resolve-scan";
+    private static final String ENDPOINT_URL = "https://test-physicalweb.sandbox.googleapis.com/v1alpha1/urls:resolve";
 
     /**
      * Callback that is run after the PWS sends a response to a resolve-scan request.
@@ -48,6 +50,14 @@ class PwsClient {
         public void onIconReceived(String iconUrl, Bitmap iconBitmap);
     }
 
+    private String apiKey() {

[agrieve, 2015/11/06 19:17:53]

nit: Should be called "getApiKey()"

[dvh, 2015/11/06 19:27:46]

Done.

+        if (ChromeVersionInfo.isStableBuild()) {
+            return GoogleAPIKeys.GOOGLE_API_KEY;
+        } else {
+            return GoogleAPIKeys.GOOGLE_API_KEY_PHYSICAL_WEB_TEST;

[newt (away), 2015/11/06 19:02:47]

Do we really want to use GOOGLE_API_KEY_PHYSICAL_WEB_TEST on Beta builds and
Dev? I'd recommend only using the test keys on local builds (or perhaps canary
and dev) if that works for your purposes.

[dvh, 2015/11/06 19:27:46]

Discussed about the plan offline.

+        }
+    }
+
     private static JSONObject createResolveScanPayload(Collection<String> urls)
             throws JSONException {
         // Encode the urls.
@@ -60,7 +70,7 @@ class PwsClient {
 
         // Organize the data into a single object.
         JSONObject jsonObject = new JSONObject();
-        jsonObject.put("objects", objects);
+        jsonObject.put("urls", objects);
         return jsonObject;
     }
 
@@ -79,12 +89,13 @@ class PwsClient {
         for (int i = 0; i < metadata.length(); i++) {
             try {
                 JSONObject obj = metadata.getJSONObject(i);
-                String requestUrl = obj.getString("id");
-                String siteUrl = obj.getString("url");
-                String iconUrl = obj.optString("icon", null);
-                String title = obj.optString("title", "");
-                String description = obj.optString("description", "");
-                pwsResults.add(new PwsResult(requestUrl, siteUrl, iconUrl, title, description));
+                JSONObject pageInfo = obj.getJSONObject("pageInfo");
+                String scannedUrl = pageInfo.getString("scannedUrl");
+                String resolvedUrl = pageInfo.getString("resolvedUrl");
+                String iconUrl = pageInfo.optString("icon", null);
+                String title = pageInfo.optString("title", "");
+                String description = pageInfo.optString("description", "");
+                pwsResults.add(new PwsResult(scannedUrl, resolvedUrl, iconUrl, title, description));
             } catch (JSONException e) {
                 Log.e(TAG, "PWS returned invalid metadata", e);
                 continue;
@@ -126,6 +137,7 @@ class PwsClient {
         HttpRequest request = null;
         try {
             JSONObject payload = createResolveScanPayload(broadcastUrls);
+            String url = ENDPOINT_URL + "?key=" + apiKey();

[agrieve, 2015/11/06 19:17:53]

variable unused?

[dvh, 2015/11/06 19:27:46]

Acknowledged.

             request = new JsonObjectHttpRequest(ENDPOINT_URL, payload, requestCallback);
         } catch (MalformedURLException e) {
             Log.e(TAG, "Error creating PWS HTTP request", e);