Merge "Add appropriate (tiny) offset to widget drops so they land correctly." into ub-launcher3-burnaby-polish

Merge "Increase spacing between icons in All Apps on tablets." into ub-launcher3-burnaby-polish

Merge "Normalizing app icons based on the standard icon guidelines" into ub-launcher3-burnaby-polish

Merge "Cleanup keyboard code." into ub-launcher3-burnaby-polish

Cleanup keyboard code.

This should be the last keyboard CL.
- Fix bug: couldn't focus All Apps button in some cases when the All
Apps column was skipped over. Also added test case for this.
- Stop explicitly passing countX and countY to handleKeyEvent, as
these had to match the matrix dimensions anyways.
- Rename createSparseMatrix() - there were 3 methods of the same name,
but all had different purposes. This is confusing both from a
readability standpoint and also when looking at stack traces.

Change-Id: I08ba8411674fcea43a608856c114dee8dbd22398

Hide the focus indicator view when going to a fullscreen page.

Bug: 25649853
Change-Id: I9f2f221020548b48d816924a1803a1a70d11c412

Fullscreen pages (e.g. Now) are focusable via keyboard.

Change-Id: I962551651aeb9ac7b57bb51b195adc3140db30d8

Make sure pages are always accessible via left/right arrow keys.

- Handle NextPageFirstItem as first focusable item in reading order
- Handle PreviousPageLastItem as last focusable item in reading order
- Check the hotseat after the workspace in both cases above
- Dpad horizontal navigation (left/right) uses these as a last
  resort (Rule3) to guarantee an item takes focus if a page exists

Note that it is necessary to search for a focusable item because
widgets are not yet focusable.

Bug: 25591057
Change-Id: I953648bd76c657d660a38427fdd4108bf9963c23

Merge "Add special column for All Apps button in FocusLogic's sparse matrix." into ub-launcher3-burnaby-polish

Add appropriate (tiny) offset to widget drops so they land correctly.

Also removed some unused parameters I found on the way.

Bug: 25191623
Change-Id: Ib9a30db106467c114cc8b54a5b13ed3e88162d56

Normalizing app icons based on the standard icon guidelines

Bug: 18245189
Change-Id: Iaadcddbe3f966733a13b2e1fb60ba09a8b3aef9a

Merge "Scaling down the icon before applying the user badge   > This also saves memory only create one new bitmap while loading instead of two" into ub-launcher3-burnaby-polish

Merge "Disallow scrolling if workspace is in transition." into ub-launcher3-burnaby-polish

Increase spacing between icons in All Apps on tablets.

Change-Id: If90332098f749295da07f4faca707a7a6965666b

Merge "Keeping the focus indicator on the widget view, while a child is focused" into ub-launcher3-burnaby-polish

Adding margin to Drag layer instead of checking for right insets at every place

Bug: 25692432
Change-Id: I853f41a17c54b30b7772b9fd8556d9465de84752

Keeping the focus indicator on the widget view, while a child is focused

Bug: 25199581
Change-Id: Ie08d1438ae0c1b5d4fcaa4d642508c1b9db0f5d4

Only fetching firstInstallTime for an activity when required

Change-Id: Ia16c05718938849f4250802d8770ddfe6dbef055

Scaling down the icon before applying the user badge
  > This also saves memory only create one new bitmap while loading instead of two

Bug: 25027252
Change-Id: Ia594dd65a30f32e99404190f649b334615c9e17f

Merge "Allowing widgets to be focusable" into ub-launcher3-burnaby-polish

Merge "Skip to the end of the folder open animation in battery saver mode." into ub-launcher3-burnaby-polish

Don't let folder be replaced with final item twice.

Bug: 25732317
Change-Id: I77b3ddc8951583bebfb908fda934c9b80ddebce0

Tying LauncherAppState lifecycle with LauncherProvider

The content provider is the first component started with the main process,
which makes it ideal for initiating LauncherAppState

Change-Id: I03db82a8949ad1376af53a0033592f421dbf0b15

