632342 - ###!!! ASSERTION: mSensor should be null. Startup called twice?: '!mSensor'

Status: RESOLVED FIXED

(Core :: Widget: Win32, defect)

Description

[[e-mail]]

Steps to reproduce problem:
1. View an accelerator-enabled bf-cacheable page
2. View a different page
3. Go back to the first page

Comment 1

[Jesse Ruderman]

I've hit this while fuzzing.

###!!! ASSERTION: mSensor should be null.  Startup called twice?: '!mSensor', file dom/system/windows/nsDeviceMotionSystem.cpp, line 140

Comment 2

[Doug Turner (:dougt)]

Attachment: patch v.1

untested (don't have a windows box) but this should work.

Comment 3

[Doug Turner (:dougt)]

http://hg.mozilla.org/integration/mozilla-inbound/rev/206148a18467

Comment 4

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Backed out for failures in content/events/test/test_bug641477.html and dom/tests/mochitest/orientation/test_bug507902.html on windows.

http://hg.mozilla.org/integration/mozilla-inbound/rev/39c1eedd0d35

Comment 5

[Joe Drew (not getting mail)]

(In reply to comment #2)
> Created attachment 546196 [details] [diff] [review] [review]
> patch v.1
> 
> untested (don't have a windows box) but this should work.

(In reply to comment #4)
> Backed out for failures in content/events/test/test_bug641477.html and
> dom/tests/mochitest/orientation/test_bug507902.html on windows.
> 
> http://hg.mozilla.org/integration/mozilla-inbound/rev/39c1eedd0d35

maybe test it on try next time :)

Comment 6

[Doug Turner (:dougt)]

Yes!

http://tbpl.mozilla.org/?tree=Try&rev=2d59c04e08bd (green)

and

http://hg.mozilla.org/integration/mozilla-inbound/rev/a0372b031aac

Comment 7

[Joe Drew (not getting mail)]

http://hg.mozilla.org/mozilla-central/rev/a0372b031aac