1131388 - [Camera][Gonk][Shinano] ~4s delay during startup while looking for batterystats service

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Camera, defect)

Description

[Mike Habicher [:mikeh] (high bugzilla latency)]

When the Camera app is started up, with all of the other issue addressed, there is still a ~4s delay. During this time, the following appears in the logcat:

02-09 19:33:24.122   939   954 I ServiceManager: Waiting for service batterystats...
02-09 19:33:25.122   939   954 I ServiceManager: Waiting for service batterystats...
02-09 19:33:26.132   939   954 I ServiceManager: Waiting for service batterystats...
02-09 19:33:27.132   939   954 I ServiceManager: Waiting for service batterystats...
02-09 19:33:28.132   939   954 I ServiceManager: Waiting for service batterystats...

Comment 1

[:gerard-majax]

While hacking on ZR, I had similar behavior. I'll reuse my gonk-misc/fakebattery code and keep you posted.

Comment 2

[:gerard-majax]

Attachment: camera-battery.patch

This will include a fake provider for this service. Maybe we would want this in Gecko directly ? You need to build, push, and run by hand for now.

Comment 3

[:gerard-majax]

Michael, did you figured out for sensorservice ? If you have nothing against this approach, I'll use this in shinano/aries device-specific init.

Comment 4

[Michael Wu [:mwu]]

Starting sensorservice via init.rc is fine. It's not much different than how android starts it - not sure why android doesn't just start it via init.rc.

Comment 5

[:gerard-majax]

Attachment: PR for gonk-misc

Comment 6

[Michael Wu [:mwu]]

Comment on attachment 8564037 [details] [review]
PR for gonk-misc

I think this is more appropriate in shinano-common, unless we start seeing more devices that want to talk to the battery stats service.

Comment 7

[:gerard-majax]

Attachment: Included in shinano-common

I have included it in my shinano-common tree

Comment 8

[:gerard-majax]

shinano-common is r+ in bug 1132921.