Revert "Reverting folder cell width on portrait phones."

This reverts commit b25e684a766362422f5693f7a2189910c1275584.

Change-Id: Ie86adc4b35c73ab7ef78d73225b437adc4751172

Reverting folder cell width on portrait phones.

The extra folder cell space added in ag/790800 looks bad on phones in
portrait mode, so reverting it here. However, we keep the extra space
on tablets or phones in landscape mode, since it provides more room
for long app names (bug 22462641).

Bug: 25662215
Change-Id: I2a37b884458ee557c9b8cff0c3edef16bfc50efb

Merge "Handle more cases in FocusHelper#handleHotseatButtonKeyEvent()." into ub-launcher3-burnaby-polish

Handle more cases in FocusHelper#handleHotseatButtonKeyEvent().

The first of these cases seems like they should already have been
present, and the last couple allow you to switch pages by hitting
arrow keys on the edges of the hotseat.

Bug: 25589939
Change-Id: I9378b209250f7dc376fa97efde979bcee2979537

Don't add offset to CellLayout's fullscreen children.

Bug: 25411084
Change-Id: Ifbcd0a53cf04a7fc29386f4f09aaf7f972e7354a

Added unit tests for b/25434120 regression.

Change-Id: I55b10274c4bb63ec6fd3138236e78a04e3eac91a

Merge "Make sure user can always get to next page with keyboard." into ub-launcher3-burnaby-polish

Merge "Stop explicitly snapping to page when navigating with keyboard." into ub-launcher3-burnaby-polish

Make sure user can always get to next page with keyboard.

There's a case where hitting the left/right arrow in the first/last
column wouldn't give focus to an icon on the next page because it is
vertically too far from the first icon to be considered. This makes
for a bad user experience, because there is no way to switch pages
with the keyboard. So now we brute-force search for an appropriate
icon if none was found in this case.

Bug: 25434120
Change-Id: Ifdead0e3b458717ccb33e2f0ec7c15f1fcce4b95

Ignoring DeadObjectException in startListening

> As of android M, the platform somtimes throws DeadObjectException
instead of TransactionTooLargeException

Bug: 25305031
Change-Id: I3a98db8a5041aef01640d16f9beb71b72bf598f1

Stop explicitly snapping to page when navigating with keyboard.

When an icon grabs focus, it already snaps to its own page, so
the explicit snapToPage() is redundant. And in the case where no icon
will grab the focus, the explicit call puts us in a weird state
because the page snaps but the focus doesn't change (see bug).

(This happens, for instance, if you are moving from an icon on the
bottom right of page 1 and the only icon on page 2 is in the top
left. In this case, the second icon won't take focus because the
angle is too wide according to FocusLogic#handleDpadHorizontal().)

Bug: 25434120
Change-Id: I17b0ef7d86426a2599bc948bf2313dca7e112364

Stop clearing focus when snapping to a new workspace page.

Clearing the focus was causing the first child in the new page to
immediately take focus, which caused FocusIndicatorView jank when
using a keyboard.

Bug: 25256728
Change-Id: I5ab31ebc3fe370d7ac9e9792b30dab3467023738

Set parallax offset to be 0.5f for default wallpapers.

This ensures they are centered as they were without any offset.

Bug: 25347076
Change-Id: Id773cd89463d47b350e60878c00e1112e0244414

Setting correct height for search bar widget options bundle.

- Seperate search bar height from its top and bottom paddings
- Report this height plus 8dps top and bottom for the widget options
- Use a separate bottom padding to adjust spacing between QSB and
  the workspace
- Add tests to verify the height of the search bar in both normal and
  tall modes (ran on Nexus 6, 7, and 10)
Change-Id: Ia7557785d1b38194eee869d0460456e8f33447d5

Merge "Disabling the desaturation of fast scroll items." into ub-launcher3-burnaby-polish

Disabling the desaturation of fast scroll items.

Change-Id: I7e90e3b35dc6473ba5e2319abcea5c0c76cbd708

Merge "Add null check in Launcher onNewIntent() before moveToDefaultScreen()." into ub-launcher3-burnaby-polish

Fixing null check when calling mutateOnMainThread

Bug: 25269169
Change-Id: I74f7fb833a374f59a16487099f1bf72b086e88ee

Add null check in Launcher onNewIntent() before moveToDefaultScreen().

There is a race condition that occurs primarily (maybe entirely) with
Android Auto, it seems because they hijack the Home intent or something
similar. I'm not exactly sure how Android Auto works, but if I pair my
phone with the Desktop Head Unit (car dashboard emulator), I can repro
the NPE fairly easily by simply force closing Android Auto and then
disconnecting my phone from the DHU. If I don't force close Android
Auto, then pressing home launches Android Auto or other apps that I
assume handle some custom intent, such as Car Home Ultra, instead of
normal Home intents such as Launcher3/Google Now Launcher. So I think
what's happening is that, when the phone is disconnected from the car,
Android Auto restores and launches the real home intent (Launcher 3)
around the same time that it destroys the previous home intent
(Android Auto, Car Home Ultra, etc.). This could cause the NPE if both
intents are actually Launcher 3, as is the case when Android Auto is
already closed, because mWorkspace is set to null in
Launcher#onDestroy() (something like onNewIntent() --> post() called
--> onDestroy() --> post() runs). This is consistent with the fact
that I can guarantee a repro if I use postDelayed() instead of post().

