- Feb 20, 2020
-
-
David Su authored
Build OsuLogin against system_current and make it available to the wifi apex. Bug: 149346045 Test: Attempted to connect to an OSU network, verify OsuLogin page pops up Change-Id: Ib71772f09d85b544f1298493ca34b10243a6a9f5
-
Yohei Yukawa authored
-
TreeHugger Robot authored
-
TreeHugger Robot authored
-
Mehdi Alizadeh authored
-
Jayant Chowdhary authored
-
TreeHugger Robot authored
-
TreeHugger Robot authored
-
TreeHugger Robot authored
-
TreeHugger Robot authored
-
TreeHugger Robot authored
-
Rambo Wang authored
-
Howard Ro authored
-
TreeHugger Robot authored
-
Trung Lam authored
* changes: Implement onDestroy for ConversationStore for when an app gets uninstalled. Add persistence of Event and EventIndex during device reboot.
-
Jackal Guo authored
-
Hai Zhang authored
-
Yohei Yukawa authored
As tracked in Bug 28750507, InputMethodManagerService#onSwitchUser() is known to be slow, and the direct reason of the UI jank in the launcher discussed Bug 139806621 is that #onSwitchUser() is running as part of IMMS#startInputOrWindowGainedFocus() that is called as a sync IPC from the UI thread of the launcher. Note that this is a relatively new behavior that was introduced in Android 10 to support per-profile IME mode (Bug 111083076). So users shouldn't see this code path unless the device is running on Android 10 with work profile enabled. What this CL does is removing that direct code path from IMMS#startInputOrWindowGainedFocus() to #onSwitchUser() by introducing a new pending result InputBindResult.ResultCode.SUCCESS_WAITING_USER_SWITCHING, which means IMMS is now performing user switching and cannot start input session right now, but it will call the client back when the IME becomes available for the target user. Note that InputMethodManager is already able to handle this kind of pending state, where IME is not yet available, since we already have a similar pending results as follows. * InputBindResult.ResultCode.SUCCESS_WAITING_USER_SWITCHING * InputBindResult.ResultCode.SUCCESS_WAITING_IME_BINDING One remaining concern is that when IMMS#switchUserOnHandlerLocked() is running with holding the giant lock (IMMS#mMethodMap), thus it's still possible that IMMS#startInputOrWindowGainedFocus() can be blocked because of lock contention, which eventually blocks client's UI thread. Although its chance wouldn't be that high, in order to tackle that scenario we need to pursue different approaches such as: * Further optimize IMMS#startInputOrWindowGainedFocus() to reduce the likelihood of lock contention. This isn't a perfect solution for this particular case but it's still worth doing. Possible candidate of optimizations are: * Bug 28750507: optimize IMMS#switchUserOnHandlerLocked() * Bug 149864769: off-load Context#unbindService() * Reconsider the use of giant lock. * Redesign the current "startInput" protocol as part of our on-going effort to redesign IME focus handling protocol (Bug 141738570) e.g. can we completely make IMMS#startInputOrWindowGainedFocus() async IPC? Bug: 139806621 Fix: 144291210 Test: atest CtsInputMethodTestCases CtsInputMethodServiceHostTestCases Test: Manually made sure that IMMS#startInputOrWindowGainedFocus() isn't blocking Launcher's UI thread when swiping home. 1. lunch aosp_coral-userdebug && make -j 2. Flash the image 3. Open the system settings 4. System -> Gestures -> System nagivation 5. Select "Gesture nagivation" 6. Install Test DPC. 7. Enable managed profile with Test DPC. 8. adb install -r \ $ANDROID_TARGET_OUT_TESTCASES/EditTextVariations/EditTextVariations.apk # deal with dex2oat 9. adb shell cmd package compile -m speed \ -f com.google.android.inputmethod.latin 10. adb shell cmd package compile -m speed \ -f com.android.inputmethod.tools.edittextvariations 11. adb shell cmd package compile -m speed \ -f com.google.android.apps.nexuslauncher 12. adb root 13. adb shell setprop pm.dexopt.disable_bg_dexopt true 14. adb reboot 15. adb shell am start -u 10 \ -n com.android.inputmethod.tools.edittextvariations/.EditTextVariations 16. Tap the first edit text on the EditTextVariations 17. Make sure that AOSP Keyboard is shown 18. adb shell am trace-ipc start 19. external/chromium-trace/systrace.py \ gfx freq am wm sched binder_driver view \ -a com.android.launcher3 -o binder.html 20. Swipe up the home button 21. Hit the enter key to terminate the systrace. 22. Check the IPC log to see how much the UI thread is blocked by IMMS#startInputOrWindowGainedFocus() Change-Id: I5a73a66d2b8acadad9b3577ebc4c17b5a25fd011
-
Alessio Balsini authored
-
Evan Rosky authored
Bug: 149828540 Test: Can use ime on keyguard Change-Id: Iae7007d6c8157cffd821e2372d19b27df869d6c7
-
TreeHugger Robot authored
-
Farid Chahla authored
When testing HID devices, it was assumed that the bus used would always be bluetooth. This caused USB hid tests to fail. This patch fixes this issue by adding a "busType" parameter to the test register file. This can be either "bluetooth" or "usb" at this time. This will break any hid test that does not specify a busType. Cherry picked from pa/1549664. Bug: 136263708 Test: atest CtsHarwareTestCases Change-Id: I44cda52505110227fa5262994d0cfe59ecb8763d
-
Varun Shah authored
-
Sarah Chin authored
-
Alec Mouri authored
-
Rambo Wang authored
-
Shu Chen authored
-
TreeHugger Robot authored
-
Alex Mang authored
-
Sarah Chin authored
Merge changes from topics "revert-10153355-dev_cap-JUKVIAUYRW", "revert-9919494-device_cap-HJIMIMXSXL" * changes: Revert "Modify PhoneCapabilities for device capabilities" Revert "PhoneCapability cleanup"
-
- Feb 19, 2020
-
-
Sudheer Shanka authored
-
Trung Lam authored
uninstalled. Change-Id: I8ad6ec2987c509d6713abdd4244c314cb05e79a1 Test: Built and tested on device. Bug: 149789130
-
Trung Lam authored
Change-Id: Ic6c403a000a776eca2a3e9678c25862693a04c2e Test: Built on device and unit tests. Bug: 149356114
-
PETER LIANG authored
-
Beth Thibodeau authored
-
TreeHugger Robot authored
-
Danning Chen authored
-
Malcolm Chen authored
Set KEY_DATA_SWITCH_VALIDATION_MIN_GAP_LONG to 24 hours to enable validation skipping if network is validated in the last 24 hours. Bug: 140070796 Test: unittest Change-Id: I07b087d731ed1661e5b0c51407ea3003769a3fd9
-
Automerger Merge Worker authored
Merge "Revert "Revert "[DexLoadReporter] Report classloader contexts di..."" am: 1ee3e70c am: 2cef69cb am: a2c3c9fe Change-Id: I6d7ba01f4083ed01eea0ec0dda0b69f27a4b1346
-
Automerger Merge Worker authored
Merge "IMS: Add support for IMS Explicit call transfer" am: 12f7f43e am: 4c0cd623 am: f65d0089 Change-Id: Ic8231acd8330c91998f0cb479a7a6fbe3ea5bda1
-