Merge "Setting statusbar and nav bar colors in theme instead of code" into ub-launcher3-burnaby-polish

Merge "Focus doesn't leave folders without also closing them." into ub-launcher3-burnaby-polish

Focus doesn't leave folders without also closing them.

- Pressing Tab wraps around to the first item from the folder name,
  and vice versa when pressing Shift+Tab.
- When tapping off the folder while editing the text, the folder
  requests focus. We handle the following cases from that state:
    - Pressing an arrow key or Tab gives focus to the first item.
    - Pressing Shift+Tab gives focus to the last item.
- Fix slight corner case where moving from folder name to an item
  didn't update mIsEditingName to false. So when clicking off of
  the folder, it gave focus to the folder (as mentioned above)
  instead of closing the folder like it usually does when icons are
  focused. Not a huge deal, but still worth fixing.
Bug: 25687579
Change-Id: I1bec844c8ccd09529a11b9e3a1d92b3bdf7b2eb3

Disallow scrolling if workspace is in transition.

Because going to overview mode scales down the workspace, it was
thinking the touch was moving even though your finger was still. If
the "movement" was large enough, it was treated as a scroll, causing
jank. This was especially prevalent on tablets due to their size.

Bug: 25779718
Change-Id: Idb7833e0087bd24ca840f6afc451bf221f6bc047

Merge "Stop requesting focus on folder if it was closed without animating." into ub-launcher3-burnaby-polish

Merge "Handle errors gracefully in WallpaperCropActivity." into ub-launcher3-burnaby-polish

Handle errors gracefully in WallpaperCropActivity.

(cherry picked from commit c984cde00840d8594a8b203748cb6e4f9967fd7c)
Now instead of crashing, it says "Couldn't load image."

Bug: 25326319
Bug: 25656670
Change-Id: I71471f4b26f7c23dee40b60772ddd798f67b409e

Stop requesting focus on folder if it was closed without animating.

Bug: 25092436
Change-Id: Ia6b72ec8c9fa02fc159457b81fef12f6fd394769

Fixing crash when uninstalling an app destroys the folder

Bug: 25666490
Bug: 25677822
Change-Id: Id71c04df0a5546ba95239ba2e3f7aabb697375a4

Revert "Revert "Reverting folder cell width on portrait phones.""

This reverts commit 1974e98f5ec4713e50ba64ec32a7607c92ddce9b.

Change-Id: I7c5d86de187989c085f676933ce3f1cbd57ed8a7

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

Skip to the end of the folder open animation in battery saver mode.

Bug: 25666809
Change-Id: I867d56a34fc5e0ad6aaabd8b6df47be05ec31966

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

Add special column for All Apps button in FocusLogic's sparse matrix.

The All Apps button creates a number of edge cases, mainly because it
causes the hotseat to sometimes have an extra column than the workspace.
Previously, we sort of swept these problems under the rug by simply
ignoring the All Apps button if other icons were present in the hotseat,
with the assumption that those other icons should get focus instead of
the All Apps button. (If possible, we want to stay in the same column
when moving from the workspace to the hotseat.) But this doesn't always
work, as in the attached bug where the hotseat doesn't get focus at all
when the All Apps button is an obvious candidate for it.

By adding a specialized column in the focus matrix for the All Apps
button, we ensure that moving down to the hotseat stays within the
original column when possible, while also allowing the focus to switch
to the All Apps button if appropriate. Furthermore, we take care to skip
over the All Apps column when necessary in order to maintain all
previous functionality.

Bug: 25590522
Change-Id: I5d6a8ee69de8834314c4689246fe7d54329b2eef

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

Allowing widgets to be focusable

> Pressing 'enter' enters the focus inside the widget.
> If the widget has only one focusable, it directly performs click
> Pressing 'esc' brings the focus back to the parent host.

Bug: 25199581
Change-Id: Ia7fd5d929f5945d61173abd349660a5efbe74afc

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

Setting statusbar and nav bar colors in theme instead of code

Change-Id: I4da50809d14f5fa4e47df47fcfd99df7b713950d

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