From 7894b7362d7eb1f223558907d3fc5d1e89ce1540 Mon Sep 17 00:00:00 2001
From: d-kunin <mr.kunin@gmail.com>
Date: Mon, 12 Aug 2013 19:06:04 +0300
Subject: [PATCH] [and] Progress and speed indication for download.

---
 android/res/layout/activity_expansion.xml     | 28 +++++----
 android/res/values-v11/styles.xml             |  2 +
 android/res/values/strings.xml                |  1 +
 android/res/values/styles.xml                 |  2 +
 .../travelguide/ExpansionActivity.java        | 60 +++++++++----------
 .../com/example/travelguide/util/Utils.java   | 27 +++++++++
 6 files changed, 77 insertions(+), 43 deletions(-)

diff --git a/android/res/layout/activity_expansion.xml b/android/res/layout/activity_expansion.xml
index 187cf52..0262d02 100644
--- a/android/res/layout/activity_expansion.xml
+++ b/android/res/layout/activity_expansion.xml
@@ -11,28 +11,36 @@
 
     <ProgressBar
         android:id="@+id/downloadProgress"
-        style="@android:style/Widget.ProgressBar.Horizontal"
+        style="@style/HorizontalProgress"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
         android:layout_centerInParent="true"
         android:max="100" />
 
+    <TextView
+        android:id="@+id/downloadSpeed"
+        style="@android:style/TextAppearance.Small.Inverse"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_alignParentLeft="true"
+        android:layout_below="@+id/downloadProgress" />
+
+    <TextView
+        android:id="@+id/downloadTotal"
+        style="@android:style/TextAppearance.Small.Inverse"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_alignParentRight="true"
+        android:layout_below="@+id/downloadProgress" />
+
     <TextView
         android:id="@+id/downloadState"
         style="@android:style/TextAppearance.Large.Inverse"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
         android:layout_above="@+id/downloadProgress"
-        android:text="Downloading ... "
         android:textColor="@color/yellow" />
 
-    <Button
-        android:id="@+id/pauseResume"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:layout_below="@id/downloadProgress"
-        android:text="Pause/Resume" />
-
     <CheckBox
         android:id="@+id/mobNetDownload"
         style="@android:style/TextAppearance.Medium.Inverse"
@@ -40,6 +48,6 @@
         android:layout_height="wrap_content"
         android:layout_alignParentBottom="true"
         android:checked="false"
-        android:text="Use mobile network" />
+        android:text="@string/use_mobile_network" />
 
 </RelativeLayout>
\ No newline at end of file
diff --git a/android/res/values-v11/styles.xml b/android/res/values-v11/styles.xml
index 69e9ad0..0d2a8ea 100644
--- a/android/res/values-v11/styles.xml
+++ b/android/res/values-v11/styles.xml
@@ -7,5 +7,7 @@
     <style name="AppBaseTheme" parent="android:Theme.Holo.Light.NoActionBar">
         <!-- API 11 theme customizations can go here. -->
     </style>
+    
+    <style name="HorizontalProgress" parent="@android:style/Widget.Holo.ProgressBar.Horizontal"/>
 
 </resources>
diff --git a/android/res/values/strings.xml b/android/res/values/strings.xml
index 02eff2b..a123e6b 100644
--- a/android/res/values/strings.xml
+++ b/android/res/values/strings.xml
@@ -6,5 +6,6 @@
     <string name="loading">Loading ...</string>
     <string name="searching">Searching ...</string>
     <string name="nothing_found">Nothing Found</string>
+    <string name="use_mobile_network">Use mobile network</string>
 
 </resources>
diff --git a/android/res/values/styles.xml b/android/res/values/styles.xml
index 342a521..824cb80 100644
--- a/android/res/values/styles.xml
+++ b/android/res/values/styles.xml
@@ -16,5 +16,7 @@
     <style name="AppTheme" parent="AppBaseTheme">
         <!-- All customizations that are NOT specific to a particular API-level can go here. -->
     </style>
+    
+    <style name="HorizontalProgress" parent="@android:style/Widget.ProgressBar.Horizontal"/>
 
 </resources>
diff --git a/android/src/com/example/travelguide/ExpansionActivity.java b/android/src/com/example/travelguide/ExpansionActivity.java
index b6fb99a..9c801e0 100644
--- a/android/src/com/example/travelguide/ExpansionActivity.java
+++ b/android/src/com/example/travelguide/ExpansionActivity.java
@@ -7,9 +7,6 @@ import android.content.Intent;
 import android.content.pm.PackageManager.NameNotFoundException;
 import android.os.Bundle;
 import android.os.Messenger;
-import android.view.View;
-import android.view.View.OnClickListener;
-import android.widget.Button;
 import android.widget.CheckBox;
 import android.widget.CompoundButton;
 import android.widget.CompoundButton.OnCheckedChangeListener;
@@ -17,6 +14,7 @@ import android.widget.ProgressBar;
 import android.widget.TextView;
 
 import com.example.travelguide.expansion.ExpansionService;
+import com.example.travelguide.util.Utils;
 import com.google.android.vending.expansion.downloader.DownloadProgressInfo;
 import com.google.android.vending.expansion.downloader.DownloaderClientMarshaller;
 import com.google.android.vending.expansion.downloader.DownloaderServiceMarshaller;
@@ -27,7 +25,7 @@ import com.google.android.vending.expansion.downloader.IStub;
 import com.susanin.travelguide.R;
 
 public class ExpansionActivity extends Activity
