Pass DesktopEnvironmentOptions form network to desktop process.

Pass DesktopEnvironmentOptions form network to desktop process.

Now DesktopEnvironmentOptions are passed to the desktop process in
StartSessionAgent IPC message. This will allow the network process
controll DesktopEnvironment configuration and particularly
DesktopCaptureOptions.

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

[Sergey Ulanov, 2016-11-23 23:57:58]

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

[Sergey Ulanov, 2016-11-23 23:58:06]

sergeyu@chromium.org changed reviewers:
+ zijiehe@chromium.org

[commit-bot: I haz the power, 2016-11-23 23:58:41]

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, 2016-11-24 02:03:55]

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

[commit-bot: I haz the power, 2016-11-24 02:03:55]

Dry run: Try jobs failed on following builders:
  blimp_linux_dbg on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT, no
build URL)

[Hzj_jie, 2016-11-27 20:42:23]

lgtm

[Hzj_jie, 2016-11-28 00:10:45]

https://codereview.chromium.org/2529533003/diff/1/remoting/host/chromoting_pa...
File remoting/host/chromoting_param_traits.h (right):

https://codereview.chromium.org/2529533003/diff/1/remoting/host/chromoting_pa...
remoting/host/chromoting_param_traits.h:91: static void Log(const param_type& p,
std::string* l);
Share we consider to add a ParamTraits<DesktopCaptureOptions>?

[Sergey Ulanov, 2016-11-28 19:54:41]

https://codereview.chromium.org/2529533003/diff/1/remoting/host/chromoting_pa...
File remoting/host/chromoting_param_traits.h (right):

https://codereview.chromium.org/2529533003/diff/1/remoting/host/chromoting_pa...
remoting/host/chromoting_param_traits.h:91: static void Log(const param_type& p,
std::string* l);
On 2016/11/28 00:10:45, Hzj_jie wrote:
> Share we consider to add a ParamTraits<DesktopCaptureOptions>?

I don't think we really need it right now.

[Sergey Ulanov, 2016-11-28 19:55:16]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2016-11-28 19:56:32]

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

[Sergey Ulanov, 2016-11-28 22:29:11]

sergeyu@chromium.org changed reviewers:
+ joedow@chromium.org

[Sergey Ulanov, 2016-11-28 22:29:11]

+joedow for OWNERS approval

[Sergey Ulanov, 2016-11-28 22:34:11]

sergeyu@chromium.org changed reviewers:
+ rsesek@chromium.org
- joedow@chromium.org

[Sergey Ulanov, 2016-11-28 22:34:11]

+rsesek for IPC changes approval

-joedow (sorry, I misread the presubmit failure)

[Robert Sesek, 2016-11-28 22:36:55]

lgtm

[commit-bot: I haz the power, 2016-11-28 22:51:05]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480362916360690, "parent_rev":
"4fd3ac90a3f640c171dbfbb00988d0adf94ab3bb", "commit_rev":
"4598a3eedcc8ce60daeaf782849d41bb8f728a85"}

[commit-bot: I haz the power, 2016-11-28 22:51:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-28 22:55:21]

Description was changed from

==========
Pass DesktopEnvironmentOptions form network to desktop process.

Now DesktopEnvironmentOptions are passed to the desktop process in
StartSessionAgent IPC message. This will allow the network process
controll DesktopEnvironment configuration and particularly
DesktopCaptureOptions.
==========

to

==========
Pass DesktopEnvironmentOptions form network to desktop process.

Now DesktopEnvironmentOptions are passed to the desktop process in
StartSessionAgent IPC message. This will allow the network process
controll DesktopEnvironment configuration and particularly
DesktopCaptureOptions.

Committed: https://crrev.com/c986f523c34dbaf0d943abae125094a72cc8c2f8
Cr-Commit-Position: refs/heads/master@{#434754}
==========

[commit-bot: I haz the power, 2016-11-28 22:55:23]

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