From c16ce866dc040010364c6ea33c317914c549c772 Mon Sep 17 00:00:00 2001
From: Matthew Simmons <simmonmt@google.com>
Date: Mon, 7 Mar 2011 13:54:45 -0500
Subject: [PATCH] code review comments

---
 MyTracks/res/layout/send_to_google_result.xml          | 10 +++++-----
 .../src/com/google/android/apps/mytracks/MyTracks.java |  2 +-
 .../apps/mytracks/io/sendtogoogle/SendDialog.java      |  8 +++-----
 3 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/MyTracks/res/layout/send_to_google_result.xml b/MyTracks/res/layout/send_to_google_result.xml
index 7e0c01405..f2d93780f 100644
--- a/MyTracks/res/layout/send_to_google_result.xml
+++ b/MyTracks/res/layout/send_to_google_result.xml
@@ -37,11 +37,11 @@ services.
       android:layout_width="wrap_content"
       android:layout_height="wrap_content"
       android:gravity="center_horizontal">
-      <!-- We don't want a divider, so we set its color to transparent. -->
-      <!-- We do this explicitly, rather than with a color resource, because -->
-      <!-- an apparent bug causes the divider to be wildly misrendered on -->
-      <!-- a Mini X10 running 2.1. -->
-   	<ListView
+    <!-- We don't want a divider, so we set its color to transparent. -->
+    <!-- We do this explicitly, rather than with a color resource, because -->
+    <!-- an apparent bug causes the divider to be wildly misrendered on -->
+    <!-- a Mini X10 running 2.1. -->
+    <ListView
         android:id="@+id/send_to_google_result_list"
         android:clickable="false"
         android:paddingTop="10dip"
diff --git a/MyTracks/src/com/google/android/apps/mytracks/MyTracks.java b/MyTracks/src/com/google/android/apps/mytracks/MyTracks.java
index 3445dda7d..f964f2fc0 100644
--- a/MyTracks/src/com/google/android/apps/mytracks/MyTracks.java
+++ b/MyTracks/src/com/google/android/apps/mytracks/MyTracks.java
@@ -1189,7 +1189,7 @@ public class MyTracks extends TabActivity implements OnTouchListener,
     //    listener will be null).  The OK button listener will initiate the
     //    share.
     //
-    // 2. The user did not pre-request a share, but the set of services to
+    // 2. The user did not pre-request a share, and the set of services to
     //    which we succeeded in uploading the track are compatible with
     //    sharing.  We'll display a share button (the share listener will be
     //    non-null), and will share the link if the user clicks it.
diff --git a/MyTracks/src/com/google/android/apps/mytracks/io/sendtogoogle/SendDialog.java b/MyTracks/src/com/google/android/apps/mytracks/io/sendtogoogle/SendDialog.java
index aca75c94f..9296de275 100644
--- a/MyTracks/src/com/google/android/apps/mytracks/io/sendtogoogle/SendDialog.java
+++ b/MyTracks/src/com/google/android/apps/mytracks/io/sendtogoogle/SendDialog.java
@@ -1,12 +1,12 @@
 /*
  * Copyright 2008 Google Inc.
- * 
+ *
  * Licensed under the Apache License, Version 2.0 (the "License"); you may not
  * use this file except in compliance with the License. You may obtain a copy of
  * the License at
- * 
+ *
  * http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
  * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
@@ -84,8 +84,6 @@ public class SendDialog extends Dialog {
             sendToFusionTablesCheckBox.isChecked() ||
             sendToDocsCheckBox.isChecked());
         groupMyMaps.setVisibility(sendToMyMapsCheckBox.isChecked() ? View.VISIBLE : View.INVISIBLE);
-        //groupDocs.setVisibility(
-        //    sendToDocsCheckBox.isChecked() ? View.VISIBLE : View.INVISIBLE);
       }
     };