Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TIMOB-10227: Kitchensink: Kitchensink crashes on relaunch after opening ... #2657

Merged
merged 1 commit into from Aug 6, 2012

Conversation

ayeung
Copy link
Contributor

@ayeung ayeung commented Jul 30, 2012

...a tab and leave the device idle for sometime.

https://jira.appcelerator.org/browse/TIMOB-10227

Test case in Jira.

…ng a tab and leave the device idle for sometime.
@ayeung
Copy link
Contributor Author

ayeung commented Jul 30, 2012

Someone from platform should do the CR, but we will want someone from QE to do the FR since I was not able to reproduce.

@eric34
Copy link
Contributor

eric34 commented Jul 30, 2012

We are trying this now. We got the Nexus S to fail after 15 minutes.

@billdawson
Copy link
Contributor

CR accepted. I'm not sure if Eric's comment means it still fails FR though? If so I imagine it's a different bug (or at least a different Exception) since the code change is definitely looking for null now.

@ayeung
Copy link
Contributor Author

ayeung commented Aug 3, 2012

QE has been busy, they will look into it this afternoon.

@eric34
Copy link
Contributor

eric34 commented Aug 6, 2012

We have been trying to reproduce this for a while. We have been unable to with the pull. We are however running into 10012 when trying to reproduce. We will recommend to Neeraj that he consider merge or rejection without being able to reproduce the fail case.

@joshthecoder
Copy link
Contributor

I will sign off on the functional test. Was able to reproduce this on a Nexus 7, but with different test steps than
the ticket linked to this PR. Following the testing steps for TIMOB-10227 I was able to reproduce the Java NPE.
This fix does appear to resolve the NPE part of the problem.

Accepted

joshthecoder added a commit that referenced this pull request Aug 6, 2012
TIMOB-10227: Kitchensink: Kitchensink crashes on relaunch after opening ...
@joshthecoder joshthecoder merged commit 510b99e into tidev:master Aug 6, 2012
@joshthecoder
Copy link
Contributor

Sorry the test steps from TIMOB-9285 reproduce this issue on the Nexus 7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants