970770 - Remove nsHapticFeedback on non-Android build

Status: NEW

(Core :: DOM: Device Interfaces, defect, P3)

Description

[Makoto Kato [:m_kato]]

This interface was created by old Fennec, but...

- Windows ports has skeleton nsHapticFeedback, but this isn't used because XPCOM interface isn't registered.
- QT and GTK ports have same one, but it isn't built now.

So we can remove it from tree.  This is used for Fennec Android only.

Comment 1

[Makoto Kato [:m_kato]]

Attachment: v1

Comment 2

[Makoto Kato [:m_kato]]

Comment on attachment 8373840 [details] [diff] [review]
v1

Desktop version doesn't use this interface.  We should remove it except to android.

Also, unix directory becomes be used by Qt, so we should rename directory name from unix to qt.

Comment 3

[Doug Turner (:dougt)]

Comment on attachment 8373840 [details] [diff] [review]
v1

sorry that this sat so long.  blassey might be able to get to it before I can.

Comment 4

[Brad Lassey [:blassey] (use needinfo?)]

Rob, are the Windows 8 tablets making use of haptic feedback?

Comment 5

[Brad Lassey [:blassey] (use needinfo?)]

Comment on attachment 8373840 [details] [diff] [review]
v1

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

Since these implementations are useless, they should definitely go. I do wonder if we actually want a working implementation on Desktop though.

Comment 6

[Robert Strong (they/them - no direct email)]

(In reply to Brad Lassey [:blassey] (use needinfo?) from comment #4)
> Rob, are the Windows 8 tablets making use of haptic feedback?
I haven't been keeping track but Jim, Brian, or Matt should know.

Comment 7

[Jim Mathies [:jimm]]

(In reply to Robert Strong [:rstrong] (use needinfo to contact me) from comment #6)
> (In reply to Brad Lassey [:blassey] (use needinfo?) from comment #4)
> > Rob, are the Windows 8 tablets making use of haptic feedback?
> I haven't been keeping track but Jim, Brian, or Matt should know.

Win8 devices support this, but afaik it's only tied to the start button on some tablets, and isn't controllable through apis. 

I think if we found a use for this down the road we could put something together at that point.