Revert of Remove HostDesktopType from AppLaunchParams

Revert of Remove HostDesktopType from AppLaunchParams (patchset #2 id:20001 of https://codereview.chromium.org/1685883003/ )

Reason for revert:
The two AppLaunchParams ctors args are the same except
for the HostDesktopType (which they don't use), but the existence of that
argument was being used as a bool to choose a constructor which did different
things.

I think this likely caused
https://code.google.com/p/chromium/issues/detail?id=587387 and
https://code.google.com/p/chromium/issues/detail?id=586862 .

Original issue's description:
> Remove HostDesktopType from AppLaunchParams
>
> R=sky@chromium.org, benwells@chromium.org
> BUG=558054
>
> Committed: https://crrev.com/99230da5e39ce56f28f05a515941fae53ba5eded
> Cr-Commit-Position: refs/heads/master@{#374814}

TBR=benwells@chromium.org,sky@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=558054, 587387, 586862
Committed: https://crrev.com/99dd35690fb24248158a61de0235e292426f31fd
Cr-Commit-Position: refs/heads/master@{#375945}

[scottmg, 2016-02-17 17:34:36]

Created Revert of Remove HostDesktopType from AppLaunchParams

[commit-bot: I haz the power, 2016-02-17 17:35:03]

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

[scottmg, 2016-02-17 17:35:36]

Description was changed from

==========
Revert of Remove HostDesktopType from AppLaunchParams (patchset #2 id:20001 of
https://codereview.chromium.org/1685883003/ )

Reason for revert:
The two AppLaunchParams ctors args are the same except
for the HostDesktopType (which they don't use), but the existence of that
argument was being used as a bool to choose a constructor which did different
things.

I think this likely caused
https://code.google.com/p/chromium/issues/detail?id=587387 and
https://code.google.com/p/chromium/issues/detail?id=586862 .

Original issue's description:
> Remove HostDesktopType from AppLaunchParams
> 
> R=sky@chromium.org, benwells@chromium.org
> BUG=558054
> 
> Committed: https://crrev.com/99230da5e39ce56f28f05a515941fae53ba5eded
> Cr-Commit-Position: refs/heads/master@{#374814}

TBR=benwells@chromium.org,sky@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=558054
==========

to

==========
Revert of Remove HostDesktopType from AppLaunchParams (patchset #2 id:20001 of
https://codereview.chromium.org/1685883003/ )

Reason for revert:
The two AppLaunchParams ctors args are the same except
for the HostDesktopType (which they don't use), but the existence of that
argument was being used as a bool to choose a constructor which did different
things.

I think this likely caused
https://code.google.com/p/chromium/issues/detail?id=587387 and
https://code.google.com/p/chromium/issues/detail?id=586862 .

Original issue's description:
> Remove HostDesktopType from AppLaunchParams
> 
> R=sky@chromium.org, benwells@chromium.org
> BUG=558054
> 
> Committed: https://crrev.com/99230da5e39ce56f28f05a515941fae53ba5eded
> Cr-Commit-Position: refs/heads/master@{#374814}

TBR=benwells@chromium.org,sky@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=558054,587387,586862
==========

[commit-bot: I haz the power, 2016-02-17 17:37:42]

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

[commit-bot: I haz the power, 2016-02-17 17:37:43]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[scottmg, 2016-02-17 17:55:18]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2016-02-17 17:57:12]

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

[commit-bot: I haz the power, 2016-02-17 18:09:06]

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

[commit-bot: I haz the power, 2016-02-17 18:09:06]

Try jobs failed on following builders:
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[scottmg, 2016-02-17 18:26:50]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2016-02-17 18:27:57]

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

[commit-bot: I haz the power, 2016-02-17 19:14:46]

Description was changed from

==========
Revert of Remove HostDesktopType from AppLaunchParams (patchset #2 id:20001 of
https://codereview.chromium.org/1685883003/ )

Reason for revert:
The two AppLaunchParams ctors args are the same except
for the HostDesktopType (which they don't use), but the existence of that
argument was being used as a bool to choose a constructor which did different
things.

I think this likely caused
https://code.google.com/p/chromium/issues/detail?id=587387 and
https://code.google.com/p/chromium/issues/detail?id=586862 .

Original issue's description:
> Remove HostDesktopType from AppLaunchParams
> 
> R=sky@chromium.org, benwells@chromium.org
> BUG=558054
> 
> Committed: https://crrev.com/99230da5e39ce56f28f05a515941fae53ba5eded
> Cr-Commit-Position: refs/heads/master@{#374814}

TBR=benwells@chromium.org,sky@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=558054,587387,586862
==========

to

==========
Revert of Remove HostDesktopType from AppLaunchParams (patchset #2 id:20001 of
https://codereview.chromium.org/1685883003/ )

Reason for revert:
The two AppLaunchParams ctors args are the same except
for the HostDesktopType (which they don't use), but the existence of that
argument was being used as a bool to choose a constructor which did different
things.

I think this likely caused
https://code.google.com/p/chromium/issues/detail?id=587387 and
https://code.google.com/p/chromium/issues/detail?id=586862 .

Original issue's description:
> Remove HostDesktopType from AppLaunchParams
> 
> R=sky@chromium.org, benwells@chromium.org
> BUG=558054
> 
> Committed: https://crrev.com/99230da5e39ce56f28f05a515941fae53ba5eded
> Cr-Commit-Position: refs/heads/master@{#374814}

TBR=benwells@chromium.org,sky@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=558054,587387,586862
==========

[commit-bot: I haz the power, 2016-02-17 19:14:47]

Committed patchset #4 (id:310001)

[commit-bot: I haz the power, 2016-02-17 19:16:24]

Description was changed from

==========
Revert of Remove HostDesktopType from AppLaunchParams (patchset #2 id:20001 of
https://codereview.chromium.org/1685883003/ )

Reason for revert:
The two AppLaunchParams ctors args are the same except
for the HostDesktopType (which they don't use), but the existence of that
argument was being used as a bool to choose a constructor which did different
things.

I think this likely caused
https://code.google.com/p/chromium/issues/detail?id=587387 and
https://code.google.com/p/chromium/issues/detail?id=586862 .

Original issue's description:
> Remove HostDesktopType from AppLaunchParams
> 
> R=sky@chromium.org, benwells@chromium.org
> BUG=558054
> 
> Committed: https://crrev.com/99230da5e39ce56f28f05a515941fae53ba5eded
> Cr-Commit-Position: refs/heads/master@{#374814}

TBR=benwells@chromium.org,sky@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=558054,587387,586862
==========

to

==========
Revert of Remove HostDesktopType from AppLaunchParams (patchset #2 id:20001 of
https://codereview.chromium.org/1685883003/ )

Reason for revert:
The two AppLaunchParams ctors args are the same except
for the HostDesktopType (which they don't use), but the existence of that
argument was being used as a bool to choose a constructor which did different
things.

I think this likely caused
https://code.google.com/p/chromium/issues/detail?id=587387 and
https://code.google.com/p/chromium/issues/detail?id=586862 .

Original issue's description:
> Remove HostDesktopType from AppLaunchParams
>
> R=sky@chromium.org, benwells@chromium.org
> BUG=558054
>
> Committed: https://crrev.com/99230da5e39ce56f28f05a515941fae53ba5eded
> Cr-Commit-Position: refs/heads/master@{#374814}

TBR=benwells@chromium.org,sky@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=558054,587387,586862

Committed: https://crrev.com/99dd35690fb24248158a61de0235e292426f31fd
Cr-Commit-Position: refs/heads/master@{#375945}
==========

[commit-bot: I haz the power, 2016-02-17 19:16:25]

Patchset 4 (id:??) landed as
https://crrev.com/99dd35690fb24248158a61de0235e292426f31fd
Cr-Commit-Position: refs/heads/master@{#375945}