Closed Bug 1120332 Opened 9 years ago Closed 9 years ago

[gonk-l] During constructing, MediaCodec always waits for 'batterystats' service for 5 seconds before giving up.

Categories

(Firefox OS Graveyard :: General, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(b2g-v2.2 fixed, b2g-master fixed)

RESOLVED FIXED
2.2 S4 (23jan)
Tracking Status
b2g-v2.2 --- fixed
b2g-master --- fixed

People

(Reporter: jhlin, Assigned: jhlin)

Details

Attachments

(1 file)

This behavior is problematic on B2G because that service is not available there.
getService() could wait up to 5 seconds when service is not available [1]; OTOH, checkService() returns immediately.
Other approaches to address this problem:
a. implement a dummy service. I wouldn't recommend this because it adds some run-time overheads, or
b. implement a  service that really works, if we want to monitor audio/video power usage.

[1] https://android.googlesource.com/platform/frameworks/native/+/master/libs/binder/IServiceManager.cpp#137
Attachment #8547444 - Flags: review?(sotaro.ikeda.g)
Comment on attachment 8547444 [details] [review]
Replace getService() with checkService().

Looks good.
Attachment #8547444 - Flags: review?(sotaro.ikeda.g) → review+
Keywords: checkin-needed
b2g-5.0.0_r6: https://github.com/mozilla-b2g/platform_frameworks_av/commit/ea531874885eed7f68802048218ed86dde927f58
Status: NEW → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S4 (23jan)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: