Last modified: 2014-08-19 17:55:12 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T67609, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 65609 - Using "random" and go home when "loading random page" then return crashes the app
Using "random" and go home when "loading random page" then return crashes the...
Status: RESOLVED FIXED
Product: Wikipedia App
Classification: Unclassified
Generic (Other open bugs)
Android (alpha)
All All
: Unprioritized normal
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-05-21 21:53 UTC by Liangent
Modified: 2014-08-19 17:55 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Liangent 2014-05-21 21:53:51 UTC
java.lang.IllegalArgumentException: View=com.android.internal.policy.impl.PhoneWindow$DecorView{428387a8 V.E..... R......D 0,0-1026,288} not attached to window manager
at android.view.WindowManagerGlobal.findViewLocked(WindowManagerGlobal.java:370)
at android.view.WindowManagerGlobal.removeView(WindowManagerGlobal.java:299)
at android.view.WindowManagerImpl.removeViewImmediate(WindowManagerImpl.java:84)
at android.app.Dialog.dismissDialog(Dialog.java:329)
at android.app.Dialog.dismiss(Dialog.java:312)
at org.wikipedia.random.RandomHandler$1$1.onFinish(RandomHandler.java:49)
at org.wikipedia.random.RandomHandler$1$1.onFinish(RandomHandler.java:29)
at org.wikipedia.concurrency.SaneAsyncTask$BackingAsyncTask.onPostExecute(SaneAsyncTask.java:110)
at org.wikipedia.concurrency.AsyncTask.finish(AsyncTask.java:641)
at org.wikipedia.concurrency.AsyncTask.access$600(AsyncTask.java:186)
at org.wikipedia.concurrency.AsyncTask$InternalHandler.handleMessage(AsyncTask.java:654)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:136)
at android.app.ActivityThread.main(ActivityThread.java:5017)
at java.lang.reflect.Method.invokeNative(Native Method)
at java.lang.reflect.Method.invoke(Method.java:515)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:779)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:595)
at de.robv.android.xposed.XposedBridge.main(XposedBridge.java:133)
at dalvik.system.NativeStart.main(Native Method)
Comment 1 Gerrit Notification Bot 2014-05-23 21:21:49 UTC
Change 135112 had a related patch set uploaded by BearND:
Fix crash using random and leaving app

https://gerrit.wikimedia.org/r/135112
Comment 2 Bernd Sitzmann 2014-05-23 22:00:17 UTC
Liangent,

I haven't been able to reproduce this yet.
When you say go home do you mean press the home button to put the app into background? 
What do you mean by "after bug 65539"?

Thanks!
Comment 3 Liangent 2014-05-24 00:27:44 UTC
The key is you must have already returned to the app (so it's a new activity now) when the random fetching task ends.
Comment 4 Bernd Sitzmann 2014-05-24 18:09:24 UTC
I also think one would want to turn on "don't keep activities" in developer options on the device to get a better chance at reproducing this.

I've closed the related OTRS tickets since we're tracking this bug here now:
Ticket#: 2014052310020631 – org.wikipedia Crash Report
Ticket#: 2014051910016565 – org.wikipedia Crash Report
Ticket#: 2014051910015744 – org.wikipedia Crash Report
Ticket#: 2014052410000401 – org.wikipedia Crash Report

Going to abandon the current try to fix this since it didn't work. Even guarding against progressDialog.dismiss() while not shown, Liangent was able to reproduce the bug. Need a better fix for this. Probably need to ensure the fragment is in resumed state.
Comment 5 Gerrit Notification Bot 2014-05-24 18:09:34 UTC
Change 135112 abandoned by BearND:
Fix crash using random and leaving app

Reason:
Need a better fix for this. Probably need to ensure the fragment is in resumed state.

https://gerrit.wikimedia.org/r/135112
Comment 6 Dan Garry 2014-08-19 17:54:15 UTC
I don't understand how to reproduce this bug. Can someone explain?
Comment 7 Yuvi Panda 2014-08-19 17:55:12 UTC
We used to have a dialog box that popped up when you hit 'random' saying 'Loading random page...'. We no longer do.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links