Revert of Add --headless flag to Windows

Revert of Add --headless flag to Windows (patchset #29 id:560001 of https://codereview.chromium.org/2762593002/ )

Reason for revert:
Testing revert on --headless flag, due to increase in child_dll size: https://bugs.chromium.org/p/chromium/issues/detail?id=714841

Original issue's description:
> Launch bug: https://bugs.chromium.org/p/chromium/issues/detail?id=705916
> 
> Add --headless flag to Windows: 
> 
> Fix browser_tests not compiling properly on windows
> Fix headless/BUILD.gn by: 
>    - Changing headless_lib to a headless component, following steps in https://chromium.googlesource.com/chromium/src/+/master/docs/component_build.md and fixing the build for linux, since it now compiles as a shared_library when component build is true.
>    - Adding neccesary HEADLESS_EXPORT (submitted in https://codereview.chromium.org/2775693003/)
>    - Disabling breakpad in windows (will fix in a separate CL)
> 
> 
> BUG=686608
> 
> Review-Url: https://codereview.chromium.org/2762593002
> Cr-Original-Commit-Position: refs/heads/master@{#466176}
> Committed: https://chromium.googlesource.com/chromium/src/+/e3c745d4b37a659afaa0358c7de7b9bc881decb5
> Review-Url: https://codereview.chromium.org/2762593002
> Cr-Commit-Position: refs/heads/master@{#466259}
> Committed: https://chromium.googlesource.com/chromium/src/+/d238dae172d63175b562f99fc988e47e5213a244

TBR=alexclarke@chromium.org,sky@chromium.org,skyostil@chromium.org,wfh@chromium.org,jzfeng@chromium.org,nick@chromium.org,hans@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=686608
Review-Url: https://codereview.chromium.org/2837093003
Cr-Commit-Position: refs/heads/master@{#466871}
Committed: https://chromium.googlesource.com/chromium/src/+/4c5022c615824a6e2ad67db5eec5a093e3111e2d

[dvallet, 2017-04-24 23:37:07]

The CQ bit was checked by dvallet@chromium.org

[dvallet, 2017-04-24 23:37:07]

Created Revert of Add --headless flag to Windows

[commit-bot: I haz the power, 2017-04-24 23:37:32]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-24 23:37:33]

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

[commit-bot: I haz the power, 2017-04-24 23:37:34]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[dvallet, 2017-04-24 23:37:54]

The CQ bit was checked by dvallet@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-24 23:38:28]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-24 23:42:11]

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

[commit-bot: I haz the power, 2017-04-24 23:42:12]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[dvallet, 2017-04-25 00:15:44]

The CQ bit was checked by dvallet@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-25 00:16:09]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-25 00:28:45]

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

[commit-bot: I haz the power, 2017-04-25 00:28:46]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[dvallet, 2017-04-25 00:53:31]

The CQ bit was checked by dvallet@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-25 00:53:56]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hans, 2017-04-25 02:03:36]

lgtm

[hans, 2017-04-25 02:04:45]

The CQ bit was unchecked by hans@chromium.org

[hans, 2017-04-25 02:04:49]

The CQ bit was checked by hans@chromium.org

[commit-bot: I haz the power, 2017-04-25 02:05:22]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-25 02:19:50]

CQ is committing da patch.

Bot data: {"patchset_id": 300001, "attempt_start_ts": 1493085889456460,
"parent_rev": "1a9ce8910080635cbaf779c1bf42efd927108371", "commit_rev":
"4c5022c615824a6e2ad67db5eec5a093e3111e2d"}

[commit-bot: I haz the power, 2017-04-25 02:20:04]

Description was changed from

