1119126 - [TextSelection] The paste icon overlaps keyboard if tapping caret quickly before keyboard shows up

Status: VERIFIED FIXED

(Core :: DOM: Selection, defect, P2)

Description

[Gerry Chang [:gchang]]

### Steps:
1. Open Messages app
2. Click pencil icon on the upper-right corner to create a new message
3. Type characters in the message body input field
4. Long press on any character to trigger utility bubble
5. Tap copy icon
6. Tap other area to make keyboard disappeared
7. Tap the input field again
8. When the caret (water drop) shows, tap the water drop right away
9. The paste icon will overlap keyboard

### Expected:
1. The paste icon should not overlap keyboard

### Actual:
1. The paste icon overlaps keyboard

### Reproduce rate
always

### Version:
Gaia-Rev        d4dac29613076bdba3cb8adc217deadb08a2ac20
Gecko-Rev       https://hg.mozilla.org/mozilla-central/rev/70de2960aa87
Build-ID        20150107160220
Version         37.0a1
Device-Name     flame
FW-Release      4.4.2

Comment 1

[Gerry Chang [:gchang]]

Attachment: The image of paste icon overlapping keyboard

Comment 2

[(Inactive after June) George Duan [:gduan] [:喬智]]

Hi Morris,
gaia need a event to update position in shortcut mode after resizing...

Comment 3

[Peter Chang[:pchang]]

(In reply to George Duan [:gduan] [:喬智] from comment #2)
> Hi Morris,
> gaia need a event to update position in shortcut mode after resizing...
George, as I discussed with Morris, gaia could help to fix this issue. Let us know if you still need gecko's help.

Comment 4

[(Inactive after June) George Duan [:gduan] [:喬智]]

Attachment: PR to master

Hi Alive,
could you review this patch?

Currently gaia handle bubble's displaying when shortcut and single tap mode, so we also need to hide it once window is resized.

Comment 5

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8549339 [details] [review]
PR to master

Squash commits please.

Also, is gecko still telling us to show the bubble again after hiding it? IMO only hide seems strange.

Comment 6

[(Inactive after June) George Duan [:gduan] [:喬智]]

Merged to master: https://github.com/mozilla-b2g/gaia/commit/57b771e6976e9ce58a4c058ee00bfcb999c304f4

This is a workaround, however, we still have some cases like iframe's iframe resize with shortcut.
Gecko should try to send the same events queue of cursor event as selection caret. (see bug 1110039)

Comment 7

[(Inactive after June) George Duan [:gduan] [:喬智]]

Hi Morris,
as offline discuss, in shortcut mode, we still need gecko to send proper event while frame is resized , which is just like selection mode. Or bubble might flash while user tap other type of input column.

Comment 8

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Working on this.

Comment 9

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Attachment: Send updateposition after reflow when selection is collapsed.

Comment 10

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Attachment: gaia wip patch.

Comment 11

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Attachment: Send updateposition after reflow when selection is collapsed v2.

Comment 12

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a5ac1e7a22b5

Comment 13

[howie [:howie]]

Triage: feature blocker.

Comment 14

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Attachment: Send updateposition after reflow when selection is collapsed v3.

Looks like test may failed if we call FlushPendingNotification inside Reflow callback. So, don't call FlushPendingNotification in the reflow callback in this patch.

Comment 15

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Attachment: Send updateposition after reflow when selection is collapsed v4.

Rebased.

Comment 16

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Attachment: Send updateposition after reflow when selection is collapsed v5.

Fix typo.

Comment 17

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

try https://treeherder.mozilla.org/#/jobs?repo=try&revision=f8037cca6072

Comment 18

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Attachment: Send updateposition after reflow when selection is collapsed v6.

Fix build failed.
try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=9d1717398813

Comment 19

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Roc, would you mind review this patch again per comment 14? Thanks.

Comment 20

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 8557778 [details] [diff] [review]
Send updateposition after reflow when selection is collapsed v6.

Review of attachment 8557778 [details] [diff] [review]:
-----------------------------------------------------------------

::: layout/base/SelectionCarets.cpp
@@ +1300,5 @@
>      // bubble in drag mode. So, don't dispatch event here.
>      if (mDragMode == NONE) {
>        DispatchSelectionStateChangedEvent(GetSelection(),
> +                                         SelectionState::Updateposition,
> +                                         false);

This is probably unwise. It's not a good idea to simply not flush and hope the layout is OK.

I suggest instead of putting this in a post-reflow callback, you put it in a scriptrunner. Running script is disabled during reflow, so your scriptrunner will run after reflow in a state where it's ok to flush layout. But you have to be careful not to trigger another reflow that will post another scriptrunner and put you in an infinite loop...

Comment 21

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Attachment: Send updateposition after reflow when selection is collapsed v7.

Use script runner to dispatch event.

Comment 22

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=ddb2ebd4ea66

Comment 23

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Attachment: Send updateposition after reflow when selection is collapsed v8.

Unfortunately, add script runner would causes infinite reflow loop... After some experiment, I found that zero range selection also means selection is collapsed. But I really only care about the selection which has range. So I add a check for range count and it looks fine. This patch also send event after scroll so that we can update bubble after scrolling as well.

Comment 24

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=5ec7cef9058d

Comment 25

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

Comment on attachment 8558340 [details] [diff] [review]
Send updateposition after reflow when selection is collapsed v8.

NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: Copy/paste bubble position may be wrong after scroll or reflow.
Testing completed: on master 
Risk to taking this patch (and alternatives if risky): low risk.
String or UUID changes made by this patch: Not available.

Comment 26

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a372a90a9475

Comment 27

[Carsten Book [:Tomcat]]

sorry had to back this out for test failures like https://treeherder.mozilla.org/logviewer.html#?job_id=6402550&repo=mozilla-inbound

Comment 28

[[:fabrice] Fabrice Desré]

https://hg.mozilla.org/projects/cypress/rev/c13e7799a147

Comment 29

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

With latest gaia, it looks good. Can be checkin again.

try https://bugzilla.mozilla.org/show_bug.cgi?id=1119126

Comment 30

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/78f919c2fb5a

Comment 31

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/78f919c2fb5a

Comment 32

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/9631a35e0d14

Comment 33

[Pi Wei Cheng [:piwei] (inactive)]

This issue is verified fixed on Flame 3.0 and 2.2.

Observed behavior: Following STR, the paste icon moves to above the message input field instead of overlapping with keyboard.

Device: Flame 3.0
BuildID: 20150406010204
Gaia: ef61ebbe5de8c2c9fc2a8f74a12455044c3b82e9
Gecko: 4fe763cbe196
Gonk: b83fc73de7b64594cd74b33e498bf08332b5d87b
Version: 40.0a1 (3.0) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0

Device: Flame 2.2
BuildID: 20150406002503
Gaia: a6351e1197d54f8624523c2db9ba1418f2aa046f
Gecko: c3335a5d3063
Gonk: ebad7da532429a6f5efadc00bf6ad8a41288a429
Version: 37.0 (2.2) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0