Closed Bug 1374482 Opened 7 years ago Closed 7 years ago

Intermittent test_accessiblecaret_selection_mode.py AccessibleCaretSelectionModeTestCase.test_long_press_changes_focus_from_textarea_to_contenteditable | AssertionError: u'A' != u''

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla58
Tracking Status
firefox57 --- fixed
firefox58 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: TYLin)

References

Details

(Keywords: intermittent-failure)

Attachments

(2 files)

Priority: -- → P5
Since you had so much luck with bug 1399204, any chance you'd be willing to look at this one next? :). Still affects Beta pretty frequently.
Flags: needinfo?(tlin)
(In reply to Ryan VanderMeulen [:RyanVM] from comment #12)
> Since you had so much luck with bug 1399204, any chance you'd be willing to
> look at this one next? :). Still affects Beta pretty frequently.

Sure. I'll take this.

test_long_press_changes_focus_from_input_to_contenteditable() fails because it selects nothing. It uses _test_minimum_select_one_character() to ensure the carets appear after changing the focus. 

I think we can test selecting a word after changing the focus to reduce the test's completity to make the test more stable.
Assignee: nobody → tlin
Status: NEW → ASSIGNED
Component: Disability Access APIs → Selection
Flags: needinfo?(tlin)
Priority: P5 → P3
Comment on attachment 8919118 [details]
Bug 1374482 Part 1 - Simplify focus changing tests for caret selection mode.

https://reviewboard.mozilla.org/r/190026/#review195228
Attachment #8919118 - Flags: review?(mtseng) → review+
Comment on attachment 8919119 [details]
Bug 1374482 Part 2 - Remove x and y parameters from _test_minimum_select_one_character().

https://reviewboard.mozilla.org/r/190028/#review195230
Attachment #8919119 - Flags: review?(mtseng) → review+
Pushed by tlin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2495b6f2d15e
Part 1 - Simplify focus changing tests for caret selection mode. r=mtseng
https://hg.mozilla.org/integration/autoland/rev/1ef55e0ab730
Part 2 - Remove x and y parameters from _test_minimum_select_one_character(). r=mtseng
https://hg.mozilla.org/mozilla-central/rev/2495b6f2d15e
https://hg.mozilla.org/mozilla-central/rev/1ef55e0ab730
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: