Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

privacy tweaks + cleanups #39

Open
wants to merge 647 commits into
base: 14
Choose a base branch
from
Open

privacy tweaks + cleanups #39

wants to merge 647 commits into from

Conversation

SirRGB
Copy link

@SirRGB SirRGB commented May 18, 2024

No description provided.

mydongistiny and others added 30 commits April 8, 2024 19:28
Get battery estimates from com.google.android.apps.turbo (Turbo.apk)
Code taken from android11 SystemUIGoogle.apk and cleaned up

[jhonboy121]:
*  use GlobalSettings class for acquiring settings
*  use proper access modifiers and finalize many vars.
*  simplify estimates processing logic
*  use modern try catch method for properly closing cursor
*  cache many variables used a lot to reduce runtime
*  use Utils class method to check whether package is installed

Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
04-07 07:52:44.272  2200  2200 E AndroidRuntime: FATAL EXCEPTION: main
04-07 07:52:44.272  2200  2200 E AndroidRuntime: Process: com.mediatek.ims, PID: 2200
04-07 07:52:44.272  2200  2200 E AndroidRuntime: java.lang.NoSuchMethodError: No static method newFromCDS([B)Lcom/android/internal/telephony/gsm/SmsMessage; in class Lcom/android/internal/telephony/gsm/SmsMessage; or its super classes (declaration of 'com.android.internal.telephony.gsm.SmsMessage' appears in /system/framework/framework.jar!classes4.dex)
04-07 07:52:44.272  2200  2200 E AndroidRuntime: 	at com.mediatek.ims.feature.MtkImsSmsImpl.newStatusReportInd(MtkImsSmsImpl.java:153)
04-07 07:52:44.272  2200  2200 E AndroidRuntime: 	at com.mediatek.ims.feature.MtkMmTelFeature$1.newStatusReportInd(MtkMmTelFeature.java:173)
04-07 07:52:44.272  2200  2200 E AndroidRuntime: 	at com.mediatek.ims.ImsService$MyHandler.handleMessage(ImsService.java:2606)
04-07 07:52:44.272  2200  2200 E AndroidRuntime: 	at android.os.Handler.dispatchMessage(Handler.java:106)
04-07 07:52:44.272  2200  2200 E AndroidRuntime: 	at android.os.Looper.loopOnce(Looper.java:201)
04-07 07:52:44.272  2200  2200 E AndroidRuntime: 	at android.os.Looper.loop(Looper.java:288)
04-07 07:52:44.272  2200  2200 E AndroidRuntime: 	at android.app.ActivityThread.main(ActivityThread.java:7870)
04-07 07:52:44.272  2200  2200 E AndroidRuntime: 	at java.lang.reflect.Method.invoke(Native Method)
04-07 07:52:44.272  2200  2200 E AndroidRuntime: 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
04-07 07:52:44.272  2200  2200 E AndroidRuntime: 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1003)

Change-Id: If37782879a9d9ddfac8283fa243858327ce78850
Signed-off-by: SamarV-121 <[email protected]>
Signed-off-by: LordShenron <[email protected]>
If WifiService disabled and one of WifiAwareService,WifiP2pService
and RttService are supported, it will raise WifiInjector instance
exception in the boot procedure.

Test: Boot device
Change-Id: If50b158a1ec3474875c771ca6377647d25038f25
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Change-Id: I3dde24312f416b1ae2d365fea683aacd8970d923
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
 Update user/profile type specification to use them and to allow
 a maximum 16 profiles per parent (supposedly, but that does not seem to work
 on its own).

Change-Id: Ifa7bf1360111dbf994e37ff696a3ae4efb645166
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Issue: calyxos#1123
Change-Id: I610afe6b63f1a56c7c54dd9957d12fc0ea182ecd
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Some apps (such as miui camera) resets the auto brightness adjustment
to 0.0f instead of correctly using Float.NaN, causing auto brightness
animation to be disabled afterwards. Prevent this scenario by
overriding it to Float.NaN ourselves.

