summaryrefslogtreecommitdiffstats
path: root/remoting/protocol
Commit message (Collapse)AuthorAgeFilesLines
* Remove even more headers from base/memory/scoped_ptr.hdcheng2016-03-251-0/+1
| | | | | | | | BUG=554298 Review URL: https://codereview.chromium.org/1828923004 Cr-Commit-Position: refs/heads/master@{#383315}
* Add more out of line copy ctors for complex classes.vmpstr2016-03-244-0/+5
| | | | | | | | | | | | | | | This patch adds the remaining copy constructors for complex classes. After this patch, it should be possible to enable the heavy class copy constructor checks by default. R=thakis@chromium.org, dcheng@chromium.org TBR=jam@chromium.org BUG=436357 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1825273002 Cr-Commit-Position: refs/heads/master@{#383131}
* Remove use of deprecated methods to cricket::VideoCapturer in ↵perkj2016-03-243-16/+9
| | | | | | | | | | | | | WebrtcVideoCapturerAdapter VideoCapturer::Pause is never used in the webrtc code base. SignalgVideoFrame should also be removed and replaced with normal method call to VideoCapturer::OnFrame. BUG=webrtc:5426 Review URL: https://codereview.chromium.org/1745923002 Cr-Commit-Position: refs/heads/master@{#383040}
* Show max latency on client's status baryuweih2016-03-232-14/+14
| | | | | | | | | | | | | Will later also consider storing max latency data in server BUG=560950 Committed: https://crrev.com/e37a467453f3814a9ecb0eb6769bd7c58b61cbae Cr-Commit-Position: refs/heads/master@{#382663} Review URL: https://codereview.chromium.org/1811833002 Cr-Commit-Position: refs/heads/master@{#382957}
* Revert of Show max latency on client's status bar (patchset #8 id:140001 of ↵newt2016-03-232-14/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1811833002/ ) Reason for revert: Caused compile failure on 64-bit Android bots. See crbug.com/597155 Original issue's description: > Show max latency on client's status bar > > Will later also consider storing max latency data in server > > BUG=560950 > > Committed: https://crrev.com/e37a467453f3814a9ecb0eb6769bd7c58b61cbae > Cr-Commit-Position: refs/heads/master@{#382663} TBR=sergeyu@chromium.org,kelvinp@chromium.org,dbeam@chromium.org,yuweih@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=560950 Review URL: https://codereview.chromium.org/1817093005 Cr-Commit-Position: refs/heads/master@{#382777}
* Show max latency on client's status baryuweih2016-03-222-14/+14
| | | | | | | | | | Will later also consider storing max latency data in server BUG=560950 Review URL: https://codereview.chromium.org/1811833002 Cr-Commit-Position: refs/heads/master@{#382663}
* net: remove UsingTCPFastOpen() method from StreamSocket classtfarina2016-03-221-4/+0
| | | | | | | | | | | | | | | Commit c4699f6 ("net: add IsUsingTCPFastOpen() method to SequencedSocketData test class") removed the usage of UsingTCPFastOpen(), so now it can be removed safely. BUG=488598 TEST=net_unittests R=mmenke@chromium.org TBR=dgozman@chromium.org,sergeyu@chromium.org Review URL: https://codereview.chromium.org/1821633003 Cr-Commit-Position: refs/heads/master@{#382604}
* Roll WebRTC 12049:12058, Libjingle 12042:12058tommi2016-03-191-1/+1
| | | | | | | | | | | | | | | | | WebRTC 12049:12058 Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/957d599..0912ecc Libjingle 12042:12058 Changes: https://chromium.googlesource.com/external/webrtc/trunk/talk.git/+log/8bc093b..35814a7 TBR=jbauch@webrtc.org BUG= Review URL: https://codereview.chromium.org/1822473002 Cr-Commit-Position: refs/heads/master@{#382204}
* Add signatures for session-description messages.sergeyu2016-03-186-38/+91
| | | | | | | | | | | | | Now the session-description messages are signed with the key generated by the authenticators and the signature is verified by the receiver. Verification can be disabled using --disable-authentication flag. BUG=547158 Review URL: https://codereview.chromium.org/1815043002 Cr-Commit-Position: refs/heads/master@{#382120}
* Remove Chrome dependency to empty videosourceinterface.hperkj2016-03-182-2/+0
| | | | | | | | | | third_party/webrtc/api/videosourceinterface.h has been merged with mediastreaminterface.h and will be removed. BUG=webrtc:5426 Review URL: https://codereview.chromium.org/1810273003 Cr-Commit-Position: refs/heads/master@{#381998}
* Remove spake2_plain authentication method.sergeyu2016-03-179-104/+45
| | | | | | | | | | It2me host can use spake2_hmac authentication method and it works properly with the current client, so there is no reason to keep spake2_plain for It2Me. Review URL: https://codereview.chromium.org/1799293002 Cr-Commit-Position: refs/heads/master@{#381744}
* Replace all uses of webrtc::VideoRendererInterface by rtc::VideoSinkInterface.nisse2016-03-172-8/+14
| | | | | | | | BUG=webrtc:5426 Review URL: https://codereview.chromium.org/1809623002 Cr-Commit-Position: refs/heads/master@{#381678}
* Add Curve25519 version of pairing authenticatorssergeyu2016-03-176-107/+255
| | | | | | | | BUG=589698 Review URL: https://codereview.chromium.org/1800823002 Cr-Commit-Position: refs/heads/master@{#381664}
* Handle pairing client ID in the negotiating authenticators.sergeyu2016-03-1714-143/+172
| | | | | | | | | | | | | | | | | | | Previously PairingAuthenticator classes were responsible for handling client ID. This meant that in case there are multiple pairing authentication methods the client would have to resend the client_id if the host chooses different authentication method. Now the client ID is handled by the negotiating layer, so client_id needs to be sent only once. Also now NegotiatingAuthenticators create PairingAuthenticators even when falling back to PIN. This ensures that <pairing-failed> tag is always present when falling back to PIN. BUG=593123 Review URL: https://codereview.chromium.org/1781173005 Cr-Commit-Position: refs/heads/master@{#381633}
* Normalize JIDs passed to Spake2Authenticator.sergeyu2016-03-161-3/+6
| | | | | | | | | | | | Spake2Authenticator requires that local_id and remote_id are the same on both ends of the connection, so JIDs need to be normalized to avoid issues when one of the ends gets JID in a different case. BUG=589698 Review URL: https://codereview.chromium.org/1803893002 Cr-Commit-Position: refs/heads/master@{#381559}
* Enable TURN on the host when using WebRTC.sergeyu2016-03-164-34/+61
| | | | | | | | BUG=577954 Review URL: https://codereview.chromium.org/1800893002 Cr-Commit-Position: refs/heads/master@{#381520}
* Use ClientAuthenticationConfig in PairingClientAuthenticatorsergeyu2016-03-1211-76/+100
| | | | | | | | | | | Moved ClientAuthenticationConfig to a separate file and now it's used in PairingClientAuthenticator. BUG=593123 Review URL: https://codereview.chromium.org/1794433002 Cr-Commit-Position: refs/heads/master@{#380872}
* Make TokenValidatorFactory ref-counted.sergeyu2016-03-125-25/+22
| | | | | | | | | This makes TokeValidatorFactory consistent with PairingRegistry and now it can be shared between multiple authenticator objects. Review URL: https://codereview.chromium.org/1788443005 Cr-Commit-Position: refs/heads/master@{#380871}
* Enable Curve25519 in host and client for PIN-based and third-party auth.sergeyu2016-03-1212-73/+171
| | | | | | | | | | | | | | | | | | | Now client and host can use Spake2Authenticator instead of V2Authenticator for both PIN-based auth and third-party auth. The plan is to use cleanup It2Me to use hashed ID, so it will be possible to remove spake2_plain method completely, so I'm not adding Curve25519 version. Also pairing authentication needs move work to enable Curve25519 (see crbug.com/593123), so it's going to be done separately. Spake2Authenticor implements SPAKE2 over Curve25519, which is defined in https://tools.ietf.org/html/draft-irtf-cfrg-spake2-03 . BUG=589698 Review URL: https://codereview.chromium.org/1780403002 Cr-Commit-Position: refs/heads/master@{#380869}
* Move NegotiatingClientAuthentication creation to ChromotingClient.sergeyu2016-03-124-45/+47
| | | | | | | | | | | | | | | | | For the new SPAKE2 authenticator we need to pass client_jid to the authenticator. This wasn't possible previously because NegotiatingClientAuthenticator was created before signaling is connected. Moved NegotiatingClientAuthentication creation to ChromotingClient. BUG=589698 Committed: https://crrev.com/279bf7c76a3e0bfbbd28748d0d02fcead88f4436 Cr-Commit-Position: refs/heads/master@{#380779} Review URL: https://codereview.chromium.org/1778023002 Cr-Commit-Position: refs/heads/master@{#380831}
* Revert of Move NegotiatingClientAuthentication creation to ChromotingClient. ↵alexmos2016-03-113-43/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (patchset #2 id:40001 of https://codereview.chromium.org/1778023002/ ) Reason for revert: Appears to be breaking compile on Linux and Mac: https://build.chromium.org/p/chromium/builders/Linux/builds/72875 https://build.chromium.org/p/chromium/builders/Mac/builds/13060 Output: FAILED: /b/build/slave/Linux/build/src/build/goma/client/gomacc ../../third_party/llvm-build/Release+Asserts/bin/clang++ ... -c ../../remoting/test/chromoting_test_fixture.cc -o obj/remoting/test/chromoting_test_driver.chromoting_test_fixture.o In file included from ../../remoting/test/chromoting_test_fixture.cc:14: In file included from ../../remoting/test/test_chromoting_client.h:14: In file included from ../../remoting/client/chromoting_client.h:19: In file included from ../../remoting/protocol/negotiating_client_authenticator.h:15: In file included from ../../remoting/protocol/negotiating_authenticator_base.h:16: ../../third_party/webrtc/libjingle/xmllite/xmlelement.h:17:10: fatal error: 'webrtc/libjingle/xmllite/qname.h' file not found #include "webrtc/libjingle/xmllite/qname.h" ^ 1 error generated. ninja: build stopped: subcommand failed. Original issue's description: > Move NegotiatingClientAuthentication creation to ChromotingClient. > > For the new SPAKE2 authenticator we need to pass client_jid > to the authenticator. This wasn't possible previously because > NegotiatingClientAuthenticator was created before signaling is > connected. Moved NegotiatingClientAuthentication creation to > ChromotingClient. > > BUG=589698 > > Committed: https://crrev.com/279bf7c76a3e0bfbbd28748d0d02fcead88f4436 > Cr-Commit-Position: refs/heads/master@{#380779} TBR=jamiewalch@chromium.org,sergeyu@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=589698 Review URL: https://codereview.chromium.org/1788943002 Cr-Commit-Position: refs/heads/master@{#380785}
* Move NegotiatingClientAuthentication creation to ChromotingClient.sergeyu2016-03-113-44/+43
| | | | | | | | | | | | | | For the new SPAKE2 authenticator we need to pass client_jid to the authenticator. This wasn't possible previously because NegotiatingClientAuthenticator was created before signaling is connected. Moved NegotiatingClientAuthentication creation to ChromotingClient. BUG=589698 Review URL: https://codereview.chromium.org/1778023002 Cr-Commit-Position: refs/heads/master@{#380779}
* Implement authenticator based on SPAKE2 implementation in boringssl.sergeyu2016-03-115-0/+517
| | | | | | | | | | | | | | The new authenticator uses SPAKE2 over Curve25519. It will be enabled in host and client in a separate CL. BUG=589698 Committed: https://crrev.com/bf336334ba59ae7cd150e9cb36a9b248d174a4eb Cr-Commit-Position: refs/heads/master@{#379972} Review URL: https://codereview.chromium.org/1759313002 Cr-Commit-Position: refs/heads/master@{#380674}
* Revert of Implement authenticator based on SPAKE2 implementation in ↵chrishall2016-03-105-517/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | boringssl. (patchset #5 id:140001 of https://codereview.chromium.org/1759313002/ ) Reason for revert: This patch added 2 more tests which are now failing on Linux MSan Tests as of build #14316 https://build.chromium.org/p/chromium.memory.fyi/builders/Linux%20MSan%20Tests/builds/14316 Spake2AuthenticatorTest.InvalidSecret (run #1): [ RUN ] Spake2AuthenticatorTest.InvalidSecret ==10538==WARNING: MemorySanitizer: use-of-uninitialized-value #0 0x14bb0fd in ?? third_party/modp_b64/modp_b64.cc:91:20 #1 0xdcf2f6 in Base64Encode base/base64.cc:18:24 #2 0x12d6e6c in EncodeBinaryValueToXml remoting/protocol/spake2_authenticator.cc:47:3 and Spake2AuthenticatorTest.SuccessfulAuth (run #1): [ RUN ] Spake2AuthenticatorTest.SuccessfulAuth ==10537==WARNING: MemorySanitizer: use-of-uninitialized-value #0 0x14bb0fd in ?? third_party/modp_b64/modp_b64.cc:91:20 #1 0xdcf2f6 in Base64Encode base/base64.cc:18:24 #2 0x12d6e6c in EncodeBinaryValueToXml remoting/protocol/spake2_authenticator.cc:47:3 Original issue's description: > Implement authenticator based on SPAKE2 implementation in boringssl. > > The new authenticator uses SPAKE2 over Curve25519. It will > be enabled in host and client in a separate CL. > > BUG=589698 > > Committed: https://crrev.com/bf336334ba59ae7cd150e9cb36a9b248d174a4eb > Cr-Commit-Position: refs/heads/master@{#379972} TBR=kelvinp@chromium.org,davidben@chromium.org,arnarb@chromium.org,sergeyu@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=589698 Review URL: https://codereview.chromium.org/1778223003 Cr-Commit-Position: refs/heads/master@{#380365}
* Replace ThirdPartyClientAuthenticator::TokenFetcher with a callback.sergeyu2016-03-107-72/+58
| | | | | | | | | Callbacks are already used to fetch PIN. It doesn't make sense to use a different style to fetch third-party tokens. Review URL: https://codereview.chromium.org/1774113005 Cr-Commit-Position: refs/heads/master@{#380316}
* Cleanup AuthenticationMethod usage.sergeyu2016-03-0914-279/+168
| | | | | | | | | | | | | | | | | | Previously AuthenticationMethod type was used in many places. Removed all dependencies on it except from NegotiatingAuthenticator classes. AuthenticationMethod has been moved to NegotiatingAuthenticatorBase::Method. Also includes the following cleanups: - removed old authentication functions from auth_util.cc which were no longer needed. - Remove HashFunction enum as it wasn't useful. BUG=589698 Review URL: https://codereview.chromium.org/1768383004 Cr-Commit-Position: refs/heads/master@{#380078}
* Implement authenticator based on SPAKE2 implementation in boringssl.sergeyu2016-03-085-0/+517
| | | | | | | | | | | The new authenticator uses SPAKE2 over Curve25519. It will be enabled in host and client in a separate CL. BUG=589698 Review URL: https://codereview.chromium.org/1759313002 Cr-Commit-Position: refs/heads/master@{#379972}
* Remove dependency on V2Authenticator from ThirdParty and pairing authenticators.sergeyu2016-03-0816-195/+168
| | | | | | | | | | | | | | | | Previously third-party and pairing authenticators were responsible for creation for the underlying SPAKE2 authenticators. Now they get a callback that they should use instead. This will allow to use them with Spake2Authenticator. Also made some minor cleanups in pairiting authenticators, particularly in the code responsible for fetching the pin after paired auth fails. BUG=589698 Review URL: https://codereview.chromium.org/1770923002 Cr-Commit-Position: refs/heads/master@{#379858}
* Support for server session cache.ryanchung2016-03-082-8/+18
| | | | | | | | | | | | Allows SSL server sockets created through a SSLServerSocketContext to share a single session cache. OpenSSL only. BUG=568650 Review URL: https://codereview.chromium.org/1518613002 Cr-Commit-Position: refs/heads/master@{#379751}
* Remove first_message argument from AuthenticatorFactory::Create().sergeyu2016-03-078-22/+11
| | | | | | | | | The first_message parameter in AuthenticatorFactory::Create() is no longer used anywhere, so it can be removed. Review URL: https://codereview.chromium.org/1769603005 Cr-Commit-Position: refs/heads/master@{#379592}
* Notify normalizing input filters on blur.jamiewalch2016-03-042-5/+19
| | | | | | | | | | | | | | | Previously, ReleaseAllKeys was not notifying the normalizing input filter because that filter exists above InputEventTracker in the input pipeline. In fact, the input "pipeline" was more like a tree, with both the touch input scaler and the pepper input handler feeding into the input tracker. This CL establishes a linear pipeline, as described in the header comment. BUG=590404 Review URL: https://codereview.chromium.org/1760633003 Cr-Commit-Position: refs/heads/master@{#379398}
* Simplify AuthenticationMethod type and PIN hash handling.sergeyu2016-03-0413-331/+209
| | | | | | | | | | | | | | | | | | | | | 1 Previously AuthenticationMethod was a class. Replaced it with a simple enum. 2 Removed SharedSecretHash struct and simplified the code that passes around PIN hash. 3 If the host config contains PIN in plain text then now the host hashes it instead of using spake2_plain authenticator method. This means that spake2_hmac is always used for Me2Me even when the PIN is not hashed in the config. This change will make it easier to add new authentication methods. BUG=589698 Review URL: https://codereview.chromium.org/1755273003 Cr-Commit-Position: refs/heads/master@{#379365}
* Roll WebRTC 11835:11842kjellander2016-03-021-1/+1
| | | | | | | | | | | | | | | | | WebRTC 11835:11842 Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/f749695..5ec6244 This updates Chromium to match the changes done in https://codereview.webrtc.org/1750593002/ It allows removing a couple of ugly GYP hacks. BUG=webrtc:4256 TBR=sergeyu@chromium.org Review URL: https://codereview.chromium.org/1755993003 Cr-Commit-Position: refs/heads/master@{#378760}
* Refactor IceTransportChannel to use non-deprecated P2PTransportChannel ↵mikescarlett2016-03-011-1/+1
| | | | | | | | | | | | | | | constructor P2PTransportChannel no longer requires a P2PTransport* parameter due to: https://codereview.chromium.org/1691673002/. so IceTransportChannel should use the new constructor now. NOPRESUBMIT=true Review URL: https://codereview.chromium.org/1735523007 Cr-Commit-Position: refs/heads/master@{#378492}
* remoting: Add out-of-line copy ctors for complex classes.vmpstr2016-02-262-0/+3
| | | | | | | | | | | | | This patch adds out of line copy constructors for classes that our clang-plugin considers heavy. This is an effort to enable copy constructor checks by default. BUG=436357 R=garykac@chromium.org, dcheng@chromium.org, thakis@chromium.org Review URL: https://codereview.chromium.org/1736633002 Cr-Commit-Position: refs/heads/master@{#377988}
* Fix WebrtcVideoCapturerAdapter to emit frames even when nothing is changing.sergeyu2016-02-251-3/+8
| | | | | | | | | | | | | | | | Previously the adapter was signaling SignalVideoFrame only when the frame has changes. Video sender in Webrtc stack may drop some frames because VideoCapturer interface assumes that SignalVideoFrame is called regularly. As results the changes were sometimes delivered to the client too late. This change is a stopgap solution until we have proper frame capture scheduling that integrates with the video sender in webrtc. BUG=547158 Review URL: https://codereview.chromium.org/1740443002 Cr-Commit-Position: refs/heads/master@{#377459}
* Fix WebrtcDataStreamAdapter to handle errors asychronously.sergeyu2016-02-243-12/+63
| | | | | | | | | | Previously WebrtcDataStreamAdapter was handling errors synchronously and the host was crashing because the data channels were destroyed when they were on the stack. Review URL: https://codereview.chromium.org/1723423002 Cr-Commit-Position: refs/heads/master@{#377350}
* net: move IsLocalhost() function into url_util.htfarina2016-02-234-4/+0
| | | | | | | | | | | | | | This patch moves the last function in net_util.h into url_util.h. We are moving it there because it also operates on a URL part (host). BUG=488531 TEST=net_unittests --gtest_filter=UrlUtilTest.IsLocalhost R=eroman@chromium.org TBR=darin@chromium.org Review URL: https://codereview.chromium.org/1726693002 Cr-Commit-Position: refs/heads/master@{#377115}
* Implement HttpIceConfigRequestsergeyu2016-02-206-1/+479
| | | | | | | | | | | In the future HttpIceConfigRequest will be used to fetch STUN/TURN configuration instead of JingleInfoRequest. BUG=577954 Review URL: https://codereview.chromium.org/1707223002 Cr-Commit-Position: refs/heads/master@{#376600}
* Enable TURN support in remoting::PortAllocatorsergeyu2016-02-192-31/+40
| | | | | | | | | | | Previously PortAllocator was ignoring IceConfig.turn_server, now it passes the list of TURN servers to cricket::PortAllocator. BUG=577954 Review URL: https://codereview.chromium.org/1706103002 Cr-Commit-Position: refs/heads/master@{#376508}
* Add support for POST requests in UrlRequest interfacesergeyu2016-02-181-1/+2
| | | | | | | | | | | Previously UrlRequest supported only GET request. Now it also supports POST. BUG=577954 Review URL: https://codereview.chromium.org/1710723002 Cr-Commit-Position: refs/heads/master@{#376247}
* Add IceConfig and IceConfigRequest.sergeyu2016-02-139-84/+303
| | | | | | | | | | | | The new IceConfig struct will be used to pass TURN configuration to PortAllocator. JingleInfoRequest has been moved to remoting/protocol and now implements IceConfigRequest. BUG=577954 Review URL: https://codereview.chromium.org/1694613002 Cr-Commit-Position: refs/heads/master@{#375328}
* Update webrtc::DesktopCapturere clients to use SharedMemoryFactorysergeyu2016-02-123-14/+22
| | | | | | | | | webrtc::DesktopCapturer::Callback::CreateSharedMemory() has been deprecated and webrtc::SharedMemoryFactory should be used instead now. Review URL: https://codereview.chromium.org/1690853002 Cr-Commit-Position: refs/heads/master@{#375224}
* Roll WebRTC 11579:11592, Libjingle 11568:11592kjellander2016-02-121-1/+1
| | | | | | | | | | | | | | | | | | | Update libjingle GYP+GN files for: * talk/session/media -> webrtc/pc move. * webrtc/media/webrtc -> webrtc/media/engine rename. WebRTC 11579:11592 Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/4ba95f2..4056bb6 Libjingle 11568:11592 Changes: https://chromium.googlesource.com/external/webrtc/trunk/talk.git/+log/a60ab43..761433d BUG=webrtc:5419,webrtc:5420 TBR=tommi@chromium.org, sergeyu@chromium.org Review URL: https://codereview.chromium.org/1690203002 Cr-Commit-Position: refs/heads/master@{#375141}
* Use UrlRequest in PortAllocator.sergeyu2016-02-1113-283/+153
| | | | | | | | | | | | | | | PepperPortAllocator and ChromiumPortAllocator were different only in how they were sending HTTP requests to allocate relay session. They were using pp::URLLoader and net::URLFetcher respectively, but otherwise contained largely the same logic. Now URL request logic is abstracted with remoting::UrlRequest interface and so a single PortAllocator implementation can be used in all cases. BUG=577954 Review URL: https://codereview.chromium.org/1681393006 Cr-Commit-Position: refs/heads/master@{#375042}
* Roll WebRTC 11523:11548, Libjingle 11522:11545kjellander2016-02-106-12/+12
| | | | | | | | | | | | | | | | | | WebRTC 11523:11548 Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/4def420..608b2be Libjingle 11522:11545 Changes: https://chromium.googlesource.com/external/webrtc/trunk/talk.git/+log/01cbe5b..bb467ca Updated libjingle GYP+GN for talk/app/webrtc move that happened in https://codereview.webrtc.org/1610243002/ TBR=tommi@chromium.org, sergeyu@chromium.org BUG=webrtc:5418 Review URL: https://codereview.chromium.org/1615433002 Cr-Commit-Position: refs/heads/master@{#374655}
* Fix IceTransportTest.TestBrokenTransportsergeyu2016-02-093-22/+36
| | | | | | | | | | | | | | | | | | | | Previously the test wasn't verifying what it's supposed to verify and was flaky, particularly on DrMemory bots. The test attemps to connect two IceTransport instances with settings that should prevent them from connecting and verifies that they fail to connect. There were two issues: 1. NetworkSettings were not passed to IceTransport objects properly, so they could still connect. 2. The test was just calling RunLoopUntilIdle() and wasn't waiting for transport failure, so in most cases the test would pass even if the transports could connect, which masked (1). Fixed both issues. BUG=585531 Review URL: https://codereview.chromium.org/1687543002 Cr-Commit-Position: refs/heads/master@{#374536}
* Avoid extra framing in WebRTC data streams.sergeyu2016-02-094-173/+133
| | | | | | | | | | | | | | | | | Previously WebRTC transport was using StreamMessagePipeAdapter to provide MessagePipe implementation for the connection. StreamMessagePipeAdapter is responsible for message framing in the old protocol, but is not necessary for webrtc data streams. Now WebrtcDataStreamAdapter::Channel implements MessagePipe interface instead of P2PStreamSocket. StreamMessagePipeAdapter is no longer used for WebRTC-based protocol and all messages are sent without the length prefix. BUG=547158 Review URL: https://codereview.chromium.org/1664883003 Cr-Commit-Position: refs/heads/master@{#374467}
* Add policy to restrict client domain for Me2Me.jamiewalch2016-02-086-55/+159
| | | | | | | | BUG=b/19316131 Review URL: https://codereview.chromium.org/1643793002 Cr-Commit-Position: refs/heads/master@{#374209}
* Roll WebRTC 11486:11495, Libjingle 11485:11495kjellander2016-02-054-4/+4
| | | | | | | | | | | | | | | | | | Includes changes for moving talk/media to webrtc/media which took place in https://codereview.webrtc.org/1587193006/ WebRTC 11486:11495 Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/052f590..e1bbb30 Libjingle 11485:11495 Changes: https://chromium.googlesource.com/external/webrtc/trunk/talk.git/+log/d70be77..abc975d TBR=perkj@chromium.org,sergeyu@chromium.org BUG=webrtc:5420 Review URL: https://codereview.chromium.org/1669023004 Cr-Commit-Position: refs/heads/master@{#373795}
generated by cgit v1.1 at 2024-05-20 23:33:18 +0000