M139 -> M138 #35

sf-jed-kyung · 2025-08-25T03:52:05Z

Update to M139

This CL enables use_siso=true by default. Googlers builds will switch to Siso, or get suggestion to run `gn clean` to switch. External developers will keep using Ninja for now. But, they will also use Siso soon. Bug: chromium:412968361 Change-Id: I064ea6147135c76a1d1eb7b8ee733b5d9b04f1dd Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393680 Reviewed-by: Mirko Bonadei <[email protected]> Reviewed-by: Christoffer Dewerin <[email protected]> Commit-Queue: Christoffer Dewerin <[email protected]> Auto-Submit: Junji Watanabe <[email protected]> Cr-Commit-Position: refs/heads/main@{#44765}

Bug: webrtc:407433458 Change-Id: I4fcc65c3884f5f9f4d196f4af2d28a3e629f7dc7 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393562 Commit-Queue: Danil Chapovalov <[email protected]> Reviewed-by: Ilya Nikolaevskiy <[email protected]> Cr-Commit-Position: refs/heads/main@{#44766}

Functionality moved to rtc_base/ cpu_info::Supports(). Bug: webrtc:42228262 Change-Id: I0ef2fc645447779aac2339580884ea042bdd315b No-Iwyu: Avoid unrelated changes Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393561 Commit-Queue: Tomas Gunnarsson <[email protected]> Reviewed-by: Tomas Gunnarsson <[email protected]> Auto-Submit: Fredrik Solenberg <[email protected]> Cr-Commit-Position: refs/heads/main@{#44768}

Bug: webrtc:407433458 Change-Id: Ifceefa4c5c116dfcc04c636cff50c7d2f0aba869 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393520 Reviewed-by: Björn Terelius <[email protected]> Commit-Queue: Danil Chapovalov <[email protected]> Cr-Commit-Position: refs/heads/main@{#44769}

All known users were migrated to FieldTrialsView directly. Bug: webrtc:42220378 Change-Id: I34c8da90bc3881d9c0fb5e4540556e5d4b160202 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393740 Auto-Submit: Danil Chapovalov <[email protected]> Reviewed-by: Per Kjellander <[email protected]> Commit-Queue: Per Kjellander <[email protected]> Cr-Commit-Position: refs/heads/main@{#44771}

Suggest to use it as the primary source for field trials in tests. When used widely, it would allow to remove quering force_field trial in test main functions, and more importnately, would avoid using global field trial string to propagate field trials from the command line. Bug: webrtc:419453427 Change-Id: I016098522966340d99857de8530fc6d6148e5479 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393220 Commit-Queue: Danil Chapovalov <[email protected]> Reviewed-by: Mirko Bonadei <[email protected]> Cr-Commit-Position: refs/heads/main@{#44772}

Bug: webrtc:42223878 Change-Id: I4efe846e2077fa6d28481103ca95adb084c05064 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393780 Commit-Queue: Björn Terelius <[email protected]> Reviewed-by: Mirko Bonadei <[email protected]> Cr-Commit-Position: refs/heads/main@{#44773}

Bug: webrtc:407433458 Change-Id: If9feced4111ffbf2137912997d567c6e854006f8 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393700 Reviewed-by: Alexander Cooper <[email protected]> Auto-Submit: Danil Chapovalov <[email protected]> Commit-Queue: Alexander Cooper <[email protected]> Cr-Commit-Position: refs/heads/main@{#44774}

Bug: webrtc:42223878 Change-Id: I4d662da8b3269dfdeb272f15fa2e5b1a65b2b444 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393781 Commit-Queue: Björn Terelius <[email protected]> Reviewed-by: Mirko Bonadei <[email protected]> Cr-Commit-Position: refs/heads/main@{#44775}

using find modules/desktop_capture/ -name "*.h" -o -name "*.cc" | grep -v win | grep -v wayland | grep -v mac | grep -v fuchsia | xargs tools_webrtc/iwyu/apply-include-cleaner followed by tools_webrtc/gn_check_autofix.py -C out/Default/ and git cl format Manual changes (mostly replacing C style headers with C++ ones) were done in PS3. PS5+PS6 brought back some platform-specific includes. BUG=webrtc:42226242 Change-Id: I4b84fa6abc320de0c734a43efa84c6f7e434c9af Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/382140 Reviewed-by: Danil Chapovalov <[email protected]> Commit-Queue: Philipp Hancke <[email protected]> Reviewed-by: Alexander Cooper <[email protected]> Cr-Commit-Position: refs/heads/main@{#44778}

Unify other usages of FieldTrials in tests to CreateTestFieldTrials Bug: webrtc:419453427 Change-Id: I535e674ddff7fe60ae032fef6925e07d41187db4 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393860 Reviewed-by: Rasmus Brandt <[email protected]> Commit-Queue: Danil Chapovalov <[email protected]> Cr-Commit-Position: refs/heads/main@{#44779}

This reverts commit 0d98c5f. Reason for revert: The CL disabled remote execution on non-corp builds unexpectedly. Bug: chromium:412968361 Original change's description: > Set use_siso=true by default in .gn > > This CL enables use_siso=true by default. > Googlers builds will switch to Siso, or get suggestion to run > `gn clean` to switch. > External developers will keep using Ninja for now. But, they will > also use Siso soon. > > Bug: chromium:412968361 > Change-Id: I064ea6147135c76a1d1eb7b8ee733b5d9b04f1dd > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393680 > Reviewed-by: Mirko Bonadei <[email protected]> > Reviewed-by: Christoffer Dewerin <[email protected]> > Commit-Queue: Christoffer Dewerin <[email protected]> > Auto-Submit: Junji Watanabe <[email protected]> > Cr-Commit-Position: refs/heads/main@{#44765} Bug: chromium:412968361 No-Presubmit: true No-Tree-Checks: true No-Try: true Change-Id: I5eb173a28e8418c27372c2b0e14230ac3d3a5e49 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393880 Reviewed-by: Mirko Bonadei <[email protected]> Reviewed-by: Christoffer Dewerin <[email protected]> Bot-Commit: [email protected] <[email protected]> Commit-Queue: Junji Watanabe <[email protected]> Cr-Commit-Position: refs/heads/main@{#44781}

Bug: webrtc:419453427 Change-Id: I7121bdf5b298bde6722084f46aab8d235c5a7262 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393800 Reviewed-by: Rasmus Brandt <[email protected]> Commit-Queue: Danil Chapovalov <[email protected]> Cr-Commit-Position: refs/heads/main@{#44782}

Fold GetCPUFeaturesARM() functionality into webrtc::cpu_info::Supports(), since it was only used to test for Neon support. Bug: webrtc:42228262 Change-Id: If72a90e4d4edd695256cb36f8e074e7c15f43f75 No-iwyu: Avoid unrelated changes Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393783 Auto-Submit: Fredrik Solenberg <[email protected]> Reviewed-by: Tomas Gunnarsson <[email protected]> Commit-Queue: Tomas Gunnarsson <[email protected]> Cr-Commit-Position: refs/heads/main@{#44783}

This reverts commit 788e02e. Reason for revert: breaks compilation downstream: desktop_capturer_differ_wrapper.cc -> desktop_capture_metadata.h" include should be conditional. Original change's description: > IWYU modules/desktop_capture/ > > using > find modules/desktop_capture/ -name "*.h" -o -name "*.cc" | grep -v win | grep -v wayland | grep -v mac | grep -v fuchsia | xargs tools_webrtc/iwyu/apply-include-cleaner > followed by > tools_webrtc/gn_check_autofix.py -C out/Default/ > and git cl format > > Manual changes (mostly replacing C style headers with C++ ones) were done in PS3. > PS5+PS6 brought back some platform-specific includes. > > BUG=webrtc:42226242 > > Change-Id: I4b84fa6abc320de0c734a43efa84c6f7e434c9af > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/382140 > Reviewed-by: Danil Chapovalov <[email protected]> > Commit-Queue: Philipp Hancke <[email protected]> > Reviewed-by: Alexander Cooper <[email protected]> > Cr-Commit-Position: refs/heads/main@{#44778} Bug: webrtc:42226242, b/420792585 No-Presubmit: true No-Tree-Checks: true No-Try: true Change-Id: I8cac586eda8fdaf9e20a6322383dd306887e8fa2 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393921 Commit-Queue: Danil Chapovalov <[email protected]> Owners-Override: Danil Chapovalov <[email protected]> Bot-Commit: [email protected] <[email protected]> Reviewed-by: Johannes Kron <[email protected]> Cr-Commit-Position: refs/heads/main@{#44784}

Bug: webrtc:419453427 Change-Id: I9f46ed080db08680c3514e751bf30d7b1616aa6d Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393782 Commit-Queue: Danil Chapovalov <[email protected]> Reviewed-by: Åsa Persson <[email protected]> Cr-Commit-Position: refs/heads/main@{#44785}

Bug: webrtc:407433458 Change-Id: Iab43e1a8014a55c9d8258d22500e49e73c28cbf9 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393922 Reviewed-by: Jakob Ivarsson‎ <[email protected]> Commit-Queue: Danil Chapovalov <[email protected]> Cr-Commit-Position: refs/heads/main@{#44786}

The new method differs from the existing method with the same name in two respects: * It reads into an InterleavedView<> instead of pointer. * It either reads exactly the requested number of samples or fails After examining the behavior in NetEq, where this method is called from, it seems that what we need is a method that reads exactly the required number of samples, not partially. A follow-up change to NetEqImpl and SyncBuffer will change SyncBuffer::GetNextAudioInterleaved() to read the requested buffer size or fail, as well as update NetEqImpl::GetAudioInternal accordingly. Bug: chromium:335805780 Change-Id: Ia78f888d52c9acb4b53b7042b31b8b0c53199e7b Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/392143 Reviewed-by: Per Åhgren <[email protected]> Reviewed-by: Jakob Ivarsson‎ <[email protected]> Commit-Queue: Tomas Gunnarsson <[email protected]> Cr-Commit-Position: refs/heads/main@{#44787}

which allows DTLS 1.3 PQC handshakes with ~1641 bytes to be fragmented into only two fragments instead of three. Increase the test network bitrate limitation a bit to avoid failures. Bug: webrtc:367395350 Change-Id: Ib1a231524e96aa859a9cb52ab867345f94cd923e Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393821 Reviewed-by: Sameer Vijaykar <[email protected]> Reviewed-by: Jonas Oreland <[email protected]> Commit-Queue: Philipp Hancke <[email protected]> Cr-Commit-Position: refs/heads/main@{#44788}

Change log: https://chromium.googlesource.com/chromium/src/+log/ba73e47453..547fced285 Full diff: https://chromium.googlesource.com/chromium/src/+/ba73e47453..547fced285 Changed dependencies * src/build: https://chromium.googlesource.com/chromium/src/build/+log/038dd242ba..2b7d244908 * src/ios: https://chromium.googlesource.com/chromium/src/ios/+log/3aa1c5d760..b07107e00f * src/testing: https://chromium.googlesource.com/chromium/src/testing/+log/8c277a0707..d45bba6967 * src/third_party: https://chromium.googlesource.com/chromium/src/third_party/+log/cd0c5ea0ee..67d6ed322c * src/third_party/android_deps/autorolled/cipd: p_9xkMekw1ax7bzdkzI9gjCXkONcxJb-xhJDLgyxxlUC..Tnd7aCKH9teskxCrm7pidQ2oKh9i8ghv6BU5_3GNGSwC * src/third_party/androidx/cipd: 4iWSQnDJqcLNK274cq1Z7mu50FgwKfBxZhbjbtnga2sC..TEqLvkwBjt4IkUwLHyGoU3S_6jVjj0wIazHc7zPp9cwC * src/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/0b1d80ab9e..321bb9ac20 * src/third_party/grpc/src: https://chromium.googlesource.com/external/github.com/grpc/grpc.git/+log/972938174f..0e59e50307 * src/third_party/llvm-libc/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libc.git/+log/9c3ae3120f..4bdcf29a5e * src/third_party/perfetto: https://chromium.googlesource.com/external/github.com/google/perfetto.git/+log/02240e4fec..e6c9e81d08 * src/tools: https://chromium.googlesource.com/chromium/src/tools/+log/3f564babcb..7565a01a4c * src/tools/luci-go: git_revision:14c1dfd95e07b23eeaff304bd29a2eb80b06fe0b..git_revision:efa549a8684c6e6aad8b21d1e92fcb2241813763 * src/tools/luci-go: git_revision:14c1dfd95e07b23eeaff304bd29a2eb80b06fe0b..git_revision:efa549a8684c6e6aad8b21d1e92fcb2241813763 * src/tools/luci-go: git_revision:14c1dfd95e07b23eeaff304bd29a2eb80b06fe0b..git_revision:efa549a8684c6e6aad8b21d1e92fcb2241813763 DEPS diff: https://chromium.googlesource.com/chromium/src/+/ba73e47453..547fced285/DEPS BUG=None Change-Id: I45af84b23f1eb48aa85282cdc89fc9e5bc63ed6d Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/394001 Commit-Queue: Autoroller <[email protected]> Bot-Commit: Autoroller <[email protected]> Cr-Commit-Position: refs/heads/main@{#44789}

This is a reland of commit 788e02e Original change's description: > IWYU modules/desktop_capture/ > > using > find modules/desktop_capture/ -name "*.h" -o -name "*.cc" | grep -v win | grep -v wayland | grep -v mac | grep -v fuchsia | xargs tools_webrtc/iwyu/apply-include-cleaner > followed by > tools_webrtc/gn_check_autofix.py -C out/Default/ > and git cl format > > Manual changes (mostly replacing C style headers with C++ ones) were done in PS3. > PS5+PS6 brought back some platform-specific includes. > > BUG=webrtc:42226242 > > Change-Id: I4b84fa6abc320de0c734a43efa84c6f7e434c9af > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/382140 > Reviewed-by: Danil Chapovalov <[email protected]> > Commit-Queue: Philipp Hancke <[email protected]> > Reviewed-by: Alexander Cooper <[email protected]> > Cr-Commit-Position: refs/heads/main@{#44778} Bug: webrtc:42226242 Change-Id: Iee8dbef0a6c21b84da6a2449a90e825b8498f601 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393940 Reviewed-by: Philipp Hancke <[email protected]> Reviewed-by: Alexander Cooper <[email protected]> Commit-Queue: Danil Chapovalov <[email protected]> Cr-Commit-Position: refs/heads/main@{#44790}

Bug: None Change-Id: I0a35d480656b82e042e32d1019b9af19a6d79a42 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/394067 Bot-Commit: [email protected] <[email protected]> Commit-Queue: [email protected] <[email protected]> Cr-Commit-Position: refs/heads/main@{#44791}

This reverts commit c2af9bf. Reason for revert: The change hasn't reduced flakiness of the test. See e.g. https://chromium-swarm.appspot.com/task?id=71305f8037899511&o=true&w=true Bug: webrtc:0 Original change's description: > Use RunLoop::Flush instead of Sleep > > This hopefully addresses flakiness in ResolveAfterDeleteDoesNotReturn > (at least on Windows) > Example run: > https://ci.chromium.org/ui/p/webrtc/builders/ci/Win64%20ASan/15860/overview > > > Bug: webrtc:0 > Change-Id: I1090dba592c15d2764c523a371bdc41aadaa0526 > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393080 > Reviewed-by: Mirko Bonadei <[email protected]> > Auto-Submit: Tomas Gunnarsson <[email protected]> > Commit-Queue: Tomas Gunnarsson <[email protected]> > Cr-Commit-Position: refs/heads/main@{#44723} Bug: webrtc:0 Change-Id: I6a65c35980a39d6a269a6bc982d8b641c2b0efbd Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/394200 Auto-Submit: Tomas Gunnarsson <[email protected]> Reviewed-by: Mirko Bonadei <[email protected]> Commit-Queue: Mirko Bonadei <[email protected]> Cr-Commit-Position: refs/heads/main@{#44792}

Bug: None Change-Id: Ic27613c72dd187066b8ccdfcc01e6763f37a876b Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/394284 Commit-Queue: [email protected] <[email protected]> Bot-Commit: [email protected] <[email protected]> Cr-Commit-Position: refs/heads/main@{#44793}

Bug: webrtc:419453427 Change-Id: I7fba1c142f70f527ab5d00911a2fc8f4643d9352 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393980 Commit-Queue: Danil Chapovalov <[email protected]> Reviewed-by: Fanny Linderborg <[email protected]> Cr-Commit-Position: refs/heads/main@{#44794}

The functions were skipping over updating the sender and receiver when the list of header extensions was changed. Now it should be better. Bug: webrtc:383078466 Change-Id: I1b93ed1ba4bbbf6c5b13861a6e21a7a7c82a5a66 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396640 Reviewed-by: Per Kjellander <[email protected]> Commit-Queue: Harald Alvestrand <[email protected]> Cr-Commit-Position: refs/heads/main@{#44951}

This verifies that earlier changes do prevent transport-cc messages from being generated. Bug: webrtc:383078466 Change-Id: I83dd306a8d656ec698d92f6d0a5e3bef7c161a3a Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/372220 Commit-Queue: Harald Alvestrand <[email protected]> Reviewed-by: Per Kjellander <[email protected]> Cr-Commit-Position: refs/heads/main@{#44952}

…ble_target_bitrate from Video allocation and encoder TargetTransferRate.stable_target_bitrate is planned to be deprecated since it is mostly unused and does not currently work well in networks limited by packet loss. Bug: webrtc:42220156, webrtc:423841921 Change-Id: Ib2a0321a6771974e679ec9a0da16e175a554ccc4 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396720 Commit-Queue: Erik Språng <[email protected]> Reviewed-by: Erik Språng <[email protected]> Auto-Submit: Per Kjellander <[email protected]> Cr-Commit-Position: refs/heads/main@{#44953}

Allow to configure field trials before construction. Pass InjectedComponents by value, thus avoiding need to worry about nullptr special case Remove TestDependencyFactory as unused: custom components are provided through InjectedComponents Update VideoQualityTest tests to propagate field trials without relying on the global field trial string Bug: webrtc:419453427 Change-Id: I64f239c5406d3f9f552b09bf38c85dfeef93c951 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396581 Commit-Queue: Danil Chapovalov <[email protected]> Reviewed-by: Åsa Persson <[email protected]> Reviewed-by: Mirko Bonadei <[email protected]> Cr-Commit-Position: refs/heads/main@{#44954}

The fix has landed. Bug: webrtc:42225697, webrtc:383078466 Change-Id: I9838c9ed582ddef05d5d1593c5a3e6d9a60543aa Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396802 Reviewed-by: Per Kjellander <[email protected]> Commit-Queue: Per Kjellander <[email protected]> Auto-Submit: Harald Alvestrand <[email protected]> Cr-Commit-Position: refs/heads/main@{#44955}

Remove deprecated PeerConfigurer::SetFieldTrials in favor of AddFieldTrials That is more restricted api, but gives PCLF more freedom in combining test provided field trials with framework provided field trials. Delete noop enable_flex_fec_support flag as framework now can enable flexfec per peer without need to rely on global field trials. Remove fallback to the global field trials. Bug: webrtc:419453427 Change-Id: I71d9f933dbc0b91ea88e6c78d85679bd5c612b75 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396082 Reviewed-by: Jeremy Leconte <[email protected]> Commit-Queue: Mirko Bonadei <[email protected]> Auto-Submit: Danil Chapovalov <[email protected]> Reviewed-by: Mirko Bonadei <[email protected]> Cr-Commit-Position: refs/heads/main@{#44957}

FrameCadenceAdapter + ZeroHz is mature enough that we don't need the histograms around anymore. No-Try: True Bug: chromium:418772311, chromium:40200151 Change-Id: I4c245421c251d044a3417efb584541795d50dd96 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396800 Reviewed-by: Ilya Nikolaevskiy <[email protected]> Commit-Queue: Markus Handell <[email protected]> Auto-Submit: Markus Handell <[email protected]> Cr-Commit-Position: refs/heads/main@{#44958}

The feature is mature enough that we don't need these kill switches and field trials anymore. Bug: chromium:40200151 Change-Id: I0344a42c8637034226925bc95f1eda10c69df977 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396801 Reviewed-by: Ilya Nikolaevskiy <[email protected]> Commit-Queue: Markus Handell <[email protected]> Cr-Commit-Position: refs/heads/main@{#44959}

This function is unnecessary, since we can set an empty list. Preliminary to eliminating the special empty-list handling. Bug: webrtc:425296984 Change-Id: I030230ec162ab3f8cd0fc82d53cf9ced68e8b716 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396620 Reviewed-by: Tomas Gunnarsson <[email protected]> Auto-Submit: Harald Alvestrand <[email protected]> Commit-Queue: Tomas Gunnarsson <[email protected]> Cr-Commit-Position: refs/heads/main@{#44960}

Removed from - AudioEncoderOpus - BitrateAllocator stable_target rate in GoogCC is not used for anything except as a stable rate for opus. In order to simplify GoogCC, it will be be deprecated since it does not work that well in loss limited network scenarios. Bug: webrtc:423841921 Change-Id: I6356ab4f11542816b6200cbb055d1a98ecebad74 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396740 Commit-Queue: Per Kjellander <[email protected]> Reviewed-by: Jakob Ivarsson‎ <[email protected]> Cr-Commit-Position: refs/heads/main@{#44961}

and reorder everything, removing the need for forward declarations. Bug: webrtc:424706384 Change-Id: I2e1307667d3b328e8ce2bb18ee801768254a4c0d Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396680 Reviewed-by: Danil Chapovalov <[email protected]> Commit-Queue: Philipp Hancke <[email protected]> Reviewed-by: Harald Alvestrand <[email protected]> Cr-Commit-Position: refs/heads/main@{#44963}

Bug: None Change-Id: I9a077c6cc45e98314d7a4737f8e1afdd9d8c600e Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396846 Bot-Commit: [email protected] <[email protected]> Commit-Queue: [email protected] <[email protected]> Cr-Commit-Position: refs/heads/main@{#44967}

It seems to be unused and useless. Bug: webrtc:425296984 Change-Id: Iee68aec438889f8c1dd7ee69ea444606f17ff485 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396860 Reviewed-by: Tomas Gunnarsson <[email protected]> Commit-Queue: Harald Alvestrand <[email protected]> Cr-Commit-Position: refs/heads/main@{#44968}

Bug: webrtc:419453427 Change-Id: I82612955c915655e00577d56692316a7b5c5cd2a Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396901 Reviewed-by: Harald Alvestrand <[email protected]> Auto-Submit: Danil Chapovalov <[email protected]> Commit-Queue: Danil Chapovalov <[email protected]> Cr-Commit-Position: refs/heads/main@{#44969}

Done using find rtc_base/ -name "*.h" -o -name "*.cc" | xargs tools_webrtc/iwyu/apply-include-cleaner followed by tools_webrtc/gn_check_autofix.py and git cl format. Some of the more platform specific changes or changes that likely require more attention were reverted manually. Those will be done in smaller CLs. Bug: webrtc:42226242 No-Iwyu: cross-platform issue in async_dns_resolver which will be handled in a follow-up. Change-Id: I963a5bd4943a780006f5371b15bbdc6a9ec87bb1 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396703 Reviewed-by: Harald Alvestrand <[email protected]> Reviewed-by: Danil Chapovalov <[email protected]> Commit-Queue: Philipp Hancke <[email protected]> Cr-Commit-Position: refs/heads/main@{#44970}

Bug: None Change-Id: Id004cdaadd2ebd6ba92e639da78756c20572263e Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/397040 Auto-Submit: Harald Alvestrand <[email protected]> Commit-Queue: Danil Chapovalov <[email protected]> Commit-Queue: Harald Alvestrand <[email protected]> Reviewed-by: Danil Chapovalov <[email protected]> Cr-Commit-Position: refs/heads/main@{#44972}

Bug: chromium:425617388 Change-Id: I139a5cede0b50c1fff3999c3761a68d2cb93ec0f Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/397080 Reviewed-by: Harald Alvestrand <[email protected]> Commit-Queue: Palak Agarwal <[email protected]> Cr-Commit-Position: refs/heads/main@{#44973}

This reverts commit 8178847. Reason for revert: Microseconds were not the root cause of the issues we observed. Original change's description: > Use milliseconds to covert NTP capture time in receiver frames. > > > No-Iwyu: No output from iwyu/apply-include-cleaner > Bug: chromium:419755714 > Change-Id: Ia5ede3e253902ad288096eb9db3bc63dc7ba15e9 > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/393400 > Commit-Queue: Guido Urdaneta <[email protected]> > Reviewed-by: Jakob Ivarsson‎ <[email protected]> > Cr-Commit-Position: refs/heads/main@{#44754} Bug: chromium:419755714 Change-Id: If48cbf8b8096471cccb154ee6aa84fe331cf58a1 No-Iwyu: No output from iwyu/apply-include-cleaner Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/395140 Commit-Queue: Guido Urdaneta <[email protected]> Reviewed-by: Jakob Ivarsson‎ <[email protected]> Cr-Commit-Position: refs/heads/main@{#44974}

…ate header extensions" This reverts commit 9496cc0. Reason for revert: Investigating downstream issue. Bug: webrtc:383078466 Original change's description: > Change SetLocalContent / SetRemoteContent to update header extensions > > The functions were skipping over updating the sender and receiver when > the list of header extensions was changed. Now it should be better. > > Bug: webrtc:383078466, b/425662432, b/426394283 > Change-Id: I1b93ed1ba4bbbf6c5b13861a6e21a7a7c82a5a66 > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/396640 > Reviewed-by: Per Kjellander <[email protected]> > Commit-Queue: Harald Alvestrand <[email protected]> > Cr-Commit-Position: refs/heads/main@{#44951} (cherry picked from commit d27b4ef) Bug: webrtc:383078466 Change-Id: Ie6db6d995021a8699c1c7aa91334ea0ea0e37c1f Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/397820 Reviewed-by: Per Kjellander <[email protected]> Reviewed-by: Harald Alvestrand <[email protected]> Commit-Queue: Mirko Bonadei <[email protected]> Bot-Commit: [email protected] <[email protected]> Owners-Override: Mirko Bonadei <[email protected]> Reviewed-by: Christoffer Dewerin <[email protected]> Cr-Original-Commit-Position: refs/heads/main@{#45012} Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/399020 Reviewed-by: Mirko Bonadei <[email protected]> Commit-Queue: Harald Alvestrand <[email protected]> Cr-Commit-Position: refs/branch-heads/7258@{#1} Cr-Branched-From: 74fa937-refs/heads/main@{#44974}

Original change's description: > Add chrome-cherry-picker account to bot allowlist > > chrome-cherry-picker@chops-service-accounts.iam.gserviceaccount.com is > being by the Chrome Cherry Picker (go/chromecherrypicker) and needs to > be able to skip the author check for presubmits. > > Bug: chromium:414375466 > Change-Id: Ib9f15dd67a4efe5346e6631135e1bcd7196b992c > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/400480 > Reviewed-by: Mirko Bonadei <[email protected]> > Reviewed-by: Björn Terelius <[email protected]> > Commit-Queue: Gennady Tsitovich <[email protected]> > Cr-Commit-Position: refs/heads/main@{#45148} Bug: chromium:431157710,chromium:414375466 Change-Id: Ib9f15dd67a4efe5346e6631135e1bcd7196b992c Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/400700 Commit-Queue: Rubber Stamper <[email protected]> Auto-Submit: Chrome Cherry Picker <chrome-cherry-picker@chops-service-accounts.iam.gserviceaccount.com> Bot-Commit: Rubber Stamper <[email protected]> Cr-Commit-Position: refs/branch-heads/7258@{#2} Cr-Branched-From: 74fa937-refs/heads/main@{#44974}

No behavior changes. (cherry picked from commit 5ff715d) Bug: webrtc:383141571, chromium:433885045, chromium:434133034 Change-Id: Ice5f3e5cbd245ddea407248a6f29c61c646e6a72 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/401740 Reviewed-by: Harald Alvestrand <[email protected]> Commit-Queue: Guido Urdaneta <[email protected]> Cr-Original-Commit-Position: refs/heads/main@{#45206} Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/402200 Cr-Commit-Position: refs/branch-heads/7258@{#3} Cr-Branched-From: 74fa937-refs/heads/main@{#44974}

Jwata and others added 30 commits May 27, 2025 01:21

Harald Alvestrand and others added 28 commits June 17, 2025 00:59

Merge branch '7258_62' into feature/jed/temp-update-to-M139

71b326e

sf-jed-kyung self-assigned this Sep 17, 2025

sf-jed-kyung changed the title ~~Update to M139~~ M139 -> M138 Sep 30, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

M139 -> M138 #35

M139 -> M138 #35

Uh oh!

sf-jed-kyung commented Aug 25, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

12 participants

M139 -> M138 #35

Are you sure you want to change the base?

M139 -> M138 #35

Uh oh!

Conversation

sf-jed-kyung commented Aug 25, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

12 participants