Add support for GYP_GENERATOR_FLAGS out dir setting for landmines

Add support for GYP_GENERATOR_FLAGS out dir setting for landmines

BUG=421894
Committed: https://crrev.com/6bad0d4fad0ee39e993d90eb2ab8050d650efa2b
Cr-Commit-Position: refs/heads/master@{#302419}

[oetuaho-nv, 2014-10-31 10:30:05]

oetuaho@nvidia.com changed reviewers:
+ scottmg@chromium.org, skyostil@google.com

[oetuaho-nv, 2014-10-31 10:30:06]

Please review, I received no response to the bug report about this so I decided
to take a stab at it myself.

[Sami, 2014-10-31 14:30:22]

skyostil@chromium.org changed reviewers:
+ skyostil@chromium.org

[Sami, 2014-10-31 14:30:22]

https://codereview.chromium.org/695753002/diff/1/build/landmines.py
File build/landmines.py (right):

https://codereview.chromium.org/695753002/diff/1/build/landmines.py#newcode49
build/landmines.py:49: output_dir =
landmine_utils.gyp_generator_flags()['output_dir']
nit: rewriting this so that both branches of the 'if' just set output_dir seems
like it would be slightly cleaner.

[oetuaho-nv, 2014-10-31 15:06:50]

Addressed feedback in patch set 2.

[Sami, 2014-10-31 15:10:01]

Thank you, lgtm.

[scottmg, 2014-10-31 16:40:14]

lgtm

https://codereview.chromium.org/695753002/diff/20001/build/landmines.py
File build/landmines.py (right):

https://codereview.chromium.org/695753002/diff/20001/build/landmines.py#newco...
build/landmines.py:47: output_dir = None
delete this line

[oetuaho-nv, 2014-11-03 08:28:55]

The CQ bit was checked by oetuaho@nvidia.com

[commit-bot: I haz the power, 2014-11-03 08:29:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/695753002/40001

[commit-bot: I haz the power, 2014-11-03 09:10:04]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2014-11-03 09:10:44]

Patchset 3 (id:??) landed as
https://crrev.com/6bad0d4fad0ee39e993d90eb2ab8050d650efa2b
Cr-Commit-Position: refs/heads/master@{#302419}

[Geoff Lang, 2014-11-03 14:50:11]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/697123002/ by geofflang@chromium.org.

The reason for reverting is: Win8 Release (NVIDIA) and Win8 Debug (NVIDIA) bots
are failing to delete temporary files in the hardware_accelerated_feature_tests.

Example failed build:
http://build.chromium.org/p/chromium.gpu/builders/Win8%20Debug%20%28NVIDIA%29...

BUG=429663.