648508 - Remove the Windows CE code for accelerometer support (follow up)

Status: VERIFIED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Ed Morley [:emorley]]

Bug 644743 removed the WinCE code for accelerometer support, however there are a few remnants left of:

> #if !defined(WINCE) && !defined(WINCE_WINDOWS_MOBILE) // normal windows.

Since the #ifdef WINCE_WINDOWS_MOBILE cases have been removed; checking
for the opposite is now presumably redundant (see bug 644743 comment 4).

Comment 1

[Ed Morley [:emorley]]

Attachment: Patch (v1)

This is my first patch, hopefully I've set up my hgrc correctly.

Ehsan, I don't have Try access - would you possibly mind vouching for level 1 so I can use try? ie per: http://www.mozilla.org/hacking/commit-access-policy/

Comment 2

[Johnny Stenback (:jst)]

Comment on attachment 524610 [details] [diff] [review]
Patch (v1)

Looks good! r=jst

Comment 3

[Ed Morley [:emorley]]

Thanks!

Comment 4

[Benjamin Stover (:stechz)]

Pushed: http://hg.mozilla.org/mozilla-central/rev/6ff088b689a3

Comment 5

[Ed Morley [:emorley]]

Thanks :-)

Comment 6

[(no longer active)]

(In reply to comment #1)
> Ehsan, I don't have Try access - would you possibly mind vouching for level 1
> so I can use try? ie per: http://www.mozilla.org/hacking/commit-access-policy/

Of course!  Just file the bug and CC me.  And thanks for your patch!  :-)

Comment 7

[Ed Morley [:emorley]]

http://mxr.mozilla.org/mozilla-central/search?string=wince&find=/dom/system/windows/
...returns 0 results.

-> Verified