| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(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}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Added GCD device ID to host config.
- Disabled email verification for GCD devices.
Disabling email verification is necessary because it is apparently not possible to retrieve an email address for a device registered through GCD.
BUG=471928
Review URL: https://codereview.chromium.org/1108203003
Cr-Commit-Position: refs/heads/master@{#328891}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
| |
BUG=471928
Review URL: https://codereview.chromium.org/1124313002
Cr-Commit-Position: refs/heads/master@{#328868}
|
|
|
|
|
|
|
|
| |
BUG=471928
Review URL: https://codereview.chromium.org/1128753002
Cr-Commit-Position: refs/heads/master@{#328867}
|
|
|
|
| |
Cr-Commit-Position: refs/heads/master@{#328844}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
It's not possible to use Xrandr to configure the display in ozone
builds. This code does not work anyway, so replace it with a stub so
that we can actually compile without putting X into the build
environment.
BUG=484324
TEST=build for chromebook pixel with
https://chromium-review.googlesource.com/#/c/269135/
Review URL: https://codereview.chromium.org/1120753009
Cr-Commit-Position: refs/heads/master@{#328813}
|
|
|
|
|
|
|
|
|
|
|
| |
This CL switches the MANUAL_Me2Me_Connect_Local_Host and MANUAL_Me2Me_v2_Alive_OnLostFocus
to JavaScript.
BUG=480028
Review URL: https://codereview.chromium.org/1126363002
Cr-Commit-Position: refs/heads/master@{#328804}
|
|
|
|
|
|
|
|
|
|
| |
IqSender was using non-normalized JIDs when comparin from field in the
response. This is not a problem at the moment, but it's still good to
have this fixed to prevent problems like crbug.com/485134 in the future.
Review URL: https://codereview.chromium.org/1131823002
Cr-Commit-Position: refs/heads/master@{#328789}
|
|
|
|
|
|
|
|
| |
BUG=478331
Review URL: https://codereview.chromium.org/1126543007
Cr-Commit-Position: refs/heads/master@{#328682}
|
|
|
|
|
|
|
|
|
|
|
| |
Files have had an extra newline appended to them so git will allow me to
create a changelist to review.
BUG=
Review URL: https://codereview.chromium.org/1035903004
Cr-Commit-Position: refs/heads/master@{#328653}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When generating signatures for the messages sent to the bot the JID
must be normalized, i.e. node and domain parts must be in lower-case.
XmppSignalStrategy::GetLocalJid() was previously returning normalized
value, but it was changed recently, which broke hosts that use account
with upper-case characters in the name.
BUG=485134
Review URL: https://codereview.chromium.org/1131653002
Cr-Commit-Position: refs/heads/master@{#328646}
|
|
|
|
|
|
|
|
|
|
| |
HostSignalingManager was added so the signaling connection outlive
HostProcess object. That's no longer necessary and so this class can be
removed as it wasn't doing anything useful anyway.
Review URL: https://codereview.chromium.org/1123973002
Cr-Commit-Position: refs/heads/master@{#328632}
|
|
|
|
|
|
|
|
|
|
|
| |
The constructor wasn't copying standard_ice field which breaks the case
when older client connects to newer host.
BUG=483325
Review URL: https://codereview.chromium.org/1125213003
Cr-Commit-Position: refs/heads/master@{#328631}
|
|
|
|
|
|
|
|
|
| |
This was broken by a recent refactoring. Exposing the function via remoting.experimental
provides a stable interface against future refactorings.
Review URL: https://codereview.chromium.org/1121343002
Cr-Commit-Position: refs/heads/master@{#328434}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
MessageLoopProxy is deprecated.
This basically does a search and replace:
MessageLoopProxy::current() -> ThreadTaskRunnerHandle::Get().
BUG=391045
Review URL: https://codereview.chromium.org/1113493005
Cr-Commit-Position: refs/heads/master@{#328413}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The main thing this fixes is the midi component. Previously midi_unittests was never compiled on any Windows component bot and the component build was all messed up. This patch makes the component build work correctly for this component.
Remove installer_util_unittests.rc and its header. These seem to be autogenerated Visual Studio files. They just include the strings resource file, but GN includes the strings resource file itself by nature of the dependencies (GYP doesn't do this).
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg
BUG=470212
Review URL: https://codereview.chromium.org/1121363002
Cr-Commit-Position: refs/heads/master@{#328383}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
SocketDataProvider (patchset #4 id:60001 of https://codereview.chromium.org/1114383003/)
Reason for revert:
Suspected cause of IOS failure https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/builds/26420/steps/net_unittests%20%28iPhone%205%20iOS%208.1%29/logs/crash%20report%20%282015-05-05-113310%29
Original issue's description:
> Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider
> so that all provider subclasses can have their state verified.
>
> Committed: https://crrev.com/bce10d97d6fc5233071427cf622aaad7abde9e19
> Cr-Commit-Position: refs/heads/master@{#328154}
>
> Committed: https://crrev.com/269875cf0d23881a7ad119bae730ff54fdd4dd0b
> Cr-Commit-Position: refs/heads/master@{#328351}
TBR=rtenneti@chromium.org,sergeyu@chromium.org,rch@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1114353005
Cr-Commit-Position: refs/heads/master@{#328375}
|
|
|
|
|
|
|
|
|
|
|
| |
so that all provider subclasses can have their state verified.
Committed: https://crrev.com/bce10d97d6fc5233071427cf622aaad7abde9e19
Cr-Commit-Position: refs/heads/master@{#328154}
Review URL: https://codereview.chromium.org/1114383003
Cr-Commit-Position: refs/heads/master@{#328351}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
SocketDataProvider (patchset #3 id:40001 of https://codereview.chromium.org/1114383003/)
Reason for revert:
Final stop on the revert train!
net_unittests on iOS Simulator (dbg) (iPhone 5 ios 8.1) should be fixed after this?
https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_(dbg)
Here's a log of the break:
https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/builds/26378/steps/net_unittests%20%28iPhone%205%20iOS%208.1%29/logs/crash%20report%20%282015-05-04-164655%29
Original issue's description:
> Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider
> so that all provider subclasses can have their state verified.
>
> Committed: https://crrev.com/bce10d97d6fc5233071427cf622aaad7abde9e19
> Cr-Commit-Position: refs/heads/master@{#328154}
TBR=rtenneti@chromium.org,sergeyu@chromium.org,rch@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1119803006
Cr-Commit-Position: refs/heads/master@{#328271}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With this change the GN and GYP webapp builds will once again produce
identical results.
The remoting_localize.py change is necessary to avoid having Fake-bidi
conflict with Hebrew.
BUG=
Review URL: https://codereview.chromium.org/1126683003
Cr-Commit-Position: refs/heads/master@{#328259}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixes
..\..\remoting/host/win/rdp_desktop_session.h(40,23) : warning(clang):
[chromium-style] Overriding method must be marked with 'override' or 'final'.
BUG=467287
TBR=sergeyu@chromium.org
Review URL: https://codereview.chromium.org/1122143002
Cr-Commit-Position: refs/heads/master@{#328251}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
setup.exe: https://build.chromium.o… (patchset #1 id:1 of https://codereview.chromium.org/1124563003/)
Reason for revert:
Broke the compile because one of these CLs is already depended on (rch@'s bce10d97d6fc5233071427cf622aaad7abde9e19).
Original issue's description:
> Revert some other possible suspects of a 0.43% size increase of setup.exe: https://build.chromium.org/p/chromium/builders/Win/builds/31562
>
> Revert "Add granular base::File tracing."
>
> This reverts commit a6e05c977096a03774e5406d63ad80c0166f9adc.
>
> Revert "Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider"
>
> This reverts commit bce10d97d6fc5233071427cf622aaad7abde9e19.
>
> Revert "Avoid unnecessary memory allocations at PlatformThread::SetName()"
>
> This reverts commit 4839a142bf95776323647c82ca9dc0725f7c4f15.
>
> TBR=tdresser@chromium.org
> BUG=none
> TEST=green sizes
>
> Committed: https://chromium.googlesource.com/chromium/src/+/c41830d6f55f85e2f5c8841db4b6ed81239b671d
TBR=tdresser@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=none
Review URL: https://codereview.chromium.org/1123833003
Cr-Commit-Position: refs/heads/master@{#328207}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
SocketDataProvider so that all provider… (patchset #1 id:1 of https://codereview.chromium.org/1127623003/)
Reason for revert:
Still not compiling. Order-dependent revert fun to ensue. End result: this CL and any dependent CL will be re-instated.
Original issue's description:
> Re-instating r328154 because it's depended on to compile.
>
> Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider so that all provider subclasses can have their state verified.
>
> Review URL: https://codereview.chromium.org/1114383003
>
> Cr-Commit-Position: refs/heads/master@{#328154}
>
> TBR=rch@chromium.org
> BUG=
>
> Committed: https://chromium.googlesource.com/chromium/src/+/2b2149aa9dfc5fbbb4ef0df2b89b5f392383f41a
TBR=rch@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=
Review URL: https://codereview.chromium.org/1123643006
Cr-Commit-Position: refs/heads/master@{#328206}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider so that all provider subclasses can have their state verified.
Review URL: https://codereview.chromium.org/1114383003
Cr-Commit-Position: refs/heads/master@{#328154}
TBR=rch@chromium.org
BUG=
Review URL: https://codereview.chromium.org/1127623003
Cr-Commit-Position: refs/heads/master@{#328205}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://build.chromium.org/p/chromium/builders/Win/builds/31562
Revert "Add granular base::File tracing."
This reverts commit a6e05c977096a03774e5406d63ad80c0166f9adc.
Revert "Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider"
This reverts commit bce10d97d6fc5233071427cf622aaad7abde9e19.
Revert "Avoid unnecessary memory allocations at PlatformThread::SetName()"
This reverts commit 4839a142bf95776323647c82ca9dc0725f7c4f15.
TBR=tdresser@chromium.org
BUG=none
TEST=green sizes
Review URL: https://codereview.chromium.org/1124563003
Cr-Commit-Position: refs/heads/master@{#328199}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Application.Mode was effectively a global variable that kept track of
which app was running (DesktopRemoting vs. AppRemoting) and which
mode (It2ME or Me2Me).
This cl removes Application.Mode and moves the Mode tracking into
the DesktopRemoting app. There are only a few remaining places in
the code that care about the It2Me vs Me2Me distinction (testing,
menu enabling, logging, ...).
Requires that crrev.com/1095203010 land first.
BUG=
Review URL: https://codereview.chromium.org/1105853004
Cr-Commit-Position: refs/heads/master@{#328179}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixes:
..\..\remoting/host/win/rdp_desktop_session.h(22,1) : warning(clang):
[chromium-style] Complex class/struct needs an explicit out-of-line destructor.
BUG=467287
Review URL: https://codereview.chromium.org/1126643002
Cr-Commit-Position: refs/heads/master@{#328169}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The PlatformThread::SetName() method used to accept 'name' parameter
as 'const char*' even though all the clients stored as 'std::string' and
thus had to convert, further inside PlatformThread::SetName() it was
converted back to 'std::string' (when passing to ThreadIdNameManager::SetName())
leading to unnecessary memory allocations.
See also: https://groups.google.com/a/chromium.org/forum/#!msg/chromium-dev/EUqoIz2iFU4/kPZ5ZK0K3gEJ
Review URL: https://codereview.chromium.org/1093223007
Cr-Commit-Position: refs/heads/master@{#328157}
|
|
|
|
|
|
|
|
| |
so that all provider subclasses can have their state verified.
Review URL: https://codereview.chromium.org/1114383003
Cr-Commit-Position: refs/heads/master@{#328154}
|
|
|
|
|
|
|
|
| |
BUG=483669
Review URL: https://codereview.chromium.org/1120223002
Cr-Commit-Position: refs/heads/master@{#328030}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The first pass of moving the other Chrome-specific build flags out of the root build config file and into the new one. This requires both ffmpeg and Blink changes to finish landing.
Originally landed as https://codereview.chromium.org/1120803003/
TBR=scottmg@chromium.org
TBR=ddorwin@chromium.org (widevine)
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg
Review URL: https://codereview.chromium.org/1123433002
Cr-Commit-Position: refs/heads/master@{#327998}
|
|
|
|
| |
Cr-Commit-Position: refs/heads/master@{#327972}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Most interfaces were storing the object in a scoped_ptr already. This
adjusts the API so that it is a little clearer of the ownership
transfer.
A number of clients put the URLFetcher in a table and do memory
management on it themselves; this is likely templatable code
for a future CL. The scope of this CL was to change the API but
no control flow changes. Making this change found one memory leak;
(http://crbug.com/482459) has been addressed separately.
BUG=371201
TESTS=net_unittests google_api_unittests
TBR=jochen@chromium.org, thakis@chromium.org, oshima@chromium.org, armansito@chromium.org, reillyg@chromium.org, rogerta@chromium.org, stuartmorgan@chromium.org, wez@chromium.org, pavely@chromium.org, rouslan@chromium.org
Review URL: https://codereview.chromium.org/1117703002
Cr-Commit-Position: refs/heads/master@{#327901}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:40001 of https://codereview.chromium.org/1120803003/)
Reason for revert:
May have caused some link errors on windows: https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN/builds/814
Original issue's description:
> Add a GN flag for the Windows multi-dll build.
>
> The first pass of moving the other Chrome-specific build flags out of the root build config file and into the new one. This requires both ffmpeg and Blink changes to finish landing.
>
> R=scottmg@chromium.org
> TBR=ddorwin@chromium.org (widevine)
>
> Committed: https://crrev.com/edff53a35780ba2f515c7e8d4f7131c91151e1a3
> Cr-Commit-Position: refs/heads/master@{#327854}
TBR=scottmg@chromium.org,brettw@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1117063004
Cr-Commit-Position: refs/heads/master@{#327861}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since it doesn't really work yet, things are slightly cleaner if we
don't try to build anything we don't yet need.
I will reenable what I can once the rest of the port is caught up.
TBR=garykac@chromium.org
BUG=431177
CQ_EXTRA_TRYBOTS=tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg
Review URL: https://codereview.chromium.org/1121623004
Cr-Commit-Position: refs/heads/master@{#327858}
|
|
|
|
|
|
|
|
|
|
|
| |
The first pass of moving the other Chrome-specific build flags out of the root build config file and into the new one. This requires both ffmpeg and Blink changes to finish landing.
R=scottmg@chromium.org
TBR=ddorwin@chromium.org (widevine)
Review URL: https://codereview.chromium.org/1120803003
Cr-Commit-Position: refs/heads/master@{#327854}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
to help diagnose stale access token in some app remoting scenarios
(tracked internally at b/20662147)
The new log entry complements an existing one that logs client connected
events in ChromotingHost::OnIncomingSession (chromoting_host.cc:302).
BUG=
Review URL: https://codereview.chromium.org/1117043002
Cr-Commit-Position: refs/heads/master@{#327822}
|
|
|
|
|
|
|
|
|
|
|
| |
Working on trimming down plugin warnings on the Windows clang build.
BUG=467287
TBR=jamiewalch@chromium.org
Review URL: https://codereview.chromium.org/1115773002
Cr-Commit-Position: refs/heads/master@{#327548}
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
| |
Hosts registered through GCD can't be brought online yet, because the
host doesn't know how to use the XMPP credentials it gets from GCD.
BUG=471928
Review URL: https://codereview.chromium.org/1111603002
Cr-Commit-Position: refs/heads/master@{#327427}
|
|
|
|
|
|
|
|
|
|
|
| |
This also makes the Note at the top of the files (to keep the GYP and
GN files in sync) a bit more noticeable.
BUG=
Review URL: https://codereview.chromium.org/1116463002
Cr-Commit-Position: refs/heads/master@{#327378}
|
|
|
|
|
|
|
|
| |
BUG=b/19609728
Review URL: https://codereview.chromium.org/1066003007
Cr-Commit-Position: refs/heads/master@{#327338}
|
|
|
|
|
|
|
|
| |
BUG=471928
Review URL: https://codereview.chromium.org/1104383002
Cr-Commit-Position: refs/heads/master@{#327219}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Recent refactoring allows us to remove many CRD-specific files from
the set of "common" JS files. This cl moves the CRD UI JS files
out of the AppRemoting build. This is in preparation for removing
Application.Mode in a follow-up cl.
Requires that crrev.com/1109453005 land first.
BUG=
Review URL: https://codereview.chromium.org/1095203010
Cr-Commit-Position: refs/heads/master@{#327205}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Without these, a GN clobber build fails, and we get warnings
during the non-compile builds.
R=garykac@chromium.org
BUG=
Review URL: https://codereview.chromium.org/1108953002
Cr-Commit-Position: refs/heads/master@{#327165}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL contains several fixes needed to make the ChromeOS GN debug component build link.
R=brettw@chromium.org, erg@chromium.org, garykac@chromium.org, kjellander@chromium.org
BUG=433082
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_chromium_gn_chromeos_dbg;tryserver.chromium.mac:mac_chromium_gn_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg
Review URL: https://codereview.chromium.org/1054963007
Cr-Commit-Position: refs/heads/master@{#327164}
|
|
|
|
|
|
|
|
|
|
| |
Since the width of this dialog is no longer fixed, this CL also updates base.resizeWindowToContent to set the width as well as the height, and adds support for centering.
BUG=b/19796166
Review URL: https://codereview.chromium.org/1081813007
Cr-Commit-Position: refs/heads/master@{#327160}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
WebKit has been fixed in
https://src.chromium.org/viewvc/blink?revision=193998&view=revision
BUG=None
TEST=trybots
R=thestig@chromium.org
TBR=sergeyu@chromium.org,cpu@chromium.org
Review URL: https://codereview.chromium.org/1053123010
Cr-Commit-Position: refs/heads/master@{#327147}
|
|
|
|
|
|
|
|
| |
BUG=471928
Review URL: https://codereview.chromium.org/1099553003
Cr-Commit-Position: refs/heads/master@{#327144}
|