[MIPS] Add build support in Skia for MIPS.

[MIPS] Add build support in Skia for MIPS.

Small change in skia.gyp to support building for MIPS arch.

BUG= https://code.google.com/p/chromium/issues/detail?id=130022
TEST=make chrome


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

[robertphillips, 2012-10-26 12:01:06]

Added Eric & Elliot

[epoger, 2012-10-29 16:57:28]

Seems fine to me (as long as you commit it after the M24 branch is cut)

I'm not chromium-y enough for my LG to mean anything.

[petarj, 2012-11-01 00:56:47]

On 2012/10/29 16:57:28, epoger wrote:
> Seems fine to me (as long as you commit it after the M24 branch is cut)
> 
> I'm not chromium-y enough for my LG to mean anything.

Any update here?
Thanks.

[reed1, 2012-11-01 13:19:33]

We have been trying hard to move as much of the gyp/gypi logic as makes sense
into skia itself, so we can update and test changes locally, before rolling into
chrome.

Is this such a change? Can we build/test skia against MIPS in our environment?
If so, I'd vote to see how much of this change belongs in skia. If not, then I
have no issue landing this CL.

[epoger, 2012-11-01 14:12:05]

Note that this change will conflict with
https://chromiumcodereview.appspot.com/11342016/ ('Fix skia build when not on
armv7.').

I fear the incentives associated with such a warning, but: whoever commits last
will have to manually merge the changes.

[epoger, 2012-11-01 14:14:47]

LGTM

On 2012/11/01 13:19:33, reed1 wrote:
> We have been trying hard to move as much of the gyp/gypi logic as makes sense
> into skia itself, so we can update and test changes locally, before rolling
into
> chrome.
> 
> Is this such a change? Can we build/test skia against MIPS in our environment?
> If so, I'd vote to see how much of this change belongs in skia. If not, then I
> have no issue landing this CL.

For now, the vast majority of the gyp configuration for building Skia within
Chrome lives within Chrome (in the file modified here).

If members of the Skia team can work with the Chrome folks to change that, I
think it would be great... but for now, I don't think that is realistic to ask
of this particular CL.

[epoger, 2012-11-01 14:15:59]

...and I believe the M24 branch was cut on Oct 30, so it should be safe now.

[commit-bot: I haz the power, 2012-11-01 15:43:33]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[reed1, 2012-11-01 15:44:03]

lgtm

[commit-bot: I haz the power, 2012-11-01 15:44:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/petarj@mips.com/11292003/1

[commit-bot: I haz the power, 2012-11-01 17:46:28]

Change committed as 165421