Change-Id: Idc39d120db306403482c7011db7c556f4f902468
Signed-off-by: LordShenron <[email protected]>
08-22 07:37:29.073  2015  2148 E AndroidRuntime: FATAL EXCEPTION: SysUiBg
08-22 07:37:29.073  2015  2148 E AndroidRuntime: Process: com.android.systemui, PID: 2015
08-22 07:37:29.073  2015  2148 E AndroidRuntime: java.lang.ClassCastException: android.graphics.drawable.LayerDrawable cannot be cast to android.graphics.drawable.Animatable
08-22 07:37:29.073  2015  2148 E AndroidRuntime: 	at com.android.systemui.media.controls.pipeline.MediaDataManager.createActionsFromState(MediaDataManager.kt:1048)
08-22 07:37:29.073  2015  2148 E AndroidRuntime: 	at com.android.systemui.media.controls.pipeline.MediaDataManager.loadMediaDataInBg(MediaDataManager.kt:846)
08-22 07:37:29.073  2015  2148 E AndroidRuntime: 	at com.android.systemui.media.controls.pipeline.MediaDataManager$loadMediaData$1.run(MediaDataManager.kt:471)
08-22 07:37:29.073  2015  2148 E AndroidRuntime: 	at android.os.Handler.handleCallback(Handler.java:942)
08-22 07:37:29.073  2015  2148 E AndroidRuntime: 	at android.os.Handler.dispatchMessage(Handler.java:99)
08-22 07:37:29.073  2015  2148 E AndroidRuntime: 	at android.os.Looper.loopOnce(Looper.java:201)
08-22 07:37:29.073  2015  2148 E AndroidRuntime: 	at android.os.Looper.loop(Looper.java:288)
08-22 07:37:29.073  2015  2148 E AndroidRuntime: 	at android.os.HandlerThread.run(HandlerThread.java:67)
08-22 07:37:29.190  1695  1695 W WindowManager: removeWindowToken: Attempted to remove non-existing token: android.os.Binder@8c1d614

Change-Id: If9794a6867acb8cf21521979c93e10221fad552a
Signed-off-by: minaripenguin <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Separate performClick from delayed message. Then we can do
performClick in advance of PRESSED_STATE_DURATION(64ms) delay.
This will benefit for click operation latency. The delayed
message is still used to clear pressed status.

