Fix sbox_validation_tests & Make isolate_driver.py include the executable itself

Fix sbox_validation_tests & Make isolate_driver.py include the executable itself

Further reduce the boilerplate in .isolate files and is necessary to add debug
symbols, to be done in a follow up.

Slightly accelerate ninja file parsing by blacklisting more extensions.

sbox_validation_tests wasn't run on Windows since
https://codereview.chromium.org/1212003003/ (~2 months ago). Thankfully this CL
uncovered this copy-paste error.

TBR=jam@chromium.org
BUG=98637
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/c6649f2e014a24a9093140a06fcadcaf2a06513c
Cr-Commit-Position: refs/heads/master@{#347027}

[M-A Ruel, 2015-09-02 14:08:43]

Patchset #1 (id:1) has been deleted

[M-A Ruel, 2015-09-02 14:08:52]

Patchset #1 (id:20001) has been deleted

[M-A Ruel, 2015-09-02 15:15:31]

Patchset #1 (id:40001) has been deleted

[M-A Ruel, 2015-09-02 15:15:55]

And finally...
https://codereview.chromium.org/1245873002/diff/60001/tools/isolate_driver.py is
the important diff, the rest is just deleting executables to confirm it works.

[M-A Ruel, 2015-09-02 15:28:20]

On 2015/09/02 15:15:55, M-A Ruel wrote:
> And finally...
> https://codereview.chromium.org/1245873002/diff/60001/tools/isolate_driver.py
is
> the important diff, the rest is just deleting executables to confirm it works.

Blah, it fails on win x64, looking.

[jam, 2015-09-02 17:14:45]

lgtm

[M-A Ruel, 2015-09-02 17:23:43]

On 2015/09/02 17:14:45, jam wrote:
> lgtm

The bug was trivial, CQ'ing (with TBR due to number of files involved)

[M-A Ruel, 2015-09-02 17:23:56]

The CQ bit was checked by maruel@chromium.org

[M-A Ruel, 2015-09-02 17:23:56]

The patchset sent to the CQ was uploaded after l-g-t-m from jam@chromium.org
Link to the patchset: https://codereview.chromium.org/1245873002/#ps70069
(title: "Silly me, had forgot .exe")

[commit-bot: I haz the power, 2015-09-02 17:24:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1245873002/70069
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1245873002/70069

[commit-bot: I haz the power, 2015-09-02 18:20:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-02 18:20:36]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[M-A Ruel, 2015-09-02 19:21:44]

The CQ bit was checked by maruel@chromium.org

[M-A Ruel, 2015-09-02 19:21:44]

The patchset sent to the CQ was uploaded after l-g-t-m from jam@chromium.org
Link to the patchset: https://codereview.chromium.org/1245873002/#ps90001
(title: "sbox_validation_tests")

[commit-bot: I haz the power, 2015-09-02 19:31:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1245873002/90001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1245873002/90001

[commit-bot: I haz the power, 2015-09-02 19:59:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-02 19:59:36]

Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromiu...)

[M-A Ruel, 2015-09-02 20:01:40]

The CQ bit was checked by maruel@chromium.org

[commit-bot: I haz the power, 2015-09-02 20:02:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1245873002/90001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1245873002/90001

[commit-bot: I haz the power, 2015-09-02 21:20:47]

Committed patchset #3 (id:90001)

[commit-bot: I haz the power, 2015-09-02 21:21:43]

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

[stgao, 2015-09-02 23:50:41]

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

[stgao, 2015-09-02 23:50:41]

Hi Marc,

This CL seems causing the compile failure in Linux official build
https://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20Linux...

It seems this CL is fixing another issue.
But Should I revert this CL first?


Among the huge log, below is the failure:
FAILED: cd ../../chrome; python ../tools/isolate_driver.py check --isolated
"../out/Release/chrome.isolated" --isolate "chrome.isolate" --path-variable
DEPTH .. --path-variable PRODUCT_DIR "../out/Release " --extra-variable
"version_full=47.0.2500.0" --config-variable "CONFIGURATION_NAME=Release"
--config-variable "OS=linux" --config-variable "asan=0" --config-variable
"branding=Chrome" --config-variable "chromeos=0" --config-variable
"component=static_library" --config-variable "disable_nacl=0" --config-variable
"enable_pepper_cdms=1" --config-variable "enable_plugins=1" --config-variable
"fastbuild=0" --config-variable "icu_use_data_file_flag=1" --config-variable
"internal_gles2_conform_tests=0" --config-variable "kasko=0" --config-variable
"lsan=0" --config-variable "msan=0" --config-variable "target_arch=x64"
--config-variable "tsan=0" --config-variable "use_custom_libcxx=0"
--config-variable "use_instrumented_libraries=0" --config-variable
"use_prebuilt_instrumented_libraries=0" --config-variable "use_openssl=1"
--config-variable "use_ozone=0" --config-variable "use_x11=1" --config-variable
"v8_use_external_startup_data=1" --config-variable "msvs_version=0"
Failed to find an input file: Input file
/b/build/slave/google-chrome-rel-linux_64/build/src/out/Release/usr/bin/eu-strip
doesn't exist

[stgao, 2015-09-03 00:11:44]

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

The reason for reverting is: This CL seems to break compile
http://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20Linux%....

If the revert doesn't fix it, I will re-land this CL..