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-3286 Enable services to be re-startable #1433

Merged
merged 16 commits into from Feb 17, 2012
Merged

TIMOB-3286 Enable services to be re-startable #1433

merged 16 commits into from Feb 17, 2012

Conversation

billdawson
Copy link
Contributor

@rusticphilosopher
Copy link
Contributor

Code reviewed and Accepted. Functional test pending

@rusticphilosopher
Copy link
Contributor

During functional testing on rooted droid 2 2.2 under V8, the service code (log) never starts executing again after the process is killed. This was done with the app currently in the foreground. The log shows the application service being restarted but then 5-10 seconds later dying and being rescheduled to start again

EDIT: it should be noted that this behavior is before startMode is changed and I have not yet tried with changed start modes

@billdawson
Copy link
Contributor Author

I just tried a device (2.2) for the first time (using 3rd party process killer) and the behavior as expected.

Do you have a logcat or something to show me?

@rusticphilosopher
Copy link
Contributor

Crash log provided on ticket for V8. Same problem exists on Rhino and seems to stem from "InputManagerService: Got RemoteException sending setActive(false) notification to pid 4935 uid 10042" for both runtimes.

@billdawson
Copy link
Contributor Author

From the trace:

    TiApplication: (main) [45,45] Titanium 1.9.0 (2012/02/15 12:20 cb70ac8)

I submitted the PR when it was already 2.0.0. I checked out cb70ac8 and checked the git log for TiJSService.java (which I changed), and it doesn't have my changes. :)

So, at least the time that you got the log, you seemed to have built against the wrong version. But maybe the original time you saw the failure was when built against the correct version.

@rusticphilosopher
Copy link
Contributor

Functional test passed. Accepted

rusticphilosopher pushed a commit that referenced this pull request Feb 17, 2012
TIMOB-3286 Enable services to be re-startable
@rusticphilosopher rusticphilosopher merged commit e412ad2 into tidev:master Feb 17, 2012
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

2 participants