| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Reland http://crrev.com/1764503002 with fixes for the Android app and
tests:
* Initialize ChromotingClientRuntime::ui_task_runner_
* Add new unittest to GN build.
* Don't call MessageLoopForUI::Start() in Android test.
TBR=nicholss
Review URL: https://codereview.chromium.org/1827573002
Cr-Commit-Position: refs/heads/master@{#382757}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
#23 id:440001 of https://codereview.chromium.org/1764503002/ )
Reason for revert:
Broke Android client and tests - see bug 596240.
Original issue's description:
> Adding container class for chromoting client runtimes.
>
> R=lambroslambrou@chromium.org
>
> Committed: https://crrev.com/b34c0f6877af54e8bdd6052bf8934dd46a3b5899
> Cr-Commit-Position: refs/heads/master@{#380979}
TBR=sergeyu@chromium.org,nicholss@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
Review URL: https://codereview.chromium.org/1821623003
Cr-Commit-Position: refs/heads/master@{#382411}
|
|
|
|
|
|
|
|
|
| |
Removed all files used to build app_remoting webapp and cleaned up
build files.
Review URL: https://codereview.chromium.org/1816653002
Cr-Commit-Position: refs/heads/master@{#382147}
|
|
|
|
|
|
|
|
|
|
| |
Reduced official build size from 7.9MB to 6.4MB. Can reenable WebRTC by changing the BUILD.gn file.
BUG=595038
Review URL: https://codereview.chromium.org/1806963002
Cr-Commit-Position: refs/heads/master@{#381636}
|
|
|
|
|
|
|
|
| |
R=lambroslambrou@chromium.org
Review URL: https://codereview.chromium.org/1764503002
Cr-Commit-Position: refs/heads/master@{#380979}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL provides remote assistance in Public Session.
Summary of changes:
1. Move the remoting.testEvents initialization to ui_mode.js
as it is only referenced there.
2. Create a separate page public_session.html that contains the
main ui for public session.
3. Use the service account email address to infer whether we
are currently in a public session.
4. Decouple remoting.HostController from host_screen.js
Previously, remoting.HostController is used to retrieve the
host version, which is problematic as that refers to the me2me
host version as opposed to the it2me host version.
BUG=508608
Review URL: https://codereview.chromium.org/1370283004
Cr-Commit-Position: refs/heads/master@{#351861}
|
|
|
|
|
|
|
|
|
|
|
| |
CRD app now bundles PNaCl version of the plugin so it doesn't need to
be built in chrome anymore.
BUG=510271
Review URL: https://codereview.chromium.org/1271523003
Cr-Commit-Position: refs/heads/master@{#341804}
|
|
|
|
|
|
|
|
|
| |
1. Removed CRD v1 app.
2. Removed support for native plugin from the webapp.
Review URL: https://codereview.chromium.org/1251273002
Cr-Commit-Position: refs/heads/master@{#341794}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Reason for revert:
TSan bot build failuire:
KeyError: '/mnt/data/b/build/slave/Chromium_Linux_TSan_Builder/build/src/remoting/remoting.gyp:remoting_webapp#target'
Error: Command '/usr/bin/python src/build/gyp_chromium' returned non-zero exit status 1 in /mnt/data/b/build/slave/Chromium_Linux_TSan_Builder/build
>
> Remove legacy CRD application types.
>
> 1. Removed CRD v1 app.
> 2. Removed support for native plugin from the webapp.
>
> Review URL: https://codereview.chromium.org/1251273002
>
> Cr-Commit-Position: refs/heads/master@{#341635}
BUG=516539
TBR=sergeyu@chromium.org
Review URL: https://codereview.chromium.org/1270693003 .
Cr-Commit-Position: refs/heads/master@{#341684}
|
|
|
|
|
|
|
|
|
| |
1. Removed CRD v1 app.
2. Removed support for native plugin from the webapp.
Review URL: https://codereview.chromium.org/1251273002
Cr-Commit-Position: refs/heads/master@{#341635}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(re-land of https://codereview.chromium.org/1202523007)
This also slightly changes the behavior of the embedded WebView. Now it
only renders content inline if it originates from the initial host that
it was told to use (otherwise it launches an external browser). This
simplifies the code (by using a single explicitly-defined
WebViewClient implementation).
BUG=499478
Review URL: https://codereview.chromium.org/1207073003
Cr-Commit-Position: refs/heads/master@{#336079}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a cross-platform change that adds credits.txt as a pre-requisite
for remoting_me2me_host. It also adds that file to the Mac bundle, which
means it is automatically installed on that platform, and adds rules to
install the file to /usr/share/doc/chrome-remote-desktop on Linux. The
Windows installer will be updated in a later CL.
It also updates the naming for the web-app credits to make the difference
clearer.
BUG=499478
Review URL: https://codereview.chromium.org/1187543004
Cr-Commit-Position: refs/heads/master@{#336023}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This allows products that are built from the Chromium source to include
third-party license attribution, even if they cannot link directly to
chrome://credits. Chrome apps such as Chromoting, for example, fall into
this category, for example.
BUG=499478
Review URL: https://codereview.chromium.org/1178323002
Cr-Commit-Position: refs/heads/master@{#334957}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This adds GN build rules for the AppRemoting sample app.
Some DesktopRemoting build rules were prettified as part of this
process (to make the App- and Desktop- remoting build rules more
similar).
This re-re-lands crrev.com/1126393002 without any additional changes
required. The original issues with the GN clobber builds are fixed in
crrev.com/1137753002. The 2nd attempt to land was rejected due to
a merge race (fixed in this cl).
BUG=471916
Review URL: https://codereview.chromium.org/1143643003
Cr-Commit-Position: refs/heads/master@{#330243}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(patchset #1 id:1 of https://codereview.chromium.org/1135933002/)
Reason for revert:
Broke GN bots, eg.:
http://build.chromium.org/p/chromium.linux/builders/Linux%20GN/builds/28281
Original issue's description:
> [Chromoting] Add GN build support for ar_sample_app target.
>
> This adds GN build rules for the AppRemoting sample app.
> Some DesktopRemoting build rules were prettified as part of this
> process (to make the App- and Desktop- remoting build rules more
> similar).
>
> This re-lands crrev.com/1126393002 without any additional changes
> required. The issues with the GN clobber builds are fixed in
> crrev.com/1137753002.
>
> BUG=471916
>
> Committed: https://crrev.com/581797d8f76d5855b7f0bed1fe912a038d1d7d25
> Cr-Commit-Position: refs/heads/master@{#329206}
TBR=jamiewalch@chromium.org,garykac@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=471916
Review URL: https://codereview.chromium.org/1136673003
Cr-Commit-Position: refs/heads/master@{#329228}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This adds GN build rules for the AppRemoting sample app.
Some DesktopRemoting build rules were prettified as part of this
process (to make the App- and Desktop- remoting build rules more
similar).
This re-lands crrev.com/1126393002 without any additional changes
required. The issues with the GN clobber builds are fixed in
crrev.com/1137753002.
BUG=471916
Review URL: https://codereview.chromium.org/1135933002
Cr-Commit-Position: refs/heads/master@{#329206}
|
|
|
|
|
|
|
|
|
|
|
|
| |
The names dom3 and dom4 reflect obsolete documentation and no longer
denote anything useful.
R=wez
TBR=jam
Review URL: https://codereview.chromium.org/1136503003
Cr-Commit-Position: refs/heads/master@{#329176}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(patchset #3 id:40001 of https://codereview.chromium.org/1126393002/)
Reason for revert:
Broke Linux GN Clobber build.
http://build.chromium.org/p/chromium.linux/builders/Linux%20GN%20Clobber/builds/49/steps/compile/logs/stdio
Original issue's description:
> [Chromoting] Add GN build support for ar_sample_app target.
>
> This adds GN build rules for the AppRemoting sample app.
> Some DesktopRemoting build rules were prettified as part of this
> process (to make the App- and Desktop- remoting build rules more
> similar).
>
> BUG=471916
>
> Committed: https://crrev.com/13a3ca1f8053b1b2d195b8f5f52b90e184287ea8
> Cr-Commit-Position: refs/heads/master@{#328871}
TBR=jamiewalch@chromium.org,garykac@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=471916
Review URL: https://codereview.chromium.org/1135603003
Cr-Commit-Position: refs/heads/master@{#328902}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This adds GN build rules for the AppRemoting sample app.
Some DesktopRemoting build rules were prettified as part of this
process (to make the App- and Desktop- remoting build rules more
similar).
BUG=471916
Review URL: https://codereview.chromium.org/1126393002
Cr-Commit-Position: refs/heads/master@{#328871}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL creates a new target webapp with browser test JavaScript files loaded.
This will enable us to run a JavaScript browser test using a regular chrome
binary to speed up development and debugging.
BUG=
Review URL: https://codereview.chromium.org/1091293004
Cr-Commit-Position: refs/heads/master@{#327438}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previous only a single sad and lonely html file (main.html) from the
Chromoting webapp build was converted to GN. With this cl, then entirety
of the Chromoting webapp has been converted to GN and produces results
identical to the GYP build.
BUG=
Review URL: https://codereview.chromium.org/1087693003
Cr-Commit-Position: refs/heads/master@{#326368}
|
|
|
|
|
|
|
|
|
|
| |
crrev.com/321815.
TBR=jamiewalch@chromium.org
Review URL: https://codereview.chromium.org/1019183005
Cr-Commit-Position: refs/heads/master@{#321844}
|
|
|
|
|
|
|
|
| |
BUG=469763
Review URL: https://codereview.chromium.org/1022983003
Cr-Commit-Position: refs/heads/master@{#321815}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This cl also cleans up some of the gyp variables related to managing JS files for HTML files:
* Remove unused params to build-html.py
* Group related vars in remoting_webapp_files.gypi
BUG=
Review URL: https://codereview.chromium.org/977903003
Cr-Commit-Position: refs/heads/master@{#319480}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With this change the Debug version of the PNaCl plugin will be copied to
the webapp directory allowing to debug PNaCl plugin (using nacl-gdb).
Also updated build-webapp.py to process all *.jinja2 input files as jinja2
templates.
This patch was previously reverted because of a problem that should be
fixed now by https://codereview.chromium.org/872673006/ .
BUG=451643
Review URL: https://codereview.chromium.org/872633006
Cr-Commit-Position: refs/heads/master@{#314873}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(patchset #1 id:1 of https://codereview.chromium.org/872633006/)
Reason for revert:
Broke compile, e.g. http://build.chromium.org/p/chromium/buildstatus?builder=Win&number=28337
Original issue's description:
> Copy debug PNaCl plugin in Dev builds of the chromoting webapp.
>
> With this change the Debug version of the PNaCl plugin will be copied to
> the webapp directory allowing to debug PNaCl plugin (using nacl-gdb).
>
> Also updated build-webapp.py to process all *.jinja2 input files as jinja2
> templates.
>
> BUG=451643
>
> Committed: https://crrev.com/65ffdafc8538b14ad0cbdf6a57d9d21e1e882a9d
> Cr-Commit-Position: refs/heads/master@{#313024}
TBR=jamiewalch@chromium.org,sergeyu@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=451643
Review URL: https://codereview.chromium.org/870273002
Cr-Commit-Position: refs/heads/master@{#313025}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With this change the Debug version of the PNaCl plugin will be copied to
the webapp directory allowing to debug PNaCl plugin (using nacl-gdb).
Also updated build-webapp.py to process all *.jinja2 input files as jinja2
templates.
BUG=451643
Review URL: https://codereview.chromium.org/872633006
Cr-Commit-Position: refs/heads/master@{#313024}
|
|
|
|
|
|
|
|
|
|
|
| |
Now remoting_webapp_v2 doesn't use PNaCl. Also added new target
remoting_webapp_v2_pnacl that does use PNaCl.
BUG=451184
Review URL: https://codereview.chromium.org/865363002
Cr-Commit-Position: refs/heads/master@{#312890}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The current AppRemoting (and Chromoting) gypi files assume that they are
being included from the src/remoting directory. This is a valid assumption
for Chromoting and the AppRemoting sample targets, but it is not true for
the additional targets which are required to live elsewhere.
This cl updates the GYPIfiles so that they support being included from
directories other than src/remoting.
It also moves the JScompile action onto the AppRemoting sample apps since it
is not necessary to run it for every AppRemoting app target.
BUG=
Review URL: https://codereview.chromium.org/855703002
Cr-Commit-Position: refs/heads/master@{#311987}
|
|
|
|
|
|
|
|
|
|
|
| |
libwebm was used in remoting, but is not needed anymore.
Committed: https://crrev.com/c053fb4fec709c4a5a72065305ee4eb2da2efaa8
Cr-Commit-Position: refs/heads/master@{#309243}
Review URL: https://codereview.chromium.org/820593002
Cr-Commit-Position: refs/heads/master@{#309295}
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit c053fb4fec709c4a5a72065305ee4eb2da2efaa8.
BUG=
TBR=sergeyu@chromium.org
Review URL: https://codereview.chromium.org/813423002
Cr-Commit-Position: refs/heads/master@{#309248}
|
|
|
|
|
|
|
|
| |
libwebm was used in remoting, but is not needed anymore.
Review URL: https://codereview.chromium.org/820593002
Cr-Commit-Position: refs/heads/master@{#309243}
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/742473002
Cr-Commit-Position: refs/heads/master@{#306336}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
remoting/webapp/crd/html -- CRD-specific HTML/CSS
remoting/webapp/crd/js -- CRD-specific JS
remoting/webapp/crd/resources -- CRD-specific resources
remoting/webapp/base/html -- common HTML/CSS
remoting/webapp/base/js -- common JS
remoting/webapp/base/resources -- common resources
The only content changes are in the GYP/GYPI files and in template_main.html. These files needed to be updated to use the new file paths.
BUG=
R=jamiewalch@chromium.org
Review URL: https://codereview.chromium.org/671513008
Cr-Commit-Position: refs/heads/master@{#300806}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since GN runs its scripts in a different current directory than GYP, I had to update the remoting html template script to be able to handle this situation.
Also includes many misc linking fixes:
- Sets enable_nacl to false.
- Add renderer context menu component
- Adds many missing files
R=sergeyu@chromium.org
Review URL: https://codereview.chromium.org/498773005
Cr-Commit-Position: refs/heads/master@{#291731}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL:
- Moves background.js to background.html
- Introduces remoting.appLauncher that allows the caller to launch and close the webapp without knowing the implementation difference between a v1 app and a v2 app.
NOTRY=true
R=jamiewalch@chromium.org
Review URL: https://codereview.chromium.org/450383003
Cr-Commit-Position: refs/heads/master@{#289096}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@289096 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change links the trusted plugin into the renderer, making it easier for
the trusted plugin to use libraries in Chromium. This removes the
ppGoogleNaClPlugin loadable module file on various platforms.
This is part of a larger effort to remove the "trusted plugin" used to
bootstrap NaCl plugins.
It introduces an "internal_module" interface for setting the value returned by
pp::Module::Get(). This is so that both the trusted plugin and the remoting
plugin can be linked into the renderer. However, I believe this is safe because
the trusted plugin runs in-process and the remoting plugin runs out-of-process.
BUG=394497
R=dmichael@chromium.org, mseaborn@chromium.org, phajdan.jr@chromium.org, thestig@chromium.org, wez@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=287071
Review URL: https://codereview.chromium.org/397243004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287580 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
The reverted change broke the ChromeOS builder, which depends on the loadable
module file existing. Argh.
This reverts commit b0b91c06bc2ed6afc0611f00d68f72b19dd06d26.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287090 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change links the trusted plugin into the renderer, making it easier for
the trusted plugin to use libraries in Chromium. This removes the
ppGoogleNaClPlugin loadable module file on various platforms.
This is part of a larger effort to remove the "trusted plugin" used to
bootstrap NaCl plugins.
It introduces an "internal_module" interface for setting the value returned by
pp::Module::Get(). This is so that both the trusted plugin and the remoting
plugin can be linked into the renderer. However, I believe this is safe because
the trusted plugin runs in-process and the remoting plugin runs out-of-process.
BUG=394497
R=dmichael@chromium.org, mseaborn@chromium.org, phajdan.jr@chromium.org, thestig@chromium.org, wez@chromium.org
Review URL: https://codereview.chromium.org/397243004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287071 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Also disabled click-to-play logic for PNaCl plugin
BUG=276739
R=jamiewalch@chromium.org
Review URL: https://codereview.chromium.org/343983003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278886 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=134215
R=weitaosu@chromium.org, wez@chromium.org
Review URL: https://codereview.chromium.org/342583002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278198 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously keyboard filters were selected using the per-platform
filename exclusion rules, and that doesn't work for PNaCl plugin. Now
both Mac and ChromeOS filters are compiled into the PNaCl plugin and
selected in runtime.
BUG=276739
Review URL: https://codereview.chromium.org/292103004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271997 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=276739
R=jamiewalch@chromium.org, rsleevi@chromium.org
Review URL: https://codereview.chromium.org/234023003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271351 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Build remoting for PNaCl
>
> BUG=276739
> R=jamiewalch@chromium.org, rsleevi@chromium.org
>
> Review URL: https://codereview.chromium.org/234023003
TBR=sergeyu@chromium.org
Review URL: https://codereview.chromium.org/285243003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271344 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=276739
R=jamiewalch@chromium.org, rsleevi@chromium.org
Review URL: https://codereview.chromium.org/234023003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271343 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reports the same connection statistics and session change events
that are currently reported by the web-app.
Caveats:
This does not provide Chrome or Webapp versions - instead it
reports the Android OS version, and the app version, under
differently-named keys.
Session termination events don't seem to be reported. This is a
limitation of the Android client that might not be easy to work around.
BUG=368015
R=rmsousa@chromium.org
Review URL: https://codereview.chromium.org/270613005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269404 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Build remoting for PNaCl
>
> BUG=276739
> R=jamiewalch@chromium.org
>
> Review URL: https://codereview.chromium.org/234023003
TBR=sergeyu@chromium.org
Review URL: https://codereview.chromium.org/254243002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@267064 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=276739
R=jamiewalch@chromium.org
Review URL: https://codereview.chromium.org/234023003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@267030 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=276739
Review URL: https://codereview.chromium.org/185163002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@254355 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added two new targets remoting_webapp_v1 and remoting_webapp_v2. Both
include the new remoting_webapp.gypi. remoting_webapp target
depends on both. Also moved these targets to remoting_client.gypi.
remoting_webapp.gypi will also be reused for webapp version that
uses PNaCl plugin.
BUG=276739
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=253800
R=jamiewalch@chromium.org
Review URL: https://codereview.chromium.org/181473008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@253885 0039d316-1c4b-4281-b951-d872f2087c98
|