-
Notifications
You must be signed in to change notification settings - Fork 28
Sysserv and Exposures #5
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
Open
JagravNaik
wants to merge
45
commits into
AOSP-CAF:o-mr1
Choose a base branch
from
JagravNaik:o-mr1
base: o-mr1
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Change-Id: Ic261d9f48e32f63f1bdafc0078ae933d5a2a1304
bug: 67864232 Change-Id: I6eb51be839df12317273830521c70a4ed80f9496 (cherry picked from commit e6b9853)
Bug:33846679 Test: I6ab6cb7a3b3151641a9f9b02b0bfc484e0a4524b This matches the old behavior, prior to switching from Java allocated pixel memory to native allocations (b/27762775). It also better matches what has happened - we ran out of memory. (Better than the current behavior - NullPointerException in the Java code, or an alternative solution of returning null, which would likely result in NPEs in the calling code.) Merged-In: I3958ed1106ac94fb1d3f30e044b620d984875211 Change-Id: I3958ed1106ac94fb1d3f30e044b620d984875211 (cherry picked from commit 5906a48)
Malformed authority segments can currently cause the parser to produce a hostname that doesn't match the hostname produced by the WHATWG URL parsing algorithm* used by browsers, which means that a URL could be seen as having a "safe" host when checked by an Android app but actually visit a different host when passed to a browser. The WHATWG URL parsing algorithm always produces a hostname based on the last @ in the authority segment, so we do the same. * https://url.spec.whatwg.org/#authority-state resets the "buffer", which is being used to build up the host name, each time an @ is found, so it has the effect of using the content between the final @ and the end of the authority section as the hostname. Bug: 68341964 Test: vogar android.net.UriTest (on NYC branch) Test: cts -m CtsNetTestCases (on NYC branch) Change-Id: Idca79f35a886de042c94d6ab66787c2e98ac8376 (cherry picked from commit cd6228d)
…87478, 3287479, 3287480, 3287517, 3287518, 3287537, 3287538, 3287539, 3287540, 3287481, 3287482, 3287483, 3287484, 3287485, 3287486, 3287487, 3287488, 3287359, 3287459, 3287360, 3287361, 3287362, 3287363, 3287364, 3287365, 3287366, 3287367, 3287489, 3287490, 3287491, 3287557, 3287577, 3287558, 3287492, 3287493, 3287597, 3287617, 3286980, 3287460, 3287494] into oc-m3-release Change-Id: Ie0cb7656011a153701e11f46f9657074178468c9
Synthetic password is double encrypted by both a random auth-bound keymaster
key and a secret derived from user password. In order to avoid a password
verification oracle without rate limiting, synthetic password needs to be
encrypted by the derived secret first, and then the auth-bound key. This
change corrects the order of encryptions, as well as adds an upgrade path to
refresh existing credentials.
Test: Running an old build with existing password, flash to new build,
verify the device unlocks successfully.
Bug: 68694819
Change-Id: Ifdaa01f3f4ddd5bb3f3d808d38f440ced729034f
Merged-In: Ifdaa01f3f4ddd5bb3f3d808d38f440ced729034f
(cherry picked from commit 78acfe7)
…17360, 3417361, 3417362, 3417363, 3416401, 3416402, 3416403, 3417778, 3417798, 3416662, 3416663, 3417799, 3417800, 3417364] into oc-m5-release Change-Id: I61799fada6cf751fc71ed0e43294b02e59b35b4a
Bug: 69383160 Test: cts-tradefed run cts -m CtsAppSecurityHostTestCases -t android.appsecurity.cts.OverlayHostTest Change-Id: I930c11716317cff1ec485a3943d1f22f07a423df Merged-In: I930c11716317cff1ec485a3943d1f22f07a423df (cherry picked from commit 80609e5)
Change-Id: Id7806920198a89e7d53ea9f973386179bfa3fe48
forward port to oreo This commit allows a themer to overlay a boolean value in config.xml to disable dynamic colors applied to the app title and app icon of each notification. PS6: Separate app title and small icon colors for more flexibility. Expose hardcoded sender text name. Oreo edits: follow bool for OverflowNumbers as well before: https://i.imgur.com/JqeLFD1.jpg after: https://i.imgur.com/GPoM0zo.jpg Change-Id: I3c7828118991ec4fc616011caf073c81f75428b4
forward ported to oreo OMS7 introduced this fine piece of code: https://github.com/SubstratumResources/platform_frameworks_base/blob/n-oms7/core/java/android/app/ResourcesManager.java#L897..#L904 // Resources.getSystem Resources are created on request and aren't tracked by // mResourceReferences. // // If overlays targeting "android" are to be used, we must create the system // resources regardless of whether they already exist, since otherwise the // information on what overlays to use would be lost. This is wasteful for most // applications, so limit this operation to the system user only. (This means // Resources.getSystem() will *not* use overlays for applications.) Replaced deprecated Resources.getSystem() with compatible method. Change-Id: I02efe27de3cc7067552964ffbaf079f9e9b5bc3e
…otifications * added a seperate color resource, so this way themer can use diff color (ex: white) for ambient and accent for normal notification. Change-Id: I65a518d9433558004289826abbf25dc374dd7ec7 Signed-off-by: Adarsh-MR <adarshmr1998@gmail.com>
forward ported and adapted to oreo-mr1 changes. This was using the volume panel drawables used also on volume panel. So with this commit themers can give different icon for either QS and volume panel expand icon. Change-Id: Ice8d8a520b9b22ba773cceb885e11c8a4bbf6d5f
Forward ported and adapted to oreo LightNavigationBar PS1: Layers Commit by setiawanjimmy Rewrite of commit by: KreAch3R Original commit by: Dave Kover Distilled from: CyanogenMod/android_frameworks_base@05ce0a6 Change-Id: I7969e952d7e08f1d12e89291512312421585b70f Signed-off-by: Adarsh-MR <adarshmr1998@gmail.com>
This is needed for the power notifications switchbar in SystemUI Tuner, amongst other things. Change-Id: I86f04840c2be46519509556b8d0061cefe26f631
It's the same problem as the booleans again. This time, it affected the adb "N" icon in the statusbar. This commit should fix this. After: http://i.imgur.com/RPh6WKK.jpg Previous commits on the same matter: OMS7 introduced this fine piece of code: https://github.com/SubstratumResources/platform_frameworks_base/blob/n-oms7/core/java/android/app/ResourcesManager.java#L897..#L904 // Resources.getSystem Resources are created on request and aren't tracked by // mResourceReferences. // // If overlays targeting "android" are to be used, we must create the system // resources regardless of whether they already exist, since otherwise the // information on what overlays to use would be lost. This is wasteful for most // applications, so limit this operation to the system user only. (This means // Resources.getSystem() will *not* use overlays for applications.) Replaced deprecated Resources.getSystem() with compatible method. Change-Id: Ibab2ce1571360a9e03043d1bf3144c89e54e1947
Change-Id: I84d3e57fe8c42d79ff6e0b6502cb6cd49aaaf91b Signed-off-by: Bryan Owens <djbryan3540@gmail.com>
Keep in mind that Background color already gets 0.5f opacity in the GlobalScreenshot class (BACKGROUND_ALPHA with some math). For the flash you can use any color value instead (full opacity or not). Change-Id: I6ac51358f7f5278d742849ebc49f580554f069cd
Credit Bryan Owens for giving guidance on what was needed. Forward ported to oreo Change-Id: I862008d1b433666333a4c77904b44a05e70472a2
Change-Id: Id4a078cdbc944fa0c0736103045a0382d49ecb80
Change-Id: I328ad1ffba231353254a8122b899bd71b5d1aa68 Signed-off-by: Adarsh-MR <adarshmr1998@gmail.com>
commit dbbd5e70cc65002df41561474b03362022dd6716
Author: Dave Kover <dkover@cyngn.com>
Date: Wed Feb 18 16:11:14 2015 -0800
Themes: Expose resolver hardcoded colors
Expose background colors of the resolver list.
Change-Id: I3a0a460c5ffe0f5057b3b9ec92faa7a3e09c9e01
commit 0343eb126f3901a3857791137f74fa805bb9d75c
Author: Thyrus11 <thyrus11@gmail.com>
Date: Sat Feb 21 07:19:42 2015 +0100
Themes: Make resolver list fully themeable
Follow-up on commit cc9e3b8fcba95b911d1cda36f7770c410058aa8b.
Change-Id: I3f006a1157db9d0b151a4fe8edf50e7edc7a0b9f
commit c7d973809488b801e8c708d740009f1233bb762e
Author: Nicholas Chum <nicholaschum@gmail.com>
Date: Sun Nov 8 05:27:28 2015 -0500
Themes: Allow Resolver List BG to be fully themed
We are able to trace the activity of the new resolver/chooser through
different
methods, thus leading us to the Java file:
\com\android\internal\app\ChooserActivity.java
Here we see that the exposed "chooser_service_row_background_color" is
available, but not the rest of the activity, so we look into R.layout's,
and we
find chooser_grid to be the only one containing hardcoded
"@color/white" values (as this is framework, we assume this is also
known as
"android:color/white" to themers).
Expose all "@color/white" values from this file to resolver_list_bg.
Change-Id: I286d92b5d1f672c8adb3c0af1951793521536d90
Change-Id: Iec7951147bbbc99aee6b06ae50c1acc7b9c01a7f
Signed-off-by: Adarsh-MR <adarshmr1998@gmail.com>
This allows the immersive mode help tooltip to be themed completely by removing hardcoded framework calls. Let the themer decide what they want the colors to be. Change-Id: Ia0927fda5e44a3ce8ef699cb018bea9b9e7ace62 Signed-off-by: Adarsh-MR <adarshmr1998@gmail.com>
This removes the forced @android:color/black tint on the permission icons during app sideload through PackageInstaller. These icons are able to be changed through framework XMLs, but this line forces a black tint (invisible on dark themes) on the icons. Let's remove this. Change-Id: I31eb5021a6d297997dbba156f98cbf47f2102b6c Signed-off-by: Adarsh-MR <adarshmr1998@gmail.com>
PS2: More descriptive color name nathanchance edit: updated for 7.1 Nougat camcory edit: update to 8.1 Change-Id: I9867ca26e7d9dacad37d2b70180a98fede0fb0e8 Signed-off-by: Adarsh-MR <adarshmr1998@gmail.com>
Change-Id: Iae67c226a197d25e82f65f2a259894756d3cad1f Signed-off-by: Simao Gomes Viana <xdevs23@outlook.com>
e.g. on landscape, or portrait when using setDivisionView in the GlobalActionsDialog Change-Id: Ie73ffcc189791192b1f6d5d2c420d2254878723d
PS: i suggest this sexy red: #ff8e0101 https://s26.postimg.org/5tf5fh8eh/jessica-bob-_Chi-ha-incastrato-_Roger-_Rabbit-.jpg :D Change-Id: I39ac0b381f0762752cb8b2a73e96c90da491c0ab
Change-Id: Iaaa6d54673e11b1c4243e754a6fb088cae534976 Signed-off-by: Adarsh-MR <adarshmr1998@gmail.com> Signed-off-by: Harsh Shandilya <harsh@prjkt.io>
…grounds Overlay config_useDarkBgNotificationIconTextTinting to enable dark bg tinting (Android 8.1 version) Change-Id: I1c6043abde22c64c2f4ff1287ca209a25e4980c4
* tag 'android-8.1.0_r18': OMS: Only allow trusted overlays to be registered. Swap the order of synthetic password wrapping Adjust Uri host parsing to use last instead of first @. Throw OOME if Bitmap.nativeCreate fails mtp: fix double free of thumbnail data Change-Id: Ib5682189200b8932f9a4fe4b7f867e5d25eea3d8
Note: Custom boot animation will not work on encrypted device Change-Id: I3b01953d0a69c033a98c0af49ee986b21652b725 Signed-off-by: Ivan Iskandar <ivan@prjkt.io>
This commit checks whether there is a preexisting file in the themed directory "/data/system/theme/audio/ui/" and if so, change the base file paths for the sound. If the file does not exist in the theme directory, then use the default sounds. At the current moment, this will require a soft reboot to work. Change-Id: I7666c2bd259443ccec442bf6059786bea3dc069e
Change-Id: I10993906384cef97e10063399f3447c78d1959df Signed-off-by: Ivan Iskandar <ivan@prjkt.io> Signed-off-by: Harsh Shandilya <msfjarvis@gmail.com>
This commit allows the framework to handle the filtering of the overlays found for OMS. Change-Id: I7646115e8f73494d726728fac58cc47aafd69d5d
The intent received by substratum and it will disable all enabled overlays. Change-Id: Ifabd57c2ea71ca93ecc2959ce09ccde3e91782dd Signed-off-by: Ivan Iskandar <ivan@prjkt.io>
Following the way "safe mode" was coded, you can now long press "volume up" during boot to automatically disable all overlays (from the current system/owner user). This should come in handy as a global "reset" mechanism. Example output: 03-12 03:22:07.090 678 678 D TEST : Disabling overlay android.GalaxyEvolution 03-12 03:22:07.176 678 678 D TEST : Disabling overlay com.android.launcher3.GalaxyEvolution 03-12 03:22:07.267 678 678 D TEST : Disabling overlay com.android.server.telecom.GalaxyEvolution Caveats: SystemServer seems to have already loaded a context based on the overlays by the time "disableOverlays" is finished, so SystemUI turns up themed, even if the overlay is correctly disabled. In the case of a user using this reset mode to "fix" a SystemUI FC, this means that the user will experience one more FC after boot (which then would trigger a SystemUI restart, loading the default theme this time, and thus ending the FC loop). Change-Id: I64fb769ea175d37a90a0804f916926b63e5b93ca
Themes are using /data/system/theme/ to push some files like LowBattery.ogg (audio notification) When the device battery trigger the low battery state, the sound is not played due to StrictMode and SystemUI is crashing. So we need that StrictMode authorize files under /system OR /data/system/theme Logcat of the issue: E AndroidRuntime: Caused by: android.os.FileUriExposedException: file:///data/system/theme/audio/ui/LowBattery.ogg exposed beyond app through Notification.sound E AndroidRuntime: at android.os.StrictMode.onFileUriExposed(StrictMode.java:1799) E AndroidRuntime: at android.net.Uri.checkFileUriExposed(Uri.java:2346) E AndroidRuntime: at android.app.NotificationManager.notifyAsUser(NotificationManager.java:300) Change-Id: I154dc4280de8eaf891772a9632283e9f547f5718 (cherry picked from commit 838f646)
This will also eliminates the need to reenable overlay after updating. Change-Id: Iac620ba4302e23822639ca7acabbda28f2fb8030 Signed-off-by: Ivan Iskandar <ivan@prjkt.io>
Change-Id: I8c9f331ca7e4c68c35ea13bed587775ee4abeb30 Signed-off-by: Ivan Iskandar <ivan@prjkt.io> Signed-off-by: Nicholas Chum <nicholas@prjkt.io> Signed-off-by: Harsh Shandilya <harsh@prjkt.io>
Complete the current feature of overlay change assets reloads implemented with 987f193 which miss the notification and other small bits in SystemUI to be reloaded. However, this commit will not fully eliminate the need of SystemUI restart when changing framework or SystemUI overlay since there is more undiscovered bits that can't/doesn't get reloaded for some kind of overlay. Change-Id: I451be5e6c65bf938706ae2a77aad1cef653c5ba5 Signed-off-by: Ivan Iskandar <ivan@prjkt.io>
* commit: SubstratumResources/platform_frameworks_base@79d56d1 added ability to seperate expand/collapse drawable from volume panel but animation are still pointing to volume drawable * Point it to right drawable to seperate it fully Change-Id: Ic959bcdd24eb2f56b3de66369cc9907ac1978b9f Signed-off-by: Adarsh-MR <adarshmr1998@gmail.com>
* changes: SystemUI: Reload more assets on overlay change base: Introduce SubstratumService OMS: Add back overlay modified callback implementation OMS: StrictMode and files under /data/system/theme/ Hold "volume up" during boot to disable all overlays ThemeSafety: Introduce App Crash Intent ApplicationsState: add filter for Substratum overlays [2/2] Extras: Add dynamic theme fonts support for O Extras: Add dynamic theme sound effects support Extras: Add dynamic theme shutdown and boot animation support
This was added with commit 7a722dc but this appears to be an incomplete attempt at doing what it should, because FLAG_OVERLAY_TRUSTED is never assigned, only checked. While we wait for Google to realise this and push a fuller commit for us to mitigate against, this bandaid does the trick. [harsh@prjkt.io: Add commit log] Change-Id: Ic8668f4d2769b9b5daeea56cd6562674258a10b1 Signed-off-by: Harsh Shandilya <harsh@prjkt.io>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.