Long-winded explanation aside, I think this fix is safe, especially
since we already have a null check for mWorkspace in onNewIntent(),
just not inside the post().

Bug: 24610231

Change-Id: I42f75b83946f375d947be1961a1f2a03a3707a84

Merge "Set search bar height before setting up views in Launcher." into ub-launcher3-burnaby-polish

Don't add cells to FocusLogic sparse matrix if they aren't focusable.

Previously, any child of ShortcutAndWidgetContainer was added to the
matrix, causing widgets (which aren't focusable) to be considered as
potential targets to gain focus when an arrow key was pressed. But if
the algorithm chose them, they couldn't take the focus so nothing
happened (i.e. the focus stayed on the app/folder it was on before).

Bug: 25126768
Change-Id: Id55fc310f7f58fb8795cce51dcefe4fd1210f788

Merge "Hide the "Set Wallpaper" action bar when clicked." into ub-launcher3-burnaby-polish

Hide the "Set Wallpaper" action bar when clicked.

This already happens everywhere except for the case where an external
app such as Photos started the wallpaper picker. So not only is this
more consistent, it also prevents crashes and other potential issues
caused by clicking "Set Wallpaper" too many times.

Bug: 25026717

Change-Id: I63c6a0ce68656e6e68094e88074a836d8c1034f9

Do not animate folder when it is closed while changing activity state

Bug: 21733536
Change-Id: I36aa3041b9300c917e320b511e00a1721e44f854

Set search bar height before setting up views in Launcher.

This ensures that the QSB widget options are set correctly the first
time they are used.

Bug: 24704753
Change-Id: I2bb13ff012b6f13ca076deed61f0b08a7037e2fa

Import translations. DO NOT MERGE
am: 6ce85953ac  -s ours

* commit '6ce85953acf0399965c6c8a75f5e6eea5a643678':
  Import translations. DO NOT MERGE

Import translations. DO NOT MERGE

Change-Id: I4f645e8f071559ad56ee7963934f1039b06d6e96
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I1cea6130293678ef0d5da961d11428f14e62ee25
Auto-generated-cl: translation import

Merge "Fixing disabled icon getting changed back to normal state on long press" into ub-launcher3-burnaby-polish

Fix regression where keyboard couldn't switch between workspace pages.

Bug: 25122302
Change-Id: Id1d7628753ce7ef37af50e8594e541dcb6bea13e

Fixing disabled icon getting changed back to normal state on long press

Change-Id: I4f35c1cbecfef7891f8b1187ba3be0af695d4fd5

Fixing ConcurrentModificationException in tests
am: ec7c9178dd

* commit 'ec7c9178dd1f17cf8479acd4e0b4575a91cfb255':
  Fixing ConcurrentModificationException in tests

Fixing ConcurrentModificationException in tests

Change-Id: I7aabc62aa249dae979366ce2871604f5822f3139

Merge "Refactoring getPreferenceKey method" into ub-launcher3-burnaby-polish

Refactoring getPreferenceKey method

This method was returnning a constant and getting inlined by proguard.

Change-Id: I87348e25b21483adc1b27d16f99dec4b73205701

Using UiSelector instead of BySelector

Change-Id: Ie912cc7291a2533f6686bdb34e108be15ce244ac

Import translations. DO NOT MERGE
am: f58538672a  -s ours

* commit 'f58538672aaeb1d0d00a9c65a773f5e251003c3b':
  Import translations. DO NOT MERGE

Import translations. DO NOT MERGE

Change-Id: If930ef94a3abe3e27123d1fe3553b0a3a453ffec
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: Iee53921c75c785c7d73dd375b4e044761a5a7246
Auto-generated-cl: translation import

Merge "Adding tests for rotation preference Updating the gradle file to run tests directly from AndroidStudio" into ub-launcher3-burnaby-polish

Adding tests for rotation preference
Updating the gradle file to run tests directly from AndroidStudio

Change-Id: Iac23dfc6f995477f2406071fbfab3a16ee58ce6f

Merge "Fix issue with source params not reporting predicted apps." into ub-launcher3-burnaby-polish

Fix issue with source params not reporting predicted apps.

Bug: 25020080
Change-Id: Iaa67ccbb3e3a6c264aaa4fa3c915faad8a69bbbc

Add LauncherCallbacks method to add additional search widget options.
am: 775455c4a8

* commit '775455c4a8266655bf43b943e969d42ac32609ec':
  Add LauncherCallbacks method to add additional search widget options.

Merge "Preventing activity to restart when keyboard is connected or keyboard type changes" into ub-launcher3-burnaby-polish

Add LauncherCallbacks method to add additional search widget options.

Bug: 25000458
Change-Id: I5e8d48617568569bc6e4f284749c3ed9fb20a68f

Close some Cursors and call super.onFinishInflate().

Android Studio gave these warnings, and they seem like good ideas.

Change-Id: I9a4d3997fd18e86e007e4511e1ac9e8a75509265

Merge "Fix "The specified child already has a parent" IllegalStateException." into ub-launcher3-burnaby-polish

Fix "The specified child already has a parent" IllegalStateException.

The problem was due to a race condition between removing a prebound
widget view from the drag layer and adding the same view to the
workspace upon dropping it; if you let go of the widget immediately
after picking it up, the latter happened before the former.

Specifically, the flow was: long-click a widget --> drop --> remove
the view from the drag layer if it's not null (it is, so nothing
happens) --> the view is finally bound/inflated and added to the drag
layer --> add the view to the workspace --> already has a parent.

