Increase default maximum desktop size and make it configurable.

remoting/tools/me2me_virtual_host.py

Index: remoting/tools/me2me_virtual_host.py
diff --git a/remoting/tools/me2me_virtual_host.py b/remoting/tools/me2me_virtual_host.py
index 57688f2404d6e8e3a5342f1a025bb8480188dc7b..b862b38a62b5f184fe27a3738c8ffe4dd366a6bf 100755
--- a/remoting/tools/me2me_virtual_host.py
+++ b/remoting/tools/me2me_virtual_host.py
@@ -34,6 +34,7 @@ import uuid
 XSESSION_COMMAND = None
 
 LOG_FILE_ENV_VAR = "CHROME_REMOTE_DESKTOP_LOG_FILE"
+MAXIMUM_SIZE_ENV_VAR = "CHROME_REMOTE_DESKTOP_MAXIMUM_DESTOP_SIZE"

[Wez, 2012/12/14 19:07:00]

typo: DESTOP

[Jamie, 2012/12/14 19:29:16]

Done.

 
 SCRIPT_PATH = sys.path[0]
 
@@ -748,7 +749,9 @@ def waitpid_handle_exceptions(pid, deadline):
 
 
 def main():
-  DEFAULT_SIZE = "2560x1600"
+  DEFAULT_SIZE = "3840x1600"

[Wez, 2012/12/14 19:07:00]

This is an insane default size, e.g. if the user connects with resize-to-client
disabled; why not set MAXIMUM_SIZE to this, or to the size the user specifies,
and have a separate default size?

[Jamie, 2012/12/14 19:29:16]

We don't have a MAXIMUM_SIZE variable--it's computed from the -s options
specified on the command-line. This is just the value that's used if no -s
options are specified on the command-line. How about having an environment
variable that can specify multiple default sizes, and having that default to,
say, 1600x1200,3840x1600? I think that should work in both the resize and
non-resize cases.

[Wez, 2012/12/14 19:30:13]

SGTM :)

[Jamie, 2012/12/14 19:42:58]

Done.

+  if os.environ.has_key(MAXIMUM_SIZE_ENV_VAR):
+    DEFAULT_SIZE = os.environ[MAXIMUM_SIZE_ENV_VAR]
   EPILOG = """This script is not intended for use by end-users.  To configure
 Chrome Remote Desktop, please install the app from the Chrome
 Web Store: https://chrome.google.com/remotedesktop"""