java lang IllegalArgumentException View not attached to window manager

Question

I have an activity that starts AsyncTask and shows progress dialog for the duration of operation  The activity is declared NOT be recreated by rotation or keyboard slide        lt activity android name   MyActivity                 android label   string app name                android configChanges  keyboardHidden orientation                 gt           lt intent-filter gt           lt  intent-filter gt       lt  activity gt    Once task completed  I dissmiss dialog  but on some phones  framework  1 5  1 6  such error is thrown   java lang IllegalArgumentException  View not attached to window manager     at android view WindowManagerImpl findViewLocked WindowManagerImpl java 356      at android view WindowManagerImpl removeView WindowManagerImpl java 201      at android view Window LocalWindowManager removeView Window java 400      at android app Dialog dismissDialog Dialog java 268      at android app Dialog access 000 Dialog java 69      at android app Dialog 1 run Dialog java 103      at android app Dialog dismiss Dialog java 252      at xxx onPostExecute xxx 1 java xxx    My code is   final Dialog dialog   new AlertDialog Builder context       setTitle  Processing           setCancelable true       create     final AsyncTask lt MyParams  Object  MyResult gt  task   new AsyncTask lt MyParams  Object  MyResult gt            Override     protected MyResult doInBackground MyParams    params               Long operation goes here             Override     protected void onPostExecute MyResult result            dialog dismiss            onCompletion result             task execute        dialog setOnCancelListener new OnCancelListener          Override     public void onCancel DialogInterface arg0            task cancel false              dialog show      From what I have read  http   bend-ing blogspot com 2008 11 properly-handle-progress-dialog-in html  and seen in Android sources  it looks like the only possible situation to get that exception is when activity was destroyed  But as I have mentioned  I forbid activity recreation for basic events   So any suggestions are very appreciated

User · Answer

Override         protected void onPostExecute Void result                super onPostExecute result                if  progressDialog    null  amp  amp  progressDialog isShowing                      Log i TAG   onPostexucte                    progressDialog dismiss

User · Answer

First of all do error handling where ever you trying to dismiss the dialog    if   progressDialog    null   amp  amp  progressDialog isShowing                  progressDialog dismiss                progressDialog   null              If that doesn t fix then dismiss it in onStop   Method of the activity      Override     protected void onStop             super onStop            if   progressDialog    null   amp  amp  progressDialog isShowing                  progressDialog dismiss                progressDialog   null

User · Answer

I think your code is correct unlike the other answer suggested  onPostExecute will run on the UI thread   That s the whole point of AsyncTask - you don t have to worry about calling runOnUiThread or deal with handlers   Furthermore  according to the docs  dismiss   can be safely called from any thread  not sure they made this the exception    Perhaps it s a timing issue where dialog dismiss   is getting called after the activity is no longer displayed   What about testing what happens if you comment out the setOnCancelListener and then exit the activity while the background task is running   Then your onPostExecute will try to dismiss an already dismissed dialog   If the app crashes you can probably just check if the dialog is open before dismissing it   I m having the exact same problem so I m going to try it out in code

User · Answer

Here is the correct solution to solving this problem   public void hideProgress         if mProgressDialog    null            if mProgressDialog isShowing        check if dialog is showing                 get the Context object that was used to great the dialog             Context context     ContextWrapper mProgressDialog getContext    getBaseContext                   if the Context used here was an activity AND it hasn t been finished or destroyed               then dismiss it             if context instanceof Activity                     if    Activity context  isFinishing    amp  amp     Activity context  isDestroyed                         mProgressDialog dismiss                  else   if the Context used wasnt an Activity  then dismiss it too                 mProgressDialog dismiss                      mProgressDialog   null            Instead of blindly catching all exceptions  this solution addresses the root of the problem  trying to dimiss a dialog when the activity used to initialize the dialog has already been finished  Working on my Nexus 4 running KitKat  but should work for all versions of Android

User · Answer

I may have a workaround   Was having the same issue  where I am loading lots of items  via the file system  into a ListView via an AsyncTask   Had the onPreExecute   firing up a ProgressDialog  and then both onPostExecute   and onCancelled    called when the task is cancelled explicitly via AsyncTask cancel    closing it via  cancel       Got the same  java lang IllegalArgumentException  View not attached to window manager  error when I was killing the dialog in the onCancelled   method of the AsyncTask  I d seen this done in the excellent Shelves app      The workaround was to create a public field in the AsyncTask that contains the ProgressDialog   public ProgressDialog mDialog    Then  in onDestroy   when I cancel my AsyncTask  I can also kill the associated dialog via   AsyncTask mDialog cancel      Calling AsyncTask cancel   DOES trigger onCancelled   in the AsyncTask  but for some reason by the time that method is called  the View has already been destroyed and thus cancelling the dialog is failing