-                               implements IDownloaderClient, OnCheckedChangeListener, OnClickListener
+                               implements IDownloaderClient, OnCheckedChangeListener
 {
 
 
@@ -37,10 +35,10 @@ public class ExpansionActivity extends Activity
   private ProgressBar mProgressBar;
   private TextView mDownloadState;
   private CheckBox mMobNetworkDownload;
-  private Button mPauseResume;
 
-  // TODO: should be binded to state
-  private boolean mIsDownloading;
+  private TextView mSpeed;
+  private TextView mTotal;
+
 
   @Override
   protected void onCreate(Bundle savedInstanceState)
@@ -61,10 +59,7 @@ public class ExpansionActivity extends Activity
         if (startResult == DownloaderClientMarshaller.NO_DOWNLOAD_REQUIRED)
           forwardToApp(); // START!
         else
-        {
-          // Set up progress and more
           setUpForDownloading();
-        }
       }
       catch (final NameNotFoundException e)
       {
@@ -95,12 +90,12 @@ public class ExpansionActivity extends Activity
 
     mProgressBar = (ProgressBar) findViewById(R.id.downloadProgress);
     mDownloadState = (TextView) findViewById(R.id.downloadState);
-    // TODO: save this preference
+
     mMobNetworkDownload = (CheckBox) findViewById(R.id.mobNetDownload);
     mMobNetworkDownload.setOnCheckedChangeListener(this);
 
-    mPauseResume = (Button) findViewById(R.id.pauseResume);
-    mPauseResume.setOnClickListener(this);
+    mSpeed = (TextView) findViewById(R.id.downloadSpeed);
+    mTotal = (TextView) findViewById(R.id.downloadTotal);
   }
 
   private void forwardToApp()
@@ -158,21 +153,36 @@ public class ExpansionActivity extends Activity
         forwardToApp();
         return;
     }
-    // TODO: add more cases
   }
 
   @Override
   public void onDownloadProgress(DownloadProgressInfo progress)
   {
-    // TODO Auto-generated method stub
-    final int currentProgressInPercent = (int) (100*progress.mOverallProgress/DATA_FILE.fileSize);
+    final int currentProgressInPercent =
+        (int) (mProgressBar.getMax()*progress.mOverallProgress/(float)progress.mOverallTotal);
+
+    String downloadSpeed = "";
+    if (progress.mCurrentSpeed > 0)
+      downloadSpeed = Utils.formatDataSize(Math.round(progress.mCurrentSpeed*1024))+ "/S";
+
+    final String progressStr = Utils.formatDataSize(progress.mOverallProgress) + "/"
+                               + Utils.formatDataSize(progress.mOverallTotal);
+
+    mSpeed.setText(downloadSpeed);
+    mTotal.setText(progressStr);
     mProgressBar.setProgress(currentProgressInPercent);
   }
 
   private void setMobileNetworkDownloadEnabled(boolean enabled)
   {
     if (mRemoteService != null)
-      mRemoteService.setDownloadFlags(enabled ? IDownloaderService.FLAGS_DOWNLOAD_OVER_CELLULAR: 0);
+      if (enabled)
+      {
+      mRemoteService.setDownloadFlags(IDownloaderService.FLAGS_DOWNLOAD_OVER_CELLULAR);
+      mRemoteService.requestContinueDownload();
+      }
+      else
+        mRemoteService.setDownloadFlags(0);
   }
 
   @Override
@@ -181,20 +191,4 @@ public class ExpansionActivity extends Activity
     if (buttonView == mMobNetworkDownload)
       setMobileNetworkDownloadEnabled(isChecked);
   }
-
-  @Override
-  public void onClick(View target)
-  {
-    if (target.getId() == R.id.pauseResume)
-      pauseResumeDownload();
-  }
-
-  private void pauseResumeDownload()
-  {
-    if (mRemoteService != null)
-    {
-      // TODO: track state and manage it
-    }
-  }
-
 }
diff --git a/android/src/com/example/travelguide/util/Utils.java b/android/src/com/example/travelguide/util/Utils.java
index d28ddef..0f7ccfe 100644
--- a/android/src/com/example/travelguide/util/Utils.java
+++ b/android/src/com/example/travelguide/util/Utils.java
@@ -1,6 +1,9 @@
 package com.example.travelguide.util;
 
 import java.io.File;
+import java.math.RoundingMode;
+import java.text.DecimalFormat;
+import java.text.NumberFormat;
 
 import android.app.Activity;
 import android.content.Context;
@@ -120,4 +123,28 @@ public class Utils
     return new MWMPoint(info.getLat(), info.getLon(), info.getName(), id);
   }
 
+
+  private final static NumberFormat df = DecimalFormat.getInstance(); // frequently used, so we dont recreate
+  static
+  {
+    df.setMinimumFractionDigits(0);
+    df.setMaximumFractionDigits(2);
+    df.setRoundingMode(RoundingMode.DOWN);
+  }
+
+  public static String formatDataSize(long bytes)
+  {
+    if (bytes < 0)
+      return "";
+    final String[] suffixes = {"B", "KB", "MB", "GB", "TB", "EB"};
+    final int powerOf1024 = (int) log1024(bytes);
+    final double shortValue = bytes/Math.pow(1024, powerOf1024);
+    return df.format(shortValue) + suffixes[powerOf1024];
+  }
+
+  public static double log1024(double value)
+  {
+    return Math.log(value)/Math.log(1024);
+  }
+
 }