Remove duplicate CPU detection code; use base::CPU instead.

Remove duplicate CPU detection code; use base::CPU instead.

BUG=none
TEST=media_unittests.


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

[Sergey Ulanov, 2012-06-08 19:11:18]

remoting - lGTM

[DaleCurtis, 2012-06-09 02:36:08]

Removed static initializers.  There should be ~0 performance difference since
all of these calls are only called once to set each procedure and the existing
code was already re-calling cpuid each time.

Resolves offline discussion with scherkus.

[scherkus (not reviewing), 2012-06-12 02:11:35]

lgtm+++++++++++

https://chromiumcodereview.appspot.com/10537082/diff/3001/media/base/yuv_conv...
File media/base/yuv_convert.cc (right):

https://chromiumcodereview.appspot.com/10537082/diff/3001/media/base/yuv_conv...
media/base/yuv_convert.cc:131: filter_proc = ChooseFilterYUVRowsProc();
ah so in reality we only execute the base::CPU() code a few times on first run
and that functionally this change doesn't change anything

[commit-bot: I haz the power, 2012-06-12 02:13:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dalecurtis@chromium.org/10537082/3001

[DaleCurtis, 2012-06-12 02:41:40]

+darin for base/cpu.h change, which appears to be a copy/paste error.

[darin (slow to review), 2012-06-12 04:43:27]

LGTM for base/ change

[commit-bot: I haz the power, 2012-06-12 04:50:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dalecurtis@chromium.org/10537082/13001

[commit-bot: I haz the power, 2012-06-12 08:29:34]

Change committed as 141627