Skip to content
Snippets Groups Projects
  1. Jun 05, 2024
  2. Jun 04, 2024
  3. Jun 03, 2024
    • Ioannis Ilkos's avatar
    • Mark Fasheh's avatar
    • Ioannis Ilkos's avatar
      Publish TracingServiceProxy after storage has been unlocked · 5a104f2b
      Ioannis Ilkos authored
      TracingServiceProxy depends on external system services to forward its
      requests to. There is an edge case where reportTrace was called before
      the encrypted storage was unlocked and before external services can be
      bound (e.g. occurs when reportTrace is concurrent to system_server
      starting up).
      
      This change defers the registration of the service until we are ready to
      accept requests. As a side-effect this also means that the device needs
      to be unlocked before the service is published.
      
      Bug: 333846370
      Test: atest PerfettoReporterTest
      Test: boot device without lock and invoke perfetto reporting (verify the report is accepted immediately)
      Test: boot device with lock and invoke perfetto reporting (verify the report is accepted after unlocking)
      Test: invoke perfetto reporting concurrent to kill -9 `pidof system_server`
      Change-Id: Ibb75c3e314d30b18cdc32e3774be1ee8022960b1
      5a104f2b
    • Mark Fasheh's avatar
      Add owners for MessageQueue related files · d2593d46
      Mark Fasheh authored
      We've been profiling and making improvements to MessageQueue for
      some time now and also have OWNERS for MessageQueueTest.java. Would
      like to be able to review future changes to the core MessageQueue
      class. I also added Message.java for good measure.
      
      Test: presubmit
      Change-Id: I961a2468c7b9a451d3b33d78a67f17cccf22cd60
      d2593d46
    • Treehugger Robot's avatar
    • Treehugger Robot's avatar
      Merge "Increase the dropbox disk quota to 20Mb for user debug builds as we add... · 06cb3424
      Treehugger Robot authored
      Merge "Increase the dropbox disk quota to 20Mb for user debug builds as we add more debug to dropbox for user debug. (cherry picked from https://googleplex-android-review.googlesource.com/q/commit:ea261f6a2a9d1034abfaf2ebdaae72c9869c8c90) Merged-In: Ib758fa6fa47cfd26e895bf8d7b9d5bde98c99461 Change-Id: Ib758fa6fa47cfd26e895bf8d7b9d5bde98c99461" into main
      06cb3424
    • Jared Duke's avatar
      Enable basic optimization for DynamicSystemInstallationService · d32c27a2
      Jared Duke authored
      While this offers minimal savings, there's no real reason to opt out of
      optimization (the default for app targets).
      
      Test: m + check APK diff
      Bug: 203241149
      Change-Id: Ib1bba31a95afa87f37aedb0251796bdb31ab65c0
      d32c27a2
    • Parth Sane's avatar
      Add tests for ServiceManager Java Binder API · 0149ca4a
      Parth Sane authored
      Bug: 333854840
      Test: atest android.os.BinderTest
      Change-Id: Iaf1fd1e5daea7103a873bc8f8aa0b78377e57714
      0149ca4a
    • Olivier Gaillard's avatar
      Increase the dropbox disk quota to 20Mb for user debug builds as we add more... · fc26cff6
      Olivier Gaillard authored
      Increase the dropbox disk quota to 20Mb for user debug builds as we add more debug to dropbox for user debug.
      (cherry picked from https://googleplex-android-review.googlesource.com/q/commit:ea261f6a2a9d1034abfaf2ebdaae72c9869c8c90)
      Merged-In: Ib758fa6fa47cfd26e895bf8d7b9d5bde98c99461
      Change-Id: Ib758fa6fa47cfd26e895bf8d7b9d5bde98c99461
      fc26cff6
    • Mitch Phillips's avatar
      Workaround bad GC of tombstone watcher. · 476ed072
      Mitch Phillips authored
      There's some bad behaviour going on that's exclusive to WatchOS
      (discovered on the Pixel Watch 2). The TombstoneWatcher class is
      responsible for watching the /data/tombstones/ directory. It is a member
      of the NativeTombstoneManager class, which is created when
      ActivityManager starts, and lives until the device is powered off.
      
      Unfortunately, some bad behaviour (unclear exactly what at this stage)
      means that the TombstoneWatcher class is spuriously garbage collected,
      even though the parent NativeTombstoneMenager class is still alive and
      reachable. I chatted with hboehm@ from the ART team, and his hypothesis
      is that because the TombstoneWatcher member is initialized in the
      constructor and never used again, some optimiser is doing the wrong
      thing and thinking that it's unused and can be GC'd.
      
      The TombstoneWatcher uses inotify in native code to execute a callback
      when a file is added to a directory, so it's clearly in use, but
      unfortunately the bug allows this class to be GC'd anyway.
      
      What this means, in practice, is that app native crashes on WatchOS are
      not being handled correctly. This includes:
       - Apps cannot get their own crashes through the
         ApplicationExitInfo::getHistoricalProcessExitReasons API.
       - Crash reports are never sent to Google, with the exception of
         tombstones that last until the next reboot, where they're one-time
         scooped up and sent to Google. This has substantially increased
         latency (requires a reboot) and also probably drops quite a number of
         reports because there's a limit of tombstones on the device, and a
         rate limit of how many are sent by GMS.
      
      Bug: 339371242
      Test: atest CtsGwpAsanTestCases on eos (Pixel Watch 2, WatchOS)
      Change-Id: I9226e4368b03bd4742fccdafde6018f145da63e6
      476ed072
    • Treehugger Robot's avatar
    • lijilou's avatar
      AccessibilityWindowsPopulator:fix CWE problem by add mLock in dump · c6feab91
      lijilou authored
      method.
      
      Accessing mDisplayInfos, mInputWindowHandlesOnDisplays and other
      collections should add mLock lock according to AOSP requirements.
      
      Test: OEM monkey test
      Bug: 344434136
      Change-Id: Icb18683680c40939e0ebd352818702f90b79179f
      c6feab91
    • b0202.jung's avatar
      Fix focused error of embedded task fragments · a209eafe
      b0202.jung authored
      If a display has freeform windowing environments such as Desktop
      or XR device, then resumed activity of task fragment will be set
      to focused app. However, it can break focused state in embedded
      task fragments.
      
      Thus, allow it only if a focused task is different from a next task.
      
      Bug: 343252725
      Test: Settings(AE) > Launch SubSettings > Check focused app/window
      Change-Id: Ib62ecb7c8e0fe453149f51c610c34d97bbde7cb1
      a209eafe
  4. Jun 02, 2024
  5. May 31, 2024
Loading