Change-Id: I95eb72c5362a1838ab16ee8211120aee8cd548e5
CRs-Fixed: 3503330
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
* Initial code based on 11.0 commit crdroidandroid/android_frameworks_base@9773423
* Removed detailed adapter - deprecated now. :(
* Simplify code and strings
* Use location as tile label when enabled
* handleClick will open settings when disabled, will open weather app when enabled
* handleLongClick will always open settings

Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
    OmniROM:

    From: maxwen <[email protected]>
    Date: Wed, 14 Oct 2020 17:17:31 +0200
    Subject: [PATCH] SystemUI: qs tiles revenge

    -screenshot

    [micky387]
     * edit to the new A12 Tiles API
     * Add secondaryLabel for the long press screenshot tile

    From: micky387 <[email protected]>
    Date: Wed, 14 Dec 2022 13:12:26 +0100
    Subject: [PATCH] SystemUI: Update ScreenshotTile after merge

Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
 * SoundSearchTile: Make default state inactive
 * Disable long click for SoundSearch tile

Squashed:

    From: El Dainosor <[email protected]>
    Date: Wed, 21 Mar 2018 20:18:21 +0000
    Subject: SoundSearchTile: Extend the use

    Because i think there's people using other apps instead of Google, so i'll use the intents from shazam and soundhound to trigger the search sound engine.
    Thanks to the tasker dev to make the intents public.

    * Add a toast informing the user about how the tile isn't working.

    Signed-off-by: Joey Huab <[email protected]>

Change-Id: Id893722185fa4f111f2730849d1c8abfffa121dc
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Toggle the primary default language

- Bring it to Oreo - LorDClockaN
- Bring it to Pie, 11, 12 - eyosen
- Bring it to T - semdoc
- Bring it to A14 - neobuddy89

Change-Id: I3e336eddb360ea1a796727657652ad9301007380
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Signed-off-by: Gustavo Mendes <[email protected]>
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
And also Get rid of old and unused detail adapter
Some other minor improvements

Signed-off-by: LordShenron <[email protected]>
The secondary accent in light mode does not have a good contrast
to the previously white icon.

Previously: https://imgur.com/a/XHpDHDJ
Now: https://imgur.com/a/QCk3QwC

Change-Id: Ib61ad4803e0e3c6cc9163408137c7a16742922b6
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Use colorAccent instead of colorAccentPrimary (matching QS)
to fix visibility in light theme.

Sample video: https://twitter.com/MishaalRahman/status/1666500007195729921

Change-Id: I8397308677414af94361920ca9c073d72b3693c1
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
When an external audio focus policy is registered, no focus event
are reported to registered audio focus listener.

This CL fixes the listener.

NPE may happen on concucrrent call of loss transient of focus
from external focus manager and abandonAudioFocus from client.
This CL protects also the setFocusRequestResultFromExtPolicy.

Test: build & basic audio

Change-Id: I31475c5c2d32593906b8dbcf9c8ddfca88695db8
Signed-off-by: Francois Gaffie <[email protected]>
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Change-Id: I3d2fc1f89fa64e1d9151efc0e10a0b7fafd6ff0a
Signed-off-by: Chet Kener <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Scrolling cache helps make short scrolls/flings smooth but will
cause stutter when long flings are made. This patch disables
scrolling cache when long flings are made.

This patch also fixes a related bug where scrolling cache will
not be enabled properly when transitioning from flinging to scrolling.

Patch Set 2: Calculate threshold based on maximum velocity (Sang Tae Park)

Change-Id: Iad52a35120212c871ffd35df6184aeb678ee44aa
Signed-off-by: Alex Naidis <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Improves responsiveness

Change-Id: I18a52ac84ba9c8274adc757e41ca2ca995d514f6
Signed-off-by: LordShenron <[email protected]>
Reduce scroll friction to a better default
value which is used in Scroller and Overscroller.

Change-Id: I66a7663a18bb80263c51f3d54a2bb1e3fe5d0b4d
Signed-off-by: Alex Naidis <[email protected]>
Signed-off-by: Adithya R <[email protected]>
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Signed-off-by: minaripenguin <[email protected]>
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Change-Id: I788dc08d35d463b91fda65e1e5d2a8dc99ce3360
Signed-off-by: minaripenguin <[email protected]>
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
- some hals fail to report success/failure (for ex. realme fp hals)

[DarkJoker360 - Switch to overlays]
[timjosten - Adapt to 12.1.0_r8 merge changes]
[ghostrider-reborn - Simplify code]

Change-Id: I442ce063280af36a04c25fcbc3dd45a90f196988
Signed-off-by: SagarMakhar <[email protected]>
Signed-off-by: DarkJoker360 <[email protected]>
Signed-off-by: Sarthak Roy <[email protected]>
Signed-off-by: Adithya R <[email protected]>
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
https://katb.in/kat9848

I'm not 100% sure, but I believe that this was the reason for a SystemUI hang,
which was responsible for the fact that the screen could not be switched on
for several seconds and just remained black.

Signed-off-by: spezi77 <[email protected]>
Signed-off-by: LordShenron <[email protected]>
* attempt to fix random short appearance of IME when creating/switching views

fixes:

02-25 07:22:30.059  2437  2437 E AndroidRuntime: FATAL EXCEPTION: main
02-25 07:22:30.059  2437  2437 E AndroidRuntime: Process: com.android.launcher3, PID: 2437
02-25 07:22:30.059  2437  2437 E AndroidRuntime: java.lang.NullPointerException: Attempt to invoke virtual method 'android.view.ViewRootImpl android.view.View.getViewRootImpl()' on a null object reference
02-25 07:22:30.059  2437  2437 E AndroidRuntime: 	at android.view.inputmethod.InputMethodManager.startInputInner(InputMethodManager.java:2274)
02-25 07:22:30.059  2437  2437 E AndroidRuntime: 	at android.view.inputmethod.InputMethodManager$H.handleMessage(InputMethodManager.java:963)
02-25 07:22:30.059  2437  2437 E AndroidRuntime: 	at android.os.Handler.dispatchMessage(Handler.java:106)
02-25 07:22:30.059  2437  2437 E AndroidRuntime: 	at android.os.Looper.loopOnce(Looper.java:201)
02-25 07:22:30.059  2437  2437 E AndroidRuntime: 	at android.os.Looper.loop(Looper.java:288)
02-25 07:22:30.059  2437  2437 E AndroidRuntime: 	at android.app.ActivityThread.main(ActivityThread.java:7921)
02-25 07:22:30.059  2437  2437 E AndroidRuntime: 	at java.lang.reflect.Method.invoke(Native Method)
02-25 07:22:30.059  2437  2437 E AndroidRuntime: 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
02-25 07:22:30.059  2437  2437 E AndroidRuntime: 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:855)

Signed-off-by: minaripenguin <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Instead of throwing IllegalStateException just fix the problem instead.

