There's now a shared defaults module for this kind of filegroup. Use
that and remove the previously duplicated prop.

Bug: 271816210
Test: m
Change-Id: Ia4547984a73179fdfc98c427482dd10246f9722c

* 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

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

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

This reverts commit 2cd4ac87.

Reason for revert: b/273575026

Change-Id: I5b9886465b8a0f71af469d303a39f670a9b0cb68

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

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

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

Test: run import script
Change-Id: Idddb7e2bb68c4faa0e0096c029902be368c8e0d1

* 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

Test: run script
Change-Id: I403ffbf74d7e986978e1f5a2f1a6fc90c256efe6

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

* changes:
  gn2bp: Remove sqlite from net_unittests
  Revert "gn2bp: Convert cc_objects to cc_static_library"

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

* changes:
  Add test for the NetworkTraceHandler
  Parse new config options and to batch in callback

* 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

* 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

Apply https://chromium-review.googlesource.com/c/chromium/src/+/4166159

Test: None
Change-Id: I297718dddd86a838367e86b8a0449d23114439b6

Test: atest CtsNetHttpTestCases
Change-Id: If24b12c7d602a3937d80e931b8bfa7f9ef7c14bc

This reverts commit 379d79d1.

Reason for revert: This is breaking the symbols of test library.

Test: None
Change-Id: I3a2e8a9a6ec6ce04038ed34489ece6a1ca053cef

For manual Android.bp change in aosp/2475845

Bug: 265674359
Test: ./update_results.sh
Change-Id: Ie94708167479a67a52bbc1c7fd629a8206585cfd

Bug: 273316506
Merged-In: Id0876b10d9a4199419b5ae9d11876d32e3da3f0c
Change-Id: Ia7c244dff703c0904f181b7081bcbe359d7ac62c

For consistency with other Callback classes, and not to force class
hierarchy on users. The historical "callback completeness" issue can be
mnitigate using interface defaults.

Test: atest CtsNetHttpTestCases
Change-Id: Ibf2aaf4396ab9defdeb4e3882cea6b1b2dbd36ee
Merged-In: Id8fcc13567da8878e848f3c6668d865e77b48146

Test: atest TetheringMetricsTest
Change-Id: Ic44f47460342300164fa6ad5c928a850ba28d1fb

Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2475445



Change-Id: I45ff707a354b5cf8ee9a505a18295f3dee9fea06
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>