From c101d2d3babd9e456666e6dbc87e7aefa6d27795 Mon Sep 17 00:00:00 2001
From: Amith Yamasani <yamasani@google.com>
Date: Fri, 11 Nov 2011 13:25:07 -0800
Subject: [PATCH] Dismiss dialogs properly on closing.

Also make the invisible activity truly invisible (was
showing a blue line behind the dialog).

Bug: 5572140
Change-Id: I67b31f0682f189c7075011e0dd0660b0fbc543d9
---
 AndroidManifest.xml                                   | 2 +-
 src/com/android/settings/SettingsLicenseActivity.java | 9 +++++----
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/AndroidManifest.xml b/AndroidManifest.xml
index 649e06c4641..68aca79ba68 100644
--- a/AndroidManifest.xml
+++ b/AndroidManifest.xml
@@ -494,7 +494,7 @@
 
         <activity android:name="SettingsLicenseActivity"
                 android:label="@string/settings_license_activity_title"
-                android:theme="@*android:style/Theme.Holo.Dialog.Alert"
+                android:theme="@android:style/Theme.Holo.Panel"
                 android:configChanges="orientation|screenSize">
             <intent-filter>
                 <action android:name="android.settings.LICENSE" />
diff --git a/src/com/android/settings/SettingsLicenseActivity.java b/src/com/android/settings/SettingsLicenseActivity.java
index 29601801afc..b9efa724bc8 100644
--- a/src/com/android/settings/SettingsLicenseActivity.java
+++ b/src/com/android/settings/SettingsLicenseActivity.java
@@ -173,10 +173,13 @@ public class SettingsLicenseActivity extends Activity {
 
     @Override
     protected void onDestroy() {
-        super.onDestroy();
-        if (mTextDlg != null) {
+        if (mTextDlg != null && mTextDlg.isShowing()) {
             mTextDlg.dismiss();
         }
+        if (mSpinnerDlg != null && mSpinnerDlg.isShowing()) {
+            mSpinnerDlg.dismiss();
+        }
+        super.onDestroy();
     }
 
     private void showPageOfText(String text) {
@@ -200,9 +203,7 @@ public class SettingsLicenseActivity extends Activity {
             @Override
             public void onPageFinished(WebView view, String url) {
                 mSpinnerDlg.dismiss();
-                mSpinnerDlg = null;
                 mTextDlg.show();
-                mTextDlg = null;
             }
         });