Alice0775 White Reporter
	Description • 9 years ago

Steps To Reproduce:
1. Open long web page (ex. http://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-central-win32/ )
2. Scroll with Mouse Wheel

Actual Results:
Page was scrolled by a page

Expected results:
Page should be scrolled by few lines

Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=50752f206a9b&tochange=0adcaee7f3bb

Regressed by: Bug 1133818

	Jacek Caban Assignee
	Comment 1 • 9 years ago

WinMouseScrollHandler.h change probably broke it. I will prepare a fix.

	Jim Mathies [:jimm]
	Comment 2 • 9 years ago

(In reply to Jacek Caban from comment #1)
> WinMouseScrollHandler.h change probably broke it. I will prepare a fix.

yep, just confirmed, flipping those two vars back to signed fixes the problem.

	Jacek Caban Assignee
	Comment 5 • 9 years ago

The fix is attached. I'm sorry for breaking this and thank you for the report.

	Jim Mathies [:jimm]
	Comment 7 • 9 years ago

Will land this as soon as inbound opens.

	Ryan VanderMeulen [:RyanVM]
	Comment 8 • 9 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/bab13c0c9e19

Please try to use more descriptive commit messages in the future (i.e. what the patch is actually doing instead of restating the problem it's solving).

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 9 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/bab13c0c9e19

	Bartosz Piec
	Comment 16 • 9 years ago

Will this be included in tomorrow's nightly?

	Lily Kim
	Comment 17 • 9 years ago

(In reply to Bartosz Piec from comment #16)
Yes.

	Sohan Roy
	Comment 18 • 9 years ago

Please reopen the bug, I am facing the problem again
https://bugzilla.mozilla.org/show_bug.cgi?id=1136547

	Andrew McCreight [:mccr8]
	Comment 19 • 9 years ago

(In reply to Sohan Roy from comment #18)
> Please reopen the bug, I am facing the problem again

Nightly hasn't updated yet.

	Lily Kim
	Comment 20 • 9 years ago

(In reply to Sohan Roy from comment #18)
Nightly updated now and it is fixed. :)

	Jim Mathies [:jimm]
	Comment 21 • 9 years ago

yep, I can confirm the fix as well.

	Liz Henry (:lizzard Please n-i to RyanVM, jcristau, or pascal)
	Comment 22 • 9 years ago

I don't think we need to track this for 39 since it's so early in the cycle and is already fixed and verified. That was fast, thanks!

We certainly had a lot of users notice and report it, so if it happens again, we're likely to hear about it.

	shivamsingha
	Comment 23 • 9 years ago

i too had this prob, its fixed after another update