Commit Graph

11075 Commits

Author SHA1 Message Date
Abodunrinwa Toki
119d10deb9 Merge "Implement Stateful TextClassifier APIs." into pi-dev 2018-03-28 20:58:47 +00:00
Jorim Jaggi
13c4b14e92 Merge "Add different animation for transitions with translucent apps" into pi-dev 2018-03-28 19:09:08 +00:00
Jan Althaus
08a75e0ee7 Merge "Change TextClassification to use RemoteActions" into pi-dev 2018-03-28 17:04:58 +00:00
TreeHugger Robot
59ecb96da9 Merge "Improve the JavaDoc for SOFT_INPUT_STATE_UNSPECIFIED" into pi-dev 2018-03-28 16:29:56 +00:00
Adrian Roos
95712fe0cb Merge "DisplayCutout: Add clarifying images to LAYOUT_IN_DISPLAY_CUTOUT_MODE_SHORT_EDGES" into pi-dev 2018-03-28 14:56:19 +00:00
Jorim Jaggi
98a9d206be Add different animation for transitions with translucent apps
None of the existing animation really work with transitions in
which a translucent app is appearing/disappearing. Add a separate
animation for that and change all existing transitions to these
new transition types in case we detect that only translucent
activities are appearing/disappearing.

Test: Sharesheet animations
Test: go/wm-smoke
Change-Id: Iffe57b7664dddf647d723c91d115ade60c12ad33
Fixes: 37953606
Fixes: 70730519
Fixes: 72649981
Fixes: 72686618
2018-03-28 16:46:41 +02:00
Adrian Roos
813a4bee76 Merge "DisplayCutout: Fix attrs.xml javadoc" into pi-dev 2018-03-28 14:29:08 +00:00
Adrian Roos
ad1da31764 DisplayCutout: Add clarifying images to LAYOUT_IN_DISPLAY_CUTOUT_MODE_SHORT_EDGES
Fixes: 76462084
Test: make doc-comment-check-docs, open $ANDROID_BUILD_TOP/out/target/common/docs/doc-comment-check/reference/android/view/WindowManager.LayoutParams.html, navigate to LAYOUT_IN_DISPLAY_CUTOUT_MODE_SHORT_EDGES
Change-Id: I8a9df979cbb062896fa874a163414a4f8a095b21
2018-03-28 15:00:45 +02:00
Jan Althaus
20d346eafe Change TextClassification to use RemoteActions
Behavioural changes:
- Every action now as an icon, and instead hints via RemoteAction whether
  the icon should be shown or not.
- Icons are now taken from the app default, not the activity.
  (That way, we can construct a lightweight resource backed Icon)