There are actually 2 problems here: one is that the bind/inflate is
asynchronous, and can therefore happen after dropping the widget view
being inflated, and the other is that the view is added to the
workspace even though the transition has barely started (we usually
ignore drops if the transition is less than half complete). It turns
out that this second problem was also due to a race condition, this
time between dropping a widget or app onto the workspace and calling
LauncherStateTransitionAnimation.dispatchOnLauncherTransitionStart().
If the drop happened before the dispatch, as in the case of the
crash, then the drop was accepted because the transition progress was
still 1.0 from the previous transition.

I fixed the first problem by removing the drag layer widget view
in Launcher where it is potentially used instead of Workspace. And I
fixed the second problem by setting mTransitionProgress to 0 in
Workspace.onLauncherTransitionPrepare().

I also added some debugging logs.

Bug: 23896857
Change-Id: I66944e6d3f23b70dea15f7fb01af0763a1bfcbda

am 28c6b963: Showing a permission prompt for direct call shortcuts

* commit '28c6b963d4ccac73ab857c839420513dd560945c':
  Showing a permission prompt for direct call shortcuts

Preventing activity to restart when keyboard is connected or keyboard type changes

Bug: 20820889
Change-Id: Ib0b21c04290607ddd67cef2003e6d89f5d6492d8

Fixing issue with overview panel buttons.

Bug: 22699895
Change-Id: I8a2f05a6008343327e34cbd3169e4056729e0b75

Increase folder cell width slightly.

The increased breathing room makes it more likely that long app names
will fit in the folder cells without being cut off.

Bug: 22462641
Change-Id: I110ede040f9e8fdddbf0c4e7a395ac71435559f3

Showing a permission prompt for direct call shortcuts

Bug: 24580949
Change-Id: Idc92f9f5979a730f43b4905af2bc47b6834dc7a7

Import translations. DO NOT MERGE

Change-Id: I30deecd0fedace5365a877a8333206e7cb015afb
Auto-generated-cl: translation import

Merge "Actually call LauncherTransitionable.onLauncherTransitionStep()." into ub-launcher3-burnaby-polish

am 37860923: Merge "Added launcher callback to specify search bar height." into ub-launcher3-burnaby

* commit '37860923619b1e467975a74764dff03ab657bdc7':
  Added launcher callback to specify search bar height.

Merge "Added launcher callback to specify search bar height." into ub-launcher3-burnaby

am d6b4037b: Grid adjustments, specifically for Ryu.

* commit 'd6b4037b093445fefa7d036da339dd549fae7a9b':
  Grid adjustments, specifically for Ryu.

Added launcher callback to specify search bar height.

The search bar can now be be normal or tall. When it is set to tall,
the hotseat and page indicators move down so that the workspace isn't
compressed quite as much.

Change-Id: Id92a946eab3a93524999f92efd847a501a95f002

Verify that the widget id is valid, before binding the widget

Bug: 23820213
Bug: 24054205
Bug: 24582322
Change-Id: I25a09b0a2a706812a7d82729b5dc554ffba84a2a

Merge "Null check in PackageInstallerCompat" into ub-launcher3-burnaby-polish

Merge "Update the widget spans after the provider changes" into ub-launcher3-burnaby-polish

Null check in PackageInstallerCompat

Bug: 24582224
Change-Id: I2cd4ca12d48b3316a8a519092118d9e2970d944d

Grid adjustments, specifically for Ryu.

> Less than 14% of the screen is used for left/right workspace padding.
  (To be clear, this means 7% for the left and 7% for the right.)
> Hotseat icons are the same size as workspace icons.
> Far left and right hotseat edges line up with workspace edges.
> All Apps button is scaled down 8dp.

Bug: 24003478

Change-Id: Idb057ab0f6ba29e7fd93feac7573ca2e07c7e839

Using 23 instead of Build.Versoin.M until the final SDK is available.

Change-Id: I9f8f6fe049afcd0ceb0797ddc803a2e77bab0437

Moving a few testing classes to a separate package

Change-Id: Ied1c063de3e938695493e4937f554686e0719dad

Merge "Fixing overscroll effect when the navigation bar is opaque" into ub-launcher3-burnaby-polish

Import translations. DO NOT MERGE

Change-Id: Ibbe258e46dd396e9186a47b3881796e12f689e33
Auto-generated-cl: translation import

Verifying that the table was successfully created during initialzation

Table creation sometimes fails scilently, which leads to a crash loop.
This way, we will try to create a table every time after crash, so
the device would eventually be able to recover.

Bug: 24263894
Change-Id: Ic0169d33f4092042423afa24320182786c544d47

am 53d3be63: (-s ours) Import translations. DO NOT MERGE

* commit '53d3be63a5ab056280279bcb13089faae0e7796f':
  Import translations. DO NOT MERGE

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 22482938
Change-Id: I758693c47b7d200da62bc1e9df4d0faaf9b147f2

am 3a3517fb: Add widget size options for QSB.

* commit '3a3517fb6c17120c6d11e5c95a0de4e81205d0ae':
  Add widget size options for QSB.

Add widget size options for QSB.

Bug: 24704753
Change-Id: I3e16c1f67a67b08b3843f7f38312447bdfa6a91e

Actually call LauncherTransitionable.onLauncherTransitionStep().

Previously, it was only called at the start and end of the transition;
now it is called as the animation interpolates. Specifically, a dummy
ValueAnimator is played alongside the transition animation and calls
dispatchOnLauncherTransitionStep() as it goes.

One place where this is important is in Workspace, where
mTransitionProgress is used to determine things like whether the
workspace should accept a drop - hence the bug that caused apps dragged
from All Apps to vanish when dropped before the transition ended.

Bug: 24215358
Change-Id: I32cd633c53557305caf84e87c9a4d4f07eef2223

Update the widget spans after the provider changes

