summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2021-10-06[automerger skipped] Merge "Update constructors for IRadioData classes" am: ↵temp_sam_202323961Sarah Chin
fa565ffc48 -s ours am skip reason: Merged-In I8480df1164548dde491c6ae6251bbc2725c3eadf with SHA-1 80b8c7f128 is already in history Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1840653 Change-Id: Iba1443da42161f4a41830081f2e1985b30444cc0
2021-10-06Merge "Update constructors for IRadioData classes"Sarah Chin
2021-10-06Merge "Update constructors for IRadioData classes" into stage-aosp-masterSarah Chin
2021-10-06Merge "Mark Soong maintainers as a last resort suggestion" am: 2885623021Jeff Sharkey
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1846454 Change-Id: I6da8604f80a290bca170bbdf1246bae1efa4cef3
2021-10-06Merge "Mark Soong maintainers as a last resort suggestion"Jeff Sharkey
2021-10-06Mark Soong maintainers as a last resort suggestionLiz Kammer
Soong having OWNERS in frameworks/base is useful to enable LSCs; however, they shouldn't appear in review suggestions except as a fallback. Test: n/a Change-Id: I86618b66e51893908478def9fc455a3dfde146f0
2021-10-06Merge "Avoid full unparcelling where possible in Bundle" am: 3ad4ecc296Treehugger Robot
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1842226 Change-Id: I0aef93d99d83f20b22753e8e1d41cfb5c4b5cdce
2021-10-06Merge "Avoid full unparcelling where possible in Bundle"Treehugger Robot
2021-10-06Update constructors for IRadioData classesSarah Chin
Test: atest FrameworksTelephonyTests Bug: 198331673 Change-Id: I8480df1164548dde491c6ae6251bbc2725c3eadf Merged-In: I8480df1164548dde491c6ae6251bbc2725c3eadf
2021-10-06Update constructors for IRadioData classesSarah Chin
Test: atest FrameworksTelephonyTests Bug: 198331673 Change-Id: I8480df1164548dde491c6ae6251bbc2725c3eadf Merged-In: I8480df1164548dde491c6ae6251bbc2725c3eadf
2021-10-06Merge "Change defusing for lazy bundles" am: fdd1771a4aBernardo Rufino
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1842142 Change-Id: Ia156038712d6a25acfea8aaa6ac1d257afa95345
2021-10-06Merge "Change defusing for lazy bundles"Bernardo Rufino
2021-10-05Merge "Fixing a build breakage caused by an incorrect link. Version code for ↵Treehugger Robot
Android T doesn't exist yet, removing the link." am: 6a0afe1846 Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1845234 Change-Id: I8249e3a7426ee9f3d2d22ba6ae093d217980acc2
2021-10-05Merge "Fixing a build breakage caused by an incorrect link. Version code for ↵Treehugger Robot
Android T doesn't exist yet, removing the link."
2021-10-05Merge "media: use MediaProperties.resolution_limit_32bit()" am: b721e9b48dTreehugger Robot
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1840914 Change-Id: Ib8b212601771c279a90eb8c29fb30bd1e5ecd0a3
2021-10-05Merge "Use AttributionSource Builder" into stage-aosp-masterTreeHugger Robot
2021-10-05Merge "media: use MediaProperties.resolution_limit_32bit()"Treehugger Robot
2021-10-05Fixing a build breakage caused by an incorrect link. Version code forJacob Hobbie
Android T doesn't exist yet, removing the link. Test: presubmit, comments only, should just be "is build fixed" Bug: 202151792 Change-Id: I2123f804aaed39eb7d31f25058c59a3c1df4d7d1
2021-10-05Merge "Revert "Revert "Cherrypicking flags and override methods for all..."" ↵Alex Johnston
am: 5c35d3bce6 Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1845059 Change-Id: I90e8a4d567fe2de9ab765806fc7bd3aba4fe0845
2021-10-05Merge "Revert "Revert "Cherrypicking flags and override methods for all...""Alex Johnston
2021-10-05Revert "Revert "Cherrypicking flags and override methods for all..."Alex Johnston
Revert "Revert "Required change for adding overload method for r..." Revert submission 1842720-revert-1836473-receiverChangesinAOSP-FWJQDTADTI Reason for revert: DroidMonitor-triggered revert due to breakage, bug b/202144225 Reverted Changes: I03409ea39:Revert "Required change for adding overload method... I583a11d2a:Revert "Cherrypicking flags and override methods f... Change-Id: I5cdf9c74d51a259fdaa43dfdc9423d22145eac60
2021-10-05Merge "Fix SystemAudioAutoInitiationAction not started" am: e6bebf94edTreehugger Robot
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1772932 Change-Id: Ifba3a65691d817ff0a4167efb225a002948ac493
2021-10-05Merge "Fix SystemAudioAutoInitiationAction not started"Treehugger Robot
2021-10-05Avoid full unparcelling where possible in BundleBernardo Rufino
and put a warning where it's not possible. This is to make sure we know when to touch bundles provided by apps. In this CL: * deepCopy() now doesn't deserialize lazy objects before copying. It doesn't copy them either, it merely passes them into the new map as the same reference. This works because we implemented fine-grained locking into each lazy value, so concurrent access won't be a problem. Furthermore, LazyValue caches the deserialized object, so we'd still honor the contract of deepCopy() that "Other types of objects (such as Parcelable or Serializable) are referenced as-is and not copied in any way". I had to perform one extra check in the synchronized block in LazyValue to guarantee this (double-checked locking), I explain that in the comments. * Removed filterValues() and codepaths that used it. This was created with the purpose of removing items whose classes weren't available to the system to prevent crashes coming from full deserialization. This is not a concern anymore with lazy bundle, hence we can remove the codepaths altogether (see email for more details). * Put warnings in javadoc of getMap() and PeristableBundle(). Test: Boots Test: atest -d android.os.cts.ParcelTest android.os.cts.BundleTest android.os.BundleTest android.os.ParcelTest Bug: 195622897 Change-Id: I14bb6a7874814f42cbcc6b5fd372c42752aa74c8
2021-10-05Change defusing for lazy bundlesBernardo Rufino
With lazy bundle, exceptions thrown by deserialization of custom items moved from being thrown in initializeFromParcelLocked() (when the bundle is first touched) to being thrown in getValueAt() (whenever the item is retrieved), which means they were escaping the defuse logic that caught those exceptions. So, now also catching these exceptions in getValueAt(), however, here we can be much less drastic, instead of erasing the bundle, we can simply remove the bad element. This also means we don't need to log wtf in initializeFromParcelLocked() if sShouldDefuse = true but the bundle is not marked as defusable, since touching the bundle doesn't carry the same consequences as before go/lazy-bundle. So, I moved that log to unparcel(itemwise = true), ie. whenever the entire bundle is deserialized on purpose. Now, 2 types of defusing can happen: 1. If the (custom) item we're retrieving caused the exception, we'll remove the item and return null. 2. If the exception was raised during partial deserialization, that is, during the read of the map and its basic types (while skipping custom types), the map will be left empty. I believe only manually written parcels would cause this type of exception and if an exception is raised here it will also be raised in the final destination too, afaict. Because of this, we can now touch app-provided bundles without fear of clobbering the data on its way to the final destination. Following conversation on previous CL, narrowed exception raised in case of unknown type to be BadParcelableException. Test: atest -d android.os.cts.ParcelTest android.os.cts.BundleTest android.os.BundleTest android.os.ParcelTest Bug: 195622897 Change-Id: I4746140f63482a9ea475aac25897a447737393e4
2021-10-05Merge "Round 3: Fix flakiness of testNativeCrash" am: 5757f89ddcTreehugger Robot
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1843421 Change-Id: I5b7038ac40d723829606c8d0a76903ad104cb94b
2021-10-05Merge "Round 3: Fix flakiness of testNativeCrash"Treehugger Robot
2021-10-05Merge "Fix the device summary shown twice issue" am: 0a80b6a3bcTreehugger Robot
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1842101 Change-Id: Id343c16720f499514a3832cc7f92fb5c25beadaf
2021-10-05Merge "Fix the device summary shown twice issue"Treehugger Robot
2021-10-05Merge "Revert "Cherrypicking flags and override methods for allowing dy..."" ↵Jacky Kao
am: 98ff8ecdaa Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1842721 Change-Id: Ieaed4fcc2fa97db551a70f4a3076b4237acfb8ab
2021-10-05Merge "Revert "Cherrypicking flags and override methods for allowing dy...""Jacky Kao
2021-10-05Revert "Cherrypicking flags and override methods for allowing dy..."Jacky Kao
Revert "Required change for adding overload method for registerR..." Revert submission 1836473-receiverChangesinAOSP Reason for revert: <INSERT REASONING HERE> Reverted Changes:Making build break I906ce9eb0:Required change for adding overload method for reg... I9a16f566c:Cherrypicking flags and override methods for allow... Change-Id: I583a11d2a23651db24acde303d522a5faae15c6b
2021-10-05Merge "Cherrypicking flags and override methods for allowing dynamically ↵Jacob Hobbie
registered receivers to mark themselves explicitly as exported or not exported." am: 316e0e8336 Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1836473 Change-Id: I39049e42feb66c3fa3f0ebdef8e175ae86d5d410
2021-10-05Merge "Cherrypicking flags and override methods for allowing dynamically ↵Jacob Hobbie
registered receivers to mark themselves explicitly as exported or not exported."
2021-10-04Merge "Delete some ide autogen files" am: 0da8689915Jeff Sharkey
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1818908 Change-Id: I9a317b53cd4aa19b6e3c54755df01dd26f5bdecf
2021-10-04Merge "Delete some ide autogen files"Jeff Sharkey
2021-10-04Merge "Parcel: obtain(IBinder) as API" am: e869287a7bSteven Moreland
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1817684 Change-Id: I2655c1c2b74b0501912fa4e90f5655db682b2932
2021-10-04Merge "Parcel: obtain(IBinder) as API"Steven Moreland
2021-10-04Merge "CTS of CtsMediaTestCases#android.media.cts.DecodeAccuracyTest may ↵John Reck
fail when video is cropped. When we use GPU to copy the data, rendering would choose filter. But we need add shrink in border to ensure the sampler not reach border like what GUI does." am: 06e16941d5 Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1801770 Change-Id: I1e34d4b25af83a4761e7a2fc81bf5b79d7b2aa08
2021-10-04Merge "CTS of CtsMediaTestCases#android.media.cts.DecodeAccuracyTest may ↵John Reck
fail when video is cropped. When we use GPU to copy the data, rendering would choose filter. But we need add shrink in border to ensure the sampler not reach border like what GUI does."
2021-10-04[automerger skipped] Merge "Migrate Cellbroadcast modules to use ↵Treehugger Robot
modules-utils" am: b0578d5fa3 -s ours am skip reason: Merged-In Ia72c92d6be150edfee3743878d988f21a585e808 with SHA-1 1aff432fda is already in history Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1834517 Change-Id: I0bc7fbc16cfce991989c1643b3371b318befb266
2021-10-04Merge "Migrate Cellbroadcast modules to use modules-utils"Treehugger Robot
2021-10-04Fix the device summary shown twice issueAlice Kuo
As getSubDeviceSummy, we should check the memberDevices's size and decide if the secondarySummary need to be shown. Bug: 201491295 Bug: 150670922 Test: pair with the A2DP/HFP headset, and check the summary Change-Id: I8bb0744067733c6e17292d6c7be1c3f110d731e3
2021-10-04Round 3: Fix flakiness of testNativeCrashYan Han
Round 2: ag/15616233 Round 1: aosp/1702009 Even with increased timeouts, testNativeCrash is not always able to get the tombstone of the crashed native app. This is because the tombstone timestamp can be more than 1000ms before the ApplicationExitInfo, causing them to not match. See b/191798394 for more details. This CL increases the maximum timestamp difference to ensure that the tombstone (almost?) always matches. Bug: 192299487 Test: atest android.app.cts.ActivityManagerAppExitInfoTest Change-Id: Ideeb93dbcb40ce9d564f159fe1f7574554c09a40
2021-10-04Merge "Improve "Lost network stack" error message" am: 73a065a41fRemi NGUYEN VAN
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1843137 Change-Id: Iefa0a4b53694828622487e673a0b4ed521e9a873
2021-10-04Merge "Improve "Lost network stack" error message"Remi NGUYEN VAN
2021-10-04Improve "Lost network stack" error messageRemi NGUYEN VAN
The "Lost network stack" message often gets reported as suspected root cause of a crash, but it is actually just a side-effect of a crash of the network_stack process, which often was just killed together with the rest of the system. Clarify the error message, so that it is clear that the root cause should be visible earlier in logs. Bug: 198681439 Change-Id: Ieb79ace34170f8bd9ea847175199590998278d57 Test: m
2021-10-01Merge "Add getActiveDevices api" into stage-aosp-masterWilliam Escande
2021-10-01Parcel: obtain(IBinder) as APISteven Moreland
There are two main usecases for this: - provide an alternative route to implement features like markSensitive (which requires only 1 function call per class, rather than a function call and flag per transaction type) - support RPC format transactions Bug: 175814583 Test: atest android.os.ParcelTest, boot, manual Change-Id: Ibaacc27a125a31e18d4b5889d05d12cc18d05624
2021-10-01CTS of CtsMediaTestCases#android.media.cts.DecodeAccuracyTest may fail when ↵John Reck
video is cropped. When we use GPU to copy the data, rendering would choose filter. But we need add shrink in border to ensure the sampler not reach border like what GUI does. Change-Id: I2c59956f3d7a7052d8e4bee587360f252a6c5549 Signed-off-by: Chun Jiang <chun.jiang@mediatek.com> Bug: 197180327 Test: CTS PixelCopyTests & DecodeAccuracyTest