History log of /frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
89ded73522265e09a03fd04f4ffccef7118bdedb 22-Sep-2017 Dake Gu <dake@google.com> leanback: fix flaky testScrollAndRemove

In short:
Skips the minScroll/maxScroll check in layout pass because
onLayoutChildren() might call scrollDirectionPrimary() to compensate
remaining scroll distance.

In long explaination:
onLayoutChildren() will compensate remaining scroll distance. Let's
say when onLayoutChildren() is called, the RecyclerView is still
scrolling toward left, there are remaining 100 pixels.
onLayoutChildren() after aligns focused child, it calls
scrollDirectionPrimary() to move all children 100 pixels toward right,
so after onLayoutChildren(), scroll animation would continue
moving toward left for 100 pixels and stops at a perfectly
aligned state. When onLayoutChildren() compensates the 100 pixels,
the delta may violate the minScroll in scrollDirectionPrimary(), we
should ignore the limit cap in layout pass.

The CL also includes a fix in WindowAlignment.updateMinMax() which
was hidden by the extra delta cap check in GridLayoutManager.

Bug: 64931938
Test: testScrollAndRemoveSample1 testScrollAndRemove
testPreferKeyLine1 testPreferKeyLine2 testPreferKeyLine10000

Change-Id: Ieaa0b9f39fae0494f0fa779cf2647721eb388896
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
7124ccd6be8ffec8b9a26587b90e09f7b6e2d4ce 25-Aug-2017 Dake Gu <dake@google.com> leanback: fix selection after updating

scrollToView() reset mFocusPosition to -1 because the view's adapter
position was just invalidated. This is a regression since
ag/1986191.

Bug: 64913451
Test: testUpdateAndSelect1 testUpdateAndSelect2 testUpdateAndSelect3
Change-Id: I1f8c152afc4ee90a2ecae498d9e69d25d0dba3d1
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
7ef219d3d0ce885caeaa2064178159b72ddb273f 11-Aug-2017 Dake Gu <dake@google.com> leanback: fix items positions when height increases

In HorizontalGridView case: when item height increases, LM first
move the item up to put new centerY at original centerY, then LM
tries a vertical smoothScrollBy which is rejected because
canScrollVertically is false. Even we made canScrollVertically true,
the result effect is not what we want: the row first jump up then
slide down. The fix disables the vertical scroll code if
there isn't an ongoing vertical scrolling.

Bug: 63903129
Test: testUpdateHeightScrollHorizontal testUpdateWidthScrollHorizontal
testUpdateWidthScrollHorizontalRtl

Change-Id: Iae706c329af4ddf2812dee4a913871549a5c4280
(cherry picked from commit ce254049426b1465ab4a50f0079949a958c410e3)
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
0437f02b702e52f5cab3e18b5e9efa90f2414342 03-Aug-2017 TreeHugger Robot <treehugger-gerrit@google.com> Merge "leanback: fix selected position after remove all" into oc-support-26.0-dev
db7f8aa22ee060d1404897b58d60365fa94d0799 02-Aug-2017 Dake Gu <dake@google.com> leanback: fix selected position after remove all

The bug is when remove all items and then add few items, the focus
position was set to last item of newly added items.

The current code tries to be smart to select the next item when
focused item is removed. But it's impossible to know if there is
item after the removed one. So stop tracking the offset change
when focused item is removed.

Bug: 64235615
Test: GridWidgetTest

Change-Id: I654e906bccff04ce59ec840f9356a17717eaba2d
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
ff5d512149869cd9e26c627d8bc6894297da0368 27-Jul-2017 Dake Gu <dake@google.com> leanback: fix crash in onMeasure when preLayout is true

In some conditions e.g. items are removed before first visible child,
the position in prelayout needs to apply offset. Uses the value
mPositionDeltaInPreLayout in onMeasure() pass.
Also onMeasure should not call getViewForPosition for a child that
is currently in RV.

Historically, leanback LayoutManager implemented custom onMeasure
before RecyclerView introduce "autoMeasure" feature, eventually (in
next major release) leanback should switch to autoMeasure and remove
the custom onMeasure().

