Cross-platform plumbing for resize-to-client

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 1438bf472f38bd86cd3853da1e0c906613f8dbd2..ef9d2fbab2069bd81d802e1b2dce17ba2650109e 100755
--- a/remoting/tools/me2me_virtual_host.py
+++ b/remoting/tools/me2me_virtual_host.py
@@ -253,7 +253,12 @@ class Desktop:
     max_height = max([height for width, height in self.sizes])
 
     try:
-      xvfb = locate_executable("Xvfb-randr")
+      # TODO(jamiewalch): This script expects to be installed alongside
+      # Xvf-randr, but that's no longer the case. Fix this once we have

[Lambros, 2012/09/13 22:00:35]

typo: Xvfb-randr

[Jamie, 2012/09/14 21:07:34]

Done.

+      # a Xvfb-randr package that installs somewhere sensible.
+      xvfb = "/usr/bin/Xvfb-randr"

[Sergey Ulanov, 2012/09/13 20:52:43]

Not directly related to this CL: maybe we can just pack Xvfb-randr in our dpk
and put it in /opt/google/chrome-remote-desktop? That way we won't need to worry
about installing this dependency.

[Jamie, 2012/09/14 21:07:34]

Lambros had some reason for preferring to keep the two packages separate--he can
probably elaborate further, but it does seem like a cleaner way of doing it.

+      if not os.path.exists(xvfb):
+        xvfb = locate_executable("Xvfb-randr")
     except Exception:
       xvfb = "Xvfb"