Cleanup: Only define various NaCl flags when PPAPI is enabled.

Cleanup: Only define various NaCl flags when PPAPI is enabled.

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

[victorhsieh, 2014-07-31 16:54:18]

lgtm

LGTM, though you'd need owners to approve

[Lei Zhang, 2014-07-31 19:03:02]

On 2014/07/31 16:54:18, victorhsieh wrote:
> lgtm
> 
> LGTM, though you'd need owners to approve

I am the owner. :)

[Lei Zhang, 2014-07-31 19:03:03]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2014-07-31 19:03:59]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/thestig@chromium.org/431833002/1

[commit-bot: I haz the power, 2014-07-31 23:13:25]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_clang_dbg on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)
  android_dbg on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg/bui...)
  win_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel/...)
  win_chromium_x64_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Lei Zhang, 2014-07-31 23:15:18]

The CQ bit was unchecked by thestig@chromium.org

[Lei Zhang, 2014-08-01 01:16:46]

Bleh, why are we building ppapi_tests.cc on Android? I'll first have to fix
that: https://codereview.chromium.org/435603006/

[Lei Zhang, 2014-08-04 18:57:49]

On 2014/08/01 01:16:46, Lei Zhang wrote:
> Bleh, why are we building ppapi_tests.cc on Android? I'll first have to fix
> that: https://codereview.chromium.org/435603006/

The other CL as landed, so this one should be able to pass on Android.

[Lei Zhang, 2014-08-04 18:58:01]

On 2014/08/04 18:57:49, Lei Zhang wrote:
> The other CL as landed, so this one should be able to pass on Android.

*has* landed

[Lei Zhang, 2014-08-04 18:58:05]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2014-08-04 18:58:34]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/thestig@chromium.org/431833002/20001

[commit-bot: I haz the power, 2014-08-05 00:00:47]

Change committed as 287406