Reorganize YUV scalers (Continued)

Reorganize YUV scalers (Continued)

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

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=101507

[Alpha Left Google, 2011-09-15 18:50:07]

Ping.

[scherkus (not reviewing), 2011-09-16 00:18:57]

sorry for the delay -- LGTM w/ one question

http://codereview.chromium.org/7888012/diff/1/media/media.gyp
File media/media.gyp (right):

http://codereview.chromium.org/7888012/diff/1/media/media.gyp#newcode385
media/media.gyp:385: 'base/simd/yuv_to_rgb_table.cc',
if the only two options are yuv_convert_simd_x86 or yuv_convert_simd_arm but
these files are present in both... any reason to keep them separated instead of
as common code?

I don't care either way -- up to you and what you find is more convenient :)

[Alpha Left Google, 2011-09-16 17:45:04]

On 2011/09/16 00:18:57, scherkus wrote:
> sorry for the delay -- LGTM w/ one question
> 
> http://codereview.chromium.org/7888012/diff/1/media/media.gyp
> File media/media.gyp (right):
> 
> http://codereview.chromium.org/7888012/diff/1/media/media.gyp#newcode385
> media/media.gyp:385: 'base/simd/yuv_to_rgb_table.cc',
> if the only two options are yuv_convert_simd_x86 or yuv_convert_simd_arm but
> these files are present in both... any reason to keep them separated instead
of
> as common code?
> 
> I don't care either way -- up to you and what you find is more convenient :)

I prefer to include them in the targets, it's harder to follow i think to
declare an extra target just for the common code.