Add Help & feedback screens to Android Chromoting client

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

 // Copyright 2013 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.accounts.Account;
 import android.accounts.AccountManager;
 import android.accounts.AccountManagerCallback;
 import android.accounts.AccountManagerFuture;
 import android.accounts.AuthenticatorException;
 import android.accounts.OperationCanceledException;
 import android.app.ActionBar;
 import android.app.Activity;
 import android.app.ProgressDialog;
 import android.content.DialogInterface;
 import android.content.Intent;
 import android.content.SharedPreferences;
 import android.content.res.Configuration;
+import android.net.Uri;
 import android.os.Bundle;
 import android.util.Log;
 import android.view.Menu;
 import android.view.MenuItem;
 import android.widget.ArrayAdapter;
 import android.widget.ListView;
 import android.widget.TextView;
 import android.widget.Toast;
 
 import org.chromium.chromoting.jni.JniInterface;
 
 import java.io.IOException;
 import java.util.Arrays;
 
 /**
  * The user interface for querying and displaying a user's host list from the directory server. It
  * also requests and renews authentication tokens using the system account manager.
  */
 public class Chromoting extends Activity implements JniInterface.ConnectionListener,
         AccountManagerCallback<Bundle>, ActionBar.OnNavigationListener,
         HostListLoader.Callback {
     /** Only accounts of this type will be selectable for authentication. */
     private static final String ACCOUNT_TYPE = "com.google";
 
     /** Scopes at which the authentication token we request will be valid. */
     private static final String TOKEN_SCOPE = "oauth2:https://www.googleapis.com/auth/chromoting " +
             "https://www.googleapis.com/auth/googletalk";
 
+    /** Web page to be displayed in the Help screen when launched from this activity. */
+    private static final String HELP_URL =
+            "http://support.google.com/chrome/?p=mobile_crd_hostslist";

[Jamie, 2014/02/25 22:01:18]

When will this URL be live?

[Lambros, 2014/02/26 21:15:45]

The URL currently shows a generic Chrome Mobile help page, until our own help
content is written and published. That's in progress, but I don't know how long
that will take.

+
     /** User's account details. */
     private Account mAccount;
 
     /** List of accounts on the system. */
     private Account[] mAccounts;
 
     /** Account auth token. */
     private String mToken;
 
     /** Helper for fetching the host list. */
@@ skipping 106 matching lines @@
             // If there is no account, don't allow the user to refresh the listing.
             mRefreshButton.setEnabled(false);
         }
 
         return super.onCreateOptionsMenu(menu);
     }
 
     /** Called whenever an action bar button is pressed. */
     @Override
     public boolean onOptionsItemSelected(MenuItem item) {
-        refreshHostList();
-        return true;
+        switch (item.getItemId()) {
+            case R.id.actionbar_directoryrefresh:
+                refreshHostList();
+                return true;
+
+            case R.id.actionbar_help:
+                {
+                    Intent intent = new Intent(this, HelpActivity.class);
+                    intent.setData(Uri.parse(HELP_URL));
+                    startActivity(intent);
+                }
+                return true;
+
+            default:
+                return super.onOptionsItemSelected(item);
+        }
     }
 
     /** Called when the user taps on a host entry. */
     public void connectToHost(HostInfo host) {
         if (host.jabberId.isEmpty() || host.publicKey.isEmpty()) {
             // TODO(lambroslambrou): If these keys are not present, treat this as a connection
             // failure and reload the host list (see crbug.com/304719).
             Toast.makeText(this, getString(R.string.error_reading_host),
                     Toast.LENGTH_LONG).show();
             return;
@@ skipping 186 matching lines @@
                 // Unreachable, but required by Google Java style and findbugs.
                 assert false : "Unreached";
         }
 
         if (dismissProgress && mProgressIndicator != null) {
             mProgressIndicator.dismiss();
             mProgressIndicator = null;
         }
     }
 }