History log of /frameworks/base/services/tests/servicestests/src/com/android/server/wm/DragDropControllerTests.java
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
97d2820cf89eab4ca7e86b4f9022b3f45d8e461f 28-Feb-2018 chaviw <chaviw@google.com> Synchronize all WindowTestsBase and WindowTestUtils methods.

Several WindowManager tests access WindowManager code that assume the WM
lock is already held. This can cause crashes in native code and cause
unexpected behavior. Added synchronize blocks around all helper methods
to avoid adding synchronized blocks in all WM tests methods.

This does not guarantee that all WM tests are synchronized, but handles
a good bulk of them for now.

The largest number of failures were from ZOrderingTests since that tests
creates a lot of windows, making synchronicity issues more likely. Ran
that test class multiple times to ensure I was unable to get the
failure.

Test: ZOrderingTests
Bug: 73966377
Change-Id: I96f586e7ea91223589caef885c87ecded555ab13
Fixes: 72976082
/frameworks/base/services/tests/servicestests/src/com/android/server/wm/DragDropControllerTests.java
fabca09f0f5cf32eeebb4c8ae793f498aa182d58 19-Dec-2017 Daichi Hirono <hirono@google.com> Remove DragDropController#prepareDrag()

The prepareDrag() call is used by apps to ask the system server to
create a surface for drag shadow. Now the app creates a surface by
itself, thus we can reduce the method.

Bug: 70818582
Test: com.android.server.wm.DragDropControllerTests,
android.server.wm.CrossAppDragAndDropTests,
manually check the drag and drop behavior on test app.
Change-Id: Iae4acbff4f9ad68faa6324beb32bf9a28dcd67be
/frameworks/base/services/tests/servicestests/src/com/android/server/wm/DragDropControllerTests.java
a1fb9be437b672f52ad64e1506038757ec060dcf 18-Dec-2017 Daichi Hirono <hirono@google.com> Create a drag shadow surface in app process,

Previoulsy a drag shadow surface is created in the system process. App needs
to call one more binder call (prepareDrag) to obtain the surface from
the system process.

The CL lets an app to create a drag shadow surface by itself. Then app
transfer the surface to system server by using reparent
API.

Bug: 70818582
Test: com.android.server.wm.DragDropControllerTests,
android.server.wm.CrossAppDragAndDropTests,
manually check the drag and drop behavior on test app.
Change-Id: I72796efffbefe78a802d7c441dea308d1cdea572
/frameworks/base/services/tests/servicestests/src/com/android/server/wm/DragDropControllerTests.java
da0748d00c645225317ed2656b8b3b0561aaaf4c 12-Dec-2017 Daichi Hirono <hirono@google.com> Fix NPE occured when null ClipData is passed to startDrag.

Previously we have NPE for the following cases:

* when specifying both DRAG_FLAG_GLOBAL and DRAG_FLAGS_URI_ACCESS, OR
* when the userIds of source and target apps are different

Bug: 70477060
Test: com.android.server.wm.DragDropControllerTests,
android.server.wm.CrossAppDragAndDropTests,
manually check the drag and drop behavior on test app.
Change-Id: I454b434e4466cb159ce23e0982012cdb21c9d002
/frameworks/base/services/tests/servicestests/src/com/android/server/wm/DragDropControllerTests.java
1821c5a0db9fa186b4e0a40fc7ff45f3dc5539a9 12-Dec-2017 Daichi Hirono <hirono@google.com> Let prepareDrag return error when 0 size drag shadow is specified.

Since ag/2837472, creating 0 size surface resulted in raising an
exception. To avoid it, the CL lets prepareDrag return error.

Bug: 70393408
Test: com.android.server.wm.DragDropControllerTests,
android.server.wm.CrossAppDragAndDropTests,
manually check the drag and drop behavior on test app.

Change-Id: Ie0c477dd9e76dd5ec8b7aed6a472e24032777cc8
/frameworks/base/services/tests/servicestests/src/com/android/server/wm/DragDropControllerTests.java