OSDN Git Service
David 'Digit' Turner [Fri, 13 Jan 2012 22:32:50 +0000 (14:32 -0800)]
Merge "media/mtp: Add missing LOCAL_C_INCLUDES"
Jean-Baptiste Queru [Fri, 13 Jan 2012 19:28:09 +0000 (11:28 -0800)]
am
c0ef548c: am
d75961ee: am
b4cee0f8: Merge "stagefright aacenc/amrwbenc: Remove trailing whitespace"
* commit '
c0ef548c748f0c3bc917eff7b50941f9cbe1df86':
stagefright aacenc/amrwbenc: Remove trailing whitespace
Jean-Baptiste Queru [Fri, 13 Jan 2012 19:25:09 +0000 (11:25 -0800)]
am
d75961ee: am
b4cee0f8: Merge "stagefright aacenc/amrwbenc: Remove trailing whitespace"
* commit '
d75961eecbfeb3f7e027a2a1f6011a26dd7d2d2d':
stagefright aacenc/amrwbenc: Remove trailing whitespace
Jean-Baptiste Queru [Fri, 13 Jan 2012 19:22:08 +0000 (11:22 -0800)]
am
b4cee0f8: Merge "stagefright aacenc/amrwbenc: Remove trailing whitespace"
* commit '
b4cee0f871e0f5a2abd6c6f05b65710b69423f2c':
stagefright aacenc/amrwbenc: Remove trailing whitespace
David 'Digit' Turner [Fri, 13 Jan 2012 13:17:29 +0000 (14:17 +0100)]
media/mtp: Add missing LOCAL_C_INCLUDES
One of the source files here is including <cutils/tztime.h> which
itself includes <bionic_time.h>, a private C library header used
to define the strftime_tz() extension (which used to be declared
in <time.h> but was moved there instead).
Add a missing C include path to let our code compile as usual.
Change-Id: I6aac2f1d3d15ad182679c81ff3f4febff74eb671
James Dong [Fri, 13 Jan 2012 00:56:58 +0000 (16:56 -0800)]
Merge "Separate sniffing from session initialization"
James Dong [Tue, 10 Jan 2012 16:24:37 +0000 (08:24 -0800)]
Separate sniffing from session initialization
This avoid lengthy/duplicate sniffing for drm plugins when a decrypt session is opened
o The change is backward compatibile in that no update is required
for existing drm plug-ins if they do not plan to provide separate
sniffer/extractor
related-to-bug:
5725548
Change-Id: I7fc4caf82d77472da4e2bc7b5d31060fb54fd84c
Glenn Kasten [Thu, 12 Jan 2012 18:30:28 +0000 (10:30 -0800)]
Merge "Use audio_format_t consistently"
The Android Open Source Project [Thu, 12 Jan 2012 17:00:21 +0000 (09:00 -0800)]
am
ce77af47: Reconcile with ics-mr1-release
* commit '
ce77af47e979d365155e4003ca2943b639600e73':
The Android Open Source Project [Thu, 12 Jan 2012 16:56:17 +0000 (08:56 -0800)]
Reconcile with ics-mr1-release
Change-Id: Ie242914fa4cfdf35675bbd2777f0817186860526
Mans Rullgard [Tue, 5 Apr 2011 14:40:06 +0000 (17:40 +0300)]
stagefright aacenc/amrwbenc: Remove trailing whitespace
This was applied in AOSP master earlier, in commit
b21611e5b0f1797e39e781d91ade023bac87637f, but when merged
into the Google internal tree in
88c2d4d26d79384f32a25bd775cf873cb678b12a, none of these
changes have been brought in. (The diff of this merge commit
and its first parent,
d94b71d, is empty.)
Therefore, reapply the rest of these changes on the files
that still are present in the current version.
Change-Id: I645a9b9134769958a11808a06163123d28d83e46
The Android Automerger [Thu, 12 Jan 2012 00:29:04 +0000 (16:29 -0800)]
merge in ics-mr1-release history after reset to ics-mr1
Glenn Kasten [Wed, 4 Jan 2012 20:38:13 +0000 (12:38 -0800)]
libmedia new can't fail on Android
Change-Id: Ie79dd5abb8078b35474bf0f1b3a6ff994a3a3360
Glenn Kasten [Wed, 11 Jan 2012 19:40:27 +0000 (11:40 -0800)]
Merge "Fix race in AudioSystem::getInputBufferSize"
Glenn Kasten [Wed, 11 Jan 2012 19:39:06 +0000 (11:39 -0800)]
Merge "Fix bug in JetPlayer::loadFromFile"
Glenn Kasten [Wed, 11 Jan 2012 19:38:21 +0000 (11:38 -0800)]
Merge "Simplify range check for audio_mode_t"
Eric Laurent [Wed, 11 Jan 2012 19:14:30 +0000 (11:14 -0800)]
am
05985a2e: am
c15cf3d2: Merge "stagefright: limit audio gap for looped playback" into ics-mr1
* commit '
05985a2e2ae5c21fbf69d2884c59ea475fce7f6d':
stagefright: limit audio gap for looped playback
Eric Laurent [Wed, 11 Jan 2012 19:12:45 +0000 (11:12 -0800)]
am
c15cf3d2: Merge "stagefright: limit audio gap for looped playback" into ics-mr1
* commit '
c15cf3d286f3d63d141b73f639a5a61ddb028029':
stagefright: limit audio gap for looped playback
Glenn Kasten [Wed, 11 Jan 2012 18:25:23 +0000 (10:25 -0800)]
Merge "Use correct type for hardware call state"
Glenn Kasten [Wed, 11 Jan 2012 18:23:34 +0000 (10:23 -0800)]
Merge "Declare IAudioTrack methods in binder opcode order"
Glenn Kasten [Wed, 11 Jan 2012 18:23:04 +0000 (10:23 -0800)]
Merge "Use consistent style of & reference for AutoMutex"
Glenn Kasten [Wed, 11 Jan 2012 18:22:45 +0000 (10:22 -0800)]
Merge "Fix missing tag in mediaserver ServiceManager: log"
Glenn Kasten [Wed, 11 Jan 2012 18:22:19 +0000 (10:22 -0800)]
Merge "AudioFlinger new can't fail"
Glenn Kasten [Wed, 11 Jan 2012 18:01:34 +0000 (10:01 -0800)]
Merge "Fix build warning"
Glenn Kasten [Tue, 10 Jan 2012 17:01:19 +0000 (09:01 -0800)]
Fix race in AudioSystem::getInputBufferSize
It was caching the recording parameters without a mutex.
Change-Id: Ic4b9f621cbc080d224c2233cf3ca3454fc0f19bd
Glenn Kasten [Tue, 10 Jan 2012 16:41:33 +0000 (08:41 -0800)]
Fix bug in JetPlayer::loadFromFile
A long pathname would corrupt the heap due to incorrect use of strncpy.
Also was using hard-coded constant 256 instead of PATH_MAX.
Change-Id: Iba382bbb38624fbc41fec91449f9814f77d4ac0d
Glenn Kasten [Sat, 7 Jan 2012 00:47:31 +0000 (16:47 -0800)]
Simplify range check for audio_mode_t
AudioSystem::setMode previously allowed negative modes, but these were
then rejected by AudioFlinger.
Now negative modes (including AUDIO_MODE_INVALID and AUDIO_MODE_CURRENT)
are explicitly disallowed.
Change-Id: I0bac8fea737c8eb1f5b6afbb893e48739f88d745
Eric Laurent [Mon, 9 Jan 2012 21:47:48 +0000 (13:47 -0800)]
stagefright: limit audio gap for looped playback
The audio gap when looping audio is mostly due to
a delay requested by the audio player when executing
the end of stream message. This is to allow the audio
to be drained out of the pipe before actually stopping the
AudioTrack.
This delay is not needed when looping as the AudioTrack
is not stopped.
The fix consists in ignoring the requested delay when the
looping flag is set in awesome player.
Issue
5800981.
Change-Id: Ib32d2930c53ecebc21ca8440e6e054c7e20db4a5
Steve Block [Mon, 9 Jan 2012 18:35:44 +0000 (18:35 +0000)]
Rename LOG_ASSERT to ALOG_ASSERT DO NOT MERGE
See https://android-git.corp.google.com/g/157519
Bug:
5449033
Change-Id: I8ceb2dba1b031a0fd68d15d146960d9ced62bbf3
Glenn Kasten [Mon, 9 Jan 2012 19:59:17 +0000 (11:59 -0800)]
Merge "By convention const goes before the type specifier"
Glenn Kasten [Tue, 3 Jan 2012 23:28:29 +0000 (15:28 -0800)]
Use cached reference to media.player service
This save unnecessary binder calls
Change-Id: I93a60efc54d9c8fb8fab706cd4477bbfd00ffec8
Glenn Kasten [Mon, 9 Jan 2012 18:25:00 +0000 (10:25 -0800)]
Merge "Replace loop by __builtin_ctz"
Glenn Kasten [Mon, 9 Jan 2012 18:24:40 +0000 (10:24 -0800)]
Merge "Clean up AudioTrack::mActive and stopped()"
Glenn Kasten [Mon, 9 Jan 2012 18:24:09 +0000 (10:24 -0800)]
Merge "suspended() and isSuspended() are const"
Steve Block [Fri, 6 Jan 2012 19:20:56 +0000 (19:20 +0000)]
Rename (IF_)LOGE(_IF) to (IF_)ALOGE(_IF) DO NOT MERGE
See https://android-git.corp.google.com/g/#/c/157220
Bug:
5449033
Change-Id: Ic9c19d30693bd56755f55906127cd6bd7126096c
The Android Open Source Project [Sat, 7 Jan 2012 02:18:36 +0000 (18:18 -0800)]
am
20090b30: Reconcile with ics-mr1-release
* commit '
20090b30340f9382a2eca6a9bed21117f7a27d7e':
The Android Open Source Project [Sat, 7 Jan 2012 02:15:13 +0000 (18:15 -0800)]
Reconcile with ics-mr1-release
Change-Id: Id0aeb30eb75c6c4aee7abbc9877aa5b16389778c
The Android Automerger [Sat, 7 Jan 2012 00:32:11 +0000 (16:32 -0800)]
merge in ics-mr1-release history after reset to ics-mr1
Eino-Ville Talvala [Fri, 6 Jan 2012 23:51:44 +0000 (15:51 -0800)]
am
4bc873e6: am
7944704e: Merge "Switch camera sounds to always use the system enforced audio stream." into ics-mr1
* commit '
4bc873e6ff73200af866f8c9efca66ad8ed23682':
Switch camera sounds to always use the system enforced audio stream.
Eino-Ville Talvala [Fri, 6 Jan 2012 23:49:17 +0000 (15:49 -0800)]
am
7944704e: Merge "Switch camera sounds to always use the system enforced audio stream." into ics-mr1
* commit '
7944704e6c4cc10105838b78d3d8403bf66a066b':
Switch camera sounds to always use the system enforced audio stream.
Glenn Kasten [Fri, 6 Jan 2012 23:03:11 +0000 (15:03 -0800)]
AudioFlinger new can't fail
Change-Id: I7dae05a5ea1c962a9975386eab1fedbbe106ffba
Glenn Kasten [Mon, 19 Dec 2011 22:09:14 +0000 (14:09 -0800)]
Fix missing tag in mediaserver ServiceManager: log
Change-Id: I2d5525ffee0b66c0bc361a02b313ec4dd61e6552
Glenn Kasten [Tue, 3 Jan 2012 22:50:23 +0000 (14:50 -0800)]
Declare IAudioTrack methods in binder opcode order
This makes it easier to match up the declarations.
Change-Id: I0996c20b2903b778b356dfe52b07b0ec634855dd
Glenn Kasten [Thu, 5 Jan 2012 23:41:56 +0000 (15:41 -0800)]
Use consistent style of & reference for AutoMutex
AutoMutex, which is a typedef for Mutex::Autolock, is overloaded for
either a reference (&) or pointer (*) parameter, but we prefer to use
the reference form when the mutex is known at compile time.
Change-Id: I3515e6d6ab7959b2356a27fa3b04fd49e42cb31e
Glenn Kasten [Wed, 4 Jan 2012 19:02:33 +0000 (11:02 -0800)]
Use correct type for hardware call state
Change-Id: Ic6d98b129e3ec653df1d8f7e829adf8dccb4f378
Eino-Ville Talvala [Thu, 5 Jan 2012 23:34:53 +0000 (15:34 -0800)]
Switch camera sounds to always use the system enforced audio stream.
Instead of picking between the music stream and the enforced audio
stream, change the camera service to always play sounds through
enforced system stream. Also update the currently-hidden CameraSound
API to match.
Bug:
5778365
Change-Id: I3cc64b1d1ff567dbac8020a665d5b19846197ff3
Glenn Kasten [Fri, 6 Jan 2012 15:46:30 +0000 (07:46 -0800)]
By convention const goes before the type specifier
Change-Id: I70203abd6a6f54e5bd9f1412800cc01212157e58
Glenn Kasten [Fri, 6 Jan 2012 15:47:26 +0000 (07:47 -0800)]
Fix build warning
Change-Id: Ic99608d0c14ed56c02f036e0bbaaae1b16bab8ba
Steve Block [Thu, 5 Jan 2012 23:22:43 +0000 (23:22 +0000)]
Rename (IF_)LOGW(_IF) to (IF_)ALOGW(_IF) DO NOT MERGE
See https://android-git.corp.google.com/g/157065
Bug:
5449033
Change-Id: I00a4b904f9449e6f93b7fd35eac28640d7929e69
Glenn Kasten [Thu, 15 Dec 2011 22:38:29 +0000 (14:38 -0800)]
Replace loop by __builtin_ctz
Using the builtin is faster on some platforms, for example on ARM it's
19 instructions instead of 13, and is O(1) instead of O(n). Of course,
track creation is an inherently slow operation, so this doesn't matter
much now. But if we add support for virtual tracks, then physical tracks
will be allocated/freed more frequently. Also just on principle ...
Change-Id: I3f590934092bd7a1869cbedbc7357928aa5cc8ff
Steve Block [Thu, 5 Jan 2012 22:07:02 +0000 (14:07 -0800)]
Merge "Rename (IF_)LOGI(_IF) to (IF_)ALOGI(_IF) DO NOT MERGE"
Glenn Kasten [Wed, 4 Jan 2012 17:36:37 +0000 (09:36 -0800)]
Use audio_format_t consistently
Was int, uint32_t, uint16_t, and uint8_t with 2-bit bitfield.
Also replace 0 by AUDIO_FORMAT_DEFAULT and replace 1 by
AUDIO_FORMAT_PCM_16_BIT.
Change-Id: Ia8804f53f1725669e368857d5bb2044917e17975
The Android Open Source Project [Thu, 5 Jan 2012 20:08:48 +0000 (12:08 -0800)]
am
d32ee5ba: Reconcile with ics-mr1-release
* commit '
d32ee5baa057992431263907ec5cf8f19b43c043':
Glenn Kasten [Tue, 3 Jan 2012 17:42:47 +0000 (09:42 -0800)]
Clean up AudioTrack::mActive and stopped()
mActive is protected by mLock; volatile is meaningless on SMP.
Fixed a couple of places where mActive was accessed without a lock:
- stopped()
- processAudioBuffer()
Added stopped_l() for cases where we already hold the lock.
Made mActive a bool not int.
Moved down a lock in setPosition that was being acquired too early.
Change-Id: I73ff368e991c0db9f9472df0b3f96fd33fcc7311
Glenn Kasten [Wed, 4 Jan 2012 19:01:11 +0000 (11:01 -0800)]
suspended() and isSuspended() are const
Change-Id: I04b95970b5a645b64e7e64fffd46d868354dda66
Glenn Kasten [Thu, 5 Jan 2012 16:20:10 +0000 (08:20 -0800)]
Merge "Remove the notion of "active track" from mixer"
The Android Open Source Project [Thu, 5 Jan 2012 15:45:28 +0000 (07:45 -0800)]
Reconcile with ics-mr1-release
Change-Id: I8a74c329fc7dd115e36d5180fab8e8680d6df4f8
Glenn Kasten [Thu, 5 Jan 2012 15:38:29 +0000 (07:38 -0800)]
Merge "Use the standard CC_LIKELY and CC_UNLIKELY macros"
Glenn Kasten [Thu, 15 Dec 2011 17:51:17 +0000 (09:51 -0800)]
Use the standard CC_LIKELY and CC_UNLIKELY macros
Several source files privately defined macros LIKELY and UNLIKELY in terms
of __builtin_expect. But <cutils/compiler.h> already has CC_LIKELY and
CC_UNLIKELY which are intended for this purpose. So rename the private
uses to use the standard names.
In addition, AudioFlinger was relying on the macro expanding to extra ( ).
Change-Id: I2494e087a0c0cac0ac998335f5e9c8ad02955873
Eric Laurent [Thu, 5 Jan 2012 01:39:12 +0000 (17:39 -0800)]
resolved conflicts for merge of
1a4b9939 to master
Change-Id: I0c910d391a38a916d8431f7d1f5b82e39e1a66c2
The Android Automerger [Thu, 5 Jan 2012 00:42:23 +0000 (16:42 -0800)]
merge in ics-mr1-release history after reset to ics-mr1
Eric Laurent [Wed, 4 Jan 2012 21:00:49 +0000 (13:00 -0800)]
am
fb578b6b: Merge "audioflinger: fix clicks on 48kHz audio." into ics-mr1
* commit '
fb578b6b88af51812b71a86ad4dd3a7de56e394b':
audioflinger: fix clicks on 48kHz audio.
Steve Block [Wed, 4 Jan 2012 20:05:49 +0000 (20:05 +0000)]
Rename (IF_)LOGI(_IF) to (IF_)ALOGI(_IF) DO NOT MERGE
See https://android-git.corp.google.com/g/156801
Bug:
5449033
Change-Id: Ib08fe86d23db91ee153e9f91a99a35c42b9208ea
Steve Block [Tue, 20 Dec 2011 16:23:08 +0000 (16:23 +0000)]
Rename (IF_)LOGD(_IF) to (IF_)ALOGD(_IF) DO NOT MERGE
See https://android-git.corp.google.com/g/156016
Bug:
5449033
Change-Id: I4c4e33bb9df3e39e11cd985e193e6fbab4635298
Eric Laurent [Fri, 23 Dec 2011 00:08:41 +0000 (16:08 -0800)]
audioflinger: fix clicks on 48kHz audio.
The calculation done in prepareTracks_l() for the minimum amount
off frames needed to mix one output buffer had 2 issues:
- the additional sample needed for interpolation was not included
- the fact that the resampler does not acknowledge the frames consumed
immediately after each mixing round but only once all frames requested have been used
was not taken into account.
Thus the number of frames available in track buffer could be considered sufficient although
it was not and the resampler would abort producing a short silence perceived as a click.
Issue
5727099.
Change-Id: I7419847a7474c7d9f9170bedd0a636132262142c
Glenn Kasten [Mon, 19 Dec 2011 23:06:39 +0000 (15:06 -0800)]
Remove the notion of "active track" from mixer
This is a first step towards making the mixer more object-oriented.
Change-Id: Ifd445d0e471023a7f5c82e934736ffc95ba1b05b
The Android Open Source Project [Tue, 20 Dec 2011 21:11:53 +0000 (13:11 -0800)]
am
21cb1290: Reconcile with ics-mr1-release
* commit '
21cb1290b578a4bceac003c276c00911fcd6c9d4':
The Android Open Source Project [Tue, 20 Dec 2011 21:08:33 +0000 (13:08 -0800)]
Reconcile with ics-mr1-release
Change-Id: I123bd5ae61ab8c4f6de1a6640faf0bab89f6aa78
Glenn Kasten [Tue, 20 Dec 2011 18:29:49 +0000 (10:29 -0800)]
Merge "Remove dead code"
Eric Laurent [Tue, 20 Dec 2011 01:26:24 +0000 (17:26 -0800)]
Merge "audio effects: rename configure command"
Eric Laurent [Fri, 16 Dec 2011 23:30:36 +0000 (15:30 -0800)]
audio effects: rename configure command
Renamed audio effect library interface command for audio format
configuration from EFFECT_CMD_CONFIGURE to EFFECT_CMD_SET_CONFIG.
This makes the naming more consistent with other exixsting commands
and allow adding a new command to get the configuration (EFFECT_CMD_GET_CONFIG).
Same change for reverse channel configuration renamed from
EFFECT_CMD_CONFIGURE_REVERSE to EFFECT_CMD_SET_CONFIG_REVERSE.
Implemented EFFECT_CMD_GET_CONFIG in exisitng effect libraries.
Change-Id: Ia7b1c620f13797fe5aceb3b0b4acbacce09fb067
Glenn Kasten [Thu, 15 Dec 2011 17:53:12 +0000 (09:53 -0800)]
Remove dead code
Change-Id: Icf23f7f90fdeb660f4015f22cf239e6d05f5d03c
Hong Teng [Fri, 16 Dec 2011 23:17:46 +0000 (15:17 -0800)]
Merge "Fix for
5754998 testPerformanceTransitionWithEffectOverlapping test failed."
Glenn Kasten [Fri, 16 Dec 2011 23:10:36 +0000 (15:10 -0800)]
Merge "Use constants for 2 and 32"
Glenn Kasten [Fri, 16 Dec 2011 23:02:30 +0000 (15:02 -0800)]
Merge "Extract out audio DSP code to utility library"
Glenn Kasten [Tue, 13 Dec 2011 19:52:35 +0000 (11:52 -0800)]
Use constants for 2 and 32
Change-Id: If820dfd58b6df258570750610a07af99598d9e53
Hong Teng [Fri, 16 Dec 2011 00:56:49 +0000 (16:56 -0800)]
Fix for
5754998 testPerformanceTransitionWithEffectOverlapping test failed.
-Fix the index is out of sync frame array boundary when seeking to end of a 3gp file.
Validation only were done for video editor cts test and Movie Studio application.
Futher validation may include media player, youtube application,.etc.
Change-Id: Ib01314f1768cfab702dbe25c307553ace0b85e77
Glenn Kasten [Fri, 16 Dec 2011 21:31:33 +0000 (13:31 -0800)]
Merge "Use switch in AudioMixer::setParameter"
Glenn Kasten [Thu, 5 May 2011 15:19:00 +0000 (08:19 -0700)]
Use switch in AudioMixer::setParameter
Replace series of if/then/elses by easier-to-read switch. Also return
void instead of status_t, since callers weren't checking it. Assert on
bad input parameters.
Change-Id: Ie1f0a297977b28501d20e1af819afed9b4750616
Glenn Kasten [Thu, 15 Dec 2011 23:32:27 +0000 (15:32 -0800)]
deleteTrackName now asserts on bad input parameter
This is safe, as the input parameter is always track->name(),
which must be valid.
Change-Id: Iea8ea3a5706c27026335526ba8851030d00681f8
The Android Automerger [Fri, 16 Dec 2011 20:01:35 +0000 (12:01 -0800)]
merge in ics-mr1-release history after reset to ics-mr1
Glenn Kasten [Thu, 15 Dec 2011 17:52:39 +0000 (09:52 -0800)]
Extract out audio DSP code to utility library
Change-Id: Ib8ce72028a7ea30e82baa518e381370e820ebbd0
The Android Open Source Project [Fri, 16 Dec 2011 16:09:57 +0000 (08:09 -0800)]
am
57ea96a4: Reconcile with ics-mr1-release
* commit '
57ea96a4b3273aa8592873355a2f79c74b1abec1':
SurfaceMediaSource: use the HW_TEXTURE usage bit
SurfaceMediaSource: use the HW_TEXTURE usage bit
The Android Open Source Project [Fri, 16 Dec 2011 16:08:07 +0000 (08:08 -0800)]
Reconcile with ics-mr1-release
Change-Id: Ib2cbd92ed2d3567e22921b66ca2202a968fdd0da
Glenn Kasten [Fri, 16 Dec 2011 15:30:14 +0000 (07:30 -0800)]
Merge "setActiveTrack and setBufferProvider can't fail"
Glenn Kasten [Fri, 16 Dec 2011 15:29:57 +0000 (07:29 -0800)]
Merge "Simplify enable/disable mixing"
Jamie Gennis [Fri, 16 Dec 2011 02:14:05 +0000 (18:14 -0800)]
SurfaceMediaSource: use the HW_TEXTURE usage bit
This change makes SurfaceMediaSource request Gralloc buffers with the
HW_TEXTURE usage bit rather than the VIDEO_ENCODER bit. It is a
temporary workaround for what is likely a Gralloc bug.
Bug:
5771063
Change-Id: I9cce615e80fc14a1644fae27ba06970c262dd179
The Android Automerger [Fri, 16 Dec 2011 04:50:02 +0000 (20:50 -0800)]
merge in ics-mr1-release history after reset to ics-mr1
Jamie Gennis [Fri, 16 Dec 2011 02:14:05 +0000 (18:14 -0800)]
SurfaceMediaSource: use the HW_TEXTURE usage bit
This change makes SurfaceMediaSource request Gralloc buffers with the
HW_TEXTURE usage bit rather than the VIDEO_ENCODER bit. It is a
temporary workaround for what is likely a Gralloc bug.
Bug:
5771063
Change-Id: I9cce615e80fc14a1644fae27ba06970c262dd179
Dave Burke [Fri, 16 Dec 2011 03:27:17 +0000 (19:27 -0800)]
am
0cc1e42d: am
ac279de9: Merge "SurfaceMediaSource: use the HW_TEXTURE usage bit" into ics-mr1
* commit '
0cc1e42d74e09c259a6da0abc9009d1abf7537b1':
SurfaceMediaSource: use the HW_TEXTURE usage bit
Dave Burke [Fri, 16 Dec 2011 03:25:07 +0000 (19:25 -0800)]
am
ac279de9: Merge "SurfaceMediaSource: use the HW_TEXTURE usage bit" into ics-mr1
* commit '
ac279de9d9fcfd59a336298cfdd5b1f4ab6d038f':
SurfaceMediaSource: use the HW_TEXTURE usage bit
Jamie Gennis [Fri, 16 Dec 2011 02:14:05 +0000 (18:14 -0800)]
SurfaceMediaSource: use the HW_TEXTURE usage bit
This change makes SurfaceMediaSource request Gralloc buffers with the
HW_TEXTURE usage bit rather than the VIDEO_ENCODER bit. It is a
temporary workaround for what is likely a Gralloc bug.
Bug:
5771063
Change-Id: I9cce615e80fc14a1644fae27ba06970c262dd179
Glenn Kasten [Fri, 16 Dec 2011 00:17:04 +0000 (16:17 -0800)]
Merge "Improve resistance to leaks for ConfigEvent"
Glenn Kasten [Fri, 16 Dec 2011 00:16:55 +0000 (16:16 -0800)]
Merge "Use NULL not 0 for pointers"
Glenn Kasten [Thu, 15 Dec 2011 23:46:46 +0000 (15:46 -0800)]
setActiveTrack and setBufferProvider can't fail
Return void, not status_t, from setActiveTrack and setBufferProvider.
These methods returned status_t, but the callers never checked the
return value. Since these aren't externally visible APIs, they now
return void, and assert on bad input parameters.
Change-Id: I530ed29484596ae41e8659826ca425149c51c2a1
Glenn Kasten [Thu, 15 Dec 2011 22:54:01 +0000 (14:54 -0800)]
Simplify enable/disable mixing
The MIXING enum isn't needed, and now returns void instead of status_t.
Change-Id: Ibe4ec24081d75ad4ab78b9c7191fc9077959c4e9
Glenn Kasten [Tue, 13 Dec 2011 19:50:00 +0000 (11:50 -0800)]
Improve resistance to leaks for ConfigEvent
A Vector of pointers is risky, as there is no ownership (and the
ThreadBase destructor was not deleting them, so if there were any left
over at end it would leak). Replaced by a Vector of values.
Change-Id: Iddde72dc30134adfcf724dec26cbe0a742509b8c
Glenn Kasten [Thu, 15 Dec 2011 18:46:14 +0000 (10:46 -0800)]
Merge "Fix build error when verbose logging is enabled"
Glenn Kasten [Tue, 13 Dec 2011 19:53:26 +0000 (11:53 -0800)]
Use NULL not 0 for pointers
Change-Id: Iab3f9abbdab617dc5a599e657ec46a0b0a002eef
Glenn Kasten [Thu, 15 Dec 2011 16:27:40 +0000 (08:27 -0800)]
Fix build error when verbose logging is enabled
Change-Id: I30f4e5d35d1b24dce2e6152a13fd0fb61414a3e2