520521 - Add touch-enabled support for Windows Mobile

Status: RESOLVED FIXED

(Core :: Widget: Win32, defect)

Description

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

+++ This bug was initially created as a clone of Bug #520176 +++

All the Pocket PC and WM Professional devices we support have touchscreens. Smartphone and WM Standdard don't

Comment 1

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

Attachment: patch

Comment 2

[Doug Turner (:dougt)]

Comment on attachment 404566 [details] [diff] [review]
patch

>+#elif defined(WINCE_WINDOWS_MOBILE)

Why not WINCE?  You worried about WINCE devices that don't have touch screens?  Is SM_DIGITIZER defined on windows mobile? 

>+        WCHAR platformType[MAX_PATH];
>+        SystemParametersInfo(SPI_GETPLATFORMTYPE, sizeof(platformType),

>+                             platformType,0);

Space after comma

Comment 3

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

(In reply to comment #2)
> (From update of attachment 404566 [details] [diff] [review])
> >+#elif defined(WINCE_WINDOWS_MOBILE)
> 
> Why not WINCE?  You worried about WINCE devices that don't have touch screens? 
I don't expect this to do the right thing on non windows mobile, so I wanted to make that explicit (it also avoids unnecessary work)

> Is SM_DIGITIZER defined on windows mobile? 
not according to msdn

Comment 4

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

pushed

Comment 5

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

pushed http://hg.mozilla.org/releases/mozilla-1.9.2/rev/9f9d709775f4