Bug: 63897029
Test: testPredictiveOnMeasureWrapContent

Change-Id: I92c70fe90dcc156fd5999bbb6358a1e84f82babb
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
da9e9ca1c05c8dae9dbb458930f5460ce152c9b5 15-Jul-2017 Dake Gu <dake@google.com> DO NOT MERGE RV: Fix onMeasure() wrong size in second run

There is some issue with ag/2480331 onMeasure fix.

In the second onMeasure() run, it should use measure result of first
run, defaultOnMeasure() gives a wrong size and causes the row being
unexpected short and cards are given a wrong Y value.

To be safe, also stop horizontal gridview's animation during transition.
This make sure that transition and RV item animator never conflicts
with each other. We cannot simply disable the RV layout during
transition, RV layout is still needed to capture the card size change
in open/close fast lane transition.

Bug: 63597044
Test: rowNotifyItemRangeChange rowNotifyItemRangeChangeWithTransition
Manually tested with play movie TV app

Change-Id: Ie6ffb99c96dc4cc914e0e92dca64cf2fde1452b0
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
4b79f71ae9a20ed0882170a15fdcbe0f3efba9d9 27-Jun-2017 Dake Gu <dake@google.com> leanback: more test for RTL alignment

Use a horizontal gridview and test both ltr and rtl.

Test: testPreferKeyLine1 testPreferKeyLine2 testPreferKeyLine10000
Bug: N/A

Change-Id: Id706a1c59331b07a761d1e0303b1cf9b437d88eb
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
2bafb2b6c1aae65b6e2169a3ecc5962858e0a4fd 27-Jun-2017 Dake Gu <dake@google.com> Revert "Revert "leanback: fix RTL alignment scroll limit" ag/2461878"

This reverts commit 720e04df8b5c85fb553da28a9a16ac3c4d270b08.

Change-Id: I7e829f652ee40b92d5608e8fd7ec624f287ed89a
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
8102c3837c3424673f9c995ae5f7e6dc52e4521c 27-Jun-2017 Aurimas Liutikas <aurimas@google.com> Merge "Revert "leanback: fix RTL alignment scroll limit" ag/2461878" into oc-support-26.0-dev
720e04df8b5c85fb553da28a9a16ac3c4d270b08 27-Jun-2017 Aurimas Liutikas <aurimas@google.com> Revert "leanback: fix RTL alignment scroll limit" ag/2461878

This change caused test breakage.

This reverts commit a9ca1446025f09e59d75fc0adb4bd10d2318c5de.

Change-Id: If83246fa86f1fe728e4e84df0d810e066d492191
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
7fdcf72c0be20e5c205f3f526d08ddca9f8896bc 27-Jun-2017 Dake Gu <dake@google.com> Merge "leanback: fix accessibility SCROLL_FORWARD for half visible item." into oc-support-26.0-dev
4297da73f24135b4448d2e36f6cf50499d095448 23-Jun-2017 Dake Gu <dake@google.com> leanback: fix accessibility SCROLL_FORWARD for half visible item.

If item is partially visible and focusable view is out side screen,
talkback will not focus to the focusable view. So we need also check
if item is fully visible.

Bug: 62663004
Test: testAccessibilityScrollForwardHalfVisible
testAccessibilityScrollBackwardHalfVisible

Change-Id: Ic946ff4e62a594cb63fcbabdfadf58a2d2587531
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
a9ca1446025f09e59d75fc0adb4bd10d2318c5de 26-Jun-2017 Dake Gu <dake@google.com> leanback: fix RTL alignment scroll limit

Bug: 62915202
Test: testItemMovedHorizontalRtl
Change-Id: Ieed9282a26461585e59769162efbfb9e5e7a4d75
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
3d079d099a4426538d5f532f947771991c7c7703 17-Jun-2017 Dake Gu <dake@google.com> leanback: fix fastRelayout for extra views added in preLayout

Then in post layout pass, fastRelayout() should invalidate after
the item that position is inconsistent with Grid position.
Previously this was relying on didStructureChange(), it is now
no longer safe to do so when prelayout may add items before first
child.

Bug: 62727612
also no longer reproduce Bug 36738635 after this change
Test: testMoveIntoPrelayoutItems

Change-Id: If67f6697b21c132ed5934bdc0fc2790ac4aae32c
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
4a28bc44a3a3b0494fa790206f72de013b5bdede 16-Jun-2017 Dake Gu <dake@google.com> leanback: fix wrong insert view index in fastRelayout

Bug: 62451353
Test: testWrongInsertViewIndexInFastRelayout
Change-Id: I2e127650d9374a2f061aa18790ec366448d0a886
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
b39fea5dfc74a6f7c9bb6d7c8bd2d8d04a8b9778 15-Jun-2017 Dake Gu <dake@google.com> leanback: clarify comments

And modify the test to be what exactly happened
in the original bug.

Bug: 62448243
Test: testLayoutWhenAViewIsInvalidated
Change-Id: Id4cfa3e97ad179474c172d6cfc45149b759097ae
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
e79b22a10835076150c04e96c52d8342b3af8409 09-Jun-2017 Keyvan Amiri <keyvana@google.com> Fixed fastRelayout when a view is invalidated

The views that are off the screen are cached and invalidated after a
layout. getAdapterPositionByView returns NO_POSITION for such views that
was causing crash in fastRelayout when getViewForPosition was called.
Now will skip laying out children when such a view is encountered.

Bug: 62448243
Test: testLayoutWhenAViewIsInvalidated
Change-Id: I5bb86682dba0bff6a4a5e77438b4735a2a720345
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
6d65d10a40b33e52a365610cb299771178a56824 27-May-2017 Dake Gu <dake@google.com> leanback: fix flaky testMoveItemToTheRight

The default wait animation timeout was same as animation duration.
Increase the default timeout to 6 seconds.

Bug: No
Test: testMoveItemToTheRight
Change-Id: Ia04080d5636a7da0d874ebfdb2c9fdaa2084c72a
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
2447d5ed8f4396fc6d1271df7f00ddf150bb5d41 27-May-2017 Dake Gu <dake@google.com> leanback: fix new flaky tests

Fails due to the size of view vs window

Bug: none
Test: testMoveItemToTheRight testPredictiveLayoutRemove2

Change-Id: I2675c87780a7931357d8539b7bc60bf7c4b78db1
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
bb1f461f210836de67bc68980409c5112889299a 27-May-2017 Dake Gu <dake@google.com> leanback: better cordinate scroll and itemanimator

Instead of scroll jump for child size change, onLayoutChildren() will
start a new scroll animation if previous scroll animation
was going to stop at unaligned position due to child size / position
changes.

Bug: 38339297
Test: testSwapAfterScroll

Change-Id: I3a5174f3a76c7269bfa5a5f6f28b1ba3875433db
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
c50aca4940c8ae508393278aa7a9c77a3f93f87d 26-May-2017 Dake Gu <dake@google.com> leanback: fix predictive animation move

1. When move items out, we need use addDisappearingView to
tell recyclerview to slide them out and those views will
not be pruned by scroll pass. This is done by adding
scrapList views in post layout fillDisappearingItems()

2. When move items out, we need check if item is moving
out side old adapter's range, prelayout needs layout extra
space for these cases.

Bug: 38339297
Test: testDontPruneMovingItem testMoveItemToTheRight
testMoveItemToTheLeft

Change-Id: I6e2c4ffeb8befb62f060247bb6c5345dc8f6bab7
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
134b0891c25facf14c53ef939846010284025ca9 20-May-2017 Dake Gu <dake@google.com> leanback: support predictive animation

1. supports predictive animation
2. no longer abort scrolling upon state.didStructureChange() is true.
So scroll animation will continue while item animation is running.

Bug: 38339297
Test: GridWidgetTest

Change-Id: If5b4a05e3dca9946395dd19204c2fcd59fa7a824
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
b6dde2cfe2fbe3151c52d92e3754288aab11a3b1 05-May-2017 Dake Gu <dake@google.com> leanback: support preferring key line over edge

