Commit Graph

1769 Commits

Author SHA1 Message Date
Vadim Tryshev
56153bbc89 Merge "Revert "Revert "Removing unused tracing""" 2022-01-26 18:43:19 +00:00
TreeHugger Robot
48fb939966 Merge "Don't start dragging widget below bottom gesture margin" 2022-01-25 18:47:18 +00:00
Brian Isganitis
c19b51e1f3 Merge "Refactor all apps to depend on activity context." 2022-01-25 18:34:30 +00:00
Alex Chau
f7f4f4d7ed Don't start dragging widget below bottom gesture margin
Bug: 215672979
Test: AddConfigWidgetTest
Change-Id: I8121c0662249ec019f4f47a402f7f78655a6b930
2022-01-25 15:18:12 +00:00
Thales Lima
ec951f7f2c Merge "Make hotseat fit one line on tablets" 2022-01-25 14:03:34 +00:00
TreeHugger Robot
90b6d5f452 Merge "Add ScreenRecord for a flaky test" 2022-01-25 04:27:51 +00:00
Brian Isganitis
1664c9f418 Refactor all apps to depend on activity context.
The current AllAppsContainerView has been split into a base class and a
subclass for activities. The base class allows using all apps with an
activity context. A-Z and work profile tabs are supported, but search
and the hotseat still require an Activity.

Test: Manual. All apps should work the same.
Bug: 204696617
Change-Id: I3b146714bc11b3b3555d97623aab5d78ac836482
2022-01-24 20:24:20 -05:00
Alex Chau
0a1f7d5de5 Add ScreenRecord for a flaky test
Bug: 215673732
Test: none
Change-Id: I175031be50e37f7c14643475e3b44bfc514022e4
2022-01-24 18:56:40 +00:00
Alex Chau
1a9ef83988 Add ScreenRecord for a flaky test
Bug: 215672979
Test: none
Change-Id: I3bfd0275a4d551c0b6d614a7a76d696f14ce3ca3
2022-01-24 15:16:24 +00:00
Vadim Tryshev
21720c9a2c Revert "Revert "Removing unused tracing""
This reverts commit 827335f209.

Reason for revert: Problem solved

Bug: 195319692
Test: presubmit
Change-Id: I084c8a6b6acdaa3ee4697fc752e817d292ba0665
2022-01-22 02:46:02 +00:00
Thales Lima
a1012904bb Make hotseat fit one line on tablets
When running on tablets in landscape, the QSB should inline with the hotseat icons. This does that and fixes small bugs that didn't account for the new hotseatBorderSpace.

Bug: 210118169
Test: atest Launcher3Tests:DeviceProfileTest
Test: visual, using HSV and Window
Change-Id: I5a89c5449bf59fde736618151eceaacca443ef67
2022-01-21 17:42:48 +00:00
TreeHugger Robot
e8f8cd2f53 Merge "Don't treat object from test package as system anomaly dialog" 2022-01-18 18:02:32 +00:00
Alex Chau
26a4e5bf26 Add screenRecord for PromiseIconUiTest
Bug: 202985412
Test: none
Change-Id: Iac74290ce872844c8a0f5d0878c8c55f6bb89bb2
2022-01-17 13:08:36 +00:00
Alex Chau
d58ae513b2 Don't treat object from test package as system anomaly dialog
- OOP test activity has an alertTitle and shouldn't be treated as system anomaly dialog

Fix: 214558538
Test: NexusLauncherOutOfProcTests / MemoryTests
Change-Id: Ia9c330c3c3f48b43fbada40a3410ccc9fb8bfbb8
2022-01-17 12:32:25 +00:00
TreeHugger Robot
29004de0d1 Merge "Limit home gesture distance to half of screen height" 2022-01-14 00:18:18 +00:00
Alex Chau
a472190c8a Limit home gesture distance to half of screen height
- Somehow when home gesture's travel distance is above certain number (~2560px in the bug's case), the touch will wrongly go to the 3P launcher instead of staying in quickstep launcehr only. As it's unusal for home gesture to be so long, limiting it to half of screen height.

Bug: 214279686
Test: presbumit and TaplTestsLauncher3#testWidgets on cf_tablet
Change-Id: I1107074556aea3011b186ba34dc830dbb1445517
2022-01-13 22:08:40 +00:00
Vadim Tryshev
d55a139843 Merge "Removing 2-button support from tests" 2022-01-13 17:59:57 +00:00
vadimt
1ac72db930 Removing 2-button support from tests
Bug: 187761685
Test: presubmit
Change-Id: I932dd8cbc7524732e3ff83a35d7034b3e5ff75d9
2022-01-12 10:37:50 -08:00
Ling Wo
7802ec4f25 Merge "Uninstall an app from workspace via TAPL" 2022-01-11 01:57:50 +00:00
My Name
5b59e75f4a Uninstall an app from workspace via TAPL
1. Implement function to uninstall app from appIcon.
2. Add test uninstall to TaplTestsLauncher3.

