зеркало из https://github.com/mozilla/gecko-dev.git
ce402fbf10
We used to scroll in addTab to make sure a new tab created by a close-tab-undo at the start or the end of the list was made visible instead of staying where it was created off the edge. We're now taking care of that in selectTab (where it should have stayed in the first place), where the select in that case occurs between the time when the new tab is added to the adapter and when the layout gets updated. In the case where the new tab is at the start, that means the check 'position < layoutManager.findFirstCompletelyVisibleItemPosition()' in selectTab reads '0 < 0', which fails (which is why we need the new check for 'position == 0'), but the check 'position > layoutManager.findLastCompletelyVisibleItemPosition()' for a tab added at the end reads 'new_lengh -1 > old_length - 1' which already passes, so we don't need a special case for undo-tab-close adds at the end in selectTab. Tabs added at the end by a normal "create new tab" still scroll for the same reason. Robotium was confused by the duplicate 'add_tab' ids from the tab strip and the tabs panel, so I renamed one of them. Also note that the 'getTabId' added to TabStripItemView for testing already exists on TabLayoutItemView, but the two classes don't share a common base. MozReview-Commit-ID: BzG2r8BSs90 --HG-- rename : mobile/android/tests/browser/robocop/src/org/mozilla/gecko/tests/testTabStripPrivacyMode.java => mobile/android/tests/browser/robocop/src/org/mozilla/gecko/tests/testTabStrip.java extra : rebase_source : b2859647d9e26cdca24e1b03065d3c62e20f7b1b extra : source : 119ee2655404e277c13d0e436fba1cad1272797e |
||
|---|---|---|
| .. | ||
| anim | ||
| color | ||
| drawable | ||
| drawable-hdpi | ||
| drawable-hdpi-v11 | ||
| drawable-large-hdpi-v11 | ||
| drawable-large-v11 | ||
| drawable-large-xhdpi-v11 | ||
| drawable-large-xxhdpi-v11 | ||
| drawable-ldrtl-hdpi | ||
| drawable-ldrtl-large-hdpi | ||
| drawable-ldrtl-large-xhdpi | ||
| drawable-ldrtl-large-xxhdpi | ||
| drawable-ldrtl-xhdpi | ||
| drawable-ldrtl-xxhdpi | ||
| drawable-nodpi | ||
| drawable-v12 | ||
| drawable-v21 | ||
| drawable-xhdpi | ||
| drawable-xhdpi-v11 | ||
| drawable-xlarge-hdpi-v11 | ||
| drawable-xlarge-xhdpi-v11 | ||
| drawable-xlarge-xxhdpi-v11 | ||
| drawable-xxhdpi | ||
| drawable-xxhdpi-v11 | ||
| drawable-xxxhdpi | ||
| layout | ||
| layout-large-v11 | ||
| layout-xlarge-v11 | ||
| menu | ||
| menu-large | ||
| menu-v11 | ||
| menu-xlarge | ||
| raw | ||
| values | ||
| values-land | ||
| values-large | ||
| values-large-land-v11 | ||
| values-large-v16 | ||
| values-sw240dp | ||
| values-sw360dp | ||
| values-sw400dp | ||
| values-v11 | ||
| values-v13 | ||
| values-v14 | ||
| values-v16 | ||
| values-v19 | ||
| values-v21 | ||
| values-w400dp | ||
| values-xlarge-land-v11 | ||
| values-xlarge-v11 | ||
| xml | ||
| xml-v11 | ||