User · Answer

The activity is declared NOT be recreated by rotation or keyboard slide    Just got the same problem  Fix for API level 13 or higer  From Android docs      Note  If your application targets API level 13 or higher  as declared by the minSdkVersion and targetSdkVersion attributes   then you should also declare the  screenSize  configuration  because it also changes when a device switches between portrait and landscape orientations    So i ve changed my manifest to this    lt activity         android name  MyActivity          android configChanges  orientation screenSize          android label  MyActivityName   gt   lt  activity gt    And now it works fine  Activity is not recreating when i rotate the phone  progress dialog and view stay the same  No error for me

User · Answer

Migh below code works for you  It works for me perfectly fine   private void viewDialog         try           Intent vpnIntent   new Intent context  UtilityVpnService class           context startService vpnIntent           final View Dialogview   View inflate getBaseContext    R layout alert open internet  null           final WindowManager LayoutParams params   new WindowManager LayoutParams                  WindowManager LayoutParams WRAP CONTENT                  WindowManager LayoutParams WRAP CONTENT                  WindowManager LayoutParams TYPE SYSTEM ALERT                  WindowManager LayoutParams FLAG NOT FOCUSABLE                           WindowManager LayoutParams FLAG NOT TOUCH MODAL   WindowManager LayoutParams FLAG DIM BEHIND                  PixelFormat TRANSLUCENT           params gravity   Gravity CENTER HORIZONTAL   Gravity CENTER VERTICAL          windowManager addView Dialogview  params            Button btn cancel    Button  Dialogview findViewById R id btn canceldialog internetblocked           Button btn okay    Button  Dialogview findViewById R id btn openmainactivity           RelativeLayout relativeLayout    RelativeLayout  Dialogview findViewById R id rellayout dialog                btn cancel setOnClickListener new View OnClickListener                      Override                 public void onClick View view                        Handler handler   new Handler Looper getMainLooper                         handler post new Runnable                              Override                         public void run                                 try                                   if  Dialogview    null                                         WindowManager  getApplicationContext   getSystemService WINDOW SERVICE   removeView Dialogview                                       windowManager removeView Dialogview                                                                   catch  final IllegalArgumentException e                                    e printStackTrace                                       Handle or log or ignore                               catch  final Exception e                                    e printStackTrace                                       Handle or log or ignore                               finally                                   try                                       if  windowManager    null  amp  amp  Dialogview    null                                            WindowManager  getApplicationContext   getSystemService WINDOW SERVICE   removeView Dialogview                                           windowManager removeView Dialogview                                                                           catch  Exception e                                        e printStackTrace                                                                                                        WindowManager  getApplicationContext   getSystemService WINDOW SERVICE   removeView Dialogview                             windowManager removeView Dialogview                                                                                                     btn okay setOnClickListener new View OnClickListener                      Override                 public void onClick View view                        Handler handler   new Handler Looper getMainLooper                         handler post new Runnable                              Override                         public void run                                             WindowManager  getApplicationContext   getSystemService WINDOW SERVICE   removeView Dialogview                               try                                   if  windowManager    null  amp  amp  Dialogview    null                                      windowManager removeView Dialogview                                   Intent intent   new Intent getBaseContext    SplashActivity class                                   intent setFlags Intent FLAG ACTIVITY NEW TASK                             intent setFlags Intent FLAG ACTIVITY CLEAR TASK                             intent setFlags Intent FLAG ACTIVITY CLEAR TOP                                     context startActivity intent                                 catch  Exception e                                    windowManager removeView Dialogview                                   e printStackTrace                                                                                                                                catch  Exception e                    windowManager removeView Dialogview               e printStackTrace                      Do not define your view globally if u call it from background service

User · Answer

Here is my  bullet proof  solution  which is compilation of all good answers that I found on this topic  thanks to  Damjan and  Kachi   Here the exception is swallowed only if all other ways of detection did not succeeded  In my case I need to close the dialog automatically and this is the only way to protect the app from crash   I hope it will help you   Please  vote and leave comments if you have remarks or better solution  Thank you   public void dismissWithCheck Dialog dialog            if  dialog    null                if  dialog isShowing                         get the Context object that was used to great the dialog                 Context context     ContextWrapper  dialog getContext    getBaseContext                        if the Context used here was an activity AND it hasn t been finished or destroyed                    then dismiss it                 if  context instanceof Activity                            Api  gt  17                     if  Build VERSION SDK INT  gt   Build VERSION CODES JELLY BEAN MR1                            if     Activity  context  isFinishing    amp  amp     Activity  context  isDestroyed                                  dismissWithTryCatch dialog                                                   else                               Api  lt  17  Unfortunately cannot check for isDestroyed                           if     Activity  context  isFinishing                                  dismissWithTryCatch dialog                                                                     else                        if the Context used wasn t an Activity  then dismiss it too                     dismissWithTryCatch dialog                             dialog   null                       public void dismissWithTryCatch Dialog dialog            try               dialog dismiss              catch  final IllegalArgumentException e                   Do nothing            catch  final Exception e                   Do nothing            finally               dialog   null

