943071 - Selection monocles aren't positioned correctly after the browser shifts due to skb display

Status: VERIFIED FIXED

(Firefox for Metro Graveyard :: Input, defect, P2)

Description

[Jim Mathies [:jimm]]

str:

1) compose a new message in gmail
2) tap the text body and enter some text
3) tap the entered text to invoke selection monocles

result: spotty support for selection, and dragging monocles sometimes doesn't work.

Comment 1

[Jim Mathies [:jimm]]

Attachment: regression fix

Comment 2

[Jim Mathies [:jimm]]

Attachment: google compose fix

Comment 3

[Jim Mathies [:jimm]]

Comment on attachment 8355222 [details] [diff] [review]
regression fix

The result of a copy paste error in bug 932783.

Comment 4

[Jim Mathies [:jimm]]

Comment on attachment 8355239 [details] [diff] [review]
google compose fix

This one is odd. We have a content editable test that applies to the body of a page. This test is currently succeeding. However in this gmail case caretPositionFromPoint fails in similar markup. The one difference is that the body is in a sub document of an iframe, which might be causing problems.

I'm going to try and work up a test case for caretPositionFromPoint and file it as a new bug in layout.

In the mean time, in cases where cpfp fails for unknown reasons, we should fall back on the selectAtPoint apis.

Comment 5

[Jim Mathies [:jimm]]

https://hg.mozilla.org/integration/fx-team/rev/fc6a9efc7bb7
https://hg.mozilla.org/integration/fx-team/rev/a65d449c77f9

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/fc6a9efc7bb7
https://hg.mozilla.org/mozilla-central/rev/a65d449c77f9

Comment 7

[Jim Mathies [:jimm]]

Comment on attachment 8355222 [details] [diff] [review]
regression fix

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 950832
User impact if declined: buggy selection control
Testing completed (on m-c, etc.): yes
Risk to taking this patch (and alternatives if risky): low
String or IDL/UUID changes made by this patch: none

Comment 8

[Jim Mathies [:jimm]]

Comment on attachment 8355239 [details] [diff] [review]
google compose fix

[Approval Request Comment]
Bug caused by (feature/regressing bug #): none
User impact if declined: gmail selection issues
Testing completed (on m-c, etc.): yes
Risk to taking this patch (and alternatives if risky): low
String or IDL/UUID changes made by this patch: none

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/c9fa98eb8103
https://hg.mozilla.org/releases/mozilla-aurora/rev/22408820aa27

Comment 10

[u279076]

Kamil, can you please verify this is fixed in the latest Nightly and Aurora builds?

Comment 11

[Kamil Jozwiak [:kjozwiak]]

Went through the following issue during iteration #21 testing. Used the following builds:
- http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014-01-16-00-40-03-mozilla-aurora/
- http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014-01-16-03-02-50-mozilla-central/

Before testing the issue, I reproduced the original issue mentioned in comment #0 using the following build:
- http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013/11/2013-11-26-03-02-01-mozilla-central/

For the following test cases, I copied several different paragraphs of text and pasted them into a draft email. From there, used the selection monocles.

- Went through the original issue outlined in comment #0
- Ensured that taping inside the body created a selection monocle that is usable
- Ensured that taping on "Email Address" and "Subject" placed the monocle in those area's when selected
- Ensured that when the text is selected, two monocles appear at the ends of the selected text
- Ensured that both of the monocles can be used to either select or deselect more text
- Ensured that the monocles can be used while the email draft is using both "full" and "snapped" views
- Ensured that the selection animation isn't janky nor has any artifacts when highlighting text
- Went through all of the above test cases using a real mouse and ensured that a user can still select text without any issues
- Went through the following test cases in full and filled view using several variations of filled view

Found a few issues relating to monocles/touch selection that affects this and everything else, created the following bugs:
- Bug # 960886
- Bug # 960889
- Bug # 960896

Comment 12

[Manuela Muntean [Away]]

Verified as fixed for iteration #22, with both latest Nightly and Aurora on Surface pro 2 with Windows 8.1 64-bit.