Fixes: Occasional force close from second qs page
Change-Id: Ie9573a53f62691dac63480655983f22dcb8ff7af
Signed-off-by: mydongistiny <[email protected]>
Signed-off-by: LordShenron <[email protected]>
Because it is not safe to use ArrayList and ArraySet
under concurrency, it is necessary to use synchronization,
otherwise ConcurrentModificationException may be thrown

Bug:254179957
Test: N/A

Change-Id: I991970f00903b6dfebd75bc42dbb2580d756c602
Signed-off-by: liyong <[email protected]>
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: LordShenron <[email protected]>
jyescas and others added 26 commits May 12, 2024 17:40
This change makes sure that the consecutive delimiters at the
beginning of the just refilled buffer are consumed.

Previously, if there were consecutive delimiters at the beginning
of the just refilled buffer, they were not being consumed and were
causing parsing errors. This was also causing fatal exceptions.

```
04-22 12:37:53.077   607   707 E AndroidRuntime: *** FATAL EXCEPTION IN SYSTEM PROCESS: CachedAppOptimizerThread
04-22 12:37:53.077   607   707 E AndroidRuntime: java.lang.NumberFormatException: invalid long: READ
04-22 12:37:53.077   607   707 E AndroidRuntime: 	at com.android.internal.util.ProcFileReader.invalidLong(ProcFileReader.java:238)
04-22 12:37:53.077   607   707 E AndroidRuntime: 	at com.android.internal.util.ProcFileReader.parseAndConsumeLong(ProcFileReader.java:220)
04-22 12:37:53.077   607   707 E AndroidRuntime: 	at com.android.internal.util.ProcFileReader.nextLong(ProcFileReader.java:183)
04-22 12:37:53.077   607   707 E AndroidRuntime: 	at com.android.internal.util.ProcFileReader.nextLong(ProcFileReader.java:172)
04-22 12:37:53.077   607   707 E AndroidRuntime: 	at com.android.internal.util.ProcFileReader.nextInt(ProcFileReader.java:246)
04-22 12:37:53.077   607   707 E AndroidRuntime: 	at com.android.internal.os.ProcLocksReader.handleBlockingFileLocks(ProcLocksReader.java:111)
04-22 12:37:53.077   607   707 E AndroidRuntime: 	at com.android.server.am.CachedAppOptimizer$FreezeHandler.handleMessage(CachedAppOptimizer.java:2267)
04-22 12:37:53.077   607   707 E AndroidRuntime: 	at android.os.Handler.dispatchMessage(Handler.java:107)
04-22 12
```

Bug: 336208153
Bug: 338416108
Bug: 331988929
Test: atest --host-unit-test-only -c  ProcFileReaderTest
Google: 3073613
Change-Id: I12813fa204d53d53687ffe6e8e3c0a85595b7e25
…EY_MISSING

Bug: 337174893
Test: manual, connect to paired but just factory reseted headset
Flag: EXEMPT, xml file change
Change-Id: I3ff5698f896b7b8caee02bf2d5f7b398c6fad418
We found that the light sensor and cached auto brightness value were
invalidated when entering doze mode, regardless of the value of
config_allowAutoBrightnessWhileDozing.

AutomaticBrightnessStrategy already checks for dozing state, using it
to decide the `state` arg to AutomaticBrightnessController.configure();
the controller should not second-guess it.

Bug: 304339170
Test: manual; go into Doze in a dark environment
Change-Id: I85d6b5872d475b164aa469ab18bb7034b073161e
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
After broadcasting a new USB_STATE that includes MTP or PTP, we now
also bind to the MtpService. This elevates the MTP process
(android.process.media) to a foreground process (without adding an
extra notification). I.e. it inherits the foreground status from
UsbDeviceManager. The MTP process needs this to prevent it from
getting frozen by the ActivityManager while the connection is active.

Before this change the error could easily be reproduced:
1. Connect and disconnect a USB device providing
   "mClass=255,mSubclass=255,mProtocol=0" (many docking stations with a
   LAN device do this).
2. Connect the phone to your PC.
3. Set "File transfer" mode.
4. File access via MTP on your PC should work for about 10 seconds but
   then stops responding.

The first step makes com.android.mtp show UI, making ActivityManager
push it down to cached state more eagerly.

This error did in practice not show up before Android U since:
* The MTP process was kept warm (unfrozen) by periodically receiving
  the BATTERY_CHANGED intent (most often at least once per minute).
