Commit Graph

17259 Commits

Author SHA1 Message Date
Wu Ahan
e7bcbbab71 Merge "Support surface only instrumentation" into sc-v2-dev am: d6d30bf7f6 am: 60894eefb5
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15256893

Change-Id: I2f43c2ba720d08446210639cb417af6dd2bc1266
2021-08-02 10:33:28 +00:00
Wu Ahan
60894eefb5 Merge "Support surface only instrumentation" into sc-v2-dev am: d6d30bf7f6
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15256893

Change-Id: I78d98f78086ace09d32fac4329c6b3357aed56b5
2021-08-02 10:23:59 +00:00
Wu Ahan
d6d30bf7f6 Merge "Support surface only instrumentation" into sc-v2-dev 2021-08-02 10:06:03 +00:00
Nataniel Borges
764792b6cb Merge "Introduce a .winscope extension for all winscope files" 2021-08-02 08:07:20 +00:00
Yohei Yukawa
d72d5a169c Move InputBindResult to com.android.internal.inputmethod
This CL mechanically moves InputBindResult from

  com.android.internal.view

to

  com.android.internal.inputmethod

without changing any observable behavior.

Bug: 192412909
Test: presubmit
Test: No lint error under core/java/com/android/internal/inputmethod
Change-Id: Ifd68c8a0a0d1b410c9a480ad39ca470fe2da5bc6
2021-07-30 13:08:44 -07:00
Yohei Yukawa
300f2ee283 Fix lint errors in InputBindResult
This is a clean-up CL of InputBindResult with no behavior change.

As a preparation to move InputBindResult into

  com.android.internal.inputmethod

this CL cleans up all the existing lint warnings in this file.

There should be no behavior change in this CL.

Bug: 192412909
Test: prebuilts/checkstyle/checkstyle.py  -f \
  frameworks/base/core/java/com/android/internal/view/InputBindResult.java
Change-Id: I5c94e6d8f91656654065c624257971ae7835a1aa
2021-07-30 13:08:13 -07:00
Yohei Yukawa
13ada96055 Merge "Simplify IInputConnectionWrapper" 2021-07-30 19:49:16 +00:00
Yohei Yukawa
3c596e3985 Merge changes I0ef1e1b4,Ia4043178
* changes:
  Revert "A unified data class to represent all the InputConnection call"
  Revert "Fix error swallowing in InputConnectionCommand unparcelling"
2021-07-30 17:47:49 +00:00
Li Li
69d46fa9b7 Merge "Freezer: fix exception when parsing /proc/locks" into sc-dev am: ef12967b75 am: 57240b0a67
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15395534

Change-Id: I8605df085ec636ea1dba1096076bee5c1d6dd4f4
2021-07-30 17:07:55 +00:00
Li Li
d7306136e8 Merge "Freezer: fix exception when parsing /proc/locks" into sc-dev am: ef12967b75 am: d2aeb490a8
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15395534

Change-Id: Ica77c69580dd69bed4e260779c1f7318307282b1
2021-07-30 17:03:54 +00:00
Li Li
d2aeb490a8 Merge "Freezer: fix exception when parsing /proc/locks" into sc-dev am: ef12967b75
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15395534

Change-Id: I28f1b15ab636063727f02414a98896ee2679c6fc
2021-07-30 16:50:50 +00:00
Li Li
ef12967b75 Merge "Freezer: fix exception when parsing /proc/locks" into sc-dev 2021-07-30 16:35:16 +00:00
Yohei Yukawa
df38b9c68c Simplify IInputConnectionWrapper
This is a mechanical refactoring CL for IInputConnectionWrapper with
no observable behavior change.

This is a preparation CL before introducing InputConnection task
cancellation (Bug 195115071).

Bug: 192412909
Test: test CtsInputMethodTestCases:InputConnectionEndToEndTest
Change-Id: I814377fd7f73d65f0d5526feebb6bf39e19809d7
2021-07-30 09:13:07 -07:00
Yohei Yukawa
7eec13b458 Revert "A unified data class to represent all the InputConnection call"
This reverts commit 2757c24880 [1].

Reason for revert:
We will try another approach to implement InputConnection task
cancellation without InputConnectionCommand.

 [1]: I86eba7185b4b0664c1b0b3da794dfc5eeddc725c

