| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
BUG=175802
Review URL: https://chromiumcodereview.appspot.com/12223124
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182390 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This will allow us to propagate it to the directory so that the client-side
web-app can identify out-of-date hosts, as discussed in the referenced bug
(option 3 in Renato's comment).
BUG=173502
Review URL: https://chromiumcodereview.appspot.com/12221123
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182379 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://chromiumcodereview.appspot.com/12251002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182366 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This CL also updates the client and host protocol handlers to the new name.
BUG=172404
Review URL: https://chromiumcodereview.appspot.com/12220092
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182115 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
document.all is not available to Apps v2, so menus were previously not
dismissable. I could use document.body.parentNode, but I can't think of
any good reason to attach the event handler to <html> instead of <body>.
Review URL: https://codereview.chromium.org/12208141
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182046 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Previously, we were asynchronously reading the manifest at start-up and using
chrome.runtime.getDetails when logging the version string. The latter is more
complicated than it needs to be, and the former doesn't work in Apps v2.
Review URL: https://codereview.chromium.org/12250011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182036 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=172404
Review URL: https://chromiumcodereview.appspot.com/12207099
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181987 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=lambroslambrou@google.com
Review URL: https://codereview.chromium.org/12226106
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181942 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Fix bugs in translating capture regions into display coordinates.
- Fix MacDeskopConfiguration to support inverse-Cartesian coordinates.
BUG=175261,174090
Review URL: https://chromiumcodereview.appspot.com/12221103
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181885 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
R=lambroslambrou@google.com
Review URL: https://codereview.chromium.org/12216006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181773 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Two simple compilation fixes and some c4267 build suppressions.
BUG=166496
BUG=167187
R=wez@chromium.org
Review URL: https://codereview.chromium.org/12225111
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181650 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
By default ATL generates locking class factories (by wrapping them in ATL::CComObjectCached) for classes hosted in a DLL. ElevatedController is compiled into a DLL but it is registered as an out-of-process class, so its class factory should not use locking.
BUG=174722
Review URL: https://chromiumcodereview.appspot.com/12209072
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181632 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
JingleMessage currently sets timeout for all messages it sends. That
doesn't make sense for transport-info messages. Removing transport-info
timeouts makes it more likely that connection can be renegotiated
after intermittent network problems. Also removed code that cleans up
list of pending requests as it's not needed anymore.
BUG=168114
Review URL: https://chromiumcodereview.appspot.com/12223059
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181606 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
As of 10.8 if you don't specify a standard output/error destination file
then output from the host service is not logged at all. This change
modifies our host service .plist to specify a specific chromoting
log file, ensures this log file is properly created
when we install the host, adds a configuration for the newsyslog
process (which will turnover/archive the log file once it goes >1MB),
and removes the log file and newsyslog configuration in the uninstaller.
It also adds some timestamps to the host startup script so that we can
better identify log events.
It also alphabetizes the strings in the .plist file.
TEST=Manual (modified files, created install package, ran and uninstalled)
BUG=171026
Review URL: https://chromiumcodereview.appspot.com/12213059
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181605 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. gksudo prompt timeout has been increased to 5 minutes,
2. gksudo is killed when the timeout expires,
3. gksudo is launched with -D instead of -m.
BUG=155920,167791
Review URL: https://chromiumcodereview.appspot.com/12207086
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181599 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=121771
TEST=Manual
Review URL: https://chromiumcodereview.appspot.com/10068020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181592 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
between connection modes and reset web-app state.
BUG=175008
Review URL: https://chromiumcodereview.appspot.com/12220080
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181591 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously SignalingConnector would reset XMPP connection whenever IP address
changes. This would cause the host to be disconnected unexpectedly when a new
network interface is activated. Now the IP change handler will only reset
XMPP connection when it is inactive. If XMPP connection is active then
network change notifications must be ignored. In that case, if the change
does affect the connection then we will get a separate notification when TCP
stack drops it.
BUG=166890
Review URL: https://codereview.chromium.org/12217093
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181586 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
The patch had suffered a bit of code-rot. This CL updates it so that it applies
cleanly again. It does not add any functionality.
NOTRY=true
Review URL: https://chromiumcodereview.appspot.com/12210096
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181585 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This will output a remoting-me2me-host-linux.zip file containing the
Linux chrome-remote-desktop packages (currently just .deb), and will
archive it along with other build artifacts like remoting-webapp.zip.
Review URL: https://codereview.chromium.org/12224068
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181551 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181400 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
By doing this we're able to reduce the size of our webapp
without impacting image quality.
TEST=Manual; verify in-app imagery and icons have no artefacts, and that .webp files are smaller than .png originals.
BUG=169382
Review URL: https://chromiumcodereview.appspot.com/12225002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181241 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
processes.
Related changes:
- The fake screen capturer now uses shared buffers when they available.
- DesktopSessionAgent frees the client end handle of the network-to-desktop pipe once the connection has been established.
BUG=134694
Review URL: https://chromiumcodereview.appspot.com/12096071
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181221 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
client_screen.js is a bit of a mixed bag of functions with only a loose
common theme. This CL pulls out the largest single feature, connection setup,
into a separate class, eliminating lots of global state as a result.
As a side-effect, it speeds up connections by parallelizing the WCS driver load
and fixes the referenced bugs.
BUG=173788,174113
Review URL: https://chromiumcodereview.appspot.com/12177003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181187 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=174722
Review URL: https://codereview.chromium.org/12220047
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181089 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
of in every client
BUG=174433
Review URL: https://codereview.chromium.org/12226028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181085 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Automated clean up of style checker errors that were missed due to the plugin
not being executed on implementation files.
BUG=115047
Review URL: https://chromiumcodereview.appspot.com/12207034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181075 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=170285
Review URL: https://chromiumcodereview.appspot.com/12211003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180918 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously, the Keystone registration took place at the end of the
script, but a trap on ERR meant that any errors within the script
caused premature exit. Since Keystone registration is critical, it
should be done first.
BUG=164647
Review URL: https://chromiumcodereview.appspot.com/12092081
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180906 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is to aid migration to Native Messaging, where the web-app will need to
pass callbacks to receive all information asynchronously from the Native
Messaging host.
BUG=173509
TEST=register, start and connect to a host, change PIN, 0 jscompile errors
Review URL: https://chromiumcodereview.appspot.com/12089099
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180905 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change adds the necessary WebP icon files in order
to support changing our icons to WebP. A separate
CL will be used to actually wire them up.
BUG=169382
TEST=Manual
TBR=wez@chromium.org
NOTRY=True
Review URL: https://codereview.chromium.org/12221030
Patch from Stephen Konig <skonig@chromium.org>.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180860 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The gypi file assumes that v8_shell exists, but doesn't add
a dependency to build it. This works with some build systems,
but not with all.
To get the bots green, just add the dependency where it's needed
for now.
BUG=174433
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=180820
Review URL: https://codereview.chromium.org/12208031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180841 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Fix js_unittest_rules.gypi dependenies
>
> The gypi file assumes that v8_shell exists, but doesn't add
> a dependency to build it. This works with some build systems,
> but not with all.
>
> To get the bots green, just add the dependency where it's needed
> for now.
>
> BUG=174433
>
> Review URL: https://codereview.chromium.org/12208031
TBR=thakis@chromium.org
Review URL: https://codereview.chromium.org/12212024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180825 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The gypi file assumes that v8_shell exists, but doesn't add
a dependency to build it. This works with some build systems,
but not with all.
To get the bots green, just add the dependency where it's needed
for now.
BUG=174433
Review URL: https://codereview.chromium.org/12208031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180820 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In C++, they're all passed as optional command line parameters (like google_apis/gaia already allows us to do with the gaia oauth urls). So one can start the host with a monstrosity such as:
out/Debug/remoting_me2me_host out/Debug/remoting_start_host --gaia-host=accounts.google.com --oauth-user-info-url=https://www.googleapis.com/oauth2/v1/userinfo --chromoting-bot-jid=remoting@bot.talk.google.com --chromoting-base-url=https://www.googleapis.com/chromoting/v1 --talk-server-host=talk.google.com --talk-server-port=5222 -[-disable-talk-server-tls] --oauth2-token-url=https://accounts.google.com/o/oauth2/token
In Javascript and IT2Me, unfortunately we don't have a command line, so instead we put all these settings in plugin_settings.js, and the javascript passes them all into the it2me plugin as appropriate. They can then be changed by manually hacking plugin_settings.js and manifest.json in the build output, changing the remoting.settings object at runtime (although any new hosts you configure that are not in the manifest's CSP won't work), or, preferrably, by passing them via environment variables to the build, and letting build-webapp.py do its magic - for example, like with this:
GOOGLE_CLIENT_ID_REMOTING=<clientid> GOOGLE_CLIENT_SECRET_REMOTING=<secret> OAUTH2_ACCOUNTS_HOST=https://accounts.google.com OAUTH2_API_HOST=https://www.googleapis.com CHROMOTING_BOT_JID=remoting@bot.talk.google.com CHROMOTING_API_HOST=https://www.googleapis.com TALK_SERVER_HOST=talk.google.com TALK_SERVER_PORT=5222 TALK_SERVER_USE_TLS=true TALK_GADGET_HOST_SUFFIX=talkgadget.google.com OAUTH2_REDIRECT_HOST_PREFIX=https://chromoting-oauth. TALK_GADGET_HOST_PREFIX=https://chromoting-client. ninja -C out/Debug remoting_webapp
I've tested with me2me (host and client) and it2me (share and access), and verified both that the defaults work as before (without command line parameters), and that I can substitute test server hostnames in all of those parameters.
BUG=
Review URL: https://chromiumcodereview.appspot.com/11346040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180622 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Previously RectangleUpdateDecoder could be reused for more than one
session which sometimes may lead to crashes.
BUG=173004
Review URL: https://codereview.chromium.org/12154004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180599 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
The code already transfers ownership of these pointers in the callbacks,
so there's no need to do it again.
BUG=162971
Review URL: https://chromiumcodereview.appspot.com/12178005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180581 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://chromiumcodereview.appspot.com/11642060
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180568 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Merged all Chromoting Host code into remoting_core.dll (Windows).
Consolidated all installable Chromoting Host code into remoting_core.dll and converted all executables into thin wrappers around entry points exposed by remoting_core.dll. This reduces size of the installer by approximately 600KB.
This CL also ensures that proper manifests will be embedded into the binaries regardless whether msbuild or ninja is used.
TBR=sergeyu@chromium.org
BUG=170200, 173052
Review URL: https://codereview.chromium.org/12189013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180468 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
caller to control the kind of desktop environment to be created.
BUG=104544
Review URL: https://chromiumcodereview.appspot.com/12087073
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180443 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Merged all Chromoting Host code into remoting_core.dll (Windows).
>
> Consolidated all installable Chromoting Host core into remoting_core.dll and converted all executables into thin wrappers around entry points exposed by remoting_core.dll. This reduces size of the installer by approximately 600KB.
>
> This is the 3rd attempt to land this CL. The previous attempt was reverted at r179373 because ninja sometimes failed to create a manifest for targets that use manifestXXX linker flags. This CL fixes that by manually embedding the manifests into binaries.
>
> BUG=170200,173052
>
> Review URL: https://codereview.chromium.org/12092117
TBR=alexeypa@chromium.org
Review URL: https://codereview.chromium.org/12179013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180440 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Consolidated all installable Chromoting Host core into remoting_core.dll and converted all executables into thin wrappers around entry points exposed by remoting_core.dll. This reduces size of the installer by approximately 600KB.
This is the 3rd attempt to land this CL. The previous attempt was reverted at r179373 because ninja sometimes failed to create a manifest for targets that use manifestXXX linker flags. This CL fixes that by manually embedding the manifests into binaries.
BUG=170200,173052
Review URL: https://codereview.chromium.org/12092117
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180433 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This broke the error handling path.
BUG=134213
Review URL: https://chromiumcodereview.appspot.com/12091112
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180298 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
This updates headers that forward-declare it and a few random places to use the namespace explicitly. There us a using declaration in file_path.h that makes the rest compile, which we can do in future passes.
Review URL: https://codereview.chromium.org/12163003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180245 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This removes the unused L-string macros in the file.
BUG=
Review URL: https://codereview.chromium.org/12090083
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179926 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
media::ScreenCapturer::Delegate. It is useful because both interfaces are implemented by the same class usually while creation of the capturer can be controlled by another (factory) class.
Review URL: https://codereview.chromium.org/12087063
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179903 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Merged all Chromoting Host code into remoting_core.dll (Windows).
>
> Consolidated all installable Chromoting Host core into remoting_core.dll and converted all executables into thin wrappers around entry points exposed by remoting_core.dll. This reduces size of the installer by approximately 600KB.
>
> This is the 2nd attempt to land this CL. It was previously reverted at r179322.
>
> BUG=170200
>
> Review URL: https://codereview.chromium.org/12088049
TBR=alexeypa@chromium.org
Review URL: https://codereview.chromium.org/12090089
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179833 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Currently, several JS files don't even have the basic "load" smoke-test
since they rely on globals not defined in the test environment. This CL
defines these globals so that they can be included in the basic test.
Review URL: https://chromiumcodereview.appspot.com/12087070
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179576 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=104544
Review URL: https://chromiumcodereview.appspot.com/12087066
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179574 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
The directory server notifies the host (via the bot) when an entry is deleted (e.g. by clicking X on the webapp), and the host exits immediately (rather than up to 5 minutes later, when its next heartbeat failed).
BUG=163572
Review URL: https://chromiumcodereview.appspot.com/12045085
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179533 0039d316-1c4b-4281-b951-d872f2087c98
|