Closed Bug 1129577 Opened 9 years ago Closed 9 years ago

[mozdevice] Triggering drags occasionally interpreted as taps

Categories

(Firefox OS Graveyard :: Gaia::PerformanceTest, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Eli, Assigned: Eli)

Details

(Keywords: perf)

Attachments

(2 files, 1 obsolete file)

When using mozdevice to trigger a drag via:

device.input.drag(250, 250, 250, 350, 10, 150)

sometimes the drag is interpreted by the touch system as a tap instead. To make this more efficient, we should provide a configurable property which can allow us to override the optimal touch frequency for a given device.

With a configurable touch frequency, we can then consume this in Raptor for a smarter drag.
Will not land until next mozdevice release.
Attachment #8559314 - Flags: review?(rwood)
Comment on attachment 8559308 [details] [review]
Link to Github pull-request: https://github.com/eliperelman/node-mozdevice/pull/3

LGTM and tested in conjunction with the raptor PR in this bug. Ran the launch test on flame and emulator.
Attachment #8559308 - Flags: review?(rwood) → review+
Attachment #8559314 - Flags: review?(rwood) → review+
Merged. Publish mozdevice@1.1.2 and gaia-raptor@0.4.1.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment on attachment 8559415 [details] [review]
Link to Github pull-request: https://github.com/eliperelman/node-mozdevice/pull/3

Time to bust out the rubber stamp.
Attachment #8559415 - Flags: review?(kgrandon)
Comment on attachment 8559415 [details] [review]
Link to Github pull-request: https://github.com/eliperelman/node-mozdevice/pull/3

Nevermind, going to kill 2 birds with one stone on another patch.
Attachment #8559415 - Attachment description: [PullReq] eliperelman:bug-1129577 to mozilla-b2g:master → Link to Github pull-request: https://github.com/eliperelman/node-mozdevice/pull/3
Attachment #8559415 - Attachment is obsolete: true
Attachment #8559415 - Flags: review?(kgrandon)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: