Closed Bug 1672726 Opened 7 months ago Closed 19 days ago

Add test for implicit pointer capture for Fission

Categories

(Core :: DOM: UI Events & Focus Handling, task)

task

Tracking

()

RESOLVED FIXED
90 Branch
Fission Milestone M8
Tracking Status
firefox90 --- fixed

People

(Reporter: edgar, Assigned: edgar)

References

(Blocks 2 open bugs)

Details

Attachments

(3 files)

I tried to add test for implicit pointer capture for fission in bug 1655239. But touch doesn't support dispatching synthesized touch event asynchronously, I also did a quick try but failed to make async dispatch work.

File this bug to investigate how to add a test for implicit pointer capture for Fission,

  • Make async dispatch touch event work.
  • Or use synthesized native touch event.

I prefer to make async dispatch touch event work if possible, given that I would like to try platform-independent approach first.

Fission Milestone: --- → M7
Severity: -- → N/A
Assignee: nobody → echen

Adding tests can be done in M8, doesn't block Beta.

Status: NEW → ASSIGNED
Fission Milestone: M7 → M8
Blocks: 1669729

So it can be used in DispatchInputOnControllerThread template.

Attachment #9215924 - Attachment description: Bug 1672726 - Add test for implicit pointer capture for touch; → Bug 1672726 - Part 3: Add test for implicit pointer capture for touch;
Attachment #9215952 - Attachment description: Bug 1672726 - Part 1: Rename MultiTouchInput::ToWidgetTouchEvent to ToWidgetEvent; → Bug 1672726 - Part 1: Rename MultiTouchInput::ToWidgetTouchEvent to ToWidgetEvent; r=botond
Attachment #9215953 - Attachment description: Bug 1672726 - Part 2: Support dispatching synthesized touch events through parent process; → Bug 1672726 - Part 2: Support dispatching synthesized touch events through parent process; r=botond
Attachment #9215924 - Attachment description: Bug 1672726 - Part 3: Add test for implicit pointer capture for touch; → Bug 1672726 - Part 3: Add test for implicit pointer capture for touch; r=smaug
Pushed by echen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/201365f33156
Part 1: Rename MultiTouchInput::ToWidgetTouchEvent to ToWidgetEvent; r=botond
https://hg.mozilla.org/integration/autoland/rev/a5bf7e5c0c11
Part 2: Support dispatching synthesized touch events through parent process; r=ipc-reviewers,botond,nika
https://hg.mozilla.org/integration/autoland/rev/05bf1014f4ec
Part 3: Add test for implicit pointer capture for touch; r=smaug
Status: ASSIGNED → RESOLVED
Closed: 19 days ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in before you can comment on or make changes to this bug.