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

diff --git a/AndroidManifest.xml b/AndroidManifest.xml
index 649e06c..68aca79 100644 (file)
--- 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 2960180..b9efa72 100644 (file)
--- 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;
             }
         });