- Legacy intents are no longer parceled for security reasons.
  (TextClassificatio wasn't parcelable in O)
- TextClassifications built with the new API will always return null for
  getIntent, but provide a getOnClickListener.
- Accessibility descriptions are now provided for action mode items.

Testing changes:
- Removed URI checks from TCM test because PendingIntent hides this info.

Bug: 73950205
Test: atest FrameworksCoreTests:TextClassificationManagerTest
Test: atest FrameworksCoreTests:TextClassificationTest
Test: atest CtsViewTestCases:TextClassificationManagerTest
Test: atest CtsViewTestCases:TextClassifierValueObjectsTest
Test: atest CtsWidgetTestCases:TextViewTest
Test: atest CtsWidgetTestCases:EditTextTest
Change-Id: I6706d2c342a8bbb9de0146a48c8b8aac9d9c7d83
2018-03-28 11:41:39 +02:00
Abodunrinwa Toki
88be5a6cee Implement Stateful TextClassifier APIs.
Outstanding work tbd in other CLs
- Introduce request objects with session Ids
- Implement character based indexing for Selection events.

This CL hides the old Logger API but still keeps running so that we can
check that the modifications to the new API does not break anything.
We will remove the old Logger once we're convinced this is stable.

Please refer to I3c9ceea0863099fc4f0a5ce5e823c648ee9c4521 for previous
reviews related to this CL.

Bug: 74461129
Test: bit FrameworksCoreTests:android.view.textclassifier.TextClassificationManagerTest
Test: bit CtsViewTestCases:android.view.textclassifier.cts.TextClassificationManagerTest
Test: bit CtsWidgetTestCases:android.widget.cts.TextViewTest
Test: bit FrameworksCoreTests:android.widget.TextViewActivityTest
Change-Id: Iea744f1fa5964b4399290c31863ebeffa99af8d3
2018-03-28 02:54:36 +01:00
Lucas Dupin
41c25cee06 Unload wallpaper bitmap and release HwuiContext
To decrease sysui memory usage

Test: adb dumpsys meminfo
Fixes: 74534423
Change-Id: I3cb4a5927e8d6b20a7aeba3fda253f0ba4cc18f6
2018-03-27 16:30:07 -07:00
Yohei Yukawa
a754785cf9 Improve the JavaDoc for SOFT_INPUT_STATE_UNSPECIFIED
SOFT_INPUT_STATE_UNSPECIFIED is a tricky mode because its de facto
spec is that the system can do whatever it thinks valid.

Despite the fact that this is the default value for newly created
Windows, unfortunatelly there is no clear spec about what would happen
regarding whether the system thinks the software keyboard should be
shown or dismissed.

This CL doesn't change the behavior, but it just attempts to clarify
the behavior in JavaDoc.

Bug: 73793727
Bug: 77152727
Test: make -j doc-comment-check-doc
Change-Id: I69dab7c7018da89ba57fe2f5c8e718ef95985307
2018-03-27 15:47:45 -07:00
TreeHugger Robot
828ffa45a1 Merge "Recover dataset picker when view fail to autofill." into pi-dev 2018-03-27 19:48:44 +00:00
Felipe Leme
49f08edf7d Recover dataset picker when view fail to autofill.
When a dataset is selected, the framework tries to autofill all views belonging
to it. But if one (or more view) failed to autofill, we should let the user
recover by tapping the view again.

This scenario typically happens when views are recycled.

Test: atest MutableAutofillIdTest#testViewGoneDuringAutofillCanStillBeFilled
Test: atest CtsAutoFillServiceTestCases # manually retrying flaky failures

Fixes: 76149637

Change-Id: I7a6352c68b4a7d5e4cb80a7346c66efd831f21c8
2018-03-27 10:50:36 -07:00
Adrian Roos
87526af33c DisplayCutout: Fix attrs.xml javadoc
Fixes an issue with the javadoc for windowLayoutInDisplayCutoutMode.
Removes @tags from enum javadocs, because they get merged into a table,
which does not support inner @tags. Instead, use <code> tags which work
in that context.

Also replaces @see tags with {@link} tags in the attr javadoc, because
the enum value table gets appended, but it is invalid to have text after
an @see tag.

Change-Id: Ib97e9aa82e173d99a467132a0d403793d7297e61
Fixes: 76448223
Test: make doc-comment-check-docs, verify output at $ANDROID_BUILD_TOP/out/target/common/docs/doc-comment-check/reference/android/R.attr.html#windowLayoutInDisplayCutoutMode
2018-03-27 17:44:19 +02:00
Derek Sollenberger
7b837616ae Remove deprecated android.graphics.Canvas APIs
Bug: 73777445
Test: compile
Change-Id: Ie68f2fbe3c3be82f3c20529d535875d4cc204b7c
2018-03-27 14:36:50 +00:00
TreeHugger Robot
992d078542 Merge "Integrate autofill session lifecycle with URL bar changes when on compat mode." into pi-dev 2018-03-27 05:15:20 +00:00
Felipe Leme
0c8ce322a3 Integrate autofill session lifecycle with URL bar changes when on compat mode.
The contents of a browser URL bar is typically changed for 2 reasons:

1.User entered a new URL.
2.Form was submitted and the URL changed.

On scenario #1, the current session should be canceled, while on #2 it should be
committed. Scenario #2 is already handled when the service sets a SaveInfo, so
this CL handles the other cases:

1.Focus on URL bar is ignored so it does not trigger a new partition.
2.If URL bar changed and service didn't set a SaveInfo, the session is canceled.

Fixes: 76027553

Test: manual test with Chrome
Test: new tests on VirtualContainerActivityCompatModeTest:
      testFocusOnUrlBarIsIgnored()
      testUrlBarChangeIgnoredWhenServiceCanSave()
      testUrlBarChangeCancelsSessionWhenServiceCannotSave
      testUrlBarChangeCancelsSessionWhenServiceReturnsNullResponse
Test: atest CtsAutoFillServiceTestCases

Change-Id: I19d2aa4c8b25def0d5eca1c59cfdc2ffe33dd388
2018-03-26 11:55:31 -07:00
TreeHugger Robot
e3d2c41344 Merge "Fix SurfaceView Background" into pi-dev 2018-03-23 23:21:04 +00:00
TreeHugger Robot
352da45bbb Merge "Properly set input type on autofill compat mode." into pi-dev 2018-03-23 22:49:25 +00:00
Todd Kopriva
1057fc37f5 Merge "docs: fixing bug 36982027 and 30636712, typos" into oc-mr1-dev
am: 5b0a4eea8d

Change-Id: Iff4044c52e7ccdf6f7f879b9d5457ce1458b21bd
2018-03-23 22:11:38 +00:00
TreeHugger Robot
87f9818319 Merge "Added TestApi for default focus highlight" into pi-dev 2018-03-23 22:04:04 +00:00
Felipe Leme
c380e18e31 Properly set input type on autofill compat mode.
Test: atest CtsAutoFillServiceTestCases:android.autofillservice.cts.VirtualContainerActivityTest#testAutofillSync\
CtsAutoFillServiceTestCases:android.autofillservice.cts.VirtualContainerActivityCompatModeTest#testAutofillSync

Fixes: 76173462

Change-Id: Ie400ed5d4426febc34b02f68d6be4d86bc11034f
2018-03-23 14:04:23 -07:00
Robert Carr
386fd70434 Fix SurfaceView Background
When refactoring to not use global transaction when setting
position  I failed to account for the forwarding done by the
background wrapper.

Bug: 76099976
Bug: 76110573
Test: Manual. go/wm-smoke
Change-Id: I2cff8e571f12e69cefbd1ea35538fd71beaf1ec4
2018-03-23 13:48:48 -07:00
John Reck
6045d66aa2 Merge "Add isPivotSet/resetPivot" into pi-dev 2018-03-23 19:26:50 +00:00
Felipe Leme
63df4e248a Merge "Improved autofill logging and dumping." into pi-dev 2018-03-23 05:48:26 +00:00
Evan Rosky
e1a0b347a0 Added TestApi for default focus highlight
Bug: 62864111
Test: View_DefaultFocusHighlightTest
Change-Id: I2a74de3274351f0a7662715e723c25f4d7f8f6ed
2018-03-22 19:37:50 -07:00
Felipe Leme
da9ea34c02 Improved autofill logging and dumping.
Bug: 76027553
Test: adb shell dumpsys autofill

Change-Id: I19ef88c71334c62292ed982e43d7a7e89b35f10e
2018-03-22 16:43:28 -07:00
TreeHugger Robot
e9a7cb8158 Merge "Fix smart_linkify_enabled flag." into pi-dev 2018-03-22 22:56:58 +00:00
rongliu
3dac23bd4d Assign -1 as default value of window type and owner uid.
Bug: 74622357
Test: Test with ag/3738015 together. Layers without buffers have window
type -1 and owner uid -1.

Change-Id: I6f010bf1327671a06874b75bc0a63a3fb010ced7
2018-03-22 13:44:29 -07:00
Abodunrinwa Toki
6563833cf3 Fix smart_linkify_enabled flag.
The flag should only ensure that smart linkify calls behave in the
legacy way instead of totally disabling linkify.
Also, to keep the flag consistent with smart_selection_enabled and
smart_text_share_enabled flags, the flag should only disable the
SmartLinkify (i.e. Linkify.addLinksAsync) feature not TextClassifier
APIs (i.e. TextClassifier.generateLinks).

Also fixes issue with non-focusable TextViews by firing the primary
action instead of showing the floating toolbar. (b/73156794)

Bug: 75967597
Bug: 73156794
Test: bit FrameworksCoreTests:android.text.util.LinkifyTest
Test: bit CtsTextTestCases:android.text.util.cts.LinkifyTest
Test: bit FrameworksCoreTests:android.view.textclassifier.TextClassificationManagerTest
Test: bit CtsWidgetTestCases:android.widget.cts.TextViewTest
Test: bit FrameworksCoreTests:android.widget.TextViewActivityTest
Test: manual - checked behaviour turning flag on/off
Test: manual - checked behaviour with TextView.setFocusableInTouchMode(true/false)
Change-Id: I541f60161b9cd63ce7e57235607500f2fb0841e7
2018-03-22 19:25:14 +00:00
TreeHugger Robot
f9593d62b3 Merge "Adds support for a fall-back model when the language-specific model is not found." into pi-dev 2018-03-22 10:50:47 +00:00
John Reck
8686e1f157 Add isPivotSet/resetPivot
Bug: 74825533
Test: CTS view tests in topic
Change-Id: Ib918bc5830d53ee0a83b13ab53ac6e8dd86c869f
2018-03-21 16:42:59 -07:00
Lukas Zilka
0fcacdddf4 Adds support for a fall-back model when the language-specific model is not found.
Bug: 74720318

Test: Builds and fall-back works.
Test: bit FrameworksCoreTests:android.view.textclassifier.TextClassificationManagerTest
Test: bit CtsViewTestCases:android.view.textclassifier.cts.TextClassificationManagerTest
Change-Id: I5491af628c8406e42ca2cc971ad3a8b708ceb315
2018-03-21 12:25:41 +01:00
TreeHugger Robot
efcd6e1ceb Merge "Fix confusing Log.e message in SpellCheckerSession" into pi-dev 2018-03-21 03:25:49 +00:00
Yohei Yukawa
0cf7b31012 Fix confusing Log.e message in SpellCheckerSession
With this CL, calling SpellCheckerSession#close() multiple times will
no longer show the following error message in logcat, which was quite
confusing.

  E SpellCheckerSession: ignoring processOrEnqueueTask due to
    unexpected mState=TASK_CLOSE scp.mWhat=TASK_CLOSE

Note that with a recent CL [1], we now more often see the above
message.  A typical scenario is:

  1. App creates SpellCheckerSession x.
  2. App calls x.close().
  3. x is garbage-collected. ART calls x.finalize(), which internally
     calls x.close() again.

 [1]: I4e00c3a2cec93d1dacff20546e481fe757279661
      9b64367193

Bug: 72974646
Fix: 72974646
Test: Made sure that no error message in question with a test app
      that calls SpellCheckerSession#close() multiple times then
      trigger SpellCheckerSession#finalize().
Change-Id: Ie9690860e6d0406dc6746cd03c28f693e65c1dde
2018-03-21 01:15:02 +00:00
kopriva
6a7f167f67 docs: fixing bug 36982027 and 30636712, typos
Test: make ds-docs

Bug: 36982027
Bug: 30636712

Change-Id: I70d6d216c714a31b7e762a203a27f6124642131e
2018-03-20 17:59:48 -07:00
TreeHugger Robot
d0661a081d Merge "Distinguish between createSurface error codes." into pi-dev 2018-03-21 00:40:19 +00:00
Andrii Kulian
982973934d Merge "Don't set override display info for ActivityView" into pi-dev 2018-03-20 22:01:27 +00:00
Andrii Kulian
f0379de259 Don't set override display info for ActivityView
ActivityView content should never be letter- or pillar-boxed
and it doesn't have overscan areas, so we can skip setting
display override info for a virtual display that backs it.

Bug: 72220802
Test: Manually resize with ActivityView demo app
Change-Id: I211449eaea50ec5d3214d63e7eb93a1d2ed2c0fd
2018-03-20 19:43:34 +00:00
TreeHugger Robot
036a2409fc Merge "Dispatching display list recreations also to transient views" into pi-dev 2018-03-19 21:09:31 +00:00
TreeHugger Robot
89b182890c Merge "SurfaceView: Expose hook for subclass to position child surfaces." into pi-dev 2018-03-19 20:32:35 +00:00
Tracy Zhou
017f44be88 Merge "Move default pip position to right above the shelf." into pi-dev 2018-03-19 20:15:14 +00:00
TreeHugger Robot
e24c95fac5 Merge "Fixed isVisibleToUserForAutofill() again." into pi-dev 2018-03-19 19:57:14 +00:00
Felipe Leme
a946d88011 Merge "Fixed AutofillManager.isAutofillUiShowing()" into pi-dev 2018-03-19 18:27:20 +00:00
Felipe Leme
cb2e83da36 Fixed AutofillManager.isAutofillUiShowing()
It was using & instead of &&

Test: not really
Bug: 75652829
Bug: 74527545

Change-Id: Ib8ca79ba6ec35d2fc97f042db2152b03d334ce30
2018-03-19 11:26:23 -07:00
Tracy Zhou
43513086d2 Move default pip position to right above the shelf.
This change takes care of the flow from WindowManagerService to
PinnedStackController, all the way to PipTouchHandler. It also
introduces a WindowManager hook that allows Launcher to pass in
shelf visibility and height. A separate change is made to send
signals from Launcher to SysUI. (ag/3724896)

Bug: 73961893
Change-Id: I2ff54e78bc2dc35c806b902464048b051a4d6394
Test: atest CtsActivityManagerDeviceTestCases:ActivityManagerPinnedStackTests
2018-03-19 17:41:08 +00:00
Felipe Leme
7008e70234 Fixed isVisibleToUserForAutofill() again.
This method should return false when the accessibility provider does not have
an accessibility info for a node. Otherwise, the Save UI is not triggered on
compat mode.

Bug: 75017711
Test: manual verification

Change-Id: Ic1759520a43b4333aef9f8f0e1d93f8d378b131c
2018-03-19 09:55:39 -07:00
John Reck
564284759f Don't use IPC in isolateProcess
Fixes: 74395652
Test: hwuiunit passes in 'shell stop' state (pseudo-isolated process),
      manually checked non-isolated processes still have working vsync
      via systrace of RT animations demo

Change-Id: I630ea011dc7eb2efa265b25673d3304b3b2510d3
2018-03-16 17:34:02 -07:00
Robert Carr
27a800afac SurfaceView: Expose hook for subclass to position child surfaces.
To position child surfaces of a SurfaceView in sync with the main view root
we need to expose the frame number and RT callback. This is of course not API
but we plan to use it with app controlled transitions, for manipulating window
surfaces within a view hierarchy.

Bug: 70341013
Test: Builds
Change-Id: Ibb3645461fac36bda8ab163867fcb2087cadee37
2018-03-16 15:15:17 -07:00