Bug: 210927656
Test: Launcher3Tests:com.android.launcher3.ui.TaplTestsLauncher3#testTryUninstallFromWorkspace
Change-Id: Iedd3bd3a46bc626fc414fd8c5bd07ebc0fa235bb
2022-01-07 21:52:07 -05:00
vadimt
672f2fc24a Generate bugreports for ANRs and other anomalies
Bug: 211907395
Test: a similar approach was implemented for SysUI tests
Change-Id: I04770e73e6adf6bda63a8362976ec550434df6f3
2022-01-07 16:05:46 -08:00
Andras Kloczl
3d8cfd22a1 Two panel workspace TAPL follow up CL
- TODO: enable test in portrait

Test: TwoPanelWorkspaceTest
Former CL: http://ag/16357467
Bug: 197631877

Change-Id: I35c326b7a6c612d2fb66c8b94b810f5131fd58d8
2022-01-07 11:27:38 +00:00
TreeHugger Robot
720762f65b Merge "Ensure to inject events outside of the insets region for right margin" 2021-12-29 05:44:38 +00:00
Prabir Pradhan
00308cfd57 Ensure to inject events outside of the insets region for right margin
The display's x-coordinates for input are in the range:
  [0, displayWidth).

Injecting input at the point (displayWidth, 0) is technically outside
of the display. After accounting for insets, the x-coordinate range for
the display is:

  [leftInset, displayWidth - rightInset).

An input event injected at x = (displayWidth - rightInset) would be
outside of this range, and would be within the right insets region.

To account for this, use a one pixel buffer when calculating the
right gesture start margin.

A similar one pixel buffer is already used for the bottom margin
calculation.


Test: atest NexusLauncherTests:
com.android.quickstep.TaplTestsQuickstep
Bug: 208849306

Change-Id: I9d5a6429cd55fdfcbf2c849e27e3d3a0a7faaa47
2021-12-24 13:14:41 +00:00
Pat Manning
ee01176810 Fix launcher scroll distance for TAPL tests.
Test is passing correctly but for the wrong reasons. If we attempted to use a different distance it would have failed.

Bug: 197630182
Test: manual
Change-Id: Ib275b89c86cdb9f75d40b9d25c8c498098b543b7
2021-12-22 09:55:53 +00:00
vadimt
abe8e3a947 Not generating memory dump for intentional leak tests
Memory dump can be generated only once, so generating it in the intentional leak test leaves us without a meaningful dump if a real leak happens later, and produces a confusing message that the dump was already generated.

Bug: 195319692
Test: presubmit
Change-Id: I35932deba0c46ddb27362467109daf74d75cbff4
2021-12-20 12:35:53 -08:00
Ling Wo
06119c9c6a Merge "delete appIcon from workspace via TAPL" into sc-v2-dev am: 361d8286cc am: 7f389d6317
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16189765

Change-Id: I4d2d76c92933e45dab99566e460c4f59804f59e7
2021-12-14 02:55:17 +00:00
Ling Wo
7f389d6317 Merge "delete appIcon from workspace via TAPL" into sc-v2-dev am: 361d8286cc
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16189765

Change-Id: Ia004c65470959ef3e7dcc3799bcda9de6b15725e
2021-12-14 02:34:50 +00:00
Ling Wo
361d8286cc Merge "delete appIcon from workspace via TAPL" into sc-v2-dev 2021-12-14 02:10:59 +00:00
Ling WO
85336c23fb delete appIcon from workspace via TAPL
1. Create a new class for DropBar icon.
2. Implement function to delete appIcon from workspace.
3. Add testDeleteFromWorkspace to TaplTestsLauncher3.

Bug: 205593009
Test: Launcher3Tests:com.android.launcher3.ui.TaplTestsLauncher3#testDeleteFromWorkspace

Change-Id: I259b909aebcdcc1c73af63a9970db43dab3e86eb
2021-12-10 18:14:41 -08:00
Alex Chau
367f6262e1 Merge "Improving memory dump diag" into sc-v2-dev am: eea4379a5d am: b944b46eeb
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16434161

Change-Id: I1af6741c983fad7b767f84747c0adadcfdbecdd2
2021-12-10 10:25:24 +00:00
Alex Chau
b944b46eeb Merge "Improving memory dump diag" into sc-v2-dev am: eea4379a5d
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16434161

Change-Id: If3a45237f8a766eb6b812392e0aa979862a87a5e
2021-12-10 10:10:38 +00:00
vadimt
bc1da4ca98 Improving memory dump diag
atest deletes artifacts directory on the device, but saves artifacts at the host machine.

Bug: 187761685
Test: presubmit
Change-Id: I1f5244ebfe6b606c4891f5fca003541b27cc944c
2021-12-09 21:58:45 +00:00
Alex Chau
992db75dae Use percentage scaling for overview thumbnail size
- Limit overview to screen ratio to be at max 70%
- Tuned row spacing and page spacing
- Added 2 more test activities to testOverviewForTablet to avoid dismissing tasks at end of grid, as more grid tasks are now shown

Bug: 207350366
Test: manual
Change-Id: Id04871464bdb3ef2d6b046977b0ffd9bd51bc230
Merged-In: Id04871464bdb3ef2d6b046977b0ffd9bd51bc230
2021-12-07 14:50:40 +00:00
Alex Chau
a5470710a3 Use percentage scaling for overview thumbnail size am: 4f7952ead4
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16389092