* The freeze timeout, DEFAULT_FREEZER_DEBOUNCE_TIMEOUT, was 600 seconds
  (changed to 10 seconds in Android U).
* The broadcast queue did deliver BATTERY_CHANGED intents also to
  frozen processes even with the option deliveryGroupPolicy=1
  (DELIVERY_GROUP_POLICY_MOST_RECENT). This is not the case in
  Android U with the new broadcast queue (BroadcastQueueModernImpl).

Bug: 301232841
Test: manual, according to steps listed above
Change-Id: Ic570381dd55152dbecb338372692bfc5caf5d7de
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
When deciding whether brightness change should be slow or fast we use
"hasAppliedAutoBrightness()". However, this value is always reset to
true by "setAutoBrightnessState()". The correct way is to save
"hasAppliedAutoBrightness()" before "setAutoBrightnessState()" and use
the saved value.

Bug: 304313764
Test: manual
Change-Id: I0ace4523813099af266a37a984cbd785b462403e
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
Signed-off-by: minaripenguin <[email protected]>
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
This change fixes https://issuetracker.google.com/issues/335003907

This change replaces
GetPrimitiveArrayCritical/ReleasePrimitiveArrayCritical with
GetByteArrayRegion/SetByteArrayRegion.

This change is needed because the critical API variants are not
appropriate for this context. See the JNI docs:

After calling GetPrimitiveArrayCritical, the native code should not run
for an extended period of time before it calls
ReleasePrimitiveArrayCritical.

In bulkTransfer, we call usb_device_bulk_transfer between
GetPrimitiveArrayCritical/ReleasePrimitiveArrayCritical.
usb_device_bulk_transfer is a blocking call so this can lead to GC
hangs.

Instead, we allocate a new buffer with malloc and then call
GetByteArrayRegion/SetByteArrayRegion before/after
usb_device_bulk_transfer, depending on the direction of the endpoint.

Test: I ran UsbTests. I also sent a "TEST UNIT READY" to a USB drive and
verified the response.

Change-Id: I5c82aac3fe99e1f11044f20091bf3a940bc8f6a7
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
 PersistableBundle store the data use the ArrayMap,so the null key may put into the map.So we should make a judgment to prevent the systemserver crash.

Bug: 337781283
Change-Id: I9ce8c86aa9c9142e3b413bdb1eaaf51099592e72
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
Bug: 337070728
Change-Id: I63707b4f9fe3bcd705e98ac2e69a5101d092de76
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
Bug: 337711090
Change-Id: If20c27c704e3db2b5960af1fe71066e1cbc6eaf4
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
mLatch.notifyAll() will not help wake the thread which is blocked by
mLatch.await, instead, it will cause system server crash.

So, mark state as failed if any package is failed to be deleted, and let
mLatch.countDown work until it comes to 0 and wake the thread blocked by
await.

Change-Id: Ida1c3c128ddef376aaaacda1358717d0d4e9ca8b
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
…ess buckets

Note: This is an upstream from Meta.

Problem:
initializeDefaultsForSystemApps() is not informing listeners of change in uid idleness.
This results in stale idleness values in components such as the network policy manager, which
has listeners to uid changes.

Solution:
call maybeInformListeners() to make sure listeners are informed of the changes

Test: Manual test + New Unit Test

 Manual Test on Meta device:
  Tested on a race condition where network policy manager queried for the idleness of a uid before
  initializeDefaultsForSystemApps() is called.

  Before the fix, network policy manager got STANDBY_BUCKET_NEVER as idleness
  and never got informed of change in uid when initializeDefaultsForSystemApps()
  sets uid idleness bucket to STANDBY_BUCKET_ACTIVE

  After the fix, network policy manager's listener was able to be informed of this change in uid.

Change-Id: Ic6b665eea4a07d133a85789bce1a1af514e7ee96
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
Using of 'instanceof' cannot determine whether 'nextParent' is non-null during the first loop in do-while, so when 'focused.getParent()' outside the loop returns null, the loop body will trigger a null pointer exception. Therefore, the conditional judgment needs to be covered in every loop.

Test:UAT
Bug: 336205346

Change-Id: I78374410d928009ea9b99630cae2fbea24ba8ab6
Signed-off-by: zhuning3 <[email protected]>
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
When the same SparseMappingTable is referenced by multiple objects as mParent,
when SparseMappingTable executes reset(), it also executes resetTable() on the Table object it holds.
wqThis is used to ensure the correspondence between the two arrays mLongs and mTable.