Bug: 194151409
Change-Id: I0ef1e1b498b77caf7704742d88c7c81944d75c42
Test: atest CtsInputMethodTestCases:InputConnectionEndToEndTest
2021-07-30 09:07:36 -07:00
Yohei Yukawa
da04fd9eb8 Revert "Fix error swallowing in InputConnectionCommand unparcelling"
This reverts commit 11694b0633 [1].

Reason for revert:
We will try another approach to implement InputConnection task
cancellation without InputConnectionCommand.

 [1]: Icd2b40e63771085cb8d88721fbdca7a089256b28

Bug: 194151409
Bug: 194567417
Change-Id: Ia404317848d74a0d8ca0606bfb07cbad36ab97c1
Test: atest CtsInputMethodTestCases:InputConnectionEndToEndTest
2021-07-30 09:06:19 -07:00
Ahan Wu
7141ab2619 Support surface only instrumentation
- Support surface only instrumentation.
- Add enums for pip and wallpaper instrumentation.

Bug: 170708749
Test: atest InteractionJankMonitorTest
Test: atest FrameTrackerTest
Change-Id: Iec1eb82d1a4756a7b85c8fbdd05f4132a74c613c
2021-07-29 19:21:31 -08:00
Daniel Solomon
070300e858 Merge "Revert "SCREEN_BRIGHTNESS - map 255 to current max"" into sc-dev am: d5e1458566
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15404418

Change-Id: Ie65fbb748337a030492d515bbea94c551398b5f5
2021-07-29 19:15:35 +00:00
Hai Zhang
0f17f27c77 Merge "Only construct real Throwable objects in AndroidFuture.readThrowable()." into sc-dev am: af04dd2ad5
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15302018

Change-Id: I1bf40fe706b0f03443a8a6fbb6e8c84b0fa44402
2021-07-29 19:12:13 +00:00
Li Li
bbe06e7b59 Freezer: fix exception when parsing /proc/locks
The original code is based on /proc/locks manpage. Unfortunately, that
manpage is never correctly updated to include the new field for blocked
locks. The new code detects the extra field. As StringTokenizer is too
heavy, ProcFileReader is used for better performance.

Bug: 194756340
Test: FrameworksUtilTests com.android.internal.util.ProcFileReaderTest
Test: FrameworksCoreTests com.android.internal.os.ProcLocksReaderTest
Test: No /proc/locks parsing exception with blocked locks
Change-Id: I4c9763f9d3091f7d84d2e4b672d7e5cb78b33f59
2021-07-29 09:18:53 -07:00
Siim Sammul
76b63d54cf Merge "Do not make a getThreadTimeMicro() call when it is not needed. This was accidentally changed with https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15080830" into sc-dev am: 6c852aeabc
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15346904

Change-Id: Ibaae003e403fc6d20002a62241ec23a0fa97f2ce
2021-07-28 18:25:11 +00:00
Alison Cichowlas
7bed52f477 Merge "Sharesheet: remove 2 separators, restyle the 3rd." into sc-dev am: 832779a555
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15377016

Change-Id: I37ce8ef56bc0cb8efd24e899406f30b7242738ae
2021-07-28 18:24:05 +00:00
Daniel Solomon
d953a29d69 Merge "Revert "SCREEN_BRIGHTNESS - map 255 to current max"" into sc-dev am: d5e1458566 am: 8c6caab15c
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15404418

Change-Id: I9419af82f3d88c6f39c403ac1b94d90482dbe02c
2021-07-28 18:24:00 +00:00
Daniel Solomon
d41a2a1ca0 Merge "Revert "SCREEN_BRIGHTNESS - map 255 to current max"" into sc-dev am: d5e1458566 am: ae50161de8
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15404418

Change-Id: I1f94be5d70453b6ef0a28d8c1a680a8880242e04
2021-07-28 18:11:11 +00:00
Daniel Solomon
ae50161de8 Merge "Revert "SCREEN_BRIGHTNESS - map 255 to current max"" into sc-dev am: d5e1458566
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15404418

Change-Id: I7782fd8704f1b7c2444b6630ce001875b19d6931
2021-07-28 17:58:09 +00:00
Daniel Solomon
d5e1458566 Merge "Revert "SCREEN_BRIGHTNESS - map 255 to current max"" into sc-dev 2021-07-28 17:42:43 +00:00
Daniel Solomon
792229b553 Revert "SCREEN_BRIGHTNESS - map 255 to current max"
This reverts commit 002e656a51.

Reason for revert: Setting brightness using adb (adb shell settings put system screen_brightness ...) results in different brightness being set by DisplayPowerController each time.