Change-Id: I18751b12e2c7fa2b376545a95426594cbadffd07
2021-12-07 14:50:04 +00:00
Alex Chau
4f7952ead4 Use percentage scaling for overview thumbnail size
- Limit overview to screen ratio to be at max 70%
- Tuned row spacing and page spacing
- Added 2 more test activities to testOverviewForTablet to avoid dismissing tasks at end of grid, as more grid tasks are now shown

Bug: 207350366
Test: manual
Change-Id: Id04871464bdb3ef2d6b046977b0ffd9bd51bc230
2021-12-07 11:00:48 +00:00
András Klöczl
0722fb1b2a Merge "Add TAPL tests for two panel workspace" into sc-v2-dev am: f4a161eef3 am: f5ee9c87b8
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16357467

Change-Id: I5e29e4e23a1a49e2c6ba9322a700d675040e5ca6
2021-12-06 14:46:54 +00:00
András Klöczl
f5ee9c87b8 Merge "Add TAPL tests for two panel workspace" into sc-v2-dev am: f4a161eef3
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16357467

Change-Id: Icc7a700ad85315c95a3ff542d63a2c7ba888bf96
2021-12-06 14:29:15 +00:00
Andras Kloczl
c820151a4c Add TAPL tests for two panel workspace
- Add test for icon dragging
- Add test for page pairing
- Add test for empty page removal
- Add dragIcon helper method

Test: TwoPanelWorkspaceTest
Bug: 197631877
Change-Id: I7ce2dd88e183626feac49b34254d00a589473944
2021-12-02 16:44:26 +00:00
Prabir Pradhan
13bdf056b1 Merge "Tapl Widgets: Don't use container width as gesture margin" into sc-v2-dev am: 79350ef1bc am: 60bbc7c74f
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16365181

Change-Id: Id7249c0341f36ec5f19e86b4dea47c9f14f4a884
2021-12-01 12:07:14 +00:00
Prabir Pradhan
60bbc7c74f Merge "Tapl Widgets: Don't use container width as gesture margin" into sc-v2-dev am: 79350ef1bc
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16365181

Change-Id: I2d7400327fc04cf07b872a0769095dfd7859cd46
2021-12-01 11:52:54 +00:00
Prabir Pradhan
79350ef1bc Merge "Tapl Widgets: Don't use container width as gesture margin" into sc-v2-dev 2021-12-01 11:39:11 +00:00
TreeHugger Robot
41e7174cb8 Merge "Fling to close tasks in the GestureNav can be too small." into sc-v2-dev am: 96f34af124 am: 0a3ae68934
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16170178

Change-Id: I05cd240dcb21a526614fde03ad5423cb3f27cfcf
2021-11-30 19:05:22 +00:00
TreeHugger Robot
0a3ae68934 Merge "Fling to close tasks in the GestureNav can be too small." into sc-v2-dev am: 96f34af124
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16170178

Change-Id: I4834f39f505dad8ffd2d0db826c4e6f6bf364f31
2021-11-30 18:47:53 +00:00
TreeHugger Robot
96f34af124 Merge "Fling to close tasks in the GestureNav can be too small." into sc-v2-dev 2021-11-30 18:34:09 +00:00
Prabir Pradhan
53ae5d85a6 Tapl Widgets: Don't use container width as gesture margin
The test used margin that was equal to the width of the container when
injecting a backward swipe. This means the swipe was injected at an x
value of 0 in the View's local coordinates. When injecting such events
into the system, the coordinates will undergo several transformations,
at which point floating point precision errors could accumulate. This
could result in the event being dispatched outside the boundary of the
view.

Change the injection margin so that the swipe is injected in the middle
of the View instead of at its edge.

Bug: 201777251
Bug: 207146693
Test: atest TaplTestsLauncher3
Change-Id: I4082c2845aef033a6fbe41070061866d6048e21d
2021-11-30 15:06:42 +00:00
Alex Chau
eab71ab652 Merge "Fix tapl tests for overview actions in fallback recents." into sc-v2-dev am: 22d7524f53 am: eb6be24779
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16182391

Change-Id: I54cf9c8d25be9a3393c122d0bf4135c051d11975
2021-11-25 18:28:10 +00:00
Alex Chau
eb6be24779 Merge "Fix tapl tests for overview actions in fallback recents." into sc-v2-dev am: 22d7524f53
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16182391

Change-Id: Iedec936c8c9c904b2be13d4589958b0ccc5f9a73
2021-11-25 18:16:18 +00:00
Alex Chau
22d7524f53 Merge "Fix tapl tests for overview actions in fallback recents." into sc-v2-dev 2021-11-25 17:58:02 +00:00
TreeHugger Robot
a6f666f12e Merge "Revert "Migrate workspace item adding tests to kotlin"" into sc-v2-dev am: da014976cd am: 6b17728794
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/16340532

Change-Id: Ifb4d6dd8bfcdd914c042dd1681bc9bbd977ce09f
2021-11-25 16:10:20 +00:00