Bug: 24003213
Change-Id: I51ad5919a808ef10385d62a48a94589eefcb06bc

Merge "Jailing the saved instance state of all the dynamically generated views" into ub-launcher3-burnaby-polish

Jailing the saved instance state of all the dynamically generated views

Using itemId instead of generating a new id for each item. This is because
if the process gets killed, View.generateId will get reset but we will still
receive the generated item id map in onRestoreInstance. This will cause
conflicts with newly generated item ids.

We wrap all the generated homescreen views inside a single sparse array. This
ensures that we do not cause any conflict with dynamically generated views in
other parts of the UI.

Bug: 16840760
Change-Id: I6fe69c2e1dd463402f51222715fae31b9d4dd240

Merge "Fixing regression when removing last item from folder." into ub-launcher3-burnaby-polish

Import translations. DO NOT MERGE

Change-Id: Id66043bc65557ef6a886bb1bec6ef18590a7056b
Auto-generated-cl: translation import

Fixing regression when removing last item from folder.

- The regression caused the folder and all its contents to be removed
  before the new item could be moved to the place of the folder.  In
  addition, in this case, the LauncherModel would get out of sync with
  the db because we move the last item to the workspace, but don’t
  remove it from the contents, so deleting the folder would remove the
  item’s mapping in sBgItemIdMap.

Bug: 24620815
Change-Id: I17137f28b0f1617a890488c7a9c5b8a9e8df0e91

am c84e6e19: Disabling widget debug

* commit 'c84e6e19363c590bbfbac17e9ab7cc61a04d552a':
  Disabling widget debug

Disabling widget debug

Change-Id: I876a904358ce455c97a7a25a34c66af5e9316469

Fixing regression in removing widgets via accessibility.

- Regression was introduced in ag/765323, but not visible in normal
  drag and drop because the view is already removed from the parent.
  However, when accessibility is enabled, it was only removing it from
  the db, but not from the parents.

Change-Id: I37483d0cd6306d270441b11e6f05247d58b88521

Merge "Do not reinflate/rebind widgets if we are going the rebind the full workspace anyway." into ub-launcher3-burnaby-polish

am 058c21db: Ensure accessibility delegate is set on all apps icons.

* commit '058c21db49f4f058b7d84c8de55d3c649918ab70':
  Ensure accessibility delegate is set on all apps icons.

Ensure accessibility delegate is set on all apps icons.

Bug: 24605237
Change-Id: I801e2aca1732c021b06d996d907f368f97277cf9

Make sure that the dimensions are not negative when drawing pending widget

Bug: 24585203
Change-Id: I3d328d0e1fe81d2704bb5b9d4b340a367bfb9249

Do not reinflate/rebind widgets if we are going the rebind the full
workspace anyway.

Bug: 23820213
Bug: 24582781
Change-Id: I2cb4927916fed64a69100beb41ee14653ea1d70a

Merge "Replace Long.compare() with implementation." into ub-launcher3-burnaby-polish

Replace Long.compare() with implementation.

- Long.compare() was added in API level 19.

Change-Id: Ic7020c941403c10daa6feacdc737c6a82676ada0

Merge "Fixing regression in search bar animation." into ub-launcher3-burnaby-polish

Fixing regression in search bar animation.

- The other case should fall through to the animation even
  when there is no search bar overlay.

Change-Id: Id91bea78b56934cc1c3379777068adc100f11dba

Merge "Adding progress information for preload icons in content description" into ub-launcher3-burnaby-polish

Merge "Fixing 'Item removed' announced twice when using accessible DnD" into ub-launcher3-burnaby-polish

Adding progress information for preload icons in content description

Bug: 24086091
Change-Id: Id8fb024973f4affd998627d30077e2764b07edc4

Merge "Don’t create dummy app info." into ub-launcher3-burnaby-polish

Don’t create dummy app info.

Bug: 24539514
Change-Id: I5d14a4d9993e072451f8e7ed9a4cea219ae5d246

am 91dd0872: (-s ours) Import translations. DO NOT MERGE

* commit '91dd087271807980638a490fd8072d28a39ada8d':
  Import translations. DO NOT MERGE

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 22482938
Change-Id: Id607b75f77fe89ee801336c8bc3e7d7db3ad59fa