- 24 Jan, 2019 40 commits
-
-
Mario Sanchez Prada authored
Use IdentityManager's API instead and only use the AccountTrackerService instance to construct instances of classes still depending on it. Bug: 922777 Change-Id: Ib21cb594a208b7ee5f15f3777f61dda570a11113 Reviewed-on: https://chromium-review.googlesource.com/c/1433617Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Cr-Commit-Position: refs/heads/master@{#625647}
-
Kush Sinha authored
|AccountManagerMigrator| migrates accounts to the new Chrome OS Account Manager. During this migration it asserts that the Primary (Device) Account is a valid (Google or Active Directory) account. This assertion is not triggered for ephemeral sessions (Guest / Incognito etc.). Unfortunately a lot of browser tests on Chrome OS run with an invalid (AccountType::UNKNOWN) account type. Fix this by removing the assertion. Bug: 915628 Change-Id: Ie6e73e7128ec57ac9b5239b534d185c4758a17e7 Reviewed-on: https://chromium-review.googlesource.com/c/1433623Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Commit-Queue: Kush Sinha <sinhak@chromium.org> Cr-Commit-Position: refs/heads/master@{#625646}
-
Bence Béky authored
Make WebSocketStreamRequestImpl keep a WeakPtr of WebSocketHandshakeStreamBase subclasses instead of a raw pointer to avoid use-after-free. While weak pointers are generally avoided in //net, this is so much cleaner and less error-prone than making WebSocketStreamRequestImpl and WebSocket{Basic,Http2}HandshakeStream notify each other of destruction, see https://crrev.com/c/1391298. Make WebSocketHandshakeStreamBase::GetWeakPtr() pure virtual and implement it in subclasses to avoid valid weak pointers pointing to partially destroyed instances. Bug: 850183 Change-Id: Ib1995b076e0fb57bd8a7fbec392fa3283b52120c Reviewed-on: https://chromium-review.googlesource.com/c/1431132Reviewed-by:
Adam Rice <ricea@chromium.org> Commit-Queue: Bence Béky <bnc@chromium.org> Cr-Commit-Position: refs/heads/master@{#625645}
-
Matt Falkenhagen authored
Follow-up to r625066 to address post-commit review comments at https://chromium-review.googlesource.com/c/chromium/src/+/1420508 Bug: 901082 Change-Id: Id222087320ade9a10f4ffc67d4769e2610b8df75 Reviewed-on: https://chromium-review.googlesource.com/c/1433896Reviewed-by:
Josh Karlin <jkarlin@chromium.org> Commit-Queue: Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#625644}
-
chrome://invalidationsTanja Gornak authored
Before this patch, the debug information available on chrome://invalidations didn't contain the instance_id token. Displaying the token there helps to debug several issues with subscription requests which are currently observed in dev/canary experiment. Bug: 801985 Change-Id: I1ade4bc28ee117ce77dc0167b9219a6aed14817b Reviewed-on: https://chromium-review.googlesource.com/c/1412024Reviewed-by:
Tim Schumann <tschumann@chromium.org> Commit-Queue: Tatiana Gornak <melandory@chromium.org> Cr-Commit-Position: refs/heads/master@{#625643}
-
Finnur Thorarinsson authored
When the property list doesn't include one of 'email', or 'tel', we should not show that data in the dialog. Also made sure to remove blank entries. NOTRY=true TBR: twellington Bug: 860467 Change-Id: Idb488f61168e4036d7acb0f1f144d10caf70510a Reviewed-on: https://chromium-review.googlesource.com/c/1433759 Commit-Queue: Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#625642}
-
Anna Malova authored
Bug: 924118 Change-Id: I19611361435e142e06440faa1a241c1a45dd76e5 Reviewed-on: https://chromium-review.googlesource.com/c/1433244Reviewed-by:
Michael van Ouwerkerk <mvanouwerkerk@chromium.org> Commit-Queue: Anna Malova <amalova@chromium.org> Cr-Commit-Position: refs/heads/master@{#625641}
-
Rayan Kanso authored
Remove check in renderer and add WP test for uploads. Bug: 871174 Change-Id: Id225cc05b6f3c065962e1481c98562ca96d87322 Reviewed-on: https://chromium-review.googlesource.com/c/1424821 Commit-Queue: Rayan Kanso <rayankans@chromium.org> Reviewed-by:
Mugdha Lakhani <nator@google.com> Reviewed-by:
Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#625640}
-
Tim Volodine authored
Ensure that headers and status line can be modified by the loader's delegate. Ensure that "Client-Via: shouldInterceptRequest" header is always present. BUG=922074,841556 Cq-Include-Trybots: master.tryserver.chromium.android:android_mojo Change-Id: I54f5cd4035bb3b56fd1364f0778c15f9eda2df3b Reviewed-on: https://chromium-review.googlesource.com/c/1425721Reviewed-by:
Richard Coles <torne@chromium.org> Commit-Queue: Tim Volodine <timvolodine@chromium.org> Cr-Commit-Position: refs/heads/master@{#625639}
-
Jan Krcal authored
This CL fixes an omission in the Wallet data bridge so that it actually reports DB errors that happen when writing sync metadata to the web data DB. This is an improvement on its own but it may also help to understand recent traffic issues. The CL omits failures from data DB writes as the current AutofillTable API does not support reporting them. Bug: 924447 Change-Id: I2136aafa5f452b23268afb2ac52c16f63769b3ad Reviewed-on: https://chromium-review.googlesource.com/c/1429641 Commit-Queue: Jan Krcal <jkrcal@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#625638}
-
Vadym Doroshenko authored
In CL https://chromium-review.googlesource.com/c/chromium/src/+/1384258 the generation in the new architecture was implemented. 2 things were missing: 1.Marking generation element for rater extension was missing in this implementation. This CL fixes that. 2. Setting aria-aucomplete attribute (it looks that it's important for screen readers). This CL fixes both of them. Bug: 866444 Change-Id: I49b459d390ee36540ed1b9b2d118930e99f86e6d Reviewed-on: https://chromium-review.googlesource.com/c/1432855Reviewed-by:
Dominic Battré <battre@chromium.org> Commit-Queue: Vadym Doroshenko <dvadym@chromium.org> Cr-Commit-Position: refs/heads/master@{#625637}
-
Oskar Sundbom authored
Bug: 924954 Tbr: ekaramad@chromium.org Change-Id: I6e85f34064d2f251dd38731cb37d7c512014d9c2 Reviewed-on: https://chromium-review.googlesource.com/c/1433619Reviewed-by:
Oskar Sundbom <ossu@chromium.org> Commit-Queue: Oskar Sundbom <ossu@chromium.org> Cr-Commit-Position: refs/heads/master@{#625636}
-
Daniel Bratell authored
Attempts to use ui/gfx code in content failed because of variable/enum shadowing. The main problem is fixed in https://chromium-review.googlesource.com/c/chromium/src/+/1424799 This fixes the remaining problems and enables the shadowing warning to prevent future problems of the same kind. All the problems are of the trivial kind, with inner blocks reusing either the name of a variable or the name of an argument, be it |cluster|, |font|, |i|, |path|, |alpha| or |line|. In one case the fix was to delete an unused local variable. Bug: 923078,794619 Change-Id: Iae8bb768c49332cbe610860e06fea9d7aebc7a4b Reviewed-on: https://chromium-review.googlesource.com/c/1421104 Commit-Queue: Daniel Bratell <bratell@opera.com> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#625635}
-
jonross authored
We would like to get coverage of browser_tests and content_browser tests when using --enable-features=VizDisplayCompositor,UseSkiaRenderer This adds those configurations to the FYI bot Linux Viz Test=skia_renderer_browser_tests, skia_renderer_content_browsertests Bug: 914491 Change-Id: I523fbace668477686e4279a3d2df4a7254caccee Reviewed-on: https://chromium-review.googlesource.com/c/1432094Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Jonathan Ross <jonross@chromium.org> Cr-Commit-Position: refs/heads/master@{#625634}
-
Rayan Kanso authored
Update the upload data pipeline by adding a specific DatabaseTask. The previous methodology asssumed unique urls. Bug: 871174 Change-Id: Ib078ca84fb29e2de61422371fee859e74be31ab8 Reviewed-on: https://chromium-review.googlesource.com/c/1422017 Commit-Queue: Rayan Kanso <rayankans@chromium.org> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Mugdha Lakhani <nator@google.com> Cr-Commit-Position: refs/heads/master@{#625633}
-
Armando Miraglia authored
Two constraints have been missing for a long time in applyConstraints in that the constraint per-se was never applied. This was caused by a check in media_stream_track.cc that would verify whether the set of constraints was empty. The constraints are also overlapping with goog* constraints, hence we needed renaming of the properties, so that the signaling to the information to the application level is correct. This change now adds testing for OverconstrainedError for these constraints, as well as fixing the check. BUG=923828 Change-Id: I26a72d8aef4f749f7309092787f7929de2239978 Reviewed-on: https://chromium-review.googlesource.com/c/1426679Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Commit-Queue: Armando Miraglia <armax@chromium.org> Cr-Commit-Position: refs/heads/master@{#625632}
-
Rayan Kanso authored
Migrate cache entries using the old format to the new one on startup. This allows for caches to store duplicate requests. Bug: 871174 Change-Id: Ie4f4fef0bd2e928680b8a9f5ff34dfd3a744db43 Reviewed-on: https://chromium-review.googlesource.com/c/1417870 Commit-Queue: Rayan Kanso <rayankans@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Mugdha Lakhani <nator@google.com> Cr-Commit-Position: refs/heads/master@{#625631}
-
Sam Maier authored
Since we are calling .get() right away anyways, it's better to just do this inline. Also, it appears that at least in some of the ways to arrive at this code, we are already running on a background thread. Bug: 878529 Change-Id: Ic9612edf5e24f09ecd22c68d16513c122d0579a6 Reviewed-on: https://chromium-review.googlesource.com/c/1427577 Commit-Queue: Sam Maier <smaier@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Auto-Submit: Sam Maier <smaier@chromium.org> Cr-Commit-Position: refs/heads/master@{#625630}
-
Thomas Tangl authored
- The 'Drive search suggestions' toggle is moved to the settings-personalization-options element. This means it is under the 'Other Google services' section. - A test setup for non-offical builds is added to the settings_personalization_options tests. - The frequently failing test 'displaying toggles depending on unified consent' is removed from the official tests. Reason: This test is not part of the CQ tests and oftentimes forgotten. Since there is no big value in keeping this test, it is removed. Bug: 916191 Change-Id: I1ab1c876f1cedcaf22c167c3a2d0b238223e42b5 Reviewed-on: https://chromium-review.googlesource.com/c/1425607 Commit-Queue: Thomas Tangl <tangltom@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#625629}
-
Sylvain Defresne authored
The class base::NoDestructor is a simpler way to define a static global value (aka a singleton) than base::{LazyInstance,Singleton} and is the recommended pattern to use. The changes were automated using a custom one-shot script, then fine tuned by hand (to remove unnecessary classes, ...). This fixes usages in src/ios/chrome/browser/suggestions. This CL was uploaded by git cl split. Bug: none Change-Id: I14071e3eb036ba351a968d6fec578b9458a52eaa Reviewed-on: https://chromium-review.googlesource.com/c/1425711 Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Commit-Queue: Mathieu Perreault <mathp@chromium.org> Reviewed-by:
Mathieu Perreault <mathp@chromium.org> Cr-Commit-Position: refs/heads/master@{#625628}
-
Mikel Astiz authored
The remaining callers (unittests) are migrated to access the object by other means. TBR=eugenebut@chromium.org Bug: 922971 Change-Id: Ibb2fc4eeabfb6ac267325e1badb20c5522c89d7b Reviewed-on: https://chromium-review.googlesource.com/c/1433235 Commit-Queue: Mikel Astiz <mastiz@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#625627}
-
Kent Tamura authored
If a validation bubble is shown on an element in a remote frame, we adjusted its position with the main frame viewport, and rendered the bubble at a position at which the remote frame can't render. In a remote frame, we should refer to 'remote viewport intersection', which is a visible rectangle of a remote frame. Bug: 910979 Change-Id: I1497f6980c6e5d3758689b4b7fd8ac89113f8540 Reviewed-on: https://chromium-review.googlesource.com/c/1433638 Commit-Queue: Kent Tamura <tkent@chromium.org> Reviewed-by:
Keishi Hattori <keishi@chromium.org> Cr-Commit-Position: refs/heads/master@{#625626}
-
Mike West authored
`URLRequest::GetUserAgent` no longer exists, nor does `URLRequest` depend upon `HTTPUserAgentSettings`. Change-Id: I4d535ddf4de7759287c1d252a1ea69364ed531f2 Reviewed-on: https://chromium-review.googlesource.com/c/1433756 Auto-Submit: Mike West <mkwst@chromium.org> Commit-Queue: Matt Menke <mmenke@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#625625}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/ed17109c24b1..e402437aad51 git log ed17109c24b1..e402437aad51 --date=short --no-merges --format='%ad %ae %s' 2019-01-24 rsavitski@google.com Merge "remove duplicate sentence in comment." Created with: gclient setdep -r src/third_party/perfetto@e402437aad51 The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=perfetto-bugs@google.com Change-Id: I662fc387f08a8b878a400ba4196d5318154d9f0c Reviewed-on: https://chromium-review.googlesource.com/c/1433835Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#625624}
-
Gabriel Charette authored
There are existing use cases for it and more are surfacing. It was also discovered that all previous manual implementations of this were unintentionally leaking state as deleting the TLS::Slot resulted in unregistering the deletion hook and thus leaking pointers stored on the main thread (e.g. between tests in the same process). On top of managing ownership automatically, ThreadLocalOwnedPointer now brings type safety (no longer need to reinterpet_cast) :) Extracted from https://chromium-review.googlesource.com/c/chromium/src/+/1379027 R=fdoray@chromium.org Bug: 707362, 851163 Change-Id: I4fd383524d4ffe7af01d9aaa292f8953a16c555b Reviewed-on: https://chromium-review.googlesource.com/c/1417115 Commit-Queue: Gabriel Charette <gab@chromium.org> Reviewed-by:
Eric Orth <ericorth@chromium.org> Reviewed-by:
Robert Liao <robliao@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#625623}
-
Tanja Gornak authored
The subscription request is retried on failure. While investigating on the server side different reasons which can result in http erros, it became clear that the subscription request shouldn't be repeated for some of the http response codes. Bug: 801985 Change-Id: I35de80f202e55a3af68e3014b5f714ce5c6d0712 Reviewed-on: https://chromium-review.googlesource.com/c/1424818 Commit-Queue: Tatiana Gornak <melandory@chromium.org> Reviewed-by:
Jan Krcal <jkrcal@chromium.org> Cr-Commit-Position: refs/heads/master@{#625622}
-
Sergio Villar Senin authored
Migrated to IdentityManager APIs which do the same thing. Bug: 922749 Change-Id: I75dcae154936211003e65f79d5a07d5698ff6e6d Reviewed-on: https://chromium-review.googlesource.com/c/1429983Reviewed-by:
Jacob Dufault <jdufault@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Commit-Queue: Sergio Villar <svillar@igalia.com> Cr-Commit-Position: refs/heads/master@{#625621}
-
Marc Treib authored
Some things mentioned there had become outdated or obsolete. Bug: 884159 Change-Id: I53d1e552e5814cae5a58d5b16c3cd51b73676318 Reviewed-on: https://chromium-review.googlesource.com/c/1433238Reviewed-by:
Mohamed Amir Yosef <mamir@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#625620}
-
Michael Hablich authored
This is a roll-back to V8 7.3.492.1. This is the most stable candidate out of the latest Canaries. If this commit results in failures in Blink please contact the Blink sheriff. They may need to make changes to Blink because of the roll-back. When in doubt please try to contact the committer and reviewers of this CL before reverting it. TBR=machenbach@chromium.org Change-Id: Ied0563b0cb78cb19bf483f189539492345b71d18 Reviewed-on: https://chromium-review.googlesource.com/c/1433240Reviewed-by:
Michael Hablich <hablich@chromium.org> Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Commit-Queue: Michael Hablich <hablich@chromium.org> Cr-Commit-Position: refs/heads/master@{#625619}
-
Oskar Sundbom authored
It's failing on bots and looks like it slipped through review unintentionally. Bug: 924930 Tbr: ccameron@chromium.org, tapted@chromium.org Change-Id: I554b2a33df7fa42f3e9770c08bb24e5eff3a79d1 Reviewed-on: https://chromium-review.googlesource.com/c/1433760 Commit-Queue: Oskar Sundbom <ossu@chromium.org> Reviewed-by:
Oskar Sundbom <ossu@chromium.org> Cr-Commit-Position: refs/heads/master@{#625618}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 880eaa9a. With Chromium commits locally applied on WPT: bcd3934d "Add a test for ServiceWorker and AppCache interaction with claim()" 8e4e5d72 "Fix preload tests failing on wpt.fyi dashboard" f643497e "[resource-timing] Modify is_secure_context_ based on resource" 20defaa4 "[LayoutNG] ComputeReplacedSize min/max width/height fix" 05cdb4d1 "Add tentative WPT tests for stale while revalidate handling." 875e732f "Revert "Add tentative WPT tests for stale while revalidate handling."" Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: guidou@chromium.org: external/wpt/mediacapture-streams NOAUTOREVERT=true TBR=foolip No-Export: true Change-Id: Ifa08abc467b4eeda8810574889d80b07ca38be11 Reviewed-on: https://chromium-review.googlesource.com/c/1433579Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#625617}
-
Finnur Thorarinsson authored
Bug: 860467 Change-Id: I0a316b47f5ca9d4d7c54d7dd4d99292fe814cdf5 Reviewed-on: https://chromium-review.googlesource.com/c/1430013 Commit-Queue: Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#625616}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/4f593680baf5..ed17109c24b1 git log 4f593680baf5..ed17109c24b1 --date=short --no-merges --format='%ad %ae %s' 2019-01-24 fmayer@google.com Merge "profiling: Forward failed unwindings to bookkeeping." 2019-01-24 hjd@google.com Merge "perfetto-ui: Make fonts work offline" Created with: gclient setdep -r src/third_party/perfetto@ed17109c24b1 The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=perfetto-bugs@google.com Change-Id: I15d213cf42ba8795edf57a37a0bba7a35c82583e Reviewed-on: https://chromium-review.googlesource.com/c/1433834Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#625615}
-
Alli Murray authored
Drive label, e.g. "SanDisk 3.0", is required to appear as the label of the root entry for a partitioned external media device in the directory tree. Bug: 918795 Change-Id: I734a9995bd7ebbc249702b5a11fd5f163332f9dd Reviewed-on: https://chromium-review.googlesource.com/c/1426238 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Reviewed-by:
Anand Mistry <amistry@chromium.org> Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#625614}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/84a9da3eb94c..2d2052fde8d4 Created with: gclient setdep -r src-internal@2d2052fde8d4 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=mmoss@chromium.org Change-Id: I5147cd6760b9fbde4e0faf56cce930fa522cd129 Reviewed-on: https://chromium-review.googlesource.com/c/1433737Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#625613}
-
Fergal Daly authored
Now selection, first-line and first-letter also work. I think this covers everything in https://www.w3.org/TR/css-pseudo-4/ that is currently supported by blink. Add WPT tests. Rename test file since it now covers pseudo elements in general. Bug: 805271 Change-Id: I16d1df2d18008ee5e080dda7dcebf4d584a4bf3c Reviewed-on: https://chromium-review.googlesource.com/c/1429779Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Fergal Daly <fergal@chromium.org> Cr-Commit-Position: refs/heads/master@{#625612}
-
Jonas Olsson authored
This is a reland of e902e08d Original change's description: > Switch to new ICE state implementation > > With this change the ice/dtls connection states should finally be compliant to the spec. > > Bug: webrtc:6145 > Change-Id: I1e7c13f22e40b534e732ecd3fc103f617306f00a > Reviewed-on: https://chromium-review.googlesource.com/c/1371392 > Commit-Queue: Jonas Olsson <jonasolsson@chromium.org> > Reviewed-by: Henrik Boström <hbos@chromium.org> > Cr-Commit-Position: refs/heads/master@{#617476} Bug: webrtc:6145 Change-Id: Ie4a09594e8a59c6598a9434c0132c1fd3543c76b Reviewed-on: https://chromium-review.googlesource.com/c/1433237Reviewed-by:
Henrik Boström <hbos@chromium.org> Commit-Queue: Jonas Olsson <jonasolsson@chromium.org> Cr-Commit-Position: refs/heads/master@{#625611}
-
Denis Kuznetsov authored
Bug: 916991 Change-Id: I4151f65520513657208d89f7bded4527a810c6bb Reviewed-on: https://chromium-review.googlesource.com/c/1427901Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Reviewed-by:
Sergey Poromov <poromov@chromium.org> Commit-Queue: Denis Kuznetsov <antrim@chromium.org> Cr-Commit-Position: refs/heads/master@{#625610}
-
Victor Vasiliev authored
R=rch@chromium.org Change-Id: Ice7bfcd52126101d273d8dc3adf455d25cb84830 Reviewed-on: https://chromium-review.googlesource.com/c/1423417Reviewed-by:
Misha Efimov <mef@chromium.org> Reviewed-by:
Ryan Hamilton <rch@chromium.org> Commit-Queue: Victor Vasiliev <vasilvv@chromium.org> Cr-Commit-Position: refs/heads/master@{#625609}
-
Oskar Sundbom authored
It's flaky. Bug: 924918 Tbr: slangley@chromium.org Change-Id: If65b35130b4ae4f6ca043720e3fed3870018dcb6 Reviewed-on: https://chromium-review.googlesource.com/c/1433758 Commit-Queue: Oskar Sundbom <ossu@chromium.org> Reviewed-by:
Oskar Sundbom <ossu@chromium.org> Cr-Commit-Position: refs/heads/master@{#625608}
-