Ensure that WebKit can use envsetup.sh

Ensure that WebKit can use envsetup.sh

Right now Android's envsetup.sh requires the chdir to be in Chromium's
src directory, whereas this will be different for WebKit. Furthermore,
disabling build_ffmpegsumo will resolve a gyp error in ffmpeg.gyp.

BUG=
TEST=

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

[Peter Beverloo, 2011-11-16 12:13:21]

Hi guys, mind having a quick look before I send this to a build/ reviewer? For
context in which this is being used, see this:
http://trac.webkit.org/browser/trunk/Source/WebKit/chromium/gyp_webkit#L81

[michaelbai, 2011-11-16 17:20:54]

lgtm

[John Grabowski, 2011-11-16 19:40:21]

http://codereview.chromium.org/8523029/diff/2001/build/android/envsetup.sh
File build/android/envsetup.sh (right):

http://codereview.chromium.org/8523029/diff/2001/build/android/envsetup.sh#ne...
build/android/envsetup.sh:79: [ -n "${WEBKIT_ANDROID_BUILD}" ]; then
This needs doc or an explanation of some kind.

[Peter Beverloo, 2011-11-17 17:10:12]

On 2011/11/16 19:40:21, John Grabowski wrote:
> http://codereview.chromium.org/8523029/diff/2001/build/android/envsetup.sh
> File build/android/envsetup.sh (right):
> 
>
http://codereview.chromium.org/8523029/diff/2001/build/android/envsetup.sh#ne...
> build/android/envsetup.sh:79: [ -n "${WEBKIT_ANDROID_BUILD}" ]; then
> This needs doc or an explanation of some kind.

Added, thanks!

Mark: could you have a look, since you reviewed the previous envsetup.sh
changes? It's a trivial change.

[Mark Mentovai, 2011-11-17 18:04:51]

LGTM, although I find comments are clearer when you don’t write “we” or “our” in
them.

[John Grabowski, 2011-11-17 18:18:40]

On 2011/11/17 18:04:51, Mark Mentovai wrote:
> LGTM, although I find comments are clearer when you don’t write “we” or “our”
in
> them.

He's Dutch; that's how verbs translate.

LGTM

[Mark Mentovai, 2011-11-17 18:21:53]

jrg@chromium.org wrote:
> He's Dutch; that's how verbs translate.

That made my day!

[Peter Beverloo, 2011-11-17 18:37:11]

Patch for landing.

Thanks Mark, I've updated the comment. I'll keep it in mind.
Yeah. Thank you too, John :-).