We have a requirement that for WINDOW_ALIGN_HIG_EDGE, when there are
limited items, we prefer the first item aligned to keyline instead
of last item aligned to bottom padding. Introduced two APIs:
setPreferKeyLineOverLowEdge and setPreferKeyLineOverHighEdge.
"prefer keyline over low" is by default false.
"prefer keyline over high" is by default true since keyline is
before high edge and user usually prefer align to lower side.

In current implementation, the scroll_max is always decided by the
last item align to edge of bottom padding.

When we prefer keyline over high edge, scroll max is no longer only
decided by last item, it will be decided by both the last item and
first item. So we will need updateScrollMin/Max altogether.

The CL also clean up GridLayoutManager code, no longer need concept
of mScrollOffsetPrimary.

Bug: 38036352
Test: testPreferKeyLine1 testPreferKeyLine2 testPreferKeyLine10000
testScrollSecondaryCannotScroll testScrollSecondaryNeedScroll

Change-Id: Ia1e03b3eb8f5d78255090380684e35e5434172bc
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
93b954ed786fcd8c25c332cb921b9a51c525dfd4 04-May-2017 Dake Gu <dake@google.com> leanback: Fix rounding error for test assertion

On nexus5x, density is 2.625, textview layout_width 50DP is
translated 131.25pixels as width.
This causes rounding errors.

Bug: 37967115
Test: GridWidgetTest
Change-Id: I5c039caf4a6150fa5c8b137b3698a4d3be1e53fe
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
bc6713f8e2fa9702675dc3827591ed401b539723 29-Apr-2017 Dake Gu <dake@google.com> leanback: fix Item alignment.

1. Add support of RTL case.

2. The existing rule of using offset negative/positive as indication
of start/end padding does not make sense, switch to use percentage,
when percentage is 0, we will use low padding, for percentage 100, we
will use high padding.
Although this changes behavior, existing apps all use low padding
and set percentage to 0, so this will not break existing apps.

Test: testItemAlignmentVertical testItemAlignmentHorizontal
testItemAlignmentHorizontalRTL
Bug: 37794661

Change-Id: I3fe4a05642fa959812f6079b335c707570c6761a
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
3719c5ffa1f4b3a92c9186374100a8a6eb9572ec 13-Apr-2017 Dake Gu <dake@google.com> Fix flaky GridWidgetTest

We should not verify times of bindToView, because
holding DOWN key might causes smoothScroller which
overshoot and bounce back, which causes re-bind
of the early items.

Bug 36702460
Test: GridWidgetTest

Change-Id: I358bc520e08e6446e0bdaae3867f8f446eaef84b
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
e36e2ce16cbd6144ccd49e0b90ae4c587a08c8f3 10-Apr-2017 Dake Gu <dake@google.com> leanback: switch to use getAdapterPosition() for accessbility node

Disappearing ITEMs that were pushed out of the RV due to inserting
other items will crash when onLeftHiddenState changes
importantForAccessibility to yes. The item's viewLayoutPosition is
invalid, so we should use AapterPosition which will return -1 and
onInitializeAccessibilityNodeInfoForItem will bail out.

This bug will happen on any LM which does not support preLayout uses
viewLayoutPosition in onInitializeAccessibilityInfo.

Bug: 37096264
Test: testAccessibilityNodeInfoOnRemovedLastItem
testAccessibilityNodeInfoOnRemovedFirstItem
simulatesAccessibilityOfItemsBeingPushedOut
testAccessibilityOfItemsBeingPushedOut

Change-Id: I2804b5b55f2cbc917155cf8b902b9487f65b8e28
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
9e537f8a5bcc66a10280e3e34e866fd36790fcf8 28-Mar-2017 Dake Gu <dake@google.com> Merge "leanback: fix alignment when appending to last item." into nyc-support-25.4-dev
am: 9522574131