Bug: 194791517
Test: atest HighBrightnessModeControllerTest
Test: manually test `settings put system screen_brightness 23` yields
      consistent results
Change-Id: I6b49253b46e4d09a1f4129d3df950c9d36eb6ede
2021-07-28 17:17:15 +00:00
Yohei Yukawa
2cab8c518f Merge "Remove @UnsupportedAppUsage from InputBindResult#CREATOR" 2021-07-28 16:23:20 +00:00
Yohei Yukawa
19ab748b9a Merge "Remove @UnsupportedAppUsage from InputBindResult#method" 2021-07-28 15:14:40 +00:00
Alison Cichowlas
82cee4bd0a Merge "Sharesheet: be more defensive about un-unparcelable intents." into sc-dev am: a1d1a38269
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15374840

Change-Id: I24bf7607ee67bfba322f0f03a62eb58745b98a79
2021-07-28 15:03:08 +00:00
Santos Cordon
28c5f05a22 Merge "SCREEN_BRIGHTNESS - map 255 to current max" into sc-dev am: e4e836a175
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15359296

Change-Id: I0e027bc0ecbdea29c8b86c1785dc514e69efdff8
2021-07-28 14:55:42 +00:00
Alison Cichowlas
b566f49ae4 Merge "Sharesheet: action bar is independent of WP tabs, should always launch as primary user." into sc-dev am: 9197f7b260
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15363687

Change-Id: I23d9a8b26412d255f50093f6b8f5cfa9bed00052
2021-07-28 14:50:52 +00:00
Hai Zhang
2af324b732 Merge "Only construct real Throwable objects in AndroidFuture.readThrowable()." into sc-dev am: af04dd2ad5 am: 831f5db04f
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15302018

Change-Id: I07948cad1d44aaaeb68f0f4c9a44e17ec104749d
2021-07-28 02:03:31 +00:00
Hai Zhang
8c0e7d693b Merge "Only construct real Throwable objects in AndroidFuture.readThrowable()." into sc-dev am: af04dd2ad5 am: 92414a05dc
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15302018

Change-Id: I269629173d5a291c3e4575b21255394cf9564b73
2021-07-28 01:58:36 +00:00
Hai Zhang
831f5db04f Merge "Only construct real Throwable objects in AndroidFuture.readThrowable()." into sc-dev am: af04dd2ad5
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15302018

Change-Id: I2afc85c266bebcb88848c57a094ed0af757b277e
2021-07-28 01:41:05 +00:00
Hai Zhang
af04dd2ad5 Merge "Only construct real Throwable objects in AndroidFuture.readThrowable()." into sc-dev 2021-07-28 01:23:12 +00:00
Yohei Yukawa
c362e57266 Remove @UnsupportedAppUsage from InputBindResult#CREATOR
We are mostly confident that this @UnsupportedAppUsage was a false
positive.

Fix: 194831602
Test: presubmit
Change-Id: Id836d48f19f6161030b1b77445777f8535bd78a3
2021-07-27 15:03:44 -07:00
Yohei Yukawa
45dc08a19d Remove @UnsupportedAppUsage from InputBindResult#method
We are mostly confident that this @UnsupportedAppUsage was a false
positive.

Fix: 194830651
Test: presubmit
Change-Id: I0b0f1932274d0ce074c94b73ee79d2c4707896cb
2021-07-27 14:55:33 -07:00
Yohei Yukawa
5f21071816 Merge "Fix error swallowing in InputConnectionCommand unparcelling" 2021-07-27 16:11:53 +00:00
Tiger Huang
17ef2f52c6 Merge "Use InsetsVisibilities to carry requested visibilities" into sc-v2-dev am: 9052763843 am: 6b52c08bbf
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15385730

Change-Id: I31b54ea52b0d977e0161458ba7e41c4b9578df9d
2021-07-27 06:48:21 +00:00
Tiger Huang
6b52c08bbf Merge "Use InsetsVisibilities to carry requested visibilities" into sc-v2-dev am: 9052763843
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15385730

Change-Id: Iac4fdec20890c2df0960488453c3ff7d2f801f8b
2021-07-27 06:34:28 +00:00
Tiger Huang
9052763843 Merge "Use InsetsVisibilities to carry requested visibilities" into sc-v2-dev 2021-07-27 06:21:13 +00:00
Siim Sammul
d75cbc78c4 Merge "Do not make a getThreadTimeMicro() call when it is not needed. This was accidentally changed with https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15080830" into sc-dev am: 6c852aeabc am: b77fb08506
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15346904

