Enable WebAudio by default for Android/x86.

Enable WebAudio by default for Android/x86.

about:flags now has a disable WebAudio flag on Android for x86, just like for
arm, as expected. I manually tested that the flag does disable WebAudio.

I verified that on desktop (linux), there is no disable (or enable)
flag, as expected. (Desktop always has WebAudio enabled, and there is
no flag to disable it.)

BUG=353809
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=260429

[Raymond Toy, 2014-03-25 16:39:09]

PTAL at this relatively small change to enable WebAudio by default on
Android/x86. The various conditional code is updated to match Android/arm where
WebAudio has been enabled by default (with an about:flags switch) for some time
now.

[jam, 2014-03-25 17:44:09]

https://codereview.chromium.org/208763003/diff/1/chrome/browser/about_flags.cc
File chrome/browser/about_flags.cc (right):

https://codereview.chromium.org/208763003/diff/1/chrome/browser/about_flags.c...
chrome/browser/about_flags.cc:571: #if defined(ARCH_CPU_ARMEL) ||
defined(ARCH_CPU_X86)
is this ifdef actually needed? afaik in our tree we only support android on arm
or x86 (i.e. not mips, are there others?), so it seems we can remove this ifdef.
also same for other files

[Raymond Toy, 2014-03-25 17:53:04]

https://codereview.chromium.org/208763003/diff/1/chrome/browser/about_flags.cc
File chrome/browser/about_flags.cc (right):

https://codereview.chromium.org/208763003/diff/1/chrome/browser/about_flags.c...
chrome/browser/about_flags.cc:571: #if defined(ARCH_CPU_ARMEL) ||
defined(ARCH_CPU_X86)
On 2014/03/25 17:44:09, jam wrote:
> is this ifdef actually needed? afaik in our tree we only support android on
arm
> or x86 (i.e. not mips, are there others?), so it seems we can remove this
ifdef.
> also same for other files

MIPS support has been requested.  But until that is available, I will remove
these ifdefs.

[jam, 2014-03-25 19:23:11]

lgtm

[Raymond Toy, 2014-03-27 20:37:17]

The CQ bit was checked by rtoy@chromium.org

[Raymond Toy, 2014-03-27 20:37:26]

On 2014/03/25 19:23:11, jam wrote:
> lgtm

Thanks!

[commit-bot: I haz the power, 2014-03-27 20:37:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtoy@google.com/208763003/90001

[cmp, 2014-03-30 16:14:00]

The CQ bit was unchecked by cmp@chromium.org

[cmp, 2014-03-30 16:16:15]

The CQ bit was checked by cmp@chromium.org

[commit-bot: I haz the power, 2014-03-30 16:16:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtoy@google.com/208763003/90001

[commit-bot: I haz the power, 2014-03-30 16:50:54]

Change committed as 260429