Change-Id: I89426dc66090db41d768a57958860ded08c02e72
af3606dbd00dd77a2b0d0950edf203247509ad4c 28-Mar-2017 Dake Gu <dake@google.com> leanback: fix alignment when appending to last item.

When append new item, it falls into fastRelayout() path
where LM didn't updateScrollMin/Max for the new item.

Bug: 32488200
Test: testAddLastItemHorizontal testAddMultipleLastItemsHorizontal
Change-Id: I660207a1fa48ff899e487b4ee457b57067ab4725
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
5c6d801d0e8ff28808a1a4b74db0519ebc22c317 14-Mar-2017 Dake Gu <dake@google.com> leanback: fix crash caused by wrong position
am: 5c4d5ee669

Change-Id: I294957d867fc1b6c79e436c975fcd1d30dc01336
5c4d5ee6697503c9c170fe240b0fe97792d83904 14-Mar-2017 Dake Gu <dake@google.com> leanback: fix crash caused by wrong position

When notifyDataSetChange() is called on Adapter that clears
its data and has stable ids, it will trigger a requestChildFocus()
in creating DISAPPEARING animation. At that time we should
use adapter position instead of view position. The adapter
position would be -1, so requestChildFocus() should do nothing.

Bug: 36139432
Test: GridWidgetTest.testRemoveLastItemWithStableId

Change-Id: Ief37156d58c2596c28002143cd14395ade24c6a5
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
abc73958d264e1eed7fd401a18be1d9ede8304eb 06-Mar-2017 Aurimas Liutikas <aurimas@google.com> A pass at fixing unchecked javac warnings.

Add types where needed.
Suppress places where we intentionally skip types.

Test: ./gradlew assemble still works
Change-Id: If1350e896948c7e27c212cf7b1a94f08b9131193
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
ee2249ba014169e1bf03bab5ba628de35d625685 05-Mar-2017 TreeHugger Robot <treehugger-gerrit@google.com> Merge "Add all missing @Override annotations."
e2104f4b5c8e3ad63570306a25e61502dfe4c418 05-Mar-2017 Aurimas Liutikas <aurimas@google.com> Add all missing @Override annotations.

Historically, we could not add @Override for certain methods due to
using older SDKs to build support library. That is no longer the case
so we can add @Override everywhere now.

Test: ./gradlew assemble assembleAndroidTest still works
Change-Id: I85e3c1157f0e253e3eab7584189f3bf95cf030e9
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
93f9a8fd25341dd983b4fa44d55723dcc21d62dd 04-Mar-2017 Dake Gu <dake@google.com> leanback: fix bad state of mFocusPosition am: b484ccb63d am: 88c7e86803
am: 85f9458aab

Change-Id: I207564714db1075f20af22e67d7fe1bc6eed5a2a
b484ccb63d546196c4b39f9a86a9c714006cfdc3 04-Mar-2017 Dake Gu <dake@google.com> leanback: fix bad state of mFocusPosition

When smoothScroller starts, the target position is cropped by adapter
size, leanback doesn't adjust its internal mFocusPosition, which
causes unable to focus to a child in onRequestFocusInDescendants().

The launcher could fix this by double check the position before pass
to setSelectionSmooth().

Bug: 32118060
Test: testSmoothScrollerOutRange
Change-Id: I5c4ea3b3b4776971f511b0d689e4f9c66bb5ae47
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
3600c90d8cd4c048574e5ce27c9463c176db4943 24-Feb-2017 Dake Gu <dake@google.com> Merge "Suppress layout handling during animateOut()" into nyc-support-25.2-dev am: 06ce76214e am: e887776d2c
am: 25f055493f

Change-Id: Id34b275429dc0f84631fa244ca1ee0d950e41985
5c05fc026b77c6387917560f8dbbbd4bff13bbf9 23-Feb-2017 Dake Gu <dake@google.com> Suppress layout handling during animateOut()

When playing video, data change of rows will automatically
pulls row in. This CL suppresses layout when animateOut()
until a specific animateIn() is called.