Change-Id: I6552265f9258ea772b1774d00bd884cc2d530202
2021-07-26 19:19:05 +00:00
Alison Cichowlas
993c186923 Merge "Sharesheet: remove 2 separators, restyle the 3rd." into sc-dev am: 832779a555 am: fa25c27e48
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15377016

Change-Id: I6d5563833b41f0f0221bbbc4fa71aedf5eb494d0
2021-07-26 19:10:45 +00:00
Alison Cichowlas
b4cf9f5ffc Merge "Sharesheet: be more defensive about un-unparcelable intents." into sc-dev am: a1d1a38269 am: ddbade4025
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15374840

Change-Id: I3f542d0d7548543331fd05a704542df7b563a0b0
2021-07-26 19:05:28 +00:00
Santos Cordon
58622a42ef Merge "SCREEN_BRIGHTNESS - map 255 to current max" into sc-dev am: e4e836a175 am: 379e94d1da
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15359296

Change-Id: I2314b5dd09df823a203088c16481fa8717cae5df
2021-07-26 18:58:37 +00:00
Alison Cichowlas
8f3b2e1c8a Merge "Sharesheet: action bar is independent of WP tabs, should always launch as primary user." into sc-dev am: 9197f7b260 am: 34ff8cdeaf
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15363687

Change-Id: I1b5085f13d39250bd58003a5a353bef7e4762054
2021-07-26 18:56:02 +00:00
Yohei Yukawa
11694b0633 Fix error swallowing in InputConnectionCommand unparcelling
This is a follow up CL to my previous CL [1], which added
InputConnectionCommand class with parcelling support.

A Parcel object may contain multiple primitive and Parcelable values.
Neither shallowing exceptions nor early-returning null solves the
fundomental problem that any subsequent read from the same Parcel
object is at risk.

Just let the higher layer decide how to deal with it.

 [1]: I86eba7185b4b0664c1b0b3da794dfc5eeddc725c
      2757c24880

Bug: 194151409
Fix: 194567417
Test: atest CtsInputMethodTestCases:InputConnectionEndToEndTest
Test: atest FrameworksCoreTests:InputConnectionCommandTest
Change-Id: Icd2b40e63771085cb8d88721fbdca7a089256b28
2021-07-26 11:02:44 -07:00
Nataniel Borges
866e966e1f Introduce a .winscope extension for all winscope files
This extension will be used to better integrate winscope traces and ABT, allowing trace files to be opened directly from ABT or buganizer

Bug: 192663722
Bug: 162813517
Test: (for winscope) build winscope, record traces, download traces, check if traces have the .winscope extension. (for flicker) run atest Flickertests. (for abt) on new extension, check if .winscope files show as "open in winscope" on buganizer
Change-Id: If669a1f41f1eab850d8a4715dbe601bb35fe01bb
2021-07-26 15:57:10 +02:00
Tiger Huang
090c0cbcb0 Use InsetsVisibilities to carry requested visibilities
InsetsState contains much more information than visibilities, such as
display frame, display cutout, rounded corners, privacy indicator
bounds, and frames of of insets sources. The control target only needs
to send the requested visibilities to WMS, so it can be too heavy to use
InsetsState.

This CL introduces an new class, InsetsVisibilities, which only contains
which type has which visibility. So it uses less memory, and it is more
efficient on copying and checking the equality.

Fix: 194186241
Test: atest InsetsVisibilitiesTest WindowAddRemovePerfTest
            InsetsControllerTest RegisterStatusBarResultTest
            CommandQueueTest LightsOutNotifControllerTest
            ActivityRecordTests DisplayContentTests
            DisplayPolicyLayoutTests InsetsPolicyTest
            InsetsSourceProviderTest InsetsStateControllerTest
            WindowFrameTests WindowManagerServiceTests WindowStateTests
Change-Id: I86c1b26b4383bfa3b924726d580e5706e13ba735
2021-07-26 21:55:34 +08:00
Siim Sammul
11dbfb0896 Merge "Do not make a getThreadTimeMicro() call when it is not needed. This was accidentally changed with https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15080830" into sc-dev am: 6c852aeabc am: 4f950d7975
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15346904

Change-Id: Ia9ceba930e67dd84a9321dd1b2b2c87aed5343de
2021-07-26 09:44:53 +00:00