Fix Java indentation issues.

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;
@@ skipping 265 matching lines @@
     }
 
     /** Called when the user touches hyperlinked text. */
     @Override
     public void onClick(View view) {
         HelpActivity.launch(this, HOST_SETUP_URL);
     }
 
     /** Called when the user taps on a host entry. */
     public void connectToHost(HostInfo host) {
-        mProgressIndicator = ProgressDialog.show(this,
-              host.name, getString(R.string.footer_connecting), true, true,
-              new DialogInterface.OnCancelListener() {
-                  @Override
-                  public void onCancel(DialogInterface dialog) {
-                      JniInterface.disconnectFromHost();
-                      mTokenFetcher = null;
-                  }
-              });
+        mProgressIndicator = ProgressDialog.show(this, host.name, getString(
+                R.string.footer_connecting), true, true,

[Lambros, 2014/10/15 20:55:51]

Optional nit:
Google style guide says "prefer to break at a higher syntactic level":
https://google-styleguide.googlecode.com/svn/trunk/javaguide.html#s4.5.1-line...

which would mean the break should be before 'getString'', not after it?

I guess the lint tool you're using isn't aware of this rule? Or this rule
doesn't apply to us?

[aurimas (slooooooooow), 2014/10/15 21:18:47]

Sadly Checkstyle does not seem to be aware of this rule :/
Fixed.

+                new DialogInterface.OnCancelListener() {
+                @Override

[Lambros, 2014/10/15 20:55:51]

Should "@Override" be indented to match up with 'public' below it?

https://google-styleguide.googlecode.com/svn/trunk/javaguide.html#s4.8.5-anno...

[aurimas (slooooooooow), 2014/10/15 21:18:48]

Yep, my bad on this one.

+                    public void onCancel(DialogInterface dialog) {
+                        JniInterface.disconnectFromHost();
+                        mTokenFetcher = null;
+                    }
+                });
         SessionConnector connector = new SessionConnector(this, this, mHostListLoader);
         assert mTokenFetcher == null;
         mTokenFetcher = createTokenFetcher(host);
         connector.connectToHost(mAccount.name, mToken, host);
     }
 
     private void refreshHostList() {
         if (mWaitingForAuthToken) {
             return;
         }
@@ skipping 186 matching lines @@
             }
         };
         return new ThirdPartyTokenFetcher(this, host.getTokenUrlPatterns(), callback);
     }
 
     public void fetchThirdPartyToken(String tokenUrl, String clientId, String scope) {
         assert mTokenFetcher != null;
         mTokenFetcher.fetchToken(tokenUrl, clientId, scope);
     }
 }