Test: atest SparseMappingTableTest
Bug: 336154577

Change-Id: I4614a5a38aad4e1316dea09db13407688f6e03d8
Signed-off-by: hupeng3 <[email protected]>
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
BackgroundThread is being overused in many scenarios, such as ams and
pms, among others. If it is used excessively in other areas, it may
cause a block in LauncherAppService.

We have experienced several internal failures, such as opening a cloned
account and then installing an application. After the application was
installed, the BackgroundThread became too busy for various reasons, and
LauncherAppService was unable to notify the launcher in time, resulting
in a very slow display of desktop icons. This delay took dozens of
seconds to occur after the application was installed.

To avoid such issues, it is recommended to open a new thread for
LauncherAppService and not share the system's BackgroundThread.

Change-Id: Ib85f973686a0586507a6f76110e3f879be9641b7
Signed-off-by: Jia Jia <[email protected]>
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
…od .

Bug: 338661733
Change-Id: I04ce021ef7838f0574875359daeb8c765d5f6a09
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
04-05 20:55:11.791  1923  1923 E AndroidRuntime: Caused by: java.util.ConcurrentModificationException
04-05 20:55:11.791  1923  1923 E AndroidRuntime: 	at android.util.ArrayMap.put(ArrayMap.java:626)
04-05 20:55:11.791  1923  1923 E AndroidRuntime: 	at com.android.systemui.tuner.TunerServiceImpl.addTunable(TunerServiceImpl.java:342)
04-05 20:55:11.791  1923  1923 E AndroidRuntime: 	at com.android.systemui.tuner.TunerServiceImpl.addTunable(TunerServiceImpl.java:314)
04-05 20:55:11.791  1923  1923 E AndroidRuntime: 	at com.android.systemui.volume.VolumeDialogComponent.<init>(VolumeDialogComponent.java:137)

Signed-off-by: minaripenguin <[email protected]>
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
…thod return var is null.

Bug: 338907035
Change-Id: I875afd3eeeba24b731e68161060285dcaf3f41aa
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: ReveRTX <[email protected]>
This is an addon for GameSpace for broadcasting about game start/stop.
Additionally, it also has special option for suppressing fullscreen
intent like incoming call.

This also squash commits:
- SystemUI: Improve GameSpace lifecycle and broadcast handling
- GameSpaceManager: Handle various case against "locking screen" scenario
- GameSpace: Fix issue with multiple users [neobuddy89]
- SystemUI: Update gamespace for A13 [Genkzsz11]
- GameSpaceManager: restrict broadcast to MANAGE_GAME_MODE holders only
- GameSpaceManager: Update TaskStack listener usage
- GameSpaceManager: Updated for A14 QPR1 [neobuddy89]

Signed-off-by: Pranav Vashi <[email protected]>
GTS don't allow any app to have the android.permission.WRITE_DEVICE_CONFIG permission
i.e we can't modify device_config property from our game space.

So write the property to our custom Settings.Secure.GAME_OVERLAY.
Then observe the value changes and set the property to the
device_config from here.

Since GameManagerService is a part of system_server,
GTS will be happy.

Change-Id: I0ebbcd6188411a583fa53904e6153482f342a03e
Signed-off-by: Pranav Vashi <[email protected]>
Media files take up a lot of spce. Deleting them cuts the size of frameworks
almost in half.

Change-Id: Ia540bdd22b5de3c141718107df62c3dc491e6cc7
Signed-off-by: Evan Anderson <[email protected]>
Signed-off-by: Anushek Prasal <[email protected]>
Signed-off-by: SirRGB <[email protected]>
PureNexus Edits:
*also remove the makefiles as well since they are useless without the videos

Change-Id: I7be80ddfbe5cd5502fbd77467216a5b67ac620c4
Signed-off-by: Anushek Prasal <[email protected]>
Signed-off-by: SirRGB <[email protected]>
This reverts commit 575a868.

Change-Id: I84bc2f0ac1df16f2bc4338b8f9300c35d309b56c
Signed-off-by: SirRGB <[email protected]>
Change-Id: I84303ee26d0232e471f44ae6eff6e41a2210e42e
Signed-off-by: spezi77 <[email protected]>
Signed-off-by: SirRGB <[email protected]>
Change-Id: Ib4ea0a5072c33fac4209aa2c63f4acbec282b0df
Signed-off-by: Pranav Vashi <[email protected]>
Signed-off-by: SirRGB <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet