- Mar 17, 2023
-
-
Stefano Duo authored
This relands commit 2cd4ac87 Merged-In: I903369dbad37b663a39e80149e25b0634386df6d Change-Id: I018121bf4d02ccc59c89de39e3d1598deb9cf67b Bug: 274094720
-
- Mar 15, 2023
-
-
Chidera Olibie authored
-
Chidera Olibie authored
Bug: 251012627 Test: atest NetHttpTestCases Change-Id: Icb8b9f38b96d9aad1a092663b6dc03a4bf578dfb
-
Wayne Ma authored
-
Dan Stahr authored
-
Chidera Olibie authored
-
Wayne Ma authored
Test: atest TetheringMetricsTest Change-Id: I1b536ff198119b07e9313f081a8e02e515c67efd
-
Dan Stahr authored
Test: atest CtsNetHttpTestCases Bug: 273626798, 273626794, 251012627 Change-Id: Ie6f4ee3f17fa8e2dc82a059bc13ad305771ed329
-
Treehugger Robot authored
-
Ryan Zuklie authored
-
Ryan Zuklie authored
* changes: Add support for dropped fields in NetworkTracing Add support for NetworkTrace aggregation threshold Add fallback in Network Tracing to old format Add event bundling in NetworkTraceHandler
-
Treehugger Robot authored
-
Ryan Zuklie authored
Interning is a feature Perfetto offers. You can store details in the intern table and associate it with an id. Then, each trace packet can reference just the id rather than the full proto contents. In our case, we already identify unique contexts, so all we need to do is give them unique IDs and record that instead. Bug: 246985031 Test: atest libnetworkstats_test Change-Id: I84f7673bc41b89390c02b8ec5460adfadbb36173
-
Ryan Zuklie authored
Dropping fields removes them from the bucketing and from the trace output (the important part being the bucketing). The most useful is dropping the local port which is often meaningless and can occasionally have very high cardinality (leading to less bundling, less aggregation and more space used by interning). Bug: 246985031 Test: atest libnetworkstats_test Change-Id: I2ae3289583fa7f29e60d92c58378f189a564bd81
-
- Mar 14, 2023
-
-
Maciej Żenczykowski authored
-
Maciej Żenczykowski authored
-
Patrick Rohr authored
This reverts commit 2cd4ac87. Reason for revert: b/273575026 Change-Id: I5b9886465b8a0f71af469d303a39f670a9b0cb68
-
Ryan Zuklie authored
The aggregation threshold determines the number of packets a particular bundle has to hit before recording only aggregate results. This allows individual high-throughput streams to be recorded at a significantly reduced trace buffer cost. Bug: 246985031 Test: atest libnetworkstats_test Change-Id: I9e3c6f013db91919be541860364764e913726cf1
-
Ryan Zuklie authored
Perfetto has been updated to understand both the bundled and individual packets, but the change hasn't yet hit stable. Until it does, fall back to using the single event format. Also fixes build warning from prior change. Note: once the change hits Perfetto stable, the two formats behave identically in the ui/querying/etc. Bug: 246985031 Test: atest libnetworkstats_test Change-Id: Ifc0e00f3c73aa1ef485f87f6ed4a9873c193ebb3
-
Xin Li authored
-
Patrick Rohr authored
Somewhat surprisingly, HandlerThread#quitSafely is *not* a synchronous method, so the tearDown() method still needs to wait for the thread to join. Test: TH Change-Id: I4297ba057b3d4e67c7b462b6995da637f2479d42
-
Patrick Rohr authored
-
Mohannad Farrag authored
-
Patrick Rohr authored
Test: run import script Change-Id: Idddb7e2bb68c4faa0e0096c029902be368c8e0d1
-
Patrick Rohr authored
* changes: cronet import: add test dependencies that were imported in aosp/2399270 cronet import: use folder.origin instead of git.origin cronet import: use public chromium repository as a source cronet import: add --git-origin-checkout-hook to run gclient
-
Patrick Rohr authored
Test: run script Change-Id: I403ffbf74d7e986978e1f5a2f1a6fc90c256efe6
-
Patrick Rohr authored
git.origin uses a bare repo which is not supported by gclient. There may be ways to get this to work including the git-origin-checkout-hook, but those are a) really slow and b) much more complex than just using a folder.origin. This change also adds support to the import script to configure the origin folder (i.e. clone the repo and run gclient) before the copybara script is invoked. Test: run import script Change-Id: I6289108271960c87583a059a553b0d0e7db8af68
-
Treehugger Robot authored
-
Treehugger Robot authored
* changes: gn2bp: Remove sqlite from net_unittests Revert "gn2bp: Convert cc_objects to cc_static_library"
-
Ryan Zuklie authored
Bundling groups PacketTraces by their various attributes (e.g. uid, tag, ports) and outputs a single Perfetto TracePacket per group. Rather than repeat this information many times, it's only written once. In most cases, this should reduce the trace size by 5x. Bug: 246985031 Test: atest libnetworkstats_test Change-Id: Ia9cb163fb4c673abdab8d442576cf4b12a98dbc6
-
Ryan Zuklie authored
* changes: Add test for the NetworkTraceHandler Parse new config options and to batch in callback
-
Chidera Olibie authored
This simply checks that setting this option does not crash cronet or hinder requests. Bug: 251012627 Test: atest CtsNetHttpTestCases Change-Id: I4726dee2206331e70bccea69c282979f2d1de160
-
Treehugger Robot authored
-
Mohannad Farrag authored
* Some net files have different include_dirs for protos generated. If the generated file is at A/B/C/src/D/E. The net file expect the export include dir to be A/B/C/src/ Test: None Change-Id: I9ebe54094ea6bd2b00645be2bf724889dbcb3c1f
-
Mohannad Farrag authored
* Remove the python3 calls and replace them with the binary right away. * Fix action_foreach where sources can be used with {{source_file_name}} Test: None Change-Id: I0c2e6fd5135de70339168b9ba5d3b7f3a6866fdc
-
Mohannad Farrag authored
Apply https://chromium-review.googlesource.com/c/chromium/src/+/4166159 Test: None Change-Id: I297718dddd86a838367e86b8a0449d23114439b6
-
Stefano Duo authored
-
Stefano Duo authored
Test: atest CtsNetHttpTestCases Change-Id: If24b12c7d602a3937d80e931b8bfa7f9ef7c14bc
-
Mohannad Farrag authored
This reverts commit 379d79d1. Reason for revert: This is breaking the symbols of test library. Test: None Change-Id: I3a2e8a9a6ec6ce04038ed34489ece6a1ca053cef
-
Treehugger Robot authored
-