Fix ninja build for Android

Fix ninja build for Android

android_ninja build script can't recognize "BUILDTYPE=Release". This
CL parses and sets the BUILDTYPE parameter in android_setup.sh.

BUG=skia:2383
Committed: http://code.google.com/p/skia/source/detail?r=14043

[qiankun, 2014-04-03 05:31:41]

PTAL.

[epoger, 2014-04-03 13:48:41]

Leon- can you please review this small fix?  It seems harmless enough to me,
although maybe we should just tell developers to set environment variables the
normal way instead:

BUILDTYPE=Release android_ninja -blah -blah
or
export BUILDTYPE=Release
android_ninja -blah -blah

[scroggo, 2014-04-03 13:52:19]

Adding Derek and Mike, who know more about this script.

[mtklein, 2014-04-03 14:26:37]

LGTM.  Seems fine to me.  I don't mind having it work this way, and I wouldn't
mind writing it as something shorter like "-t Debug" "-t Release" either.

For what it's worth, I've always set BUILDTYPE through the environment or called
platform_tools/android/bin/android_ninja -C out/config/*/Release.  The later -C
overrides the default in the script.

https://codereview.chromium.org/221283007/diff/1/platform_tools/android/bin/a...
File platform_tools/android/bin/android_setup.sh (right):

https://codereview.chromium.org/221283007/diff/1/platform_tools/android/bin/a...
platform_tools/android/bin/android_setup.sh:22: elif [[ $(echo "$1" | grep
"^BUILDTYPE=") != "" ]];
I'd prefer just
    elif echo $1 | grep -q ^BUILDTYPE=; then
but I guess the rest of the file sets the odd precedent.

[epoger, 2014-04-03 14:30:07]

Since mtklein has LGed it, I'll check the box...

[epoger, 2014-04-03 14:30:11]

The CQ bit was checked by epoger@google.com

[commit-bot: I haz the power, 2014-04-03 14:30:14]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/qiankun.miao@intel.com/221283007/1

[commit-bot: I haz the power, 2014-04-03 14:48:19]

Change committed as 14043