Bug: 35399351
Test: testAnimateOutBlockLayout testAnimateOutBlockSmoothScroll
testAnimateOutBlockScrollTo
Change-Id: Ie91137687e96f0d48a674c410041b9412c8945d6
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
acb3523a1af65a3a3440d720d4712d187c1ae32b 17-Feb-2017 Dake Gu <dake@google.com> Merge "leanback: Fix BaseGridView.animateIn()" into nyc-support-25.2-dev am: a1ee39dad4 am: 549ba57e73
am: 51f0998eda

Change-Id: I78c374683300dcd8ca03774958ce18ebf05547b2
8e06d698aa47dfdea344d2529f2879489eb4610f 16-Feb-2017 Dake Gu <dake@google.com> leanback: Fix BaseGridView.animateIn()

We dont need animateIn() since layout manager will slide in
items when user refocus or scroll to the item.
What is nicer is that the refocus logic already handles the
half way slide back. It is wrong that animateIn() slides a
fixed distance.

Also added support for animateOut for HorizontalGridView.

Add missing implementation of smoothScrollToPosition()

Bug: 35399351
Test: testAnimateOutResetByScrollTo testAnimateOutResetByFocusChange
testHorizontalAnimateOutResetByScrollTo testHorizontalAnimateOutRtl

Change-Id: Ia24c10ecf812cc1609acfa529b226ad87bf8da3a
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
35232c6eaeb9b99f390cb8ef0ac83bf45fa0b3fa 01-Feb-2017 Aurimas Liutikas <aurimas@google.com> Update test sizes for all currently @SmallTest.

SmallTests are supposed to complete in <200ms, Medium in <1s, and Large in <30s.
This CL updates the size of all the SmallTest based on excecution times from
presubmit test runners.

Test: None
Bug: 34854821
Change-Id: Iec73f7b6ef63b27cd3d8b19a8e8c7a9b91ff6729
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
b50e46706ac654b071942bd7f6ba059c14c9c29a 12-Oct-2016 Dake Gu <dake@google.com> Fix flaky GridWidgetTest more

Test: testFocusToFirstItem GuidedStepFragmentTest
Bug: None

Change-Id: I815284eb38ae181b18fe229a89c66eb288b5bfb2
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
21ac4a7e2cc753b4e4eeaa7edb9696add157ebf8 08-Oct-2016 Dake Gu <dake@google.com> Fix some flaky test

Use onLayoutCompleted() callback to wait layout.

Bug: 31771962
Test: GridWidgetTest

Change-Id: I30b72ed046bf2b7d24db7c6cd920e6b5825d5593
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
580515488b23ac8a39795d11aef985c0f05c467e 05-Oct-2016 Dake Gu <dake@google.com> leanback test: add more analysis info for flaky test

Test: nothing changed
Bug: none
Change-Id: Ie0bf45a04525b3d858a36e4330e6ea0f92af4904
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
03eae4e82fccd027e003a8d1544d3e9023404b14 03-Oct-2016 Dake Gu <dake@google.com> leanback: dont clearFocus in layoutInit

clearFocus() triggers unecessary focus search.

Test: added two test cases and run against all widget test
Bug: none
Change-Id: I78bad294d70b79d9d28c7d9015db8dca11081dfb
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
b3034c5df0d06f3dbfa466af19b1f014967f3d75 29-Sep-2016 Dake Gu <dake@google.com> Leanback: Fix flaky tests

Test: modified
Bug: 31771962

Change-Id: Id44203e3ed696bd63d2661f0a4e67a7e3a889d1b
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
42e7d6fafcde7bfe261dd7d8d75ee53ca0cd6790 28-Sep-2016 Aurimas Liutikas <aurimas@google.com> Replace usage of Instrumentation#runOnMainSync with ActivityTestRule#runOnUiThread.

ActivityTestRule#runOnUiThread handles exceptions correctly and thus should be used
in place of Instrumentation#runOnMainSync. This changes most of these use-cases
except for tricky cases.

