When notifying the framework of an incoming call, a lock is held
in both the MmTelFeature and MmTelFeatureConnection. This can cause
a deadlock if the MmTelFeatureConnection is also processing an
event, such as sending/acknowleging an SMS message.
Remove the lock around the listener methods to the framework, since
it is not needed.
Bug: 155083563
Test: atest CtsTelephonyTestCases:ImsServiceTest FrameworksTelephonyTests
Change-Id: I39f7192c6f79e215ef989797870f5f501197cd08
For example, if share sheet is fully expanded in personal profile,
when we go to the work tab, the direct share targets should
also be expanded there.
Fixes: 154523836
Test: manual
Test: atest ChooserActivityTest
Change-Id: I9346d7485f7498fc038c86bf5f8550c8ea6d0b48
Some callers weren't getting the @Provides version of the controller and
thus not initializing it properly
Test: manual
Fixes: 154638737
Change-Id: I8ce4373b563ca307e2892ffa35047d95f04a705d
Some minor fixes:
* Remove icon next to structure name in Add controls
* Remove "other apps" when there's no other app to switch.
* Change footer buttons in apps screen.
Test: manual
Fixes: 154672630
Fixes: 154673161
Fixes: 154631120
Change-Id: I45be5c78b97a0301b77fa11125a294c66903dbef
Updates the colors used when the dark theme is on. When a
screenshot is taken, checks for the current dark theme status and
updates the assets if necessary.
Test: manual
Bug: 146560663
Change-Id: Iccc727924e04febc5be25e2205775b52ef082f6a
This CL does the following changes:
- Save and restore the resolver drawer layout height.
This is necessary because when we toggle darkmode,
we don't go through handleLayoutChange, thus we don't
recalculate the height the standard way.
- Disables saving state for the viewpager.
This is necessary, because the content is
incorrectly restored afterwards. It's better to
reload it instead.
- Saves and restores the last selected tab. This is
necessary because with the viewpager state saving
disabled, it defaults to the first tab.
I considered updating the configChanges attribute in
the manifest to include dark mode toggling, but it
gave unsatisfactory result - the share sheet only
partially became dark.
Test: manual
Test: atest ChooserActivityTest
Test: atest ResolverActivityTest
Fixes: 154526324
Change-Id: I99e159ea7ca413f8acbc0429a43ce5aba32a1b28
* updatePersonalAppsSuspension is invoked for all events relevant
to profile maximum time off: user stopped, user unlocked,
system boot, deadline alarm goes off,
setManagedProfileMaximumTimeOff called.
* It takes all relecant bits of state into account: policy,
current deadline, user state. It calculates the new state
of the deadline, notification and alarm and makes appropriate
changes (e.g. schedules the alarm, posts notification, suspens
apps).
* Updated package manager query flags so that even when personal
apps are being suspended while the user is locked, it includes
non direct boot aware apps as well.
Test: manual, with TestDPC
Test: atest OrgOwnedProfileOwnerTest#testWorkProfileMaximumTimeOff
Test: atest com.android.server.devicepolicy.DevicePolicyManagerTest
Test: atest OrgOwnedProfileOwnerTest#testPersonalAppsSuspensionNormalApp
Bug: 149075510
Change-Id: I94d2582c7af91a5d97e67d2baf2e15f0a6d5ffa9
Recently, a CTS that ensured the existence of a system gallery was
added. However, this test was failing on GSI since the default holder
for the role is com.android.gallery which doesn't exist on GSI.
This change replaces the default holder with a package that exists on
all builds, including GSI and AOSP.
Test: build
Fix: 153939224
Change-Id: I4265f1d840f8c440f2e9134206893ed3fd0ea465
Considering the tight character restriction in the Battery page, we
updated the following battery descriptions:
Less than 15 min remaining -> Less than 15 min left
More than 2 days remaining -> More than 2 day left
Bug: 143475287
Test: manual and robotest
Change-Id: I2092bdca61585010b9a6e343e647ab33ceabcdda
All carriers discarded the requirement to put up a standing
notification when tethering is on. Thus, remove the "tethering
is on" notification.
Bug: 154438388
Test: atest TetheringTests
Change-Id: Ife3915837b6b7b83d3eaaa84b71b6409ff37b71c
Merged-In: Ife3915837b6b7b83d3eaaa84b71b6409ff37b71c
(cherry picked from commit 0171c07d05, aosp/1289107)
The CL that moved the initialization of the tethering offload
config HAL from C++ to Java caused the code not to retry
fetching the service if it is not ready when tethering is
started. This is because the C++ version of getService() retries,
but the Java version only retries if getService(true) is called.
Make the new code retry as well.
b/152430668#comment4 asserts that the fetch will be retried only
if the service is installed on the device, so the retries should
be attempted (and thus should not have any startup time impact)
on devices that do not support tethering offload.
Bug: 152430668
Test: builds, boots, tethering offload works
Merged-In: I093f127d90b2aa1b13eb0748378a24726d419472
Change-Id: I093f127d90b2aa1b13eb0748378a24726d419472
Some strings are not used, so just remove them.
Bug: 154445061
Test: atest TetheringTests
Change-Id: Ie67f7f4d0863d58c4c5206e445c1dbdfe3f65f60
Merged-In: Ie67f7f4d0863d58c4c5206e445c1dbdfe3f65f60
(cherry picked from commit 59eeedc045, aosp/1289100)
The operation applies to a local transaction, so it should not
touch the global transaction.
Bug: 143734779
Test atest AccessibilityWindowManagerTest#
setAccessibilityWindowIdToSurfaceMetadata
Test: Enable accessibility service (e.g. TalkBack) and check logcat
shouldn't have "Call to SurfaceControl.closeTransaction without
matching openTransaction".
Change-Id: I5a2879656471c85f91931e385e4873b4499ed977
Previously, MetricsManagers would unregister themselves as a
PullUidProvider for a given ConfigKey in the destructor. This caused all
pulls to fail after a config update because the new MetricsManager would
register itself before the old MetricsManager was destructed and
unregistered. This resulted in the old MetricsManager removing the new
config since they shared the same config key. The fix is for the
PullerManager to check that the PullUidProviders are equal in the
unregister function before actually erasing it.
Test: bit statsd_test:* (wrote a failing test that now passes).
Test: statsd_localdrive to manually update a config, ensured pulls still
worked.
Bug: 154544328
Change-Id: Id7af3b3b407e24bee74fc34bd1c2b9e0575e9c9e
We change to reference the end() of the correct iterator.
Test: TreeHugger
Bug: 150764186
Change-Id: I380d41a916184c90c3a38848b0a039ea56006f47
Merged-In: I380d41a916184c90c3a38848b0a039ea56006f47
(cherry picked from commit ef05f714c4)
Keep the windowing mode of WallpaperWindowToken as fullscreen.
Bug: 151472924
Test: atest WallpaperControllerTests
Change-Id: Idd9e25fc592b90abf14526ea61396fa46020abf9
