summaryrefslogtreecommitdiffstats
path: root/media
Commit message (Collapse)AuthorAgeFilesLines
* Adding delay estimation to input audio.xians@chromium.org2011-10-049-11/+38
| | | | | | | | | | | | | | On linux, snd_pcm_delay API will be used to get the delay value from ALSA; On mac, the AudioQueue implementation will just use the buffer size as delay value since we may not use this impl for recording; the low-latency AudioUnits implementation will use some AudioUnitGetProperty() API to get the accurate delay value from the driver; On windows, for the Wave implementation, based on the tests we do, it should be OK to use the buffer size as the delay value; for the to-be-implemented core audio implementation, we will see what is the appropriate way to get the delay there. Review URL: http://codereview.chromium.org/8068005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104002 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash on media teardown when triggered by pipeline errorvrk@google.com2011-10-042-5/+62
| | | | | | | | | | | | | This CL prohibits the media pipeline from posting non-teardown related tasks onto its message loop during teardown. The CL includes a unit test to test for this. BUG=96849 TEST=media_unittests Review URL: http://codereview.chromium.org/8116019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103974 0039d316-1c4b-4281-b951-d872f2087c98
* Fix support for yuv_422 pixel format.scherkus@chromium.org2011-10-048-27/+81
| | | | | | | | | | | | Added pix_fmt field to the VideoDecoderConfig class. The pixel format is passed to the codec_context_ and used to correctly initialize VideoFrames. Patch by shadi@chromium.org: http://codereview.chromium.org/8052002/ BUG=95642 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103961 0039d316-1c4b-4281-b951-d872f2087c98
* Fix issue with getting description of test case with unicode character in ↵imasaki@google.com2011-10-047-17/+38
| | | | | | | | | | | | | | | | | the layout test analyzer. Due to this issue, some test group fail (e.g., fast/encoding). The solution is to return empty string as description in that case. Other changes are added: * Fix several unit test issue * support svg file for layout tests * sort out command line generation in the runner. * add owner's email address for a given test group in the dashboard (if available in run config file) Review URL: http://codereview.chromium.org/8118024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103958 0039d316-1c4b-4281-b951-d872f2087c98
* Modification based on user's feedback in the layout test analyzer.imasaki@google.com2011-10-036-51/+143
| | | | | | | | Most of the modifications are cosmetic change. Review URL: http://codereview.chromium.org/8103013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103732 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up VideoDecoderConfig and replace VideoCodecInfo with a bool.scherkus@chromium.org2011-10-0110-111/+109
| | | | | | | | | | | Similar to AudioDecoderConfig that was introduced in r102183, add Initialize() and IsValidConfig() to VideoDecoderConfig and update documentation. This helps pave the way to remove DemuxerStream::GetAVStream(). Since natural_size isn't used by neither VideoDecoderConfig nor VideoDecodeEngines, remove it from both and replace VideoCodecInfo with a bool. Review URL: http://codereview.chromium.org/8084021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103603 0039d316-1c4b-4281-b951-d872f2087c98
* Remove no-longer-necessary conditional mention of an already-included source ↵fischman@chromium.org2011-10-011-5/+0
| | | | | | | | | | | | file. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/7966030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103599 0039d316-1c4b-4281-b951-d872f2087c98
* Simplify some callbacks-related code now that Bind can curry Callback->Closure.fischman@chromium.org2011-09-304-55/+18
| | | | | | | | | | BUG=none TEST=media_unittests, trybots Review URL: http://codereview.chromium.org/8099001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103553 0039d316-1c4b-4281-b951-d872f2087c98
* Improve ChunkDemuxer so AppendData() calls no longer have to be done on ↵acolwell@chromium.org2011-09-3011-134/+415
| | | | | | | | | | | | element boundries. BUG=86536 TEST=ChunkDemuxerTest::TestAppendingInPieces Review URL: http://codereview.chromium.org/7981024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103543 0039d316-1c4b-4281-b951-d872f2087c98
* gcc 4.6 warnings cleanuppph34r@gmail.com2011-09-302-9/+1
| | | | | | | | | | BUG=87490 TEST=compile with gcc 4.6 Review URL: http://codereview.chromium.org/8050008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103513 0039d316-1c4b-4281-b951-d872f2087c98
* Follow-up cleanup promised during r103376's CR.fischman@chromium.org2011-09-3013-198/+69
| | | | | | | | | | | | | | | | | | | | | | | - mock_task.{h,cc} is gone. - MockCallback is now MockClosure, and its commentary brought up to date (the commentary checked in was a mix of old and attempt-at-new that never materialized) - NewExpectedCallback is NewExpectedClosure. A bit of background on FooCallback vs. FooCB: when acolwell@ & I did the first conversions to the new world, everything was named FooCallback. I proposed using FooCB for the migrated ones as a way to both easily visually differentiate as well as save characters (!). Now that we have an additional "don't typedef Closures" guideline I like having FooCB for non-closure new-style callbacks, and FooClosure for new-style closures. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/8085017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103464 0039d316-1c4b-4281-b951-d872f2087c98
* Removing mfplayer and mfdecoder tools.scherkus@chromium.org2011-09-308-1846/+0
| | | | | | | | They've been unused for a year and are trivial to revert should we need them again. Review URL: http://codereview.chromium.org/8083011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103430 0039d316-1c4b-4281-b951-d872f2087c98
* Here are more exports needed for content_unittests to link in the component ↵dpranke@chromium.org2011-09-302-2/+5
| | | | | | | | | | | | build. R=jam@chromium.org, darin@chromium.org BUG=90442 TEST= Review URL: http://codereview.chromium.org/8054037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103394 0039d316-1c4b-4281-b951-d872f2087c98
* Convert all of media/ over to the new base::{Bind,Callback} hotness.fischman@chromium.org2011-09-2987-1120/+781
| | | | | | | | | | | | | | | | | | | Mostly this was a rote conversion, replacing: - Pass-by-pointer CallbackN<>'s become pass-by-const-ref Callback<>'s. - scoped_ptr<CallbackN<>> members become Callback<> members. - several dedicated FooCallback typedefs became base::Closure. Because it was only used in a small handful of places and only in one place profitably, I deleted AutoCallbackRunner. Because it tickles a Bind bug I disabled mfdecoder in .gyp (about to get deleted in a scherkus CL). BUG=none TEST=media_tests, trybots Review URL: http://codereview.chromium.org/8071007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103376 0039d316-1c4b-4281-b951-d872f2087c98
* Remove no-longer-used FFmpeg index-related helpers.fischman@chromium.org2011-09-293-196/+0
| | | | | | | | | | | | These methods served AdaptiveDemuxer which died in r103279. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/8084005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103334 0039d316-1c4b-4281-b951-d872f2087c98
* Remove AdaptiveDemuxer since it's been replaced with ChunkingDemuxer.fischman@chromium.org2011-09-297-1398/+2
| | | | | | | | | | BUG=none TEST=trybots Review URL: http://codereview.chromium.org/8065023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103279 0039d316-1c4b-4281-b951-d872f2087c98
* Re-enable SSSE3-optimized functions.hbono@chromium.org2011-09-292-2/+18
| | | | | | | | | | This change fixes a stupid problem in getcpuid() (I forgot declaring cpuid breaks rbx) and re-enables my SSSE3-optimized RGB->YUV conversion. BUG=none TEST=YUVConvertTest.* Review URL: http://codereview.chromium.org/7979016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103237 0039d316-1c4b-4281-b951-d872f2087c98
* Properly scope the lifetime of the |PPB_VideoCapture_Impl|.viettrungluu@chromium.org2011-09-282-1/+15
| | | | | | | | | | | | | | | | | | Since it/its proxy are |media::VideoCapture::EventHandler|s, they must remain alive from the |StartCapture()| call until |OnRemoved()| is received. (Precisely, the |PPB_VideoCapture_Impl| owns a |PlatformVideoCapture(Impl)|, which owns a |VideoCaptureHandlerProxy|. Keeping the |PPB_VideoCapture_Impl| alive keeps all these things alive.) BUG=none TEST=Closing a page with a PPAPI plugin using the video capture interface doesn't cause the renderer to crash. Other similar things should also work more or less properly (and at least not crash). Review URL: http://codereview.chromium.org/8052023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103138 0039d316-1c4b-4281-b951-d872f2087c98
* Add OnRemoved() in VideoCapture::EventHandler APIwjia@chromium.org2011-09-273-0/+11
| | | | | | | | | | This is to allow client to know when the event handler can be deleted. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/8037055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103016 0039d316-1c4b-4281-b951-d872f2087c98
* Fix the bug caused by refactoring in the layout test analyzer.imasaki@google.com2011-09-271-3/+2
| | | | | | Review URL: http://codereview.chromium.org/8060018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102995 0039d316-1c4b-4281-b951-d872f2087c98
* Make media/ PIC-clean on 32-bit Linux.agl@chromium.org2011-09-271-2/+5
| | | | | | | | | | | | This change sets the asm defines so that code in media/ doesn't require text relocations on 32-bit Linux. BUG=87704 TEST=Try running Chrome on Fedora Review URL: http://codereview.chromium.org/8046012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102920 0039d316-1c4b-4281-b951-d872f2087c98
* The runner script runs the analyzer with various parameters (especially, ↵imasaki@google.com2011-09-237-117/+504
| | | | | | | | | | | | group names defined in the CSV file). The runner also creates dashboard page to summarize the results in simple HTML table. Also, fixed the issue when there is no previous result to compare in the result directory. In that case, it still updates the trend graph. Review URL: http://codereview.chromium.org/7918010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102568 0039d316-1c4b-4281-b951-d872f2087c98
* Exclude ffmpeg from the build when building for Android.peter@chromium.org2011-09-231-65/+126
| | | | | | | | | | | | | | | | Conditionalize the ffmpeg dependency and exclude files which rely on it for the media and media_unittest targets. Disable the ffmpeg_unittests, ffmpeg_tests and media_bench for Android targets as they require it. This change also conditionalizes the inclusion of yasm_compile.gyp, as yasm is only used for the ia32 and x86 target architectures. BUG= TEST= Review URL: http://codereview.chromium.org/7977023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102474 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r102332 - export more symbols needed for the component builddpranke@chromium.org2011-09-231-1/+1
| | | | | | | | | | | | | of content and also move a couple of functions to content that should've been moved earlier. BUG=90442 TBR=jam@chromium.org,allanwoj@chromium.org TEST= Review URL: http://codereview.chromium.org/7969028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102405 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Fix more symbol exports and a compile failure for the component ↵dpranke@chromium.org2011-09-221-1/+1
| | | | | | | | | | | | | build of" TBR=jam@chromium.org BUG=90442 TEST= Review URL: http://codereview.chromium.org/7967013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102334 0039d316-1c4b-4281-b951-d872f2087c98
* Fix more symbol exports and a compile failure for the component build ofdpranke@chromium.org2011-09-221-1/+1
| | | | | | | | | | | | content. R=jam@chromium.org,crogers@google.com,allanwoj@chromium.org BUG=90442 TEST=none Review URL: http://codereview.chromium.org/7981049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102332 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce AudioDecoderConfig to migrate away from GetAVStream().scherkus@chromium.org2011-09-2126-85/+426
| | | | | | | | | Instead add DemuxerStream::audio_decoder_config() to break FFmpegAudioDecoder's dependency on the AVCodecContext object maintained by FFmpegDemuxer. Review URL: http://codereview.chromium.org/7867051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102183 0039d316-1c4b-4281-b951-d872f2087c98
* Delete Tracked, and move Location to its own file.ajwong@chromium.org2011-09-211-0/+1
| | | | | | | | | | | | | The Birth/Death tracking of tasks has been moved out-of-band into MessageLoop's PendingTask structure. Thus, Task no longer needs to inherit from Tracked. Since Task was the only child of Tracked, delete the Tracked class and move Location to its own file. BUG=none TEST=builds Review URL: http://codereview.chromium.org/7879006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102132 0039d316-1c4b-4281-b951-d872f2087c98
* Add a lock around getting and setting of source_ to avoid possible compiler ↵ajwong@chromium.org2011-09-212-11/+32
| | | | | | | | | | | | | | reorderings. Also makes it very explicit that multi-threaded access is occuring. BUG=24801 TEST=will see how TSan handles this. Review URL: http://codereview.chromium.org/7888055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102067 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r101418: Fix aspect ratio and clarify video frame dimensionsvrk@google.com2011-09-1926-206/+199
| | | | | | | | | | | | Fixes shared build errors for windows and linux. BUG=18941,94861 TEST=shared builds compile TBR=acolwell Review URL: http://codereview.chromium.org/7932005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101808 0039d316-1c4b-4281-b951-d872f2087c98
* Remove base/scoped_ptr.hakalin@chromium.org2011-09-182-2/+2
| | | | | | | | | | | Fix remaining users to use base/memory/scoped_ptr.h. BUG= TEST= Review URL: http://codereview.chromium.org/7930009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101715 0039d316-1c4b-4281-b951-d872f2087c98
* Rewriting FFmpegAudioDecoder and eliminating DecoderBase.scherkus@chromium.org2011-09-178-658/+257
| | | | | | | | | BUG=93379 TEST=media_unittests, layout tests Review URL: http://codereview.chromium.org/6901135 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101607 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 101551 - Add script to generate and run layout tests for problematic ↵vrk@google.com2011-09-163-211/+0
| | | | | | | | | | | | | | video files. BUG=NONE TEST=NONE Review URL: http://codereview.chromium.org/7888020 TBR=vrk@google.com Review URL: http://codereview.chromium.org/7918033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101570 0039d316-1c4b-4281-b951-d872f2087c98
* Replace AudioDecoderConfig with simple accessors on AudioDecoder.scherkus@chromium.org2011-09-1617-103/+73
| | | | | | | | AudioDecoderConfig was being used as a struct for outputting format information to an AudioRenderer. This was confusing and in contrast to VideoDecoderConfig, which is is actually used as input to configure a video decoder! Review URL: http://codereview.chromium.org/7796033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101563 0039d316-1c4b-4281-b951-d872f2087c98
* Add script to generate and run layout tests for problematic video files.vrk@google.com2011-09-163-0/+211
| | | | | | | | | BUG=NONE TEST=NONE Review URL: http://codereview.chromium.org/7888020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101551 0039d316-1c4b-4281-b951-d872f2087c98
* Reorganize YUV scalers (Continued)hclam@chromium.org2011-09-1616-1857/+111
| | | | | | | | | | | | | | | | | | | | After rewriting assembly code in YASM we can finally move all these different versions of files in the same folder, i.e. media/base/simd. After this change the main entry point will be: yuv_convert.cc and yuv_convert.h It then calls into the internal functions under media/base/simd. After this change I'll move all the color conversion files to media/csc. BUG=None TEST=Tree is gree Review URL: http://codereview.chromium.org/7888012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101507 0039d316-1c4b-4281-b951-d872f2087c98
* Write tests for FFmpegAudioDecoder so I can rewrite the class sans regressions.scherkus@chromium.org2011-09-164-2/+206
| | | | | | | | | TEST=media_unittests BUG=96773 Review URL: http://codereview.chromium.org/7891050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101429 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 101418 - Fix aspect ratio and clarify video frame dimensionsvrk@google.com2011-09-1625-196/+206
| | | | | | | | | | | | BUG=18941,94861 TEST=video-aspect-ratio.html Review URL: http://codereview.chromium.org/7864009 TBR=vrk@google.com Review URL: http://codereview.chromium.org/7919006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101420 0039d316-1c4b-4281-b951-d872f2087c98
* Fix aspect ratio and clarify video frame dimensionsvrk@google.com2011-09-1625-206/+196
| | | | | | | | | BUG=18941,94861 TEST=video-aspect-ratio.html Review URL: http://codereview.chromium.org/7864009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101418 0039d316-1c4b-4281-b951-d872f2087c98
* Remove hardcoded text 'media' in the layout test analyzer.imasaki@google.com2011-09-152-92/+121
| | | | | | | | | Added functionality to run the analzyer for a general layout test group such as 'compositing'. Review URL: http://codereview.chromium.org/7888061 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101318 0039d316-1c4b-4281-b951-d872f2087c98
* Add API to query for the audio buffer size.rtoy@google.com2011-09-152-0/+22
| | | | | | | | | | | | | This is essentially identical to the GetAudioHardwareSampleRate API, except we get the "optimal" hardware buffer size to be used. BUG=None TEST=None Review URL: http://codereview.chromium.org/7837030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101249 0039d316-1c4b-4281-b951-d872f2087c98
* There is a complain from Valgrind about invalid memory access in ↵xians@chromium.org2011-09-153-14/+23
| | | | | | | | | | | | | | snd_device_name_hint(-1, ..) // -1 means all cards. It looks like the problem is actually because we use -1 to loop through all the soundcards. And this patch will loop through the soundcard manually, which should resolve the Valgrind problem. Bug=96207 Test=media_unittests with Valgrind Review URL: http://codereview.chromium.org/7888011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101240 0039d316-1c4b-4281-b951-d872f2087c98
* Windows-only change -- issue is worst on Windows because of pre-buffering. ↵enal@chromium.org2011-09-152-2/+24
| | | | | | | | | | | | | | | | | On Linux problem is much milder, and on Mac we cannot yield high-priority audio thread... Very short-term change: while working on proper long-term solution, workaround for race condition that causes clicks and bad audio stream. Yield if reader was called too quickly after previous call, as there are good chances renderer thread did not fill the buffer yet. BUG=http://code.google.com/p/chromium/issues/detail?id=61022 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=99236 Review URL: http://codereview.chromium.org/7755001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101231 0039d316-1c4b-4281-b951-d872f2087c98
* Fixing dropped seeks.serya@chromium.org2011-09-151-1/+1
| | | | | | | | | | BUG=92251,chromium-os:19888 TEST=None Review URL: http://codereview.chromium.org/7890012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101197 0039d316-1c4b-4281-b951-d872f2087c98
* Add one more vorbis packet for testing purposes.scherkus@chromium.org2011-09-142-2/+6
| | | | | | | | TBR=acolwell Review URL: http://codereview.chromium.org/7888043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101090 0039d316-1c4b-4281-b951-d872f2087c98
* Resubmit - Rewrite color space conversions suite using YASM"hclam@chromium.org2011-09-1428-192/+1891
| | | | | | | | | | | | I'll watch the official buildbot this time. TBR=ajwong, dhollowa BUG=None TEST=None Review URL: http://codereview.chromium.org/7891039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101067 0039d316-1c4b-4281-b951-d872f2087c98
* Improve the speed of YUVConvertTest.SideBySideRGB.hbono@chromium.org2011-09-141-7/+9
| | | | | | | | | | This change just removes redundant checks in YUVConvertTest.SideBySideRGB to avoid this test from taking long time (especially on our valgrind bots). BUG=none TEST=YUVConvertTest.SideBySideRGB Review URL: http://codereview.chromium.org/7867033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101054 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 100915 - Rewrite color space conversions suite using YASMdhollowa@chromium.org2011-09-1428-1916/+191
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Failed on official Linux 32 builder. (x486?) http://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20Linux/builds/11240/steps/compile/logs/stdio In file included from media/base/yuv_convert.cc:33: /usr/lib/gcc/i486-linux-gnu/4.4.3/include/emmintrin.h:32:3:error: #error "SSE2 instruction set not enabled" make: *** [out/Release/obj.target/yuv_convert/media/base/yuv_convert.o] Error 1 Besides rewrite assembly code in YASM, this change also do runtime selection of routine depending processor support. There are couple advantages in this YASM version. 1. Performance improvement Using YASM we can push less to the stack, this results in ~5% improvement on 32-bits system. 2. MMX and SSE versions are separate SSE version use movntq while MMX version use movq. 3. 64-bits version works universally This means that 64-bits YASM code compile and run on 64-bits windows, mac and linux. While previously it only runs on linux 64. 4. Handling PIC much better PIC code runs and build on 32-bits and 64-bits system. This is better than before which only works on 32-bits mac. 5. Easier maintenance This code uses x86asm.inc for a bunch of nice macros. Having just one YASM verion is better than having inline VC assembly and don't have to worry inline assemly has different register names on mac and linux. I tested on linux 32-bits, linux 32-bits PIC and linux 64-bits that this code is at least as fast as before. BUG=None TEST=media_unittests runs and all passing Review URL: http://codereview.chromium.org/7858003 TBR=hclam@chromium.org Review URL: http://codereview.chromium.org/7887025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101008 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 100916 - Fix builddhollowa@chromium.org2011-09-142-0/+25
| | | | | | | | | | | | | | | | | | | | Failed on official Linux 32 builder. (x486?) http://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20Linux/builds/11240/steps/compile/logs/stdio In file included from media/base/yuv_convert.cc:33: /usr/lib/gcc/i486-linux-gnu/4.4.3/include/emmintrin.h:32:3:error: #error "SSE2 instruction set not enabled" make: *** [out/Release/obj.target/yuv_convert/media/base/yuv_convert.o] Error 1 BUG=None TEST=None Review URL: http://codereview.chromium.org/7892009 TBR=hclam@chromium.org Review URL: http://codereview.chromium.org/7885020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101007 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 100917 - Not use SSSE3 code that was merged accidentallydhollowa@chromium.org2011-09-141-1/+17
| | | | | | | | | | | | | | | | | | | | | | | | Failed on official Linux 32 builder. (x486?) http://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20Linux/builds/11240/steps/compile/logs/stdio In file included from media/base/yuv_convert.cc:33: /usr/lib/gcc/i486-linux-gnu/4.4.3/include/emmintrin.h:32:3:error: #error "SSE2 instruction set not enabled" make: *** [out/Release/obj.target/yuv_convert/media/base/yuv_convert.o] Error 1 Disable SSSE3 conversion code as tests are not running. TBR=hbono@chromium.org BUG=None TEST=None Review URL: http://codereview.chromium.org/7888014 TBR=hclam@chromium.org Review URL: http://codereview.chromium.org/7890023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101006 0039d316-1c4b-4281-b951-d872f2087c98