Bug: 31801302
Test: ran support-design:connectedCheck and support-appcompat:connectedCheck
Change-Id: I70904aef89e6bbc2a3cff1b3be697258d666e316
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
754cb29c50f09a83251dd4bb633ba445b2411adb 27-Sep-2016 Aurimas Liutikas <aurimas@google.com> Switch from deprecated size annotations to new ones.

Switching from android.test.suitebuilder.annotation.*Test
to android.support.test.filters.*Test.

Bug: 30074170
Test: manual - ran small appcompat tests, they still run and pass
Change-Id: Idce58b988c8e1a809f7f5fae67e0e235159d9fc5
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
3103f63e99d47573823957f7aa34308555873221 20-Sep-2016 Aurimas Liutikas <aurimas@google.com> Fix all operator wrapping issues in leanback library.

Test: code still compiles, there should be no code/logic changes.
Change-Id: I6e97e2ba429c323022055263540c3fb6d0003822
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
9480eb4e94e429726b07eba133ffbc6450b8d795 09-Sep-2016 Alan Viverette <alanv@google.com> resolve merge conflicts of 7243922 to master

Change-Id: I34d800e77c4b1a440eb67191b4697213a2e6239e
c39d9c75590eca86a5e7e32a8824ba04a0d42e9b 25-Aug-2016 Alan Viverette <alanv@google.com> Add @RestrictTo(GROUP_ID) annotations to match @hide docs annotations

Also removes some unnecessary @hide annotations on classes that wouldn't
have shown up in docs anyway due to package/private visibility or NO_DOCS.

Bug: 27937193
Change-Id: Iab127a5a3ce57a2d61965d3cef782621eb3859f5
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
06dd2b62f48746fbb1447b171db882d0a458368e 25-Aug-2016 Dake Gu <dake@google.com> Convert GridWidgetTest to AndroidJUnit4

Change-Id: Idea0f1d83c133d2654d0e81b3df758f266026f92
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
d1f491d37de7420febb0345cb6b254f6eef196eb 18-Aug-2016 Dake Gu <dake@google.com> Merge "Fix GridWidgetTest.testFocusableViewAvailable"
61afde67696f46ecb82eb0a26038c6fbf3383c2c 18-Aug-2016 Dake Gu <dake@google.com> Fix GridWidgetTest.testFocusableViewAvailable

Bug 30943873

Change-Id: Id1175e0fdf1bdb801acbb8853a62bf128b5b2f77
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
fad2335f169d36b7b6f2c0ec8ddfe6c0094c2072 16-Aug-2016 Aurimas Liutikas <aurimas@google.com> Annotate more support lib tests with size annotations.

Turns out we were not running these tests because they are not
annotated with @SmallTest, @MediumTest, or @LargeTest.

Bug: 30892341
Change-Id: I4ed870d4c1a456c1fcd446243179c005653a041d
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
7a38d7478bb7a0ec2bfc87e50b77c93fe5f66bfe 22-Jun-2016 Dake Gu <dake@google.com> leanback: add test case for selection with adapter change

Bug 29279610

Change-Id: Iec421d841033bafc41f8dc836d1a8e8e541e5ce0
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
88c1cf67c6437e5e0f386b65b9e79605a111bd03 31-May-2016 Dake Gu <dake@google.com> GridLayoutManager: call focusableViewAvailable() inside layout pass

Onboard fragment layouts VerticalGridView, then starts a transition.
focusableViewAvailable() is called in a post runnable, causing a
scroll animation run together with transition.

Move focusableViewAvailable() call into layout pass, if framework
focuses to the view, gridview will align to the view in layout pass,
avoids extra scroll animation.

Bug: 28274171

Change-Id: Id348746c4d93a23527e7e829801e0a63e4622d77
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java
7d7bf9541e0203ec67df0fe53367e2c126355b9e 07-Jun-2016 susnata <susnata@google.com> Moving Leanback unit tests.

Moving Leanback unit tests from frameworks/support/v17/tests
to frameworks/support/v17/leanback/tests.

Change-Id: I1dbfe73434a61d721f8666911c688930a8cb32cb
/frameworks/support/v17/leanback/tests/java/android/support/v17/leanback/widget/GridWidgetTest.java