==========
Revert of Add --headless flag to Windows (patchset #29 id:560001 of
https://codereview.chromium.org/2762593002/ )

Reason for revert:
Testing revert on --headless flag, due to increase in child_dll size:
https://bugs.chromium.org/p/chromium/issues/detail?id=714841

Original issue's description:
> Launch bug: https://bugs.chromium.org/p/chromium/issues/detail?id=705916
> 
> Add --headless flag to Windows: 
> 
> Fix browser_tests not compiling properly on windows
> Fix headless/BUILD.gn by: 
>    - Changing headless_lib to a headless component, following steps in
https://chromium.googlesource.com/chromium/src/+/master/docs/component_build.md
and fixing the build for linux, since it now compiles as a shared_library when
component build is true.
>    - Adding neccesary HEADLESS_EXPORT (submitted in
https://codereview.chromium.org/2775693003/)
>    - Disabling breakpad in windows (will fix in a separate CL)
> 
> 
> BUG=686608
> 
> Review-Url: https://codereview.chromium.org/2762593002
> Cr-Original-Commit-Position: refs/heads/master@{#466176}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e3c745d4b37a659afaa0358c7de7...
> Review-Url: https://codereview.chromium.org/2762593002
> Cr-Commit-Position: refs/heads/master@{#466259}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d238dae172d63175b562f99fc988...

TBR=alexclarke@chromium.org,sky@chromium.org,skyostil@chromium.org,wfh@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=686608
==========

to

==========
Revert of Add --headless flag to Windows (patchset #29 id:560001 of
https://codereview.chromium.org/2762593002/ )

Reason for revert:
Testing revert on --headless flag, due to increase in child_dll size:
https://bugs.chromium.org/p/chromium/issues/detail?id=714841

Original issue's description:
> Launch bug: https://bugs.chromium.org/p/chromium/issues/detail?id=705916
> 
> Add --headless flag to Windows: 
> 
> Fix browser_tests not compiling properly on windows
> Fix headless/BUILD.gn by: 
>    - Changing headless_lib to a headless component, following steps in
https://chromium.googlesource.com/chromium/src/+/master/docs/component_build.md
and fixing the build for linux, since it now compiles as a shared_library when
component build is true.
>    - Adding neccesary HEADLESS_EXPORT (submitted in
https://codereview.chromium.org/2775693003/)
>    - Disabling breakpad in windows (will fix in a separate CL)
> 
> 
> BUG=686608
> 
> Review-Url: https://codereview.chromium.org/2762593002
> Cr-Original-Commit-Position: refs/heads/master@{#466176}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e3c745d4b37a659afaa0358c7de7...
> Review-Url: https://codereview.chromium.org/2762593002
> Cr-Commit-Position: refs/heads/master@{#466259}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d238dae172d63175b562f99fc988...

TBR=alexclarke@chromium.org,sky@chromium.org,skyostil@chromium.org,wfh@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=686608

Review-Url: https://codereview.chromium.org/2837093003
Cr-Commit-Position: refs/heads/master@{#466871}
Committed:
https://chromium.googlesource.com/chromium/src/+/4c5022c615824a6e2ad67db5eec5...
==========

[commit-bot: I haz the power, 2017-04-25 02:20:05]

Committed patchset #3 (id:300001) as
https://chromium.googlesource.com/chromium/src/+/4c5022c615824a6e2ad67db5eec5...

[findit-for-me, 2017-04-25 03:14:11]

Findit(https://goo.gl/kROfz5) identified this CL at revision 466871 as the
culprit for
failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3Itb...

[dvallet, 2017-04-25 04:02:08]

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

The reason for reverting is: Revert this to fix Mac Build issue.

[dvallet, 2017-04-25 04:27:20]

Description was changed from

==========
Revert of Add --headless flag to Windows (patchset #29 id:560001 of
https://codereview.chromium.org/2762593002/ )

Reason for revert:
Testing revert on --headless flag, due to increase in child_dll size:
https://bugs.chromium.org/p/chromium/issues/detail?id=714841

Original issue's description:
> Launch bug: https://bugs.chromium.org/p/chromium/issues/detail?id=705916
> 
> Add --headless flag to Windows: 
> 
> Fix browser_tests not compiling properly on windows
> Fix headless/BUILD.gn by: 
>    - Changing headless_lib to a headless component, following steps in
https://chromium.googlesource.com/chromium/src/+/master/docs/component_build.md
and fixing the build for linux, since it now compiles as a shared_library when
component build is true.
>    - Adding neccesary HEADLESS_EXPORT (submitted in
https://codereview.chromium.org/2775693003/)
>    - Disabling breakpad in windows (will fix in a separate CL)
> 
> 
> BUG=686608
> 
> Review-Url: https://codereview.chromium.org/2762593002
> Cr-Original-Commit-Position: refs/heads/master@{#466176}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e3c745d4b37a659afaa0358c7de7...
> Review-Url: https://codereview.chromium.org/2762593002
> Cr-Commit-Position: refs/heads/master@{#466259}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d238dae172d63175b562f99fc988...

TBR=alexclarke@chromium.org,sky@chromium.org,skyostil@chromium.org,wfh@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=686608

Review-Url: https://codereview.chromium.org/2837093003
Cr-Commit-Position: refs/heads/master@{#466871}
Committed:
https://chromium.googlesource.com/chromium/src/+/4c5022c615824a6e2ad67db5eec5...
==========

to

==========
Revert of Add --headless flag to Windows (patchset #29 id:560001 of
https://codereview.chromium.org/2762593002/ )

Reason for revert:
Testing revert on --headless flag, due to increase in child_dll size:
https://bugs.chromium.org/p/chromium/issues/detail?id=714841

Original issue's description:
> Launch bug: https://bugs.chromium.org/p/chromium/issues/detail?id=705916
> 
> Add --headless flag to Windows: 
> 
> Fix browser_tests not compiling properly on windows
> Fix headless/BUILD.gn by: 
>    - Changing headless_lib to a headless component, following steps in
https://chromium.googlesource.com/chromium/src/+/master/docs/component_build.md
and fixing the build for linux, since it now compiles as a shared_library when
component build is true.
>    - Adding neccesary HEADLESS_EXPORT (submitted in
https://codereview.chromium.org/2775693003/)
>    - Disabling breakpad in windows (will fix in a separate CL)
> 
> 
> BUG=686608
> 
> Review-Url: https://codereview.chromium.org/2762593002
> Cr-Original-Commit-Position: refs/heads/master@{#466176}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e3c745d4b37a659afaa0358c7de7...
> Review-Url: https://codereview.chromium.org/2762593002
> Cr-Commit-Position: refs/heads/master@{#466259}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d238dae172d63175b562f99fc988...

TBR=alexclarke@chromium.org,sky@chromium.org,skyostil@chromium.org,wfh@chromi...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=686608

Review-Url: https://codereview.chromium.org/2837093003
Cr-Commit-Position: refs/heads/master@{#466871}
Committed:
https://chromium.googlesource.com/chromium/src/+/4c5022c615824a6e2ad67db5eec5...
==========

[dvallet, 2017-04-25 04:29:19]

The CQ bit was checked by dvallet@chromium.org

[dvallet, 2017-04-25 04:29:20]

The patchset sent to the CQ was uploaded after l-g-t-m from hans@chromium.org
Link to the patchset: https://codereview.chromium.org/2837093003/#ps320001
(title: "Fix mac build")

[commit-bot: I haz the power, 2017-04-25 04:29:45]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-25 04:36:54]

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

[commit-bot: I haz the power, 2017-04-25 04:36:55]

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

[dvallet, 2017-04-25 04:37:14]

On 2017/04/25 04:29:45, commit-bot: I haz the power wrote:
> CQ is trying da patch.
> 
> Follow status at:
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

I'm submitting the fix within this patch. It was a rebase issue.

[dvallet, 2017-04-25 04:47:42]

Patchset #4 (id:320001) has been deleted

[dvallet, 2017-04-25 04:48:35]

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

The reason for reverting is: Revert is breaking Mac build.

[dvallet, 2017-04-25 06:50:58]

On 2017/04/25 04:48:35, dvallet wrote:
> A revert of this CL (patchset #3 id:300001) has been created in
> https://codereview.chromium.org/2840693003/ by mailto:dvallet@chromium.org.
> 
> The reason for reverting is: Revert is breaking Mac build.

This CL should fix the issue with no need to revert:
https://codereview.chromium.org/2840713003/

[dvallet, 2017-04-25 06:51:03]

On 2017/04/25 04:48:35, dvallet wrote:
> A revert of this CL (patchset #3 id:300001) has been created in
> https://codereview.chromium.org/2840693003/ by mailto:dvallet@chromium.org.
> 
> The reason for reverting is: Revert is breaking Mac build.

This CL should fix the issue with no need to revert:
https://codereview.chromium.org/2840713003/

[dvallet, 2017-04-25 06:51:06]

On 2017/04/25 04:48:35, dvallet wrote:
> A revert of this CL (patchset #3 id:300001) has been created in
> https://codereview.chromium.org/2840693003/ by mailto:dvallet@chromium.org.
> 
> The reason for reverting is: Revert is breaking Mac build.

This CL should fix the issue with no need to revert:
https://codereview.chromium.org/2840713003/