Chromium Code Reviews| Index: remoting/android/java/src/org/chromium/chromoting/Chromoting.java |
| diff --git a/remoting/android/java/src/org/chromium/chromoting/Chromoting.java b/remoting/android/java/src/org/chromium/chromoting/Chromoting.java |
| index be378904553fec2c44085415258644ed5443d4a1..7393fef07839d1f20ec24113176c732ccdcd5d2d 100644 |
| --- a/remoting/android/java/src/org/chromium/chromoting/Chromoting.java |
| +++ b/remoting/android/java/src/org/chromium/chromoting/Chromoting.java |
| @@ -283,15 +283,15 @@ public class Chromoting extends Activity implements JniInterface.ConnectionListe |
| /** 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 b
aurimas (slooooooooow)
2014/10/15 21:18:47
Sadly Checkstyle does not seem to be aware of this
|
| + new DialogInterface.OnCancelListener() { |
| + @Override |
|
Lambros
2014/10/15 20:55:51
Should "@Override" be indented to match up with 'p
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); |