User · Answer

I had the same problem  while using a button to sync a list from the server  1  I click the button 2  A progress dialog shows up while dowloading the list from server 3  I turn the device to another orientation 4  java lang IllegalArgumentException  View not attached to window manager on postExecute   of the AsyncTask during progress dismiss     As I tried the fix I figured that even if the problem doesn t occur my list wasn t showing all items   I figured that what I wanted was for the AsyncTask to finish  and dismiss the dialog  before the activity beign destroyed  so I made the asynctask object an attribute and overrided the onDestroy   method   If the asynctask takes a lot of time the user maybe will feel that the device is slow  but I think that s the price he pays for trying to change the device orientation while the progress dialog is showing up  And even if it takes some time the app doesn t crash   private AsyncTask lt Boolean  Void  Boolean gt  atask    Override protected void onDestroy         if  atask  null          try               atask get              catch  InterruptedException e              catch  ExecutionException e                  super onDestroy

User · Answer

Use this    if  dialog  null  amp  amp   dialog isShowing     dialog dismiss

User · Answer

I had the same problem  you can solve it by    Override protected void onPostExecute MyResult result        try           if   this mDialog    null   amp  amp  this mDialog isShowing                  this mDialog dismiss                    catch  final IllegalArgumentException e               Handle or log or ignore       catch  final Exception e               Handle or log or ignore       finally           this mDialog   null

User · Answer

I too get this error sometimes when I dismiss dialog and finish activity from onPostExecute method  I guess sometimes activity gets finished before dialog successfully dismisses   Simple  yet effective solution that works for me   Override protected void onPostExecute MyResult result        try           if   this mDialog    null   amp  amp  this mDialog isShowing                  this mDialog dismiss                    catch  final IllegalArgumentException e               Handle or log or ignore       catch  final Exception e               Handle or log or ignore       finally           this mDialog   null

User · Answer

What worked for me most of the time is to verify whether the Activity is not finishing   if   mActivity isFinishing          dialog dismiss

User · Answer

i agree a opinion of  Damjan   if you use many dialogs  should close all dialog in onDestroy   or onStop    then you may be able to reduce the frequency  java lang IllegalArgumentException  View not attached to window manager  exception occurs    Override protected void onDestroy         Log d TAG   called onDestroy        mDialog dismiss        super onDestroy         but little exceed    to make it more clear  you prevent to show any dialog after onDestroy called  i don t use as below  but it s clear   private boolean mIsDestroyed   false   private void showDialog         closeDialog         if  mIsDestroyed            Log d TAG   called onDestroy   already             return             mDialog   new AlertDialog this           setTitle  title            setMessage  This is DialogTest            setNegativeButton  Cancel   new DialogInterface OnClickListener                 public void onClick DialogInterface dialog  int which                    dialog dismiss                                      create        mDialog show       private void closeDialog         if  mDialog    null            mDialog dismiss              Override protected void onDestroy         Log d TAG   called onDestroy        mIsDestroyed   true      closeDialog        super onDestroy          good luck

User · Answer

alex    I could be wrong here  but I suspect that multiple phones  in the wild  have a bug that causes  them to switch orientation on applications that are marked as statically oriented  This happens quite a bit on my personal phone  and on many of the test phones our group uses  including droid  n1  g1  hero   Typically an app marked as statically oriented  perhaps vertically  will lay itself out for a second or two using a horizontal orientation  and then immediately switch back  End result is that even though you don t want your app to switch orientation  you have to be prepared that it may  I don t know under what exact conditions this behavior can be reproduced  I don t know if it is specific to a version of Android  All I know is that I have seen it happen plenty of times     I would recommend using the solution provided in the link you posted that suggests overriding the Activity onCreateDialog method and letting the Android OS manage the lifecycle of your Dialogs  It looks to me like even though you don t want your activity to switch orientations  it is switching orientation somewhere  You can try to track down a method that will always prevent orientation switching  but I am trying to tell you that I personally don t believe there is a foolproof way that works on all current Android phones in the market

[android] java.lang.IllegalArgumentException: View not attached to window manager

Examples related to android